ath9k: call hw initializer directly
authorLuis R. Rodriguez <lrodriguez@atheros.com>
Mon, 3 Aug 2009 19:24:39 +0000 (12:24 -0700)
committerJohn W. Linville <linville@tuxdriver.com>
Tue, 4 Aug 2009 20:44:28 +0000 (16:44 -0400)
ath9k_hw_attach() was going first through some device id verifier,
and then calling some other helper which was doing the real hardware
initialization. Lets just do the devid checks within the real worker
by calling a helper ath9k_hw_devid_supported().

Signed-off-by: Luis R. Rodriguez <lrodriguez@atheros.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/ath/ath9k/hw.c

index ff2875b233dad551faf0b30f70ec5c5949c9e677..d8ae289a09b4e691da301d1e520aeee19532ca89 100644 (file)
@@ -608,12 +608,35 @@ static int ath9k_hw_post_attach(struct ath_hw *ah)
        return 0;
 }
 
-static int ath9k_hw_do_attach(struct ath_hw *ah,
-                             struct ath_softc *sc)
+static bool ath9k_hw_devid_supported(u16 devid)
+{
+       switch (devid) {
+       case AR5416_DEVID_PCI:
+       case AR5416_DEVID_PCIE:
+       case AR5416_AR9100_DEVID:
+       case AR9160_DEVID_PCI:
+       case AR9280_DEVID_PCI:
+       case AR9280_DEVID_PCIE:
+       case AR9285_DEVID_PCIE:
+       case AR5416_DEVID_AR9287_PCI:
+       case AR5416_DEVID_AR9287_PCIE:
+               return true;
+       default:
+               break;
+       }
+       return false;
+}
+
+int ath9k_hw_attach(struct ath_hw *ah, struct ath_softc *sc)
 {
        int r;
        u32 i, j;
 
+       if (!ath9k_hw_devid_supported(ah->hw_version.devid)) {
+               r = -EOPNOTSUPP;
+               goto bad;
+       }
+
        ath9k_hw_newstate(ah);
        ath9k_hw_set_defaults(ah);
 
@@ -1183,25 +1206,6 @@ void ath9k_hw_detach(struct ath_hw *ah)
        kfree(ah);
 }
 
-int ath9k_hw_attach(struct ath_hw *ah, struct ath_softc *sc)
-{
-       switch (ah->hw_version.devid) {
-       case AR5416_DEVID_PCI:
-       case AR5416_DEVID_PCIE:
-       case AR5416_AR9100_DEVID:
-       case AR9160_DEVID_PCI:
-       case AR9280_DEVID_PCI:
-       case AR9280_DEVID_PCIE:
-       case AR9285_DEVID_PCIE:
-       case AR5416_DEVID_AR9287_PCI:
-       case AR5416_DEVID_AR9287_PCIE:
-               return ath9k_hw_do_attach(ah, sc);
-       default:
-               break;
-       }
-       return -EOPNOTSUPP;
-}
-
 /*******/
 /* INI */
 /*******/
@@ -2898,7 +2902,7 @@ void ath9k_hw_configpcipowersave(struct ath_hw *ah, int restore)
                /*
                 * AR9280 2.0 or later chips use SerDes values from the
                 * initvals.h initialized depending on chipset during
-                * ath9k_hw_do_attach()
+                * ath9k_hw_attach()
                 */
                for (i = 0; i < ah->iniPcieSerdes.ia_rows; i++) {
                        REG_WRITE(ah, INI_RA(&ah->iniPcieSerdes, i, 0),