[edk2-devel] [PATCH v2 14/17] Silicon/Ampere: Update ArmPlatformLib to work with changed ARM_CORE_INFO

[Nhi Pham via groups.io]

On 27/03/2022 05:00, Leif Lindholm wrote:
> On Sat, Mar 26, 2022 at 16:12:07 +0700, Nhi Pham via groups.io wrote:
>>> Would you be happy for me to fold that into
>>> "AmpereAltraPkg, JadePkg: Add ACPI support", or would you be able to
>>> submit a v6 of that patch only?
>>>
>>> Best Regards,
>>>
>>> Leif
>> Thanks much for the patch. The MPIDR decoding matches with Rebecca's update
>> for the ArmPlatformLib earlier and the ARM_CORE_INFO is just consumed in the
>> AcpiSrat.c. So, that is good for now. Please help fold that into the ACPI
>> patch when merging the rest of the Mt. Jade support patchset.
> Done. I just pushed up to and including patch 29 as
> 41628dcf3332..e18e208e7105.
Thanks, Leif. I'm happy to hear that.
>
> (I will take a look at patch 30 on Monday, but I couldn't find that I
> had signed off on it anywhere.)

It seems like you have not reviewed the v5 of patch 30 yet.

Best regards,

Nhi

>
> Apologies for the delay in getting this merged.
>
>> In the future, I will follow up with a patch that makes the representation
>> of MPIDR in the ARM_CORE_INFO the same as in MADT table.
> That sounds ideal, thanks.
>
> Best Regards,
>
> Leif


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#88145): https://edk2.groups.io/g/devel/message/88145
Mute This Topic: https://groups.io/mt/87777839/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-