| Message ID | 20220727185012.3255200-4-saravanak@google.com (mailing list archive) |
|---|---|
| State | Not Applicable |
| Headers | show |
| Series | Bring back driver_deferred_probe_check_state() for now | expand |
| Context | Check | Description |
|---|---|---|
| netdev/tree_selection | success | Guessing tree name failed - patch did not apply |
On Wed, 27 Jul 2022 at 20:50, Saravana Kannan <saravanak@google.com> wrote: > > This reverts commit 5a46079a96451cfb15e4f5f01f73f7ba24ef851a. > > There are a few more issues to fix that have been reported in the thread > for the original series [1]. We'll need to fix those before this will > work. So, revert it for now. > > [1] - https://lore.kernel.org/lkml/20220601070707.3946847-1-saravanak@google.com/ > > Signed-off-by: Saravana Kannan <saravanak@google.com> Acked-by: Ulf Hansson <ulf.hansson@linaro.org> Kind regards Uffe > --- > drivers/base/power/domain.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/base/power/domain.c b/drivers/base/power/domain.c > index 3e86772d5fac..739e52cd4aba 100644 > --- a/drivers/base/power/domain.c > +++ b/drivers/base/power/domain.c > @@ -2730,7 +2730,7 @@ static int __genpd_dev_pm_attach(struct device *dev, struct device *base_dev, > mutex_unlock(&gpd_list_lock); > dev_dbg(dev, "%s() failed to find PM domain: %ld\n", > __func__, PTR_ERR(pd)); > - return -ENODEV; > + return driver_deferred_probe_check_state(base_dev); > } > > dev_dbg(dev, "adding to PM domain %s\n", pd->name); > -- > 2.37.1.359.gd136c6c3e2-goog >
* Saravana Kannan <saravanak@google.com> [700101 02:00]: > This reverts commit 5a46079a96451cfb15e4f5f01f73f7ba24ef851a. > > There are a few more issues to fix that have been reported in the thread > for the original series [1]. We'll need to fix those before this will > work. So, revert it for now. This fixes booting for several TI 32-bit ARM SoCs such as am335x and dra7. Please add a proper fixes tag for this patch though: Fixes: 5a46079a9645 ("PM: domains: Delete usage of driver_deferred_probe_check_state()") Reviewed-by: Tony Lindgren <tony@atomide.com> Tested-by: Tony Lindgren <tony@atomide.com>
diff --git a/drivers/base/power/domain.c b/drivers/base/power/domain.c index 3e86772d5fac..739e52cd4aba 100644 --- a/drivers/base/power/domain.c +++ b/drivers/base/power/domain.c @@ -2730,7 +2730,7 @@ static int __genpd_dev_pm_attach(struct device *dev, struct device *base_dev, mutex_unlock(&gpd_list_lock); dev_dbg(dev, "%s() failed to find PM domain: %ld\n", __func__, PTR_ERR(pd)); - return -ENODEV; + return driver_deferred_probe_check_state(base_dev); } dev_dbg(dev, "adding to PM domain %s\n", pd->name);
This reverts commit 5a46079a96451cfb15e4f5f01f73f7ba24ef851a. There are a few more issues to fix that have been reported in the thread for the original series [1]. We'll need to fix those before this will work. So, revert it for now. [1] - https://lore.kernel.org/lkml/20220601070707.3946847-1-saravanak@google.com/ Signed-off-by: Saravana Kannan <saravanak@google.com> --- drivers/base/power/domain.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)