diff mbox series

[wireless] wifi: mt76: mt7996: fix invalid interface combinations

Message ID 20241007135133.5336-1-shayne.chen@mediatek.com (mailing list archive)
State New
Delegated to: Felix Fietkau
Headers show
Series [wireless] wifi: mt76: mt7996: fix invalid interface combinations | expand

Commit Message

Shayne Chen Oct. 7, 2024, 1:51 p.m. UTC
Setting beacon_int_min_gcd and NL80211_IFTYPE_ADHOC in the same interface
combination is invalid, which will trigger the following warning trace
and get error returned from wiphy_register().

[   10.080325] Call trace:
[   10.082761]  wiphy_register+0xc4/0x76c [cfg80211]
[   10.087465]  ieee80211_register_hw+0x800/0xac4 [mac80211]
[   10.092868]  mt76_register_device+0x16c/0x2c0 [mt76]
[   10.097829]  mt7996_register_device+0x740/0x844 [mt7996e]

Fix this by removing unused adhoc iftype.

Fixes: 948f65249868 ("wifi: mt76: mt7996: advertize beacon_int_min_gcd")
Reported-by: Frank Wunderlich <frank-w@public-files.de>
Signed-off-by: Shayne Chen <shayne.chen@mediatek.com>
---
 drivers/net/wireless/mediatek/mt76/mt7996/init.c | 3 ---
 1 file changed, 3 deletions(-)

Comments

Frank Wunderlich Oct. 9, 2024, 5:23 p.m. UTC | #1
Hi Shayne

> Gesendet: Montag, 07. Oktober 2024 um 15:51 Uhr
> Von: "Shayne Chen" <shayne.chen@mediatek.com>
> Betreff: [PATCH wireless] wifi: mt76: mt7996: fix invalid interface combinations
>
> Setting beacon_int_min_gcd and NL80211_IFTYPE_ADHOC in the same interface
> combination is invalid, which will trigger the following warning trace
> and get error returned from wiphy_register().
>
> [   10.080325] Call trace:
> [   10.082761]  wiphy_register+0xc4/0x76c [cfg80211]
> [   10.087465]  ieee80211_register_hw+0x800/0xac4 [mac80211]
> [   10.092868]  mt76_register_device+0x16c/0x2c0 [mt76]
> [   10.097829]  mt7996_register_device+0x740/0x844 [mt7996e]
>
> Fix this by removing unused adhoc iftype.
>
> Fixes: 948f65249868 ("wifi: mt76: mt7996: advertize beacon_int_min_gcd")
> Reported-by: Frank Wunderlich <frank-w@public-files.de>
> Signed-off-by: Shayne Chen <shayne.chen@mediatek.com>

thx for the patch. I can confirm it fixes the issue/trace.

regards
diff mbox series

Patch

diff --git a/drivers/net/wireless/mediatek/mt76/mt7996/init.c b/drivers/net/wireless/mediatek/mt76/mt7996/init.c
index f6a010b61a8a..2d36c7325137 100644
--- a/drivers/net/wireless/mediatek/mt76/mt7996/init.c
+++ b/drivers/net/wireless/mediatek/mt76/mt7996/init.c
@@ -16,9 +16,6 @@ 
 
 static const struct ieee80211_iface_limit if_limits[] = {
 	{
-		.max = 1,
-		.types = BIT(NL80211_IFTYPE_ADHOC)
-	}, {
 		.max = 16,
 		.types = BIT(NL80211_IFTYPE_AP)
 #ifdef CONFIG_MAC80211_MESH