On Mon, 2025-08-18 at 15:49 +0530, Amith A wrote: > > + * @NL80211_ATTR_INCUMBENT_SIGNAL_INTERFERENCE_BITMAP: u32 attribute specifying > + * the signal interference bitmap detected on the operating bandwidth for > + * %NL80211_CMD_INCUMBENT_SIGNAL_DETECT. Each bit represents a 20 MHz > + * segment, lowest bit corresponds to the lowest 20 MHz segment, in the > + * operating bandwidth where the interference is detected. I think you should clarify how this interacts with puncturing. Are disabled subchannels not considered part of the chandef, so if e.g. you have an 80 MHz channel with one disabled subchannel, only the lowest _three_ bits are taken into account, rather than the usual four for an 80 MHz chandef? Or, perhaps more easily, are disabled subchannels part of the bitmap and then must be zero since there cannot be incumbent signal on them that matters, since we don't even use them? Should the code perhaps then check that those bits are indeed zero? Or something else entirely? johannes
