lkml.org 
[lkml]   [2011]   [Jul]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: Build regressions/improvements in v3.0-rc7 (sound/isa/)
On Mon, 25 Jul 2011 09:37:53 +0200 Takashi Iwai wrote:

> At Sun, 24 Jul 2011 12:26:24 -0700,
> Randy Dunlap wrote:
> >
> > On Wed, 13 Jul 2011 21:33:30 +0200 Geert Uytterhoeven wrote:
> >
> > > + src/sound/isa/ad1816a/ad1816a_lib.c: error: implicit declaration of function 'snd_dma_disable': => 544
> > > + src/sound/isa/ad1816a/ad1816a_lib.c: error: implicit declaration of function 'snd_dma_pointer': => 302
> > > + src/sound/isa/ad1816a/ad1816a_lib.c: error: implicit declaration of function 'snd_dma_program': => 244
> > > + src/sound/isa/es1688/es1688_lib.c: error: implicit declaration of function 'snd_dma_pointer': => 509
> > > + src/sound/isa/es1688/es1688_lib.c: error: implicit declaration of function 'snd_dma_program': => 417
> > > + src/sound/isa/gus/gus_dma.c: error: implicit declaration of function 'snd_dma_disable': => 177
> > > + src/sound/isa/gus/gus_dma.c: error: implicit declaration of function 'snd_dma_program': => 79
> > > + src/sound/isa/gus/gus_pcm.c: error: implicit declaration of function 'snd_dma_pointer': => 619
> > > + src/sound/isa/gus/gus_pcm.c: error: implicit declaration of function 'snd_dma_program': => 591
> > > + src/sound/isa/sb/sb16_main.c: error: implicit declaration of function 'snd_dma_pointer': => 456
> > > + src/sound/isa/sb/sb16_main.c: error: implicit declaration of function 'snd_dma_program': => 276
> > > + src/sound/isa/sb/sb8_main.c: error: implicit declaration of function 'snd_dma_pointer': => 425
> > > + src/sound/isa/sb/sb8_main.c: error: implicit declaration of function 'snd_dma_program': => 172
> > > + src/sound/isa/sscape.c: error: implicit declaration of function 'snd_dma_program': => 481
> > > + src/sound/isa/wss/wss_lib.c: error: implicit declaration of function 'snd_dma_disable': => 1695
> > > + src/sound/isa/wss/wss_lib.c: error: implicit declaration of function 'snd_dma_pointer': => 1160
> > > + src/sound/isa/wss/wss_lib.c: error: implicit declaration of function 'snd_dma_program': => 1025
> >
> > All of these source files do #include <sound/core.h>,
> > but these missing/implicit functions are conditional:
> >
> > #ifdef CONFIG_ISA_DMA_API
> > #define DMA_MODE_NO_ENABLE 0x0100
> >
> > void snd_dma_program(unsigned long dma, unsigned long addr, unsigned int size, unsigned short mode);
> > void snd_dma_disable(unsigned long dma);
> > unsigned int snd_dma_pointer(unsigned long dma, unsigned int size);
> > #endif
> >
> > so should these drivers all depend on ISA_DMA_API ?
>
> Yes, these are ISA sound cards, so they must have CONFIG_ISA and
> CONFIG_ISA_DMA. Although CONFIG_SND_SB8_DSP, CONFIG_SND_SB16_DSP
> and CONFIG_SND_WSS_LIB have no direct dependency, but the drivers
> selecting these are all in "if SND_ISA" in sound/isa/Kconfig.

Hm, yes, they are.

> > or should only pieces of each driver depend on ISA_DMA_API ?
>
> I wonder which configuration makes it possible.
> Did you see a similar problem with randconfig?

No. This parisc allmodconfig file from linux-next builds has
CONFIG_SND_ISA=y but it does not have CONFIG_ISA_DMA_API enabled. Weird.

Geert, do you know how this happens?

parisc config: http://kisskb.ellerman.id.au/kisskb/buildresult/4323458/config/
from this failed build: http://kisskb.ellerman.id.au/kisskb/buildresult/4323458/

thanks,
---
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***


\
 
 \ /
  Last update: 2011-07-25 18:13    [W:0.076 / U:0.880 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site