From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [216.205.24.124]) by mx.groups.io with SMTP id smtpd.web10.30531.1610955729299694914 for ; Sun, 17 Jan 2021 23:42:09 -0800 Authentication-Results: mx.groups.io; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=XwCuGjPH; spf=pass (domain: redhat.com, ip: 216.205.24.124, mailfrom: lersek@redhat.com) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1610955727; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=rqEpvjLezNLJ+6392CEK22+qScxW/U565CTaJiny95k=; b=XwCuGjPHLAIx9BJfcNbgHiqfZ+ii1K2R35lBGibsqjCZ3xBrMCb9o7yhmsVStsm6+cB0wb whxzYh7PhpZoFNxA7WmjX/91KY+W2fGHQZ+qkkZOJ7dgd/HJdRYO8NadLxZo9LOxt3HFUp tZ4+eZNVu8ZV5d/xvBVCYuJxzENOtew= Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-397-kXEydAFEP-i9MsanLmHRjA-1; Mon, 18 Jan 2021 02:42:04 -0500 X-MC-Unique: kXEydAFEP-i9MsanLmHRjA-1 Received: from smtp.corp.redhat.com (int-mx05.intmail.prod.int.phx2.redhat.com [10.5.11.15]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id EFCCE1800D42; Mon, 18 Jan 2021 07:42:00 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-113-141.ams2.redhat.com [10.36.113.141]) by smtp.corp.redhat.com (Postfix) with ESMTP id 744F05D72E; Mon, 18 Jan 2021 07:41:56 +0000 (UTC) Subject: Re: [edk2-devel] [edk2-platforms PATCH 0/3] add OrderedCollectionLib class resolution To: Leif Lindholm , devel@edk2.groups.io Cc: Agyeman Prince , Ard Biesheuvel , Chasel Chiu , Deepika Kethi Reddy , Jeremy Soller , Kathappan Esakkithevar , Kelly Steele , Marcin Wojtas , Meenakshi Aggarwal , Michael D Kinney , Nate DeSimone , =?UTF-8?Q?Philippe_Mathieu-Daud=c3=a9?= , Rangasai V Chaganty , Thomas Abraham , Wenyi Xie , Yi Qian , Zailiang Sun , "Ard Biesheuvel (ARM address)" References: <20210113082843.9095-1-lersek@redhat.com> <20210115220448.GV1664@vanye> From: "Laszlo Ersek" Message-ID: <4c1f0fe2-f409-5c55-091f-6ddc95a339f5@redhat.com> Date: Mon, 18 Jan 2021 08:41:55 +0100 MIME-Version: 1.0 In-Reply-To: <20210115220448.GV1664@vanye> X-Scanned-By: MIMEDefang 2.79 on 10.5.11.15 Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=lersek@redhat.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 8bit On 01/15/21 23:04, Leif Lindholm wrote: > 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 :) Yes, that was a surprise. > > Many thanks for fixing up BeagleBoard as well. > > For the relevant patches: > Acked-by: Leif Lindholm Thanks! Laszlo > >> On 01/13/21 09:28, Laszlo Ersek wrote: >>> The edk2 patch >>> >>> ShellPkg/ShellCommandLib: add ShellSortFileList >>> >>> for 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 >>> Cc: Ard Biesheuvel >>> Cc: Chasel Chiu >>> Cc: Deepika Kethi Reddy >>> Cc: Jeremy Soller >>> Cc: Kathappan Esakkithevar >>> Cc: Kelly Steele >>> Cc: Leif Lindholm >>> Cc: Marcin Wojtas >>> Cc: Meenakshi Aggarwal >>> Cc: Michael D Kinney >>> Cc: Nate DeSimone >>> Cc: Philippe Mathieu-Daudé >>> Cc: Rangasai V Chaganty >>> Cc: Thomas Abraham >>> Cc: Wenyi Xie >>> Cc: Yi Qian >>> Cc: Zailiang Sun >>> >>> 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 >>> >> >