Re: [PATCH] bttv driver II

From: Gerd Knorr (kraxel@goldbach.in-berlin.de)
Date: Mon May 22 2000 - 14:50:31 EST


On Mon, 22 May 2000, Rui Sousa wrote:

> > > $CONFIG_SOUND
> > > must have the same value. It's not what I wrote above but there is some
> > > dependence
> > > (trough the msp3400.c file).
> >
> > The msp3400 driver is not required
>
> obj-$(CONFIG_VIDEO_BT848) += bttv.o msp3400.o \
> tda7432.o tda8425.o tda985x.o tda9875.o tea6300.o tea6420.o

msp3400.c should compile fine with sound disabled, the mixer stuff is'nt
included then (is'nt required to decode the TV audio signal, its just a
additional goodie).

I plan to move the mixer stuff to another module because (a) it seems to
be quite confusing and (b) some of the other TV audio chips listed above
have have volume/treble/bass registers too, so it would be nice to have
one generic tvmixer module...

> Which may be why sometimes I need to generate interrupts
> (change channels, overwrite the window,...) to get the image to sync again...

Thats probably another problem. It happens if bttv wants to reset the
bt848 because of too many errors. The reset involves some actions
(calling set_pll() for example) which can't be done from irq context,
thats why bttv's irq handler just stops dma and sets a flag. Next time
some program calls a ioctl() bttv resets the chip an restarts everything.
If someone has a more clever way to handle this, hints are welcome...

  Gerd

-- 
Protecting the children is a good way to get a lot of adults who cant
stand up for themselves.		-- seen in some sig on /.

- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.rutgers.edu Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Tue May 23 2000 - 21:00:22 EST