On Tue, Apr 6, 2010 at 12:44 PM, Dan Carpenter <error27@xxxxxxxxx> wrote: > We were storing -1 as an unsigned int and as a result the effect of > passing -1 was the same as using 1. > > Signed-off-by: Dan Carpenter <error27@xxxxxxxxx> > > diff --git a/drivers/platform/x86/asus-laptop.c b/drivers/platform/x86/asus-laptop.c > index 475ab50..f532336 100644 > --- a/drivers/platform/x86/asus-laptop.c > +++ b/drivers/platform/x86/asus-laptop.c > @@ -78,15 +78,15 @@ static uint wapf = 1; > module_param(wapf, uint, 0644); > MODULE_PARM_DESC(wapf, "WAPF value"); > > -static uint wlan_status = 1; > -static uint bluetooth_status = 1; > +static int wlan_status = 1; > +static int bluetooth_status = 1; > > -module_param(wlan_status, uint, 0644); > +module_param(wlan_status, int, 0644); > MODULE_PARM_DESC(wlan_status, "Set the wireless status on boot " > "(0 = disabled, 1 = enabled, -1 = don't do anything). " > "default is 1"); > > -module_param(bluetooth_status, uint, 0644); > +module_param(bluetooth_status, int, 0644); > MODULE_PARM_DESC(bluetooth_status, "Set the wireless status on boot " > "(0 = disabled, 1 = enabled, -1 = don't do anything). " > "default is 1"); > -- Thanks ! Acked-by: Corentin Chary <corentincj@xxxxxxxxxx> -- Corentin Chary http://xf.iksaif.net -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
