[edk2-devel] [`edk2-devel][PATCH] UefiPayloadPkg: Build a HOB from bootloader ACPI table

[Guo Dong]

Hi Ray,

In this patch, we added a shared file AcpiTable.c for both universal payload and non-universal payload.
The exposed API from this file is:   EFI_STATUS  BuildHobFromAcpi ( IN   UINT64 AcpiTableBase);
This function will build an ACPI board HOB based on the information from ACPI table.

For universal payload, it calls this function to build a hob for other modules. The main function is very simple and clear.

For non-universal payload, ACPI board HOB is used in the ParseMemoryInfo() callback for PCIE base info.
So we could get this HOB from the caller, or get this HOB inside the callback. I select to do it inside the callback.

Thanks,
Guo

-----Original Message-----
From: Ni, Ray <ray.ni at intel.com> 
Sent: Saturday, September 25, 2021 7:48 PM
To: Dong, Guo <guo.dong at intel.com>; devel at edk2.groups.io
Cc: Ma, Maurice <maurice.ma at intel.com>; You, Benjamin <benjamin.you at intel.com>
Subject: RE: [`edk2-devel][PATCH] UefiPayloadPkg: Build a HOB from bootloader ACPI table


-  Status = ParseMemoryInfo (MemInfoCallbackMmio, &AcpiBoardInfo);

+  Status = ParseMemoryInfo (MemInfoCallbackMmio, NULL);

Guo,
I am curious why you changed this part.
Without this change, MemInfoCallbackMmio() can get the AcpiBoardInfo from the parameter.
With the change, it has to locate the Guided HOB itself.


Other parts look good to me.

Thanks,
Ray



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