On Wed, 2011-01-05 at 14:49 -0500, Bob Copeland wrote: > On Wed, Jan 5, 2011 at 2:40 PM, Joe Perches <joe@xxxxxxxxxxx> wrote: > > On Wed, 2011-01-05 at 14:22 -0500, John W. Linville wrote: > >> diff --git a/drivers/net/wireless/ath/ath.h b/drivers/net/wireless/ath/ath.h > > [] > >> + bool nohwcrypt; > >> diff --git a/drivers/net/wireless/ath/ath5k/base.c b/drivers/net/wireless/ath/ath5k/base.c > >> -int modparam_nohwcrypt; > >> +static int modparam_nohwcrypt; > >> module_param_named(nohwcrypt, modparam_nohwcrypt, bool, S_IRUGO); > > bool/int matching? > It used to be the case that you had to use ints here. Google tells me > that changed about 2 years ago, so bool should be ok. bool is OK. modparam_nohwcrypt is declared int and should be declared bool. Also should be declared and used as bool in ath9k and ath9k_htc too. -- 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