From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by mx.groups.io with SMTP id smtpd.web12.311.1591219022918539700 for ; Wed, 03 Jun 2020 14:17:03 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@intel.onmicrosoft.com header.s=selector2-intel-onmicrosoft-com header.b=h4JW7PXc; spf=pass (domain: intel.com, ip: 192.55.52.93, mailfrom: jorge.hernandez.beltran@intel.com) IronPort-SDR: C8f+v4pUD1/lHLV7V1Utu9Fr8vVywUfCofmoOGLuWkICVPzVs6LNNh7/aARry7FLek6Dc5963F dwHp4290uwZA== X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga002.jf.intel.com ([10.7.209.21]) by fmsmga102.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 03 Jun 2020 14:17:01 -0700 IronPort-SDR: elBwUaaMOcvDjyziz4s2XDXeasLQDDMU5lsoQFesBZqon7smzcgu13qCnp5NQGz2PFLEPbJDCG gvHJEs+qCGPA== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.73,469,1583222400"; d="scan'208";a="287164290" Received: from fmsmsx106.amr.corp.intel.com ([10.18.124.204]) by orsmga002.jf.intel.com with ESMTP; 03 Jun 2020 14:17:01 -0700 Received: from FMSEDG001.ED.cps.intel.com (10.1.192.133) by FMSMSX106.amr.corp.intel.com (10.18.124.204) with Microsoft SMTP Server (TLS) id 14.3.439.0; Wed, 3 Jun 2020 14:17:00 -0700 Received: from NAM11-BN8-obe.outbound.protection.outlook.com (104.47.58.177) by edgegateway.intel.com (192.55.55.68) with Microsoft SMTP Server (TLS) id 14.3.439.0; Wed, 3 Jun 2020 14:17:00 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=R59nHt52F2DDGUG3jacZ5WWji5548u2DPko3UwzijGV6/s42ewKHXW7V0GMcQCvRs7g/nUNhyK2BfkMcbnIZCo2HRPh3TZseYu2ycGCroHUW5pE1C6BZiX3u8MNZsboWyE4fJ1wOqYbL5CMgD5yDEPwPzl/awVQ5hW0IYXQgHPaX+XS0/pIPHBg75yqzDJ/RQg2rdx6I6bKwvsjlzgrCe66Sgv7n37nmspOowRICTfm62dn5wCEzNMRW7pP96eHkEieoasCXCtBx7KMLrJ5Hv80W9uKQJDI+UAv5TqMoeaB5GkO/o4OoSgCUfmoG72dwomgC2YTYtQHfP8DioEy+1w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=efkrCvpMBAPfoKSP1hT0rwniQFJdQCecKeQ7eGxGKy0=; b=lwtTecX1L6qneuRSgs2yroaQADgdQg9RFbTNZuiqZUQBWO4+J56ns7/4eYIYocj6hhiO87mBHT55pCnPNSSshv7SUuzW5+vFwiUM7dm0m1W4pCDL2Nt//Fd/t8L3xZZ0aBmxlH/HfQcTz1Va3QywgcOtSJ6q3iXZIT5XjLbV/k4WxjLWduMb6PIh+WRAGhBTtBwB14j1L6qQpO76OSuJGFRrycqH3sFDZvjvdc5DTznhG0nmu3Pmcon3UePfPf7VhR+l1Z1fj5Qo1/7U+z3sKmL74hC6mo3PTuQBEIQYMdFsnArGOpAX5AMHjdqRT5TXG+ze55aMMhbiBd565BhSqw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=intel.com; dmarc=pass action=none header.from=intel.com; dkim=pass header.d=intel.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=intel.onmicrosoft.com; s=selector2-intel-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=efkrCvpMBAPfoKSP1hT0rwniQFJdQCecKeQ7eGxGKy0=; b=h4JW7PXcM3JjN7jRSDM3o+OIVaNSC8o3fXr0M8Gzh6tv6F5lRM1PGUo8ORUXaOCW3DJ2XgMaM6S18K66odIcA38G5u+EfeeD+FqJHMK/ljdlBAVR3aN5dDbgvXEtx3LDQcBQTLxqNLEQ5sMVXuatAxu6fZA1qc8vzxoyg+/HVuQ= Received: from MW3PR11MB4588.namprd11.prod.outlook.com (2603:10b6:303:54::19) by MW3PR11MB4668.namprd11.prod.outlook.com (2603:10b6:303:54::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3045.17; Wed, 3 Jun 2020 21:16:58 +0000 Received: from MW3PR11MB4588.namprd11.prod.outlook.com ([fe80::61c5:809c:cd86:d4c1]) by MW3PR11MB4588.namprd11.prod.outlook.com ([fe80::61c5:809c:cd86:d4c1%7]) with mapi id 15.20.3045.024; Wed, 3 Jun 2020 21:16:57 +0000 From: "Hernandez Beltran, Jorge" To: "Lohr, Paul A" , "Gao, Liming" , "devel@edk2.groups.io" Subject: Re: [Patch 1/1] [Edk2Platforms] FitGen: Update FitGen tool to be compliance with the FIT specification revision 1.2 Thread-Topic: [Patch 1/1] [Edk2Platforms] FitGen: Update FitGen tool to be compliance with the FIT specification revision 1.2 Thread-Index: AQHWNQ7dH+h/BeCJY0SgU+WraRwf8ajG/ScAgAAQtICAAFzQUA== Date: Wed, 3 Jun 2020 21:16:57 +0000 Message-ID: References: <20200528164102.31576-1-jorge.hernandez.beltran@intel.com> In-Reply-To: Accept-Language: en-US X-Mentions: liming.gao@intel.com X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-version: 11.2.0.6 dlp-product: dlpe-windows dlp-reaction: no-action authentication-results: intel.com; dkim=none (message not signed) header.d=none;intel.com; dmarc=none action=none header.from=intel.com; x-originating-ip: [189.205.200.38] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 9baf6ace-de1e-419f-8584-08d8080372f5 x-ms-traffictypediagnostic: MW3PR11MB4668: x-ms-exchange-transport-forked: True x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:1728; x-forefront-prvs: 04238CD941 x-ms-exchange-senderadcheck: 1 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: yAn/qPoXuY/LsAYWmZbfmHRkm+CF9H0itnEAkDDTl8tGFZpchqfkqNgo4qZJXRDhD9UOVbfB1mcO1Oo31S46Pprj/11CAIAkVtzL4OUt7oc34z/kEITGluEczXVlejywChDW4bEZ/xe7vbFY0FLkPejpyBEGeHQ+UI5rgeyUysnqnFdRVdc8lAuypjhTeAAhVe4nimvMCUFFAZlaLblZE5en4rI9m8bVjrU/bgEjAUYk8mMD7o20686o3ULZLaAScI/EgGlmYNr1Te+C+KhClb7SbmELn6Ze5OF+4vBIN4Ezpxn4kCOmC8zjcdIZNzcKXEEqbiKau0zAGdbTIoTkaaFCqgAYw6utWyL13NapuLiG/egmBH6LnH+yRHEARV565d1ChGhsAcH+KkwHTTSf3w== x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:MW3PR11MB4588.namprd11.prod.outlook.com;PTR:;CAT:NONE;SFTY:;SFS:(376002)(396003)(366004)(346002)(39860400002)(136003)(53546011)(26005)(478600001)(7696005)(86362001)(55016002)(83380400001)(966005)(110136005)(33656002)(9686003)(8936002)(316002)(71200400001)(5660300002)(6506007)(66946007)(76116006)(30864003)(8676002)(64756008)(186003)(52536014)(66476007)(15650500001)(2906002)(66556008)(66446008);DIR:OUT;SFP:1102; x-ms-exchange-antispam-messagedata: +1VZiH3MvNQSOYr4Nca7PfHJgbCwApeqi3lAHz2KwYIo+FZUceuBCfJ6CsrUb9zHXoC4zL2jF7XOh3c5P3x4INBirCpLEZfkIntqXmBSA9iugooVj93nmTXwpYA/0MrHbUg7tLeu3gHsnE5PA/Aoc+ayLVNeg5w23Z+b1Vbk3pyRSCfTvqe9Wzc3NM+ZHo48r0OgcXKbME2mkjO4WgsGFebmwNSvSnoIdLNRfHgKYHeLQisdtcq4QxOPxKL9U4RnQl5vFz81UIR4TH7QNYibQ7XiJD+08sqLX8M/jbFmeD08q+0kdrCzWEelvsaM1g4+IahNUPqjc6QMRLjoQiX4MdAA+cd7izIlEivjczyimRehozzGcxEFuEtiHOgS4PBx2fFqe5mw1a2xfH1fz+2KZd3jfkoaqfFpT+I84Q/xTs+yXXlrEqtHBMHxh79+2EykkvvG7sLfwueVOhI1fssc64BRKcyJGZBOLGPt3fvFasUft8Befq1oPD7CGK+rchhU MIME-Version: 1.0 X-MS-Exchange-CrossTenant-Network-Message-Id: 9baf6ace-de1e-419f-8584-08d8080372f5 X-MS-Exchange-CrossTenant-originalarrivaltime: 03 Jun 2020 21:16:57.8218 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: 8qm/Khv1j13sZQ/J7ttttoH3ybcI6G8/2bJprUPi2bdeZnMIWHFV63vPfKKZfe6C5720sJghmiCLPqYrbD1UGjS8v538FhW+4Tj1nBHahTNY9zMuht5KpHO8DtOC4mh2 X-MS-Exchange-Transport-CrossTenantHeadersStamped: MW3PR11MB4668 Return-Path: jorge.hernandez.beltran@intel.com X-OriginatorOrg: intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Done. I fixed the commit message issues regarding the format and I sent the= updated patch with "git send-email --compose --to=3Ddevel@edk2.groups.io..= ." command again. I added "-v2 --annotate" to indicate it was version 2 of the patch but I gu= ess it didn't work, subject from new email doesn't have the "[Patch v2 1/1]= " part I was expecting even after I set it manually when composing "the em= ail". @Gao, Liming If you know how to force subject to reflect the version, "[Pat= ch v2 1/1]", I would appreciate the help... -----Original Message----- From: Lohr, Paul A =20 Sent: Wednesday, June 3, 2020 10:31 AM To: Gao, Liming ; Hernandez Beltran, Jorge ; devel@edk2.groups.io Subject: RE: [Patch 1/1] [Edk2Platforms] FitGen: Update FitGen tool to be c= ompliance with the FIT specification revision 1.2 Thanks Liming. I agree the code changes look good. Paul A. Lohr - Server Firmware Enabling 512.239.9073 (cell) 512.794.5044 (work) -----Original Message----- From: Gao, Liming Sent: Wednesday, June 3, 2020 9:31 AM To: Hernandez Beltran, Jorge ; devel@edk= 2.groups.io Cc: Lohr, Paul A Subject: RE: [Patch 1/1] [Edk2Platforms] FitGen: Update FitGen tool to be c= ompliance with the FIT specification revision 1.2 Jorge: The patch is good. But, the commit message is too long than 80. Please up= date the commit message to be short. You can use edk2\BaseTools\Scripts\PatchCheck.py tool to check the patch = format. =20 =20 With the commit message change, Reviewed-by: Liming Gao Thanks Liming > -----Original Message----- > From: Hernandez Beltran, Jorge > Sent: Friday, May 29, 2020 12:41 AM > To: devel@edk2.groups.io > Cc: Gao, Liming ; Lohr, Paul A=20 > ; Hernandez Beltran, Jorge=20 > > Subject: [Patch 1/1] [Edk2Platforms] FitGen: Update FitGen tool to be=20 > compliance with the FIT specification revision 1.2 >=20 > BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=3D2687 >=20 > FIT specification revision 1.2 was recently released to the open commun= ity. > Revision 1.2 updates CSE Secure Boot Rules in section 4.12, and adds 2 = new entry > sub-types used to distinguish the CSE entries. >=20 > Signed-off-by: Jorge Hernandez Beltran=20 > > --- > Silicon/Intel/Tools/FitGen/FitGen.c | 152 +++++++++++++++++++++++++++---= ------ > Silicon/Intel/Tools/FitGen/FitGen.h | 2 +- > 2 files changed, 114 insertions(+), 40 deletions(-) >=20 > diff --git a/Silicon/Intel/Tools/FitGen/FitGen.c > b/Silicon/Intel/Tools/FitGen/FitGen.c > index 75d8932d90ac..c4006e69c822 100644 > --- a/Silicon/Intel/Tools/FitGen/FitGen.c > +++ b/Silicon/Intel/Tools/FitGen/FitGen.c > @@ -226,6 +226,8 @@ typedef struct { > #define FIT_TABLE_TYPE_BOOT_POLICY_MANIFEST 12 > #define FIT_TABLE_TYPE_BIOS_DATA_AREA 13 > #define FIT_TABLE_TYPE_CSE_SECURE_BOOT 16 > +#define FIT_TABLE_SUBTYPE_FIT_PATCH_MANIFEST 12 > +#define FIT_TABLE_SUBTYPE_ACM_MANIFEST 13 >=20 > // > // With OptionalModule Address isn't known until free space has been=20 > @@ -236,6 +238,7 @@ typedef struct { // typedef struct { > UINT32 Type; > + UINT32 SubType; // Used by OptionalModule only > UINT32 Address; > UINT8 *Buffer; // Used by OptionalModule only > UINT32 Size; > @@ -295,7 +298,7 @@ Returns: > --*/ > { > printf ( > - "%s - Tiano IA32/X64 FIT table generation Utility for FIT spec revis= ion 1.1."" Version %i.%i\n\n", > + "%s - Tiano IA32/X64 FIT table generation Utility for FIT spec=20 > + revision 1.2."" Version %i.%i\n\n", > UTILITY_NAME, > UTILITY_MAJOR_VERSION, > UTILITY_MINOR_VERSION > @@ -334,7 +337,7 @@ Returns: > "\t[-U |]\n" > "\t[-B ] [-B ...] [-V ]\n" > "\t[-M ] [-M ...]|[-U=20 > | MicrocodeRegionSize>|] [-V ]\n" > - "\t[-O RecordType ||| [-V > ]] [-O ... [-V ...]]\n" > + "\t[-O RecordType + RecordDataSize>| RecordDataSize>||| [-V ]] [-O ... [-V ...]]\n" > "\t[-P RecordType [-V ]] [-P ... [-V ...]]\n" > , UTILITY_NAME); > printf (" Where:\n"); > @@ -366,6 +369,7 @@ Returns: > printf ("\tRecordDataSize - FIT entry record data size.\n"); > printf ("\tRecordDataGuid - FIT entry record data GUID.\n"); > printf ("\tRecordBinFile - FIT entry record data binary file.= \n"); > + printf ("\tCseRecordSubType - FIT entry record subtype. Use to f= urther distinguish CSE entries (see FIT spec revision 1.2 chapter > 4.12).\n"); > printf ("\tFitEntryDefaultVersion - The default version for all FIT=20 > table entries. 0x%04x is used if this is not specified.\n", DEFAULT_FIT_E= NTRY_VERSION); > printf ("\tFitHeaderVersion - The version for FIT header. (Overr= ide default version)\n"); > printf ("\tStartupAcmVersion - The version for StartupAcm. (Overr= ide default version)\n"); > @@ -857,6 +861,7 @@ Returns: > UINT8 *FileBuffer; > UINT32 FileSize; > UINT32 Type; > + UINT32 SubType; > UINT8 *MicrocodeFileBuffer; > UINT8 *MicrocodeFileBufferRaw; > UINT32 MicrocodeFileSize; > @@ -1608,26 +1613,22 @@ Returns: > } > Type =3D xtoi (argv[Index + 1]); > // > - // 1st, try GUID > + // 1st, try CSE entry sub-type > // > - if (IsGuidData (argv[Index + 2], &Guid)) { > - FileBuffer =3D FindFileFromFvByGuid (FdBuffer, FdSize, &Guid, &Fil= eSize); > - if (FileBuffer =3D=3D NULL) { > - Error (NULL, 0, 0, "-O Parameter incorrect, GUID not found!", "%= s", argv[Index + 2]); > - // not found > - return 0; > - } > - if (FileSize >=3D 0x80000000) { > - Error (NULL, 0, 0, "-O Parameter incorrect, FileSize too large!"= , NULL); > - return 0; > + SubType =3D 0; > + if (Type =3D=3D FIT_TABLE_TYPE_CSE_SECURE_BOOT) { > + if (Index + 3 >=3D argc) { > + break; > } > - FileBuffer =3D (UINT8 *)MEMORY_TO_FLASH (FileBuffer, FdBuffer, FdS= ize); > - Index +=3D 3; > - } else { > + SubType =3D xtoi (argv[Index + 2]); > // > - // 2nd, try file > + // try file > // > - Status =3D ReadInputFile (argv[Index + 2], &FileBuffer, &FileSize,= NULL); > + if (SubType !=3D FIT_TABLE_SUBTYPE_FIT_PATCH_MANIFEST && SubType != =3D FIT_TABLE_SUBTYPE_ACM_MANIFEST) { > + Error (NULL, 0, 0, "-O Parameter incorrect, SubType unsupported!= ", NULL); > + return 0; > + } > + Status =3D ReadInputFile (argv[Index + 3], &FileBuffer,=20 > + &FileSize, NULL); > if (Status =3D=3D STATUS_SUCCESS) { > if (FileSize >=3D 0x80000000) { > Error (NULL, 0, 0, "-O Parameter incorrect, FileSize too=20 > large!", NULL); @@ -1640,48 +1641,90 @@ Returns: > // Assume the file size should < 2G. > // > FileSize |=3D 0x80000000; > + Index +=3D 4; > + } else { > + if (Status =3D=3D STATUS_WARNING) { > + Error (NULL, 0, 0, "-O Parameter incorrect, Unable to open fil= e", argv[Index + 3]); > + } > + return 0; > + } > + } else { > + // > + // 2nd, try GUID > + // > + if (IsGuidData (argv[Index + 2], &Guid)) { > + FileBuffer =3D FindFileFromFvByGuid (FdBuffer, FdSize, &Guid, &F= ileSize); > + if (FileBuffer =3D=3D NULL) { > + Error (NULL, 0, 0, "-O Parameter incorrect, GUID not found!", = "%s", argv[Index + 2]); > + // not found > + return 0; > + } > + if (FileSize >=3D 0x80000000) { > + Error (NULL, 0, 0, "-O Parameter incorrect, FileSize too large= !", NULL); > + return 0; > + } > + FileBuffer =3D (UINT8 *)MEMORY_TO_FLASH (FileBuffer, FdBuffer,=20 > + FdSize); > Index +=3D 3; > } else { > // > - // 3rd, try > + // 3rd, try file > // > - if (Index + 3 >=3D argc) { > - break; > - } > - if ((strcmp (argv[Index + 2], "RESERVE") =3D=3D 0) || > - (strcmp (argv[Index + 2], "reserve") =3D=3D 0)) { > - FileSize =3D xtoi (argv[Index + 3]); > + Status =3D ReadInputFile (argv[Index + 2], &FileBuffer, &FileSiz= e, NULL); > + if (Status =3D=3D STATUS_SUCCESS) { > if (FileSize >=3D 0x80000000) { > Error (NULL, 0, 0, "-O Parameter incorrect, FileSize too=20 > large!", NULL); > + free (FileBuffer); > return 0; > } > - FileBuffer =3D malloc (FileSize); > - if (FileBuffer =3D=3D NULL) { > - Error (NULL, 0, 0, "No sufficient memory to allocate!", NULL= ); > - return 0; > - } > - SetMem (FileBuffer, FileSize, 0xFF); > // > // Set the most significant bit > // It means the data in memory, not in flash yet. > // Assume the file size should < 2G. > // > FileSize |=3D 0x80000000; > - Index +=3D 4; > + Index +=3D 3; > } else { > // > - // 4th, try > + // 4th, try > // > if (Index + 3 >=3D argc) { > break; > } > - FileBuffer =3D (UINT8 *) (UINTN) xtoi (argv[Index + 2]); > - FileSize =3D xtoi (argv[Index + 3]); > - if (FileSize >=3D 0x80000000) { > - Error (NULL, 0, 0, "-O Parameter incorrect, FileSize too lar= ge!", NULL); > - return 0; > + if ((strcmp (argv[Index + 2], "RESERVE") =3D=3D 0) || > + (strcmp (argv[Index + 2], "reserve") =3D=3D 0)) { > + FileSize =3D xtoi (argv[Index + 3]); > + if (FileSize >=3D 0x80000000) { > + Error (NULL, 0, 0, "-O Parameter incorrect, FileSize too l= arge!", NULL); > + return 0; > + } > + FileBuffer =3D malloc (FileSize); > + if (FileBuffer =3D=3D NULL) { > + Error (NULL, 0, 0, "No sufficient memory to allocate!", NU= LL); > + return 0; > + } > + SetMem (FileBuffer, FileSize, 0xFF); > + // > + // Set the most significant bit > + // It means the data in memory, not in flash yet. > + // Assume the file size should < 2G. > + // > + FileSize |=3D 0x80000000; > + Index +=3D 4; > + } else { > + // > + // 5th, try > + // > + if (Index + 3 >=3D argc) { > + break; > + } > + FileBuffer =3D (UINT8 *) (UINTN) xtoi (argv[Index + 2]); > + FileSize =3D xtoi (argv[Index + 3]); > + if (FileSize >=3D 0x80000000) { > + Error (NULL, 0, 0, "-O Parameter incorrect, FileSize too l= arge!", NULL); > + return 0; > + } > + Index +=3D 4; > } > - Index +=3D 4; > } > } > } > @@ -1691,6 +1734,9 @@ Returns: > return 0; > } > =20 > gFitTableContext.OptionalModule[gFitTableContext.OptionalModuleNumber] > .Type =3D Type; > + if (gFitTableContext.OptionalModule[gFitTableContext.OptionalModuleN= umber].Type =3D=3D FIT_TABLE_TYPE_CSE_SECURE_BOOT) { > + gFitTableContext.OptionalModule[gFitTableContext.OptionalModuleNum= ber].SubType =3D SubType; > + } > gFitTableContext.OptionalModule[gFitTableContext.OptionalModuleNumbe= r].Address =3D (UINT32) (UINTN) FileBuffer; > =20 > gFitTableContext.OptionalModule[gFitTableContext.OptionalModuleNumber] > .Buffer =3D FileBuffer; > =20 > gFitTableContext.OptionalModule[gFitTableContext.OptionalModuleNumber].Si= ze =3D FileSize; @@ -2055,6 +2101,23 @@ Returns: > return ; > } >=20 > +CHAR8 *mFitCseSubTypeStr[] =3D { > + "CSE_RSVD ", > + "CSE_K_HASH1", > + "CSE_M_HASH ", > + "CSE_BPOLICY", > + "CSE_OTHR_BP", > + "CSE_OEMSMIP", > + "CSE_MRCDATA", > + "CSE_IBBL_H ", > + "CSE_IBB_H ", > + "CSE_OEM_ID ", > + "CSEOEMSKUID", > + "CSE_BD_IND ", > + "CSE_FPM ", > + "CSE_ACMM " > +}; > + > CHAR8 *mFitTypeStr[] =3D { > " ", > "MICROCODE ", > @@ -2103,6 +2166,14 @@ Returns: > return mFitSignatureInHeader; > } > if (FitEntry->Type < sizeof (mFitTypeStr)/sizeof(mFitTypeStr[0])) { > + if (FitEntry->Type =3D=3D FIT_TABLE_TYPE_CSE_SECURE_BOOT) { > + // > + // "Reserved" field is used to distinguish CSE Secure Boot entries= (see FIT spec revision 1.2) > + // > + if (FitEntry->Rsvd < sizeof (mFitCseSubTypeStr)/sizeof(mFitCseSubT= ypeStr[0])) { > + return mFitCseSubTypeStr[FitEntry->Rsvd]; > + } > + } > return mFitTypeStr[FitEntry->Type]; > } else { > return " "; > @@ -2675,6 +2746,9 @@ Returns: > *(UINT32 *)&FitEntry[FitIndex].Size[0] =3D gFitTableContext.Optional= Module[Index].Size; > FitEntry[FitIndex].Version =3D (UINT16)gFitTableContext.= OptionalModule[Index].Version; > FitEntry[FitIndex].Type =3D (UINT8)gFitTableContext.O= ptionalModule[Index].Type; > + if (FitEntry[FitIndex].Type =3D=3D FIT_TABLE_TYPE_CSE_SECURE_BOOT) { > + FitEntry[FitIndex].Rsvd =3D (UINT8)gFitTableContext.O= ptionalModule[Index].SubType; > + } > FitEntry[FitIndex].C_V =3D 0; > FitEntry[FitIndex].Checksum =3D 0; > FitIndex++; > diff --git a/Silicon/Intel/Tools/FitGen/FitGen.h > b/Silicon/Intel/Tools/FitGen/FitGen.h > index cb9274b4175e..abad2d8799c8 100644 > --- a/Silicon/Intel/Tools/FitGen/FitGen.h > +++ b/Silicon/Intel/Tools/FitGen/FitGen.h > @@ -31,7 +31,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent //=20 > Utility version information // #define UTILITY_MAJOR_VERSION 0=20 > -#define UTILITY_MINOR_VERSION 61 > +#define UTILITY_MINOR_VERSION 62 > #define UTILITY_DATE __DATE__ >=20 > // > -- > 2.16.2.windows.1