Re: [PATCH 4/7] OvmfPkg/QemuFwCfgDxeLib: allow UEFI_DRIVER modules

["Philippe Mathieu-Daudé" <philmd@redhat.com>]

On 4/24/20 9:53 AM, Laszlo Ersek wrote:
> We don't distribute UEFI_DRIVER modules stand-alone that were built as
> part of an OVMF platform. OVMF's UEFI_DRIVERs are allowed to inherit
> platform dependencies.
> 
> By enabling UEFI_DRIVERs to consume QemuFwCfgDxeLib, we can hook
> fw_cfg-based NULL class libraries into UEFI drivers, e.g. in order to set
> dynamic PCDs.
> 
> Cc: Ard Biesheuvel <ard.biesheuvel@arm.com>
> Cc: Jordan Justen <jordan.l.justen@intel.com>
> Cc: Per Sundstrom <per_sundstrom@yahoo.com>
> Cc: Philippe Mathieu-Daudé <philmd@redhat.com>
> Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2681
> Signed-off-by: Laszlo Ersek <lersek@redhat.com>
> ---
>   OvmfPkg/Library/QemuFwCfgLib/QemuFwCfgDxeLib.inf | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/OvmfPkg/Library/QemuFwCfgLib/QemuFwCfgDxeLib.inf b/OvmfPkg/Library/QemuFwCfgLib/QemuFwCfgDxeLib.inf
> index c331c5e7220a..48899ff1236a 100644
> --- a/OvmfPkg/Library/QemuFwCfgLib/QemuFwCfgDxeLib.inf
> +++ b/OvmfPkg/Library/QemuFwCfgLib/QemuFwCfgDxeLib.inf
> @@ -16,7 +16,7 @@ [Defines]
>     FILE_GUID                      = 80474090-55e7-4c28-b25c-9f236ba41f28
>     MODULE_TYPE                    = BASE
>     VERSION_STRING                 = 1.0
> -  LIBRARY_CLASS                  = QemuFwCfgLib|DXE_DRIVER DXE_RUNTIME_DRIVER DXE_SMM_DRIVER
> +  LIBRARY_CLASS                  = QemuFwCfgLib|DXE_DRIVER DXE_RUNTIME_DRIVER DXE_SMM_DRIVER UEFI_DRIVER
>   
>     CONSTRUCTOR                    = QemuFwCfgInitialize
>   
> 

Reviewed-by: Philippe Mathieu-Daude <philmd@redhat.com>