From e2ef8b9a684fcee4bb52c706020bfa5fca796665 Mon Sep 17 00:00:00 2001 From: Benjamin You Date: Tue, 14 Nov 2017 16:41:13 +0800 Subject: [PATCH] CorebootModulePkg/CbSupportDxe: Remove duplicated IO Space addition Since UefiCpuPkg's CpuDxe Driver already adds Local Apic's MMIO space to GCD, CorebootModulePkg's CbSupportDxe should not do this again. Doing this again causes error return status from GCD service, and ASSERT (FALSE) with debug build, so the duplicated addition is removed. Cc: Maurice Ma Cc: Prince Agyeman Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Benjamin You Reviewed-by: Maurice Ma --- CorebootModulePkg/CbSupportDxe/CbSupportDxe.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/CorebootModulePkg/CbSupportDxe/CbSupportDxe.c b/CorebootModulePkg/CbSupportDxe/CbSupportDxe.c index 24bacf815c..c526c9e871 100755 --- a/CorebootModulePkg/CbSupportDxe/CbSupportDxe.c +++ b/CorebootModulePkg/CbSupportDxe/CbSupportDxe.c @@ -140,9 +140,6 @@ CbDxeEntryPoint ( // // Report MMIO/IO Resources // - Status = CbReserveResourceInGcd (TRUE, EfiGcdMemoryTypeMemoryMappedIo, 0xFEE00000, SIZE_1MB, 0, SystemTable); // LAPIC - ASSERT_EFI_ERROR (Status); - Status = CbReserveResourceInGcd (TRUE, EfiGcdMemoryTypeMemoryMappedIo, 0xFEC00000, SIZE_4KB, 0, SystemTable); // IOAPIC ASSERT_EFI_ERROR (Status);