net: fec_mxc: simplify fec_get_miibus
authorPeng Fan <peng.fan@nxp.com>
Wed, 28 Mar 2018 12:54:12 +0000 (20:54 +0800)
committerJoe Hershberger <joe.hershberger@ni.com>
Fri, 13 Apr 2018 20:47:37 +0000 (15:47 -0500)
No need to provide two prototype for this function.
Use ulong for the first parameter, then this function
could be shared for DM/non DM case.

Signed-off-by: Peng Fan <peng.fan@nxp.com>
Acked-by: Joe Hershberger <joe.hershberger@ni.com>
drivers/net/fec_mxc.c
include/netdev.h

index 450cc0b8ea2137005be64881d38bf3c98b8f0513..74d36f479ec9b59301cb2002c94e783986c0e564 100644 (file)
@@ -1012,18 +1012,9 @@ static void fec_free_descs(struct fec_priv *fec)
        free(fec->tbd_base);
 }
 
-#ifdef CONFIG_DM_ETH
-struct mii_dev *fec_get_miibus(struct udevice *dev, int dev_id)
-#else
-struct mii_dev *fec_get_miibus(uint32_t base_addr, int dev_id)
-#endif
+struct mii_dev *fec_get_miibus(ulong base_addr, int dev_id)
 {
-#ifdef CONFIG_DM_ETH
-       struct fec_priv *priv = dev_get_priv(dev);
-       struct ethernet_regs *eth = priv->eth;
-#else
-       struct ethernet_regs *eth = (struct ethernet_regs *)(ulong)base_addr;
-#endif
+       struct ethernet_regs *eth = (struct ethernet_regs *)base_addr;
        struct mii_dev *bus;
        int ret;
 
@@ -1283,7 +1274,7 @@ static int fecmxc_probe(struct udevice *dev)
        fec_reg_setup(priv);
        priv->dev_id = (dev_id == -1) ? 0 : dev_id;
 
-       bus = fec_get_miibus(dev, dev_id);
+       bus = fec_get_miibus((ulong)priv->eth, dev_id);
        if (!bus) {
                ret = -ENOMEM;
                goto err_mii;
index 86d28ade14b6d5c14c9013428f923d2ce41629ad..68c6d49cbfc9835d3761d38e258acb3b477bcc2d 100644 (file)
@@ -117,11 +117,7 @@ static inline int pci_eth_init(bd_t *bis)
        return num;
 }
 
-#ifdef CONFIG_DM_ETH
-struct mii_dev *fec_get_miibus(struct udevice *dev, int dev_id);
-#else
-struct mii_dev *fec_get_miibus(uint32_t base_addr, int dev_id);
-#endif
+struct mii_dev *fec_get_miibus(ulong base_addr, int dev_id);
 
 #ifdef CONFIG_PHYLIB
 struct phy_device;