[PATCHv2 2/2] OMAP3: PM: Consistently report cpuidle C-state's names

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



From: Eduardo Valentin <eduardo.valentin@xxxxxxxxx>

This patch changes the naming convention for OMAP3 cpuidle
C-states. Currently, depending on which Cstates are enabled/disabled,
there is a inconsistency between C-state name and what power domain
states it is supposed to reach when that C-state is selected.

For instance, if all cstates are enabled, "C1" means mpu on and
core on. But if this exact cstate is disabled, then "C1" doesn't mean
same power domains state configuration anymore.

So, this patch tries changing the cstates naming convention in order
to proper and consistently report the cstates names.

Signed-off-by: Eduardo Valentin <eduardo.valentin@xxxxxxxxx>
---
 arch/arm/mach-omap2/cpuidle34xx.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/arm/mach-omap2/cpuidle34xx.c b/arch/arm/mach-omap2/cpuidle34xx.c
index 50fe9ab..7ab9992 100644
--- a/arch/arm/mach-omap2/cpuidle34xx.c
+++ b/arch/arm/mach-omap2/cpuidle34xx.c
@@ -367,7 +367,7 @@ int __init omap3_idle_init(void)
 			omap3_enter_idle_bm : omap3_enter_idle;
 		if (cx->type == OMAP3_STATE_C1)
 			dev->safe_state = state;
-		sprintf(state->name, "C%d", count+1);
+		sprintf(state->name, "C%d", cx->type + 1);
 		sprintf(state->desc, "MPU=%s NEON=%s CORE=%s",
 			pd_states[cx->mpu_state], pd_states[cx->mpu_state],
 			pd_states[cx->core_state]);
-- 
1.6.5.7.g9ecb2

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux