Skip to contentSkip to conversation

ath9k: Remove redundant max_streams assignment

PATCH New Sujith Manoharan 2 participants 3 messages Feb 13, 2012 Qualcomm Atheros ATH9K wireless driver

lore
Share & tools
lore

Sujith Manoharan

author
#0 lore raw

PATCH 1/7 ath9k: Remove redundant max_streams assignment

Signed-off-by Sujith Manoharan <[email protected]>
---
 drivers/net/wireless/ath/ath9k/init.c |    2 --
 1 files changed, 0 insertions(+), 2 deletions(-)
drivers/net/wireless/ath/ath9k/init.c +0 -2
index 53a005d..a625c9a 100644
@@ -258,8 +258,6 @@ static void setup_ht_cap(struct ath_softc *sc,
 
 	if (AR_SREV_9330(ah) || AR_SREV_9485(ah))
 		max_streams = 1;
-	else if (AR_SREV_9462(ah))
-		max_streams = 2;
 	else if (AR_SREV_9300_20_OR_LATER(ah))
 		max_streams = 3;
 	else
Signature · 1 line
1.7.9
#1 lore raw

PATCH 1/7 ath9k: Remove redundant max_streams assignment

Quoted 10 lines · Sujith Manoharan
Signed-off-by Sujith Manoharan <[email protected]>
---
 drivers/net/wireless/ath/ath9k/init.c |    2 --
 1 files changed, 0 insertions(+), 2 deletions(-)
drivers/net/wireless/ath/ath9k/init.c +0 -0
index 53a005d..a625c9a 100644

drivers/net/wireless/ath/ath9k/init.c static void setup_ht_cap(struct ath_softc *sc, · line 263

Quoted 8 lines
drivers/net/wireless/ath/ath9k/init.c +0 -2
@@ -258,8 +258,6 @@ static void setup_ht_cap(struct ath_softc *sc,
 
 	if (AR_SREV_9330(ah) || AR_SREV_9485(ah))
 		max_streams = 1;
-	else if (AR_SREV_9462(ah))
-		max_streams = 2;
 	else if (AR_SREV_9300_20_OR_LATER(ah))
 		max_streams = 3;
 	else

I think AR9462 is also matched by AR_SREV_9300_20_OR_LATER(ah), so that one may not be redundant.

- Felix

author
#2 lore raw

PATCH 1/7 ath9k: Remove redundant max_streams assignment

Quoted 2 lines · Felix Fietkau

I think AR9462 is also matched by AR_SREV_9300_20_OR_LATER(ah), so that one may not be redundant.

Ah, nice catch.

John, can you please drop this patch ?

Sujith