On 05/06/2011 09:51 AM, Felix Fietkau wrote:
On 2011-05-06 6:29 PM, Ben Greear wrote:
On 05/05/2011 05:44 PM, Felix Fietkau wrote:
This should fix the second issue:
--- a/drivers/net/wireless/ath/ath9k/main.c
+++ b/drivers/net/wireless/ath/ath9k/main.c
@@ -1722,6 +1722,10 @@ static int ath9k_sta_add(struct ieee8021
struct ath_node *an = (struct ath_node *) sta->drv_priv;
struct ieee80211_key_conf ps_key = { };
+ if (vif->opmode != NL80211_IFTYPE_AP&&
+ vif->opmode != NL80211_IFTYPE_AP_VLAN)
+ return 0;
+
ath_node_attach(sc, sta);
an->ps_key = ath_key_config(common, vif, sta,&ps_key);
This doesn't compile..appears there is no vif->opmode member?
Also, if you are trying to compare the hardware opmode, I'm not
sure that would be correct since you can be in AP mode and still
have stations. Maybe I'm just confused though...
I sent the wrong version of the patch, forgot to refresh it.
Here's the correct one:
--- a/drivers/net/wireless/ath/ath9k/main.c
+++ b/drivers/net/wireless/ath/ath9k/main.c
@@ -1722,6 +1722,10 @@ static int ath9k_sta_add(struct ieee8021
struct ath_node *an = (struct ath_node *) sta->drv_priv;
struct ieee80211_key_conf ps_key = { };
+ if (vif->type != NL80211_IFTYPE_AP&&
+ vif->type != NL80211_IFTYPE_AP_VLAN)
+ return 0;
+
ath_node_attach(sc, sta);
an->ps_key = ath_key_config(common, vif, sta,&ps_key);
I think maybe you want that logic inserted below the ath_node_attach?
I tried it as you have it above, guessing that is what you meant, and
it crashed with null pointer exception....
Compiling with the logic below node_attach now...
Ben
--
Ben Greear <greearb@xxxxxxxxxxxxxxx>
Candela Technologies Inc http://www.candelatech.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
