| Message ID | 20190425090413.10700-1-ulf.hansson@linaro.org |
|---|---|
| Headers | show |
| Series | PM / Domains: Improve support for CPUs in genpd | expand |
On Thu, Apr 25, 2019 at 11:04 AM Ulf Hansson <ulf.hansson@linaro.org> wrote: > > Recently genpd was extended to cope with devices belonging to CPUs. However, > attaching CPU devices via genpd_dev_pm_attach_by_id|name() doesn't work, > because of the virtual device that genpd allocates in this path. > > In this series, this limitation is addressed, together with a few other related > fixes/cleanups. > > Ulf Hansson (4): > PM / Domains: Use the base device for > driver_deferred_probe_check_state() > PM / Domains: Drop unused in-parameter to some genpd functions > PM / Domains: Search for the CPU device outside the genpd lock > PM / Domains: Allow to attach a CPU via > genpd_dev_pm_attach_by_id|name() > > drivers/base/power/domain.c | 73 ++++++++++++++++++------------------- > include/linux/pm_domain.h | 1 + > 2 files changed, 36 insertions(+), 38 deletions(-) Are there any dependencies between this and the series you've recently posted?
On Thu, 25 Apr 2019 at 11:56, Rafael J. Wysocki <rafael@kernel.org> wrote: > > On Thu, Apr 25, 2019 at 11:04 AM Ulf Hansson <ulf.hansson@linaro.org> wrote: > > > > Recently genpd was extended to cope with devices belonging to CPUs. However, > > attaching CPU devices via genpd_dev_pm_attach_by_id|name() doesn't work, > > because of the virtual device that genpd allocates in this path. > > > > In this series, this limitation is addressed, together with a few other related > > fixes/cleanups. > > > > Ulf Hansson (4): > > PM / Domains: Use the base device for > > driver_deferred_probe_check_state() > > PM / Domains: Drop unused in-parameter to some genpd functions > > PM / Domains: Search for the CPU device outside the genpd lock > > PM / Domains: Allow to attach a CPU via > > genpd_dev_pm_attach_by_id|name() > > > > drivers/base/power/domain.c | 73 ++++++++++++++++++------------------- > > include/linux/pm_domain.h | 1 + > > 2 files changed, 36 insertions(+), 38 deletions(-) > > Are there any dependencies between this and the series you've recently posted? Yep. I should have stated that, sorry. This should be applied on top. Kind regards Uffe
On Thu, Apr 25, 2019 at 12:11 PM Ulf Hansson <ulf.hansson@linaro.org> wrote: > > On Thu, 25 Apr 2019 at 11:56, Rafael J. Wysocki <rafael@kernel.org> wrote: > > > > On Thu, Apr 25, 2019 at 11:04 AM Ulf Hansson <ulf.hansson@linaro.org> wrote: > > > > > > Recently genpd was extended to cope with devices belonging to CPUs. However, > > > attaching CPU devices via genpd_dev_pm_attach_by_id|name() doesn't work, > > > because of the virtual device that genpd allocates in this path. > > > > > > In this series, this limitation is addressed, together with a few other related > > > fixes/cleanups. > > > > > > Ulf Hansson (4): > > > PM / Domains: Use the base device for > > > driver_deferred_probe_check_state() > > > PM / Domains: Drop unused in-parameter to some genpd functions > > > PM / Domains: Search for the CPU device outside the genpd lock > > > PM / Domains: Allow to attach a CPU via > > > genpd_dev_pm_attach_by_id|name() > > > > > > drivers/base/power/domain.c | 73 ++++++++++++++++++------------------- > > > include/linux/pm_domain.h | 1 + > > > 2 files changed, 36 insertions(+), 38 deletions(-) > > > > Are there any dependencies between this and the series you've recently posted? > > Yep. I should have stated that, sorry. This should be applied on top. On top of which series?
On Thu, 25 Apr 2019 at 12:14, Rafael J. Wysocki <rafael@kernel.org> wrote: > > On Thu, Apr 25, 2019 at 12:11 PM Ulf Hansson <ulf.hansson@linaro.org> wrote: > > > > On Thu, 25 Apr 2019 at 11:56, Rafael J. Wysocki <rafael@kernel.org> wrote: > > > > > > On Thu, Apr 25, 2019 at 11:04 AM Ulf Hansson <ulf.hansson@linaro.org> wrote: > > > > > > > > Recently genpd was extended to cope with devices belonging to CPUs. However, > > > > attaching CPU devices via genpd_dev_pm_attach_by_id|name() doesn't work, > > > > because of the virtual device that genpd allocates in this path. > > > > > > > > In this series, this limitation is addressed, together with a few other related > > > > fixes/cleanups. > > > > > > > > Ulf Hansson (4): > > > > PM / Domains: Use the base device for > > > > driver_deferred_probe_check_state() > > > > PM / Domains: Drop unused in-parameter to some genpd functions > > > > PM / Domains: Search for the CPU device outside the genpd lock > > > > PM / Domains: Allow to attach a CPU via > > > > genpd_dev_pm_attach_by_id|name() > > > > > > > > drivers/base/power/domain.c | 73 ++++++++++++++++++------------------- > > > > include/linux/pm_domain.h | 1 + > > > > 2 files changed, 36 insertions(+), 38 deletions(-) > > > > > > Are there any dependencies between this and the series you've recently posted? > > > > Yep. I should have stated that, sorry. This should be applied on top. > > On top of which series? [PATCH 0/3] PM / Domains: Improve support for multi PM domains and [PATCH v13 0/4] PM / Domains: Support hierarchical CPU arrangement (PSCI/ARM) Kind regards Uffe
On Thu, Apr 25, 2019 at 12:17 PM Ulf Hansson <ulf.hansson@linaro.org> wrote: > > On Thu, 25 Apr 2019 at 12:14, Rafael J. Wysocki <rafael@kernel.org> wrote: > > > > On Thu, Apr 25, 2019 at 12:11 PM Ulf Hansson <ulf.hansson@linaro.org> wrote: > > > > > > On Thu, 25 Apr 2019 at 11:56, Rafael J. Wysocki <rafael@kernel.org> wrote: > > > > > > > > On Thu, Apr 25, 2019 at 11:04 AM Ulf Hansson <ulf.hansson@linaro.org> wrote: > > > > > > > > > > Recently genpd was extended to cope with devices belonging to CPUs. However, > > > > > attaching CPU devices via genpd_dev_pm_attach_by_id|name() doesn't work, > > > > > because of the virtual device that genpd allocates in this path. > > > > > > > > > > In this series, this limitation is addressed, together with a few other related > > > > > fixes/cleanups. > > > > > > > > > > Ulf Hansson (4): > > > > > PM / Domains: Use the base device for > > > > > driver_deferred_probe_check_state() > > > > > PM / Domains: Drop unused in-parameter to some genpd functions > > > > > PM / Domains: Search for the CPU device outside the genpd lock > > > > > PM / Domains: Allow to attach a CPU via > > > > > genpd_dev_pm_attach_by_id|name() > > > > > > > > > > drivers/base/power/domain.c | 73 ++++++++++++++++++------------------- > > > > > include/linux/pm_domain.h | 1 + > > > > > 2 files changed, 36 insertions(+), 38 deletions(-) > > > > > > > > Are there any dependencies between this and the series you've recently posted? > > > > > > Yep. I should have stated that, sorry. This should be applied on top. > > > > On top of which series? > > [PATCH 0/3] PM / Domains: Improve support for multi PM domains > > and > > [PATCH v13 0/4] PM / Domains: Support hierarchical CPU arrangement (PSCI/ARM) OK, thanks!
On Thursday, April 25, 2019 12:32:24 PM CEST Rafael J. Wysocki wrote: > On Thu, Apr 25, 2019 at 12:17 PM Ulf Hansson <ulf.hansson@linaro.org> wrote: > > > > On Thu, 25 Apr 2019 at 12:14, Rafael J. Wysocki <rafael@kernel.org> wrote: > > > > > > On Thu, Apr 25, 2019 at 12:11 PM Ulf Hansson <ulf.hansson@linaro.org> wrote: > > > > > > > > On Thu, 25 Apr 2019 at 11:56, Rafael J. Wysocki <rafael@kernel.org> wrote: > > > > > > > > > > On Thu, Apr 25, 2019 at 11:04 AM Ulf Hansson <ulf.hansson@linaro.org> wrote: > > > > > > > > > > > > Recently genpd was extended to cope with devices belonging to CPUs. However, > > > > > > attaching CPU devices via genpd_dev_pm_attach_by_id|name() doesn't work, > > > > > > because of the virtual device that genpd allocates in this path. > > > > > > > > > > > > In this series, this limitation is addressed, together with a few other related > > > > > > fixes/cleanups. > > > > > > > > > > > > Ulf Hansson (4): > > > > > > PM / Domains: Use the base device for > > > > > > driver_deferred_probe_check_state() > > > > > > PM / Domains: Drop unused in-parameter to some genpd functions > > > > > > PM / Domains: Search for the CPU device outside the genpd lock > > > > > > PM / Domains: Allow to attach a CPU via > > > > > > genpd_dev_pm_attach_by_id|name() > > > > > > > > > > > > drivers/base/power/domain.c | 73 ++++++++++++++++++------------------- > > > > > > include/linux/pm_domain.h | 1 + > > > > > > 2 files changed, 36 insertions(+), 38 deletions(-) > > > > > > > > > > Are there any dependencies between this and the series you've recently posted? > > > > > > > > Yep. I should have stated that, sorry. This should be applied on top. > > > > > > On top of which series? > > > > [PATCH 0/3] PM / Domains: Improve support for multi PM domains > > > > and > > > > [PATCH v13 0/4] PM / Domains: Support hierarchical CPU arrangement (PSCI/ARM) > > OK, thanks! > This series has been applied, thanks!
Recently genpd was extended to cope with devices belonging to CPUs. However, attaching CPU devices via genpd_dev_pm_attach_by_id|name() doesn't work, because of the virtual device that genpd allocates in this path. In this series, this limitation is addressed, together with a few other related fixes/cleanups. Ulf Hansson (4): PM / Domains: Use the base device for driver_deferred_probe_check_state() PM / Domains: Drop unused in-parameter to some genpd functions PM / Domains: Search for the CPU device outside the genpd lock PM / Domains: Allow to attach a CPU via genpd_dev_pm_attach_by_id|name() drivers/base/power/domain.c | 73 ++++++++++++++++++------------------- include/linux/pm_domain.h | 1 + 2 files changed, 36 insertions(+), 38 deletions(-) -- 2.17.1