ASoC: codecs: simple-amplifier: Use dev_err_probe() helper
authorKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Tue, 14 Dec 2021 02:08:28 +0000 (11:08 +0900)
committerMark Brown <broonie@kernel.org>
Mon, 20 Dec 2021 12:47:08 +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-8-kuninori.morimoto.gx@renesas.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/codecs/simple-amplifier.c

index b30fc1f894e1b6d2a9acd92c99e35b1c1df99d6c..d306c585b52b0900eadeaed99f8299ed8803b9c9 100644 (file)
@@ -69,7 +69,6 @@ static int simple_amp_probe(struct platform_device *pdev)
 {
        struct device *dev = &pdev->dev;
        struct simple_amp *priv;
-       int err;
 
        priv = devm_kzalloc(dev, sizeof(*priv), GFP_KERNEL);
        if (priv == NULL)
@@ -78,12 +77,9 @@ static int simple_amp_probe(struct platform_device *pdev)
 
        priv->gpiod_enable = devm_gpiod_get_optional(dev, "enable",
                                                     GPIOD_OUT_LOW);
-       if (IS_ERR(priv->gpiod_enable)) {
-               err = PTR_ERR(priv->gpiod_enable);
-               if (err != -EPROBE_DEFER)
-                       dev_err(dev, "Failed to get 'enable' gpio: %d", err);
-               return err;
-       }
+       if (IS_ERR(priv->gpiod_enable))
+               return dev_err_probe(dev, PTR_ERR(priv->gpiod_enable),
+                                    "Failed to get 'enable' gpio");
 
        return devm_snd_soc_register_component(dev,
                                               &simple_amp_component_driver,