diff options
author | Randy Dunlap <randy.dunlap@oracle.com> | 2010-08-10 08:46:44 -0700 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2010-08-31 14:51:53 -0700 |
commit | 8922967ea36eac4c2f5eaa78ec43d781340ccc31 (patch) | |
tree | fb3450357f6766b8e28e2025212ae9ce5e097d16 /drivers/staging/xgifb/XGI_main.h | |
parent | 6a4ca0399255260bb04fa4e718625ede0c745e59 (diff) | |
download | kernel_samsung_smdk4412-8922967ea36eac4c2f5eaa78ec43d781340ccc31.zip kernel_samsung_smdk4412-8922967ea36eac4c2f5eaa78ec43d781340ccc31.tar.gz kernel_samsung_smdk4412-8922967ea36eac4c2f5eaa78ec43d781340ccc31.tar.bz2 |
Staging: xgifb: fix lots of sparse warnings
Fix many sparse warnings about data or functions being static.
Fix many sparse warnings about data or functions not being used
(put them inside #if 0/#endif blocks).
Fix sparse warnings about 0 being used for NULL.
Fixed a small bit of source formatting when those lines were being
modified anyway, but there is still lots of this yet to be done.
Signed-off-by: Randy Dunlap <randy.dunlap@oracle.com>
Cc: devel@driverdev.osuosl.org
Cc: Arnaud Patard <apatard@mandriva.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/staging/xgifb/XGI_main.h')
-rw-r--r-- | drivers/staging/xgifb/XGI_main.h | 27 |
1 files changed, 12 insertions, 15 deletions
diff --git a/drivers/staging/xgifb/XGI_main.h b/drivers/staging/xgifb/XGI_main.h index 06f62ee..72448e8 100644 --- a/drivers/staging/xgifb/XGI_main.h +++ b/drivers/staging/xgifb/XGI_main.h @@ -350,20 +350,17 @@ static int enable_dstn = 0; static int XGIfb_ypan = -1; -int XGIfb_accel = 0; - - static int XGIfb_hwcursor_size = 0; static int XGIfb_CRT2_write_enable = 0; -int XGIfb_crt2type = -1; /* TW: CRT2 type (for overriding autodetection) */ -int XGIfb_tvplug = -1; /* PR: Tv plug type (for overriding autodetection) */ +static int XGIfb_crt2type = -1; /* TW: CRT2 type (for overriding autodetection) */ +static int XGIfb_tvplug = -1; /* PR: Tv plug type (for overriding autodetection) */ -int XGIfb_queuemode = -1; /* TW: Use MMIO queue mode by default (310/325 series only) */ +static int XGIfb_queuemode = -1; /* TW: Use MMIO queue mode by default (310/325 series only) */ -unsigned char XGIfb_detectedpdc = 0; +static unsigned char XGIfb_detectedpdc = 0; -unsigned char XGIfb_detectedlcda = 0xff; +static unsigned char XGIfb_detectedlcda = 0xff; @@ -372,10 +369,10 @@ unsigned char XGIfb_detectedlcda = 0xff; /* XGIfb_info XGIfbinfo; */ /* TW: Hardware extension; contains data on hardware */ -struct xgi_hw_device_info XGIhw_ext; +static struct xgi_hw_device_info XGIhw_ext; /* TW: XGI private structure */ -struct vb_device_info XGI_Pr; +static struct vb_device_info XGI_Pr; /* card parameters */ static unsigned long XGIfb_mmio_size = 0; @@ -392,7 +389,7 @@ typedef enum _XGI_CMDTYPE { /* mode table */ /* NOT const - will be patched for 1280x960 mode number chaos reasons */ -struct _XGIbios_mode { +static struct _XGIbios_mode { char name[15]; u8 mode_no; u16 vesa_mode_no_1; /* "XGI defined" VESA mode number */ @@ -491,17 +488,17 @@ static int xgifb_mode_idx = 1; #else static int xgifb_mode_idx = -1; /* Use a default mode if we are inside the kernel */ #endif -u8 XGIfb_mode_no = 0; -u8 XGIfb_rate_idx = 0; +static u8 XGIfb_mode_no = 0; +static u8 XGIfb_rate_idx = 0; /* TW: CR36 evaluation */ -const unsigned short XGI300paneltype[] = +static const unsigned short XGI300paneltype[] = { LCD_UNKNOWN, LCD_800x600, LCD_1024x768, LCD_1280x1024, LCD_1280x960, LCD_640x480, LCD_1024x600, LCD_1152x768, LCD_1024x768, LCD_1024x768, LCD_1024x768, LCD_1024x768, LCD_1024x768, LCD_1024x768, LCD_1024x768 }; -const unsigned short XGI310paneltype[] = +static const unsigned short XGI310paneltype[] = { LCD_UNKNOWN, LCD_800x600, LCD_1024x768, LCD_1280x1024, LCD_640x480, LCD_1024x600, LCD_1152x864, LCD_1280x960, LCD_1152x768, LCD_1400x1050,LCD_1280x768, LCD_1600x1200, |