[edk2-devel] [edk2-platforms][PATCH v1 00/10] Add variableFlashInfoLib

Michael D Kinney michael.d.kinney at intel.com
Fri Apr 22 23:15:20 UTC 2022


Hi Michael,

1) This patch series is dependent on the edk2 repo patch series that adds the 
   VariableFlashInfoLib.  This patch series cannot be committed until the 
   edk2 changes are committed because it will break all the edk2-platforms
   if this one is committed first.

2) The related edk2 changes are a breaking change for all downstream platforms
   that also need to add VariableFlashInfoLib to their DSC files.

   Should breaking changes like this also require a separate email that
   is sent to devel at edk2.groups.io, announce at edk2.groups.io, and discuss at edk2.groups.io
   to increase the visibility of breaking changes?


Series Reviewed-by: Michael D Kinney <michael.d.kinney at intel.com>


Best regards,

Mike

> -----Original Message-----
> From: devel at edk2.groups.io <devel at edk2.groups.io> On Behalf Of Michael Kubacki
> Sent: Wednesday, April 20, 2022 1:44 PM
> To: devel at edk2.groups.io
> Cc: Chang, Abner <abner.chang at hpe.com>; Ard Biesheuvel <ardb+tianocore at kernel.org>; Chiu, Chasel <chasel.chiu at intel.com>;
> Schaefer, Daniel <daniel.schaefer at hpe.com>; Dong, Eric <eric.dong at intel.com>; Gilbert Chen <gilbert.chen at hpe.com>; Graeme Gregory
> <graeme at nuviainc.com>; Ilias Apalodimas <ilias.apalodimas at linaro.org>; Jeremy Linton <jeremy.linton at arm.com>; Leif Lindholm
> <quic_llindhol at quicinc.com>; Gao, Liming <gaoliming at byosoft.com.cn>; Masami Hiramatsu <masami.hiramatsu at linaro.org>; Desimone,
> Nathaniel L <nathaniel.l.desimone at intel.com>; Radoslaw Biernacki <rad at semihalf.com>; Sami Mujawar <sami.mujawar at arm.com>
> Subject: [edk2-devel] [edk2-platforms][PATCH v1 00/10] Add variableFlashInfoLib
> 
> From: Michael Kubacki <michael.kubacki at 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/88787
> 
> Cc: Abner Chang <abner.chang at hpe.com>
> Cc: Ard Biesheuvel <ardb+tianocore at kernel.org>
> Cc: Chasel Chiu <chasel.chiu at intel.com>
> Cc: Daniel Schaefer <daniel.schaefer at hpe.com>
> Cc: Eric Dong <eric.dong at intel.com>
> Cc: Gilbert Chen <gilbert.chen at hpe.com>
> Cc: Graeme Gregory <graeme at nuviainc.com>
> Cc: Ilias Apalodimas <ilias.apalodimas at linaro.org>
> Cc: Jeremy Linton <jeremy.linton at arm.com>
> Cc: Leif Lindholm <quic_llindhol at quicinc.com>
> Cc: Liming Gao <gaoliming at byosoft.com.cn>
> Cc: Masami Hiramatsu <masami.hiramatsu at linaro.org>
> Cc: Nate DeSimone <nathaniel.l.desimone at intel.com>
> Cc: Radoslaw Biernacki <rad at semihalf.com>
> Cc: Sami Mujawar <sami.mujawar at arm.com>
> Signed-off-by: Michael Kubacki <michael.kubacki at microsoft.com>
> 
> Michael Kubacki (10):
>   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
> 
>  Platform/AMD/OverdriveBoard/OverdriveBoard.dsc                             | 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 +
>  12 files changed, 12 insertions(+)
> 
> --
> 2.28.0.windows.1
> 
> 
> 
> -=-=-=-=-=-=
> Groups.io Links: You receive all messages sent to this group.
> View/Reply Online (#89148): https://edk2.groups.io/g/devel/message/89148
> Mute This Topic: https://groups.io/mt/90592905/1643496
> Group Owner: devel+owner at edk2.groups.io
> Unsubscribe: https://edk2.groups.io/g/devel/unsub [michael.d.kinney at intel.com]
> -=-=-=-=-=-=
> 



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#89240): https://edk2.groups.io/g/devel/message/89240
Mute This Topic: https://groups.io/mt/90592905/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