On 17-02-17, 15:58, Kevin Hilman wrote: > Viresh Kumar <viresh.ku...@linaro.org> writes: > > > With runtime PM, the devices get suspended while the system is up and > > running in order to save power. At such times, it is important to > > re-evaluate the required performance state of the domain, in order to > > choose a lower state if possible. > > > > This patch updates the genpd suspend/resume callbacks to do that. > > > > Signed-off-by: Viresh Kumar <viresh.ku...@linaro.org> > > Doesn't this assume that a device in the domain would need to change > performance state while runtime suspended. How would that happen? > > Rather than adding this here, I would think that drivers would instead > remove any QoS requests before going into runtime suspend, which would > trigger an update before runtime suspending.
Okay, lets leave it for the drivers, at least for the time being. I will drop this patch. -- viresh