On Tue, Nov 23, 2004 at 01:09:15AM +0100, Luca RIsolia wrote:

> > > Since format conversion is generally not allowed in the kernel anymore,
> > > I'll probably have to remove it eventually. No great loss though; the
> > > code should work just as well from userspace ;)
> 
> At the moment, V4L1 does not support the Bayer format. I think the
> "ovcamchip" module still needs to output in YUV format by default, while
> providing the Bayer format as an option, otherwise V4L1 drivers depending
> on ovcamchip will stop working.

Ok, so you're suggesting both to keep the Bayer decoding and output in
YUV ...

> 
> > (I'll have to do this, since the robot needs every CPU cycle it can get,
> > and cannot waste them on video frame copying.)
> 
> We all want every CPU cycle we can get, so we should really avoid
> convertions in kernel space :)
> 
> > However, I believe a simple Bayer decoding implementation (like mine)
> > should be included in the kernel driver, since only GnomeMeeting so far
> > supports Bayer decoding in userspace.
> 
> This is not the best approach. Apart from the disadvantages in terms
> of performance, no applications will be ever patched if colorspace
> convertions are done in kernel-space. So I suggest that you fix your
> applications or libraries.

... and to remove the Bayer decoding and output in BGGR8 only? 

Well, the driver won't be able to output in YUV, since that'd require
a real colorspace conversion. At this time it's giving output in BGR24,
converted from BGGR8.

I want BGR24 as an option in addition to the default raw BGGR8.

> > Interesting. For my robotics project I simply flipped the camera and
> > removed the software flip implementation altogether.
> 
> To preserve the R,G or B origin in the output from the image sensor,
> you should shift the active pixel area (w=2a,h=2b) one pixel in the
> appropriate direction.
 
I can't. The camera is sending 640x480 pixels out. If I shifted the
active area I'd get a non-standard (smaller) image size. I'll see
whether the image size being sent could be increased or shifted in HW,
because the sensor size is a little larger, but I seriously doubt it -
the window registers seem to be ignored in raw mode.
-- 
Vojtech Pavlik
SuSE Labs, SuSE CR


-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now. 
http://productguide.itmanagersjournal.com/
_______________________________________________
[EMAIL PROTECTED]
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel

Reply via email to