Wright Feng <[email protected]> writes: > From: Chi-hsien Lin <[email protected]> > > Setting IBSS mode when stopping AP may affect functionality of INFRA > mode on another interface. For instance, if user creates and stops > hostap daemon on virtual interface, then association cannot work on > primary interface because default BSS has been set to IBSS mode in > firmware side. The IBSS mode should be set when cfg80211 changes the > interface. > > Reviewed-by: Wright Feng <[email protected]> > Signed-off-by: Chi-hsien Lin <[email protected]> > Signed-off-by: Wright Feng <[email protected]> > --- > drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c > b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c > index 944b83c..ce2a5b6 100644 > --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c > +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c > @@ -4658,9 +4658,6 @@ static int brcmf_cfg80211_stop_ap(struct wiphy *wiphy, > struct net_device *ndev) > err = brcmf_fil_cmd_int_set(ifp, BRCMF_C_SET_AP, 0); > if (err < 0) > brcmf_err("setting AP mode failed %d\n", err); > - err = brcmf_fil_cmd_int_set(ifp, BRCMF_C_SET_INFRA, 0); > - if (err < 0) > - brcmf_err("setting INFRA mode failed %d\n", err);
The title talks about setting IBSS mode but the code looks to be about setting INFRA mode. Is this correct? -- Kalle Valo
