Messages in this thread | | | From | "Rafael J. Wysocki" <> | Subject | Re: [PATCH v3 2/2] PM / Runtime: Add pm_runtime_enable_recursive | Date | Wed, 20 May 2015 01:39:25 +0200 |
| |
On Tuesday, May 19, 2015 01:49:15 PM Alan Stern wrote: > On Tue, 19 May 2015, Tomeu Vizoso wrote: > > > This function makes less cumbersome to enable runtime PM in a device and > > its descendants. > > > > Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora.com> > > I don't see the point of this. In the scenario you have in mind, are > the device and all its descendants registered by the same > subsystem/driver? If they are, can't the subsystem/driver code enable > runtime PM for each of them when they are registered, by adding a > single call in the right spot? > > If they don't all belong to the same subsystem/driver, who is going to > call your pm_runtime_enable_recursive routine? No single caller will > have the right to enable runtime PM for all these devices.
Agreed.
-- I speak only for myself. Rafael J. Wysocki, Intel Open Source Technology Center.
| |