* [PATCH 1/4] SecurityPkg: enable TPM components to build for ARM and AARCH64 @ 2018-11-13 22:12 Cohen, Eugene 2018-11-13 22:21 ` Yao, Jiewen 0 siblings, 1 reply; 5+ messages in thread From: Cohen, Eugene @ 2018-11-13 22:12 UTC (permalink / raw) To: edk2-devel@lists.01.org, Yao, Jiewen, Zhang, Chao B Cc: Bin, Sung-Uk (빈성욱) SecurityPkg: enable TPM components to build for ARM and AARCH64 Contributed-under: TianoCore Contribution Agreement 1.1 Cc: Chao Zhang <chao.b.zhang@intel.com> Cc: Jiewen Yao <jiewen.yao@intel.com> Signed-off-by: Eugene Cohen <eugene@hp.com> --- SecurityPkg/SecurityPkg.dsc | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/SecurityPkg/SecurityPkg.dsc b/SecurityPkg/SecurityPkg.dsc index 68a2953..6fb9ad2 100644 --- a/SecurityPkg/SecurityPkg.dsc +++ b/SecurityPkg/SecurityPkg.dsc @@ -53,6 +53,7 @@ IntrinsicLib|CryptoPkg/Library/IntrinsicLib/IntrinsicLib.inf OpensslLib|CryptoPkg/Library/OpensslLib/OpensslLib.inf IoLib|MdePkg/Library/BaseIoLibIntrinsic/BaseIoLibIntrinsic.inf + TpmIoLib|SecurityPkg/Library/TpmIoLibMmio/TpmIoLibMmio.inf TpmCommLib|SecurityPkg/Library/TpmCommLib/TpmCommLib.inf PlatformSecureLib|SecurityPkg/Library/PlatformSecureLibNull/PlatformSecureLibNull.inf TcgPhysicalPresenceLib|SecurityPkg/Library/DxeTcgPhysicalPresenceLib/DxeTcgPhysicalPresenceLib.inf @@ -199,7 +200,7 @@ [Components.IA32, Components.X64, Components.ARM, Components.AARCH64] SecurityPkg/Library/AuthVariableLib/AuthVariableLib.inf -[Components.IA32, Components.X64] +[Components.IA32, Components.X64 Components.ARM, Components.AARCH64] # SecurityPkg/UserIdentification/PwdCredentialProviderDxe/PwdCredentialProviderDxe.inf # SecurityPkg/UserIdentification/UsbCredentialProviderDxe/UsbCredentialProviderDxe.inf SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfigDxe.inf -- 2.7.4 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH 1/4] SecurityPkg: enable TPM components to build for ARM and AARCH64 2018-11-13 22:12 [PATCH 1/4] SecurityPkg: enable TPM components to build for ARM and AARCH64 Cohen, Eugene @ 2018-11-13 22:21 ` Yao, Jiewen 2018-11-13 22:49 ` Cohen, Eugene 2018-11-13 22:59 ` Kinney, Michael D 0 siblings, 2 replies; 5+ messages in thread From: Yao, Jiewen @ 2018-11-13 22:21 UTC (permalink / raw) To: Cohen, Eugene, edk2-devel@lists.01.org, Zhang, Chao B; +Cc: Bin, Sung-Uk (???) HI Eugene Thanks to enable SPI TPM chip. In general, I am OK on this patch series. There are some additional work here. 1) Please split this patch to 2. The TpmIoLib is not present in at this point of time. We should add it after TpmIoLib instance is added. 2) Since this patch series adds the dependency of TpmIoLib, please update *all* impacted platform in EDKII repo and EDKII platform repo. We need make sure this patch series does not break any existing platform build. 3) I hope, (if possible) you can provide one *real example* on how to add SPI instance, to demonstrate the usage and value of this one more layer abstraction. Thank you Yao Jiewen > -----Original Message----- > From: Cohen, Eugene [mailto:eugene@hp.com] > Sent: Wednesday, November 14, 2018 6:13 AM > To: edk2-devel@lists.01.org; Yao, Jiewen <jiewen.yao@intel.com>; Zhang, > Chao B <chao.b.zhang@intel.com> > Cc: Bin, Sung-Uk (빈성욱) <sunguk-bin@hp.com> > Subject: [PATCH 1/4] SecurityPkg: enable TPM components to build for ARM > and AARCH64 > > SecurityPkg: enable TPM components to build for ARM and AARCH64 > > Contributed-under: TianoCore Contribution Agreement 1.1 > Cc: Chao Zhang <chao.b.zhang@intel.com> > Cc: Jiewen Yao <jiewen.yao@intel.com> > Signed-off-by: Eugene Cohen <eugene@hp.com> > --- > SecurityPkg/SecurityPkg.dsc | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/SecurityPkg/SecurityPkg.dsc b/SecurityPkg/SecurityPkg.dsc > index 68a2953..6fb9ad2 100644 > --- a/SecurityPkg/SecurityPkg.dsc > +++ b/SecurityPkg/SecurityPkg.dsc > @@ -53,6 +53,7 @@ > IntrinsicLib|CryptoPkg/Library/IntrinsicLib/IntrinsicLib.inf > OpensslLib|CryptoPkg/Library/OpensslLib/OpensslLib.inf > IoLib|MdePkg/Library/BaseIoLibIntrinsic/BaseIoLibIntrinsic.inf > + TpmIoLib|SecurityPkg/Library/TpmIoLibMmio/TpmIoLibMmio.inf > TpmCommLib|SecurityPkg/Library/TpmCommLib/TpmCommLib.inf > > PlatformSecureLib|SecurityPkg/Library/PlatformSecureLibNull/PlatformSecu > reLibNull.inf > > TcgPhysicalPresenceLib|SecurityPkg/Library/DxeTcgPhysicalPresenceLib/Dxe > TcgPhysicalPresenceLib.inf > @@ -199,7 +200,7 @@ > [Components.IA32, Components.X64, Components.ARM, > Components.AARCH64] > SecurityPkg/Library/AuthVariableLib/AuthVariableLib.inf > > -[Components.IA32, Components.X64] > +[Components.IA32, Components.X64 Components.ARM, > Components.AARCH64] > # > SecurityPkg/UserIdentification/PwdCredentialProviderDxe/PwdCredentialPr > oviderDxe.inf > # > SecurityPkg/UserIdentification/UsbCredentialProviderDxe/UsbCredentialPro > viderDxe.inf > > SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfi > gDxe.inf > -- > 2.7.4 ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 1/4] SecurityPkg: enable TPM components to build for ARM and AARCH64 2018-11-13 22:21 ` Yao, Jiewen @ 2018-11-13 22:49 ` Cohen, Eugene 2018-11-14 0:28 ` Yao, Jiewen 2018-11-13 22:59 ` Kinney, Michael D 1 sibling, 1 reply; 5+ messages in thread From: Cohen, Eugene @ 2018-11-13 22:49 UTC (permalink / raw) To: Yao, Jiewen, edk2-devel@lists.01.org, Zhang, Chao B Cc: Bin, Sung-Uk (빈성욱) Jiewen, 1 and 2 - okay, no problem, working on it. 3 - we are developing the SPI TPM support as we speak. Our SPI controller is proprietary so there's no value in trying to share that driver. Our internal SPI driver currently does not support the PI spec SPI_IO_PROTOCOL but are looking at adopting it - if so then we could contribute the TpmIoLibSpi instance which would connect the TPM stack to the SPI_IO_PROTOCOL. Is this what you wanted or is there something more specific you were looking for? Thanks, Eugene > -----Original Message----- > From: Yao, Jiewen <jiewen.yao@intel.com> > Sent: Tuesday, November 13, 2018 3:22 PM > To: Cohen, Eugene <eugene@hp.com>; edk2-devel@lists.01.org; Zhang, > Chao B <chao.b.zhang@intel.com> > Cc: Bin, Sung-Uk (빈성욱) <sunguk-bin@hp.com> > Subject: RE: [PATCH 1/4] SecurityPkg: enable TPM components to build for > ARM and AARCH64 > > HI Eugene > Thanks to enable SPI TPM chip. > In general, I am OK on this patch series. > > There are some additional work here. > 1) Please split this patch to 2. The TpmIoLib is not present in at this point of > time. We should add it after TpmIoLib instance is added. > > 2) Since this patch series adds the dependency of TpmIoLib, please update > *all* impacted platform in EDKII repo and EDKII platform repo. > We need make sure this patch series does not break any existing platform > build. > > 3) I hope, (if possible) you can provide one *real example* on how to add > SPI instance, to demonstrate the usage and value of this one more layer > abstraction. > > Thank you > Yao Jiewen > > > > -----Original Message----- > > From: Cohen, Eugene [mailto:eugene@hp.com] > > Sent: Wednesday, November 14, 2018 6:13 AM > > To: edk2-devel@lists.01.org; Yao, Jiewen <jiewen.yao@intel.com>; > > Zhang, Chao B <chao.b.zhang@intel.com> > > Cc: Bin, Sung-Uk (빈성욱) <sunguk-bin@hp.com> > > Subject: [PATCH 1/4] SecurityPkg: enable TPM components to build for > > ARM and AARCH64 > > > > SecurityPkg: enable TPM components to build for ARM and AARCH64 > > > > Contributed-under: TianoCore Contribution Agreement 1.1 > > Cc: Chao Zhang <chao.b.zhang@intel.com> > > Cc: Jiewen Yao <jiewen.yao@intel.com> > > Signed-off-by: Eugene Cohen <eugene@hp.com> > > --- > > SecurityPkg/SecurityPkg.dsc | 3 ++- > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > diff --git a/SecurityPkg/SecurityPkg.dsc b/SecurityPkg/SecurityPkg.dsc > > index 68a2953..6fb9ad2 100644 > > --- a/SecurityPkg/SecurityPkg.dsc > > +++ b/SecurityPkg/SecurityPkg.dsc > > @@ -53,6 +53,7 @@ > > IntrinsicLib|CryptoPkg/Library/IntrinsicLib/IntrinsicLib.inf > > OpensslLib|CryptoPkg/Library/OpensslLib/OpensslLib.inf > > IoLib|MdePkg/Library/BaseIoLibIntrinsic/BaseIoLibIntrinsic.inf > > + TpmIoLib|SecurityPkg/Library/TpmIoLibMmio/TpmIoLibMmio.inf > > TpmCommLib|SecurityPkg/Library/TpmCommLib/TpmCommLib.inf > > > > > PlatformSecureLib|SecurityPkg/Library/PlatformSecureLibNull/PlatformSe > > PlatformSecureLib|cu > > reLibNull.inf > > > > > TcgPhysicalPresenceLib|SecurityPkg/Library/DxeTcgPhysicalPresenceLib/D > > TcgPhysicalPresenceLib|xe > > TcgPhysicalPresenceLib.inf > > @@ -199,7 +200,7 @@ > > [Components.IA32, Components.X64, Components.ARM, > Components.AARCH64] > > SecurityPkg/Library/AuthVariableLib/AuthVariableLib.inf > > > > -[Components.IA32, Components.X64] > > +[Components.IA32, Components.X64 Components.ARM, > > Components.AARCH64] > > # > > > SecurityPkg/UserIdentification/PwdCredentialProviderDxe/PwdCredential > P > > r > > oviderDxe.inf > > # > > > SecurityPkg/UserIdentification/UsbCredentialProviderDxe/UsbCredentialP > > ro > > viderDxe.inf > > > > > SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootCon > fi > > gDxe.inf > > -- > > 2.7.4 ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 1/4] SecurityPkg: enable TPM components to build for ARM and AARCH64 2018-11-13 22:49 ` Cohen, Eugene @ 2018-11-14 0:28 ` Yao, Jiewen 0 siblings, 0 replies; 5+ messages in thread From: Yao, Jiewen @ 2018-11-14 0:28 UTC (permalink / raw) To: Cohen, Eugene, edk2-devel@lists.01.org, Zhang, Chao B; +Cc: Bin, Sung-Uk (???) Thanks Eugene For 3, I do not think you need expose your IP. There are 2 purposes I propose. 1) To prove this patch works for SPI. We need a real example to use this TpmIoLib. It is OK to keep it close source if it is IP. I just want to make sure this approach is feasible. 2) To demonstrate how to do that. If you can use EFI_SPI_PROTOCOL, that would be perfect. Thank you Yao Jiewen > -----Original Message----- > From: Cohen, Eugene [mailto:eugene@hp.com] > Sent: Wednesday, November 14, 2018 6:49 AM > To: Yao, Jiewen <jiewen.yao@intel.com>; edk2-devel@lists.01.org; Zhang, > Chao B <chao.b.zhang@intel.com> > Cc: Bin, Sung-Uk (빈성욱) <sunguk-bin@hp.com> > Subject: RE: [PATCH 1/4] SecurityPkg: enable TPM components to build for > ARM and AARCH64 > > Jiewen, > > 1 and 2 - okay, no problem, working on it. > > 3 - we are developing the SPI TPM support as we speak. Our SPI controller > is proprietary so there's no value in trying to share that driver. Our internal > SPI driver currently does not support the PI spec SPI_IO_PROTOCOL but are > looking at adopting it - if so then we could contribute the TpmIoLibSpi > instance which would connect the TPM stack to the SPI_IO_PROTOCOL. Is > this what you wanted or is there something more specific you were looking > for? > > Thanks, > > Eugene > > > -----Original Message----- > > From: Yao, Jiewen <jiewen.yao@intel.com> > > Sent: Tuesday, November 13, 2018 3:22 PM > > To: Cohen, Eugene <eugene@hp.com>; edk2-devel@lists.01.org; Zhang, > > Chao B <chao.b.zhang@intel.com> > > Cc: Bin, Sung-Uk (빈성욱) <sunguk-bin@hp.com> > > Subject: RE: [PATCH 1/4] SecurityPkg: enable TPM components to build for > > ARM and AARCH64 > > > > HI Eugene > > Thanks to enable SPI TPM chip. > > In general, I am OK on this patch series. > > > > There are some additional work here. > > 1) Please split this patch to 2. The TpmIoLib is not present in at this point of > > time. We should add it after TpmIoLib instance is added. > > > > 2) Since this patch series adds the dependency of TpmIoLib, please update > > *all* impacted platform in EDKII repo and EDKII platform repo. > > We need make sure this patch series does not break any existing platform > > build. > > > > 3) I hope, (if possible) you can provide one *real example* on how to add > > SPI instance, to demonstrate the usage and value of this one more layer > > abstraction. > > > > Thank you > > Yao Jiewen > > > > > > > -----Original Message----- > > > From: Cohen, Eugene [mailto:eugene@hp.com] > > > Sent: Wednesday, November 14, 2018 6:13 AM > > > To: edk2-devel@lists.01.org; Yao, Jiewen <jiewen.yao@intel.com>; > > > Zhang, Chao B <chao.b.zhang@intel.com> > > > Cc: Bin, Sung-Uk (빈성욱) <sunguk-bin@hp.com> > > > Subject: [PATCH 1/4] SecurityPkg: enable TPM components to build for > > > ARM and AARCH64 > > > > > > SecurityPkg: enable TPM components to build for ARM and AARCH64 > > > > > > Contributed-under: TianoCore Contribution Agreement 1.1 > > > Cc: Chao Zhang <chao.b.zhang@intel.com> > > > Cc: Jiewen Yao <jiewen.yao@intel.com> > > > Signed-off-by: Eugene Cohen <eugene@hp.com> > > > --- > > > SecurityPkg/SecurityPkg.dsc | 3 ++- > > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > > > diff --git a/SecurityPkg/SecurityPkg.dsc b/SecurityPkg/SecurityPkg.dsc > > > index 68a2953..6fb9ad2 100644 > > > --- a/SecurityPkg/SecurityPkg.dsc > > > +++ b/SecurityPkg/SecurityPkg.dsc > > > @@ -53,6 +53,7 @@ > > > IntrinsicLib|CryptoPkg/Library/IntrinsicLib/IntrinsicLib.inf > > > OpensslLib|CryptoPkg/Library/OpensslLib/OpensslLib.inf > > > IoLib|MdePkg/Library/BaseIoLibIntrinsic/BaseIoLibIntrinsic.inf > > > + TpmIoLib|SecurityPkg/Library/TpmIoLibMmio/TpmIoLibMmio.inf > > > TpmCommLib|SecurityPkg/Library/TpmCommLib/TpmCommLib.inf > > > > > > > > PlatformSecureLib|SecurityPkg/Library/PlatformSecureLibNull/PlatformSe > > > PlatformSecureLib|cu > > > reLibNull.inf > > > > > > > > TcgPhysicalPresenceLib|SecurityPkg/Library/DxeTcgPhysicalPresenceLib/D > > > TcgPhysicalPresenceLib|xe > > > TcgPhysicalPresenceLib.inf > > > @@ -199,7 +200,7 @@ > > > [Components.IA32, Components.X64, Components.ARM, > > Components.AARCH64] > > > SecurityPkg/Library/AuthVariableLib/AuthVariableLib.inf > > > > > > -[Components.IA32, Components.X64] > > > +[Components.IA32, Components.X64 Components.ARM, > > > Components.AARCH64] > > > # > > > > > SecurityPkg/UserIdentification/PwdCredentialProviderDxe/PwdCredential > > P > > > r > > > oviderDxe.inf > > > # > > > > > SecurityPkg/UserIdentification/UsbCredentialProviderDxe/UsbCredentialP > > > ro > > > viderDxe.inf > > > > > > > > SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootCon > > fi > > > gDxe.inf > > > -- > > > 2.7.4 ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 1/4] SecurityPkg: enable TPM components to build for ARM and AARCH64 2018-11-13 22:21 ` Yao, Jiewen 2018-11-13 22:49 ` Cohen, Eugene @ 2018-11-13 22:59 ` Kinney, Michael D 1 sibling, 0 replies; 5+ messages in thread From: Kinney, Michael D @ 2018-11-13 22:59 UTC (permalink / raw) To: Yao, Jiewen, Cohen, Eugene, edk2-devel@lists.01.org, Zhang, Chao B, Kinney, Michael D Cc: Bin, Sung-Uk (???) Jiewen, There are I2C examples for TPM12 in the QuarkPlatformPkg. Would that we a good example too? Could this new lib class be used for both TPM12 and TPM20 devices? Mike > -----Original Message----- > From: edk2-devel [mailto:edk2-devel- > bounces@lists.01.org] On Behalf Of Yao, Jiewen > Sent: Tuesday, November 13, 2018 2:22 PM > To: Cohen, Eugene <eugene@hp.com>; edk2- > devel@lists.01.org; Zhang, Chao B > <chao.b.zhang@intel.com> > Cc: Bin, Sung-Uk (???) <sunguk-bin@hp.com> > Subject: Re: [edk2] [PATCH 1/4] SecurityPkg: enable TPM > components to build for ARM and AARCH64 > > HI Eugene > Thanks to enable SPI TPM chip. > In general, I am OK on this patch series. > > There are some additional work here. > 1) Please split this patch to 2. The TpmIoLib is not > present in at this point of time. We should add it > after TpmIoLib instance is added. > > 2) Since this patch series adds the dependency of > TpmIoLib, please update *all* impacted platform in > EDKII repo and EDKII platform repo. > We need make sure this patch series does not break any > existing platform build. > > 3) I hope, (if possible) you can provide one *real > example* on how to add SPI instance, to demonstrate the > usage and value of this one more layer abstraction. > > Thank you > Yao Jiewen > > > > -----Original Message----- > > From: Cohen, Eugene [mailto:eugene@hp.com] > > Sent: Wednesday, November 14, 2018 6:13 AM > > To: edk2-devel@lists.01.org; Yao, Jiewen > <jiewen.yao@intel.com>; Zhang, > > Chao B <chao.b.zhang@intel.com> > > Cc: Bin, Sung-Uk (빈성욱) <sunguk-bin@hp.com> > > Subject: [PATCH 1/4] SecurityPkg: enable TPM > components to build for ARM > > and AARCH64 > > > > SecurityPkg: enable TPM components to build for ARM > and AARCH64 > > > > Contributed-under: TianoCore Contribution Agreement > 1.1 > > Cc: Chao Zhang <chao.b.zhang@intel.com> > > Cc: Jiewen Yao <jiewen.yao@intel.com> > > Signed-off-by: Eugene Cohen <eugene@hp.com> > > --- > > SecurityPkg/SecurityPkg.dsc | 3 ++- > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > diff --git a/SecurityPkg/SecurityPkg.dsc > b/SecurityPkg/SecurityPkg.dsc > > index 68a2953..6fb9ad2 100644 > > --- a/SecurityPkg/SecurityPkg.dsc > > +++ b/SecurityPkg/SecurityPkg.dsc > > @@ -53,6 +53,7 @@ > > > IntrinsicLib|CryptoPkg/Library/IntrinsicLib/IntrinsicLi > b.inf > > > OpensslLib|CryptoPkg/Library/OpensslLib/OpensslLib.inf > > > IoLib|MdePkg/Library/BaseIoLibIntrinsic/BaseIoLibIntrin > sic.inf > > + > TpmIoLib|SecurityPkg/Library/TpmIoLibMmio/TpmIoLibMmio. > inf > > > TpmCommLib|SecurityPkg/Library/TpmCommLib/TpmCommLib.in > f > > > > > PlatformSecureLib|SecurityPkg/Library/PlatformSecureLib > Null/PlatformSecu > > reLibNull.inf > > > > > TcgPhysicalPresenceLib|SecurityPkg/Library/DxeTcgPhysic > alPresenceLib/Dxe > > TcgPhysicalPresenceLib.inf > > @@ -199,7 +200,7 @@ > > [Components.IA32, Components.X64, Components.ARM, > > Components.AARCH64] > > > SecurityPkg/Library/AuthVariableLib/AuthVariableLib.inf > > > > -[Components.IA32, Components.X64] > > +[Components.IA32, Components.X64 Components.ARM, > > Components.AARCH64] > > # > > > SecurityPkg/UserIdentification/PwdCredentialProviderDxe > /PwdCredentialPr > > oviderDxe.inf > > # > > > SecurityPkg/UserIdentification/UsbCredentialProviderDxe > /UsbCredentialPro > > viderDxe.inf > > > > > SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/S > ecureBootConfi > > gDxe.inf > > -- > > 2.7.4 > > _______________________________________________ > edk2-devel mailing list > edk2-devel@lists.01.org > https://lists.01.org/mailman/listinfo/edk2-devel ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2018-11-14 0:28 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2018-11-13 22:12 [PATCH 1/4] SecurityPkg: enable TPM components to build for ARM and AARCH64 Cohen, Eugene 2018-11-13 22:21 ` Yao, Jiewen 2018-11-13 22:49 ` Cohen, Eugene 2018-11-14 0:28 ` Yao, Jiewen 2018-11-13 22:59 ` Kinney, Michael D
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox