From: "Krzysztof Koch" <krzysztof.koch@arm.com>
To: "Philippe Mathieu-Daudé" <philmd@redhat.com>,
"devel@edk2.groups.io" <devel@edk2.groups.io>
Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>,
Leif Lindholm <leif.lindholm@linaro.org>,
Michael D Kinney <michael.d.kinney@intel.com>,
Sami Mujawar <Sami.Mujawar@arm.com>,
Matteo Carlini <Matteo.Carlini@arm.com>, nd <nd@arm.com>
Subject: Re: [edk2-devel] [edk2-platforms][PATCH v1 1/1] Platform/ARM/ArmJunoDxe: Fix 'unused variable' error for DynamicTables
Date: Mon, 20 Jan 2020 09:46:09 +0000 [thread overview]
Message-ID: <HE1PR0801MB178820DAF32CE0117388827A84320@HE1PR0801MB1788.eurprd08.prod.outlook.com> (raw)
In-Reply-To: <9fbf43f5-4bba-dd41-4590-86b4d9ab71a3@redhat.com>
Sure! I don't mind changing the commit message. Shall I resubmit the patch?
Kind regards,
Krzysztof
-----Original Message-----
From: Philippe Mathieu-Daudé <philmd@redhat.com>
Sent: Monday, January 20, 2020 9:15
To: devel@edk2.groups.io; Krzysztof Koch <Krzysztof.Koch@arm.com>
Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>; Leif Lindholm <leif.lindholm@linaro.org>; Michael D Kinney <michael.d.kinney@intel.com>; Sami Mujawar <Sami.Mujawar@arm.com>; Matteo Carlini <Matteo.Carlini@arm.com>; nd <nd@arm.com>
Subject: Re: [edk2-devel] [edk2-platforms][PATCH v1 1/1] Platform/ARM/ArmJunoDxe: Fix 'unused variable' error for DynamicTables
On 1/9/20 10:35 AM, Krzysztof Koch wrote:
> Fix 'mAcpiRegistration defined but not used' compiler error for
> Dynamic Tables Framework builds.
>
> Do not declare the mAcpiRegistration variable in the Dynamic Tables
> Framework context.
>
> An earlier patch made mAcpiRegistration variable STATIC. Reducing the
Can we change "An earlier patch" -> "Commit 484a683ab9f"?
> scope made scope resolution simpler, thus, the 'defined but not used'
> error got raised.
>
> Signed-off-by: Krzysztof Koch <krzysztof.koch@arm.com>
Reviewed-by: Philippe Mathieu-Daude <philmd@redhat.com>
> ---
>
> Changes can be seen at:
> https://github.com/KrzysztofKoch1/edk2-platforms/tree/723_fix_unused_v
> ariable_juno_dxe_v1
>
> Notes:
> v1:
> - Fix 'defined but not used' Dynamic Tables build error
> [Krzysztof]
>
> Platform/ARM/JunoPkg/Drivers/ArmJunoDxe/ArmJunoDxe.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/Platform/ARM/JunoPkg/Drivers/ArmJunoDxe/ArmJunoDxe.c
> b/Platform/ARM/JunoPkg/Drivers/ArmJunoDxe/ArmJunoDxe.c
> index
> e3b64992e02c003f7e3a6636340a6afa573f477a..cd7a205f90f131560419cfd744dd
> 5c22df726c99 100644
> --- a/Platform/ARM/JunoPkg/Drivers/ArmJunoDxe/ArmJunoDxe.c
> +++ b/Platform/ARM/JunoPkg/Drivers/ArmJunoDxe/ArmJunoDxe.c
> @@ -1,6 +1,6 @@
> /** @file
> *
> -* Copyright (c) 2013-2019, ARM Limited. All rights reserved.
> +* Copyright (c) 2013-2020, ARM Limited. All rights reserved.
> *
> * SPDX-License-Identifier: BSD-2-Clause-Patent
> *
> @@ -30,9 +30,9 @@
> #ifndef DYNAMIC_TABLES_FRAMEWORK
> // This GUID must match the FILE_GUID in ArmPlatformPkg/ArmJunoPkg/AcpiTables/AcpiTables.inf
> STATIC CONST EFI_GUID mJunoAcpiTableFile = { 0xa1dd808e, 0x1e95,
> 0x4399, { 0xab, 0xc0, 0x65, 0x3c, 0x82, 0xe8, 0x53, 0x0c } }; -#endif
> -
> STATIC VOID *mAcpiRegistration = NULL;
> +#endif
> +
> STATIC VOID *mPciIoNotificationRegistration = NULL;
>
> /**
> --
> 'Guid(CE165669-3EF3-493F-B85D-6190EE5B9759)'
>
>
>
>
>
prev parent reply other threads:[~2020-01-20 9:46 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-01-09 9:35 [edk2-platforms][PATCH v1 1/1] Platform/ARM/ArmJunoDxe: Fix 'unused variable' error for DynamicTables Krzysztof Koch
2020-01-09 9:45 ` [edk2-devel] " Sami Mujawar
2020-01-20 9:14 ` Philippe Mathieu-Daudé
2020-01-20 9:46 ` Krzysztof Koch [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=HE1PR0801MB178820DAF32CE0117388827A84320@HE1PR0801MB1788.eurprd08.prod.outlook.com \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox