[edk2-devel] [edk2-platforms PATCH v4 11/12] RISC-V Platforms: add SecureBootVariableLib class resolution

[Daniel Schaefer]

Reviewed-by: Daniel Schaefer <daniel.schaefer at hpe.com>
________________________________
From: devel at edk2.groups.io <devel at edk2.groups.io> on behalf of Abner Chang <abner.chang at hpe.com>
Sent: Thursday, July 1, 2021 11:50
To: devel at edk2.groups.io <devel at edk2.groups.io>; gjb at semihalf.com <gjb at semihalf.com>
Cc: leif at nuviainc.com <leif at nuviainc.com>; ardb+tianocore at kernel.org <ardb+tianocore at kernel.org>; Samer.El-Haj-Mahmoud at arm.com <Samer.El-Haj-Mahmoud at arm.com>; sunny.Wang at arm.com <sunny.Wang at arm.com>; mw at semihalf.com <mw at semihalf.com>; upstream at semihalf.com <upstream at semihalf.com>; jiewen.yao at intel.com <jiewen.yao at intel.com>; jian.j.wang at intel.com <jian.j.wang at intel.com>; min.m.xu at intel.com <min.m.xu at intel.com>; lersek at redhat.com <lersek at redhat.com>; sami.mujawar at arm.com <sami.mujawar at arm.com>; afish at apple.com <afish at apple.com>; ray.ni at intel.com <ray.ni at intel.com>; jordan.l.justen at intel.com <jordan.l.justen at intel.com>; rebecca at bsdio.com <rebecca at bsdio.com>; grehan at freebsd.org <grehan at freebsd.org>; thomas.abraham at arm.com <thomas.abraham at arm.com>; chasel.chiu at intel.com <chasel.chiu at intel.com>; nathaniel.l.desimone at intel.com <nathaniel.l.desimone at intel.com>; gaoliming at byosoft.com.cn <gaoliming at byosoft.com.cn>; eric.dong at intel.com <eric.dong at intel.com>; michael.d.kinney at intel.com <michael.d.kinney at intel.com>; zailiang.sun at intel.com <zailiang.sun at intel.com>; yi.qian at intel.com <yi.qian at intel.com>; graeme at nuviainc.com <graeme at nuviainc.com>; rad at semihalf.com <rad at semihalf.com>; pete at akeo.ie <pete at akeo.ie>
Subject: Re: [edk2-devel] [edk2-platforms PATCH v4 11/12] RISC-V Platforms: add SecureBootVariableLib class resolution

Reviewed-by: Abner Chang <abner.chang at hpe.com>

> -----Original Message-----
> From: devel at edk2.groups.io [mailto:devel at edk2.groups.io] On Behalf Of
> Grzegorz Bernacki
> Sent: Wednesday, June 30, 2021 8:34 PM
> To: devel at edk2.groups.io
> Cc: leif at nuviainc.com; ardb+tianocore at kernel.org; Samer.El-Haj-
> Mahmoud at arm.com; sunny.Wang at arm.com; mw at semihalf.com;
> upstream at semihalf.com; jiewen.yao at intel.com; jian.j.wang at intel.com;
> min.m.xu at intel.com; lersek at redhat.com; sami.mujawar at arm.com;
> afish at apple.com; ray.ni at intel.com; jordan.l.justen at intel.com;
> rebecca at bsdio.com; grehan at freebsd.org; thomas.abraham at arm.com;
> chasel.chiu at intel.com; nathaniel.l.desimone at intel.com;
> gaoliming at byosoft.com.cn; eric.dong at intel.com;
> michael.d.kinney at intel.com; zailiang.sun at intel.com; yi.qian at intel.com;
> graeme at nuviainc.com; rad at semihalf.com; pete at akeo.ie; Grzegorz
> Bernacki <gjb at semihalf.com>
> Subject: [edk2-devel] [edk2-platforms PATCH v4 11/12] RISC-V Platforms:
> add SecureBootVariableLib class resolution
>
> The edk2 patch
>   SecurityPkg: Create library for setting Secure Boot variables.
>
> removes generic functions from SecureBootConfigDxe and places
> them into SecureBootVariableLib. This patch adds SecureBootVariableLib
> mapping for each RICS-V platform which uses SecureBootConfigDxe.
>
> Signed-off-by: Grzegorz Bernacki <gjb at semihalf.com>
> ---
>  Platform/SiFive/U5SeriesPkg/FreedomU500VC707Board/U500.dsc           | 1 +
>
> Platform/SiFive/U5SeriesPkg/FreedomU540HiFiveUnleashedBoard/U540.dsc
> | 1 +
>  2 files changed, 2 insertions(+)
>
> diff --git a/Platform/SiFive/U5SeriesPkg/FreedomU500VC707Board/U500.dsc
> b/Platform/SiFive/U5SeriesPkg/FreedomU500VC707Board/U500.dsc
> index b91823ceeb..fc5ba2a07f 100644
> --- a/Platform/SiFive/U5SeriesPkg/FreedomU500VC707Board/U500.dsc
> +++ b/Platform/SiFive/U5SeriesPkg/FreedomU500VC707Board/U500.dsc
> @@ -122,6 +122,7 @@
>    OpensslLib|CryptoPkg/Library/OpensslLib/OpensslLib.inf
>
> TpmMeasurementLib|SecurityPkg/Library/DxeTpmMeasurementLib/DxeTp
> mMeasurementLib.inf
>    AuthVariableLib|SecurityPkg/Library/AuthVariableLib/AuthVariableLib.inf
> +
> SecureBootVariableLib|SecurityPkg/Library/SecureBootVariableLib/SecureB
> ootVariableLib.inf
>  !else
>
> TpmMeasurementLib|MdeModulePkg/Library/TpmMeasurementLibNull/Tp
> mMeasurementLibNull.inf
>
> AuthVariableLib|MdeModulePkg/Library/AuthVariableLibNull/AuthVariableL
> ibNull.inf
> diff --git
> a/Platform/SiFive/U5SeriesPkg/FreedomU540HiFiveUnleashedBoard/U540.d
> sc
> b/Platform/SiFive/U5SeriesPkg/FreedomU540HiFiveUnleashedBoard/U540.
> dsc
> index 0eafe29880..71add8ff9a 100644
> ---
> a/Platform/SiFive/U5SeriesPkg/FreedomU540HiFiveUnleashedBoard/U540.d
> sc
> +++
> b/Platform/SiFive/U5SeriesPkg/FreedomU540HiFiveUnleashedBoard/U540.
> dsc
> @@ -122,6 +122,7 @@
>    OpensslLib|CryptoPkg/Library/OpensslLib/OpensslLib.inf
>
> TpmMeasurementLib|SecurityPkg/Library/DxeTpmMeasurementLib/DxeTp
> mMeasurementLib.inf
>    AuthVariableLib|SecurityPkg/Library/AuthVariableLib/AuthVariableLib.inf
> +
> SecureBootVariableLib|SecurityPkg/Library/SecureBootVariableLib/SecureB
> ootVariableLib.inf
>  !else
>
> TpmMeasurementLib|MdeModulePkg/Library/TpmMeasurementLibNull/Tp
> mMeasurementLibNull.inf
>
> AuthVariableLib|MdeModulePkg/Library/AuthVariableLibNull/AuthVariableL
> ibNull.inf
> --
> 2.25.1
>
>
>
>
>








-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#77388): https://edk2.groups.io/g/devel/message/77388
Mute This Topic: https://groups.io/mt/83891041/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/edk2-devel-archive/attachments/20210701/def19ced/attachment.htm>