diff options
author | Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> | 2013-03-19 09:41:32 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2013-03-20 12:17:59 -0400 |
commit | fc0c0900408e05758a0df17c1924ca837fafca5e (patch) | |
tree | e6c2b366491a8c4db3f653597a50f70343a49c94 /drivers/net/ethernet/renesas | |
parent | 0582b7d15f8a7ae53dd2128b8eb01567b3fd2277 (diff) |
sh_eth: check TSU registers ioremap() error
One must check the result of ioremap() -- in this case it prevents potential
kernel oops when initializing TSU registers further on...
Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ethernet/renesas')
-rw-r--r-- | drivers/net/ethernet/renesas/sh_eth.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/drivers/net/ethernet/renesas/sh_eth.c b/drivers/net/ethernet/renesas/sh_eth.c index c87862812ead..bf5e3cf97c4d 100644 --- a/drivers/net/ethernet/renesas/sh_eth.c +++ b/drivers/net/ethernet/renesas/sh_eth.c | |||
@@ -2446,6 +2446,11 @@ static int sh_eth_drv_probe(struct platform_device *pdev) | |||
2446 | } | 2446 | } |
2447 | mdp->tsu_addr = ioremap(rtsu->start, | 2447 | mdp->tsu_addr = ioremap(rtsu->start, |
2448 | resource_size(rtsu)); | 2448 | resource_size(rtsu)); |
2449 | if (mdp->tsu_addr == NULL) { | ||
2450 | ret = -ENOMEM; | ||
2451 | dev_err(&pdev->dev, "TSU ioremap failed.\n"); | ||
2452 | goto out_release; | ||
2453 | } | ||
2449 | mdp->port = devno % 2; | 2454 | mdp->port = devno % 2; |
2450 | ndev->features = NETIF_F_HW_VLAN_FILTER; | 2455 | ndev->features = NETIF_F_HW_VLAN_FILTER; |
2451 | } | 2456 | } |