Hey Billy,
On Sun, 2003-02-23 at 19:57, Billy Biggs wrote:
> Just a warning to V4L application developers, the rivatv capture
> driver does format conversion from UYVY to everything else it claims to
> support. This answers part of the performance problems users of tvtime
> were experiencing when using this driver, since we ask for YUY2, and the
> driver converts.
I share your concerns. 1) rivatv will never make it into the kernel like
this (see the philips webcam driver as a perfect example of what happens
to such drivers), and 2) it does more evil things.
Its vpx32xx module, for example, handles vpx3214c, vpx3216b, vpx3220a,
vpx3224d, vpx3225d chipsets. However, the vpx322[45]d on one side and
the others on the other side are incompatible. The result is a messy
driver which uses if (vpx322[45]d) { .. } else { .. } statements all
over the place.
The zoran driver has a rewritten vpx3220 module which works for the
vpx3214c, vpx3216b, vpx3220a chipsets. I'm working on a vpx3224 module
that works for the vpx3224d, vpx3225d, vpx3224e, vpx3225e and vpx3226e
(all of these are compatible). I.m.o., this is the correct way of
handling this. I've discussed this with Laurent and he agreed
completely. I'd like to know the opinions of others on this too (is
there any 'official' policy on these issues?).
Maybe we should try to discuss this with the rivatv developers?
Ronald
--
Ronald Bultje <rbultje@xxxxxxxxxxxxxxxxxxx>
Linux Video/Multimedia developer
