[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CACMJSetxpEL0pomsiZffXr3bFYEGLMBvOcZDZHkizxRx92HVQg@mail.gmail.com>
Date: Fri, 12 Jul 2024 11:09:07 +0200
From: Bartosz Golaszewski <bartosz.golaszewski@...aro.org>
To: Arnd Bergmann <arnd@...db.de>
Cc: Bartosz Golaszewski <brgl@...ev.pl>, Naresh Kamboju <naresh.kamboju@...aro.org>,
open list <linux-kernel@...r.kernel.org>, lkft-triage@...ts.linaro.org,
Dan Carpenter <dan.carpenter@...aro.org>,
Luiz Augusto von Dentz <luiz.von.dentz@...el.com>
Subject: Re: next: arm64: defconfig: gcc-8: drivers/bluetooth/hci_qca.c:2501:2:
error: label at end of compound statement
On Fri, 12 Jul 2024 at 11:04, Arnd Bergmann <arnd@...db.de> wrote:
>
> On Fri, Jul 12, 2024, at 10:34, Bartosz Golaszewski wrote:
>
> > The actual code looks like this now:
> >
> > case QCA_WCN7850:
> > if (power->vregs_on)
> > qca_power_shutdown(&qcadev->serdev_hu);
> > break;
> > default:
> >
> > What can be done to silence this warning? Or should we just ignore it because
> > it's gcc 8?
>
> clang-18 and gcc-10 still warn as well:
>
> 5:1: warning: label at end of compound statement is a C23 extension [-Wc23-extensions]
>
> It's easy enough to fix it by dropping the redundant 'default:'
> line or adding a 'break;' Luiz just committed a fix, see below.
>
> Arnd
>
> commit f14c0bb78769f2670fdd8bcd28ca5543a7601c33
> Author: Luiz Augusto von Dentz <luiz.von.dentz@...el.com>
> Date: Wed Jul 10 22:30:57 2024 -0400
>
> Bluetooth: hci_qca: Fix build error
>
> This fixes the following build error introduced by a887c8dede8e
> ("Bluetooth: hci_qca: schedule a devm action for disabling the clock"):
>
> drivers/bluetooth/hci_qca.c: In function ‘qca_serdev_remove’:
> drivers/bluetooth/hci_qca.c:2501:2: error: label at end of compound statement
> 2501 | default:
> | ^~~~~~~
>
> Fixes: a887c8dede8e ("Bluetooth: hci_qca: schedule a devm action for disabling the clock")
> Signed-off-by: Luiz Augusto von Dentz <luiz.von.dentz@...el.com>
>
> diff --git a/drivers/bluetooth/hci_qca.c b/drivers/bluetooth/hci_qca.c
> index 030153d468bf..ca6466676902 100644
> --- a/drivers/bluetooth/hci_qca.c
> +++ b/drivers/bluetooth/hci_qca.c
> @@ -2499,6 +2499,7 @@ static void qca_serdev_remove(struct serdev_device *serdev)
> qca_power_shutdown(&qcadev->serdev_hu);
> break;
> default:
> + break;
> }
>
> hci_uart_unregister_device(&qcadev->serdev_hu);
>
Ah, I saw the break in the file and thought it was me who had it in
the code in the first place but obviously not. Thanks, nevermind my
comment.
Bart
Powered by blists - more mailing lists