From 5ed3b5624f91b2738a90598ea8a9c085cfab851e Mon Sep 17 00:00:00 2001 From: Krzysztof Koch Date: Fri, 28 Jun 2019 18:24:38 +0800 Subject: [PATCH] ShellPkg: acpiview: Remove duplicate indentation in IORT parser Remove redundant whitespace characters at the beginning of the strings describing IORT table field names. When dumping ACPI table contents, the indentation level for printing field names is controled using the 'Indent' argument to the 'ParseAcpi' function. In the IORT acpiview parser, both 'Indent' and extra whitespace characters are used for indentation, which results in excess indentation. Signed-off-by: Krzysztof Koch Reviewed-by: Alexei Fedorov Reviewed-by: Zhichao Gao Reviewed-by: Sami Mujawar --- .../Parsers/Iort/IortParser.c | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Iort/IortParser.c b/ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Iort/IortParser.c index a91a4f9db1..93f78e1a97 100644 --- a/ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Iort/IortParser.c +++ b/ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Iort/IortParser.c @@ -131,19 +131,19 @@ STATIC CONST ACPI_PARSER IortNodeSmmuV1V2Parser[] = { An ACPI_PARSER array describing the SMMUv1/2 Node Interrupt Array. **/ STATIC CONST ACPI_PARSER InterruptArrayParser[] = { - {L" Interrupt GSIV", 4, 0, L"0x%x", NULL, NULL, NULL, NULL}, - {L" Flags", 4, 4, L"0x%x", NULL, NULL, NULL, NULL} + {L"Interrupt GSIV", 4, 0, L"0x%x", NULL, NULL, NULL, NULL}, + {L"Flags", 4, 4, L"0x%x", NULL, NULL, NULL, NULL} }; /** An ACPI_PARSER array describing the IORT ID Mapping. **/ STATIC CONST ACPI_PARSER IortNodeIdMappingParser[] = { - {L" Input base", 4, 0, L"0x%x", NULL, NULL, NULL, NULL}, - {L" Number of IDs", 4, 4, L"0x%x", NULL, NULL, NULL, NULL}, - {L" Output base", 4, 8, L"0x%x", NULL, NULL, NULL, NULL}, - {L" Output reference", 4, 12, L"0x%x", NULL, NULL, NULL, NULL}, - {L" Flags", 4, 16, L"0x%x", NULL, NULL, NULL, NULL} + {L"Input base", 4, 0, L"0x%x", NULL, NULL, NULL, NULL}, + {L"Number of IDs", 4, 4, L"0x%x", NULL, NULL, NULL, NULL}, + {L"Output base", 4, 8, L"0x%x", NULL, NULL, NULL, NULL}, + {L"Output reference", 4, 12, L"0x%x", NULL, NULL, NULL, NULL}, + {L"Flags", 4, 16, L"0x%x", NULL, NULL, NULL, NULL} }; /** @@ -170,14 +170,14 @@ STATIC CONST ACPI_PARSER IortNodeItsParser[] = { ValidateItsIdMappingCount, ValidateItsIdArrayReference ), - {L" Number of ITSs", 4, 16, L"%d", NULL, (VOID**)&ItsCount, NULL} + {L"Number of ITSs", 4, 16, L"%d", NULL, (VOID**)&ItsCount, NULL} }; /** An ACPI_PARSER array describing the ITS ID. **/ STATIC CONST ACPI_PARSER ItsIdParser[] = { - { L" GIC ITS Identifier", 4, 0, L"%d", NULL, NULL, NULL } + { L"GIC ITS Identifier", 4, 0, L"%d", NULL, NULL, NULL } }; /**