Re: [edk2-platforms][PATCH v2 00/11] Add VariableFlashInfoLib

["Ard Biesheuvel" <ardb@kernel.org>]

On Tue, 26 Apr 2022 at 04:19, <mikuback@linux.microsoft.com> wrote:
>
> From: Michael Kubacki <michael.kubacki@microsoft.com>
>
> REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3479
>
> Adds an instance for the library class VariableFlashInfoLib that
> was recently introduced in MdeModulePkg. This change is made to
> allow the new variable driver to build that has a dependency on
> this library class and does not require any further platform
> changes.
>
> Note that this series is for an upcoming change in MdeModulePkg
> and requires that change to be present locally for testing.
>
> https://edk2.groups.io/g/devel/message/89296
>
> V2 Changes:
> - Add patch "VExpressPkg: Add VariableFlashInfoLib"
>
> Cc: Abner Chang <abner.chang@hpe.com>
> Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
> Cc: Chasel Chiu <chasel.chiu@intel.com>
> Cc: Daniel Schaefer <daniel.schaefer@hpe.com>
> Cc: Eric Dong <eric.dong@intel.com>
> Cc: Gilbert Chen <gilbert.chen@hpe.com>
> Cc: Graeme Gregory <graeme@nuviainc.com>
> Cc: Ilias Apalodimas <ilias.apalodimas@linaro.org>
> Cc: Jeremy Linton <jeremy.linton@arm.com>
> Cc: Leif Lindholm <quic_llindhol@quicinc.com>
> Cc: Liming Gao <gaoliming@byosoft.com.cn>
> Cc: Masami Hiramatsu <masami.hiramatsu@linaro.org>
> Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> Cc: Radoslaw Biernacki <rad@semihalf.com>
> Cc: Sami Mujawar <sami.mujawar@arm.com>
> Cc: Thomas Abraham <thomas.abraham@arm.com>
> Signed-off-by: Michael Kubacki <michael.kubacki@microsoft.com>
>
> Michael Kubacki (11):
>   OverdriveBoard: Add VariableFlashInfoLib
>   BeagleBoardPkg: Add VariableFlashInfoLib
>   MinPlatformPkg: Add VariableFlashInfoLib
>   CelloBoard: Add VariableFlashInfoLib
>   SbsaQemu: Add VariableFlashInfoLib
>   RaspberryPi: Add VariableFlashInfoLib
>   U5SeriesPkg: Add VariableFlashInfoLib
>   Socionext: Add VariableFlashInfoLib
>   Softiron: Add VariableFlashInfoLib
>   PlatformStandaloneMmPkg: Add VariableFlashInfoLib
>   VExpressPkg: Add VariableFlashInfoLib
>

For the series

Acked-by: Ard Biesheuvel <ardb@kernel.org>

*However* can we please fix this in the common frameworks? None of
this would be necessary if we could define a default resolution when
declaring a library class. That way, this problem would be fixed for
everybody not just the platforms that are in the open source repo.



>  Platform/AMD/OverdriveBoard/OverdriveBoard.dsc                             | 1 +
>  Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc                               | 1 +
>  Platform/BeagleBoard/BeagleBoardPkg/BeagleBoardPkg.dsc                     | 1 +
>  Platform/Intel/MinPlatformPkg/Include/Dsc/CoreCommonLib.dsc                | 1 +
>  Platform/LeMaker/CelloBoard/CelloBoard.dsc                                 | 1 +
>  Platform/Qemu/SbsaQemu/SbsaQemu.dsc                                        | 1 +
>  Platform/RaspberryPi/RPi3/RPi3.dsc                                         | 1 +
>  Platform/RaspberryPi/RPi4/RPi4.dsc                                         | 1 +
>  Platform/SiFive/U5SeriesPkg/FreedomU500VC707Board/U500.dsc                 | 1 +
>  Platform/SiFive/U5SeriesPkg/FreedomU540HiFiveUnleashedBoard/U540.dsc       | 1 +
>  Platform/Socionext/SynQuacerEvalBoard/SynQuacerEvalBoard.dsc               | 1 +
>  Platform/SoftIron/Overdrive1000Board/Overdrive1000Board.dsc                | 1 +
>  Platform/StandaloneMm/PlatformStandaloneMmPkg/PlatformStandaloneMmRpmb.dsc | 1 +
>  13 files changed, 13 insertions(+)
>
> --
> 2.28.0.windows.1
>