From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga17.intel.com (mga17.intel.com [192.55.52.151]) by mx.groups.io with SMTP id smtpd.web12.14961.1591194679331986348 for ; Wed, 03 Jun 2020 07:31:19 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@intel.onmicrosoft.com header.s=selector2-intel-onmicrosoft-com header.b=q+KyvhbW; spf=pass (domain: intel.com, ip: 192.55.52.151, mailfrom: liming.gao@intel.com) IronPort-SDR: 0PmbqqJKwnOcP3+BrEywP82S2HZnDowij42+MF+6zzELOs3qPgaXjYA+JqW+aMcwCq/2nbY/5G AmGJC4iPMmsA== X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga006.fm.intel.com ([10.253.24.20]) by fmsmga107.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 03 Jun 2020 07:31:17 -0700 IronPort-SDR: 3wjKEHwn7bgxh8jpLOJcc7vIqeW7meKIoKSLF54CCbJrX+jQTaNdoHfMw8u/xVGyciRIxjfO6H 4AhsWco8zz5g== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.73,467,1583222400"; d="scan'208";a="471088527" Received: from orsmsx106.amr.corp.intel.com ([10.22.225.133]) by fmsmga006.fm.intel.com with ESMTP; 03 Jun 2020 07:31:16 -0700 Received: from orsmsx601.amr.corp.intel.com (10.22.229.14) by ORSMSX106.amr.corp.intel.com (10.22.225.133) with Microsoft SMTP Server (TLS) id 14.3.439.0; Wed, 3 Jun 2020 07:31:16 -0700 Received: from orsmsx610.amr.corp.intel.com (10.22.229.23) by ORSMSX601.amr.corp.intel.com (10.22.229.14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.1713.5; Wed, 3 Jun 2020 07:31:16 -0700 Received: from ORSEDG001.ED.cps.intel.com (10.7.248.4) by orsmsx610.amr.corp.intel.com (10.22.229.23) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256) id 15.1.1713.5 via Frontend Transport; Wed, 3 Jun 2020 07:31:16 -0700 Received: from NAM02-CY1-obe.outbound.protection.outlook.com (104.47.37.59) by edgegateway.intel.com (134.134.137.100) with Microsoft SMTP Server (TLS) id 14.3.439.0; Wed, 3 Jun 2020 07:31:15 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=SnzsywpV3VKeq5mwsnN5z1Hv45rcoqf8m9PnVsg1TAIHubmG35OLr4w0HRsky8DJ2gGfXG5BU+4S/8o+LAGEluhoRCCoIJ6jSCJ/KqzTTsYTWko5Qk+VUmWoay46RUdCXLo36h4pkIS/J+Ey3KXLEF2YWA2Hla5Gmnf1UCplnJT0xsla5Di3gXcNJKkNK0+hshtHKLKWJ8dCWdnPx5woG0VQ1RR3j1jjaDmE1m3rwxpl9dGDEzSjbKwXaorOa75PwvXlnNl/LbLhxkyrgSN+pgQyzJa54YAweh5XemdGzEXIONdttG2T5DR8Q0H8p+xdBpDZeZwKEuojzxjmBwYgwQ== 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=2rlx9RTaCRTG2u0dy3our5urXdd4oMUcZ53UUbq7zIE=; b=FNuJAQdibdhwRFiffE+hkj+n8s1Sa2N2bwJS+rsTX1izLE6PMi+h/Em4wdmuTl+i3u0237j39KCOwKLEUPVe3nhWctj4rGmcS/4bDa+3V8jy60Oa2dbh/k/yRivSDaA0LiiLIY82arNhrFbv1rxM/vf56a1QqSDAnOpchdK1crEVkgHMOBUatq+ptiQXEZtgYDtWgyQXwHs0YYlk7uYXLWCx1l1r0kI7FzpS7IPQj8ndiGzT2DucYhj/hATa+RSF062ElwXqXk3/mzUaCK79BhRNm7osaEmw1EnOPNI4ZBNltdl9ayykg4uTN2swAnpEKykfT9/IrghH4CTRpCBDMw== 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=2rlx9RTaCRTG2u0dy3our5urXdd4oMUcZ53UUbq7zIE=; b=q+KyvhbWrusLiZvKRLMcBi4PSA4uMH6j8i+AL75aBOoi219lNs7fH83pLLbg/OpbCEZNX6jWUVC/NO6yWvgpVjyRLlREQ6T62h7r8nLl2SEcy5+5eKqlOr9L0UhDFHqx/rxmaW7xiyoNRDobwZiHep0m2p88z2CGqsNuzYMU3po= Received: from MWHPR11MB1630.namprd11.prod.outlook.com (2603:10b6:301:e::7) by MWHPR11MB1246.namprd11.prod.outlook.com (2603:10b6:300:29::15) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3045.21; Wed, 3 Jun 2020 14:31:14 +0000 Received: from MWHPR11MB1630.namprd11.prod.outlook.com ([fe80::50fb:d63c:5958:a67e]) by MWHPR11MB1630.namprd11.prod.outlook.com ([fe80::50fb:d63c:5958:a67e%7]) with mapi id 15.20.3045.022; Wed, 3 Jun 2020 14:31:14 +0000 From: "Liming Gao" To: "Hernandez Beltran, Jorge" , "devel@edk2.groups.io" CC: "Lohr, Paul A" 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: AQHWNQ7dzHnTprhwP06kAxy/lE/dBajG/KcA Date: Wed, 3 Jun 2020 14:31:14 +0000 Message-ID: References: <20200528164102.31576-1-jorge.hernandez.beltran@intel.com> In-Reply-To: <20200528164102.31576-1-jorge.hernandez.beltran@intel.com> Accept-Language: en-US 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: [192.55.52.204] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 68b41f78-c0d1-4275-0ab7-08d807cac4f9 x-ms-traffictypediagnostic: MWHPR11MB1246: x-ms-exchange-transport-forked: True x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:80; x-forefront-prvs: 04238CD941 x-ms-exchange-senderadcheck: 1 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: i59rCwYsfCp1FqE/6zzK5/NmtSu1o9O5IXk/nvgFQCBXTp7LdKHOyfRHtc8MIMkwXntA5Mpj2ist09E7UexjG5gYH4ofF9N6UBGzpBf8Z3C1Ed+3x1hxIQQtK8AngdvVCEPfGkjhkHr06PHoFWBwhSN/xIYndYvoOD7i7LxXNlq0Ndf8+1GQb5YoLnd6KXaJY+9hF7DvI4zEtj5QkomxeYLxsSGNAxQiTPj01L458BQZm5Z0Js9NZEWGw8RtWqELAtbBg1BILEPMqBWcDFecrBQZzan+C3080LdjoFrthX/4/EyxQOwj+I9gbyCpCulxT7nkiYH+oX/hwgnnoj15WFrOCFim0CNoGX1aywMUdI1t67wcBV7FK42sGvLRWo6yppzVWot9qjCL6hq26tU/tw== x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:MWHPR11MB1630.namprd11.prod.outlook.com;PTR:;CAT:NONE;SFTY:;SFS:(396003)(39860400002)(366004)(376002)(346002)(136003)(186003)(478600001)(110136005)(55016002)(86362001)(7696005)(966005)(30864003)(2906002)(52536014)(33656002)(26005)(66446008)(6506007)(83380400001)(5660300002)(8936002)(8676002)(4326008)(66556008)(66946007)(316002)(71200400001)(66476007)(107886003)(64756008)(76116006)(15650500001)(9686003)(53546011);DIR:OUT;SFP:1102; x-ms-exchange-antispam-messagedata: NQrHPOGh7VFgdn8RwHvFkywEr1UUrXrq6cX9aNwkkwlm4401MDCA1AUpFRBRNkINKrtDmGGA540KOIbVewAWp8W3j5fN/iAA9FBaqnmYH49ldmeeDZmAYgCIwMHD09bvMZkqATsKZelovVOJ+XuxfE7QZUjjShXJqiCsyLSo9LY6HZrh/mSNueo/O3zFGXgbLnjkuqQ9qsUDgBQjVaqzLKP4iooPahku8ecoYg/tacvaAGRI21Vn6RaYdGm4IfXVzA2/zuz7nfxe9T3jsAYvc8blyjEoACr+QoXUbyWt6kHKZfPLpVIe9xgL/Z49KSCyJUqQTt2W832sVciuwI76Gxjl7hg2CZh4onDID/4YVr/mY2PVHHjwJ3s9ne2kexAQhzNPNfPQMcZqEhzRDr+/zHHL5eyCOulkNI7mBw51ecAQEr/ALo7V9Onc2E6rd/6/S2wAcWgmdupzKqqZw44hWLEF6twCDhFrv2QHKmvcYULqpuInWnJT6h3nygIopOo9 MIME-Version: 1.0 X-MS-Exchange-CrossTenant-Network-Message-Id: 68b41f78-c0d1-4275-0ab7-08d807cac4f9 X-MS-Exchange-CrossTenant-originalarrivaltime: 03 Jun 2020 14:31:14.0869 (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: DFzVmWysme1D62O5Ma6TjuCjYEDPP3xTM3v19nPRtgCzX/B2x0JKePt1j/sNdC29yHPpyjCSNffRRn6+qJ/ynw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: MWHPR11MB1246 Return-Path: liming.gao@intel.com X-OriginatorOrg: intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable 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 ; Hernandez Beltran, Jorge > > Subject: [Patch 1/1] [Edk2Platforms] FitGen: Update FitGen tool to be com= pliance 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 > --- > 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/Fi= tGen/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 > @@ -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 revis= ion 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 | MicrocodeRegionSize>|] [-V ]\n" > - "\t[-O RecordType ||| [-V > ]] [-O ... [-V ...]]\n" > + "\t[-O RecordType | 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 ta= ble entries. 0x%04x is used if this is not specified.\n", > DEFAULT_FIT_ENTRY_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, &FileSize,= NULL); > if (Status =3D=3D STATUS_SUCCESS) { > if (FileSize >=3D 0x80000000) { > Error (NULL, 0, 0, "-O Parameter incorrect, FileSize too 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, FdS= ize); > 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 lar= ge!", 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; > } > gFitTableContext.OptionalModule[gFitTableContext.OptionalModuleNumbe= r].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; > gFitTableContext.OptionalModule[gFitTableContext.OptionalModuleNumbe= r].Buffer =3D FileBuffer; > gFitTableContext.OptionalModule[gFitTableContext.OptionalModuleNumbe= r].Size =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/Fi= tGen/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 > // Utility version information > // > #define UTILITY_MAJOR_VERSION 0 > -#define UTILITY_MINOR_VERSION 61 > +#define UTILITY_MINOR_VERSION 62 > #define UTILITY_DATE __DATE__ >=20 > // > -- > 2.16.2.windows.1