On Mon, Oct 15, 2012 at 02:56:36PM +0530, Sujith Manoharan wrote:
> Rajkumar Manoharan wrote:
> > Signed-off-by: Rajkumar Manoharan <rmanohar@xxxxxxxxxxxxxxxx>
> > ---
> > drivers/net/wireless/ath/ath9k/ar9003_eeprom.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/net/wireless/ath/ath9k/ar9003_eeprom.c b/drivers/net/wireless/ath/ath9k/ar9003_eeprom.c
> > index f3784eb..8621e37 100644
> > --- a/drivers/net/wireless/ath/ath9k/ar9003_eeprom.c
> > +++ b/drivers/net/wireless/ath/ath9k/ar9003_eeprom.c
> > @@ -3601,7 +3601,7 @@ static void ar9003_hw_ant_ctrl_apply(struct ath_hw *ah, bool is2ghz)
> > * 7:4 R/W SWITCH_TABLE_COM_SPDT_WLAN_IDLE
> > * SWITCH_TABLE_COM_SPDT_WLAN_IDLE
> > */
> > - if (AR_SREV_9462_20_OR_LATER(ah)) {
> > + if (AR_SREV_9462_20_OR_LATER(ah) || AR_SREV_9565(ah)) {
> > value = ar9003_switch_com_spdt_get(ah, is2ghz);
> > REG_RMW_FIELD(ah, AR_PHY_GLB_CONTROL,
> > AR_SWITCH_TABLE_COM_SPDT_ALL, value);
>
> A cleaner way would be to fix the AR_SREV_9462_20_OR_LATER macro to resemble
> all the other *_LATER macros (based on the macVersion).
>
I hate *_LATER macros. Its always confusing. Since SPDT is applicable only for
9462 & 9565, I remove the _LATER in v2.
-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
