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