aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorUlf Hansson <ulf.hansson@linaro.org>2018-04-26 04:53:06 -0400
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>2018-05-14 16:58:45 -0400
commit88a9769e609a7f3b762a2fe88555c5602758346b (patch)
tree2cc9aa01dcc7d92b325e58f8bce09bb694112f76
parentd21bc89eb98cac7a7af6476f41cbdf10f082dd6b (diff)
driver core: Respect all error codes from dev_pm_domain_attach()
The limitation of being able to check only for -EPROBE_DEFER from dev_pm_domain_attach() has been removed. Hence let's respect all error codes and bail out accordingly. Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org> Acked-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
-rw-r--r--drivers/base/platform.c17
1 files changed, 8 insertions, 9 deletions
diff --git a/drivers/base/platform.c b/drivers/base/platform.c
index 8075ddc70a17..9460139d9b02 100644
--- a/drivers/base/platform.c
+++ b/drivers/base/platform.c
@@ -572,17 +572,16 @@ static int platform_drv_probe(struct device *_dev)
572 return ret; 572 return ret;
573 573
574 ret = dev_pm_domain_attach(_dev, true); 574 ret = dev_pm_domain_attach(_dev, true);
575 if (ret != -EPROBE_DEFER) { 575 if (ret)
576 if (drv->probe) { 576 goto out;
577 ret = drv->probe(dev); 577
578 if (ret) 578 if (drv->probe) {
579 dev_pm_domain_detach(_dev, true); 579 ret = drv->probe(dev);
580 } else { 580 if (ret)
581 /* don't fail if just dev_pm_domain_attach failed */ 581 dev_pm_domain_detach(_dev, true);
582 ret = 0;
583 }
584 } 582 }
585 583
584out:
586 if (drv->prevent_deferred_probe && ret == -EPROBE_DEFER) { 585 if (drv->prevent_deferred_probe && ret == -EPROBE_DEFER) {
587 dev_warn(_dev, "probe deferral not supported\n"); 586 dev_warn(_dev, "probe deferral not supported\n");
588 ret = -ENXIO; 587 ret = -ENXIO;