ShellPkg: Correct smbiosview strings and conditions for SMBIOS Type9

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4826

Using smbiosview to dump SMBIOS Type9,
some code condition and string are incorrect.

Signed-off-by: HoraceX Lien <horacex.lien@intel.com>
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Reviewed-by: Liming Gao <gaoliming@byosoft.com.cn>
This commit is contained in:
HoraceX Lien 2024-08-08 16:50:22 +08:00 committed by mergify[bot]
parent 319835abb8
commit 7cde720e51
2 changed files with 10 additions and 11 deletions

View File

@ -3173,7 +3173,7 @@ DisplaySystemSlotId (
//
case 0x04:
ShellPrintHiiEx (-1, -1, NULL, STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_LOGICAL_MICRO_CHAN), gShellDebug1HiiHandle);
if ((SlotId > 0) && (SlotId < 15)) {
if ((SlotId > 0) && (SlotId <= 15)) {
ShellPrintHiiEx (-1, -1, NULL, STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_ONE_VAR_D), gShellDebug1HiiHandle, SlotId);
} else {
ShellPrintHiiEx (-1, -1, NULL, STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_ERROR_NOT_1_15), gShellDebug1HiiHandle);
@ -3182,11 +3182,11 @@ DisplaySystemSlotId (
break;
//
// EISA
// Slot Type: EISA
//
case 0x05:
ShellPrintHiiEx (-1, -1, NULL, STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_LOGICAL_EISA_NUM), gShellDebug1HiiHandle);
if ((SlotId > 0) && (SlotId < 15)) {
if ((SlotId > 0) && (SlotId <= 15)) {
ShellPrintHiiEx (-1, -1, NULL, STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_ONE_VAR_D), gShellDebug1HiiHandle, SlotId);
} else {
ShellPrintHiiEx (-1, -1, NULL, STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_ERROR_NOT_1_15), gShellDebug1HiiHandle);
@ -3202,21 +3202,20 @@ DisplaySystemSlotId (
break;
//
// PCMCIA
// Slot Type: PCMCIA
//
case 0x07:
ShellPrintHiiEx (-1, -1, NULL, STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_IDENTIFIES_ADAPTER_NUM), gShellDebug1HiiHandle, SlotId);
break;
//
// Slot Type: PCI-E
// Slot Type: PCI 66MHz Capable, AGP, PCI-E, etc
//
case 0xA5:
ShellPrintHiiEx (-1, -1, NULL, STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_VALUE_PRESENT), gShellDebug1HiiHandle, SlotId);
break;
default:
if (((SlotType >= 0x0E) && (SlotType <= 0x12)) || ((SlotType >= 0xA6) && (SlotType <= 0xC4))) {
if (((SlotType >= 0x0E) && (SlotType <= 0x13)) ||
((SlotType >= 0x1F) && (SlotType <= 0x25)) ||
((SlotType >= 0xA5) && (SlotType <= 0xC6)))
{
ShellPrintHiiEx (-1, -1, NULL, STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_VALUE_PRESENT), gShellDebug1HiiHandle, SlotId);
} else {
ShellPrintHiiEx (-1, -1, NULL, STRING_TOKEN (STR_SMBIOSVIEW_PRINTINFO_UNDEFINED_SLOT_ID), gShellDebug1HiiHandle);

View File

@ -1924,7 +1924,7 @@ TABLE_ITEM SystemSlotHeightTable[] = {
},
{
0x02,
L" Unkown"
L" Unknown"
},
{
0x03,