On Fri, Dec 07, 2012 at 03:31:55PM +0530, Sujith Manoharan wrote: > From: Sujith Manoharan <c_manoha@xxxxxxxxxxxxxxxx> > > The PAPRD training control registers have to be > programmed with values that depend on the chip. This patch > ensures that the correct values are chosen for the chip > in use. > > Signed-off-by: Sujith Manoharan <c_manoha@xxxxxxxxxxxxxxxx> > --- > drivers/net/wireless/ath/ath9k/ar9003_paprd.c | 36 +++++++++++++++++++++------ > 1 file changed, 29 insertions(+), 7 deletions(-) > > diff --git a/drivers/net/wireless/ath/ath9k/ar9003_paprd.c b/drivers/net/wireless/ath/ath9k/ar9003_paprd.c > index 0ed3846..2cd23e5 100644 > --- a/drivers/net/wireless/ath/ath9k/ar9003_paprd.c > +++ b/drivers/net/wireless/ath/ath9k/ar9003_paprd.c > @@ -204,7 +204,20 @@ static int ar9003_paprd_setup_single_table(struct ath_hw *ah) > AR_PHY_PAPRD_TRAINER_CNTL1_CF_PAPRD_AGC2_SETTLING, 28); > REG_RMW_FIELD(ah, AR_PHY_PAPRD_TRAINER_CNTL1, > AR_PHY_PAPRD_TRAINER_CNTL1_CF_CF_PAPRD_TRAIN_ENABLE, 1); > - val = AR_SREV_9462(ah) ? 0x91 : 147; > + > + if (AR_SREV_9485(ah)) { > + val = 148; > + } else { > + if (IS_CHAN_2GHZ(ah->curchan)) { > + if (AR_SREV_9462(ah) || AR_SREV_9565(ah)) > + val = 0x91; Please change the hex to decimal for the sake of uniformity > + else > + val = 147; > + } else { > + val = 137; > + } > + } -Rajkumar -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html