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@lip6.fr Signed-off-by: Fengguang Wu fengguang.wu@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);