ath9k: remove redundant status variable
authorMinghao Chi <chi.minghao@zte.com.cn>
Wed, 12 Jan 2022 08:07:51 +0000 (08:07 +0000)
committerKalle Valo <quic_kvalo@quicinc.com>
Mon, 17 Jan 2022 12:43:02 +0000 (14:43 +0200)
Return value directly instead of taking this in another redundant
variable.

Reported-by: Zeal Robot <zealci@zte.com.cn>
Signed-off-by: Minghao Chi <chi.minghao@zte.com.cn>
Signed-off-by: CGEL ZTE <cgel.zte@gmail.com>
Signed-off-by: Kalle Valo <quic_kvalo@quicinc.com>
Link: https://lore.kernel.org/r/20220112080751.667316-1-chi.minghao@zte.com.cn
drivers/net/wireless/ath/ath9k/eeprom.c

index e6b3cd4..efb7889 100644 (file)
@@ -670,8 +670,6 @@ void ath9k_hw_get_gain_boundaries_pdadcs(struct ath_hw *ah,
 
 int ath9k_hw_eeprom_init(struct ath_hw *ah)
 {
-       int status;
-
        if (AR_SREV_9300_20_OR_LATER(ah))
                ah->eep_ops = &eep_ar9300_ops;
        else if (AR_SREV_9287(ah)) {
@@ -685,7 +683,5 @@ int ath9k_hw_eeprom_init(struct ath_hw *ah)
        if (!ah->eep_ops->fill_eeprom(ah))
                return -EIO;
 
-       status = ah->eep_ops->check_eeprom(ah);
-
-       return status;
+       return ah->eep_ops->check_eeprom(ah);
 }