On 2018-07-06 17:09, Johannes Berg wrote:
On Wed, 2018-07-04 at 11:43 +0530, Tamizh chelvam wrote:
> > --- a/include/uapi/linux/nl80211.h
> > +++ b/include/uapi/linux/nl80211.h
> > @@ -1249,6 +1249,7 @@ enum nl80211_commands {
> > NL80211_CMD_CONTROL_PORT_FRAME,
> >
> > NL80211_CMD_SET_STA_MON,
> > + NL80211_CMD_NOTIFY_STA_MON,
>
> Missing documentation - but again, why not unify it with the existing
> event? Or can't we because that might confuse older applications?
> Perhaps anyway we should unicast these notifications? Not sure though,
> perhaps one app could set them up and the other might care?
>
This new command introduced as per the previous discussion
https://patchwork.kernel.org/patch/10168685/
https://patchwork.kernel.org/patch/10240697/
Is my understanding correct ?
I guess you're right. To my defense, this was like almost half a year
ago :)
Yeah;)
I just referred this code from cfg80211_prepare_cqm. Do you want me to
change this to struct based?
Please, I guess we can fix up cqm too later.
Sure, I'll fix it in the next version.
Thanks,
Tamizh.
