From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [63.128.21.124]) by mx.groups.io with SMTP id smtpd.web10.1644.1610702867248323600 for ; Fri, 15 Jan 2021 01:27:47 -0800 Authentication-Results: mx.groups.io; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=bnWe5Vwr; spf=pass (domain: redhat.com, ip: 63.128.21.124, mailfrom: philmd@redhat.com) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1610702866; 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=MXIm+GnEt2voV5GSIMYLPLJ33ZaN2YcxTR1l6rQxLQo=; b=bnWe5Vwr7rUSR6FukFK4pRIJKN5s2vp++iI4N/19gHbva+5V677g2apT66tVCADawgnlBX fVoyhPq6QvO43BAq3MnfbqkQDK74lGkuzWGWRG78LGNBjf5PV8B3r3OocFutA14xoBBOuF d+6E0edV2DRA+29p1CVrKzqCtfkFM80= Received: from mail-ej1-f69.google.com (mail-ej1-f69.google.com [209.85.218.69]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-418-5U_ejwOdO2-9z-r7cKxKsA-1; Fri, 15 Jan 2021 04:27:42 -0500 X-MC-Unique: 5U_ejwOdO2-9z-r7cKxKsA-1 Received: by mail-ej1-f69.google.com with SMTP id jg11so2734442ejc.23 for ; Fri, 15 Jan 2021 01:27:42 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=MXIm+GnEt2voV5GSIMYLPLJ33ZaN2YcxTR1l6rQxLQo=; b=qzGhEEakUUenALXRTCHTNcj1ltEMNRjceJyshEZcqIps0S/4sUhHuOMQsFI8uI6jT+ OZK4dhGXu9L4/SytT5kswJ/1OL3tJ9+gxuAwadA+VefxTRs03pbT2zaP8b7fD2jaLAI+ UfRe0LY1hsXMdyldA3/gXnfxAazttHk4IXWEcXFudnUQ7ynO/hEQxiRD3g9rmxFFPb8l 24S71Ry1xiLgZHGF3MR9Di/szN58hulCA2TlRFkmt2SG4V0GmJ9MYvaCJcsShBu1dVLj j2rzVasQ8B833f8ujJ8HENkCNNMJVZ7albwq2EQJvuX8W8qkO+Ou9NmqQN+ggrkNWa6L Ztwg== X-Gm-Message-State: AOAM533qWcgTgh2kpFfglswcM41Ml66bhFGj58/0YLqpHN+6qAT0AaZf QU82BiN9znQb3hXrmNbBb/x2njFQyMLFbrDGtzZOeUBfbAKNuJ4eJW5L7Q4xfw6/hPz8YvFEgEt 7z5V+fe+2vxXo9A== X-Received: by 2002:a05:6402:307c:: with SMTP id bs28mr8976218edb.186.1610702861288; Fri, 15 Jan 2021 01:27:41 -0800 (PST) X-Google-Smtp-Source: ABdhPJxHcE3XmFQzTaFRWJdN1jqGmNQm6EZQNWLXjraHFt3cVjp61D46UcmLEfg7RcpxcPgitpTnmg== X-Received: by 2002:a05:6402:307c:: with SMTP id bs28mr8976190edb.186.1610702861074; Fri, 15 Jan 2021 01:27:41 -0800 (PST) Return-Path: Received: from [192.168.1.36] (13.red-83-57-169.dynamicip.rima-tde.net. [83.57.169.13]) by smtp.gmail.com with ESMTPSA id h16sm2990511eji.110.2021.01.15.01.27.39 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 15 Jan 2021 01:27:40 -0800 (PST) Subject: Re: [edk2-platforms PATCH 0/3] add OrderedCollectionLib class resolution To: Laszlo Ersek , 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 , Rangasai V Chaganty , Thomas Abraham , Wenyi Xie , Yi Qian , Zailiang Sun References: <20210113082843.9095-1-lersek@redhat.com> From: =?UTF-8?B?UGhpbGlwcGUgTWF0aGlldS1EYXVkw6k=?= Message-ID: Date: Fri, 15 Jan 2021 10:27:38 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.6.0 MIME-Version: 1.0 In-Reply-To: <20210113082843.9095-1-lersek@redhat.com> Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=philmd@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 1/13/21 9:28 AM, 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(+) Reviewed-by: Philippe Mathieu-Daude