* Re: [edk2-devel] [PATCH] MdePkg,ShellPkg: Add ProcessorUpgradeSocketLGA4677 from SMBIOS 3.5.0
[not found] <16A8DA167D1CA920.13648@groups.io>
@ 2021-09-28 2:10 ` Zeng, Star
2021-09-28 2:11 ` Ni, Ray
0 siblings, 1 reply; 3+ messages in thread
From: Zeng, Star @ 2021-09-28 2:10 UTC (permalink / raw)
To: devel@edk2.groups.io, Zeng, Star
Cc: gaoliming@byosoft.com.cn, Kinney, Michael D, Liu, Zhiguang,
Ni, Ray, Gao, Zhichao
+ Maintainers and Reviewers
-----Original Message-----
From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Zeng, Star
Sent: 2021年9月28日 10:04
To: devel@edk2.groups.io
Cc: Zeng, Star <star.zeng@intel.com>
Subject: [edk2-devel] [PATCH] MdePkg,ShellPkg: Add ProcessorUpgradeSocketLGA4677 from SMBIOS 3.5.0
This patch adds ProcessorUpgradeSocketLGA4677 definition into Smbios.h from SMBIOS 3.5.0 and entry into QueryTable.c.
It also adds ProcessorUpgradeSocketLGA4189 and ProcessorUpgradeSocketLGA1200 into from SMBIOS 3.4.0 and entries into QueryTable.c.
Signed-off-by: Star Zeng <star.zeng@intel.com>
---
MdePkg/Include/IndustryStandard/SmBios.h | 7 +++++--
.../SmbiosView/QueryTable.c | 14 +++++++++++++-
2 files changed, 18 insertions(+), 3 deletions(-)
diff --git a/MdePkg/Include/IndustryStandard/SmBios.h b/MdePkg/Include/IndustryStandard/SmBios.h
index 6918f58cce44..2c2b32b8d462 100644
--- a/MdePkg/Include/IndustryStandard/SmBios.h
+++ b/MdePkg/Include/IndustryStandard/SmBios.h
@@ -1,7 +1,7 @@
/** @file Industry Standard Definitions of SMBIOS Table Specification v3.3.0. -Copyright (c) 2006 - 2019, Intel Corporation. All rights reserved.<BR>+Copyright (c) 2006 - 2021, Intel Corporation. All rights reserved.<BR> (C) Copyright 2015-2017 Hewlett Packard Enterprise Development LP<BR> (C) Copyright 2015 - 2019 Hewlett Packard Enterprise Development LP<BR> SPDX-License-Identifier: BSD-2-Clause-Patent@@ -810,7 +810,10 @@ typedef enum {
ProcessorUpgradeSocketLGA2066 = 0x39, ProcessorUpgradeSocketBGA1392 = 0x3A, ProcessorUpgradeSocketBGA1510 = 0x3B,- ProcessorUpgradeSocketBGA1528 = 0x3C+ ProcessorUpgradeSocketBGA1528 = 0x3C,+ ProcessorUpgradeSocketLGA4189 = 0x3D,+ ProcessorUpgradeSocketLGA1200 = 0x3E,+ ProcessorUpgradeSocketLGA4677 = 0x3F } PROCESSOR_UPGRADE; ///diff --git a/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/QueryTable.c b/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/QueryTable.c
index 7fc9d38a3b03..c312a7f8f227 100644
--- a/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/QueryTable.c
+++ b/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/QueryTable.
+++ c
@@ -2,7 +2,7 @@
Build a table, each item is (Key, Info) pair. And give a interface of query a string out of a table. - Copyright (c) 2005 - 2019, Intel Corporation. All rights reserved.<BR>+ Copyright (c) 2005 - 2021, Intel Corporation. All rights reserved.<BR> (C) Copyright 2016-2019 Hewlett Packard Enterprise Development LP<BR> SPDX-License-Identifier: BSD-2-Clause-Patent @@ -589,6 +589,18 @@ TABLE_ITEM ProcessorUpgradeTable[] = {
{ 0x3C, L"Socket BGA1528"+ },+ {+ 0x3D,+ L"Socket LGA4189"+ },+ {+ 0x3E,+ L"Socket LGA1200"+ },+ {+ 0x3F,+ L"Socket LGA4677" } }; --
2.27.0.windows.1
-=-=-=-=-=-=
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#81189): https://edk2.groups.io/g/devel/message/81189
Mute This Topic: https://groups.io/mt/85916590/1779220
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [star.zeng@intel.com] -=-=-=-=-=-=
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [edk2-devel] [PATCH] MdePkg,ShellPkg: Add ProcessorUpgradeSocketLGA4677 from SMBIOS 3.5.0
2021-09-28 2:10 ` [edk2-devel] [PATCH] MdePkg,ShellPkg: Add ProcessorUpgradeSocketLGA4677 from SMBIOS 3.5.0 Zeng, Star
@ 2021-09-28 2:11 ` Ni, Ray
2021-09-28 2:39 ` Zeng, Star
0 siblings, 1 reply; 3+ messages in thread
From: Ni, Ray @ 2021-09-28 2:11 UTC (permalink / raw)
To: Zeng, Star, devel@edk2.groups.io
Cc: gaoliming@byosoft.com.cn, Kinney, Michael D, Liu, Zhiguang,
Gao, Zhichao
Star,
It might be better to split the patch to two patches.
one is to change MdePkg adding the definitions.
The other is to change ShellPkg consuming the definitions.
> -----Original Message-----
> From: Zeng, Star <star.zeng@intel.com>
> Sent: Tuesday, September 28, 2021 10:11 AM
> To: devel@edk2.groups.io; Zeng, Star <star.zeng@intel.com>
> Cc: gaoliming@byosoft.com.cn; Kinney, Michael D <michael.d.kinney@intel.com>; Liu, Zhiguang <zhiguang.liu@intel.com>; Ni,
> Ray <ray.ni@intel.com>; Gao, Zhichao <zhichao.gao@intel.com>
> Subject: RE: [edk2-devel] [PATCH] MdePkg,ShellPkg: Add ProcessorUpgradeSocketLGA4677 from SMBIOS 3.5.0
>
> + Maintainers and Reviewers
>
> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Zeng, Star
> Sent: 2021年9月28日 10:04
> To: devel@edk2.groups.io
> Cc: Zeng, Star <star.zeng@intel.com>
> Subject: [edk2-devel] [PATCH] MdePkg,ShellPkg: Add ProcessorUpgradeSocketLGA4677 from SMBIOS 3.5.0
>
> This patch adds ProcessorUpgradeSocketLGA4677 definition into Smbios.h from SMBIOS 3.5.0 and entry into QueryTable.c.
> It also adds ProcessorUpgradeSocketLGA4189 and ProcessorUpgradeSocketLGA1200 into from SMBIOS 3.4.0 and entries into
> QueryTable.c.
>
> Signed-off-by: Star Zeng <star.zeng@intel.com>
> ---
> MdePkg/Include/IndustryStandard/SmBios.h | 7 +++++--
> .../SmbiosView/QueryTable.c | 14 +++++++++++++-
> 2 files changed, 18 insertions(+), 3 deletions(-)
>
> diff --git a/MdePkg/Include/IndustryStandard/SmBios.h b/MdePkg/Include/IndustryStandard/SmBios.h
> index 6918f58cce44..2c2b32b8d462 100644
> --- a/MdePkg/Include/IndustryStandard/SmBios.h
> +++ b/MdePkg/Include/IndustryStandard/SmBios.h
> @@ -1,7 +1,7 @@
> /** @file Industry Standard Definitions of SMBIOS Table Specification v3.3.0. -Copyright (c) 2006 - 2019, Intel Corporation. All
> rights reserved.<BR>+Copyright (c) 2006 - 2021, Intel Corporation. All rights reserved.<BR> (C) Copyright 2015-2017 Hewlett
> Packard Enterprise Development LP<BR> (C) Copyright 2015 - 2019 Hewlett Packard Enterprise Development LP<BR> SPDX-
> License-Identifier: BSD-2-Clause-Patent@@ -810,7 +810,10 @@ typedef enum {
> ProcessorUpgradeSocketLGA2066 = 0x39, ProcessorUpgradeSocketBGA1392 = 0x3A, ProcessorUpgradeSocketBGA1510
> = 0x3B,- ProcessorUpgradeSocketBGA1528 = 0x3C+ ProcessorUpgradeSocketBGA1528 = 0x3C,+
> ProcessorUpgradeSocketLGA4189 = 0x3D,+ ProcessorUpgradeSocketLGA1200 = 0x3E,+ ProcessorUpgradeSocketLGA4677 =
> 0x3F } PROCESSOR_UPGRADE; ///diff --git a/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/QueryTable.c
> b/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/QueryTable.c
> index 7fc9d38a3b03..c312a7f8f227 100644
> --- a/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/QueryTable.c
> +++ b/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/QueryTable.
> +++ c
> @@ -2,7 +2,7 @@
> Build a table, each item is (Key, Info) pair. And give a interface of query a string out of a table. - Copyright (c) 2005 - 2019,
> Intel Corporation. All rights reserved.<BR>+ Copyright (c) 2005 - 2021, Intel Corporation. All rights reserved.<BR> (C) Copyright
> 2016-2019 Hewlett Packard Enterprise Development LP<BR> SPDX-License-Identifier: BSD-2-Clause-Patent @@ -589,6 +589,18
> @@ TABLE_ITEM ProcessorUpgradeTable[] = {
> { 0x3C, L"Socket BGA1528"+ },+ {+ 0x3D,+ L"Socket LGA4189"+ },+ {+ 0x3E,+ L"Socket LGA1200"+ },+ {+ 0x3F,+
> L"Socket LGA4677" } }; --
> 2.27.0.windows.1
>
>
>
> -=-=-=-=-=-=
> Groups.io Links: You receive all messages sent to this group.
> View/Reply Online (#81189): https://edk2.groups.io/g/devel/message/81189
> Mute This Topic: https://groups.io/mt/85916590/1779220
> Group Owner: devel+owner@edk2.groups.io
> Unsubscribe: https://edk2.groups.io/g/devel/unsub [star.zeng@intel.com] -=-=-=-=-=-=
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [edk2-devel] [PATCH] MdePkg,ShellPkg: Add ProcessorUpgradeSocketLGA4677 from SMBIOS 3.5.0
2021-09-28 2:11 ` Ni, Ray
@ 2021-09-28 2:39 ` Zeng, Star
0 siblings, 0 replies; 3+ messages in thread
From: Zeng, Star @ 2021-09-28 2:39 UTC (permalink / raw)
To: Ni, Ray, devel@edk2.groups.io
Cc: gaoliming@byosoft.com.cn, Kinney, Michael D, Liu, Zhiguang,
Gao, Zhichao
Good comment.
Saw 0db89a661f38b10012ff4f62e1853bfc48efd462 does so for both MdePkg and ShellPkg, but that is different for fixing typo which must change both MdePkg and ShellPkg in same patch.
Please check V2.
Thanks,
Star
-----Original Message-----
From: Ni, Ray <ray.ni@intel.com>
Sent: 2021年9月28日 10:12
To: Zeng, Star <star.zeng@intel.com>; devel@edk2.groups.io
Cc: gaoliming@byosoft.com.cn; Kinney, Michael D <michael.d.kinney@intel.com>; Liu, Zhiguang <zhiguang.liu@intel.com>; Gao, Zhichao <zhichao.gao@intel.com>
Subject: RE: [edk2-devel] [PATCH] MdePkg,ShellPkg: Add ProcessorUpgradeSocketLGA4677 from SMBIOS 3.5.0
Star,
It might be better to split the patch to two patches.
one is to change MdePkg adding the definitions.
The other is to change ShellPkg consuming the definitions.
> -----Original Message-----
> From: Zeng, Star <star.zeng@intel.com>
> Sent: Tuesday, September 28, 2021 10:11 AM
> To: devel@edk2.groups.io; Zeng, Star <star.zeng@intel.com>
> Cc: gaoliming@byosoft.com.cn; Kinney, Michael D
> <michael.d.kinney@intel.com>; Liu, Zhiguang <zhiguang.liu@intel.com>;
> Ni, Ray <ray.ni@intel.com>; Gao, Zhichao <zhichao.gao@intel.com>
> Subject: RE: [edk2-devel] [PATCH] MdePkg,ShellPkg: Add
> ProcessorUpgradeSocketLGA4677 from SMBIOS 3.5.0
>
> + Maintainers and Reviewers
>
> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Zeng,
> Star
> Sent: 2021年9月28日 10:04
> To: devel@edk2.groups.io
> Cc: Zeng, Star <star.zeng@intel.com>
> Subject: [edk2-devel] [PATCH] MdePkg,ShellPkg: Add
> ProcessorUpgradeSocketLGA4677 from SMBIOS 3.5.0
>
> This patch adds ProcessorUpgradeSocketLGA4677 definition into Smbios.h from SMBIOS 3.5.0 and entry into QueryTable.c.
> It also adds ProcessorUpgradeSocketLGA4189 and
> ProcessorUpgradeSocketLGA1200 into from SMBIOS 3.4.0 and entries into QueryTable.c.
>
> Signed-off-by: Star Zeng <star.zeng@intel.com>
> ---
> MdePkg/Include/IndustryStandard/SmBios.h | 7 +++++--
> .../SmbiosView/QueryTable.c | 14 +++++++++++++-
> 2 files changed, 18 insertions(+), 3 deletions(-)
>
> diff --git a/MdePkg/Include/IndustryStandard/SmBios.h
> b/MdePkg/Include/IndustryStandard/SmBios.h
> index 6918f58cce44..2c2b32b8d462 100644
> --- a/MdePkg/Include/IndustryStandard/SmBios.h
> +++ b/MdePkg/Include/IndustryStandard/SmBios.h
> @@ -1,7 +1,7 @@
> /** @file Industry Standard Definitions of SMBIOS Table Specification v3.3.0. -Copyright (c) 2006 - 2019, Intel Corporation. All
> rights reserved.<BR>+Copyright (c) 2006 - 2021, Intel Corporation. All
> rights reserved.<BR> (C) Copyright 2015-2017 Hewlett Packard
> Enterprise Development LP<BR> (C) Copyright 2015 - 2019 Hewlett
> Packard Enterprise Development LP<BR> SPDX-
> License-Identifier: BSD-2-Clause-Patent@@ -810,7 +810,10 @@ typedef enum {
> ProcessorUpgradeSocketLGA2066 = 0x39, ProcessorUpgradeSocketBGA1392 = 0x3A, ProcessorUpgradeSocketBGA1510
> = 0x3B,- ProcessorUpgradeSocketBGA1528 = 0x3C+ ProcessorUpgradeSocketBGA1528 = 0x3C,+
> ProcessorUpgradeSocketLGA4189 = 0x3D,+ ProcessorUpgradeSocketLGA1200 = 0x3E,+ ProcessorUpgradeSocketLGA4677 =
> 0x3F } PROCESSOR_UPGRADE; ///diff --git
> a/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/QueryTable.c
> b/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/QueryTable.c
> index 7fc9d38a3b03..c312a7f8f227 100644
> ---
> a/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/QueryTable.c
> +++ b/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/QueryTable.
> +++ c
> @@ -2,7 +2,7 @@
> Build a table, each item is (Key, Info) pair. And give a interface of query a string out of a table. - Copyright (c) 2005 - 2019,
> Intel Corporation. All rights reserved.<BR>+ Copyright (c) 2005 - 2021, Intel Corporation. All rights reserved.<BR> (C) Copyright
> 2016-2019 Hewlett Packard Enterprise Development LP<BR> SPDX-License-Identifier: BSD-2-Clause-Patent @@ -589,6 +589,18
> @@ TABLE_ITEM ProcessorUpgradeTable[] = {
> { 0x3C, L"Socket BGA1528"+ },+ {+ 0x3D,+ L"Socket LGA4189"+ },+ {+ 0x3E,+ L"Socket LGA1200"+ },+ {+ 0x3F,+
> L"Socket LGA4677" } }; --
> 2.27.0.windows.1
>
>
>
> -=-=-=-=-=-=
> Groups.io Links: You receive all messages sent to this group.
> View/Reply Online (#81189):
> https://edk2.groups.io/g/devel/message/81189
> Mute This Topic: https://groups.io/mt/85916590/1779220
> Group Owner: devel+owner@edk2.groups.io
> Unsubscribe: https://edk2.groups.io/g/devel/unsub
> [star.zeng@intel.com] -=-=-=-=-=-=
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-09-28 2:39 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <16A8DA167D1CA920.13648@groups.io>
2021-09-28 2:10 ` [edk2-devel] [PATCH] MdePkg,ShellPkg: Add ProcessorUpgradeSocketLGA4677 from SMBIOS 3.5.0 Zeng, Star
2021-09-28 2:11 ` Ni, Ray
2021-09-28 2:39 ` Zeng, Star
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox