[edk2-devel] [PATCH v2 1/2] UefiPayloadPkg: Store the real size of the MMCONF window
Ma, Maurice
maurice.ma at intel.com
Mon Jul 20 15:22:15 UTC 2020
Hi Marcello,
Could you put a little bit more details in the patch commit message ?
Thanks
Maurice
> -----Original Message-----
> From: Marcello Sylvester Bauer <marcello.bauer at 9elements.com>
> Sent: Thursday, July 16, 2020 4:48
> To: devel at edk2.groups.io
> Cc: Patrick Rudolph <patrick.rudolph at 9elements.com>; Christian Walter
> <christian.walter at 9elements.com>; Ma, Maurice <maurice.ma at intel.com>;
> Desimone, Nathaniel L <nathaniel.l.desimone at intel.com>; Zeng, Star
> <star.zeng at intel.com>; Kinney, Michael D <michael.d.kinney at intel.com>; Gao,
> Liming <liming.gao at intel.com>
> Subject: [PATCH v2 1/2] UefiPayloadPkg: Store the real size of the MMCONF
> window
>
> From: Patrick Rudolph <patrick.rudolph at 9elements.com>
>
> This will fix issues with the PciBusDxe.
>
> Signed-off-by: Patrick Rudolph <patrick.rudolph at 9elements.com>
> Signed-off-by: Marcello Sylvester Bauer <marcello.bauer at 9elements.com>
> Cc: Patrick Rudolph <patrick.rudolph at 9elements.com>
> Cc: Christian Walter <christian.walter at 9elements.com>
> Cc: Maurice Ma <maurice.ma at intel.com>
> Cc: Nate DeSimone <nathaniel.l.desimone at intel.com>
> Cc: Star Zeng <star.zeng at intel.com>
> Cc: Michael D Kinney <michael.d.kinney at intel.com>
> Cc: Liming Gao <liming.gao at intel.com>
> ---
> UefiPayloadPkg/Include/Guid/AcpiBoardInfoGuid.h | 1 +
> UefiPayloadPkg/BlSupportPei/BlSupportPei.c | 3 +++
> 2 files changed, 4 insertions(+)
>
> diff --git a/UefiPayloadPkg/Include/Guid/AcpiBoardInfoGuid.h
> b/UefiPayloadPkg/Include/Guid/AcpiBoardInfoGuid.h
> index fe783fe5e14c..043b748ae4a9 100644
> --- a/UefiPayloadPkg/Include/Guid/AcpiBoardInfoGuid.h
> +++ b/UefiPayloadPkg/Include/Guid/AcpiBoardInfoGuid.h
> @@ -24,6 +24,7 @@ typedef struct {
> UINT64 PmTimerRegBase;
>
> UINT64 ResetRegAddress;
>
> UINT64 PcieBaseAddress;
>
> + UINT64 PcieBaseSize;
>
> } ACPI_BOARD_INFO;
>
>
>
> #endif
>
> diff --git a/UefiPayloadPkg/BlSupportPei/BlSupportPei.c
> b/UefiPayloadPkg/BlSupportPei/BlSupportPei.c
> index 22972453117a..a7e99f9ec6de 100644
> --- a/UefiPayloadPkg/BlSupportPei/BlSupportPei.c
> +++ b/UefiPayloadPkg/BlSupportPei/BlSupportPei.c
> @@ -240,8 +240,10 @@ Done:
> if (MmCfgHdr != NULL) {
>
> MmCfgBase =
> (EFI_ACPI_MEMORY_MAPPED_ENHANCED_CONFIGURATION_SPACE_BASE_A
> DDRESS_ALLOCATION_STRUCTURE *)((UINT8*) MmCfgHdr + sizeof
> (*MmCfgHdr));
>
> AcpiBoardInfo->PcieBaseAddress = MmCfgBase->BaseAddress;
>
> + AcpiBoardInfo->PcieBaseSize = (MmCfgBase->EndBusNumber + 1 -
> MmCfgBase->StartBusNumber) * 4096 * 32 * 8;
>
> } else {
>
> AcpiBoardInfo->PcieBaseAddress = 0;
>
> + AcpiBoardInfo->PcieBaseSize = 0;
>
> }
>
> DEBUG ((DEBUG_INFO, "PmCtrl Reg 0x%lx\n", AcpiBoardInfo-
> >PmCtrlRegBase));
>
> DEBUG ((DEBUG_INFO, "PmTimer Reg 0x%lx\n", AcpiBoardInfo-
> >PmTimerRegBase));
>
> @@ -250,6 +252,7 @@ Done:
> DEBUG ((DEBUG_INFO, "PmEvt Reg 0x%lx\n", AcpiBoardInfo->PmEvtBase));
>
> DEBUG ((DEBUG_INFO, "PmGpeEn Reg 0x%lx\n", AcpiBoardInfo-
> >PmGpeEnBase));
>
> DEBUG ((DEBUG_INFO, "PcieBaseAddr 0x%lx\n", AcpiBoardInfo-
> >PcieBaseAddress));
>
> + DEBUG ((DEBUG_INFO, "PcieBaseSize 0x%lx\n", AcpiBoardInfo-
> >PcieBaseSize));
>
>
>
> //
>
> // Verify values for proper operation
>
> --
> 2.27.0
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#62848): https://edk2.groups.io/g/devel/message/62848
Mute This Topic: https://groups.io/mt/75539545/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-
More information about the edk2-devel-archive
mailing list