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.muja...@arm.com> 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=gmail....@groups.io> Date: Saturday, 24 April 2021 at 03:58 To: devel@edk2.groups.io <devel@edk2.groups.io> Cc: Adrián Herrera <adr.her.arc...@gmail.com> 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...@gmail.com> --- .../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 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#74427): https://edk2.groups.io/g/devel/message/74427 Mute This Topic: https://groups.io/mt/82327317/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-