[PATCH 4/6] arm: omap3: add manual control for mpu_iva voltdm usecounting

Tero Kristo [off-list ref] writes:

On Tue, 2012-02-14 at 11:35 -0800, Kevin Hilman wrote:

quoted

Tero Kristo [off-list ref] writes:

quoted

mpu_iva voltdm usecount is now decreased during idle. This will
allow the voltagedomain to reflect actual usage, and will allow
proper usage of vc callbacks.

I don't follow why this is needed in the idle path.

Why aren't the usecounts from clock/powerdomain disables (enables) causing this
voltdm disable (enable) to happen automatcially?

The main issue is mpu powerdomain which doesn't contain any clocks that
are controlled actively. Without adding a "virtual" usecount
enable/disable hook, the usecount for mpu_iva will always remain at 0
(or just follow the iva domain on omap3.) 

Rather than faking the MPU clock gating by disabling the voltage/power
domains, why not just gate the MPU clock in idle?   That would trigger
all the rest and would also be much clearer what is happening in the
idle path.

We want the usecount to reflect the actual idling of this domain also,
so that we can control SR / auto-ret based on MPU activity.

Yes, makes sense now.  Thanks for the explanation.

quoted

If these are needed, seems like they should be in the
pwrdm_[pre|post]_transition() hooks so they can be generic.

This is actually a good idea, I could move them here. I could also add a
manual triggering for core domain here, as I have a feeling it might be
good to have so that core will also follow the idle cycle more tightly
(basically as a virtual follow for sdrc clock which is idled by HW
during wfi.)

In the comments on the new version, you should also describe in detail
the reasons for the "manual" idling of CORE (due to clocks that are
hardware managed.)

Kevin