All of lore.kernel.org
 help / color / mirror / Atom feed
* [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.