diff options
| author |   Varka Bhadram <varkabhadram@gmail.com> | 2014-10-24 07:42:09 +0530 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2014-10-27 19:16:01 -0400 |
| commit | c960804f536fe810fa7c8c5eb0dffa8c9aa21390 (patch) | |
| tree | 4277117dfa7057cb8291c3f48604c8e386df3a36 /drivers/net/ethernet/renesas/sh_eth.c | |
| parent | ethernet: marvell: remove unnecessary check (diff) | |
| download | linux-dev-c960804f536fe810fa7c8c5eb0dffa8c9aa21390.tar.xz linux-dev-c960804f536fe810fa7c8c5eb0dffa8c9aa21390.zip | |
ethernet: renesas: remove unnecessary check
devm_ioremap_resource checks platform_get_resource() return value.
We can remove the duplicate check here.
Signed-off-by: Varka Bhadram <varkab@cdac.in>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to '')
| -rw-r--r-- | drivers/net/ethernet/renesas/sh_eth.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/drivers/net/ethernet/renesas/sh_eth.c b/drivers/net/ethernet/renesas/sh_eth.c index 60e9c2cd051e..dbe860650bfe 100644 --- a/drivers/net/ethernet/renesas/sh_eth.c +++ b/drivers/net/ethernet/renesas/sh_eth.c @@ -2769,10 +2769,6 @@ static int sh_eth_drv_probe(struct platform_device *pdev) /* get base addr */ res = platform_get_resource(pdev, IORESOURCE_MEM, 0); - if (unlikely(res == NULL)) { - dev_err(&pdev->dev, "invalid resource\n"); - return -EINVAL; - } ndev = alloc_etherdev(sizeof(struct sh_eth_private)); if (!ndev) @@ -2781,8 +2777,6 @@ static int sh_eth_drv_probe(struct platform_device *pdev) pm_runtime_enable(&pdev->dev); pm_runtime_get_sync(&pdev->dev); - /* The sh Ether-specific entries in the device structure. */ - ndev->base_addr = res->start; devno = pdev->id; if (devno < 0) devno = 0; @@ -2806,6 +2800,8 @@ static int sh_eth_drv_probe(struct platform_device *pdev) goto out_release; } + ndev->base_addr = res->start; + spin_lock_init(&mdp->lock); mdp->pdev = pdev; |