diff options
author | Dmitry Torokhov <dtor@insightbb.com> | 2007-05-03 00:52:51 -0400 |
---|---|---|
committer | Dmitry Torokhov <dtor@insightbb.com> | 2007-05-03 00:52:51 -0400 |
commit | 78167236e23bb3c80d2b35b693e578a6e56b1171 (patch) | |
tree | 2287a478c80f5eaa79738d4b541bd5a0ddd42de5 /drivers/input/joystick/analog.c | |
parent | dec3eb01c2409ca8276c1152c167add66a37d1ba (diff) |
Input: drivers/joystick - fix various sparse warnings
Fix various issues pointed by sparse:
- module_param_array_named() takes unsigned int as number
of parameters argument
- shadowing of global variables is not healthy. I think there was
once a bug in db9 caused by it.
Signed-off-by: Dmitry Torokhov <dtor@mail.ru>
Diffstat (limited to 'drivers/input/joystick/analog.c')
-rw-r--r-- | drivers/input/joystick/analog.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/input/joystick/analog.c b/drivers/input/joystick/analog.c index 1c1afb5d4684..bdd157c1ebf8 100644 --- a/drivers/input/joystick/analog.c +++ b/drivers/input/joystick/analog.c | |||
@@ -53,7 +53,7 @@ MODULE_LICENSE("GPL"); | |||
53 | #define ANALOG_PORTS 16 | 53 | #define ANALOG_PORTS 16 |
54 | 54 | ||
55 | static char *js[ANALOG_PORTS]; | 55 | static char *js[ANALOG_PORTS]; |
56 | static int js_nargs; | 56 | static unsigned int js_nargs; |
57 | static int analog_options[ANALOG_PORTS]; | 57 | static int analog_options[ANALOG_PORTS]; |
58 | module_param_array_named(map, js, charp, &js_nargs, 0); | 58 | module_param_array_named(map, js, charp, &js_nargs, 0); |
59 | MODULE_PARM_DESC(map, "Describes analog joysticks type/capabilities"); | 59 | MODULE_PARM_DESC(map, "Describes analog joysticks type/capabilities"); |