kevin/linux
1
Fork 0
Code

thermal/drivers/mediatek: Clean up redundant dev_err_probe()

Browse Source 
Referring to platform_get_irq()'s definition, the return value has
already been checked if ret < 0, and printed via dev_err_probe().
Calling dev_err_probe() one more time outside platform_get_irq()
is obviously redundant.

Removing dev_err_probe() outside platform_get_irq() to clean up
above problem.

Signed-off-by: Chen Jiahao <chenjiahao16@huawei.com>
Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
Link: https://lore.kernel.org/r/20230802094527.988842-1-chenjiahao16@huawei.com
This commit is contained in:
Chen Jiahao 2023-08-02 17:45:27 +08:00 committed by Daniel Lezcano
parent c51592a95f
commit e9b1de73b7
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
drivers/thermal/mediatek/lvts_thermal.c
View File

@ -1216,7 +1216,7 @@ static int lvts_probe(struct platform_device *pdev)
irq = platform_get_irq(pdev, 0);
if (irq < 0)
return dev_err_probe(dev, irq, "No irq resource\n");
return irq;
ret = lvts_domain_init(dev, lvts_td, lvts_data);
if (ret)