diff mbox series

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

Message ID 20211009212847.1781986-1-chunkeey@gmail.com (mailing list archive)
State Accepted
Commit 03469e79fee9e8e908dae3bd1a80bcd9a66f2a88
Delegated to: Kalle Valo
Headers show
Series [v1] ath9k: support DT ieee80211-freq-limit property to limit channels | expand

Commit Message

Christian Lamparter Oct. 9, 2021, 9:28 p.m. UTC
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(+)

Comments

Kalle Valo Oct. 11, 2021, 6:27 a.m. UTC | #1
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.
Kalle Valo Oct. 13, 2021, 5:46 a.m. UTC | #2
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
diff mbox series

Patch

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)