From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by mx.groups.io with SMTP id smtpd.web11.8283.1578652887728686253 for ; Fri, 10 Jan 2020 02:41:27 -0800 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 192.55.52.115, mailfrom: ray.ni@intel.com) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga005.fm.intel.com ([10.253.24.32]) by fmsmga103.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 10 Jan 2020 02:41:27 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.69,416,1571727600"; d="scan'208";a="422075069" Received: from fmsmsx108.amr.corp.intel.com ([10.18.124.206]) by fmsmga005.fm.intel.com with ESMTP; 10 Jan 2020 02:41:27 -0800 Received: from fmsmsx114.amr.corp.intel.com (10.18.116.8) by FMSMSX108.amr.corp.intel.com (10.18.124.206) with Microsoft SMTP Server (TLS) id 14.3.439.0; Fri, 10 Jan 2020 02:41:27 -0800 Received: from shsmsx153.ccr.corp.intel.com (10.239.6.53) by FMSMSX114.amr.corp.intel.com (10.18.116.8) with Microsoft SMTP Server (TLS) id 14.3.439.0; Fri, 10 Jan 2020 02:41:26 -0800 Received: from shsmsx104.ccr.corp.intel.com ([169.254.5.197]) by SHSMSX153.ccr.corp.intel.com ([169.254.12.89]) with mapi id 14.03.0439.000; Fri, 10 Jan 2020 18:41:25 +0800 From: "Ni, Ray" To: "Kinney, Michael D" CC: "Dong, Eric" , "lersek@redhat.com" , "Gao, Liming" , "Fu, Siyuan" , "devel@edk2.groups.io" Subject: Re: [PATCH v2 1/2] MdePkg: Add header file for Firmware Interface Table specification. Thread-Topic: [PATCH v2 1/2] MdePkg: Add header file for Firmware Interface Table specification. Thread-Index: AQHVx2NfLpP3bSwTQkaIlKH469JOaafjtRTA Date: Fri, 10 Jan 2020 10:41:25 +0000 Message-ID: <734D49CCEBEEF84792F5B80ED585239D5C3E871D@SHSMSX104.ccr.corp.intel.com> References: <22247fcc9c3a4f5fe237ff2e9ec289fb41ceb3ec.1578535686.git.siyuan.fu@intel.com> <4A89E2EF3DFEDB4C8BFDE51014F606A14E58F6BF@SHSMSX104.ccr.corp.intel.com> In-Reply-To: <4A89E2EF3DFEDB4C8BFDE51014F606A14E58F6BF@SHSMSX104.ccr.corp.intel.com> Accept-Language: en-US, zh-CN X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Return-Path: ray.ni@intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Siyuan, FIT is Intel-only interface that's not consumed by other vendors. We may ne= ed to keep it in IntelSiliconPkg. I proposed this movement earlier in offline discussion with you but based o= n new discussion with Mike, I withdrew my proposal. Sorry about that! + Mike for additional comments. Thanks, Ray > -----Original Message----- > From: Gao, Liming > Sent: Friday, January 10, 2020 11:09 AM > To: Fu, Siyuan ; devel@edk2.groups.io > Cc: Kinney, Michael D ; Dong, Eric > ; Ni, Ray ; lersek@redhat.com > Subject: RE: [PATCH v2 1/2] MdePkg: Add header file for Firmware Interfac= e > Table specification. >=20 > Reviewed-by: Liming Gao >=20 > > -----Original Message----- > > From: Fu, Siyuan > > Sent: Thursday, January 9, 2020 10:14 AM > > To: devel@edk2.groups.io > > Cc: Kinney, Michael D ; Gao, Liming > ; Dong, Eric ; Ni, Ray > > ; lersek@redhat.com > > Subject: [PATCH v2 1/2] MdePkg: Add header file for Firmware Interface = Table > specification. > > > > This patch add FirmwareInterfaceTable.h for the Firmware Interface Tabl= e > > BIOS specification. > > > > This is to remove future edk2 dependency on edk2-platforms repo. The fi= le > > content comes from > > edk2-platforms\Silicon\Intel\IntelSiliconPkg\Include\IndustryStandard > > > > BZ link: https://tianocore.acgmultimedia.com/show_bug.cgi?id=3D2449 > > > > Cc: Michael D Kinney > > Cc: Liming Gao > > Signed-off-by: Siyuan Fu > > --- > > .../IndustryStandard/FirmwareInterfaceTable.h | 76 +++++++++++++++++++ > > 1 file changed, 76 insertions(+) > > create mode 100644 > MdePkg/Include/IndustryStandard/FirmwareInterfaceTable.h > > > > diff --git a/MdePkg/Include/IndustryStandard/FirmwareInterfaceTable.h > b/MdePkg/Include/IndustryStandard/FirmwareInterfaceTable.h > > new file mode 100644 > > index 0000000000..be3e34ae1b > > --- /dev/null > > +++ b/MdePkg/Include/IndustryStandard/FirmwareInterfaceTable.h > > @@ -0,0 +1,76 @@ > > +/** @file > > + Industry Standard Definitions of Firmware Interface Table BIOS Speci= fication > 1.0. > > + > > + Copyright (c) 2016 - 2020, Intel Corporation. All rights reserved. > > + SPDX-License-Identifier: BSD-2-Clause-Patent > > + > > +**/ > > + > > +#ifndef __FIRMWARE_INTERFACE_TABLE_H__ > > +#define __FIRMWARE_INTERFACE_TABLE_H__ > > + > > +// > > +// FIT Entry type definitions > > +// > > +#define FIT_TYPE_00_HEADER 0x00 > > +#define FIT_TYPE_01_MICROCODE 0x01 > > +#define FIT_TYPE_02_STARTUP_ACM 0x02 > > +#define FIT_TYPE_07_BIOS_STARTUP_MODULE 0x07 > > +#define FIT_TYPE_08_TPM_POLICY 0x08 > > +#define FIT_TYPE_09_BIOS_POLICY 0x09 > > +#define FIT_TYPE_0A_TXT_POLICY 0x0A > > +#define FIT_TYPE_0B_KEY_MANIFEST 0x0B > > +#define FIT_TYPE_0C_BOOT_POLICY_MANIFEST 0x0C > > +#define FIT_TYPE_10_CSE_SECURE_BOOT 0x10 > > +#define FIT_TYPE_2D_TXTSX_POLICY 0x2D > > +#define FIT_TYPE_2F_JMP_DEBUG_POLICY 0x2F > > +#define FIT_TYPE_7F_SKIP 0x7F > > + > > +#define FIT_POINTER_ADDRESS 0xFFFFFFC0 ///< Fixed addr= ess at 4G > - 40h > > + > > +#define FIT_TYPE_VERSION 0x0100 > > + > > +#define FIT_TYPE_00_SIGNATURE SIGNATURE_64 ('_', 'F', 'I', 'T', '_', = ' ', ' ', ' ') > > + > > +#pragma pack(1) > > + > > +typedef struct { > > + // > > + // Address is the base address of the firmware component > > + // must be aligned on 16 byte boundary > > + // > > + UINT64 Address; > > + // > > + // Size is the span of the component in multiple of 16 bytes > > + // > > + UINT8 Size[3]; > > + // > > + // Reserved must be set to 0 > > + // > > + UINT8 Reserved; > > + // > > + // Component's version number in binary coded decimal (BCD) format. > > + // For the FIT header entry, the value in this field will indicate t= he revision > > + // number of the FIT data structure. The upper byte of the revision = field > > + // indicates the major revision and the lower byte indicates the min= or > revision. > > + // > > + UINT16 Version; > > + // > > + // FIT types 0x00 to 0x7F > > + // > > + UINT8 Type : 7; > > + // > > + // Checksum Valid indicates whether component has valid checksum. > > + // > > + UINT8 C_V : 1; > > + // > > + // Component's checksum. The modulo sum of all the bytes in the > component and > > + // the value in this field (Chksum) must add up to zero. This field = is only > > + // valid if the C_V flag is non-zero. > > + // > > + UINT8 Chksum; > > +} FIRMWARE_INTERFACE_TABLE_ENTRY; > > + > > +#pragma pack() > > + > > +#endif > > -- > > 2.19.1.windows.1