Re: [ath9k-devel] [PATCH 1/7] ath9k: Remove redundant max_streams assignment

2012-02-13 Thread Felix Fietkau
On 2012-02-13 7:20 AM, Sujith Manoharan wrote:
 Signed-off-by: Sujith Manoharan c_man...@qca.qualcomm.com
 ---
  drivers/net/wireless/ath/ath9k/init.c |2 --
  1 files changed, 0 insertions(+), 2 deletions(-)
 
 diff --git a/drivers/net/wireless/ath/ath9k/init.c 
 b/drivers/net/wireless/ath/ath9k/init.c
 index 53a005d..a625c9a 100644
 --- a/drivers/net/wireless/ath/ath9k/init.c
 +++ b/drivers/net/wireless/ath/ath9k/init.c
 @@ -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
___
ath9k-devel mailing list
ath9k-devel@lists.ath9k.org
https://lists.ath9k.org/mailman/listinfo/ath9k-devel


Re: [ath9k-devel] [PATCH 1/7] ath9k: Remove redundant max_streams assignment

2012-02-13 Thread Sujith Manoharan
Felix Fietkau wrote:
 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
___
ath9k-devel mailing list
ath9k-devel@lists.ath9k.org
https://lists.ath9k.org/mailman/listinfo/ath9k-devel