Commit f4e1f9bb authored by Wei Yongjun's avatar Wei Yongjun Committed by Bjorn Andersson

remoteproc: qcom: wcnss: Fix return value check in wcnss_probe()

In case of error, the function devm_ioremap_resource() returns ERR_PTR()
and never returns NULL. The NULL test in the return value check should
be replaced with IS_ERR().
Signed-off-by: default avatarWei Yongjun <weiyongjun1@huawei.com>
Signed-off-by: default avatarBjorn Andersson <bjorn.andersson@linaro.org>
parent aed361ad
...@@ -528,8 +528,8 @@ static int wcnss_probe(struct platform_device *pdev) ...@@ -528,8 +528,8 @@ static int wcnss_probe(struct platform_device *pdev)
res = platform_get_resource_byname(pdev, IORESOURCE_MEM, "pmu"); res = platform_get_resource_byname(pdev, IORESOURCE_MEM, "pmu");
mmio = devm_ioremap_resource(&pdev->dev, res); mmio = devm_ioremap_resource(&pdev->dev, res);
if (!mmio) { if (IS_ERR(mmio)) {
ret = -ENOMEM; ret = PTR_ERR(mmio);
goto free_rproc; goto free_rproc;
}; };
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment