On Thu, 2025-04-24 at 11:50 +0200, Arend van Spriel wrote: > > Looked at other drivers implementing this callback and here are the results: > > [wil6210] wil_cfg80211_change_bss(): does exactly the same thing. > [wilc1000] change_bss(): worse! it accepts everything and does nothing. > [rtl8723bs] cfg80211_rtw_change_bss(): same. just an empty callback. OK, though I guess other drivers being bad doesn't mean this one should be :) > If this patch means Infineon is (mildly) regaining interest in upstream > wifi development let's not discourage them. Fair. I didn't mean to discourage. I just think to meaningfully contribute upstream people should follow the list. And even review other people's patches. I've been meaning to make that more of a requirement, since I can't possibly meaningfully review everything I now need to merge. https://lore.kernel.org/linux-wireless/21896d2788b8bc6c7fcb534cd43e75671a57f494.camel@xxxxxxxxxxxxxxxx/ > I do watch the > linux-wireless list on occasion but I am a bit lost on your remark. What > do you mean by tree tags. You mean the "wifi:" prefix? But then I am > confused about the "subject prefix" remark. Oh, well I guess terminology: https://lore.kernel.org/linux-wireless/ec3a3d891acfe5ed8763271a1df4151d75daf25f.camel@xxxxxxxxxxxxxxxx/ johannes
