Re: [PATCH 3/3] arm_scpi: modify to support acpi

From: Sudeep Holla
Date: Thu Nov 24 2022 - 06:10:22 EST


Hi Wang,

On Thu, Nov 24, 2022 at 03:09:31PM +0800, Wang Honghui wrote:
> arm_scpi: modify to support acpi
>

1. The commit message is pretty useless. I don't get the complete picture
of why you need this change or what you are trying to do in this change.

2. I am unable to see the series as a whole, just have 2 versions of 3/3
in my inbox.

3. This is not the correct way to use SCPI or mailbox in ACPI. You need
to use PCC and PCC OpRegion to achieve what you want. The whole SCPI
protocol details gets abstracted in the firmware(ACPI ASL)

Hope this helps. So in short, NACK for this approach for ACPI.

--
Regards,
Sudeep