[coreboot-gerrit] Change in coreboot[master]: sec/intel/txt: Add and fill in BIOS Specification info

[Angel Pons (Code Review)]

Angel Pons has submitted this change. ( https://review.coreboot.org/c/coreboot/+/46493 )

Change subject: sec/intel/txt: Add and fill in BIOS Specification info
......................................................................

sec/intel/txt: Add and fill in BIOS Specification info

This is not critical to function, but is nice to have.

Change-Id: Ieb5f41f3e4c5644a31606434916c35542d35617a
Signed-off-by: Angel Pons 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/46493
Reviewed-by: Arthur Heymans 
Tested-by: build bot (Jenkins) 
---
M src/security/intel/txt/ramstage.c
1 file changed, 8 insertions(+), 0 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Arthur Heymans: Looks good to me, approved

diff --git a/src/security/intel/txt/ramstage.c b/src/security/intel/txt/ramstage.c
index 8d9f5d9..5d0d121 100644
--- a/src/security/intel/txt/ramstage.c
+++ b/src/security/intel/txt/ramstage.c
@@ -298,6 +298,7 @@
     */
    struct {
    	struct txt_biosdataregion bdr;
+		struct txt_bios_spec_ver_element spec;
    	struct txt_heap_acm_element heap_acm;
    	struct txt_extended_data_element_header end;
    } __packed data = {0};
@@ -350,6 +351,13 @@
    data.bdr.support_acpi_ppi = 0;
    data.bdr.platform_type = 0;

+	/* Fill in the version of the used TXT BIOS Specification */
+	data.spec.header.type = HEAP_EXTDATA_TYPE_BIOS_SPEC_VER;
+	data.spec.header.size = sizeof(data.spec);
+	data.spec.ver_major = 2;
+	data.spec.ver_minor = 1;
+	data.spec.ver_revision = 0;
+
    /* Extended elements - ACM addresses */
    data.heap_acm.header.type = HEAP_EXTDATA_TYPE_ACM;
    data.heap_acm.header.size = sizeof(data.heap_acm);