On Mon, Mar 30, 2009 at 8:14 AM, Jiri Slaby <jirislaby@xxxxxxxxx> wrote: >> +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. Ok, I'll do it as a follow-up if John doesn't mind. -- Bob Copeland %% www.bobcopeland.com -- 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