On 03/30/2009 02:05 PM, Bob Copeland wrote: > --- a/drivers/net/wireless/ath5k/base.c > +++ b/drivers/net/wireless/ath5k/base.c > @@ -64,6 +64,10 @@ static int modparam_nohwcrypt; > module_param_named(nohwcrypt, modparam_nohwcrypt, int, 0444); > MODULE_PARM_DESC(nohwcrypt, "Disable hardware encryption."); > > +static int modparam_all_channels; > +module_param_named(all_channels, modparam_all_channels, int, 0444); Make it bool and use S_* constants for perms, please (S_IRUGO). Maybe do it as a followup also cleaning up the former one. -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html