summaryrefslogtreecommitdiff
path: root/linux/drivers/media/video/usbvideo/ibmcam.c
diff options
context:
space:
mode:
authorMauro Carvalho Chehab <mchehab@redhat.com>2008-10-11 08:47:28 -0300
committerMauro Carvalho Chehab <mchehab@redhat.com>2008-10-11 08:47:28 -0300
commit21e61b2c212edb8d0c30ed8098b04dd1b74bb000 (patch)
tree5089f9d0674d0a23933006184ab4a46c39058d12 /linux/drivers/media/video/usbvideo/ibmcam.c
parent59bd3f15af822cf71839e0cfc59f5979b6e2e6c7 (diff)
downloadmediapointer-dvb-s2-21e61b2c212edb8d0c30ed8098b04dd1b74bb000.tar.gz
mediapointer-dvb-s2-21e61b2c212edb8d0c30ed8098b04dd1b74bb000.tar.bz2
af9013/m5602: Fix compilation at in-kernel
From: Mauro Carvalho Chehab <mchehab@redhat.com> This patch fixes a few errors that were breaking in-kernel compilation. The first one were at m5602 that were including "gspca.h" from another directory. A trivial change at m5602/Makefile fixed the issue. A harder to fix error were caused by the fact that both af9013 and m5602 were declaring an external var called 'debug'. If you compile both modules in-kernel, you'll have troubles. This is even worse, since there are two other drivers outside the subsystem with a similar mistake. To make things worse, gcc do a very great job of fooling the developers, by blaming the wrong lines anf files when such error occurs. So, I was having errors like: drivers/media/dvb/built-in.o: In function `dvb_usercopy': /home/v4l/tokernel/git/drivers/media/dvb/dvb-core/dvbdev.c:403: multiple definition of `debug' drivers/media/video/built-in.o:/home/v4l/tokernel/git/drivers/media/video/bt819.c:264: first defined here or: drivers/media/dvb/built-in.o: In function `dvb_usercopy': /home/v4l/tokernel/git/drivers/media/dvb/dvb-core/dvbdev.c:403: multiple definition of `debug' drivers/media/video/built-in.o:/home/v4l/tokernel/git/drivers/media/video/vpx3220.c:125: first defined here The two above errors caused by m5602 driver. After fixing m5602, I got this: drivers/built-in.o: In function `SiS_SetGroup2': /home/v4l/tokernel/git/drivers/video/sis/init301.c:6996: multiple definition of `debug' arch/x86/kernel/built-in.o:/home/v4l/tokernel/git/arch/x86/kernel/entry_32.S:810: first defined here This one caused by af9013. This patch fixes those erros by prefixing the debug vars with the name of the module, without changing the name of the parameter. Next time, please prepend all extern vars with the name of the module. To avoid bisect troubles, I've folded those changes with the original commits that added those modules at my -git tree. kernel-sync: CC: Jean-Francois Moine <moinejf@free.fr> CC: Antti Palosaari <crope@iki.fi> CC: Erik Andrén <erik.andren@gmail.com> Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'linux/drivers/media/video/usbvideo/ibmcam.c')
0 files changed, 0 insertions, 0 deletions