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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-Id: <201106292117.40272.mitov@issp.bas.bg>
Date:	Wed, 29 Jun 2011 21:17:39 +0300
From:	Marin Mitov <mitov@...p.bas.bg>
To:	Greg KH <greg@...ah.com>
Cc:	linux-kernel@...r.kernel.org
Subject: Re: [PATCH] staging: use videobuf2 framework for drivers/staging/dt3155v4l driver

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