On 1/19/2023 7:37 AM, Johannes Berg wrote:
Frankly I'm not happy for this to be stuck into set_interface() like
that - can we add it to set_channel() only or something? At least there
should be too? And read it only if the channel is actually touched? This
feels very ad-hoc.
johannes
Hi Johannes,
I will work on these comments.
Secondly, this RFC uses ieee80211_valid_disable_subchannel_bitmap()
defined in following RFC you sent:
https://patchwork.kernel.org/project/linux-wireless/patch/20220325140859.e48bf244f157.I3547481d49f958389f59dfeba3fcc75e72b0aa6e@changeid/
Is there is any plan for the next version?
If not, I can move the validation function defined in cfg80211 to
mac80211 from the following patch-set:
https://patchwork.kernel.org/project/linux-wireless/patch/20220214223051.3610-3-quic_alokad@xxxxxxxxxxx/
You had asked me about exporting this during the review as well.
Please let me know so that I can incorporate accordingly.
Thanks.
