Skip to content

Commit

Permalink
UefiPayloadPkg/UefiPayloadPkg.dsc: Move out the PCD settings from UiA…
Browse files Browse the repository at this point in the history
…pp scope

Setting the PCDs only in the UiApp scope does not seem not to be
propagated to the DasharoModulesPkg libraries when FixedPcdGetBool is
used with those PCDs. Move away the PCD settings to the global PCD
section in the DSC.

Signed-off-by: Michał Żygowski <[email protected]>
  • Loading branch information
miczyg1 committed Jul 1, 2024
1 parent 85983e5 commit d130aec
Showing 1 changed file with 7 additions and 5 deletions.
12 changes: 7 additions & 5 deletions UefiPayloadPkg/UefiPayloadPkg.dsc
Original file line number Diff line number Diff line change
Expand Up @@ -464,6 +464,13 @@
gEfiMdePkgTokenSpaceGuid.PcdPerformanceLibraryPropertyMask|0x1
!endif

!if $(DASHARO_SYSTEM_FEATURES_ENABLE) == TRUE
gDasharoSystemFeaturesTokenSpaceGuid.PcdShowMenu|TRUE
gDasharoSystemFeaturesTokenSpaceGuid.PcdShowIommuOptions|$(IOMMU_ENABLE)
gDasharoSystemFeaturesTokenSpaceGuid.PcdShowSerialPortMenu|$(SERIAL_TERMINAL)
gDasharoSystemFeaturesTokenSpaceGuid.PcdShowPs2Option|$(PS2_KEYBOARD_ENABLE)
!endif

[PcdsPatchableInModule.common]
gEfiMdePkgTokenSpaceGuid.PcdReportStatusCodePropertyMask|0x7
gEfiMdePkgTokenSpaceGuid.PcdDebugPrintErrorLevel|0x8000004F
Expand Down Expand Up @@ -677,11 +684,6 @@
NULL|DasharoModulePkg/Library/DasharoSystemFeaturesUiLib/DasharoSystemFeaturesUiLib.inf
NULL|MdeModulePkg/Library/BootManagerUiLib/BootManagerUiLib.inf
NULL|MdeModulePkg/Library/BootMaintenanceManagerUiLib/BootMaintenanceManagerUiLib.inf
<PcdsFixedAtBuild>
gDasharoSystemFeaturesTokenSpaceGuid.PcdShowMenu|$(DASHARO_SYSTEM_FEATURES_ENABLE)
gDasharoSystemFeaturesTokenSpaceGuid.PcdShowIommuOptions|$(IOMMU_ENABLE)
gDasharoSystemFeaturesTokenSpaceGuid.PcdShowSerialPortMenu|$(SERIAL_TERMINAL)
gDasharoSystemFeaturesTokenSpaceGuid.PcdShowPs2Option|$(PS2_KEYBOARD_ENABLE)
}
MdeModulePkg/Application/BootManagerMenuApp/BootManagerMenuApp.inf
!if $(RAM_DISK_ENABLE) == TRUE
Expand Down

0 comments on commit d130aec

Please sign in to comment.