Re: [PATCH 1/1] V4L: stk11xx, add a new webcam driver

["Markus Rechberger" <mrechberger@gmail.com>]

On 5/28/07, Mauro Carvalho Chehab <mchehab@infradead.org> wrote:
> Em Seg, 2007-05-28 às 17:14 +0200, Markus Rechberger escreveu:
>
> > > > +/*
> > > > + * Bayer conversion
> > > > + */
> > >
> > > We don't do format conversions in kernel. Instead, you should return a
> > > proper Bayer Fourcc format (like V4L2_PIX_FMT_SBGGR8).
> > >
> >
> > It's ok in his case since most userspace applications do not support
> > this format, it helps noone if there's a driver out there which isn't
> > supported by most userspace applications and I don't expect that all
> > applications will add a conversion for that format soon.
> > This would moreover raise the question about a userspace library,
> > though there are many more important issues which are outstanding and
> > which got smashed down.
>
> As Luca pointed, if we add conversion for one driver, we should add for
> the rest.
>
> Instead, it would be better if Jiri sends the decoding and the rescaling
> stuff as a patch to v4l2-apps/lib, starting the API decoding library.
> Once we have a library, we can ask the userspace developers to use it
> for the formats not recognized by their userspace apps.

I think it would be better to evaluate existing solutions (eg. libpw).
Instead starting another one (which would lack any design since I
haven't seen any thread that was picking together requirements),
improving that existing one might also be helpful for applications
like ekiga.
As for libpw it also supports colourspace conversion and software
scaling, the used algorithms seem to need some improvement, but alot
work has already been done there.
Of course other libraries should also be taken into account, maybe
someone knows something better which is more complete.

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