[0/4] PM / Domains: Improve support for CPUs in genpd

Message ID 20190425090413.10700-1-ulf.hansson@linaro.org
Headers show
Series
  • PM / Domains: Improve support for CPUs in genpd
Related show

Message

Ulf Hansson April 25, 2019, 9:04 a.m.
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

Comments

Rafael J. Wysocki April 25, 2019, 9:56 a.m. | #1
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?
Ulf Hansson April 25, 2019, 10:11 a.m. | #2
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
Rafael J. Wysocki April 25, 2019, 10:14 a.m. | #3
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?
Ulf Hansson April 25, 2019, 10:17 a.m. | #4
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
Rafael J. Wysocki April 25, 2019, 10:32 a.m. | #5
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!
Rafael J. Wysocki May 1, 2019, 10:42 a.m. | #6
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!