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

["Philippe Mathieu-Daudé" <philmd@redhat.com>]

On 1/13/21 9:28 AM, 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(+)

Reviewed-by: Philippe Mathieu-Daude <philmd@redhat.com>