On 2018-03-27 18:26, Johannes Berg wrote:
On Tue, 2018-03-27 at 12:18 +0530, Tamizh chelvam wrote:
Currently bw and smps_mode are u8 type value in sta_opmode_info
structure. This values filled in mac80211 from ieee80211_sta_rx_bandwidth and ieee80211_smps_mode. These enum values are specific to mac80211 and
userspace/cfg80211 doesn't know about that. This patchset change its
data type in the sta_opmode_info structure and mapping from mac80211
specific enum to nl80211 enum value.

Good catch.

Tamizh chelvam (3):
  cfg80211: fix data type of sta_opmode_info parameter
  mac80211: Use proper smps_mode enum in sta opmode event
  mac80211: Use proper chan_width enum in sta opmode event

Note :
  * Is this mac80211 approach sufficient ? or whether some more
    complete cleanup would be preferred ?

Not sure what you mean? The patches look good to me.

I just wanted to confirm is this approach fine or not! I'll send a patch format

Reply via email to