From mboxrd@z Thu Jan 1 00:00:00 1970 From: Randy Dunlap Subject: Re: linux-next: Tree for May 1 (media/usb/stk1160) Date: Wed, 01 May 2013 13:23:25 -0700 Message-ID: <518179BD.3010407@infradead.org> References: <20130501183734.7ad1efca2d06e75432edabbd@canb.auug.org.au> <518157EB.3010700@infradead.org> <20130501192845.GA18811@free.fr> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: linux-kbuild-owner@vger.kernel.org To: David Rientjes Cc: "Yann E. MORIN" , Stephen Rothwell , linux-next@vger.kernel.org, linux-kernel@vger.kernel.org, linux-media , linux-kbuild@vger.kernel.org List-Id: linux-next.vger.kernel.org On 05/01/13 12:58, David Rientjes wrote: > On Wed, 1 May 2013, Yann E. MORIN wrote: > >>> When CONFIG_SND=m and CONFIG_SND_AC97_CODEC=m and >>> CONFIG_VIDEO_STK1160=y >>> CONFIG_VIDEO_STK1160_AC97=y >>> >>> drivers/built-in.o: In function `stk1160_ac97_register': >>> (.text+0x122706): undefined reference to `snd_card_create' >>> drivers/built-in.o: In function `stk1160_ac97_register': >>> (.text+0x1227b2): undefined reference to `snd_ac97_bus' >>> drivers/built-in.o: In function `stk1160_ac97_register': >>> (.text+0x1227cd): undefined reference to `snd_card_free' >>> drivers/built-in.o: In function `stk1160_ac97_register': >>> (.text+0x12281b): undefined reference to `snd_ac97_mixer' >>> drivers/built-in.o: In function `stk1160_ac97_register': >>> (.text+0x122832): undefined reference to `snd_card_register' >>> drivers/built-in.o: In function `stk1160_ac97_unregister': >>> (.text+0x12285e): undefined reference to `snd_card_free' >>> >>> >>> This kconfig fragment: >>> config VIDEO_STK1160_AC97 >>> bool "STK1160 AC97 codec support" >>> depends on VIDEO_STK1160 && SND > > This doesn't depend on SND, it depends on SND=y. > -- Maybe this option *should* depend on SND=y, but that's not what the kconfig syntax says. The kconfig language does not care if the variable is a bool or a tristate when evaluating a depends expression AFAICT (but I am only reading Documentation/kbuild/kconfig-language.txt, not the source code). -- ~Randy