From: "Ard Biesheuvel" <ardb@kernel.org>
To: devel@edk2.groups.io, dandan.bi@intel.com,
Laszlo Ersek <lersek@redhat.com>, Andrew Fish <afish@apple.com>
Cc: Leif Lindholm <leif@nuviainc.com>,
Michael D Kinney <michael.d.kinney@intel.com>,
Liming Gao <gaoliming@byosoft.com.cn>
Subject: Re: [edk2-devel] [edk2-platforms] [patch 00/35] Consume RegisterFilterLibNull instance
Date: Tue, 16 Mar 2021 16:00:46 +0100 [thread overview]
Message-ID: <CAMj1kXGgP10-Af70j6wOVYzfthqm9xduuvjV5p8oGvcnZwdATg@mail.gmail.com> (raw)
In-Reply-To: <20210316145428.35616-1-dandan.bi@intel.com>
On Tue, 16 Mar 2021 at 15:56, Dandan Bi <dandan.bi@intel.com> wrote:
>
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
> RFC: https://edk2.groups.io/g/devel/message/72530
>
>
> Add RegisterFilterLibNull in dsc files in edk2-platforms repo,
> which will be consumed by IoLib and BaseLib.
>
> This is the following update in edk2-platforms repo for the change in edk2,
> which will add RegisterFilterLib dependency for IoLib and BaseLib to filter/trace
> port IO/MMIO/MSR access.
> https://edk2.groups.io/g/devel/message/72754
>
> Cc: Leif Lindholm <leif@nuviainc.com>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Liming Gao <gaoliming@byosoft.com.cn>
>
It is a bit disappointing that we have to update every platform in
existence again to apply a change to a core module.
Is there really not a better way to provide a 'default' resolution for
a library class? Maybe a change to the .DEC format, so that the file
which defines the library class can provide a resolution that is used
if none is provided by the .DSC file?
> Dandan Bi (35):
> Drivers/ASIX: Consume RegisterFilterLibNull instance
> Drivers/DisplayLink: Consume RegisterFilterLibNull instance
> Drivers/OptionRomPkg: Consume RegisterFilterLibNull instance
> Features/Debugging: Consume RegisterFilterLibNull instance
> Features/Network: Consume RegisterFilterLibNull instance
> Features/OutOfBandManagement: Consume RegisterFilterLibNull instance
> Features/PowerManagement: Consume RegisterFilterLibNull instance
> Features/SystemInformation: Consume RegisterFilterLibNull instance
> Features/UserInterface: Consume RegisterFilterLibNull instance
> Platform/AMD: Consume RegisterFilterLibNull instance
> Platform/ARM: Consume RegisterFilterLibNull instance
> Platform/BeagleBoard: Consume RegisterFilterLibNull instance
> Platform/BoardModulePkg: Consume RegisterFilterLibNull instance
> Platform/MinPlatformPkg: Consume RegisterFilterLibNull instance
> Platform/QuarkPlatformPkg: Consume RegisterFilterLibNull instance
> Platform/Vlv2TbltDevicePkg: Consume RegisterFilterLibNull instance
> Platform/LeMaker: Consume RegisterFilterLibNull instance
> Platform/Qemu: Consume RegisterFilterLibNull instance
> Platform/RaspberryPi: Consume RegisterFilterLibNull instance
> Platform/RISC-V: Consume RegisterFilterLibNull instance
> Platform/SiFive: Consume RegisterFilterLibNull instance
> Platform/Socionext: Consume RegisterFilterLibNull instance
> Platform/SoftIron: Consume RegisterFilterLibNull instance
> Silicon/Hisilicon: Consume RegisterFilterLibNull instance
> Silicon/CoffeelakeSiliconPkg: Consume RegisterFilterLibNull instance
> Silicon/IntelSiliconPkg: Consume RegisterFilterLibNull instance
> Silicon/KabylakeSiliconPkg: Consume RegisterFilterLibNull instance
> Silicon/QuarkSocPkg: Consume RegisterFilterLibNull instance
> Silicon/TigerlakeSiliconPkg: Consume RegisterFilterLibNull instance
> Silicon/Marvell: Consume RegisterFilterLibNull instance
> Silicon/NXP: Consume RegisterFilterLibNull instance
> Silicon/Openmoko: Consume RegisterFilterLibNull instance
> Silicon/RISC_V: Consume RegisterFilterLibNull instance
> Silicon/Synopsys/DesignWare: Consume RegisterFilterLibNull instance
> Silicon/TexasInstruments: Consume RegisterFilterLibNull instance
>
> Drivers/ASIX/Asix.dsc | 1 +
> Drivers/DisplayLink/DisplayLinkPkg/DisplayLinkPkg.dsc | 1 +
> Drivers/OptionRomPkg/OptionRomPkg.dsc | 3 ++-
> .../Debugging/AcpiDebugFeaturePkg/Include/AcpiDebugFeature.dsc | 3 ++-
> .../Debugging/BeepDebugFeaturePkg/Include/BeepDebugFeature.dsc | 3 ++-
> .../PostCodeDebugFeaturePkg/Include/PostCodeDebugFeature.dsc | 3 ++-
> .../Debugging/Usb3DebugFeaturePkg/Include/Usb3DebugFeature.dsc | 3 ++-
> .../Intel/Network/NetworkFeaturePkg/Include/NetworkFeature.dsc | 3 ++-
> .../OutOfBandManagement/IpmiFeaturePkg/Include/IpmiFeature.dsc | 3 ++-
> .../OutOfBandManagement/SpcrFeaturePkg/Include/SpcrFeature.dsc | 3 ++-
> .../Intel/PowerManagement/S3FeaturePkg/Include/S3Feature.dsc | 3 ++-
> .../SmbiosFeaturePkg/Include/SmbiosFeature.dsc | 3 ++-
> .../Intel/UserInterface/LogoFeaturePkg/Include/LogoFeature.dsc | 3 ++-
> .../UserAuthFeaturePkg/Include/UserAuthFeature.dsc | 3 ++-
> .../Include/VirtualKeyboardFeature.dsc | 3 ++-
> Platform/AMD/OverdriveBoard/OverdriveBoard.dsc | 1 +
> Platform/ARM/SgiPkg/PlatformStandaloneMm.dsc | 1 +
> Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc | 1 +
> Platform/BeagleBoard/BeagleBoardPkg/BeagleBoardPkg.dsc | 3 ++-
> Platform/Intel/BoardModulePkg/BoardModulePkg.dsc | 3 ++-
> Platform/Intel/MinPlatformPkg/Include/Dsc/CoreCommonLib.dsc | 3 ++-
> Platform/Intel/QuarkPlatformPkg/Quark.dsc | 1 +
> Platform/Intel/QuarkPlatformPkg/QuarkMin.dsc | 1 +
> Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgIA32.dsc | 3 ++-
> Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgX64.dsc | 3 ++-
> Platform/LeMaker/CelloBoard/CelloBoard.dsc | 1 +
> Platform/Qemu/SbsaQemu/SbsaQemu.dsc | 1 +
> Platform/RISC-V/PlatformPkg/RiscVPlatformPkg.dsc | 1 +
> Platform/RaspberryPi/RPi3/RPi3.dsc | 3 ++-
> Platform/RaspberryPi/RPi4/RPi4.dsc | 3 ++-
> Platform/SiFive/U5SeriesPkg/FreedomU500VC707Board/U500.dsc | 1 +
> .../U5SeriesPkg/FreedomU540HiFiveUnleashedBoard/U540.dsc | 1 +
> Platform/Socionext/DeveloperBox/DeveloperBox.dsc.inc | 1 +
> Platform/Socionext/SynQuacerEvalBoard/SynQuacerEvalBoard.dsc | 1 +
> Platform/SoftIron/Overdrive1000Board/Overdrive1000Board.dsc | 1 +
> Silicon/Hisilicon/Hisilicon.dsc.inc | 1 +
> Silicon/Intel/CoffeelakeSiliconPkg/CoffeelakeSiliconPkg.dsc | 1 +
> Silicon/Intel/IntelSiliconPkg/IntelSiliconPkg.dsc | 3 ++-
> Silicon/Intel/KabylakeSiliconPkg/KabylakeSiliconPkg.dsc | 3 ++-
> Silicon/Intel/QuarkSocPkg/QuarkSocPkg.dsc | 3 ++-
> Silicon/Intel/TigerlakeSiliconPkg/TigerlakeSiliconPkg.dsc | 1 +
> Silicon/Marvell/Armada7k8k/Armada7k8k.dsc.inc | 1 +
> Silicon/NXP/NxpQoriqLs.dsc.inc | 1 +
> Silicon/Openmoko/Openmoko.dsc | 1 +
> Silicon/RISC-V/ProcessorPkg/RiscVProcessorPkg.dsc | 1 +
> Silicon/Synopsys/DesignWare/DesignWare.dsc | 1 +
> Silicon/TexasInstruments/Omap35xxPkg/Omap35xxPkg.dsc | 1 +
> 47 files changed, 70 insertions(+), 23 deletions(-)
>
> --
> 2.18.0.windows.1
>
>
>
>
>
>
next prev parent reply other threads:[~2021-03-16 15:00 UTC|newest]
Thread overview: 50+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-03-16 14:53 [edk2-platforms] [patch 00/35] Consume RegisterFilterLibNull instance Dandan Bi
2021-03-16 14:53 ` [edk2-platforms] [patch 01/35] Drivers/ASIX: " Dandan Bi
2021-03-16 14:53 ` [edk2-platforms] [patch 02/35] Drivers/DisplayLink: " Dandan Bi
2021-03-16 14:53 ` [edk2-platforms] [patch 03/35] Drivers/OptionRomPkg: " Dandan Bi
2021-03-16 14:53 ` [edk2-platforms] [patch 04/35] Features/Debugging: " Dandan Bi
2021-03-16 14:53 ` [edk2-platforms] [patch 05/35] Features/Network: " Dandan Bi
2021-03-16 14:53 ` [edk2-platforms] [patch 06/35] Features/OutOfBandManagement: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 07/35] Features/PowerManagement: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 08/35] Features/SystemInformation: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 09/35] Features/UserInterface: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 10/35] Platform/AMD: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 11/35] Platform/ARM: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 12/35] Platform/BeagleBoard: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 13/35] Platform/BoardModulePkg: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 14/35] Platform/MinPlatformPkg: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 15/35] Platform/QuarkPlatformPkg: " Dandan Bi
2021-03-16 15:04 ` Steele, Kelly
2021-03-16 14:54 ` [edk2-platforms] [patch 16/35] Platform/Vlv2TbltDevicePkg: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 17/35] Platform/LeMaker: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 18/35] Platform/Qemu: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 19/35] Platform/RaspberryPi: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 20/35] Platform/RISC-V: " Dandan Bi
2021-03-17 8:14 ` Abner Chang
2021-03-16 14:54 ` [edk2-platforms] [patch 21/35] Platform/SiFive: " Dandan Bi
2021-03-17 8:14 ` Abner Chang
2021-03-16 14:54 ` [edk2-platforms] [patch 22/35] Platform/Socionext: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 23/35] Platform/SoftIron: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 24/35] Silicon/Hisilicon: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 25/35] Silicon/CoffeelakeSiliconPkg: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 26/35] Silicon/IntelSiliconPkg: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 27/35] Silicon/KabylakeSiliconPkg: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 28/35] Silicon/QuarkSocPkg: " Dandan Bi
2021-03-16 15:05 ` Steele, Kelly
2021-03-16 14:54 ` [edk2-platforms] [patch 29/35] Silicon/TigerlakeSiliconPkg: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 30/35] Silicon/Marvell: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 31/35] Silicon/NXP: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 32/35] Silicon/Openmoko: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 33/35] Silicon/RISC_V: " Dandan Bi
2021-03-17 8:14 ` Abner Chang
2021-03-16 14:54 ` [edk2-platforms] [patch 34/35] Silicon/Synopsys/DesignWare: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 35/35] Silicon/TexasInstruments: " Dandan Bi
2021-03-16 15:00 ` Ard Biesheuvel [this message]
2021-03-16 16:23 ` [edk2-devel] [edk2-platforms] [patch 00/35] " Laszlo Ersek
2021-03-17 3:04 ` 回复: " gaoliming
2021-03-17 15:05 ` Dandan Bi
2021-03-17 17:08 ` Laszlo Ersek
2021-03-18 4:43 ` 回复: " gaoliming
2021-03-18 13:25 ` Laszlo Ersek
2021-03-18 13:42 ` Ni, Ray
2021-03-19 1:20 ` 回复: " gaoliming
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CAMj1kXGgP10-Af70j6wOVYzfthqm9xduuvjV5p8oGvcnZwdATg@mail.gmail.com \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox