Re: [edk2-devel] [edk2-platforms PATCH 0/3] add OrderedCollectionLib class resolution

["Leif Lindholm" <leif@nuviainc.com>]

Hi Laszlo,

On Fri, Jan 15, 2021 at 09:50:57 +0100, Laszlo Ersek wrote:
> Hi All,
> 
> sorry about the message; it's been only two days indeed.
> 
> This patch series is trivial, it adds one lib class resolution to each
> of 22 platforms in edk2-platforms.
> 
> I build-tested the platforms modified by this change, so I'm planning to
> push the series to edk2-platforms one week after posting it -- even if I
> get no feedback. Thus, the reason I'm sending this email isn't really to
> ping or urge reviewers, it's more like announcing my plan.
> 
> Thanks,
> Laszlo

Apologies, I woke up Wednesday to the news we had been acquired by
Qualcomm, and the week got a bit hazy at that point :)

Many thanks for fixing up BeagleBoard as well.

For the relevant patches:
Acked-by: Leif Lindholm <leif@nuviainc.com>

> On 01/13/21 09:28, Laszlo Ersek wrote:
> > The edk2 patch
> > 
> >   ShellPkg/ShellCommandLib: add ShellSortFileList
> > 
> > for <https://bugzilla.tianocore.org/show_bug.cgi?id=3151> will make the
> > 
> >   ShellPkg/Library/UefiShellCommandLib/UefiShellCommandLib.inf
> > 
> > instance dependent on the OrderedCollectionLib class. Because the shell
> > binaries in several silicon and platform DSCs in edk2-platforms consume
> > the above UefiShellCommandLib instance, resolve the OrderedCollectionLib
> > class in those DSCs.
> > 
> > Cc: Agyeman Prince <prince.agyeman@intel.com>
> > Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
> > Cc: Chasel Chiu <chasel.chiu@intel.com>
> > Cc: Deepika Kethi Reddy <deepika.kethi.reddy@intel.com>
> > Cc: Jeremy Soller <jeremy@system76.com>
> > Cc: Kathappan Esakkithevar <kathappan.esakkithevar@intel.com>
> > Cc: Kelly Steele <kelly.steele@intel.com>
> > Cc: Leif Lindholm <leif@nuviainc.com>
> > Cc: Marcin Wojtas <mw@semihalf.com>
> > Cc: Meenakshi Aggarwal <meenakshi.aggarwal@nxp.com>
> > Cc: Michael D Kinney <michael.d.kinney@intel.com>
> > Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> > Cc: Philippe Mathieu-Daudé <philmd@redhat.com>
> > Cc: Rangasai V Chaganty <rangasai.v.chaganty@intel.com>
> > Cc: Thomas Abraham <thomas.abraham@arm.com>
> > Cc: Wenyi Xie <xiewenyi2@huawei.com>
> > Cc: Yi Qian <yi.qian@intel.com>
> > Cc: Zailiang Sun <zailiang.sun@intel.com>
> > 
> > Thanks
> > Laszlo
> > 
> > Laszlo Ersek (3):
> >   Platform/BeagleBoard: add missing TimeBaseLib resolution
> >   ARM Silicon and Platforms: add OrderedCollectionLib class resolution
> >   Intel Platforms: add OrderedCollectionLib class resolution
> > 
> >  Platform/AMD/OverdriveBoard/OverdriveBoard.dsc                          | 1 +
> >  Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc                            | 1 +
> >  Platform/BeagleBoard/BeagleBoardPkg/BeagleBoardPkg.dsc                  | 2 ++
> >  Platform/Hisilicon/D03/D03.dsc                                          | 1 +
> >  Platform/Hisilicon/HiKey/HiKey.dsc                                      | 1 +
> >  Platform/Hisilicon/HiKey960/HiKey960.dsc                                | 1 +
> >  Platform/Intel/CometlakeOpenBoardPkg/CometlakeURvp/OpenBoardPkg.dsc     | 1 +
> >  Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.dsc         | 1 +
> >  Platform/Intel/KabylakeOpenBoardPkg/KabylakeRvp3/OpenBoardPkg.dsc       | 1 +
> >  Platform/Intel/QuarkPlatformPkg/Quark.dsc                               | 1 +
> >  Platform/Intel/QuarkPlatformPkg/QuarkMin.dsc                            | 1 +
> >  Platform/Intel/SimicsOpenBoardPkg/BoardX58Ich10/OpenBoardPkg.dsc        | 1 +
> >  Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgIA32.dsc                    | 1 +
> >  Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgX64.dsc                     | 1 +
> >  Platform/Intel/WhiskeylakeOpenBoardPkg/UpXtreme/OpenBoardPkg.dsc        | 1 +
> >  Platform/Intel/WhiskeylakeOpenBoardPkg/WhiskeylakeURvp/OpenBoardPkg.dsc | 1 +
> >  Platform/LeMaker/CelloBoard/CelloBoard.dsc                              | 1 +
> >  Platform/Socionext/DeveloperBox/DeveloperBox.dsc                        | 1 +
> >  Platform/Socionext/SynQuacerEvalBoard/SynQuacerEvalBoard.dsc            | 1 +
> >  Platform/SoftIron/Overdrive1000Board/Overdrive1000Board.dsc             | 1 +
> >  Silicon/Marvell/Armada7k8k/Armada7k8k.dsc.inc                           | 1 +
> >  Silicon/NXP/NxpQoriqLs.dsc.inc                                          | 1 +
> >  22 files changed, 23 insertions(+)
> > 
> > 
> > base-commit: 47b4a1db02873ffe864486dfcc8a87c409bd3d4d
> > 
>