On Tue, 2023-08-22 at 14:17 +0800, Wen Gong wrote: > > > > Anyway it would seem better for a utility function to have clearer > > defined output, I think? > > > > johannes > > Do you mean move the logic of set bss_conf->power_type to another new > function from > > ieee80211_chandef_he_6ghz_oper()? > No, not really. But we could have a pointer to the bss_conf or even the particular value, as an obvious output from the function. johannes