On Sun, Dec 18, 2011 at 8:37 PM, Arik Nemtsov <anamtsov@xxxxxxxxx> wrote: > On Sun, Dec 18, 2011 at 20:25, Eliad Peller <eliad@xxxxxxxxxx> wrote: >> Implement the change_interface callback by simply removing the >> current vif and adding a new one after updating the vif type. >> >> Signed-off-by: Eliad Peller <eliad@xxxxxxxxxx> >> --- >> drivers/net/wireless/wl12xx/main.c | 11 +++++++++++ >> 1 files changed, 11 insertions(+), 0 deletions(-) >> >> diff --git a/drivers/net/wireless/wl12xx/main.c b/drivers/net/wireless/wl12xx/main.c >> index c305841..86a7ee3 100644 >> --- a/drivers/net/wireless/wl12xx/main.c >> +++ b/drivers/net/wireless/wl12xx/main.c >> @@ -2269,6 +2269,16 @@ out: >> cancel_work_sync(&wl->recovery_work); >> } >> >> +static int wl12xx_op_change_interface(struct ieee80211_hw *hw, >> + struct ieee80211_vif *vif, >> + enum nl80211_iftype new_type, bool p2p) >> +{ >> + wl1271_op_remove_interface(hw, vif); >> + >> + vif->type = ieee80211_iftype_p2p(new_type, p2p); > > Isn't this internal? it's usually read-only (for the low-level drivers), but here we have to change it... otoh, i've just noticed that i forgot to update vif->p2p as well :) Eliad. -- 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