Hi James, > -----Original Message----- > From: [email protected] [mailto:[email protected]] > Sent: 2017年11月22日 3:31 > To: Xinming Hu <[email protected]> > Cc: Linux Wireless <[email protected]>; Kalle Valo > <[email protected]>; Brian Norris <[email protected]>; Dmitry > Torokhov <[email protected]>; [email protected]; Zhiyuan Yang > <[email protected]>; Tim Song <[email protected]>; Cathy Luo > <[email protected]>; James Cao <[email protected]>; Ganapathi Bhat > <[email protected]>; Ellie Reeves <[email protected]> > Subject: Re: [PATCH v2] mwifiex: do not support change AP interface to station > mode > > On Tue, Nov 21, 2017 at 08:03:35PM +0800, Xinming Hu wrote: > > Firmware do not support change interface from micro-ap mode to station > > mode, forbidden this operation in driver accordingly. > > "forbidden" should be "forbid", for correct tense. > > "in driver" is redundant and can be removed. > > "accordingly" is also redundant. > > Perhaps "Firmware do not support change interface from micro-ap mode to > station mode, forbid this operation." >
Ok, it looks better. > > Signed-off-by: Cathy Luo <[email protected]> > > Signed-off-by: Xinming Hu <[email protected]> > > --- > > v2: remove unnecessary sta/uap combo check(James Cameron) > > > > drivers/net/wireless/marvell/mwifiex/cfg80211.c | 1 - > > 1 file changed, 1 deletion(-) > > > > diff --git a/drivers/net/wireless/marvell/mwifiex/cfg80211.c > > b/drivers/net/wireless/marvell/mwifiex/cfg80211.c > > index 6e0d9a9..4d45df8 100644 > > --- a/drivers/net/wireless/marvell/mwifiex/cfg80211.c > > +++ b/drivers/net/wireless/marvell/mwifiex/cfg80211.c > > @@ -1180,7 +1180,6 @@ static int mwifiex_deinit_priv_params(struct > mwifiex_private *priv) > > case NL80211_IFTYPE_AP: > > switch (type) { > > case NL80211_IFTYPE_ADHOC: > > Change interface type from micro-ap to adhoc is supported? Yes, it is supported in FW. > > > - case NL80211_IFTYPE_STATION: > > return mwifiex_change_vif_to_sta_adhoc(dev, curr_iftype, > > type, params); > > break; > > -- > > 1.9.1 > > > > -- > James Cameron > http://quozl.netrek.org/
