Re: [PATCH 21/27] qtnfmac: extend "IE set" TLV to include frame type info
From: Sergey Matyukevich <hidden>
Date: 2017-08-30 12:21:14
From: Sergey Matyukevich <hidden>
Date: 2017-08-30 12:21:14
Two points here. First, it looks like there is a problem here inherited from the existing implementation. We go through payload, but in fact we pass to cfg80211_inform_bss only the last QTN_TLV_ID_IE_SET element. Second, it looks like QTN_TLV_ID_IE_SET here should be treated in the same way as in qtnf_event_handle_sta_assoc, to avoid confusion. In other words, either we use only QTN_TLV_ID_IE_SET in both cases, or switch to specific qlink_tlv_ie_set elements.
Ignore the second point: it is addressed by the next patch. Regards, Sergey