From: "Liming Gao" <liming.gao@intel.com>
To: "devel@edk2.groups.io" <devel@edk2.groups.io>,
"Gao, Zhichao" <zhichao.gao@intel.com>,
Abner Chang <abner.chang@hpe.com>
Cc: "Carsey, Jaben" <jaben.carsey@intel.com>,
"Ni, Ray" <ray.ni@intel.com>,
Leif Lindholm <leif.lindholm@linaro.org>,
Gilbert Chen <gilbert.chen@hpe.com>
Subject: Re: [edk2-devel] [PATCH v7 3/3] ShellPkg/UefiShellDebug1CommandsLib: Update to support SmBios 3.3.0
Date: Thu, 17 Oct 2019 03:49:33 +0000 [thread overview]
Message-ID: <4A89E2EF3DFEDB4C8BFDE51014F606A14E51D729@SHSMSX104.ccr.corp.intel.com> (raw)
In-Reply-To: <3CE959C139B4C44DBEA1810E3AA6F9000B857018@SHSMSX101.ccr.corp.intel.com>
Push @b8fc5c03e5b5551b083f7efe91a051faa00c1e27..e797a806a8d090a09a782fe10ba2765154579d4b
Please close BZ 2202.
Thanks
Liming
>-----Original Message-----
>From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of
>Gao, Zhichao
>Sent: Wednesday, October 16, 2019 8:45 AM
>To: Abner Chang <abner.chang@hpe.com>; devel@edk2.groups.io
>Cc: Carsey, Jaben <jaben.carsey@intel.com>; Ni, Ray <ray.ni@intel.com>; Leif
>Lindholm <leif.lindholm@linaro.org>; Gilbert Chen <gilbert.chen@hpe.com>
>Subject: Re: [edk2-devel] [PATCH v7 3/3]
>ShellPkg/UefiShellDebug1CommandsLib: Update to support SmBios 3.3.0
>
>
>
>> -----Original Message-----
>> From: Abner Chang [mailto:abner.chang@hpe.com]
>> Sent: Tuesday, October 15, 2019 3:46 PM
>> To: devel@edk2.groups.io
>> Cc: abner.chang@hpe.com; Carsey, Jaben <jaben.carsey@intel.com>; Ni,
>> Ray <ray.ni@intel.com>; Gao, Zhichao <zhichao.gao@intel.com>; Leif
>> Lindholm <leif.lindholm@linaro.org>; Gilbert Chen <gilbert.chen@hpe.com>
>> Subject: [PATCH v7 3/3] ShellPkg/UefiShellDebug1CommandsLib: Update to
>> support SmBios 3.3.0
>>
>> Shell command smbiosview updates for SMBIOS 3.3.0 spec.
>>
>> Bugzilla link,
>> https://bugzilla.tianocore.org/show_bug.cgi?id=2202
>>
>> Signed-off-by: Abner Chang <abner.chang@hpe.com>
>>
>> Cc: Jaben Carsey <jaben.carsey@intel.com>
>> Cc: Ray Ni <ray.ni@intel.com>
>> Cc: Zhichao Gao <zhichao.gao@intel.com>
>> Cc: Leif Lindholm <leif.lindholm@linaro.org>
>> Cc: Gilbert Chen <gilbert.chen@hpe.com>
>> ---
>> .../SmbiosView/PrintInfo.c | 21 ++++++-
>> .../SmbiosView/QueryTable.c | 65 +++++++++++++++++++++-
>> .../SmbiosView/PrintInfo.h | 13 ++++-
>> .../SmbiosView/SmbiosViewStrings.uni | 3 +-
>> .../UefiShellDebug1CommandsLib.uni | 3 +-
>> 5 files changed, 100 insertions(+), 5 deletions(-)
>>
>> diff --git
>> a/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/PrintInfo.c
>> b/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/PrintInfo.c
>> index cfd7a2b..a75caff 100644
>> ---
>> a/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/PrintInfo.c
>> +++
>> b/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/PrintInfo.c
>> @@ -3,7 +3,7 @@
>>
>> Copyright (c) 2005 - 2018, Intel Corporation. All rights reserved.<BR>
>> (C) Copyright 2014 Hewlett-Packard Development Company, L.P.<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
>>
>> **/
>> @@ -1169,6 +1169,13 @@ SmbiosPrintStructure (
>> break;
>>
>> //
>> + // Processor Additional Information (Type 44) // case 44:
>> + DisplayProcessorArchitectureType (Struct->Type44-
>> >ProcessorSpecificBlock.ProcessorArchType, Option);
>> + break;
>> +
>> + //
>> // Inactive (Type 126)
>> //
>> case 126:
>> @@ -2350,6 +2357,18 @@ DisplayProcessorFamily2 (
>> Print (L"Video Processor\n");
>> break;
>>
>> + case 0x200:
>> + Print (L"RISC-V RV32\n");
>> + break;
>> +
>> + case 0x201:
>> + Print (L"RISC-V RV64\n");
>> + break;
>> +
>> + case 0x202:
>> + Print (L"RISC-V RV128\n");
>> + break;
>> +
>> default:
>> ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN
>> (STR_SMBIOSVIEW_PRINTINFO_UNDEFINED_PROC_FAMILY),
>> gShellDebug1HiiHandle);
>> }
>> diff --git
>> a/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/QueryTable.
>> c
>> b/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/QueryTable.
>> c
>> index bfb7c3b..87f446e 100644
>> ---
>> a/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/QueryTable.
>> c
>> +++
>> b/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/QueryTable.
>> +++ c
>> @@ -3,7 +3,7 @@
>> And give a interface of query a string out of a table.
>>
>> Copyright (c) 2005 - 2018, Intel Corporation. All rights reserved.<BR>
>> - (C) Copyright 2016-2017 Hewlett Packard Enterprise Development LP<BR>
>> + (C) Copyright 2016-2019 Hewlett Packard Enterprise Development LP<BR>
>> SPDX-License-Identifier: BSD-2-Clause-Patent
>>
>> **/
>> @@ -620,6 +620,10 @@ TABLE_ITEM ProcessorCharacteristicsTable[] = {
>> {
>> 7,
>> L" Power/Performance Control"
>> + },
>> + {
>> + 8,
>> + L" 128-bit Capable"
>> }
>> };
>>
>> @@ -3268,6 +3272,44 @@ TABLE_ITEM MCHostInterfaceTypeTable[] = {
>> },
>> };
>>
>> +TABLE_ITEM ProcessorArchitectureTypesTable[] = {
>> + {
>> + 0,
>> + L" Reserved "
>> + },
>> + {
>> + 1,
>> + L" IA32 (x86) "
>> + },
>> + {
>> + 2,
>> + L" x64 (x86-64, intel64, AMD64, EM64T) "
>> + },
>> + {
>> + 3,
>> + L" Intel Itanium architecture "
>> + },
>> + {
>> + 4,
>> + L" 32-bit ARM (Aarch32) "
>> + },
>> + {
>> + 5,
>> + L" 64-bit ARM (Aarch64) "
>> + },
>> + {
>> + 6,
>> + L" 32-bit RISC-V (RV32) "
>> + },
>> + {
>> + 7,
>> + L" 64-bit RISC-V (RV64) "
>> + },
>> + {
>> + 8,
>> + L" 128-bit RISC-V (RV128) "
>> + }
>> +};
>>
>> TABLE_ITEM StructureTypeInfoTable[] = {
>> {
>> @@ -3447,6 +3489,10 @@ TABLE_ITEM StructureTypeInfoTable[] = {
>> L" TPM Device"
>> },
>> {
>> + 44,
>> + L" Processor Additional Information"
>> + },
>> + {
>> 0x7E,
>> L" Inactive"
>> },
>> @@ -4697,6 +4743,23 @@ DisplayMCHostInterfaceType ( }
>>
>> /**
>> + Display Processor Architecture Type (Type 44).
>> +
>> + @param[in] Key The key of the structure.
>> + @param[in] Option The optional information.
>> +**/
>> +VOID
>> +DisplayProcessorArchitectureType (
>> + IN UINT8 Key,
>> + IN UINT8 Option
>> + )
>> +{
>> + ShellPrintHiiEx(-1,-1,NULL,STRING_TOKEN
>> +(STR_SMBIOSVIEW_QUERYTABLE_PROCESSOR_ARCH_TYPE),
>> +gShellDebug1HiiHandle);
>
>Refer to the CSS spec 5.2.2.6, "Always put space before an open parenthesis".
>Please change to ShellPrintHiiEx (...);
>
>> + PRINT_INFO_OPTION (Key, Option);
>> + PRINT_TABLE_ITEM (ProcessorArchitectureTypesTable, Key); }
>> +
>> +/**
>> Display the structure type information.
>>
>> @param[in] Key The key of the structure.
>> diff --git
>> a/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/PrintInfo.h
>> b/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/PrintInfo.h
>> index a8c32ab..77a86fc 100644
>> ---
>> a/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/PrintInfo.h
>> +++
>> b/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/PrintInfo.h
>> @@ -2,7 +2,7 @@
>> Module to clarify the element info of the smbios structure.
>>
>> Copyright (c) 2005 - 2015, Intel Corporation. All rights reserved.<BR>
>> - (C) Copyright 2017 Hewlett Packard Enterprise Development LP<BR>
>> + (C) Copyright 2017 - 2019 Hewlett Packard Enterprise Development
>> + LP<BR>
>> SPDX-License-Identifier: BSD-2-Clause-Patent
>>
>> **/
>> @@ -427,4 +427,15 @@ DisplayTpmDeviceCharacteristics (
>> IN UINT8 Option
>> );
>>
>> +/**
>> + Display Processor Architecture Type (Type 44).
>> +
>> + @param[in] Key The key of the structure.
>> + @param[in] Option The optional information.
>
>Here it is not force, but better to align the comment.
>There is no need to send another version patch. Fix them before push.
>With the mentioned addressed, Reviewed-by: Zhichao Gao
><zhichao.gao@intel.com>
>
>Thanks,
>Zhichao
>
>> +**/
>> +VOID
>> +DisplayProcessorArchitectureType (
>> + IN UINT8 Key,
>> + IN UINT8 Option
>> + );
>> #endif
>> diff --git
>>
>a/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/SmbiosView
>> Strings.uni
>>
>b/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/SmbiosView
>> Strings.uni
>> index 94aff91..7fbcc9a 100644
>> ---
>>
>a/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/SmbiosView
>> Strings.uni
>> +++
>>
>b/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/SmbiosView
>> S
>> +++ trings.uni
>> @@ -2,7 +2,7 @@
>> //
>> // Copyright (c) 2005 - 2018, Intel Corporation. All rights reserved.<BR> // (C)
>> Copyright 2014-2015 Hewlett-Packard Development Company, L.P.<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 // // Module Name:
>> @@ -447,6 +447,7 @@
>> #string STR_SMBIOSVIEW_QUERYTABLE_BMC_INTERFACE_TYPE
>> #language en-US "BMC Interface Type:"
>> #string STR_SMBIOSVIEW_QUERYTABLE_MC_HOST_INTERFACE_TYPE
>> #language en-US "MC Host Interface Type:"
>> #string STR_SMBIOSVIEW_QUERYTABLE_STRUCT_TYPE #language
>> en-US "Structure Type:"
>> +#string STR_SMBIOSVIEW_QUERYTABLE_PROCESSOR_ARCH_TYPE
>> #language en-US "Processor Architecture Type:"
>> #string STR_SMBIOSVIEW_SMBIOSVIEW_ONE_VAR_ARGV
>> #language en-US "%s "
>> #string STR_SMBIOSVIEW_SMBIOSVIEW_QUERY_STRUCT_COND
>> #language en-US "Query Structure, conditions are:\r\n"
>> #string STR_SMBIOSVIEW_SMBIOSVIEW_QUERYTYPE_RANDOM
>> #language en-US "QueryType = Random \r\n"
>> diff --git
>>
>a/ShellPkg/Library/UefiShellDebug1CommandsLib/UefiShellDebug1Comman
>> dsLib.uni
>>
>b/ShellPkg/Library/UefiShellDebug1CommandsLib/UefiShellDebug1Comman
>> dsLib.uni
>> index 42a9295..6693be2 100644
>> ---
>>
>a/ShellPkg/Library/UefiShellDebug1CommandsLib/UefiShellDebug1Comman
>> dsLib.uni
>> +++
>>
>b/ShellPkg/Library/UefiShellDebug1CommandsLib/UefiShellDebug1Comman
>> d
>> +++ sLib.uni
>> @@ -2,7 +2,7 @@
>> //
>> // Copyright (c) 2010 - 2018, Intel Corporation. All rights reserved.<BR> // (C)
>> Copyright 2013-2015 Hewlett-Packard Development Company, L.P.<BR> -//
>> (C) Copyright 2016 Hewlett Packard Enterprise Development LP<BR>
>> +// (C) Copyright 2016 - 2019 Hewlett Packard Enterprise Development
>> +LP<BR>
>> // SPDX-License-Identifier: BSD-2-Clause-Patent // // Module Name:
>> @@ -976,6 +976,7 @@
>> " 41 - Onboard Devices Extended Information\r\n"
>> " 42 - Management Controller Host Interface\r\n"
>> " 43 - TPM Device\r\n"
>> +" 44 - Processor Additional Information\r\n"
>> " 2. Enter the SmbiosHandle parameter in hexadecimal format.\r\n"
>> " Do not use the '0x' prefix format for hexadecimal values.\r\n"
>> " 3. Internal commands:\r\n"
>> --
>> 2.7.4
>
>
>
prev parent reply other threads:[~2019-10-17 3:49 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-10-15 7:45 [PATCH v7 3/3] ShellPkg/UefiShellDebug1CommandsLib: Update to support SmBios 3.3.0 Abner Chang
2019-10-16 0:45 ` Gao, Zhichao
2019-10-17 3:49 ` Liming Gao [this message]
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=4A89E2EF3DFEDB4C8BFDE51014F606A14E51D729@SHSMSX104.ccr.corp.intel.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