Commit ffe57033 authored by Wei Yongjun's avatar Wei Yongjun Committed by Mauro Carvalho Chehab
Browse files

[media] VPU: mediatek: fix return value check in mtk_vpu_probe()



In case of error, the function devm_clk_get() returns ERR_PTR()
and not returns NULL. The NULL test in the return value check
should be replaced with IS_ERR().
Signed-off-by: default avatarWei Yongjun <yongjun_wei@trendmicro.com.cn>
Signed-off-by: default avatarHans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@s-opensource.com>
parent 980e0b36
...@@ -788,9 +788,9 @@ static int mtk_vpu_probe(struct platform_device *pdev) ...@@ -788,9 +788,9 @@ static int mtk_vpu_probe(struct platform_device *pdev)
/* Get VPU clock */ /* Get VPU clock */
vpu->clk = devm_clk_get(dev, "main"); vpu->clk = devm_clk_get(dev, "main");
if (!vpu->clk) { if (IS_ERR(vpu->clk)) {
dev_err(dev, "get vpu clock failed\n"); dev_err(dev, "get vpu clock failed\n");
return -EINVAL; return PTR_ERR(vpu->clk);
} }
platform_set_drvdata(pdev, vpu); platform_set_drvdata(pdev, vpu);
......
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