[edk2-platforms/devel-MinPlatform][PATCH v1 1/1] MinPlatformPkg: Include Tcg2Smm ACPI table RAW section

public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed

* [edk2-platforms/devel-MinPlatform][PATCH v1 1/1] MinPlatformPkg: Include Tcg2Smm ACPI table RAW section
@ 2019-04-05  6:48 Kubacki, Michael A
  2019-04-08  3:06 ` Chiu, Chasel
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Kubacki, Michael A @ 2019-04-05  6:48 UTC (permalink / raw)
  To: devel; +Cc: Chasel Chiu, Jiewen Yao, Nate DeSimone, Ruiyu Ni,
	Michael D Kinney

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1684

Updates the INF statement for Tcg2Smm to include a RuleOverride so
the Tpm.asl ACPI table is included as a RAW section in the FFS for
Tcg2Smm.

This allows Tcg2Smm to find the section during boot and publish the
ACPI table.

Cc: Chasel Chiu <chasel.chiu@intel.com>
Cc: Jiewen Yao <jiewen.yao@intel.com>
Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
Cc: Ruiyu Ni <ruiyu.ni@intel.com>
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Michael Kubacki <michael.a.kubacki@intel.com>
---
 Platform/Intel/MinPlatformPkg/Include/Fdf/CoreSecurityLateInclude.fdf | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/Platform/Intel/MinPlatformPkg/Include/Fdf/CoreSecurityLateInclude.fdf b/Platform/Intel/MinPlatformPkg/Include/Fdf/CoreSecurityLateInclude.fdf
index ccb083a0a6..2171fe12e5 100644
--- a/Platform/Intel/MinPlatformPkg/Include/Fdf/CoreSecurityLateInclude.fdf
+++ b/Platform/Intel/MinPlatformPkg/Include/Fdf/CoreSecurityLateInclude.fdf
@@ -1,7 +1,7 @@
 ## @file
 #  FDF file of Platform.
 #
-# Copyright (c) 2017, Intel Corporation. All rights reserved.<BR>
+# Copyright (c) 2017 - 2019, Intel Corporation. All rights reserved.<BR>
 #
 # This program and the accompanying materials are licensed and made available under
 # the terms and conditions of the BSD License which accompanies this distribution.
@@ -20,6 +20,6 @@ INF  SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfigDxe.i
 !if gMinPlatformPkgTokenSpaceGuid.PcdTpm2Enable == TRUE
 INF  SecurityPkg/Tcg/MemoryOverwriteControl/TcgMor.inf
 INF  SecurityPkg/Tcg/Tcg2Dxe/Tcg2Dxe.inf
-INF  SecurityPkg/Tcg/Tcg2Smm/Tcg2Smm.inf
+INF  RuleOverride = DRIVER_ACPITABLE SecurityPkg/Tcg/Tcg2Smm/Tcg2Smm.inf
 INF  SecurityPkg/Tcg/Tcg2Config/Tcg2ConfigDxe.inf
 !endif
-- 
2.16.2.windows.1


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [edk2-platforms/devel-MinPlatform][PATCH v1 1/1] MinPlatformPkg: Include Tcg2Smm ACPI table RAW section
  2019-04-05  6:48 [edk2-platforms/devel-MinPlatform][PATCH v1 1/1] MinPlatformPkg: Include Tcg2Smm ACPI table RAW section Kubacki, Michael A
@ 2019-04-08  3:06 ` Chiu, Chasel
  2019-04-08  3:28 ` [edk2-devel] " chasel.chiu
  2019-04-08  5:33 ` Chiu, Chasel
  2 siblings, 0 replies; 4+ messages in thread
From: Chiu, Chasel @ 2019-04-08  3:06 UTC (permalink / raw)
  To: Kubacki, Michael A, devel@edk2.groups.io
  Cc: Yao, Jiewen, Desimone, Nathaniel L, Ni, Ray, Kinney, Michael D


Reviewed-by: Chasel Chiu <chasel.chiu@intel.com>

> -----Original Message-----
> From: Kubacki, Michael A
> Sent: Friday, April 5, 2019 2:48 PM
> To: devel@edk2.groups.io
> Cc: Chiu, Chasel <chasel.chiu@intel.com>; Yao, Jiewen <jiewen.yao@intel.com>;
> Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>; Ni, Ray
> <ray.ni@intel.com>; Kinney, Michael D <michael.d.kinney@intel.com>
> Subject: [edk2-platforms/devel-MinPlatform][PATCH v1 1/1] MinPlatformPkg:
> Include Tcg2Smm ACPI table RAW section
> 
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1684
> 
> Updates the INF statement for Tcg2Smm to include a RuleOverride so the Tpm.asl
> ACPI table is included as a RAW section in the FFS for Tcg2Smm.
> 
> This allows Tcg2Smm to find the section during boot and publish the ACPI table.
> 
> Cc: Chasel Chiu <chasel.chiu@intel.com>
> Cc: Jiewen Yao <jiewen.yao@intel.com>
> Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> Cc: Ruiyu Ni <ruiyu.ni@intel.com>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Michael Kubacki <michael.a.kubacki@intel.com>
> ---
>  Platform/Intel/MinPlatformPkg/Include/Fdf/CoreSecurityLateInclude.fdf | 4
> ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git
> a/Platform/Intel/MinPlatformPkg/Include/Fdf/CoreSecurityLateInclude.fdf
> b/Platform/Intel/MinPlatformPkg/Include/Fdf/CoreSecurityLateInclude.fdf
> index ccb083a0a6..2171fe12e5 100644
> --- a/Platform/Intel/MinPlatformPkg/Include/Fdf/CoreSecurityLateInclude.fdf
> +++ b/Platform/Intel/MinPlatformPkg/Include/Fdf/CoreSecurityLateInclude.
> +++ fdf
> @@ -1,7 +1,7 @@
>  ## @file
>  #  FDF file of Platform.
>  #
> -# Copyright (c) 2017, Intel Corporation. All rights reserved.<BR>
> +# Copyright (c) 2017 - 2019, Intel Corporation. All rights
> +reserved.<BR>
>  #
>  # This program and the accompanying materials are licensed and made available
> under  # the terms and conditions of the BSD License which accompanies this
> distribution.
> @@ -20,6 +20,6 @@ INF
> SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfigD
> xe.i
>  !if gMinPlatformPkgTokenSpaceGuid.PcdTpm2Enable == TRUE  INF
> SecurityPkg/Tcg/MemoryOverwriteControl/TcgMor.inf
>  INF  SecurityPkg/Tcg/Tcg2Dxe/Tcg2Dxe.inf
> -INF  SecurityPkg/Tcg/Tcg2Smm/Tcg2Smm.inf
> +INF  RuleOverride = DRIVER_ACPITABLE
> +SecurityPkg/Tcg/Tcg2Smm/Tcg2Smm.inf
>  INF  SecurityPkg/Tcg/Tcg2Config/Tcg2ConfigDxe.inf
>  !endif
> --
> 2.16.2.windows.1


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [edk2-devel] [edk2-platforms/devel-MinPlatform][PATCH v1 1/1] MinPlatformPkg: Include Tcg2Smm ACPI table RAW section
  2019-04-05  6:48 [edk2-platforms/devel-MinPlatform][PATCH v1 1/1] MinPlatformPkg: Include Tcg2Smm ACPI table RAW section Kubacki, Michael A
  2019-04-08  3:06 ` Chiu, Chasel
@ 2019-04-08  3:28 ` chasel.chiu
  2019-04-08  5:33 ` Chiu, Chasel
  2 siblings, 0 replies; 4+ messages in thread
From: chasel.chiu @ 2019-04-08  3:28 UTC (permalink / raw)
  To: Kubacki, Michael A, devel

[-- Attachment #1: Type: text/plain, Size: 52 bytes --]

Reviewed-by: Chasel Chiu < chasel.chiu@intel.com >

[-- Attachment #2: Type: text/html, Size: 218 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [edk2-platforms/devel-MinPlatform][PATCH v1 1/1] MinPlatformPkg: Include Tcg2Smm ACPI table RAW section
  2019-04-05  6:48 [edk2-platforms/devel-MinPlatform][PATCH v1 1/1] MinPlatformPkg: Include Tcg2Smm ACPI table RAW section Kubacki, Michael A
  2019-04-08  3:06 ` Chiu, Chasel
  2019-04-08  3:28 ` [edk2-devel] " chasel.chiu
@ 2019-04-08  5:33 ` Chiu, Chasel
  2 siblings, 0 replies; 4+ messages in thread
From: Chiu, Chasel @ 2019-04-08  5:33 UTC (permalink / raw)
  To: Kubacki, Michael A, devel@edk2.groups.io
  Cc: Yao, Jiewen, Desimone, Nathaniel L, Ni, Ray, Kinney, Michael D


Reviewed-by: Chasel Chiu <chasel.chiu@intel.com>

> -----Original Message-----
> From: Kubacki, Michael A
> Sent: Friday, April 5, 2019 2:48 PM
> To: devel@edk2.groups.io
> Cc: Chiu, Chasel <chasel.chiu@intel.com>; Yao, Jiewen <jiewen.yao@intel.com>;
> Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>; Ni, Ray
> <ray.ni@intel.com>; Kinney, Michael D <michael.d.kinney@intel.com>
> Subject: [edk2-platforms/devel-MinPlatform][PATCH v1 1/1] MinPlatformPkg:
> Include Tcg2Smm ACPI table RAW section
> 
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1684
> 
> Updates the INF statement for Tcg2Smm to include a RuleOverride so the Tpm.asl
> ACPI table is included as a RAW section in the FFS for Tcg2Smm.
> 
> This allows Tcg2Smm to find the section during boot and publish the ACPI table.
> 
> Cc: Chasel Chiu <chasel.chiu@intel.com>
> Cc: Jiewen Yao <jiewen.yao@intel.com>
> Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> Cc: Ruiyu Ni <ruiyu.ni@intel.com>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Michael Kubacki <michael.a.kubacki@intel.com>
> ---
>  Platform/Intel/MinPlatformPkg/Include/Fdf/CoreSecurityLateInclude.fdf | 4
> ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git
> a/Platform/Intel/MinPlatformPkg/Include/Fdf/CoreSecurityLateInclude.fdf
> b/Platform/Intel/MinPlatformPkg/Include/Fdf/CoreSecurityLateInclude.fdf
> index ccb083a0a6..2171fe12e5 100644
> --- a/Platform/Intel/MinPlatformPkg/Include/Fdf/CoreSecurityLateInclude.fdf
> +++ b/Platform/Intel/MinPlatformPkg/Include/Fdf/CoreSecurityLateInclude.
> +++ fdf
> @@ -1,7 +1,7 @@
>  ## @file
>  #  FDF file of Platform.
>  #
> -# Copyright (c) 2017, Intel Corporation. All rights reserved.<BR>
> +# Copyright (c) 2017 - 2019, Intel Corporation. All rights
> +reserved.<BR>
>  #
>  # This program and the accompanying materials are licensed and made available
> under  # the terms and conditions of the BSD License which accompanies this
> distribution.
> @@ -20,6 +20,6 @@ INF
> SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfigD
> xe.i
>  !if gMinPlatformPkgTokenSpaceGuid.PcdTpm2Enable == TRUE  INF
> SecurityPkg/Tcg/MemoryOverwriteControl/TcgMor.inf
>  INF  SecurityPkg/Tcg/Tcg2Dxe/Tcg2Dxe.inf
> -INF  SecurityPkg/Tcg/Tcg2Smm/Tcg2Smm.inf
> +INF  RuleOverride = DRIVER_ACPITABLE
> +SecurityPkg/Tcg/Tcg2Smm/Tcg2Smm.inf
>  INF  SecurityPkg/Tcg/Tcg2Config/Tcg2ConfigDxe.inf
>  !endif
> --
> 2.16.2.windows.1


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2019-04-08  5:33 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-04-05  6:48 [edk2-platforms/devel-MinPlatform][PATCH v1 1/1] MinPlatformPkg: Include Tcg2Smm ACPI table RAW section Kubacki, Michael A
2019-04-08  3:06 ` Chiu, Chasel
2019-04-08  3:28 ` [edk2-devel] " chasel.chiu
2019-04-08  5:33 ` Chiu, Chasel

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox