diff options
author | Paul Walmsley <paul@pwsan.com> | 2012-01-30 09:47:24 (GMT) |
---|---|---|
committer | Kevin Hilman <khilman@ti.com> | 2012-03-05 23:38:02 (GMT) |
commit | e68e8093ed570f9272665112d13d4c5811536680 (patch) | |
tree | db72b5962b7991e0bfba2f0d182b4c13de9be23a /kernel/params.c | |
parent | 506c7d7931317813b3142f57d44bf113102a2a8f (diff) | |
download | linux-fsl-qoriq-e68e8093ed570f9272665112d13d4c5811536680.tar.xz |
ARM: OMAP2+: PM: clean up omap_set_pwrdm_state()
Clean up a few different parts of omap_set_pwrdm_state():
- Remove a superfluous call to pwrdm_state_switch(). Not needed
unless LOWPOWERSTATECHANGE is used, because the state switch code is
called by either clkdm_sleep() or clkdm_allow_idle().
- Add code to wait for the power state transition in the OMAP4+ low
power state change. This is speculative, so I would particularly
appreciate feedback on this part.
- Remove a superfluous call to pwrdm_read_pwrst().
- Update variable names to be more meaningful (hopefully) and precise.
- Fix an error path bug that would not place the clockdomain back into
hardware-supervised idle or sleep mode if the power state could not
be programmed.
The documentation for this function still needs major improvements;
that's left for a later patch.
Signed-off-by: Paul Walmsley <paul@pwsan.com>
Cc: Rajendra Nayak <rnayak@ti.com>
Tested-by: Tero Kristo <t-kristo@ti.com>
Acked-by: Santosh Shilimkar <santosh.shilimkar@ti.com>
Tested-by: Kevin Hilman <khilman@ti.com>
Signed-off-by: Kevin Hilman <khilman@ti.com>
Diffstat (limited to 'kernel/params.c')
0 files changed, 0 insertions, 0 deletions