On Thu, Jul 29, 2010 at 6:38 PM, Amit Kucheria <amit.kuche...@linaro.org> wrote:
> On 10 Jul 29, Amit Arora wrote:
>>
>> Currently the PowerTOP tool is x86 centric and does not work on other 
>> architectures, unless some changes are made.
>>
>> This patchset removes the hardcoded values for the P and C states. It does 
>> that in two steps
>> 1) Implements the OMAP3 support in PowerTOP.
>> 2) Makes the code generic enough for all boards
>>
>> I have tested this only on my desktop. I am yet to receive an ARM based 
>> board, and hence couldn't test it there. So, any kind of testing and/or 
>> feedback are most welcome! Thanks!
>>
>>
>> Amit Arora (2):
>>   Add OMAP3 support to PowerTOP
>>   Remove hardcoded c and p states
>>
>>  Makefile          |    2 +-
>>  cpufreqstats.c    |    3 +-
>>  display.c         |   23 +++++++++-----
>>  generic_cstates.c |   63 ++++++++++++++++++++++++++++++++++++++++
>>  powertop.c        |   82 
>> +++++++++++++++++++++++++++++++++++++++++++++++++----
>>  powertop.h        |   11 ++++---
>>  6 files changed, 162 insertions(+), 22 deletions(-)
>>  create mode 100644 generic_cstates.c
>
> With these patches applied, I lose the entire bottom section of "Top causes
> for wakeups". Reverting the patches brings them back. I'm testing this on my
> PC yet.

I tested it on my dual core desktop along with my laptop. Strange that
I don't see this issue.

Can you please maximize the size of the window and try again ? This
might give us some hint.

Thanks!
--
Regards,
Amit Arora
>
> /Amit
>
>

_______________________________________________
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev

Reply via email to