On Thu, 3 Jun 2021 at 13:15, Mark Brown [off-list ref] wrote:

On Thu, Jun 03, 2021 at 12:20:57PM +0200, Ulf Hansson wrote:

quoted

On Thu, 3 Jun 2021 at 11:34, Ulf Hansson [off-list ref] wrote:

quoted

quoted

Recent changes in genpd drops and restore performance state votes for
devices during runtime PM.

quoted

After a second thought, it looks like we maybe should defer to apply
this final patch of the series. At least until we figured out how to
address the below issue:

quoted

So, I noticed that we have things like "regulator-fixed-domain", that
uses "required-opps" to enable/disable a regulator through the
dev_pm_set_performance_state() interface. We likely don't want to drop
the performance state internally in genpd when genpd_suspend_noirq()
gets called, for the corresponding struct device for the regulator.

quoted

I guess if genpd should drop performance states like $subject patch
suggest, we need some kind of additional coordination, that allows a
subsystem/driver to inform genpd when it should avoid it. Or something
along those lines.

I'm not sure what you're looking for from me here - was there a concrete
question or somehing?

Nope, not really, sorry if that was not clear.

I just wanted to loop you in, as to make sure that we don't change
something at the PM domain level, which may not fit well with the
regulator implementation.

Kind regards
Uffe