* [PATCH] mailbox: mtk-gpueb: Add missing 'static' to mailbox ops struct
@ 2025-10-13 8:09 Nicolas Frattaroli
2025-10-15 14:58 ` AngeloGioacchino Del Regno
0 siblings, 1 reply; 2+ messages in thread
From: Nicolas Frattaroli @ 2025-10-13 8:09 UTC (permalink / raw)
To: Jassi Brar, Matthias Brugger, AngeloGioacchino Del Regno, Chia-I Wu
Cc: linux-kernel, linux-arm-kernel, linux-mediatek, kernel,
kernel test robot, Nicolas Frattaroli
mtk_gpueb_mbox_ops should be declared static. However, due to its const
nature, this specifier was missed, as it compiled fine without it and
with no warning by the compiler.
arc-linux-gcc (GCC) 12.5.0 doesn't seem to like it however, so add the
static to fix that.
Reported-by: kernel test robot <lkp@intel.com>
Closes: https://lore.kernel.org/oe-kbuild-all/202510100629.3nGvrhEU-lkp@intel.com/
Fixes: dbca0eabb821 ("mailbox: add MediaTek GPUEB IPI mailbox")
Signed-off-by: Nicolas Frattaroli <nicolas.frattaroli@collabora.com>
---
drivers/mailbox/mtk-gpueb-mailbox.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/mailbox/mtk-gpueb-mailbox.c b/drivers/mailbox/mtk-gpueb-mailbox.c
index 925bcf21f650..f6d2beccd91b 100644
--- a/drivers/mailbox/mtk-gpueb-mailbox.c
+++ b/drivers/mailbox/mtk-gpueb-mailbox.c
@@ -200,7 +200,7 @@ static bool mtk_gpueb_mbox_last_tx_done(struct mbox_chan *chan)
return !(readl(ch->ebm->mbox_ctl + GPUEB_MBOX_CTL_TX_STS) & BIT(ch->num));
}
-const struct mbox_chan_ops mtk_gpueb_mbox_ops = {
+static const struct mbox_chan_ops mtk_gpueb_mbox_ops = {
.send_data = mtk_gpueb_mbox_send_data,
.startup = mtk_gpueb_mbox_startup,
.shutdown = mtk_gpueb_mbox_shutdown,
---
base-commit: cb6649f6217c0331b885cf787f1d175963e2a1d2
change-id: 20251013-mt8196-gpueb-mbox-fix-fe6c3053994f
Best regards,
--
Nicolas Frattaroli <nicolas.frattaroli@collabora.com>
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] mailbox: mtk-gpueb: Add missing 'static' to mailbox ops struct
2025-10-13 8:09 [PATCH] mailbox: mtk-gpueb: Add missing 'static' to mailbox ops struct Nicolas Frattaroli
@ 2025-10-15 14:58 ` AngeloGioacchino Del Regno
0 siblings, 0 replies; 2+ messages in thread
From: AngeloGioacchino Del Regno @ 2025-10-15 14:58 UTC (permalink / raw)
To: Nicolas Frattaroli, Jassi Brar, Matthias Brugger, Chia-I Wu
Cc: linux-kernel, linux-arm-kernel, linux-mediatek, kernel,
kernel test robot
Il 13/10/25 10:09, Nicolas Frattaroli ha scritto:
> mtk_gpueb_mbox_ops should be declared static. However, due to its const
> nature, this specifier was missed, as it compiled fine without it and
> with no warning by the compiler.
>
> arc-linux-gcc (GCC) 12.5.0 doesn't seem to like it however, so add the
> static to fix that.
>
> Reported-by: kernel test robot <lkp@intel.com>
> Closes: https://lore.kernel.org/oe-kbuild-all/202510100629.3nGvrhEU-lkp@intel.com/
> Fixes: dbca0eabb821 ("mailbox: add MediaTek GPUEB IPI mailbox")
> Signed-off-by: Nicolas Frattaroli <nicolas.frattaroli@collabora.com>
Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2025-10-15 14:58 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-10-13 8:09 [PATCH] mailbox: mtk-gpueb: Add missing 'static' to mailbox ops struct Nicolas Frattaroli
2025-10-15 14:58 ` AngeloGioacchino Del Regno
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®