On 5/7/2014 3:04 PM, Al Stone wrote:
On 05/06/2014 08:10 AM, Ashwin Chaugule wrote:
This series contains experimental patchwork for PCC and CPPC as defined in the current ACPI5.0a spec. The PCC patch was tested on an ARMv7 platform and on a hacked up PCC emulator. The CPPC part is only compile tested. Both are very much a WIP. Any feedback and help with testing would be highly appreciated!
Ashwin Chaugule (2): ACPI: Add support for Platform Communication Channel CPPC: Initial support for Collaborative Processor Performance Control
arch/arm64/Kconfig | 2 + drivers/acpi/Kconfig | 10 ++ drivers/acpi/Makefile | 2 +- drivers/acpi/pcc.c | 209 +++++++++++++++++++++++++++++ drivers/cpufreq/Kconfig | 11 +- drivers/cpufreq/Makefile | 1 + drivers/cpufreq/cppc-cpufreq.c | 298 +++++++++++++++++++++++++++++++++++++++++ 7 files changed, 531 insertions(+), 2 deletions(-) create mode 100644 drivers/acpi/pcc.c create mode 100644 drivers/cpufreq/cppc-cpufreq.c
I think I'm liking this patch set. There are a couple of minor things -- e.g., alignment of the comments in the definition of enum cppc_pcc_regs -- but for the most part, looks pretty good.
For upstreaming, I think it would be wise to provide a way to test this (as part of the patch set) so it can be demonstrated it works to most anyone that wants to try it. If that could be done in such a way so that it's arch-independent, that would be even cooler. Barring that, I think it's critical that others know how to reproduce what's been done, so at a minimum there needs to be detailed data on how to test this.
HP plans to test it on an x86 platform of ours that has the firmware support and will post results. We'd like to see one driver that supports x86 and ARM.
-- ljk