wl12xx/wlcore: initial split of probe
authorLuciano Coelho <coelho@ti.com>
Mon, 21 Nov 2011 16:55:51 +0000 (18:55 +0200)
committerLuciano Coelho <coelho@ti.com>
Thu, 12 Apr 2012 05:43:56 +0000 (08:43 +0300)
We need to set some parameters (eg. partition and register tables)
during probe of the lower driver, so split the probe function, leaving
most of it in wlcore, but moving the hw struct allocation to the lower
driver.

Signed-off-by: Luciano Coelho <coelho@ti.com>
drivers/net/wireless/ti/wl12xx/main.c
drivers/net/wireless/ti/wlcore/main.c
drivers/net/wireless/ti/wlcore/wlcore.h

index e6c350232f0ce4797dfc0ce7f55f5bdb3e9656fd..c4fc93f5718114f2afb3e63d1ec970e7574b7da5 100644 (file)
 #include <linux/module.h>
 #include <linux/platform_device.h>
 
+#include <linux/err.h>
+
 #include "../wlcore/wlcore.h"
+#include "../wlcore/debug.h"
+
+static int __devinit wl12xx_probe(struct platform_device *pdev)
+{
+       struct wl1271 *wl;
+       struct ieee80211_hw *hw;
+
+       hw = wlcore_alloc_hw();
+       if (IS_ERR(hw)) {
+               wl1271_error("can't allocate hw");
+               return PTR_ERR(hw);
+       }
+
+       wl = hw->priv;
+
+       return wlcore_probe(wl, pdev);
+}
 
 static const struct platform_device_id wl12xx_id_table[] __devinitconst = {
        { "wl12xx", 0 },
@@ -31,7 +50,7 @@ static const struct platform_device_id wl12xx_id_table[] __devinitconst = {
 MODULE_DEVICE_TABLE(platform, wl12xx_id_table);
 
 static struct platform_driver wl12xx_driver = {
-       .probe          = wlcore_probe,
+       .probe          = wl12xx_probe,
        .remove         = __devexit_p(wlcore_remove),
        .id_table       = wl12xx_id_table,
        .driver = {
index 651536ee816d5d891aa25a93cccb13d672275a9f..3cbc774f105767f129d2f3577ba468342a307ef8 100644 (file)
@@ -5287,7 +5287,7 @@ static int wl1271_init_ieee80211(struct wl1271 *wl)
 
 #define WL1271_DEFAULT_CHANNEL 0
 
-static struct ieee80211_hw *wl1271_alloc_hw(void)
+struct ieee80211_hw *wlcore_alloc_hw(void)
 {
        struct ieee80211_hw *hw;
        struct wl1271 *wl;
@@ -5412,8 +5412,9 @@ err_hw_alloc:
 
        return ERR_PTR(ret);
 }
+EXPORT_SYMBOL_GPL(wlcore_alloc_hw);
 
-static int wl1271_free_hw(struct wl1271 *wl)
+int wlcore_free_hw(struct wl1271 *wl)
 {
        /* Unblock any fwlog readers */
        mutex_lock(&wl->mutex);
@@ -5447,6 +5448,7 @@ static int wl1271_free_hw(struct wl1271 *wl)
 
        return 0;
 }
+EXPORT_SYMBOL_GPL(wlcore_free_hw);
 
 static irqreturn_t wl12xx_hardirq(int irq, void *cookie)
 {
@@ -5477,22 +5479,12 @@ static irqreturn_t wl12xx_hardirq(int irq, void *cookie)
        return IRQ_WAKE_THREAD;
 }
 
-int __devinit wlcore_probe(struct platform_device *pdev)
+int __devinit wlcore_probe(struct wl1271 *wl, struct platform_device *pdev)
 {
        struct wl12xx_platform_data *pdata = pdev->dev.platform_data;
-       struct ieee80211_hw *hw;
-       struct wl1271 *wl;
        unsigned long irqflags;
-       int ret = -ENODEV;
-
-       hw = wl1271_alloc_hw();
-       if (IS_ERR(hw)) {
-               wl1271_error("can't allocate hw");
-               ret = PTR_ERR(hw);
-               goto out;
-       }
+       int ret;
 
-       wl = hw->priv;
        wl->irq = platform_get_irq(pdev, 0);
        wl->ref_clock = pdata->board_ref_clock;
        wl->tcxo_clock = pdata->board_tcxo_clock;
@@ -5521,7 +5513,7 @@ int __devinit wlcore_probe(struct platform_device *pdev)
                wl->irq_wake_enabled = true;
                device_init_wakeup(wl->dev, 1);
                if (pdata->pwr_in_suspend)
-                       hw->wiphy->wowlan.flags = WIPHY_WOWLAN_ANY;
+                       wl->hw->wiphy->wowlan.flags = WIPHY_WOWLAN_ANY;
 
        }
        disable_irq(wl->irq);
@@ -5555,7 +5547,7 @@ int __devinit wlcore_probe(struct platform_device *pdev)
                goto out_hw_pg_ver;
        }
 
-       return 0;
+       goto out;
 
 out_hw_pg_ver:
        device_remove_file(wl->dev, &dev_attr_hw_pg_ver);
@@ -5567,7 +5559,7 @@ out_irq:
        free_irq(wl->irq, wl);
 
 out_free_hw:
-       wl1271_free_hw(wl);
+       wlcore_free_hw(wl);
 
 out:
        return ret;
@@ -5584,7 +5576,7 @@ int __devexit wlcore_remove(struct platform_device *pdev)
        }
        wl1271_unregister_hw(wl);
        free_irq(wl->irq, wl);
-       wl1271_free_hw(wl);
+       wlcore_free_hw(wl);
 
        return 0;
 }
index e0187d7b534487fae4c46cb68ef32f415095c507..4cabf971da510ead8b7d3a6f94e31bc5d5e52b90 100644 (file)
 #ifndef __WLCORE_H__
 #define __WLCORE_H__
 
-int __devinit wlcore_probe(struct platform_device *pdev);
+#include "wl12xx.h"
+
+int __devinit wlcore_probe(struct wl1271 *wl, struct platform_device *pdev);
 int __devexit wlcore_remove(struct platform_device *pdev);
+struct ieee80211_hw *wlcore_alloc_hw(void);
+int wlcore_free_hw(struct wl1271 *wl);
+
 
 #endif /* __WLCORE_H__ */