From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id E7B7E817B2 for ; Mon, 9 Jan 2017 01:19:36 -0800 (PST) Received: from fmsmga006.fm.intel.com ([10.253.24.20]) by orsmga101.jf.intel.com with ESMTP; 09 Jan 2017 01:19:36 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.33,339,1477983600"; d="scan'208";a="51623943" Received: from shzintpr01.sh.intel.com (HELO [10.253.24.20]) ([10.239.4.80]) by fmsmga006.fm.intel.com with ESMTP; 09 Jan 2017 01:19:35 -0800 To: Linson Augustine , edk2-devel@lists.01.org References: <20170109085859.7732-1-linson.augustine@hpe.com> Cc: michael.d.kinney@intel.com, liming.gao@intel.com, star.zeng@intel.com From: "Zeng, Star" Message-ID: <21cdd05a-a9ef-8b8f-01ec-a6caf7e92157@intel.com> Date: Mon, 9 Jan 2017 17:19:05 +0800 User-Agent: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.5.1 MIME-Version: 1.0 In-Reply-To: <20170109085859.7732-1-linson.augustine@hpe.com> Subject: Re: [PATCH v1 1/1] MdePkg/SmBios.h: Add new defines for SMBIOS record type 43 X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 09 Jan 2017 09:19:37 -0000 Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Hi Linson, On 2017/1/9 16:58, Linson Augustine wrote: > Added defintions for the new SMBIOS Type 43 record. Typo "defintions" should be "definitions". I noticed that the new type 43 is added in new SMBIOS spec 3.1.0, but there are also other definitions and structure fields added in new SMBIOS spec 3.1.0, will you plan to add them too? Thanks, Star > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Augustine Linson P > --- > MdePkg/Include/IndustryStandard/SmBios.h | 19 ++++++++++++++++++- > 1 file changed, 18 insertions(+), 1 deletion(-) > > diff --git a/MdePkg/Include/IndustryStandard/SmBios.h b/MdePkg/Include/IndustryStandard/SmBios.h > index c66422fc3a..debc85af6e 100644 > --- a/MdePkg/Include/IndustryStandard/SmBios.h > +++ b/MdePkg/Include/IndustryStandard/SmBios.h > @@ -2,7 +2,7 @@ > Industry Standard Definitions of SMBIOS Table Specification v3.0.0. > > Copyright (c) 2006 - 2015, Intel Corporation. All rights reserved.
> -(C) Copyright 2015 Hewlett Packard Enterprise Development LP
> +(C) Copyright 2015-2017 Hewlett Packard Enterprise Development LP
> This program and the accompanying materials are licensed and made available under > the terms and conditions of the BSD License that accompanies this distribution. > The full text of the license may be found at > @@ -97,6 +97,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. > #define SMBIOS_TYPE_ADDITIONAL_INFORMATION 40 > #define SMBIOS_TYPE_ONBOARD_DEVICES_EXTENDED_INFORMATION 41 > #define SMBIOS_TYPE_MANAGEMENT_CONTROLLER_HOST_INTERFACE 42 > +#define SMBIOS_TYPE_TPM_DEVICE 43 > > /// > /// Inactive type is added from SMBIOS 2.2. Reference SMBIOS 2.6, chapter 3.3.43. > @@ -2359,6 +2360,21 @@ typedef struct { > } SMBIOS_TABLE_TYPE42; > > /// > +/// TPM Device (Type 43). > +/// > +typedef struct { > + SMBIOS_STRUCTURE Hdr; > + UINT8 VendorID[4]; > + UINT8 MajorSpecVersion; > + UINT8 MinorSpecVersion; > + UINT32 FirmwareVersion1; > + UINT32 FirmwareVersion2; > + UINT8 Description; > + UINT64 Characteristics; > + UINT32 OemDefined; > +} SMBIOS_TABLE_TYPE43; > + > +/// > /// Inactive (Type 126) > /// > typedef struct { > @@ -2420,6 +2436,7 @@ typedef union { > SMBIOS_TABLE_TYPE40 *Type40; > SMBIOS_TABLE_TYPE41 *Type41; > SMBIOS_TABLE_TYPE42 *Type42; > + SMBIOS_TABLE_TYPE43 *Type43; > SMBIOS_TABLE_TYPE126 *Type126; > SMBIOS_TABLE_TYPE127 *Type127; > UINT8 *Raw; >