Hi Marek On Mon, 29 Aug 2011, Marek Szyprowski wrote: > This patch changes the order of operations during stream on call. Now the > buffers are first queued to the driver and then the start_streaming method > is called. > > This resolves the most common case when the driver needs to know buffer > addresses to enable dma engine and start streaming. Additional parameter > to start_streaming method have been added to simplify drivers code. The > driver are now obliged to check if the number of queued buffers is high > enough to enable hardware streaming. If not - it can return an error. In > such case all the buffers that have been pre-queued are invalidated. > > This patch also updates all videobuf2 clients to work properly with the > changed order of operations. > > Signed-off-by: Marek Szyprowski <m.szyprowski@xxxxxxxxxxx> > Signed-off-by: Kyungmin Park <kyungmin.park@xxxxxxxxxxx> > CC: Pawel Osciak <pawel@xxxxxxxxxx> > CC: Guennadi Liakhovetski <g.liakhovetski@xxxxxx> > CC: Hans Verkuil <hverkuil@xxxxxxxxx> > CC: Tomasz Stanislawski <t.stanislaws@xxxxxxxxxxx> > CC: Sylwester Nawrocki <s.nawrocki@xxxxxxxxxxx> > CC: Kamil Debski <k.debski@xxxxxxxxxxx> > CC: Jonathan Corbet <corbet@xxxxxxx> > CC: Josh Wu <josh.wu@xxxxxxxxx> > CC: Hans de Goede <hdegoede@xxxxxxxxxx> > CC: Paul Mundt <lethal@xxxxxxxxxxxx> > --- > > Hello, > > This is yet another version of the patch that introduces significant > changes in the vb2 streamon operation. I've decided to remove the > additional parameter to buf_queue callback and added a few cleanups here > and there. This patch also includes an update for all vb2 clients. Just for the record: These are not all vb2 clients. A simple grep for something like vb2_ops gives you also drivers/media/video/mx3_camera.c drivers/media/video/sh_mobile_ceu_camera.c Thanks Guennadi --- Guennadi Liakhovetski, Ph.D. Freelance Open-Source Software Developer http://www.open-technology.de/ -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
