extcon: intel-mrfld: Switch to use dev_err_probe()
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Thu, 22 Feb 2024 05:41:22 +0000 (14:41 +0900)
committerChanwoo Choi <cw00.choi@samsung.com>
Wed, 8 May 2024 16:03:38 +0000 (01:03 +0900)
Switch to use dev_err_probe() to simplify the error path and
unify a message template.

Link: https://lore.kernel.org/lkml/20231222161954.2955905-1-andriy.shevchenko@linux.intel.com/
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Acked-by: MyungJoo Ham <myungjoo.ham@samsung.com>
Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com>
drivers/extcon/extcon-intel-mrfld.c

index cd1a5f230077ca0de266b9b75b2557f10f2bdfc0..e96947fb76ee49dbae13fd53ec36eb218fc37e65 100644 (file)
@@ -217,24 +217,18 @@ static int mrfld_extcon_probe(struct platform_device *pdev)
                return -ENOMEM;
 
        ret = devm_extcon_dev_register(dev, data->edev);
-       if (ret < 0) {
-               dev_err(dev, "can't register extcon device: %d\n", ret);
-               return ret;
-       }
+       if (ret < 0)
+               return dev_err_probe(dev, ret, "can't register extcon device\n");
 
        ret = devm_request_threaded_irq(dev, irq, NULL, mrfld_extcon_interrupt,
                                        IRQF_ONESHOT | IRQF_SHARED, pdev->name,
                                        data);
-       if (ret) {
-               dev_err(dev, "can't register IRQ handler: %d\n", ret);
-               return ret;
-       }
+       if (ret)
+               return dev_err_probe(dev, ret, "can't register IRQ handler\n");
 
        ret = regmap_read(regmap, BCOVE_ID, &id);
-       if (ret) {
-               dev_err(dev, "can't read PMIC ID: %d\n", ret);
-               return ret;
-       }
+       if (ret)
+               return dev_err_probe(dev, ret, "can't read PMIC ID\n");
 
        data->id = id;