ASoC: codecs: ak4118: Use dev_err_probe() helper
authorKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Tue, 14 Dec 2021 02:08:22 +0000 (11:08 +0900)
committerMark Brown <broonie@kernel.org>
Mon, 20 Dec 2021 12:47:01 +0000 (12:47 +0000)
Use the dev_err_probe() helper, instead of open-coding the same
operation.

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Link: https://lore.kernel.org/r/20211214020843.2225831-2-kuninori.morimoto.gx@renesas.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/codecs/ak4118.c

index e0a6451..2e6bafd 100644 (file)
@@ -374,20 +374,14 @@ static int ak4118_i2c_probe(struct i2c_client *i2c,
        i2c_set_clientdata(i2c, ak4118);
 
        ak4118->reset = devm_gpiod_get(&i2c->dev, "reset", GPIOD_OUT_HIGH);
-       if (IS_ERR(ak4118->reset)) {
-               ret = PTR_ERR(ak4118->reset);
-               if (ret != -EPROBE_DEFER)
-                       dev_err(&i2c->dev, "Failed to get reset: %d\n", ret);
-               return ret;
-       }
+       if (IS_ERR(ak4118->reset))
+               return dev_err_probe(&i2c->dev, PTR_ERR(ak4118->reset),
+                                    "Failed to get reset\n");
 
        ak4118->irq = devm_gpiod_get(&i2c->dev, "irq", GPIOD_IN);
-       if (IS_ERR(ak4118->irq)) {
-               ret = PTR_ERR(ak4118->irq);
-               if (ret != -EPROBE_DEFER)
-                       dev_err(&i2c->dev, "Failed to get IRQ: %d\n", ret);
-               return ret;
-       }
+       if (IS_ERR(ak4118->irq))
+               return dev_err_probe(&i2c->dev, PTR_ERR(ak4118->irq),
+                                    "Failed to get IRQ\n");
 
        ret = devm_request_threaded_irq(&i2c->dev, gpiod_to_irq(ak4118->irq),
                                        NULL, ak4118_irq_handler,