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.web09.1501.1610700670487854491 for ; Fri, 15 Jan 2021 00:51:10 -0800 Authentication-Results: mx.groups.io; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=Pfeo2CtR; 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=1610700669; h=from:from:reply-to: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=yo6W6CfsMeHN5no9tHS7VLjHYeUaMWc2ygm9fyScksY=; b=Pfeo2CtRLl8h4f+MT76Gcarza4dsg0aOohJZXTPJMs7hWa6GEqAuqGrPS8Gh+nildK5+kN I3XBGANQccWX1lw5Eb038OToMdFVLh6e+NNJFBAYAvjDJIFD9xUKZ9cFjwC1KQSa0IbJgc YrAyDaegJMxqnrJrcqGizMehezQg3S0= 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-121-4flL25YnNoi-Kx_5ZkA4Kw-1; Fri, 15 Jan 2021 03:51:05 -0500 X-MC-Unique: 4flL25YnNoi-Kx_5ZkA4Kw-1 Received: from smtp.corp.redhat.com (int-mx08.intmail.prod.int.phx2.redhat.com [10.5.11.23]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id AA67C1927800; Fri, 15 Jan 2021 08:51:02 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-112-223.ams2.redhat.com [10.36.112.223]) by smtp.corp.redhat.com (Postfix) with ESMTP id 30EAF19C71; Fri, 15 Jan 2021 08:50:57 +0000 (UTC) Subject: Re: [edk2-devel] [edk2-platforms PATCH 0/3] add OrderedCollectionLib class resolution From: "Laszlo Ersek" To: devel@edk2.groups.io Cc: Agyeman Prince , Ard Biesheuvel , Chasel Chiu , Deepika Kethi Reddy , Jeremy Soller , Kathappan Esakkithevar , Kelly Steele , Leif Lindholm , 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)" Reply-To: devel@edk2.groups.io, lersek@redhat.com References: <20210113082843.9095-1-lersek@redhat.com> Message-ID: Date: Fri, 15 Jan 2021 09:50:57 +0100 MIME-Version: 1.0 In-Reply-To: <20210113082843.9095-1-lersek@redhat.com> X-Scanned-By: MIMEDefang 2.84 on 10.5.11.23 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 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 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 >