orinoco: firmware: consistently compile out fw cache support if not requested
authorAndrey Borzenkov <arvidjaar@mail.ru>
Sat, 28 Feb 2009 20:09:09 +0000 (23:09 +0300)
committerJohn W. Linville <linville@tuxdriver.com>
Mon, 16 Mar 2009 22:09:26 +0000 (18:09 -0400)
Currently part of support for FW caching is unconditionally compiled
in even if it is never used. Consistently remove caching support if
not requested by user.

Signed-off-by: Andrey Borzenkov <arvidjaar@mail.ru>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/orinoco/fw.c
drivers/net/wireless/orinoco/fw.h
drivers/net/wireless/orinoco/main.c
drivers/net/wireless/orinoco/orinoco.h

index e7abb45d6753bc5cad4aba31604d529f7caf95ee..1084b43e04bc62b1c6b9c52eae0047911e1673d9 100644 (file)
@@ -70,6 +70,19 @@ static const char *validate_fw(const struct orinoco_fw_header *hdr, size_t len)
        return NULL;
 }
 
+#if defined(CONFIG_HERMES_CACHE_FW_ON_INIT) || defined(CONFIG_PM_SLEEP)
+static inline const struct firmware *
+orinoco_cached_fw_get(struct orinoco_private *priv, bool primary)
+{
+       if (primary)
+               return priv->cached_pri_fw;
+       else
+               return priv->cached_fw;
+}
+#else
+#define orinoco_cached_fw_get(priv, primary) (NULL)
+#endif
+
 /* Download either STA or AP firmware into the card. */
 static int
 orinoco_dl_firmware(struct orinoco_private *priv,
@@ -107,7 +120,7 @@ orinoco_dl_firmware(struct orinoco_private *priv,
        if (err)
                goto free;
 
-       if (!priv->cached_fw) {
+       if (!orinoco_cached_fw_get(priv, false)) {
                err = request_firmware(&fw_entry, firmware, priv->dev);
 
                if (err) {
@@ -117,7 +130,7 @@ orinoco_dl_firmware(struct orinoco_private *priv,
                        goto free;
                }
        } else
-               fw_entry = priv->cached_fw;
+               fw_entry = orinoco_cached_fw_get(priv, false);
 
        hdr = (const struct orinoco_fw_header *) fw_entry->data;
 
@@ -170,7 +183,7 @@ orinoco_dl_firmware(struct orinoco_private *priv,
 
 abort:
        /* If we requested the firmware, release it. */
-       if (!priv->cached_fw)
+       if (!orinoco_cached_fw_get(priv, false))
                release_firmware(fw_entry);
 
 free:
@@ -273,20 +286,20 @@ symbol_dl_firmware(struct orinoco_private *priv,
        int ret;
        const struct firmware *fw_entry;
 
-       if (!priv->cached_pri_fw) {
+       if (!orinoco_cached_fw_get(priv, true)) {
                if (request_firmware(&fw_entry, fw->pri_fw, priv->dev) != 0) {
                        printk(KERN_ERR "%s: Cannot find firmware: %s\n",
                               dev->name, fw->pri_fw);
                        return -ENOENT;
                }
        } else
-               fw_entry = priv->cached_pri_fw;
+               fw_entry = orinoco_cached_fw_get(priv, true);
 
        /* Load primary firmware */
        ret = symbol_dl_image(priv, fw, fw_entry->data,
                              fw_entry->data + fw_entry->size, 0);
 
-       if (!priv->cached_pri_fw)
+       if (!orinoco_cached_fw_get(priv, true))
                release_firmware(fw_entry);
        if (ret) {
                printk(KERN_ERR "%s: Primary firmware download failed\n",
@@ -294,19 +307,19 @@ symbol_dl_firmware(struct orinoco_private *priv,
                return ret;
        }
 
-       if (!priv->cached_fw) {
+       if (!orinoco_cached_fw_get(priv, false)) {
                if (request_firmware(&fw_entry, fw->sta_fw, priv->dev) != 0) {
                        printk(KERN_ERR "%s: Cannot find firmware: %s\n",
                               dev->name, fw->sta_fw);
                        return -ENOENT;
                }
        } else
-               fw_entry = priv->cached_fw;
+               fw_entry = orinoco_cached_fw_get(priv, false);
 
        /* Load secondary firmware */
        ret = symbol_dl_image(priv, fw, fw_entry->data,
                              fw_entry->data + fw_entry->size, 1);
-       if (!priv->cached_fw)
+       if (!orinoco_cached_fw_get(priv, false))
                release_firmware(fw_entry);
        if (ret) {
                printk(KERN_ERR "%s: Secondary firmware download failed\n",
@@ -340,9 +353,9 @@ int orinoco_download(struct orinoco_private *priv)
        return err;
 }
 
+#if defined(CONFIG_HERMES_CACHE_FW_ON_INIT) || defined(CONFIG_PM_SLEEP)
 void orinoco_cache_fw(struct orinoco_private *priv, int ap)
 {
-#if defined(CONFIG_HERMES_CACHE_FW_ON_INIT) || defined(CONFIG_PM_SLEEP)
        const struct firmware *fw_entry = NULL;
        const char *pri_fw;
        const char *fw;
@@ -362,12 +375,10 @@ void orinoco_cache_fw(struct orinoco_private *priv, int ap)
                if (request_firmware(&fw_entry, fw, priv->dev) == 0)
                        priv->cached_fw = fw_entry;
        }
-#endif
 }
 
 void orinoco_uncache_fw(struct orinoco_private *priv)
 {
-#if defined(CONFIG_HERMES_CACHE_FW_ON_INIT) || defined(CONFIG_PM_SLEEP)
        if (priv->cached_pri_fw)
                release_firmware(priv->cached_pri_fw);
        if (priv->cached_fw)
@@ -375,5 +386,5 @@ void orinoco_uncache_fw(struct orinoco_private *priv)
 
        priv->cached_pri_fw = NULL;
        priv->cached_fw = NULL;
-#endif
 }
+#endif
index 2290f0845d598f389bb2d732d7f89ce64351f611..89fc26d25b0644a1539c4cc844a56207aa923006 100644 (file)
@@ -10,7 +10,12 @@ struct orinoco_private;
 
 int orinoco_download(struct orinoco_private *priv);
 
+#if defined(CONFIG_HERMES_CACHE_FW_ON_INIT) || defined(CONFIG_PM_SLEEP)
 void orinoco_cache_fw(struct orinoco_private *priv, int ap);
 void orinoco_uncache_fw(struct orinoco_private *priv);
+#else
+#define orinoco_cache_fw(priv, ap) do { } while(0)
+#define orinoco_uncache_fw(priv) do { } while (0)
+#endif
 
 #endif /* _ORINOCO_FW_H_ */
index e9b1db77a73595e11d9025f95e0206f1b7144b44..345593c4accbe3c2f007acd9836e66a9ff8375e0 100644 (file)
@@ -2580,8 +2580,10 @@ struct net_device
        netif_carrier_off(dev);
        priv->last_linkstatus = 0xffff;
 
+#if defined(CONFIG_HERMES_CACHE_FW_ON_INIT) || defined(CONFIG_PM_SLEEP)
        priv->cached_pri_fw = NULL;
        priv->cached_fw = NULL;
+#endif
 
        /* Register PM notifiers */
        orinoco_register_pm_notifier(priv);
index f3f94b28ce6df3b57b3f65f48e77aac2163abd22..8e5a72cc297f9c051c7bc3493f42db09884207b0 100644 (file)
@@ -159,9 +159,11 @@ struct orinoco_private {
        unsigned int tkip_cm_active:1;
        unsigned int key_mgmt:3;
 
+#if defined(CONFIG_HERMES_CACHE_FW_ON_INIT) || defined(CONFIG_PM_SLEEP)
        /* Cached in memory firmware to use during ->resume. */
        const struct firmware *cached_pri_fw;
        const struct firmware *cached_fw;
+#endif
 
        struct notifier_block pm_notifier;
 };