Re: [Powertop] [PATCH] fix cpuidle state name parsing

2012-08-01 Thread Rajagopal Venkat
On 1 August 2012 03:34, Chris Ferron wrote: > > 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. > > > > Pl

Re: [Powertop] [PATCH] fix cpuidle state name parsing

2012-07-31 Thread Chris Ferron
> 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. > Im not sure this is doing what yo

Re: [Powertop][PATCH] fix cpuidle state name parsing

2012-07-30 Thread Rajagopal Venkat
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 wrot

Re: [Powertop][PATCH] fix cpuidle state name parsing

2012-07-12 Thread Rajagopal Venkat
Can someone consider this patch for merge? On 23 June 2012 00:08, Rajagopal Venkat wrote: > parse cpuidle C state based on sysfs file entry(stateX) > instead of state name/description > > Signed-off-by: Rajagopal Venkat > --- > src/cpu/abstract_cpu.cpp |2 +- > 1 file changed, 1 insertion(

[Powertop][PATCH] fix cpuidle state name parsing

2012-06-22 Thread Rajagopal Venkat
parse cpuidle C state based on sysfs file entry(stateX) instead of state name/description Signed-off-by: Rajagopal Venkat --- 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