From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by mx.groups.io with SMTP id smtpd.web10.4408.1665393683844965786 for ; Mon, 10 Oct 2022 02:21:24 -0700 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: arm.com, ip: 217.140.110.172, mailfrom: pierre.gondois@arm.com) Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id ABF151480; Mon, 10 Oct 2022 02:21:29 -0700 (PDT) Received: from pierre123.arm.com (unknown [10.57.34.234]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPA id B89B13F792; Mon, 10 Oct 2022 02:21:22 -0700 (PDT) From: "PierreGondois" To: devel@edk2.groups.io Cc: Sami Mujawar , Alexei Fedorov Subject: [PATCH 06/14] DynamicTablesPkg: Fix wrong/missing fields in CmObjParser Date: Mon, 10 Oct 2022 11:20:50 +0200 Message-Id: <20221010092058.118714-7-Pierre.Gondois@arm.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20221010092058.118714-1-Pierre.Gondois@arm.com> References: <20221010092058.118714-1-Pierre.Gondois@arm.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable From: Pierre Gondois Add missing fields to the following CmObjParser objects: - EArmObjGicDInfo - EArmObjCacheInfo and fix wrong formatting of: - EArmObjLpiInfo Signed-off-by: Pierre Gondois --- .../ConfigurationManagerObjectParser.c | 24 ++++++++++--------- 1 file changed, 13 insertions(+), 11 deletions(-) diff --git a/DynamicTablesPkg/Library/Common/TableHelperLib/Configuration= ManagerObjectParser.c b/DynamicTablesPkg/Library/Common/TableHelperLib/Co= nfigurationManagerObjectParser.c index 040aaa4cbb17..2126beba8b9f 100644 --- a/DynamicTablesPkg/Library/Common/TableHelperLib/ConfigurationManager= ObjectParser.c +++ b/DynamicTablesPkg/Library/Common/TableHelperLib/ConfigurationManager= ObjectParser.c @@ -303,7 +303,8 @@ STATIC CONST CM_OBJ_PARSER CmArmProcHierarchyInfoPar= ser[] =3D { { "ParentToken", sizeof (CM_OBJECT_TOKEN), "0x%p", NULL= }, { "GicCToken", sizeof (CM_OBJECT_TOKEN), "0x%p", NULL= }, { "NoOfPrivateResources", 4, "0x%x", NULL= }, - { "PrivateResourcesArrayToken", sizeof (CM_OBJECT_TOKEN), "0x%p", NULL= } + { "PrivateResourcesArrayToken", sizeof (CM_OBJECT_TOKEN), "0x%p", NULL= }, + { "LpiToken", sizeof (CM_OBJECT_TOKEN), "0x%p", NULL= }, }; =20 /** A parser for EArmObjCacheInfo. @@ -315,7 +316,8 @@ STATIC CONST CM_OBJ_PARSER CmArmCacheInfoParser[] =3D= { { "NumberOfSets", 4, "0x%x", NULL }, { "Associativity", 4, "0x%x", NULL }, { "Attributes", 1, "0x%x", NULL }, - { "LineSize", 2, "0x%x", NULL } + { "LineSize", 2, "0x%x", NULL }, + { "CacheId", 4, "0x%x", NULL }, }; =20 /** A parser for EArmObjProcNodeIdInfo. @@ -400,14 +402,14 @@ STATIC CONST CM_OBJ_PARSER AcpiGenericAddressParse= r[] =3D { /** A parser for EArmObjLpiInfo. */ STATIC CONST CM_OBJ_PARSER CmArmLpiInfoParser[] =3D { - { "MinResidency", 4, = "0x%x", NULL }, - { "WorstCaseWakeLatency", 4, = "0x%x", NULL }, - { "Flags", 4, = "0x%x", NULL }, - { "ArchFlags", 4, = "0x%x", NULL }, - { "ResCntFreq", 4, = "0x%x", NULL }, - { "EnableParentState", 4, = "0x%x", NULL }, - { "IsInteger", 1, = "%d", NULL }, - { "IntegerEntryMethod", 8, = "0x%llx", NULL }, + { "MinResidency", 4, = "0x%x", NULL }, + { "WorstCaseWakeLatency", 4, = "0x%x", NULL }, + { "Flags", 4, = "0x%x", NULL }, + { "ArchFlags", 4, = "0x%x", NULL }, + { "ResCntFreq", 4, = "0x%x", NULL }, + { "EnableParentState", 4, = "0x%x", NULL }, + { "IsInteger", 1, = "%d", NULL }, + { "IntegerEntryMethod", 8, = "0x%llx", NULL }, { "RegisterEntryMethod", sizeof (EFI_ACPI_6_3_GENERIC_ADDRESS_STR= UCTURE), NULL, NULL, AcpiGenericAddressParser, ARRAY_SIZE (AcpiGenericAddressParser) }, @@ -417,7 +419,7 @@ STATIC CONST CM_OBJ_PARSER CmArmLpiInfoParser[] =3D = { { "UsageCounterRegister", sizeof (EFI_ACPI_6_3_GENERIC_ADDRESS_STR= UCTURE), NULL, NULL, AcpiGenericAddressParser, ARRAY_SIZE (AcpiGenericAddressParser) }, - { "StateName", 16, = "0x%a", NULL }, + { "StateName", 16, = "NULL", PrintString }, }; =20 /** A parser for EArmObjPciAddressMapInfo. --=20 2.25.1