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 >
I understand that it is logical to keep the forward-ported TI patch separated from the generic code you wrote in the beginning. But I think we should merge the patches since the bulk of the changes in the 2nd patch is just the renaming of the file. More comments in other email. /Amit _______________________________________________ linaro-dev mailing list linaro-dev@lists.linaro.org http://lists.linaro.org/mailman/listinfo/linaro-dev