* [PATCH] ASoC: SDCA: Add stub for sdca_fdl_free_state()
@ 2026-08-13 1:26 yaolu
2026-08-13 8:35 ` Charles Keepax
0 siblings, 1 reply; 2+ messages in thread
From: yaolu @ 2026-08-13 1:26 UTC (permalink / raw)
To: ckeepax, mstrozek, perex, tiwai
Cc: broonie, linux-sound, patches, linux-kernel, Lu Yao, k2ci
From: Lu Yao <yaolu@kylinos.cn>
Commit 0880082c27b6 ("ASoC: SDCA: Remove devm from primary IRQ
cleanup") added a call to sdca_fdl_free_state() in sdca_interrupts.c,
but only declared it in the CONFIG_SND_SOC_SDCA_FDL enabled section of
sdca_fdl.h. Building SDCA IRQ support without FDL support therefore
fails with:
sound/soc/sdca/sdca_interrupts.c:471:56: error: 'sdca_fdl_free_state' undeclared (first use in this function); did you mean 'sdca_jack_free_state'?
Fixes: 0880082c27b6 ("ASoC: SDCA: Remove devm from primary IRQ cleanup")
Signed-off-by: Lu Yao <yaolu@kylinos.cn>
Reported-by: k2ci <kernel-bot@kylinos.cn>
---
include/sound/sdca_fdl.h | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/include/sound/sdca_fdl.h b/include/sound/sdca_fdl.h
index dc33927b82bd..bc3600cdd2ec 100644
--- a/include/sound/sdca_fdl.h
+++ b/include/sound/sdca_fdl.h
@@ -83,6 +83,10 @@ static inline int sdca_fdl_alloc_state(struct sdca_interrupt *interrupt)
return 0;
}
+static inline void sdca_fdl_free_state(struct sdca_interrupt *interrupt)
+{
+}
+
static inline int sdca_fdl_process(struct sdca_interrupt *interrupt)
{
return 0;
--
2.25.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] ASoC: SDCA: Add stub for sdca_fdl_free_state()
2026-08-13 1:26 [PATCH] ASoC: SDCA: Add stub for sdca_fdl_free_state() yaolu
@ 2026-08-13 8:35 ` Charles Keepax
0 siblings, 0 replies; 2+ messages in thread
From: Charles Keepax @ 2026-08-13 8:35 UTC (permalink / raw)
To: yaolu
Cc: mstrozek, perex, tiwai, broonie, linux-sound, patches,
linux-kernel, k2ci
On Thu, Aug 13, 2026 at 09:26:36AM +0800, yaolu@kylinos.cn wrote:
> From: Lu Yao <yaolu@kylinos.cn>
>
> Commit 0880082c27b6 ("ASoC: SDCA: Remove devm from primary IRQ
> cleanup") added a call to sdca_fdl_free_state() in sdca_interrupts.c,
> but only declared it in the CONFIG_SND_SOC_SDCA_FDL enabled section of
> sdca_fdl.h. Building SDCA IRQ support without FDL support therefore
> fails with:
>
> sound/soc/sdca/sdca_interrupts.c:471:56: error: 'sdca_fdl_free_state' undeclared (first use in this function); did you mean 'sdca_jack_free_state'?
>
> Fixes: 0880082c27b6 ("ASoC: SDCA: Remove devm from primary IRQ cleanup")
> Signed-off-by: Lu Yao <yaolu@kylinos.cn>
> Reported-by: k2ci <kernel-bot@kylinos.cn>
> ---
This was already fixed here:
https://lore.kernel.org/linux-sound/20260730130602.3747053-1-ckeepax@opensource.cirrus.com/T/#u
Thanks,
Charles
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2026-08-13 8:35 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-08-13 1:26 [PATCH] ASoC: SDCA: Add stub for sdca_fdl_free_state() yaolu
2026-08-13 8:35 ` Charles Keepax
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.