Virtualization
/
edk2
mirror of https://gitlab.com/qemu-project/edk2.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

DynamicTablesPkg: Use %a formatter in AmlDbgPrint 

The correct formatter to print a CHAR8 char in edk2 is '%a'.
Replace the '%s' formatters by '%a'.

Reviewed-by: Sami Mujawar <sami.mujawar@arm.com>
Signed-off-by: Pierre Gondois <Pierre.Gondois@arm.com>
This commit is contained in:
Pierre Gondois 2021-09-30 08:48:18 +01:00 committed by mergify[bot]
parent 96e006b37e
commit 235ff9fcd1
1 changed files with 8 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
DynamicTablesPkg/Library/Common/AmlLib/AmlDbgPrint/AmlDbgPrint.c
View File

@ -2,7 +2,7 @@
AML Print Function. AML Print Function.
Copyright (c) 2010 - 2018, Intel Corporation. All rights reserved. <BR> Copyright (c) 2010 - 2018, Intel Corporation. All rights reserved. <BR>
Copyright (c) 2019 - 2020, Arm Limited. All rights reserved.<BR> Copyright (c) 2019 - 2021, Arm Limited. All rights reserved.<BR>
SPDX-License-Identifier: BSD-2-Clause-Patent SPDX-License-Identifier: BSD-2-Clause-Patent
**/ **/
@ -199,7 +199,7 @@ AmlDbgPrintNodeHeader (
DEBUG (( DEBUG ((
DEBUG_INFO, DEBUG_INFO,
"%3d | %-15s | ", "%3d | %-15a | ",
Level, Level,
NodeTypeStrTbl[Node->NodeType] NodeTypeStrTbl[Node->NodeType]
)); ));
@ -227,7 +227,7 @@ AmlDbgPrintDataNode (
AmlDbgPrintNodeHeader ((AML_NODE_HEADER*)DataNode, Level); AmlDbgPrintNodeHeader ((AML_NODE_HEADER*)DataNode, Level);
DEBUG ((DEBUG_INFO, "%-36s | ", NodeDataTypeStrTbl[DataNode->DataType])); DEBUG ((DEBUG_INFO, "%-36a | ", NodeDataTypeStrTbl[DataNode->DataType]));
DEBUG ((DEBUG_INFO, "0x%04x | ", DataNode->Size)); DEBUG ((DEBUG_INFO, "0x%04x | ", DataNode->Size));
if ((DataNode->DataType == EAmlNodeDataTypeNameString) || if ((DataNode->DataType == EAmlNodeDataTypeNameString) ||
@ -300,13 +300,13 @@ AmlDbgPrintObjectNode (
// Print a string corresponding to the field object OpCode/SubOpCode. // Print a string corresponding to the field object OpCode/SubOpCode.
if (AmlNodeHasAttribute (ObjectNode, AML_IS_FIELD_ELEMENT)) { if (AmlNodeHasAttribute (ObjectNode, AML_IS_FIELD_ELEMENT)) {
DEBUG ((DEBUG_INFO, "%-15s ", AmlGetFieldOpCodeStr ( DEBUG ((DEBUG_INFO, "%-15a ", AmlGetFieldOpCodeStr (
ObjectNode->AmlByteEncoding->OpCode, ObjectNode->AmlByteEncoding->OpCode,
0 0
))); )));
} else { } else {
// Print a string corresponding to the object OpCode/SubOpCode. // Print a string corresponding to the object OpCode/SubOpCode.
DEBUG ((DEBUG_INFO, "%-15s | ", AmlGetOpCodeStr ( DEBUG ((DEBUG_INFO, "%-15a | ", AmlGetOpCodeStr (
ObjectNode->AmlByteEncoding->OpCode, ObjectNode->AmlByteEncoding->OpCode,
ObjectNode->AmlByteEncoding->SubOpCode) ObjectNode->AmlByteEncoding->SubOpCode)
)); ));
@ -378,19 +378,19 @@ AmlDbgPrintTableHeader (
DEBUG ((DEBUG_INFO, "Lvl | Node Type |\n")); DEBUG ((DEBUG_INFO, "Lvl | Node Type |\n"));
DEBUG (( DEBUG ((
DEBUG_INFO, DEBUG_INFO,
" | %-15s | Signature| Length | Rev | CSum | OemId | " " | %-15a | Signature| Length | Rev | CSum | OemId | "
"OemTableId | OemRev | CreatorId| CreatorRev\n", "OemTableId | OemRev | CreatorId| CreatorRev\n",
NodeTypeStrTbl[EAmlNodeRoot] NodeTypeStrTbl[EAmlNodeRoot]
)); ));
DEBUG (( DEBUG ((
DEBUG_INFO, DEBUG_INFO,
" | %-15s | Op | SubOp| OpName | MaxI| Attribute | " " | %-15a | Op | SubOp| OpName | MaxI| Attribute | "
"PkgLen | NodeName (opt)\n", "PkgLen | NodeName (opt)\n",
NodeTypeStrTbl[EAmlNodeObject] NodeTypeStrTbl[EAmlNodeObject]
)); ));
DEBUG (( DEBUG ((
DEBUG_INFO, DEBUG_INFO,
" | %-15s | Data Type | Size | " " | %-15a | Data Type | Size | "
"Buffer\n", "Buffer\n",
NodeTypeStrTbl[EAmlNodeData] NodeTypeStrTbl[EAmlNodeData]
)); ));