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@xxxxxxx> Signed-off-by: Fengguang Wu <fengguang.wu@xxxxxxxxx> --- 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); _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel