On Wed, Nov 23, 2005 at 02:17:45PM -0500, Michael Krufky wrote:
> Adrian Bunk wrote:
>
> >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.
> >
> >
> We need to keep the -DHAVE_FOO stuff there, in order to satisfy the
> following requirements:
>
> 1) To allow the option of only selecting those frontends required by
> specific dvb hardware, without forcing all modules to be loaded... This
> feature is optional, and I implemented it in response to the demand from
> some hybrid v4l/dvb device users, (and myself) Why force a driver to
> load every frontend module if it isnt required by the hardware? --
> apparantly the implementation was less than perfect. I had originally
> intended for this to live in -mm for a bit, but when the merge window
> came around, Mauro had sent it upstream before I had the chance to
> create alternate patches for linus' tree.
>
> 2) (more importantly) To allow v4l-kernel cvs to retain backwards
> compatability with older kernels..
>
> I had originally tried to rename these to use the Kconfig variables, but
> LKML people asked for it to be changed back.
>
> Please do not remove this feature -- if it is broken, then we should try
> to fix it, rather than remove it. If the specific frontend selection
> isn't working, then I guess we can revert back to the old behavior where
> every frontend is forced, but I would rather not.
I do not yet know how to fix it, but configurations like
CONFIG_VIDEO_CX88_DVB=y, CONFIG_DVB_CX22702=m are currently compile
errors.
> Michael Krufky
cu
Adrian
--
"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
