lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Tue, 8 Feb 2011 14:22:14 -0800 From: Randy Dunlap <randy.dunlap@...cle.com> To: linux-kernel@...r.kernel.org Cc: akpm@...ux-foundation.org, gregkh@...e.de, rmthomas@...olus.org Subject: Re: + staging-easycap-fix-build-when-snd-is-not-enabled.patch added to -mm tree Andrew, Please drop this patch. Tomas Winkler has supplied a better one which I have tested/acked. (sent only to staging/driver-devel mailing list) On Tue, 08 Feb 2011 13:35:37 -0800 akpm@...ux-foundation.org wrote: > > The patch titled > staging/easycap: fix build when SND is not enabled > has been added to the -mm tree. Its filename is > staging-easycap-fix-build-when-snd-is-not-enabled.patch > > Before you just go and hit "reply", please: > a) Consider who else should be cc'ed > b) Prefer to cc a suitable mailing list as well > c) Ideally: find the original patch on the mailing list and do a > reply-to-all to that, adding suitable additional cc's > > *** Remember to use Documentation/SubmitChecklist when testing your code *** > > See http://userweb.kernel.org/~akpm/stuff/added-to-mm.txt to find > out what to do about this > > The current -mm tree may be found at http://userweb.kernel.org/~akpm/mmotm/ > > ------------------------------------------------------ > Subject: staging/easycap: fix build when SND is not enabled > From: Randy Dunlap <randy.dunlap@...cle.com> > > Fix easycap build when CONFIG_SOUND is enabled but CONFIG_SND is > not enabled. > > These functions are only built when CONFIG_SND is enabled, so the > driver should depend on SND. > This means that having SND enabled is required for the (obsolete) > EASYCAP_OSS config option. > > drivers/built-in.o: In function `easycap_usb_disconnect': > easycap_main.c:(.text+0x2aba20): undefined reference to `snd_card_free' > drivers/built-in.o: In function `easycap_alsa_probe': > (.text+0x2b784b): undefined reference to `snd_card_create' > drivers/built-in.o: In function `easycap_alsa_probe': > (.text+0x2b78fb): undefined reference to `snd_pcm_new' > drivers/built-in.o: In function `easycap_alsa_probe': > (.text+0x2b7916): undefined reference to `snd_pcm_set_ops' > drivers/built-in.o: In function `easycap_alsa_probe': > (.text+0x2b795b): undefined reference to `snd_card_register' > drivers/built-in.o: In function `easycap_alsa_probe': > (.text+0x2b79d8): undefined reference to `snd_card_free' > drivers/built-in.o: In function `easycap_alsa_probe': > (.text+0x2b7a78): undefined reference to `snd_card_free' > drivers/built-in.o: In function `easycap_alsa_complete': > (.text+0x2b7e68): undefined reference to `snd_pcm_period_elapsed' > drivers/built-in.o:(.data+0x2cae8): undefined reference to `snd_pcm_lib_ioctl' > > Signed-off-by: Randy Dunlap <randy.dunlap@...cle.com> > Cc: R.M. Thomas <rmthomas@...olus.org> > Cc: Greg KH <gregkh@...e.de> > Signed-off-by: Andrew Morton <akpm@...ux-foundation.org> > --- > > drivers/staging/easycap/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff -puN drivers/staging/easycap/Kconfig~staging-easycap-fix-build-when-snd-is-not-enabled drivers/staging/easycap/Kconfig > --- a/drivers/staging/easycap/Kconfig~staging-easycap-fix-build-when-snd-is-not-enabled > +++ a/drivers/staging/easycap/Kconfig > @@ -1,6 +1,6 @@ > config EASYCAP > tristate "EasyCAP USB ID 05e1:0408 support" > - depends on USB && VIDEO_DEV && SOUND > + depends on USB && VIDEO_DEV && SND > > ---help--- > This is an integrated audio/video driver for EasyCAP cards with > _ --- ~Randy *** Remember to use Documentation/SubmitChecklist when testing your code *** -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists