On 6/22/22 07:39, Pranav Madhu wrote:
Hi Pierre,
Thanks for reviewing this patch series. Please find my reply inline.
Regards,
Pranav
-----Original Message-----
From: Pierre Gondois <pierre.gond...@arm.com>
Sent: Tuesday, June 21, 2022 2:15 PM
To: devel@edk2.groups.io; Pranav Madhu <pranav.ma...@arm.com>;
Thomas Abraham <thomas.abra...@arm.com>
Cc: Sami Mujawar <sami.muja...@arm.com>; Ard Biesheuvel
<ardb+tianoc...@kernel.org>; nd <n...@arm.com>
Subject: Re: [edk2-devel] [edk2-platforms][PATCH V1 0/7] Fix ACPI Low
Power Idle states for RD platforms
Hello Pranav,
Would it be possible to extend this change to the Juno aswell as the platform
has a Null Residency Counter and doesn't support OS Initiated LPI ?
It can be changed for Juno as well, but I do not have access to any Juno boards
and this patch series is focusing on RD platforms. The Juno board update can be
posted as a separate patch.
Ok sure.
Another remark is that for RdV1MC, RdN2, RdV1 and RdN2Cfg1 platforms,
CPUs don't need to be encapsulated anymore in cluster objects
Cf:
Device (CL00) { // Cluster 0
Name (_HID, "ACPI0010")
Name (_UID, 0)
...
}
since Clusters don't have any additional information. But keeping them is
correct aswell.
These platforms have a minimal cluster logic but does not support low power
states. But the cluster encapsulation represents the platform topology
accurately. So it would be better to keep the cluster encapsulation.
Ok yes, this was just so other people are aware of this.
Reviewed-by: Pierre Gondois <pierre.gond...@arm.com>
Regards,
Pierre
----------
Groups.io Links:
You receive all messages sent to this group.
View/Reply Online (#90633)
<https://edk2.groups.io/g/devel/message/90633> | | Mute This Topic
<https://groups.io/mt/91652410/1821310> | New Topic
<https://edk2.groups.io/g/devel/post>
Your Subscription <https://edk2.groups.io/g/devel/editsub/1821310> |
Contact Group Owner <mailto:devel+ow...@edk2.groups.io> |
Unsubscribe
<https://edk2.groups.io/g/devel/unsub> [pierre.gond...@arm.com]
_._,_._,_
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#90714): https://edk2.groups.io/g/devel/message/90714
Mute This Topic: https://groups.io/mt/91652410/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
