* [PATCH v1] ath9k: support DT ieee80211-freq-limit property to limit channels @ 2021-10-09 21:28 Christian Lamparter 2021-10-11 6:27 ` Kalle Valo 2021-10-13 5:46 ` Kalle Valo 0 siblings, 2 replies; 3+ messages in thread From: Christian Lamparter @ 2021-10-09 21:28 UTC (permalink / raw) To: linux-wireless, ath9k-devel; +Cc: Kalle Valo The common DT property can be used to limit the available channels but ath9k has to manually call wiphy_read_of_freq_limits(). Signed-off-by: Christian Lamparter <chunkeey@gmail.com> --- I would have put this into ath9k_of_init(). But it didn't work there. The reason is that in ath9k_of_init() the channels and bands are not yet registered in the wiphy struct. So there isn't any channel to flag as disabled. --- drivers/net/wireless/ath/ath9k/init.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/net/wireless/ath/ath9k/init.c b/drivers/net/wireless/ath/ath9k/init.c index 1568730fc01e..0344d6f27275 100644 --- a/drivers/net/wireless/ath/ath9k/init.c +++ b/drivers/net/wireless/ath/ath9k/init.c @@ -1094,6 +1094,8 @@ int ath9k_init_device(u16 devid, struct ath_softc *sc, ARRAY_SIZE(ath9k_tpt_blink)); #endif + wiphy_read_of_freq_limits(hw->wiphy); + /* Register with mac80211 */ error = ieee80211_register_hw(hw); if (error) -- 2.33.0 ^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH v1] ath9k: support DT ieee80211-freq-limit property to limit channels 2021-10-09 21:28 [PATCH v1] ath9k: support DT ieee80211-freq-limit property to limit channels Christian Lamparter @ 2021-10-11 6:27 ` Kalle Valo 2021-10-13 5:46 ` Kalle Valo 1 sibling, 0 replies; 3+ messages in thread From: Kalle Valo @ 2021-10-11 6:27 UTC (permalink / raw) To: Christian Lamparter; +Cc: linux-wireless, ath9k-devel Christian Lamparter <chunkeey@gmail.com> writes: > The common DT property can be used to limit the available channels > but ath9k has to manually call wiphy_read_of_freq_limits(). > > Signed-off-by: Christian Lamparter <chunkeey@gmail.com> > > --- > > I would have put this into ath9k_of_init(). But it didn't work there. > The reason is that in ath9k_of_init() the channels and bands are not yet > registered in the wiphy struct. So there isn't any channel to flag as > disabled. I think this is good info to have in the commit log, so in the pending branch I did that. -- https://patchwork.kernel.org/project/linux-wireless/list/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v1] ath9k: support DT ieee80211-freq-limit property to limit channels 2021-10-09 21:28 [PATCH v1] ath9k: support DT ieee80211-freq-limit property to limit channels Christian Lamparter 2021-10-11 6:27 ` Kalle Valo @ 2021-10-13 5:46 ` Kalle Valo 1 sibling, 0 replies; 3+ messages in thread From: Kalle Valo @ 2021-10-13 5:46 UTC (permalink / raw) To: Christian Lamparter; +Cc: linux-wireless, ath9k-devel Christian Lamparter <chunkeey@gmail.com> wrote: > The common DT property can be used to limit the available channels > but ath9k has to manually call wiphy_read_of_freq_limits(). > > I would have put this into ath9k_of_init(). But it didn't work there. > The reason is that in ath9k_of_init() the channels and bands are not yet > registered in the wiphy struct. So there isn't any channel to flag as > disabled. > > Signed-off-by: Christian Lamparter <chunkeey@gmail.com> > Signed-off-by: Kalle Valo <kvalo@codeaurora.org> Patch applied to ath-next branch of ath.git, thanks. 03469e79fee9 ath9k: support DT ieee80211-freq-limit property to limit channels -- https://patchwork.kernel.org/project/linux-wireless/patch/20211009212847.1781986-1-chunkeey@gmail.com/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-10-13 5:47 UTC | newest] Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-10-09 21:28 [PATCH v1] ath9k: support DT ieee80211-freq-limit property to limit channels Christian Lamparter 2021-10-11 6:27 ` Kalle Valo 2021-10-13 5:46 ` Kalle Valo
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).