From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by mx.groups.io with SMTP id smtpd.web11.10041.1651239663223825267 for ; Fri, 29 Apr 2022 06:41:03 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=Fx90VfW0; spf=pass (domain: kernel.org, ip: 139.178.84.217, mailfrom: ardb@kernel.org) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 7607962300 for ; Fri, 29 Apr 2022 13:41:02 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 5C82DC385B3 for ; Fri, 29 Apr 2022 13:41:01 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1651239661; bh=rzQZjz3ahmO/zN5+de1nu/Ng7MwtlOL9NcBpgOTqWsQ=; h=References:In-Reply-To:From:Date:Subject:To:Cc:From; b=Fx90VfW0lz4f7xfAklWHt5wLwJr8oCyiT4eZ0HyBgnMy5HPJxKb8DCwMW5RtkbpTp R2IldVFIsjcTgz7slFM95+bIt4/v8WkECcfQiAa4iBJjUDoQZmvw7ilt5aw2bv/ni6 1G659h/RVu2GIKoZvT62V+9fedHd/CFMW8aTo27xl7BZHIEp2bSv4qU3A3BKNenRIT mBeUENXk1EhLQEnKqV1dDKqr/iGkT2n2gLiBHojPQbFmmSQq1y0yNuguWFu27dkYD0 u4U0O7FN4TliBNtRIKIEvtWSwpQdYEdIhfnnU6nG5VkbySUiuSyfn2v/lXYYyIGsZT p+C3OmSXkqY+Q== Received: by mail-oa1-f45.google.com with SMTP id 586e51a60fabf-d39f741ba0so8127008fac.13 for ; Fri, 29 Apr 2022 06:41:01 -0700 (PDT) X-Gm-Message-State: AOAM532lVwxiirMmXMxxIrncXthnqNRJM/cHDf/DcTcuHstoByclcgSL jCVN4phXTBtUF+8CAOcgeRR4Fyvf0mmFVjLiFlE= X-Google-Smtp-Source: ABdhPJwETmmm/OzCJtruiFHKT9/H38IMM+/YNWSV+1E2TiJxY1fClP4WBcOkLIeMQXn15Ajipb3PQpIIwDCQqZdmGOg= X-Received: by 2002:a05:6870:eaa5:b0:da:b3f:2b45 with SMTP id s37-20020a056870eaa500b000da0b3f2b45mr1355163oap.228.1651239660472; Fri, 29 Apr 2022 06:41:00 -0700 (PDT) MIME-Version: 1.0 References: <20220426021834.1392-1-mikuback@linux.microsoft.com> In-Reply-To: <20220426021834.1392-1-mikuback@linux.microsoft.com> From: "Ard Biesheuvel" Date: Fri, 29 Apr 2022 15:40:48 +0200 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [edk2-platforms][PATCH v2 00/11] Add VariableFlashInfoLib To: Michael Kubacki Cc: edk2-devel-groups-io , Abner Chang , Ard Biesheuvel , Chasel Chiu , Daniel Schaefer , Eric Dong , Gilbert Chen , Graeme Gregory , Ilias Apalodimas , Jeremy Linton , Leif Lindholm , Liming Gao , Masami Hiramatsu , Nate DeSimone , Radoslaw Biernacki , Sami Mujawar , Thomas Abraham Content-Type: text/plain; charset="UTF-8" On Tue, 26 Apr 2022 at 04:19, wrote: > > From: Michael Kubacki > > REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3479 > > Adds an instance for the library class VariableFlashInfoLib that > was recently introduced in MdeModulePkg. This change is made to > allow the new variable driver to build that has a dependency on > this library class and does not require any further platform > changes. > > Note that this series is for an upcoming change in MdeModulePkg > and requires that change to be present locally for testing. > > https://edk2.groups.io/g/devel/message/89296 > > V2 Changes: > - Add patch "VExpressPkg: Add VariableFlashInfoLib" > > Cc: Abner Chang > Cc: Ard Biesheuvel > Cc: Chasel Chiu > Cc: Daniel Schaefer > Cc: Eric Dong > Cc: Gilbert Chen > Cc: Graeme Gregory > Cc: Ilias Apalodimas > Cc: Jeremy Linton > Cc: Leif Lindholm > Cc: Liming Gao > Cc: Masami Hiramatsu > Cc: Nate DeSimone > Cc: Radoslaw Biernacki > Cc: Sami Mujawar > Cc: Thomas Abraham > Signed-off-by: Michael Kubacki > > Michael Kubacki (11): > OverdriveBoard: Add VariableFlashInfoLib > BeagleBoardPkg: Add VariableFlashInfoLib > MinPlatformPkg: Add VariableFlashInfoLib > CelloBoard: Add VariableFlashInfoLib > SbsaQemu: Add VariableFlashInfoLib > RaspberryPi: Add VariableFlashInfoLib > U5SeriesPkg: Add VariableFlashInfoLib > Socionext: Add VariableFlashInfoLib > Softiron: Add VariableFlashInfoLib > PlatformStandaloneMmPkg: Add VariableFlashInfoLib > VExpressPkg: Add VariableFlashInfoLib > For the series Acked-by: Ard Biesheuvel *However* can we please fix this in the common frameworks? None of this would be necessary if we could define a default resolution when declaring a library class. That way, this problem would be fixed for everybody not just the platforms that are in the open source repo. > Platform/AMD/OverdriveBoard/OverdriveBoard.dsc | 1 + > Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc | 1 + > Platform/BeagleBoard/BeagleBoardPkg/BeagleBoardPkg.dsc | 1 + > Platform/Intel/MinPlatformPkg/Include/Dsc/CoreCommonLib.dsc | 1 + > Platform/LeMaker/CelloBoard/CelloBoard.dsc | 1 + > Platform/Qemu/SbsaQemu/SbsaQemu.dsc | 1 + > Platform/RaspberryPi/RPi3/RPi3.dsc | 1 + > Platform/RaspberryPi/RPi4/RPi4.dsc | 1 + > Platform/SiFive/U5SeriesPkg/FreedomU500VC707Board/U500.dsc | 1 + > Platform/SiFive/U5SeriesPkg/FreedomU540HiFiveUnleashedBoard/U540.dsc | 1 + > Platform/Socionext/SynQuacerEvalBoard/SynQuacerEvalBoard.dsc | 1 + > Platform/SoftIron/Overdrive1000Board/Overdrive1000Board.dsc | 1 + > Platform/StandaloneMm/PlatformStandaloneMmPkg/PlatformStandaloneMmRpmb.dsc | 1 + > 13 files changed, 13 insertions(+) > > -- > 2.28.0.windows.1 >