Common pattern of handling deferred probe can be simplified with
dev_err_probe().  Less code and also it prints the error value.

Signed-off-by: Krzysztof Kozlowski <k...@kernel.org>
---
 drivers/power/supply/cpcap-battery.c | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/drivers/power/supply/cpcap-battery.c 
b/drivers/power/supply/cpcap-battery.c
index 90eba364664b..295611b3b15e 100644
--- a/drivers/power/supply/cpcap-battery.c
+++ b/drivers/power/supply/cpcap-battery.c
@@ -747,11 +747,8 @@ static int cpcap_battery_init_iio(struct 
cpcap_battery_ddata *ddata)
        return 0;
 
 out_err:
-       if (error != -EPROBE_DEFER)
-               dev_err(ddata->dev, "could not initialize VBUS or ID IIO: %i\n",
-                       error);
-
-       return error;
+       return dev_err_probe(ddata->dev, error,
+                            "could not initialize VBUS or ID IIO\n");
 }
 
 /* Calibrate coulomb counter */
-- 
2.17.1

Reply via email to