On Tue, Feb 14, 2006 at 04:21:50PM +0100, Axel Thimm wrote: > On Tue, Feb 14, 2006 at 09:57:00AM -0500, Andrew Dodd wrote: > > Quoting Axel Thimm <[EMAIL PROTECTED]>: > > > > >> This isn't true if you need Samsung tuner support - the kernel > > >> modules do not have support for the Samsung TCPN tuner on new > > >> PVR-500s, and so in the case of PVR-500 boards with the new > > >> Samsung tuner, you have to go back to the "old way" of doing > > >> things. Of course, if they're using RPMs that didn't have the > > >> Kbuild files modified to build those modules in the first > > >> place, things won't work anyway.
> > > Hans turned off the build of certain kernel modules for 2.6.15
> > > because the support is now in the kernel. Did you try the
> > > Samsung tuner support with 2.6.15? If ivtv 0.4.2 had better
> > > tuner kernel modules than 2.6.15, Hans wouldn't had turned it
> > > off.
>
> > No, it makes sense to turn building of ivtv's own tveeprom/tuner
> > modules off by default, since they are of no benefit to 95%+ of
> > ivtv users
>
> Oh, no, we want 100% !!! :)
>
> > - only those who have Samsung tuners on a recent PVR-500 (not
> > supported by the 2.6.15 kernel modules, as Samsung support was added
> > in early-to-mid January, which is after 2.6.15's release) have a
> > need for the ivtv tuner modules, which is why getting them to build
> > requires modifying one of the Kbuild files in the ivtv distribution.
>
> Then that's a bug in ivtv 0.4.2 logic of deciding which kernel modules
> to build. I'll look into it.
>
> > The end conclusion is: Do not try to use precompiled RPMs for a PVR-500
> > with
> > the new Samsung tuner, at least not with kernel 2.6.15 + ivtv 0.4.2.
>
> As I see it 0.4.2 needs an extra functional patch for Samasung & NTSC
> and fixing its build targets for 2.6.15 (e.g. including
> tuner.ko). Correct?
Here is a patch for allowing 2.6.15 to build the neccessary modules:
Index: ivtv/branches/0.4/driver/Kbuild
===================================================================
--- ivtv/branches/0.4/driver/Kbuild (revision 3162)
+++ ivtv/branches/0.4/driver/Kbuild (working copy)
@@ -5,12 +5,15 @@
ivtv-cards.o v4l1-compat.o ivtv-yuv.o
ivtv-fb-objs := ivtv-osd.o ivtv-queue.o
-NO_SUP_MODULES := $(shell test $(SUBLEVEL) -ge 15 -a $(PATCHLEVEL) -ge 6 -a
"$(CONFIG_VIDEO_DECODER)" -a "$(CONFIG_VIDEO_AUDIO_DECODER)" -a
"$(CONFIG_VIDEO_TUNER)" && echo 1)
-ifneq ($(NO_SUP_MODULES),1)
+NO_DECODER_MODULES := $(shell test $(SUBLEVEL) -ge 15 -a $(PATCHLEVEL) -ge 6
-a "$(CONFIG_VIDEO_DECODER)" -a "$(CONFIG_VIDEO_AUDIO_DECODER)" && echo 1)
+NO_TUNER_MODULES := $(shell test $(SUBLEVEL) -ge 16 -a $(PATCHLEVEL) -ge 6 -a
"$(CONFIG_VIDEO_TUNER)" && echo 1)
+ifneq ($(NO_DECODER_MODULES),1)
cx25840-objs := cx25840-core.o cx25840-audio.o cx25840-firmware.o cx25840-vbi.o
obj-m += msp3400.o saa7115.o cx25840.o wm8775.o saa7127.o cs53l32a.o
+endif
+ifneq ($(NO_TUNER_MODULES),1)
ifndef NO_IVTV_TUNER
obj-m += tuner.o tveeprom.o tda9887.o
endif
--
Axel.Thimm at ATrpms.net
pgpRBGadnqwQY.pgp
Description: PGP signature
_______________________________________________ ivtv-devel mailing list [email protected] http://ivtvdriver.org/mailman/listinfo/ivtv-devel
