On 10/22/24 06:29, Johannes Berg wrote:
On Thu, 2024-10-10 at 13:40 -0700, greearb@xxxxxxxxxxxxxxx wrote:
From: Ben Greear <greearb@xxxxxxxxxxxxxxx>
If changed is '0', then the ieee80211_hw_config takes no
action, so just remove the call in
__ieee809211_recalc_txpower()
Hmm. I think it may have done something in the past, so perhaps this is
a bug that needs to be fixed instead?
- if (!local->use_chanctx)
- changed |= ieee80211_hw_conf_chan(local);
in "wifi: mac80211: simplify non-chanctx drivers".
But maybe that's exactly the previous patch? Would be nice though to
have that in the commit message, or if it's related maybe even squash
them?
I'm not sure exactly how it is all supposed to work, but at least with the patches
I posted, the mtk7996 driver works again. The mtk7996 driver does still have
code to handle CONF_CHANGE_POWER, so possibly the code you reference above
that if re-added would allow ieee80211_hw_config to configure txpower should
be re-added.
If this is what you mean by previous patch:
mac80211: fix user-power when emulating chanctx
then I don't think it should be squashed with anything, but if you want me to squash it,
let me know and I will do so and repost.
Thanks,
Ben
johannes
--
Ben Greear <greearb@xxxxxxxxxxxxxxx>
Candela Technologies Inc http://www.candelatech.com
