On Fri, 20 Apr 2018 15:01:22 +0200, Mauro Carvalho Chehab wrote: > > Em Fri, 20 Apr 2018 09:51:29 -0300 > Mauro Carvalho Chehab <mchehab@xxxxxxxxxxxxxxxx> escreveu: > > > Em Fri, 20 Apr 2018 14:37:46 +0200 > > Takashi Iwai <tiwai@xxxxxxx> escreveu: > > > > > On Fri, 20 Apr 2018 14:32:15 +0200, > > > Mauro Carvalho Chehab wrote: > > > > > > > > All sound drivers that don't depend on PNP can be safelly > > > > build with COMPILE_TEST, as ISA provides function stubs to > > > > be used for such purposes. > > > > > > > > As a side effect, with this change, the radio-miropcm20 > > > > can now be built outside i386 with COMPILE_TEST. > > > > > > > > Signed-off-by: Mauro Carvalho Chehab <mchehab@xxxxxxxxxxxxxxxx> > > > > --- > > > > drivers/media/radio/Kconfig | 3 ++- > > > > sound/isa/Kconfig | 9 +++++---- > > > > 2 files changed, 7 insertions(+), 5 deletions(-) > > > > > > > > diff --git a/drivers/media/radio/Kconfig b/drivers/media/radio/Kconfig > > > > index d363726e9eb1..8fa403c7149e 100644 > > > > --- a/drivers/media/radio/Kconfig > > > > +++ b/drivers/media/radio/Kconfig > > > > @@ -372,7 +372,8 @@ config RADIO_GEMTEK_PROBE > > > > > > > > config RADIO_MIROPCM20 > > > > tristate "miroSOUND PCM20 radio" > > > > - depends on ISA && ISA_DMA_API && VIDEO_V4L2 && SND > > > > + depends on ISA || COMPILE_TEST > > > > + depends on ISA_DMA_API && VIDEO_V4L2 && SND > > > > select SND_ISA > > > > select SND_MIRO > > > > ---help--- > > > > diff --git a/sound/isa/Kconfig b/sound/isa/Kconfig > > > > index cb54d9c0a77f..d2a6cdd0395c 100644 > > > > --- a/sound/isa/Kconfig > > > > +++ b/sound/isa/Kconfig > > > > @@ -20,7 +20,8 @@ config SND_SB16_DSP > > > > > > > > menuconfig SND_ISA > > > > bool "ISA sound devices" > > > > - depends on ISA && ISA_DMA_API > > > > + depends on ISA || COMPILE_TEST > > > > + depends on ISA_DMA_API > > > > default y > > > > help > > > > Support for sound devices connected via the ISA bus. > > > > @@ -38,7 +39,7 @@ config SND_ADLIB > > > > > > > > config SND_AD1816A > > > > tristate "Analog Devices SoundPort AD1816A" > > > > - depends on PNP > > > > + depends on PNP && ISA > > > > select ISAPNP > > > > select SND_OPL3_LIB > > > > select SND_MPU401_UART > > > > > > Just from curiosity: what's the reason for this explicit CONFIG_ISA > > > dependency? What error did you get? > > > > Kconfig complains with "select ISAPNP": > > > > WARNING: unmet direct dependencies detected for ISAPNP > > Depends on [n]: PNP [=y] && ISA [=n] > > Selected by [y]: > > - SND_AD1816A [=y] && SOUND [=y] && !UML && SND [=y] && SND_ISA [=y] && PNP [=y] > > > > Because it is declared as: > > > > config ISAPNP > > bool "ISA Plug and Play support" > > depends on ISA > > > > I could have tried to change ISAPNP to depends on ISA || COMPILE_TEST, > > but I suspect that would touch on yet another subsystem and has > > the potential to point to other things that need changes, as > > a lot more drivers will be selected. > > > > Anyway, after a quick look at include/linux/isapnp.h, I suspect > > that this can work. > > > > I'll run some tests here. > > Yes, removing the ISAPNP dependency if COMPILE_TEST is trivial too. > > Just sent a separate patch to be applied after this one with such > removal. > > I opted to make it as a separate patch as, if the drivers there > fail to build on some weird architecture, we won't need to discard > this one. OK, then feel free to take my ack: Acked-by: Takashi Iwai <tiwai@xxxxxxx> But still it'd be better to mention about the CONFIG_ISA change, even though it'll be removed again after the ISAPNP COMIPLE_TEST patch. thanks, Takashi