Re: [PATCH] Platform/ARM/VExpressPkg: Fix unused but set


Sami Mujawar
 

Hi Adrian,

 

Thank you for this patch.

 

The changes look good to me. Can you let me know which compiler toolchain flagged up this error/build crash, please?

 

Reviewed-by: Sami Mujawar <sami.mujawar@...>

 

Regards,

 

Sami Mujawar

 

From: devel@edk2.groups.io <devel@edk2.groups.io> on behalf of Adrián Herrera via groups.io <adr.her.arc.95@...>
Date: Saturday, 24 April 2021 at 03:58
To: devel@edk2.groups.io <devel@edk2.groups.io>
Cc: Adrián Herrera <adr.her.arc.95@...>
Subject: [edk2-devel] [PATCH] Platform/ARM/VExpressPkg: Fix unused but set

Remove unused but set variables in GetArmNameSpaceObject. These caused a
build crash due to -Werror=unused-but-set-variable.

Signed-off-by: Adrián Herrera <adr.her.arc.95@...>
---
 .../ConfigurationManagerDxe/ConfigurationManager.c          | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/Platform/ARM/VExpressPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManager.c b/Platform/ARM/VExpressPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManager.c
index e99fbb654f..d169cd2c5d 100644
--- a/Platform/ARM/VExpressPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManager.c
+++ b/Platform/ARM/VExpressPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManager.c
@@ -721,8 +721,6 @@ GetArmNameSpaceObject (
   UINTN                             Smmuv3Count;


   UINTN                             ItsGroupCount;


   UINTN                             ItsIdentifierArrayCount;


-  UINTN                             RootComplexCount;


-  UINTN                             DeviceIdMappingArrayCount;


   UINTN                             PciConfigSpaceCount;


 


   if ((This == NULL) || (CmObject == NULL)) {


@@ -739,15 +737,11 @@ GetArmNameSpaceObject (
     Smmuv3Count = 1;


     ItsGroupCount = 1;


     ItsIdentifierArrayCount = ARRAY_SIZE (PlatformRepo->ItsIdentifierArray);


-    RootComplexCount = 1;


-    DeviceIdMappingArrayCount = ARRAY_SIZE (PlatformRepo->DeviceIdMapping);


     PciConfigSpaceCount = 1;


   } else {


     Smmuv3Count = 0;


     ItsGroupCount = 0;


     ItsIdentifierArrayCount = 0;


-    RootComplexCount = 0;


-    DeviceIdMappingArrayCount = 0;


     PciConfigSpaceCount = 0;


   }


 


--
2.30.0





Join devel@edk2.groups.io to automatically receive all group messages.