Re: [Alsa-devel] [2.6 patch] let SND_SUPPORT_OLD_API depend on SND_PCM

[Takashi Iwai]

At Tue, 10 Jan 2006 18:08:36 +0100,
Adrian Bunk wrote:
> 
> On Tue, Jan 10, 2006 at 06:00:29PM +0100, Takashi Iwai wrote:
> > At Tue, 10 Jan 2006 17:44:49 +0100,
> > Adrian Bunk wrote:
> > > 
> > > SND_SUPPORT_OLD_API only has an effect if SND_PCM is set.
> > > 
> > > 
> > > Signed-off-by: Adrian Bunk <bunk@xxxxxxxxx>
> > > 
> > > --- linux-2.6.15-mm2-full/sound/core/Kconfig.old	2006-01-10 17:35:35.000000000 +0100
> > > +++ linux-2.6.15-mm2-full/sound/core/Kconfig	2006-01-10 17:36:07.000000000 +0100
> > > @@ -124,7 +124,7 @@
> > >  
> > >  config SND_SUPPORT_OLD_API
> > >  	bool "Support old ALSA API"
> > > -	depends on SND
> > > +	depends on SND_PCM
> > >  	default y
> > >  	help
> > >  	  Say Y here to support the obsolete ALSA PCM API (ver.0.9.0 rc3
> > 
> > Does it work?  CONFIG_SND_PCM is selected by the drivers.  So, it will
> > be N until any drivers are selected.
> 
> It does work and "make oldconfig" handles it perfectly, but I understand 
> your point that it might be accidentially set to N if a user later 
> selects a driver in "make {menu,x}oldconfig", and that this case is most 
> likely better handled without my patch.

Yes, exactly.  IMO, it's safer to avoid this dependency check.


Takashi
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/