This patch fixes powertop to display cpuidle states on platforms where cpuidle stateX directory name field does not contain "CX" string. On some platforms, the name field contains meaningful strings like WFI, Sleep, DeepSleep.
Please review the patch.
On 23 June 2012 00:08, Rajagopal Venkat rajagopal.venkat@linaro.org wrote:
parse cpuidle C state based on sysfs file entry(stateX) instead of state name/description
Signed-off-by: Rajagopal Venkat rajagopal.venkat@linaro.org
src/cpu/abstract_cpu.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/cpu/abstract_cpu.cpp b/src/cpu/abstract_cpu.cpp index cd4eba0..72969fc 100644 --- a/src/cpu/abstract_cpu.cpp +++ b/src/cpu/abstract_cpu.cpp @@ -147,7 +147,7 @@ void abstract_cpu::insert_cstate(const char *linux_name, const char *human_name, strcpy(state->linux_name, linux_name); strcpy(state->human_name, human_name);
c = human_name;
c = linux_name; while (*c) { if (strcmp(linux_name, "active")==0) { state->line_level = LEVEL_C0;-- 1.7.9.5