Re: [PATCH] SecurityPkg: Disable TPM interrupt in DEC

["Yao, Jiewen" <jiewen.yao@intel.com>]

Thanks.

Would you please update comment with - "Disable TPM interrupt support in DEC by default to keep compatibility"

Reviewed-by: Jiewen.yao@intel.com



> -----Original Message-----
> From: Zhang, Chao B
> Sent: Tuesday, January 30, 2018 9:17 AM
> To: edk2-devel@lists.01.org
> Cc: Yao, Jiewen <jiewen.yao@intel.com>; Long, Qin <qin.long@intel.com>;
> Zhang, Chao B <chao.b.zhang@intel.com>
> Subject: [PATCH] SecurityPkg: Disable TPM interrupt in DEC
> 
> Disable TPM interrupt support in DEC
> 
> Cc: Yao Jiewen <jiewen.yao@intel.com>
> Cc: Long Qin <qin.long@intel.com>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Chao Zhang <chao.b.zhang@intel.com>
> ---
>  SecurityPkg/SecurityPkg.dec | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/SecurityPkg/SecurityPkg.dec b/SecurityPkg/SecurityPkg.dec
> index d2741f6..983fb0e 100644
> --- a/SecurityPkg/SecurityPkg.dec
> +++ b/SecurityPkg/SecurityPkg.dec
> @@ -453,12 +453,12 @@
>    ## Indicate current TPM2 Interrupt Number reported by _CRS control
> method.<BR><BR>
>    # TPM2 Interrupt feature is disabled If the pcd is set to 0.<BR>
>    # @Prompt Current TPM2 Interrupt Number
> -
> gEfiSecurityPkgTokenSpaceGuid.PcdTpm2CurrentIrqNum|0x0C|UINT32|0x0001
> 001C
> +
> gEfiSecurityPkgTokenSpaceGuid.PcdTpm2CurrentIrqNum|0x00|UINT32|0x0001
> 001C
> 
>    ## Indicate platform possible TPM2 Interrupt Number reported by _PRS
> control method.<BR><BR>
>    # Possible TPM2 Interrupt Number Buffer will not be reported if TPM2
> Interrupt feature is disabled.<BR>
>    # @Prompt Possible TPM2 Interrupt Number buffer
> -  gEfiSecurityPkgTokenSpaceGuid.PcdTpm2PossibleIrqNumBuf|{0x0C, 0x00,
> 0x00, 0x00}|VOID*|0x0001001D
> +  gEfiSecurityPkgTokenSpaceGuid.PcdTpm2PossibleIrqNumBuf|{0x00, 0x00,
> 0x00, 0x00}|VOID*|0x0001001D
> 
>  [PcdsDynamic, PcdsDynamicEx]
> 
> --
> 1.9.5.msysgit.1