Date: Fri, 4 Nov 2011 22:56:48 +0000 (UTC) From: Adrian Chadd <adrian@FreeBSD.org> To: src-committers@freebsd.org, svn-src-user@freebsd.org Subject: svn commit: r227096 - user/adrian/if_ath_tx/sys/dev/ath/ath_hal/ar9002 Message-ID: <201111042256.pA4MumNM038014@svn.freebsd.org>
next in thread | raw e-mail | index | archive | help
Author: adrian Date: Fri Nov 4 22:56:48 2011 New Revision: 227096 URL: http://svn.freebsd.org/changeset/base/227096 Log: Since the v14 eeprom code already returns default parameter values for txgain/rxgain types, we don't have to check the version here. Just trust the eeprom code to give us the correct value. Modified: user/adrian/if_ath_tx/sys/dev/ath/ath_hal/ar9002/ar9280_attach.c Modified: user/adrian/if_ath_tx/sys/dev/ath/ath_hal/ar9002/ar9280_attach.c ============================================================================== --- user/adrian/if_ath_tx/sys/dev/ath/ath_hal/ar9002/ar9280_attach.c Fri Nov 4 22:53:52 2011 (r227095) +++ user/adrian/if_ath_tx/sys/dev/ath/ath_hal/ar9002/ar9280_attach.c Fri Nov 4 22:56:48 2011 (r227096) @@ -131,10 +131,6 @@ ar9280InitPLL(struct ath_hal *ah, const OS_DELAY(RTC_PLL_SETTLE_DELAY); OS_REG_WRITE(ah, AR_RTC_SLEEP_CLK, AR_RTC_SLEEP_DERIVED_CLK); } - -/* XXX shouldn't be here! */ -#define EEP_MINOR(_ah) \ - (AH_PRIVATE(_ah)->ah_eeversion & AR5416_EEP_VER_MINOR_MASK) /* * Attach for an AR9280 part. @@ -312,54 +308,42 @@ ar9280Attach(uint16_t devid, HAL_SOFTC s AR5416_PWR_TABLE_OFFSET_DB, (int) pwr_table_offset); if (AR_SREV_MERLIN_20(ah)) { - if (EEP_MINOR(ah) >= AR5416_EEP_MINOR_VER_17) { - /* setup rxgain table */ - switch (ath_hal_eepromGet(ah, AR_EEP_RXGAIN_TYPE, - AH_NULL)) { - case AR5416_EEP_RXGAIN_13dB_BACKOFF: - HAL_INI_INIT(&ahp9280->ah_ini_rxgain, - ar9280Modes_backoff_13db_rxgain_v2, 6); - break; - case AR5416_EEP_RXGAIN_23dB_BACKOFF: - HAL_INI_INIT(&ahp9280->ah_ini_rxgain, - ar9280Modes_backoff_23db_rxgain_v2, 6); - break; - case AR5416_EEP_RXGAIN_ORIG: - HAL_INI_INIT(&ahp9280->ah_ini_rxgain, - ar9280Modes_original_rxgain_v2, 6); - break; - default: - HALASSERT(AH_FALSE); - goto bad; - } - } else { - /* Default to original RX gain */ + /* setup rxgain table */ + switch (ath_hal_eepromGet(ah, AR_EEP_RXGAIN_TYPE, + AH_NULL)) { + case AR5416_EEP_RXGAIN_13dB_BACKOFF: + HAL_INI_INIT(&ahp9280->ah_ini_rxgain, + ar9280Modes_backoff_13db_rxgain_v2, 6); + break; + case AR5416_EEP_RXGAIN_23dB_BACKOFF: + HAL_INI_INIT(&ahp9280->ah_ini_rxgain, + ar9280Modes_backoff_23db_rxgain_v2, 6); + break; + case AR5416_EEP_RXGAIN_ORIG: HAL_INI_INIT(&ahp9280->ah_ini_rxgain, ar9280Modes_original_rxgain_v2, 6); + break; + default: + HALASSERT(AH_FALSE); + goto bad; } } if (AR_SREV_MERLIN_20(ah)) { - if (EEP_MINOR(ah) >= AR5416_EEP_MINOR_VER_19) { - /* setp txgain table */ - switch (ath_hal_eepromGet(ah, AR_EEP_TXGAIN_TYPE, - AH_NULL)) { - case AR5416_EEP_TXGAIN_HIGH_POWER: - HAL_INI_INIT(&ahp9280->ah_ini_txgain, - ar9280Modes_high_power_tx_gain_v2, 6); - break; - case AR5416_EEP_TXGAIN_ORIG: - HAL_INI_INIT(&ahp9280->ah_ini_txgain, - ar9280Modes_original_tx_gain_v2, 6); - break; - default: - HALASSERT(AH_FALSE); - goto bad; - } - } else { - /* Default to original TX gain */ + /* setp txgain table */ + switch (ath_hal_eepromGet(ah, AR_EEP_TXGAIN_TYPE, + AH_NULL)) { + case AR5416_EEP_TXGAIN_HIGH_POWER: + HAL_INI_INIT(&ahp9280->ah_ini_txgain, + ar9280Modes_high_power_tx_gain_v2, 6); + break; + case AR5416_EEP_TXGAIN_ORIG: HAL_INI_INIT(&ahp9280->ah_ini_txgain, ar9280Modes_original_tx_gain_v2, 6); + break; + default: + HALASSERT(AH_FALSE); + goto bad; } }
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201111042256.pA4MumNM038014>