media: i2c: ov9282: Switch to use dev_err_probe helper
authorAlexander Stein <alexander.stein@ew.tq-group.com>
Tue, 13 Dec 2022 14:07:27 +0000 (15:07 +0100)
committerMauro Carvalho Chehab <mchehab@kernel.org>
Mon, 6 Feb 2023 07:36:03 +0000 (08:36 +0100)
In the probe path, dev_err() can be replaced with dev_err_probe()
which will check if error code is -EPROBE_DEFER and prints the
error name. It also sets the defer probe reason which can be
checked later through debugfs. It's more simple in error path.

Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
drivers/media/i2c/ov9282.c

index 8216644494bbf6088d0cd46abc91dff21b20930f..7f46cac38aab48b463485878a0b7f544c4bbb946 100644 (file)
@@ -1142,10 +1142,9 @@ static int ov9282_parse_hw_config(struct ov9282 *ov9282)
        }
 
        ret = ov9282_configure_regulators(ov9282);
-       if (ret) {
-               dev_err(ov9282->dev, "Failed to get power regulators\n");
-               return ret;
-       }
+       if (ret)
+               return dev_err_probe(ov9282->dev, ret,
+                                    "Failed to get power regulators\n");
 
        rate = clk_get_rate(ov9282->inclk);
        if (rate != OV9282_INCLK_RATE) {