Re: [PATCH] edk2-firmware: fix PlatformStandaloneMmRpmb.dsc build

[Jon Mason <jdmason@kudzu.us>]

On Tue, Jan 14, 2025 at 11:41:54AM +0200, Mikko Rapeli wrote:
> With backported patch from upstream. Error was:
> 
> | build.py...
> | /home/builder/src/base/repo/build/tmp_rockpi4b/work/rockpi4b-poky-linux/edk2-firmware/202408/edk2/edk2-platforms/Platform/StandaloneMm/PlatformS
> tandaloneMmPkg/PlatformStandaloneMmRpmb.dsc(...): error 4000: Instance of library class [HobPrintLib] is not found
> |       in [/home/builder/src/base/repo/build/tmp_rockpi4b/work/rockpi4b-poky-linux/edk2-firmware/202408/edk2/StandaloneMmPkg/Core/StandaloneMmCor
> e.inf] [AARCH64]
> |       consumed by module [/home/builder/src/base/repo/build/tmp_rockpi4b/work/rockpi4b-poky-linux/edk2-firmware/202408/edk2/StandaloneMmPkg/Core
> /StandaloneMmCore.inf]
> 
> Signed-off-by: Mikko Rapeli <mikko.rapeli@linaro.org>

I'm seeing this patch fail to apply (as part of the edk2 build
process) on every edk2 platform
https://gitlab.com/jonmason00/meta-arm/-/jobs/8844673217
https://gitlab.com/jonmason00/meta-arm/-/jobs/8844673227
https://gitlab.com/jonmason00/meta-arm/-/jobs/8844673281
https://gitlab.com/jonmason00/meta-arm/-/jobs/8844673312
https://gitlab.com/jonmason00/meta-arm/-/jobs/8844673331
https://gitlab.com/jonmason00/meta-arm/-/jobs/8844673346

Thanks,
Jon

> ---
>  meta-arm/recipes-bsp/uefi/edk2-firmware.inc   |  1 +
>  .../0001-Platform-StMmRpmb-Fix-build.patch    | 38 +++++++++++++++++++
>  2 files changed, 39 insertions(+)
>  create mode 100644 meta-arm/recipes-bsp/uefi/files/0001-Platform-StMmRpmb-Fix-build.patch
> 
> diff --git a/meta-arm/recipes-bsp/uefi/edk2-firmware.inc b/meta-arm/recipes-bsp/uefi/edk2-firmware.inc
> index 42668d9d..c10154aa 100644
> --- a/meta-arm/recipes-bsp/uefi/edk2-firmware.inc
> +++ b/meta-arm/recipes-bsp/uefi/edk2-firmware.inc
> @@ -20,6 +20,7 @@ SRCBRANCH_edk2_platforms = "master"
>  SRC_URI = "\
>      ${SRC_URI_EDK2};branch=${SRCBRANCH_edk2_platforms};name=edk2;destsuffix=edk2 \
>      ${SRC_URI_EDK2_PLATFORMS};branch=${SRCBRANCH_edk2};name=edk2-platforms;destsuffix=edk2/edk2-platforms \
> +    file://0001-Platform-StMmRpmb-Fix-build.patch;patchdir=edk2-platforms \
>  "
>  
>  SRCREV_FORMAT         = "edk2_edk2-platforms"
> diff --git a/meta-arm/recipes-bsp/uefi/files/0001-Platform-StMmRpmb-Fix-build.patch b/meta-arm/recipes-bsp/uefi/files/0001-Platform-StMmRpmb-Fix-build.patch
> new file mode 100644
> index 00000000..234ccf34
> --- /dev/null
> +++ b/meta-arm/recipes-bsp/uefi/files/0001-Platform-StMmRpmb-Fix-build.patch
> @@ -0,0 +1,38 @@
> +From ee185eda7b7ec9a0c82827fea197fbb73d971f93 Mon Sep 17 00:00:00 2001
> +From: Ard Biesheuvel <ardb@kernel.org>
> +Date: Thu, 29 Aug 2024 21:21:50 +0200
> +Subject: [PATCH] Platform/StMmRpmb: Fix build
> +
> +Add some missing library class resolutions relating to changes in the
> +core upstream EDK2 repo.
> +
> +Signed-off-by: Ard Biesheuvel <ardb@kernel.org>
> +Reviewed-by: Ilias Apalodimas <ilias.apalodimas@linaro.org>
> +---
> + .../PlatformStandaloneMmPkg/PlatformStandaloneMmRpmb.dsc       | 3 +++
> + 1 file changed, 3 insertions(+)
> +
> +Upstream-Status: Backport
> +
> +diff --git a/Platform/StandaloneMm/PlatformStandaloneMmPkg/PlatformStandaloneMmRpmb.dsc b/Platform/StandaloneMm/PlatformStandaloneMmPkg/PlatformStandaloneMmRpmb.dsc
> +index 653029c52f..54c7a63b41 100644
> +--- a/Platform/StandaloneMm/PlatformStandaloneMmPkg/PlatformStandaloneMmRpmb.dsc
> ++++ b/Platform/StandaloneMm/PlatformStandaloneMmPkg/PlatformStandaloneMmRpmb.dsc
> +@@ -41,11 +41,14 @@
> +   ExtractGuidedSectionLib|EmbeddedPkg/Library/PrePiExtractGuidedSectionLib/PrePiExtractGuidedSectionLib.inf
> +   FvLib|StandaloneMmPkg/Library/FvLib/FvLib.inf
> +   HobLib|StandaloneMmPkg/Library/StandaloneMmCoreHobLib/StandaloneMmCoreHobLib.inf
> ++  HobPrintLib|MdeModulePkg/Library/HobPrintLib/HobPrintLib.inf
> ++  ImagePropertiesRecordLib|MdeModulePkg/Library/ImagePropertiesRecordLib/ImagePropertiesRecordLib.inf
> +   IoLib|MdePkg/Library/BaseIoLibIntrinsic/BaseIoLibIntrinsic.inf
> +   NULL|MdePkg/Library/BaseStackCheckLib/BaseStackCheckLib.inf
> +   MemLib|StandaloneMmPkg/Library/StandaloneMmMemLib/StandaloneMmMemLib.inf
> +   MemoryAllocationLib|StandaloneMmPkg/Library/StandaloneMmCoreMemoryAllocationLib/StandaloneMmCoreMemoryAllocationLib.inf
> +   PcdLib|MdePkg/Library/BasePcdLibNull/BasePcdLibNull.inf
> ++  PeCoffGetEntryPointLib|MdePkg/Library/BasePeCoffGetEntryPointLib/BasePeCoffGetEntryPointLib.inf
> +   PeCoffLib|MdePkg/Library/BasePeCoffLib/BasePeCoffLib.inf
> +   PrintLib|MdePkg/Library/BasePrintLib/BasePrintLib.inf
> +   VariablePolicyLib|MdeModulePkg/Library/VariablePolicyLib/VariablePolicyLib.inf
> +-- 
> +2.43.0
> +
> -- 
> 2.43.0
> 
>