[edk2-devel] [PATCH edk2-platforms v1 2/3] Platform/ARM: Juno: Print debug values in right order

PierreGondois posted 3 patches 3 years, 9 months ago
[edk2-devel] [PATCH edk2-platforms v1 2/3] Platform/ARM: Juno: Print debug values in right order
Posted by PierreGondois 3 years, 9 months ago
From: Pierre Gondois <Pierre.Gondois@arm.com>

The values printed in the 'DEBUG' are not printed in the right
order. Re-order them.

Signed-off-by: Pierre Gondois <Pierre.Gondois@arm.com>
---
 .../ConfigurationManagerDxe/ConfigurationManager.c              | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManager.c b/Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManager.c
index 977aba5da5c4..d147bbc317db 100644
--- a/Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManager.c
+++ b/Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManager.c
@@ -790,8 +790,8 @@ HandleCmObjectSearchPlatformRepo (
   DEBUG ((
     DEBUG_INFO,
     "INFO: Token = 0x%p, CmObjectId = %x, Ptr = 0x%p, Size = %d, Count = %d\n",
-    CmObjectId,
     (VOID*)Token,
+    CmObjectId,
     CmObjectDesc->Data,
     CmObjectDesc->Size,
     CmObjectDesc->Count
-- 
2.25.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#89351): https://edk2.groups.io/g/devel/message/89351
Mute This Topic: https://groups.io/mt/90732526/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH edk2-platforms v1 2/3] Platform/ARM: Juno: Print debug values in right order
Posted by Sami Mujawar 3 years, 6 months ago
Hi Pierre,

Thank you for this fix.

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

Regards,

Sami Mujawar

On 27/04/2022 03:59 pm, Pierre.Gondois@arm.com wrote:
> From: Pierre Gondois <Pierre.Gondois@arm.com>
>
> The values printed in the 'DEBUG' are not printed in the right
> order. Re-order them.
>
> Signed-off-by: Pierre Gondois <Pierre.Gondois@arm.com>
> ---
>   .../ConfigurationManagerDxe/ConfigurationManager.c              | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManager.c b/Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManager.c
> index 977aba5da5c4..d147bbc317db 100644
> --- a/Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManager.c
> +++ b/Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManager.c
> @@ -790,8 +790,8 @@ HandleCmObjectSearchPlatformRepo (
>     DEBUG ((
>       DEBUG_INFO,
>       "INFO: Token = 0x%p, CmObjectId = %x, Ptr = 0x%p, Size = %d, Count = %d\n",
> -    CmObjectId,
>       (VOID*)Token,
> +    CmObjectId,
>       CmObjectDesc->Data,
>       CmObjectDesc->Size,
>       CmObjectDesc->Count


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#91461): https://edk2.groups.io/g/devel/message/91461
Mute This Topic: https://groups.io/mt/90732526/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-