From: "Thotala, Gopi" <gopi.thotala@intel.com>
To: Rebecca Cran <rebecca@bsdio.com>,
"devel@edk2.groups.io" <devel@edk2.groups.io>
Subject: Re: [edk2-devel] [PATCH] MdePkg/Include: Smbios Specification 3.4.0 changes
Date: Mon, 14 Jun 2021 15:39:37 +0000 [thread overview]
Message-ID: <MWHPR1101MB23363067F31EBAF55F8674819B319@MWHPR1101MB2336.namprd11.prod.outlook.com> (raw)
In-Reply-To: <B220D3D2-19D6-4B10-BB3C-ACE19EA2BAE0@bsdio.com>
[-- Attachment #1.1: Type: text/plain, Size: 596 bytes --]
Attached V2 patch after typo correction.
Thanks
Gopi
From: Rebecca Cran <rebecca@bsdio.com>
Sent: Sunday, June 13, 2021 9:34 AM
To: devel@edk2.groups.io; Thotala, Gopi <gopi.thotala@intel.com>
Subject: Re: [edk2-devel] [PATCH] MdePkg/Include: Smbios Specification 3.4.0 changes
There’s a typo of ‘persistent’ in:
// Optane DC Presistent Memory in SMBIOS spec 3.4.0
Rebecca Cran
On Jun 2, 2021, at 10:46 AM, Thotala, Gopi <gopi.thotala@intel.com<mailto:gopi.thotala@intel.com>> wrote:
Initial patch submitted for review.
<MdePkg-Include-Smbios-Specification-3.4.0-changes.patch>
[-- Attachment #1.2: Type: text/html, Size: 3494 bytes --]
[-- Attachment #2: V2-MdePkg-Include-Smbios-Specification-3.4.0-changes.patch --]
[-- Type: application/octet-stream, Size: 4021 bytes --]
From b588e209f7b0666c4bcd2d731669e2675428e06e Mon Sep 17 00:00:00 2001
From: gthotala <gopi.thotala@intel.com>
Date: Wed, 2 Jun 2021 08:46:58 -0700
Subject: [PATCH] MdePkg/Include: Smbios Specification 3.4.0 changes
Included new entries in Smbios type9 table, modified existing entries
in Smbios type16 and 17 tables as per Smbios specs 3.4.0
Signed-off-by: Thotala Gopi <gopi.thotala@intel.com>
---
MdePkg/Include/IndustryStandard/SmBios.h | 30 ++++++++++++-------
.../SmbiosView/QueryTable.c | 8 ++---
2 files changed, 24 insertions(+), 14 deletions(-)
diff --git a/MdePkg/Include/IndustryStandard/SmBios.h b/MdePkg/Include/IndustryStandard/SmBios.h
index cc023b7369..6918f58cce 100644
--- a/MdePkg/Include/IndustryStandard/SmBios.h
+++ b/MdePkg/Include/IndustryStandard/SmBios.h
@@ -1393,7 +1393,10 @@ typedef struct {
UINT8 HotPlugDevicesSupported :1;
UINT8 SmbusSignalSupported :1;
UINT8 BifurcationSupported :1;
- UINT8 Reserved :4; ///< Set to 0.
+ UINT8 AsyncSurpriseRemoval :1;
+ UINT8 FlexbusSlotCxl10Capable :1;
+ UINT8 FlexbusSlotCxl20Capable :1;
+ UINT8 Reserved :1; ///< Set to 0.
} MISC_SLOT_CHARACTERISTICS2;
///
@@ -1435,6 +1438,12 @@ typedef struct {
UINT8 DataBusWidth;
UINT8 PeerGroupingCount;
MISC_SLOT_PEER_GROUP PeerGroups[1];
+ //
+ // Add for smbios 3.4
+ //
+ UINT8 SlotInformation;
+ UINT8 SlotPhysicalWidth;
+ UINT16 SlotPitch;
} SMBIOS_TABLE_TYPE9;
///
@@ -1628,7 +1637,7 @@ typedef enum {
MemoryArrayLocationPc98C24AddonCard = 0xA1,
MemoryArrayLocationPc98EAddonCard = 0xA2,
MemoryArrayLocationPc98LocalBusAddonCard = 0xA3,
- MemoryArrayLocationCXLFlexbus10AddonCard = 0xA4
+ MemoryArrayLocationCXLAddonCard = 0xA4
} MEMORY_ARRAY_LOCATION;
///
@@ -1763,17 +1772,18 @@ typedef struct {
/// Memory Device - Memory Technology
///
typedef enum {
- MemoryTechnologyOther = 0x01,
- MemoryTechnologyUnknown = 0x02,
- MemoryTechnologyDram = 0x03,
- MemoryTechnologyNvdimmN = 0x04,
- MemoryTechnologyNvdimmF = 0x05,
- MemoryTechnologyNvdimmP = 0x06,
+ MemoryTechnologyOther = 0x01,
+ MemoryTechnologyUnknown = 0x02,
+ MemoryTechnologyDram = 0x03,
+ MemoryTechnologyNvdimmN = 0x04,
+ MemoryTechnologyNvdimmF = 0x05,
+ MemoryTechnologyNvdimmP = 0x06,
//
// This definition is updated to represent Intel
- // Optane DC Presistent Memory in SMBIOS spec 3.3.0
+ // Optane DC Persistent Memory in SMBIOS spec 3.4.0
//
- MemoryTechnologyIntelPersistentMemory = 0x07
+ MemoryTechnologyIntelOptanePersistentMemory = 0x07
+
} MEMORY_DEVICE_TECHNOLOGY;
///
diff --git a/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/QueryTable.c b/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/QueryTable.c
index 39a3e3c089..7fc9d38a3b 100644
--- a/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/QueryTable.c
+++ b/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/QueryTable.c
@@ -2300,8 +2300,8 @@ TABLE_ITEM PMALocationTable[] = {
L" PC-98/Local bus add-on card"
},
{
- MemoryArrayLocationCXLFlexbus10AddonCard,
- L" CXL Flexbus 1.0 add-on card"
+ MemoryArrayLocationCXLAddonCard,
+ L" CXL add-on card"
}
};
@@ -2650,8 +2650,8 @@ TABLE_ITEM MemoryDeviceMemoryTechnologyTable[] = {
L" NVDIMM-P"
},
{
- MemoryTechnologyIntelPersistentMemory,
- L" Intel Optane DC Persistent Memory"
+ MemoryTechnologyIntelOptanePersistentMemory,
+ L" Intel Optane Persistent Memory"
}
};
--
2.18.0.windows.1
next prev parent reply other threads:[~2021-06-14 15:39 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-06-02 16:24 [edk2-devel] [PATCH] MdePkg/Include: Smbios Specification 3.4.0 changes Thotala, Gopi
2021-06-03 1:00 ` 回复: " gaoliming
2021-06-13 16:33 ` Rebecca Cran
2021-06-14 15:39 ` Thotala, Gopi [this message]
2021-06-17 2:10 ` 回复: " gaoliming
2021-06-17 3:53 ` Thotala, Gopi
2021-06-21 1:11 ` 回复: " gaoliming
2021-07-08 15:34 ` Thotala, Gopi
2021-07-09 1:00 ` 回复: " gaoliming
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=MWHPR1101MB23363067F31EBAF55F8674819B319@MWHPR1101MB2336.namprd11.prod.outlook.com \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox