[alsa-devel] [PATCH] ASoC: sun4i-i2s: fix odd_ptr_err.cocci warnings

Julia Lawall julia.lawall at lip6.fr
Tue Oct 4 18:23:24 CEST 2016


sound/soc/sunxi/sun4i-i2s.c:633:5-11: inconsistent IS_ERR and PTR_ERR on line 635.

 PTR_ERR should normally access the value just tested by IS_ERR

Generated by: scripts/coccinelle/tests/odd_ptr_err.cocci

Signed-off-by: Julia Lawall <julia.lawall at lip6.fr>
Signed-off-by: Fengguang Wu <fengguang.wu at intel.com>
---

Please take the patch only if it's a positive warning. Thanks!

 sun4i-i2s.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

--- a/sound/soc/sunxi/sun4i-i2s.c
+++ b/sound/soc/sunxi/sun4i-i2s.c
@@ -632,7 +632,7 @@ static int sun4i_i2s_probe(struct platfo
 	reset_apb = devm_reset_control_get(&pdev->dev, "apb_reset");
 	if (IS_ERR(reset_apb)) {
 		dev_err(&pdev->dev, "Can't get apb reset\n");
-		return PTR_ERR(i2s->mod_clk);
+		return PTR_ERR(reset_apb);
 	}

 	ret = reset_control_deassert(reset_apb);


More information about the Alsa-devel mailing list