Re: [PATCH] staging: use videobuf2 framework for drivers/staging/dt3155v4l driver

[Marin Mitov <mitov@issp.bas.bg>]

On Wednesday, June 29, 2011 07:21:24 am Greg KH wrote:
> On Wed, Jun 29, 2011 at 06:46:02AM +0300, Marin Mitov wrote:
> > On Tuesday, June 28, 2011 11:25:15 pm Greg KH wrote:
> > > On Sat, May 28, 2011 at 09:45:27PM +0300, Marin Mitov wrote:
> > > > This patch transforms drivers/staging/dt3155v4l driver to use videobuf2 framework.
> > > > Tested and works with "xawtv -f".
> > > > 
> > > > Either streaming API or read method should be selected during kernel configuration.
> > > 
> > > I've applied this, but why would you ever not want to use the streaming
> > > api?
> > 
> > There are no reasons for that, sure, but for now xawtv do not work if both are
> > specified in the driver. I have to debug it.
> 
> Ok, but why not just remove the read method then?  Or do some userspace
> programs still require it?

In current version if you select streaming API you effectively remove read method.
If you select read method you effectively remove streaming API. At lest this is
what userland programs are seeing as driver capabilities.

For now I see some changes in videobuf2-core are in discussion. When they become part
of the kernel I will test the driver again.

Thanks.

Marin Mitov

> 
> greg k-h
>