stmmac: let stmmac_dvr_probe take a struct of resources
authorJoachim Eastwood <manabian@gmail.com>
Wed, 20 May 2015 18:03:07 +0000 (20:03 +0200)
committerDavid S. Miller <davem@davemloft.net>
Thu, 21 May 2015 22:57:26 +0000 (18:57 -0400)
Creat a struct that contain all the resources that needs to be
assigned to the priv struct in stmmac_dvr_probe. This makes it
possible to factor out more common code from the other probe
functions and also use this struct to hold the resources as
they are fetched.

Signed-off-by: Joachim Eastwood <manabian@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/stmicro/stmmac/stmmac.h
drivers/net/ethernet/stmicro/stmmac/stmmac_main.c
drivers/net/ethernet/stmicro/stmmac/stmmac_pci.c
drivers/net/ethernet/stmicro/stmmac/stmmac_platform.c

index 2ac9552d1fa385953e261ff3797c74b8d5ad4add..b1687adb8e02695062a252a74cd131858000d5be 100644 (file)
 #include <linux/ptp_clock_kernel.h>
 #include <linux/reset.h>
 
+struct stmmac_resources {
+       void __iomem *addr;
+       const char *mac;
+       int wol_irq;
+       int lpi_irq;
+       int irq;
+};
+
 struct stmmac_tx_info {
        dma_addr_t buf;
        bool map_as_page;
@@ -131,7 +139,7 @@ int stmmac_suspend(struct net_device *ndev);
 int stmmac_dvr_remove(struct net_device *ndev);
 struct stmmac_priv *stmmac_dvr_probe(struct device *device,
                                     struct plat_stmmacenet_data *plat_dat,
-                                    void __iomem *addr);
+                                    struct stmmac_resources *res);
 void stmmac_disable_eee_mode(struct stmmac_priv *priv);
 bool stmmac_eee_init(struct stmmac_priv *priv);
 
index 396eb59768be5a7b7ccfc02622b08d1b060e96ed..c73570f9327994e03099ae09600280778586977e 100644 (file)
@@ -2797,7 +2797,7 @@ static int stmmac_hw_init(struct stmmac_priv *priv)
  * stmmac_dvr_probe
  * @device: device pointer
  * @plat_dat: platform data pointer
- * @addr: iobase memory address
+ * @res: stmmac resource pointer
  * Description: this is the main probe function used to
  * call the alloc_etherdev, allocate the priv structure.
  * Return:
@@ -2806,7 +2806,7 @@ static int stmmac_hw_init(struct stmmac_priv *priv)
  */
 struct stmmac_priv *stmmac_dvr_probe(struct device *device,
                                     struct plat_stmmacenet_data *plat_dat,
-                                    void __iomem *addr)
+                                    struct stmmac_resources *res)
 {
        int ret = 0;
        struct net_device *ndev = NULL;
@@ -2825,8 +2825,15 @@ struct stmmac_priv *stmmac_dvr_probe(struct device *device,
        stmmac_set_ethtool_ops(ndev);
        priv->pause = pause;
        priv->plat = plat_dat;
-       priv->ioaddr = addr;
-       priv->dev->base_addr = (unsigned long)addr;
+       priv->ioaddr = res->addr;
+       priv->dev->base_addr = (unsigned long)res->addr;
+
+       priv->dev->irq = res->irq;
+       priv->wol_irq = res->wol_irq;
+       priv->lpi_irq = res->lpi_irq;
+
+       if (res->mac)
+               memcpy(priv->dev->dev_addr, res->mac, ETH_ALEN);
 
        dev_set_drvdata(device, priv);
 
index 4743d0017d7996580c4ce2accb78b1ee0d099cd1..2c663ec2dfb3e700f449e655a539cb78b0aede7a 100644 (file)
@@ -163,6 +163,7 @@ static int stmmac_pci_probe(struct pci_dev *pdev,
 {
        struct stmmac_pci_info *info = (struct stmmac_pci_info *)id->driver_data;
        struct plat_stmmacenet_data *plat;
+       struct stmmac_resources res;
        struct stmmac_priv *priv;
        int i;
        int ret;
@@ -214,13 +215,16 @@ static int stmmac_pci_probe(struct pci_dev *pdev,
 
        pci_enable_msi(pdev);
 
-       priv = stmmac_dvr_probe(&pdev->dev, plat, pcim_iomap_table(pdev)[i]);
+       memset(&res, 0, sizeof(res));
+       res.addr = pcim_iomap_table(pdev)[i];
+       res.wol_irq = pdev->irq;
+       res.irq = pdev->irq;
+
+       priv = stmmac_dvr_probe(&pdev->dev, plat, &res);
        if (IS_ERR(priv)) {
                dev_err(&pdev->dev, "%s: main driver probe failed\n", __func__);
                return PTR_ERR(priv);
        }
-       priv->dev->irq = pdev->irq;
-       priv->wol_irq = pdev->irq;
 
        dev_dbg(&pdev->dev, "STMMAC PCI driver registration completed\n");
 
index 8d7e160c2c5b4db4a0a0edba353fb813dd7a4a8a..af4f0358f13fd93303f204a1bf7847e7fa480594 100644 (file)
@@ -252,25 +252,25 @@ static int stmmac_probe_config_dt(struct platform_device *pdev,
  */
 int stmmac_pltfr_probe(struct platform_device *pdev)
 {
+       struct stmmac_resources stmmac_res;
        int ret = 0;
        struct resource *res;
        struct device *dev = &pdev->dev;
-       void __iomem *addr = NULL;
        struct stmmac_priv *priv = NULL;
        struct plat_stmmacenet_data *plat_dat = NULL;
-       const char *mac = NULL;
-       int irq, wol_irq, lpi_irq;
+
+       memset(&stmmac_res, 0, sizeof(stmmac_res));
 
        /* Get IRQ information early to have an ability to ask for deferred
         * probe if needed before we went too far with resource allocation.
         */
-       irq = platform_get_irq_byname(pdev, "macirq");
-       if (irq < 0) {
-               if (irq != -EPROBE_DEFER) {
+       stmmac_res.irq = platform_get_irq_byname(pdev, "macirq");
+       if (stmmac_res.irq < 0) {
+               if (stmmac_res.irq != -EPROBE_DEFER) {
                        dev_err(dev,
                                "MAC IRQ configuration information not found\n");
                }
-               return irq;
+               return stmmac_res.irq;
        }
 
        /* On some platforms e.g. SPEAr the wake up irq differs from the mac irq
@@ -280,21 +280,21 @@ int stmmac_pltfr_probe(struct platform_device *pdev)
         * In case the wake up interrupt is not passed from the platform
         * so the driver will continue to use the mac irq (ndev->irq)
         */
-       wol_irq = platform_get_irq_byname(pdev, "eth_wake_irq");
-       if (wol_irq < 0) {
-               if (wol_irq == -EPROBE_DEFER)
+       stmmac_res.wol_irq = platform_get_irq_byname(pdev, "eth_wake_irq");
+       if (stmmac_res.wol_irq < 0) {
+               if (stmmac_res.wol_irq == -EPROBE_DEFER)
                        return -EPROBE_DEFER;
-               wol_irq = irq;
+               stmmac_res.wol_irq = stmmac_res.irq;
        }
 
-       lpi_irq = platform_get_irq_byname(pdev, "eth_lpi");
-       if (lpi_irq == -EPROBE_DEFER)
+       stmmac_res.lpi_irq = platform_get_irq_byname(pdev, "eth_lpi");
+       if (stmmac_res.lpi_irq == -EPROBE_DEFER)
                return -EPROBE_DEFER;
 
        res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-       addr = devm_ioremap_resource(dev, res);
-       if (IS_ERR(addr))
-               return PTR_ERR(addr);
+       stmmac_res.addr = devm_ioremap_resource(dev, res);
+       if (IS_ERR(stmmac_res.addr))
+               return PTR_ERR(stmmac_res.addr);
 
        plat_dat = dev_get_platdata(&pdev->dev);
 
@@ -314,7 +314,7 @@ int stmmac_pltfr_probe(struct platform_device *pdev)
        plat_dat->unicast_filter_entries = 1;
 
        if (pdev->dev.of_node) {
-               ret = stmmac_probe_config_dt(pdev, plat_dat, &mac);
+               ret = stmmac_probe_config_dt(pdev, plat_dat, &stmmac_res.mac);
                if (ret) {
                        pr_err("%s: main dt probe failed", __func__);
                        return ret;
@@ -335,21 +335,12 @@ int stmmac_pltfr_probe(struct platform_device *pdev)
                        return ret;
        }
 
-       priv = stmmac_dvr_probe(&(pdev->dev), plat_dat, addr);
+       priv = stmmac_dvr_probe(&(pdev->dev), plat_dat, &stmmac_res);
        if (IS_ERR(priv)) {
                pr_err("%s: main driver probe failed", __func__);
                return PTR_ERR(priv);
        }
 
-       /* Copy IRQ values to priv structure which is now avaialble */
-       priv->dev->irq = irq;
-       priv->wol_irq = wol_irq;
-       priv->lpi_irq = lpi_irq;
-
-       /* Get MAC address if available (DT) */
-       if (mac)
-               memcpy(priv->dev->dev_addr, mac, ETH_ALEN);
-
        pr_debug("STMMAC platform driver registration completed");
 
        return 0;