wl12xx: Use a single fw for both STA and AP roles
authorArik Nemtsov <arik@wizery.com>
Wed, 17 Aug 2011 07:45:48 +0000 (10:45 +0300)
committerLuciano Coelho <coelho@ti.com>
Mon, 22 Aug 2011 09:35:22 +0000 (12:35 +0300)
Firmware >= 6/7.3.0.0.75 (wl127x/wl128x) supports both
STA and AP roles.

Signed-off-by: Arik Nemtsov <arik@wizery.com>
Signed-off-by: Eliad Peller <eliad@wizery.com>
Signed-off-by: Luciano Coelho <coelho@ti.com>
drivers/net/wireless/wl12xx/main.c
drivers/net/wireless/wl12xx/sdio.c
drivers/net/wireless/wl12xx/spi.c
drivers/net/wireless/wl12xx/wl12xx.h

index f91875e379a20ed07088bd8f55158c4c06df19c9..7b29573ef2443bf8f2ad82369d955f24af728aa1 100644 (file)
@@ -1043,25 +1043,10 @@ static int wl1271_fetch_firmware(struct wl1271 *wl)
        const char *fw_name;
        int ret;
 
-       switch (wl->bss_type) {
-       case BSS_TYPE_AP_BSS:
-               if (wl->chip.id == CHIP_ID_1283_PG20)
-                       fw_name = WL128X_AP_FW_NAME;
-               else
-                       fw_name = WL127X_AP_FW_NAME;
-               break;
-       case BSS_TYPE_IBSS:
-       case BSS_TYPE_STA_BSS:
-               if (wl->chip.id == CHIP_ID_1283_PG20)
-                       fw_name = WL128X_FW_NAME;
-               else
-                       fw_name = WL1271_FW_NAME;
-               break;
-       default:
-               wl1271_error("no compatible firmware for bss_type %d",
-                            wl->bss_type);
-               return -EINVAL;
-       }
+       if (wl->chip.id == CHIP_ID_1283_PG20)
+               fw_name = WL128X_FW_NAME;
+       else
+               fw_name = WL127X_FW_NAME;
 
        wl1271_debug(DEBUG_BOOT, "booting firmware %s", fw_name);
 
@@ -1090,7 +1075,6 @@ static int wl1271_fetch_firmware(struct wl1271 *wl)
        }
 
        memcpy(wl->fw, fw->data, wl->fw_len);
-       wl->fw_bss_type = wl->bss_type;
        ret = 0;
 
 out:
@@ -1361,8 +1345,7 @@ static int wl1271_chip_wakeup(struct wl1271 *wl)
                goto out;
        }
 
-       /* Make sure the firmware type matches the BSS type */
-       if (wl->fw == NULL || wl->fw_bss_type != wl->bss_type) {
+       if (wl->fw == NULL) {
                ret = wl1271_fetch_firmware(wl);
                if (ret < 0)
                        goto out;
@@ -1796,9 +1779,6 @@ static int wl1271_op_start(struct ieee80211_hw *hw)
         *
         * The MAC address is first known when the corresponding interface
         * is added. That is where we will initialize the hardware.
-        *
-        * In addition, we currently have different firmwares for AP and managed
-        * operation. We will know which to boot according to interface type.
         */
 
        return 0;
@@ -4393,7 +4373,6 @@ struct ieee80211_hw *wl1271_alloc_hw(void)
        wl->hw_pg_ver = -1;
        wl->bss_type = MAX_BSS_TYPE;
        wl->set_bss_type = MAX_BSS_TYPE;
-       wl->fw_bss_type = MAX_BSS_TYPE;
        wl->last_tx_hlid = 0;
        wl->ap_ps_map = 0;
        wl->ap_fw_ps_map = 0;
index 5cf18c2c23f09ee3fdd35a3aceafc322f3770580..ac2e5661397c89816ad12eac1224a99d776fb785 100644 (file)
@@ -412,7 +412,5 @@ module_exit(wl1271_exit);
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Luciano Coelho <coelho@ti.com>");
 MODULE_AUTHOR("Juuso Oikarinen <juuso.oikarinen@nokia.com>");
-MODULE_FIRMWARE(WL1271_FW_NAME);
+MODULE_FIRMWARE(WL127X_FW_NAME);
 MODULE_FIRMWARE(WL128X_FW_NAME);
-MODULE_FIRMWARE(WL127X_AP_FW_NAME);
-MODULE_FIRMWARE(WL128X_AP_FW_NAME);
index e0b3736d7e1921886d7b3e1882efb9eed11beaaa..0f971867786014bbb56e2c7a074109a604ec619a 100644 (file)
@@ -486,8 +486,6 @@ module_exit(wl1271_exit);
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Luciano Coelho <coelho@ti.com>");
 MODULE_AUTHOR("Juuso Oikarinen <juuso.oikarinen@nokia.com>");
-MODULE_FIRMWARE(WL1271_FW_NAME);
+MODULE_FIRMWARE(WL127X_FW_NAME);
 MODULE_FIRMWARE(WL128X_FW_NAME);
-MODULE_FIRMWARE(WL127X_AP_FW_NAME);
-MODULE_FIRMWARE(WL128X_AP_FW_NAME);
 MODULE_ALIAS("spi:wl1271");
index 5b00a84acab2242b84652b6f41da2758288340ec..dc0013e9b9f7ec1dd7256afc402687af1dece119 100644 (file)
@@ -130,10 +130,8 @@ extern u32 wl12xx_debug_level;
 
 
 
-#define WL1271_FW_NAME "ti-connectivity/wl1271-fw-2.bin"
-#define WL128X_FW_NAME "ti-connectivity/wl128x-fw.bin"
-#define WL127X_AP_FW_NAME "ti-connectivity/wl1271-fw-ap.bin"
-#define WL128X_AP_FW_NAME "ti-connectivity/wl128x-fw-ap.bin"
+#define WL127X_FW_NAME "ti-connectivity/wl127x-fw-3.bin"
+#define WL128X_FW_NAME "ti-connectivity/wl128x-fw-3.bin"
 
 /*
  * wl127x and wl128x are using the same NVS file name. However, the
@@ -405,7 +403,6 @@ struct wl1271 {
 
        u8 *fw;
        size_t fw_len;
-       u8 fw_bss_type;
        void *nvs;
        size_t nvs_len;