From: "Zhiguang Liu" <zhiguang.liu@intel.com>
To: "devel@edk2.groups.io" <devel@edk2.groups.io>,
"gaoliming@byosoft.com.cn" <gaoliming@byosoft.com.cn>,
"Dong, Eric" <eric.dong@intel.com>,
"Chiu, Chasel" <chasel.chiu@intel.com>,
"Desimone, Nathaniel L" <nathaniel.l.desimone@intel.com>
Cc: "Tian, Hot" <hot.tian@intel.com>
Subject: Re: [edk2-devel] 回复: [PATCH] Platform\Intel\MinPlatformPkg: Add Variable Policy library
Date: Wed, 25 Nov 2020 08:10:08 +0000 [thread overview]
Message-ID: <CY4PR11MB168720E0A3364B427A6525E990FA0@CY4PR11MB1687.namprd11.prod.outlook.com> (raw)
In-Reply-To: <007401d6c2ce$680d1e30$38275a90$@byosoft.com.cn>
Hi Maintainers,
Can you help push this patch? This is a little urgent.
It has already got R-B from Chasel and Liming
Thanks
Zhiguang
> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of gaoliming
> Sent: Wednesday, November 25, 2020 9:58 AM
> To: Liu, Zhiguang <zhiguang.liu@intel.com>; devel@edk2.groups.io
> Cc: Chiu, Chasel <chasel.chiu@intel.com>; Desimone, Nathaniel L
> <nathaniel.l.desimone@intel.com>; Dong, Eric <eric.dong@intel.com>
> Subject: [edk2-devel] 回复: [PATCH] Platform\Intel\MinPlatformPkg: Add
> Variable Policy library
>
> Reviewed-by: Liming Gao <gaoliming@byosoft.com.cn>
>
> > -----邮件原件-----
> > 发件人: Zhiguang Liu <zhiguang.liu@intel.com>
> > 发送时间: 2020年11月24日 15:35
> > 收件人: devel@edk2.groups.io
> > 抄送: Chasel Chiu <chasel.chiu@intel.com>; Nate DeSimone
> > <nathaniel.l.desimone@intel.com>; Liming Gao
> > <gaoliming@byosoft.com.cn>; Eric Dong <eric.dong@intel.com>
> > 主题: [PATCH] Platform\Intel\MinPlatformPkg: Add Variable Policy library
> >
> > Makes the changes because of the new added library instances:
> > VariablePolicyLib
> > and VariablePolicyHelperLib
> >
> > Cc: Chasel Chiu <chasel.chiu@intel.com>
> > Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> > Cc: Liming Gao <gaoliming@byosoft.com.cn>
> > Cc: Eric Dong <eric.dong@intel.com>
> >
> > Signed-off-by: Zhiguang Liu <zhiguang.liu@intel.com>
> > ---
> > Platform/Intel/MinPlatformPkg/Include/Dsc/CoreCommonLib.dsc | 2 ++
> > Platform/Intel/MinPlatformPkg/Include/Dsc/CoreDxeInclude.dsc | 1 +
> > Platform/Intel/MinPlatformPkg/Include/Dsc/CoreDxeLib.dsc | 1 +
> > 3 files changed, 4 insertions(+)
> >
> > diff --git
> > a/Platform/Intel/MinPlatformPkg/Include/Dsc/CoreCommonLib.dsc
> > b/Platform/Intel/MinPlatformPkg/Include/Dsc/CoreCommonLib.dsc
> > index 23ab53dbfe..46bb2cecae 100644
> > --- a/Platform/Intel/MinPlatformPkg/Include/Dsc/CoreCommonLib.dsc
> > +++ b/Platform/Intel/MinPlatformPkg/Include/Dsc/CoreCommonLib.dsc
> > @@ -160,3 +160,5 @@
> > SmmMemLib|MdePkg/Library/SmmMemLib/SmmMemLib.inf
> >
> >
> >
> > SmbusLib|MdePkg/Library/BaseSmbusLibNull/BaseSmbusLibNull.inf
> >
> > +
> > VariablePolicyLib|MdeModulePkg/Library/VariablePolicyLib/VariablePolic
> > VariablePolicyLib|yLib
> > .inf
> >
> > +
> > VariablePolicyHelperLib|MdeModulePkg/Library/VariablePolicyHelperLib/V
> > VariablePolicyHelperLib|ari
> > ablePolicyHelperLib.inf
> > \ No newline at end of file
> > diff --git
> > a/Platform/Intel/MinPlatformPkg/Include/Dsc/CoreDxeInclude.dsc
> > b/Platform/Intel/MinPlatformPkg/Include/Dsc/CoreDxeInclude.dsc
> > index f0e578f8cc..d7ed8f98cd 100644
> > --- a/Platform/Intel/MinPlatformPkg/Include/Dsc/CoreDxeInclude.dsc
> > +++ b/Platform/Intel/MinPlatformPkg/Include/Dsc/CoreDxeInclude.dsc
> > @@ -36,6 +36,7 @@
> > <LibraryClasses>
> >
> > NULL|MdeModulePkg/Library/VarCheckUefiLib/VarCheckUefiLib.inf
> >
> > NULL|MdeModulePkg/Library/VarCheckHiiLib/VarCheckHiiLib.inf
> >
> > +
> > NULL|MdeModulePkg/Library/VarCheckPolicyLib/VarCheckPolicyLib.inf
> >
> > }
> >
> > !else
> >
> > MdeModulePkg/Universal/Variable/RuntimeDxe/VariableRuntimeDxe.inf
> > {
> >
> > diff --git a/Platform/Intel/MinPlatformPkg/Include/Dsc/CoreDxeLib.dsc
> > b/Platform/Intel/MinPlatformPkg/Include/Dsc/CoreDxeLib.dsc
> > index 026ddf4cd7..fa9098d525 100644
> > --- a/Platform/Intel/MinPlatformPkg/Include/Dsc/CoreDxeLib.dsc
> > +++ b/Platform/Intel/MinPlatformPkg/Include/Dsc/CoreDxeLib.dsc
> > @@ -115,6 +115,7 @@
> > TimerLib|PcAtChipsetPkg/Library/AcpiTimerLib/DxeAcpiTimerLib.inf
> >
> > !endif
> >
> > BaseCryptLib|CryptoPkg/Library/BaseCryptLib/RuntimeCryptLib.inf
> >
> > +
> > VariablePolicyLib|MdeModulePkg/Library/VariablePolicyLib/VariablePolic
> > VariablePolicyLib|yLib
> > RuntimeDxe.inf
> >
> >
> >
> > [LibraryClasses.common.UEFI_DRIVER]
> >
> >
> >
> Tpm2DeviceLib|SecurityPkg/Library/Tpm2DeviceLibTcg2/Tpm2DeviceLibTcg2
> > .inf
> >
> > --
> > 2.25.1.windows.1
>
>
>
>
>
>
>
next prev parent reply other threads:[~2020-11-25 8:10 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-11-24 7:34 [PATCH] Platform\Intel\MinPlatformPkg: Add Variable Policy library Zhiguang Liu
2020-11-24 8:42 ` Chiu, Chasel
2020-11-25 1:58 ` 回复: " gaoliming
2020-11-25 8:10 ` Zhiguang Liu [this message]
2020-11-25 8:18 ` [edk2-devel] " Dong, Eric
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=CY4PR11MB168720E0A3364B427A6525E990FA0@CY4PR11MB1687.namprd11.prod.outlook.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