[PATCH v1] ath9k: support DT ieee80211-freq-limit property to limit channels

[PATCH v1] ath9k: support DT ieee80211-freq-limit property to limit channels

[Christian Lamparter]

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

Re: [PATCH v1] ath9k: support DT ieee80211-freq-limit property to limit channels

[Kalle Valo]

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

Re: [PATCH v1] ath9k: support DT ieee80211-freq-limit property to limit channels

[Kalle Valo]

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