lkml.org 
[lkml]   [2005]   [Nov]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
SubjectRe: Linux 2.6.15-rc2
On Tue, Nov 22, 2005 at 11:36:48PM -0500, Gene Heskett wrote:
>...
> Well, I just went thru it again, and turned off everything but the
> cx8800 and ORv51132 stuffs, and now I get this at the and of the
> 'makeit' script I use here:
>
> WARNING:
> /lib/modules/2.6.15-rc2/kernel/drivers/media/video/cx88/cx88-dvb.ko
> needs unknown symbol mt352_attach
> WARNING:
> /lib/modules/2.6.15-rc2/kernel/drivers/media/video/cx88/cx88-dvb.ko
> needs unknown symbol nxt200x_attach
> WARNING:
> /lib/modules/2.6.15-rc2/kernel/drivers/media/video/cx88/cx88-dvb.ko
> needs unknown symbol mt352_write
> WARNING:
> /lib/modules/2.6.15-rc2/kernel/drivers/media/video/cx88/cx88-dvb.ko
> needs unknown symbol lgdt330x_attach
> WARNING:
> /lib/modules/2.6.15-rc2/kernel/drivers/media/video/cx88/cx88-dvb.ko
> needs unknown symbol cx22702_attach
>...

Nice catch and thanks for your report.

The bug is obvious. A possible patch is below (and at least
drivers/media/video/saa7134/Makefile contains the same bug),
but I'd really prfer getting rid of the -DHAVE_* stuff in the
Makefiles and using Kconfig variables instead.

Would such a patch be accepted?

> Cheers, Gene

cu
Adrian

BTW: Please don't strip the Cc whenreplying to linux-kernel.



--- linux-2.6.15-rc2/drivers/media/video/cx88/Makefile.old 2005-11-23 18:34:07.000000000 +0100
+++ linux-2.6.15-rc2/drivers/media/video/cx88/Makefile 2005-11-23 18:34:18.000000000 +0100
@@ -9,21 +9,21 @@
EXTRA_CFLAGS += -I$(src)/..
EXTRA_CFLAGS += -I$(srctree)/drivers/media/dvb/dvb-core
EXTRA_CFLAGS += -I$(srctree)/drivers/media/dvb/frontends
-ifneq ($(CONFIG_VIDEO_BUF_DVB),n)
+ifneq ($(CONFIG_VIDEO_BUF_DVB),)
EXTRA_CFLAGS += -DHAVE_VIDEO_BUF_DVB=1
endif
-ifneq ($(CONFIG_DVB_CX22702),n)
+ifneq ($(CONFIG_DVB_CX22702),)
EXTRA_CFLAGS += -DHAVE_CX22702=1
endif
-ifneq ($(CONFIG_DVB_OR51132),n)
+ifneq ($(CONFIG_DVB_OR51132),)
EXTRA_CFLAGS += -DHAVE_OR51132=1
endif
-ifneq ($(CONFIG_DVB_LGDT330X),n)
+ifneq ($(CONFIG_DVB_LGDT330X),)
EXTRA_CFLAGS += -DHAVE_LGDT330X=1
endif
-ifneq ($(CONFIG_DVB_MT352),n)
+ifneq ($(CONFIG_DVB_MT352),)
EXTRA_CFLAGS += -DHAVE_MT352=1
endif
-ifneq ($(CONFIG_DVB_NXT200X),n)
+ifneq ($(CONFIG_DVB_NXT200X),)
EXTRA_CFLAGS += -DHAVE_NXT200X=1
endif
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
\
 
 \ /
  Last update: 2005-11-23 18:45    [W:0.112 / U:22.008 seconds]
©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site