* [PATCH] ALSA: aw2-alsa: select SND_PCM
@ 2026-11-13 6:46 Jude Gyimah
2026-11-13 7:57 ` Takashi Iwai
0 siblings, 1 reply; 2+ messages in thread
From: Jude Gyimah @ 2026-11-13 6:46 UTC (permalink / raw)
To: perex, tiwai; +Cc: linux-sound, linux-kernel, Jude Gyimah
Several stanzas in sound/pci/Kconfig select SND_PCM.
SND_AW2 selects nothing at all.
aw2-alsa calls snd_pcm_new, which sound/core/pcm.c provides under SND_PCM.
aw2-alsa.c:(.text+0x554): undefined reference to `snd_pcm_new'
4 symbols are undefined in all.
The driver can be built into vmlinux while SND_PCM is a module, in which
case the symbols are not there at link time.
Select rather than depend on: SND_PCM has no prompt of its own and is only
ever enabled by a select, so a depends on would leave SND_AW2
unselectable.
Reproduced on v7.3, arm64, with CONFIG_SND_AW2=y and CONFIG_SND_PCM=m.
Found with kreach, a static check that compares what Kconfig allows
against the symbols the compiled objects actually reference.
Fixes: 98f2a97f207a ("[ALSA] Emagic Audiowerk 2 ALSA driver.")
Signed-off-by: Jude Gyimah <jude.gyimah87@gmail.com>
---
sound/pci/Kconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/sound/pci/Kconfig b/sound/pci/Kconfig
index 6366f72b3667..9c192d51ff44 100644
--- a/sound/pci/Kconfig
+++ b/sound/pci/Kconfig
@@ -141,6 +141,7 @@ config SND_AU8830
config SND_AW2
tristate "Emagic Audiowerk 2"
+ select SND_PCM
help
Say Y here to include support for Emagic Audiowerk 2 soundcards.
--
2.43.0
^ permalink raw reply [flat|nested] 2+ messages in thread* Re: [PATCH] ALSA: aw2-alsa: select SND_PCM
2026-11-13 6:46 [PATCH] ALSA: aw2-alsa: select SND_PCM Jude Gyimah
@ 2026-11-13 7:57 ` Takashi Iwai
0 siblings, 0 replies; 2+ messages in thread
From: Takashi Iwai @ 2026-11-13 7:57 UTC (permalink / raw)
To: Jude Gyimah; +Cc: perex, tiwai, linux-sound, linux-kernel
On Fri, 13 Nov 2026 07:46:14 +0100,
Jude Gyimah wrote:
>
> Several stanzas in sound/pci/Kconfig select SND_PCM.
> SND_AW2 selects nothing at all.
>
> aw2-alsa calls snd_pcm_new, which sound/core/pcm.c provides under SND_PCM.
>
> aw2-alsa.c:(.text+0x554): undefined reference to `snd_pcm_new'
>
> 4 symbols are undefined in all.
>
> The driver can be built into vmlinux while SND_PCM is a module, in which
> case the symbols are not there at link time.
>
> Select rather than depend on: SND_PCM has no prompt of its own and is only
> ever enabled by a select, so a depends on would leave SND_AW2
> unselectable.
>
> Reproduced on v7.3, arm64, with CONFIG_SND_AW2=y and CONFIG_SND_PCM=m.
>
> Found with kreach, a static check that compares what Kconfig allows
> against the symbols the compiled objects actually reference.
>
> Fixes: 98f2a97f207a ("[ALSA] Emagic Audiowerk 2 ALSA driver.")
> Signed-off-by: Jude Gyimah <jude.gyimah87@gmail.com>
Applied now. It's interesting why this wasn't caught by build tests
for long time...
thanks,
Takashi
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2026-11-13 7:57 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-11-13 6:46 [PATCH] ALSA: aw2-alsa: select SND_PCM Jude Gyimah
2026-11-13 7:57 ` Takashi Iwai
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®