net: moxa: Use devm_platform_get_and_ioremap_resource()
[ Upstream commit 35cba15a504bf4f585bb9d78f47b22b28a1a06b2 ] Use devm_platform_get_and_ioremap_resource() to simplify code and avoid a null-ptr-deref by checking 'res' in it. Signed-off-by: Yang Yingliang <yangyingliang@huawei.com> Signed-off-by: David S. Miller <davem@davemloft.net> Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:

committed by
Greg Kroah-Hartman

parent
278dc34b71
commit
8e4da40142
@@ -481,14 +481,13 @@ static int moxart_mac_probe(struct platform_device *pdev)
|
|||||||
priv->ndev = ndev;
|
priv->ndev = ndev;
|
||||||
priv->pdev = pdev;
|
priv->pdev = pdev;
|
||||||
|
|
||||||
res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
|
priv->base = devm_platform_get_and_ioremap_resource(pdev, 0, &res);
|
||||||
ndev->base_addr = res->start;
|
|
||||||
priv->base = devm_ioremap_resource(p_dev, res);
|
|
||||||
if (IS_ERR(priv->base)) {
|
if (IS_ERR(priv->base)) {
|
||||||
dev_err(p_dev, "devm_ioremap_resource failed\n");
|
dev_err(p_dev, "devm_ioremap_resource failed\n");
|
||||||
ret = PTR_ERR(priv->base);
|
ret = PTR_ERR(priv->base);
|
||||||
goto init_fail;
|
goto init_fail;
|
||||||
}
|
}
|
||||||
|
ndev->base_addr = res->start;
|
||||||
|
|
||||||
spin_lock_init(&priv->txlock);
|
spin_lock_init(&priv->txlock);
|
||||||
|
|
||||||
|
Reference in New Issue
Block a user