Re: [PATCH 0/2] Make cppc acpi driver aware of pcc subspace ids

[Prakash, Prashanth]

On 6/28/2017 3:20 PM, Rafael J. Wysocki wrote:
> On Tuesday, June 13, 2017 02:17:09 PM George Cherian wrote:
>> The current cppc acpi driver works with only one pcc subspace id.
>> It maintains and registers only one pcc channel even if the acpi table has 
>> different pcc subspace ids. 
>>
>> As per ACPI 6.2 spec  all PCC registers, for all processors in the same 
>> performance domain (as defined by _PSD), must be defined to be in the same
>> subspace. The series tries to address the same by making cppc acpi driver 
>> aware of multiple possible pcc subspace ids.
>>
>> Patch 1 : In preparation to share the MAX_PCC_SUBSPACE definition with cppc acpi
>>           driver
>> Patch 2 : Make the cppc acpi driver aware of multiple pcc subspace ids.
>>
>>
>> George Cherian (2):
>>   mailbox: PCC: Move the MAX_PCC_SUBSPACES definition to header file
>>   ACPI / CPPC: Make cppc acpi driver aware of pcc subspace ids
>>
>>  drivers/acpi/cppc_acpi.c | 179 +++++++++++++++++++++++++----------------------
>>  drivers/mailbox/pcc.c    |   1 -
>>  include/acpi/pcc.h       |   1 +
>>  3 files changed, 97 insertions(+), 84 deletions(-)
> I need Ashwing and Prashanth to look at this series and let me know what they
> think.
Rafael/George,

I was on vacation last few weeks, I will go through this series this week and provide
feedback. Sorry about the delay.

--
Thanks,
Prashanth