Adrian Bunk wrote:
> This patch contains the possible cleanups including the following:
> - every file should #include the headers containing the prototypes for
> it's global functions
> - make needlessly global functions static
> - kyro/STG4000Interface.h: #include video/kyro.h and linux/pci.h
> instead of a manual "struct pci_dev"
> - i810_main.{c,h}: prototypes for static functions belong to the
> C file
>
>
> Signed-off-by: Adrian Bunk <[email protected]>
>
> ---
>
> drivers/video/arcfb.c | 8 +--
> drivers/video/console/softcursor.c | 2
> drivers/video/i810/i810-i2c.c | 1
> drivers/video/i810/i810_accel.c | 1
> drivers/video/i810/i810_gtf.c | 1
> drivers/video/i810/i810_main.c | 51 ++++++++++++++++++--
> drivers/video/i810/i810_main.h | 56 +---------------------
> drivers/video/kyro/STG4000InitDevice.c | 1
> drivers/video/kyro/STG4000Interface.h | 3 -
> drivers/video/kyro/STG4000OverlayDevice.c | 1
> drivers/video/matrox/matroxfb_g450.c | 2
> drivers/video/nvidia/nv_hw.c | 1
> drivers/video/tdfxfb.c | 2
> 13 files changed, 68 insertions(+), 62 deletions(-)
>
> --- linux-2.6.14-rc5-mm1-full/drivers/video/console/softcursor.c.old 2005-11-06 00:31:15.000000000 +0100
> +++ linux-2.6.14-rc5-mm1-full/drivers/video/console/softcursor.c 2005-11-06 00:31:30.000000000 +0100
> @@ -17,6 +17,8 @@
> #include <asm/uaccess.h>
> #include <asm/io.h>
>
> +#include "fbcon.h"
I don't think softcursor needs anything in fbcon.h. The rest looks okay.
Tony
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
