I forgot that iwl3945 registration is separate from iwlagn. Signed-off-by: Johannes Berg <johannes@xxxxxxxxxxxxxxxx> --- Hi Kalle, Reinette, This looks like my mistake, sorry! > [ 17.489314] WARNING: at net/wireless/core.c:291 wiphy_register+0xa7/0x1bf [cfg80211]() > And the warning from code: > > if (WARN_ON(wiphy->max_scan_ssids < 1)) > return -EINVAL; I thought 3945 were already unified so that the setup code was common, but it turns out not to be. This patch will fix it. Alternatively the scanning rework patches you (Reinette) have will also fix it -- do you just want to send those to John instead? johannes drivers/net/wireless/iwlwifi/iwl3945-base.c | 2 ++ 1 file changed, 2 insertions(+) --- wireless-testing.orig/drivers/net/wireless/iwlwifi/iwl3945-base.c 2009-04-09 09:53:35.000000000 +0200 +++ wireless-testing/drivers/net/wireless/iwlwifi/iwl3945-base.c 2009-04-09 09:53:57.000000000 +0200 @@ -4913,6 +4913,8 @@ static int iwl3945_setup_mac(struct iwl_ hw->wiphy->custom_regulatory = true; + hw->wiphy->max_scan_ssids = 1; /* WILL FIX */ + /* Default value; 4 EDCA QOS priorities */ hw->queues = 4; -- 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
