Prev: CONFIG_HIGHMEM or CONFIG_HIGHMEM64G in illegal_highdma()
Next: [PATCH] Updated: Reworked Cell OProfile: SPU mutex lock fix
From: Krzysztof Helt on 7 May 2008 13:00
On Wed, 7 May 2008 19:09:44 +0800
<JosephChan(a)via.com.tw> wrote:

> Signed-off-by: Joseph Chan <josephchan(a)via.com.tw>
>
> diff -Nur a/drivers/video/via/viafbdev.h b/drivers/video/via/viafbdev.h
> --- a/drivers/video/via/viafbdev.h 1969-12-31 19:00:00.000000000 -0500
> +++ b/drivers/video/via/viafbdev.h 2008-04-29 04:27:46.000000000 -0400
> @@ -0,0 +1,114 @@

(...)
> +
> +struct viafb_par {
> + struct fb_var_screeninfo var;
> + int bpp;
> + int hres;
> + int vres;
> + int linelength;
> + int vclk; /*in MHz */
> +
> + int vtotal;
> + int vdispend;
> + int vsyncstart;
> + int vsyncend;
> + int vblankstart;
> + int vblankend;
> +
> + int htotal;
> + int hdispend;
> + int hsyncstart;
> + int hsyncend;
> + int hblankstart;
> + int hblankend;
> +
> + int accel; /* Acceleration Flags */

A random pick of "accel", "hblankstart" and "hblankend" shows that they are only set and never checked. Try to remove all unused fields here.

Regards,
Krzysztof

----------------------------------------------------------------------
Lepszy niz zakladki.
Kliknij >>> http://link.interia.pl/f1db1

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
 | 
Pages: 1
Prev: CONFIG_HIGHMEM or CONFIG_HIGHMEM64G in illegal_highdma()
Next: [PATCH] Updated: Reworked Cell OProfile: SPU mutex lock fix