net: qcom/emac: fix return value check in emac_sgmii_config()
authorWei Yongjun <weiyongjun1@huawei.com>
Sat, 1 Oct 2016 09:12:29 +0000 (09:12 +0000)
committerDavid S. Miller <davem@davemloft.net>
Tue, 4 Oct 2016 01:53:03 +0000 (21:53 -0400)
In case of error, the function ioremap() returns NULL pointer
not ERR_PTR(). The IS_ERR() test in the return value check
should be replaced with NULL test.

Also add check for return value of platform_get_resource().

Fixes: 54e19bc74f33 ("net: qcom/emac: do not use devm on internal
phy pdev")
Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com>
Acked-by: Timur Tabi <timur@codeaurora.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/qualcomm/emac/emac-sgmii.c

index 3d2c05a40d2c7ea6084f5484f461343d5f6e326f..75c1b530e39e8118f1788d0647988b97df260d23 100644 (file)
@@ -740,9 +740,14 @@ int emac_sgmii_config(struct platform_device *pdev, struct emac_adapter *adpt)
 
        /* Base address is the first address */
        res = platform_get_resource(sgmii_pdev, IORESOURCE_MEM, 0);
+       if (!res) {
+               ret = -EINVAL;
+               goto error_put_device;
+       }
+
        phy->base = ioremap(res->start, resource_size(res));
-       if (IS_ERR(phy->base)) {
-               ret = PTR_ERR(phy->base);
+       if (!phy->base) {
+               ret = -ENOMEM;
                goto error_put_device;
        }
 
@@ -750,8 +755,8 @@ int emac_sgmii_config(struct platform_device *pdev, struct emac_adapter *adpt)
        res = platform_get_resource(sgmii_pdev, IORESOURCE_MEM, 1);
        if (res) {
                phy->digital = ioremap(res->start, resource_size(res));
-               if (IS_ERR(phy->digital)) {
-                       ret = PTR_ERR(phy->digital);
+               if (!phy->digital) {
+                       ret = -ENOMEM;
                        goto error_unmap_base;
                }
        }