net: ethernet: support of_get_mac_address new ERR_PTR error

There was NVMEM support added to of_get_mac_address, so it could now
return ERR_PTR encoded error values, so we need to adjust all current
users of of_get_mac_address to this new fact.

While at it, remove superfluous is_valid_ether_addr as the MAC address
returned from of_get_mac_address is always valid and checked by
is_valid_ether_addr anyway.

Fixes: d01f449c008a ("of_net: add NVMEM support to of_get_mac_address")
Signed-off-by: Petr Štetiar <ynezz@true.cz>
Signed-off-by: David S. Miller <davem@davemloft.net>

diff --git a/drivers/net/ethernet/aeroflex/greth.c b/drivers/net/ethernet/aeroflex/greth.c
index 47e5984f16fbdda511a686aed2b5144990083516..7c5cf0224a70023f1e14f0baad9b4c100b26e3b3 100644 (file)
--- a/drivers/net/ethernet/aeroflex/greth.c
+++ b/drivers/net/ethernet/aeroflex/greth.c
@@ -1459,7 +1459,7 @@ static int greth_of_probe(struct platform_device *ofdev)
                const u8 *addr;
 
                addr = of_get_mac_address(ofdev->dev.of_node);
-               if (addr) {
+               if (!IS_ERR(addr)) {
                        for (i = 0; i < 6; i++)
                                macaddr[i] = (unsigned int) addr[i];
                } else {

diff --git a/drivers/net/ethernet/allwinner/sun4i-emac.c b/drivers/net/ethernet/allwinner/sun4i-emac.c
index e1acafa82214133044765421109a84ff2168dd91..37ebd890ef519a9fdee51bf9cc5f8d0946d9e2e5 100644 (file)
--- a/drivers/net/ethernet/allwinner/sun4i-emac.c
+++ b/drivers/net/ethernet/allwinner/sun4i-emac.c
@@ -870,7 +870,7 @@ static int emac_probe(struct platform_device *pdev)
 
        /* Read MAC-address from DT */
        mac_addr = of_get_mac_address(np);
-       if (mac_addr)
+       if (!IS_ERR(mac_addr))
                memcpy(ndev->dev_addr, mac_addr, ETH_ALEN);
 
        /* Check if the MAC address is valid, if not get a random one */

diff --git a/drivers/net/ethernet/altera/altera_tse_main.c b/drivers/net/ethernet/altera/altera_tse_main.c
index aa1d1f5339d2a4b78ee2092d0c9683e1262e2581..877e67f4344b498e8d7907dea60feeeb4e5daf65 100644 (file)
--- a/drivers/net/ethernet/altera/altera_tse_main.c
+++ b/drivers/net/ethernet/altera/altera_tse_main.c
@@ -1537,7 +1537,7 @@ static int altera_tse_probe(struct platform_device *pdev)
 
        /* get default MAC address from device tree */
        macaddr = of_get_mac_address(pdev->dev.of_node);
-       if (macaddr)
+       if (!IS_ERR(macaddr))
                ether_addr_copy(ndev->dev_addr, macaddr);
        else
                eth_hw_addr_random(ndev);

diff --git a/drivers/net/ethernet/arc/emac_main.c b/drivers/net/ethernet/arc/emac_main.c
index ff3d68532f5fbca48c9e19dca103cce3f055b6f6..7f89ad5c336d66d9676dac2f571ed02199ecd2df 100644 (file)
--- a/drivers/net/ethernet/arc/emac_main.c
+++ b/drivers/net/ethernet/arc/emac_main.c
@@ -960,7 +960,7 @@ int arc_emac_probe(struct net_device *ndev, int interface)
        /* Get MAC address from device tree */
        mac_addr = of_get_mac_address(dev->of_node);
 
-       if (mac_addr)
+       if (!IS_ERR(mac_addr))
                memcpy(ndev->dev_addr, mac_addr, ETH_ALEN);
        else
                eth_hw_addr_random(ndev);

diff --git a/drivers/net/ethernet/aurora/nb8800.c b/drivers/net/ethernet/aurora/nb8800.c
index f62deeb6e9418859d52c7b379718942f9d92752d..3c4967eecef12cf43b0890f2a66440fd1096c6ac 100644 (file)
--- a/drivers/net/ethernet/aurora/nb8800.c
+++ b/drivers/net/ethernet/aurora/nb8800.c
@@ -1463,7 +1463,7 @@ static int nb8800_probe(struct platform_device *pdev)
        dev->irq = irq;
 
        mac = of_get_mac_address(pdev->dev.of_node);
-       if (mac)
+       if (!IS_ERR(mac))
                ether_addr_copy(dev->dev_addr, mac);
 
        if (!is_valid_ether_addr(dev->dev_addr))

diff --git a/drivers/net/ethernet/broadcom/bcmsysport.c b/drivers/net/ethernet/broadcom/bcmsysport.c
index 4e87a303f83e7a2556b9fdcba32b493f7624cf63..c623896e3ccb857ea47649da4da305685ce4929f 100644 (file)
--- a/drivers/net/ethernet/broadcom/bcmsysport.c
+++ b/drivers/net/ethernet/broadcom/bcmsysport.c
@@ -2505,7 +2505,7 @@ static int bcm_sysport_probe(struct platform_device *pdev)
 
        /* Initialize netdevice members */
        macaddr = of_get_mac_address(dn);
-       if (!macaddr || !is_valid_ether_addr(macaddr)) {
+       if (IS_ERR(macaddr)) {
                dev_warn(&pdev->dev, "using random Ethernet MAC\n");
                eth_hw_addr_random(dev);
        } else {

diff --git a/drivers/net/ethernet/broadcom/bgmac-bcma.c b/drivers/net/ethernet/broadcom/bgmac-bcma.c
index 6fe074c1588bb7c4568be9ace93cd5f754bab33a..34d18302b1a38e179be8615101f979768a1fd4fb 100644 (file)
--- a/drivers/net/ethernet/broadcom/bgmac-bcma.c
+++ b/drivers/net/ethernet/broadcom/bgmac-bcma.c
@@ -132,7 +132,7 @@ static int bgmac_probe(struct bcma_device *core)
                mac = of_get_mac_address(bgmac->dev->of_node);
 
        /* If no MAC address assigned via device tree, check SPROM */
-       if (!mac) {
+       if (IS_ERR_OR_NULL(mac)) {
                switch (core->core_unit) {
                case 0:
                        mac = sprom->et0mac;

diff --git a/drivers/net/ethernet/broadcom/bgmac-platform.c b/drivers/net/ethernet/broadcom/bgmac-platform.c
index 894eda5b13cf8fd0daf2fec8cf5131679708027b..6dc0dd91ad11e815e1f2fbcb7efbf64c1e36d078 100644 (file)
--- a/drivers/net/ethernet/broadcom/bgmac-platform.c
+++ b/drivers/net/ethernet/broadcom/bgmac-platform.c
@@ -193,7 +193,7 @@ static int bgmac_probe(struct platform_device *pdev)
        bgmac->dma_dev = &pdev->dev;
 
        mac_addr = of_get_mac_address(np);
-       if (mac_addr)
+       if (!IS_ERR(mac_addr))
                ether_addr_copy(bgmac->net_dev->dev_addr, mac_addr);
        else
                dev_warn(&pdev->dev, "MAC address not present in device tree\n");

diff --git a/drivers/net/ethernet/broadcom/genet/bcmgenet.c b/drivers/net/ethernet/broadcom/genet/bcmgenet.c
index 4fd973571e4cd245d819c75f345de737d0fb4269..374b9ff05c88727ed485dfcb0d27e79a23c97180 100644 (file)
--- a/drivers/net/ethernet/broadcom/genet/bcmgenet.c
+++ b/drivers/net/ethernet/broadcom/genet/bcmgenet.c
@@ -3476,7 +3476,7 @@ static int bcmgenet_probe(struct platform_device *pdev)
 
        if (dn) {
                macaddr = of_get_mac_address(dn);
-               if (!macaddr) {
+               if (IS_ERR(macaddr)) {
                        dev_err(&pdev->dev, "can't find MAC address\n");
                        err = -EINVAL;
                        goto err;

diff --git a/drivers/net/ethernet/cavium/octeon/octeon_mgmt.c b/drivers/net/ethernet/cavium/octeon/octeon_mgmt.c
index 5359c1021f428b04bc13ad2de9cc437fc47596cb..15b1130aa4aee45602c3185d6e0c9df1610b1284 100644 (file)
--- a/drivers/net/ethernet/cavium/octeon/octeon_mgmt.c
+++ b/drivers/net/ethernet/cavium/octeon/octeon_mgmt.c
@@ -1503,7 +1503,7 @@ static int octeon_mgmt_probe(struct platform_device *pdev)
 
        mac = of_get_mac_address(pdev->dev.of_node);
 
-       if (mac)
+       if (!IS_ERR(mac))
                memcpy(netdev->dev_addr, mac, ETH_ALEN);
        else
                eth_hw_addr_random(netdev);

diff --git a/drivers/net/ethernet/cavium/thunder/thunder_bgx.c b/drivers/net/ethernet/cavium/thunder/thunder_bgx.c
index 81c281ada63b32bab35fa1da242852c0bb83135f..a65be851124f4b31841afaf62c859a4af75231f7 100644 (file)
--- a/drivers/net/ethernet/cavium/thunder/thunder_bgx.c
+++ b/drivers/net/ethernet/cavium/thunder/thunder_bgx.c
@@ -1484,7 +1484,7 @@ static int bgx_init_of_phy(struct bgx *bgx)
                        break;
 
                mac = of_get_mac_address(node);
-               if (mac)
+               if (!IS_ERR(mac))
                        ether_addr_copy(bgx->lmac[lmac].mac, mac);
 
                SET_NETDEV_DEV(&bgx->lmac[lmac].netdev, &bgx->pdev->dev);

diff --git a/drivers/net/ethernet/davicom/dm9000.c b/drivers/net/ethernet/davicom/dm9000.c
index c2586f44c29dc8a42eab98369ac4f52260c82e1a..953ee5616801113de773a3f6fc60a4773b92e133 100644 (file)
--- a/drivers/net/ethernet/davicom/dm9000.c
+++ b/drivers/net/ethernet/davicom/dm9000.c
@@ -1412,7 +1412,7 @@ static struct dm9000_plat_data *dm9000_parse_dt(struct device *dev)
                pdata->flags |= DM9000_PLATF_NO_EEPROM;
 
        mac_addr = of_get_mac_address(np);
-       if (mac_addr)
+       if (!IS_ERR(mac_addr))
                memcpy(pdata->dev_addr, mac_addr, sizeof(pdata->dev_addr));
 
        return pdata;

diff --git a/drivers/net/ethernet/ethoc.c b/drivers/net/ethernet/ethoc.c
index 0f3e7f21c6fa59e363fc2ee779f2fd1943022642..71da0490521b834eff62bcdf4216fd128c61ef9c 100644 (file)
--- a/drivers/net/ethernet/ethoc.c
+++ b/drivers/net/ethernet/ethoc.c
@@ -1153,7 +1153,7 @@ static int ethoc_probe(struct platform_device *pdev)
                const void *mac;
 
                mac = of_get_mac_address(pdev->dev.of_node);
-               if (mac)
+               if (!IS_ERR(mac))
                        ether_addr_copy(netdev->dev_addr, mac);
                priv->phy_id = -1;
        }

diff --git a/drivers/net/ethernet/ezchip/nps_enet.c b/drivers/net/ethernet/ezchip/nps_enet.c
index 659f1ad37e96a823a4ee3cba04be905d5c9c5dee..b4ce261550873a0b47c9ce0365862f126c20bd3f 100644 (file)
--- a/drivers/net/ethernet/ezchip/nps_enet.c
+++ b/drivers/net/ethernet/ezchip/nps_enet.c
@@ -616,7 +616,7 @@ static s32 nps_enet_probe(struct platform_device *pdev)
 
        /* set kernel MAC address to dev */
        mac_addr = of_get_mac_address(dev->of_node);
-       if (mac_addr)
+       if (!IS_ERR(mac_addr))
                ether_addr_copy(ndev->dev_addr, mac_addr);
        else
                eth_hw_addr_random(ndev);

diff --git a/drivers/net/ethernet/freescale/fec_main.c b/drivers/net/ethernet/freescale/fec_main.c
index a96ad20ee4843e9cdd02c55a3e7286a51679efea..aa7d4e27c5d121497c8511dff2c300a81d351990 100644 (file)
--- a/drivers/net/ethernet/freescale/fec_main.c
+++ b/drivers/net/ethernet/freescale/fec_main.c
@@ -1655,7 +1655,7 @@ static void fec_get_mac(struct net_device *ndev)
                struct device_node *np = fep->pdev->dev.of_node;
                if (np) {
                        const char *mac = of_get_mac_address(np);
-                       if (mac)
+                       if (!IS_ERR(mac))
                                iap = (unsigned char *) mac;
                }
        }

diff --git a/drivers/net/ethernet/freescale/fec_mpc52xx.c b/drivers/net/ethernet/freescale/fec_mpc52xx.c
index c1968b3ecec87cf8fcd807c4f34d0be4cc6e27e8..7b7e526869a7f8c04b262e3408581df99fc709fe 100644 (file)
--- a/drivers/net/ethernet/freescale/fec_mpc52xx.c
+++ b/drivers/net/ethernet/freescale/fec_mpc52xx.c
@@ -902,7 +902,7 @@ static int mpc52xx_fec_probe(struct platform_device *op)
         * First try to read MAC address from DT
         */
        mac_addr = of_get_mac_address(np);
-       if (mac_addr) {
+       if (!IS_ERR(mac_addr)) {
                memcpy(ndev->dev_addr, mac_addr, ETH_ALEN);
        } else {
                struct mpc52xx_fec __iomem *fec = priv->fec;

diff --git a/drivers/net/ethernet/freescale/fman/mac.c b/drivers/net/ethernet/freescale/fman/mac.c
index 3c21486c6c8479186ba927f7f7f19a8c9f015314..9cd2c28d17df93390d6b57481d14eb56da6cc3db 100644 (file)
--- a/drivers/net/ethernet/freescale/fman/mac.c
+++ b/drivers/net/ethernet/freescale/fman/mac.c
@@ -724,7 +724,7 @@ static int mac_probe(struct platform_device *_of_dev)
 
        /* Get the MAC address */
        mac_addr = of_get_mac_address(mac_node);
-       if (!mac_addr) {
+       if (IS_ERR(mac_addr)) {
                dev_err(dev, "of_get_mac_address(%pOF) failed\n", mac_node);
                err = -EINVAL;
                goto _return_of_get_parent;

diff --git a/drivers/net/ethernet/freescale/fs_enet/fs_enet-main.c b/drivers/net/ethernet/freescale/fs_enet/fs_enet-main.c
index 7c548ed535da5babe24c342243c82d737f257034..90ea7a115d0fb9a140d012551959bd943084873c 100644 (file)
--- a/drivers/net/ethernet/freescale/fs_enet/fs_enet-main.c
+++ b/drivers/net/ethernet/freescale/fs_enet/fs_enet-main.c
@@ -1014,7 +1014,7 @@ static int fs_enet_probe(struct platform_device *ofdev)
        spin_lock_init(&fep->tx_lock);
 
        mac_addr = of_get_mac_address(ofdev->dev.of_node);
-       if (mac_addr)
+       if (!IS_ERR(mac_addr))
                memcpy(ndev->dev_addr, mac_addr, ETH_ALEN);
 
        ret = fep->ops->allocate_bd(ndev);

diff --git a/drivers/net/ethernet/freescale/gianfar.c b/drivers/net/ethernet/freescale/gianfar.c
index 45fcc96be90e06abb348843653313779918573f1..df13c693b03850b23c357b98997c3f29fff289cb 100644 (file)
--- a/drivers/net/ethernet/freescale/gianfar.c
+++ b/drivers/net/ethernet/freescale/gianfar.c
@@ -872,7 +872,7 @@ static int gfar_of_init(struct platform_device *ofdev, struct net_device **pdev)
 
        mac_addr = of_get_mac_address(np);
 
-       if (mac_addr)
+       if (!IS_ERR(mac_addr))
                memcpy(dev->dev_addr, mac_addr, ETH_ALEN);
 
        if (model && !strcasecmp(model, "TSEC"))

diff --git a/drivers/net/ethernet/freescale/ucc_geth.c b/drivers/net/ethernet/freescale/ucc_geth.c
index eb3e65e8868f9f65047ab24e8094f56e307bb64b..216e99af2b5a1dc26d336240a76985a7d56c4c8c 100644 (file)
--- a/drivers/net/ethernet/freescale/ucc_geth.c
+++ b/drivers/net/ethernet/freescale/ucc_geth.c
@@ -3910,7 +3910,7 @@ static int ucc_geth_probe(struct platform_device* ofdev)
        }
 
        mac_addr = of_get_mac_address(np);
-       if (mac_addr)
+       if (!IS_ERR(mac_addr))
                memcpy(dev->dev_addr, mac_addr, ETH_ALEN);
 
        ugeth->ug_info = ug_info;

diff --git a/drivers/net/ethernet/hisilicon/hisi_femac.c b/drivers/net/ethernet/hisilicon/hisi_femac.c
index 2c2808830e95723604b7356514ea64ca123fac3a..96c32ae320b0f128aef1084be052027a135bb5da 100644 (file)
--- a/drivers/net/ethernet/hisilicon/hisi_femac.c
+++ b/drivers/net/ethernet/hisilicon/hisi_femac.c
@@ -870,7 +870,7 @@ static int hisi_femac_drv_probe(struct platform_device *pdev)
                           phy_modes(phy->interface));
 
        mac_addr = of_get_mac_address(node);
-       if (mac_addr)
+       if (!IS_ERR(mac_addr))
                ether_addr_copy(ndev->dev_addr, mac_addr);
        if (!is_valid_ether_addr(ndev->dev_addr)) {
                eth_hw_addr_random(ndev);

diff --git a/drivers/net/ethernet/hisilicon/hix5hd2_gmac.c b/drivers/net/ethernet/hisilicon/hix5hd2_gmac.c
index e5d853b7b454f4651e2dd2575c191559c4527cd1..b1cb58f0aaf64a45785fe04d2edc2733b289336f 100644 (file)
--- a/drivers/net/ethernet/hisilicon/hix5hd2_gmac.c
+++ b/drivers/net/ethernet/hisilicon/hix5hd2_gmac.c
@@ -1229,7 +1229,7 @@ static int hix5hd2_dev_probe(struct platform_device *pdev)
        }
 
        mac_addr = of_get_mac_address(node);
-       if (mac_addr)
+       if (!IS_ERR(mac_addr))
                ether_addr_copy(ndev->dev_addr, mac_addr);
        if (!is_valid_ether_addr(ndev->dev_addr)) {
                eth_hw_addr_random(ndev);

diff --git a/drivers/net/ethernet/lantiq_xrx200.c b/drivers/net/ethernet/lantiq_xrx200.c
index d29104de0d53c06ae431b92bd6aa75cb579bd5ee..cda641ef89afaca7c236bed2f263ecb218b1be7c 100644 (file)
--- a/drivers/net/ethernet/lantiq_xrx200.c
+++ b/drivers/net/ethernet/lantiq_xrx200.c
@@ -478,7 +478,7 @@ static int xrx200_probe(struct platform_device *pdev)
        }
 
        mac = of_get_mac_address(np);
-       if (mac && is_valid_ether_addr(mac))
+       if (!IS_ERR(mac))
                ether_addr_copy(net_dev->dev_addr, mac);
        else
                eth_hw_addr_random(net_dev);

diff --git a/drivers/net/ethernet/marvell/mv643xx_eth.c b/drivers/net/ethernet/marvell/mv643xx_eth.c
index 292a668ce88e6f75580910eb1997e3b791a27d72..07e254fc96effec9428f5247ebcaf6b222476d3f 100644 (file)
--- a/drivers/net/ethernet/marvell/mv643xx_eth.c
+++ b/drivers/net/ethernet/marvell/mv643xx_eth.c
@@ -2749,7 +2749,7 @@ static int mv643xx_eth_shared_of_add_port(struct platform_device *pdev,
        }
 
        mac_addr = of_get_mac_address(pnp);
-       if (mac_addr)
+       if (!IS_ERR(mac_addr))
                memcpy(ppd.mac_addr, mac_addr, ETH_ALEN);
 
        mv643xx_eth_property(pnp, "tx-queue-size", ppd.tx_queue_size);

diff --git a/drivers/net/ethernet/marvell/mvneta.c b/drivers/net/ethernet/marvell/mvneta.c
index a715277ecf81c9494521a816a470f4d57e40f80e..8186135883ed965f62ec76586649c3eafb63786f 100644 (file)
--- a/drivers/net/ethernet/marvell/mvneta.c
+++ b/drivers/net/ethernet/marvell/mvneta.c
@@ -4563,7 +4563,7 @@ static int mvneta_probe(struct platform_device *pdev)
        }
 
        dt_mac_addr = of_get_mac_address(dn);
-       if (dt_mac_addr) {
+       if (!IS_ERR(dt_mac_addr)) {
                mac_from = "device tree";
                memcpy(dev->dev_addr, dt_mac_addr, ETH_ALEN);
        } else {

diff --git a/drivers/net/ethernet/marvell/pxa168_eth.c b/drivers/net/ethernet/marvell/pxa168_eth.c
index 35f2142aac5e328b5068850e22a7c77bed63b60b..ce037e8530fa651d719cd9c4c678ccacfd909370 100644 (file)
--- a/drivers/net/ethernet/marvell/pxa168_eth.c
+++ b/drivers/net/ethernet/marvell/pxa168_eth.c
@@ -1461,7 +1461,7 @@ static int pxa168_eth_probe(struct platform_device *pdev)
        if (pdev->dev.of_node)
                mac_addr = of_get_mac_address(pdev->dev.of_node);
 
-       if (mac_addr && is_valid_ether_addr(mac_addr)) {
+       if (!IS_ERR_OR_NULL(mac_addr)) {
                ether_addr_copy(dev->dev_addr, mac_addr);
        } else {
                /* try reading the mac address, if set by the bootloader */

diff --git a/drivers/net/ethernet/marvell/sky2.c b/drivers/net/ethernet/marvell/sky2.c
index 8b3495ee2b6eb7aeeabdb4449e21e05f61562435..c4050ec594f4ee1ad9de7642d07ed715392c4809 100644 (file)
--- a/drivers/net/ethernet/marvell/sky2.c
+++ b/drivers/net/ethernet/marvell/sky2.c
@@ -4808,7 +4808,7 @@ static struct net_device *sky2_init_netdev(struct sky2_hw *hw, unsigned port,
         * 2) from internal registers set by bootloader
         */
        iap = of_get_mac_address(hw->pdev->dev.of_node);
-       if (iap)
+       if (!IS_ERR(iap))
                memcpy(dev->dev_addr, iap, ETH_ALEN);
        else
                memcpy_fromio(dev->dev_addr, hw->regs + B2_MAC_1 + port * 8,

diff --git a/drivers/net/ethernet/mediatek/mtk_eth_soc.c b/drivers/net/ethernet/mediatek/mtk_eth_soc.c
index 53abe925ecb11cb14de40c4f3f23ab67b49b396d..f9fbb3ffa3a61e696a8078a98f41be1496a9ac45 100644 (file)
--- a/drivers/net/ethernet/mediatek/mtk_eth_soc.c
+++ b/drivers/net/ethernet/mediatek/mtk_eth_soc.c
@@ -2028,7 +2028,7 @@ static int __init mtk_init(struct net_device *dev)
        const char *mac_addr;
 
        mac_addr = of_get_mac_address(mac->of_node);
-       if (mac_addr)
+       if (!IS_ERR(mac_addr))
                ether_addr_copy(dev->dev_addr, mac_addr);
 
        /* If the mac address is invalid, use random mac address  */

diff --git a/drivers/net/ethernet/micrel/ks8851.c b/drivers/net/ethernet/micrel/ks8851.c
index 7849119d407aef1a7b92d6b0e047a8f74b4867f7..b44172a901edbeb5e899c8c9642cd1c84fbca1e3 100644 (file)
--- a/drivers/net/ethernet/micrel/ks8851.c
+++ b/drivers/net/ethernet/micrel/ks8851.c
@@ -425,7 +425,7 @@ static void ks8851_init_mac(struct ks8851_net *ks)
        const u8 *mac_addr;
 
        mac_addr = of_get_mac_address(ks->spidev->dev.of_node);
-       if (mac_addr) {
+       if (!IS_ERR(mac_addr)) {
                memcpy(dev->dev_addr, mac_addr, ETH_ALEN);
                ks8851_write_mac_addr(dev);
                return;

diff --git a/drivers/net/ethernet/micrel/ks8851_mll.c b/drivers/net/ethernet/micrel/ks8851_mll.c
index c946841c0a066d2e7eabd059092ed6cbbb156b01..dc76b0d152348bfb6b95abad8ef2f36e11d64c35 100644 (file)
--- a/drivers/net/ethernet/micrel/ks8851_mll.c
+++ b/drivers/net/ethernet/micrel/ks8851_mll.c
@@ -1327,7 +1327,7 @@ static int ks8851_probe(struct platform_device *pdev)
        /* overwriting the default MAC address */
        if (pdev->dev.of_node) {
                mac = of_get_mac_address(pdev->dev.of_node);
-               if (mac)
+               if (!IS_ERR(mac))
                        memcpy(ks->mac_addr, mac, ETH_ALEN);
        } else {
                struct ks8851_mll_platform_data *pdata;

diff --git a/drivers/net/ethernet/nxp/lpc_eth.c b/drivers/net/ethernet/nxp/lpc_eth.c
index 89d17399fb5accd910564e13c788b309a522bcb3..da138edddd3256905741ff146db7967429ec3ab1 100644 (file)
--- a/drivers/net/ethernet/nxp/lpc_eth.c
+++ b/drivers/net/ethernet/nxp/lpc_eth.c
@@ -1368,7 +1368,7 @@ static int lpc_eth_drv_probe(struct platform_device *pdev)
 
        if (!is_valid_ether_addr(ndev->dev_addr)) {
                const char *macaddr = of_get_mac_address(np);
-               if (macaddr)
+               if (!IS_ERR(macaddr))
                        memcpy(ndev->dev_addr, macaddr, ETH_ALEN);
        }
        if (!is_valid_ether_addr(ndev->dev_addr))

diff --git a/drivers/net/ethernet/qualcomm/qca_spi.c b/drivers/net/ethernet/qualcomm/qca_spi.c
index 97f92953bdb9c071deaf705a607589a12d064a53..b28360bc2255e85bc5bdfb45b1313268f1f5e833 100644 (file)
--- a/drivers/net/ethernet/qualcomm/qca_spi.c
+++ b/drivers/net/ethernet/qualcomm/qca_spi.c
@@ -966,7 +966,7 @@ qca_spi_probe(struct spi_device *spi)
 
        mac = of_get_mac_address(spi->dev.of_node);
 
-       if (mac)
+       if (!IS_ERR(mac))
                ether_addr_copy(qca->net_dev->dev_addr, mac);
 
        if (!is_valid_ether_addr(qca->net_dev->dev_addr)) {

diff --git a/drivers/net/ethernet/qualcomm/qca_uart.c b/drivers/net/ethernet/qualcomm/qca_uart.c
index db6068cd7a1f21473e9c440850c3f80894439e29..590616846cd168735ee701e1724f333441a1d2ad 100644 (file)
--- a/drivers/net/ethernet/qualcomm/qca_uart.c
+++ b/drivers/net/ethernet/qualcomm/qca_uart.c
@@ -351,7 +351,7 @@ static int qca_uart_probe(struct serdev_device *serdev)
 
        mac = of_get_mac_address(serdev->dev.of_node);
 
-       if (mac)
+       if (!IS_ERR(mac))
                ether_addr_copy(qca->net_dev->dev_addr, mac);
 
        if (!is_valid_ether_addr(qca->net_dev->dev_addr)) {

diff --git a/drivers/net/ethernet/renesas/ravb_main.c b/drivers/net/ethernet/renesas/ravb_main.c
index 9618c4881c83a47475da7016932a60ef5fc46a26..d3ffcf5b445a0d4a43071f4b05b7e6355e867885 100644 (file)
--- a/drivers/net/ethernet/renesas/ravb_main.c
+++ b/drivers/net/ethernet/renesas/ravb_main.c
@@ -111,7 +111,7 @@ static void ravb_set_buffer_align(struct sk_buff *skb)
  */
 static void ravb_read_mac_address(struct net_device *ndev, const u8 *mac)
 {
-       if (mac) {
+       if (!IS_ERR(mac)) {
                ether_addr_copy(ndev->dev_addr, mac);
        } else {
                u32 mahr = ravb_read(ndev, MAHR);

diff --git a/drivers/net/ethernet/renesas/sh_eth.c b/drivers/net/ethernet/renesas/sh_eth.c
index e33af371b169ea54ae6449deb72902e51b02ca82..4d4be66125836b021ff9ad2935f288f5a2af4d67 100644 (file)
--- a/drivers/net/ethernet/renesas/sh_eth.c
+++ b/drivers/net/ethernet/renesas/sh_eth.c
@@ -3193,7 +3193,7 @@ static struct sh_eth_plat_data *sh_eth_parse_dt(struct device *dev)
        pdata->phy_interface = ret;
 
        mac_addr = of_get_mac_address(np);
-       if (mac_addr)
+       if (!IS_ERR(mac_addr))
                memcpy(pdata->mac_addr, mac_addr, ETH_ALEN);
 
        pdata->no_ether_link =

diff --git a/drivers/net/ethernet/samsung/sxgbe/sxgbe_platform.c b/drivers/net/ethernet/samsung/sxgbe/sxgbe_platform.c
index fbd00cb0cb7d7204e97b9b11d8ae058c95148834..d2bc9412ba03b1d77c0320441bb0f7ac4440786a 100644 (file)
--- a/drivers/net/ethernet/samsung/sxgbe/sxgbe_platform.c
+++ b/drivers/net/ethernet/samsung/sxgbe/sxgbe_platform.c
@@ -124,7 +124,7 @@ static int sxgbe_platform_probe(struct platform_device *pdev)
        }
 
        /* Get MAC address if available (DT) */
-       if (mac)
+       if (!IS_ERR_OR_NULL(mac))
                ether_addr_copy(priv->dev->dev_addr, mac);
 
        /* Get the TX/RX IRQ numbers */

diff --git a/drivers/net/ethernet/socionext/sni_ave.c b/drivers/net/ethernet/socionext/sni_ave.c
index bb6d5fb73035b340b989f368d90bad5c806a6d05..51a7b48db4bca4b16a517be4dbec68744939b502 100644 (file)
--- a/drivers/net/ethernet/socionext/sni_ave.c
+++ b/drivers/net/ethernet/socionext/sni_ave.c
@@ -1599,7 +1599,7 @@ static int ave_probe(struct platform_device *pdev)
        ndev->max_mtu = AVE_MAX_ETHFRAME - (ETH_HLEN + ETH_FCS_LEN);
 
        mac_addr = of_get_mac_address(np);
-       if (mac_addr)
+       if (!IS_ERR(mac_addr))
                ether_addr_copy(ndev->dev_addr, mac_addr);
 
        /* if the mac address is invalid, use random mac address */

diff --git a/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c b/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c
index 5ab2733e15e2101e58d887813aba8671fa0b9e7c..5678b869cbffdfe5d10120d96f1e618d98f8d208 100644 (file)
--- a/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c
+++ b/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c
@@ -4262,7 +4262,7 @@ int stmmac_dvr_probe(struct device *device,
        priv->wol_irq = res->wol_irq;
        priv->lpi_irq = res->lpi_irq;
 
-       if (res->mac)
+       if (!IS_ERR_OR_NULL(res->mac))
                memcpy(priv->dev->dev_addr, res->mac, ETH_ALEN);
 
        dev_set_drvdata(device, priv->dev);

diff --git a/drivers/net/ethernet/ti/cpsw.c b/drivers/net/ethernet/ti/cpsw.c
index e37680654a13044bc32985eb77899f8105f4bdbd..b18eeb05b9930feb23d563767b2ef272891406ea 100644 (file)
--- a/drivers/net/ethernet/ti/cpsw.c
+++ b/drivers/net/ethernet/ti/cpsw.c
@@ -2232,7 +2232,7 @@ static int cpsw_probe_dt(struct cpsw_platform_data *data,
 
 no_phy_slave:
                mac_addr = of_get_mac_address(slave_node);
-               if (mac_addr) {
+               if (!IS_ERR(mac_addr)) {
                        memcpy(slave_data->mac_addr, mac_addr, ETH_ALEN);
                } else {
                        ret = ti_cm_get_macid(&pdev->dev, i,

diff --git a/drivers/net/ethernet/ti/netcp_core.c b/drivers/net/ethernet/ti/netcp_core.c
index 01d4ca331f8c37e1c4a1c9753ae9689dfc88ff26..642843945031c222a5f5225832c83df11aa87145 100644 (file)
--- a/drivers/net/ethernet/ti/netcp_core.c
+++ b/drivers/net/ethernet/ti/netcp_core.c
@@ -2037,7 +2037,7 @@ static int netcp_create_interface(struct netcp_device *netcp_device,
                devm_release_mem_region(dev, res.start, size);
        } else {
                mac_addr = of_get_mac_address(node_interface);
-               if (mac_addr)
+               if (!IS_ERR(mac_addr))
                        ether_addr_copy(ndev->dev_addr, mac_addr);
                else
                        eth_random_addr(ndev->dev_addr);

diff --git a/drivers/net/ethernet/wiznet/w5100.c b/drivers/net/ethernet/wiznet/w5100.c
index d8ba512f166ad38f10f6a01c1ef10c564bdd808e..b0052933993b58d3c9f62531c9ad00ecf6d8c8a6 100644 (file)
--- a/drivers/net/ethernet/wiznet/w5100.c
+++ b/drivers/net/ethernet/wiznet/w5100.c
@@ -1164,7 +1164,7 @@ int w5100_probe(struct device *dev, const struct w5100_ops *ops,
        INIT_WORK(&priv->setrx_work, w5100_setrx_work);
        INIT_WORK(&priv->restart_work, w5100_restart_work);
 
-       if (mac_addr)
+       if (!IS_ERR_OR_NULL(mac_addr))
                memcpy(ndev->dev_addr, mac_addr, ETH_ALEN);
        else
                eth_hw_addr_random(ndev);

diff --git a/drivers/net/ethernet/xilinx/ll_temac_main.c b/drivers/net/ethernet/xilinx/ll_temac_main.c
index 985199100b7d270a7a7b57d5f637b6a4848ad645..f389a819f058f4a76510d14878624d5cbf6cbc84 100644 (file)
--- a/drivers/net/ethernet/xilinx/ll_temac_main.c
+++ b/drivers/net/ethernet/xilinx/ll_temac_main.c
@@ -1252,7 +1252,7 @@ static int temac_probe(struct platform_device *pdev)
        if (temac_np) {
                /* Retrieve the MAC address */
                addr = of_get_mac_address(temac_np);
-               if (!addr) {
+               if (IS_ERR(addr)) {
                        dev_err(&pdev->dev, "could not find MAC address\n");
                        return -ENODEV;
                }

diff --git a/drivers/net/ethernet/xilinx/xilinx_axienet_main.c b/drivers/net/ethernet/xilinx/xilinx_axienet_main.c
index 4041c75997ba5ed52803b69b4c2e19bf41dee8e8..108fbc7f125a2fabfc2380d745ff1027b50475e2 100644 (file)
--- a/drivers/net/ethernet/xilinx/xilinx_axienet_main.c
+++ b/drivers/net/ethernet/xilinx/xilinx_axienet_main.c
@@ -1596,7 +1596,7 @@ static int axienet_probe(struct platform_device *pdev)
 
        /* Retrieve the MAC address */
        mac_addr = of_get_mac_address(pdev->dev.of_node);
-       if (!mac_addr) {
+       if (IS_ERR(mac_addr)) {
                dev_err(&pdev->dev, "could not find MAC address\n");
                goto free_netdev;
        }

diff --git a/drivers/net/ethernet/xilinx/xilinx_emaclite.c b/drivers/net/ethernet/xilinx/xilinx_emaclite.c
index fc38692da71e0c72da9931a3b189f58179fc66fe..69117075356327aa9904dc26f2ab8064dd9dd3fb 100644 (file)
--- a/drivers/net/ethernet/xilinx/xilinx_emaclite.c
+++ b/drivers/net/ethernet/xilinx/xilinx_emaclite.c
@@ -1165,7 +1165,7 @@ static int xemaclite_of_probe(struct platform_device *ofdev)
        lp->rx_ping_pong = get_bool(ofdev, "xlnx,rx-ping-pong");
        mac_address = of_get_mac_address(ofdev->dev.of_node);
 
-       if (mac_address) {
+       if (!IS_ERR(mac_address)) {
                /* Set the MAC address. */
                memcpy(ndev->dev_addr, mac_address, ETH_ALEN);
        } else {

diff --git a/net/ethernet/eth.c b/net/ethernet/eth.c
index fddcee38c1da63da5a3613058df975bd3b5f615e..4b2b222377ace9235294cbbc7cf6456485723769 100644 (file)
--- a/net/ethernet/eth.c
+++ b/net/ethernet/eth.c
@@ -560,7 +560,7 @@ int eth_platform_get_mac_address(struct device *dev, u8 *mac_addr)
        addr = NULL;
        if (dp)
                addr = of_get_mac_address(dp);
-       if (!addr)
+       if (IS_ERR_OR_NULL(addr))
                addr = arch_get_platform_mac_address();
 
        if (!addr)