lkml.org 
[lkml]   [2015]   [Feb]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH 2/3] media: omap3isp: ispvideo: drop driver specific isp_video_fh
Date
Hi Prabhakar,

Thank you for the patch.

On Monday 23 February 2015 20:19:32 Lad Prabhakar wrote:
> From: "Lad, Prabhakar" <prabhakar.csengg@gmail.com>
>
> this patch drops driver specific isp_video_fh, as this
> can be handled by core.

I'm afraid it's not that simple.

The omap3isp driver stores video queues per file handle for a reason. This was
design to permit creating a high-resolution still image capture queue and
prepare buffers ahead of time, to avoid the large delay due to cache
management as prepare time when taking the snapshot.

Now this use case has been partially solved by VIDIOC_CREATE_BUFS, but we're
still missing a VIDIOC_DESTROY_BUFS to make it work completely. That needs to
be solved first.

> Signed-off-by: Lad, Prabhakar <prabhakar.csengg@gmail.com>
> ---
> drivers/media/platform/omap3isp/ispvideo.c | 128 ++++++++++---------------
> drivers/media/platform/omap3isp/ispvideo.h | 13 +--
> 2 files changed, 49 insertions(+), 92 deletions(-)

--
Regards,

Laurent Pinchart



\
 
 \ /
  Last update: 2015-02-24 01:41    [W:0.477 / U:0.268 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site