Re: [PATCH] media: omap3isp: video: Use v4l2_get_timestamp()

2015-03-14 Thread Sakari Ailus
Thanks!

On Fri, Mar 13, 2015 at 02:31:27AM +0200, Laurent Pinchart wrote:
 Replace the open-coded copy by a function call.
 
 Signed-off-by: Laurent Pinchart laurent.pinch...@ideasonboard.com

Acked-by: Sakari Ailus sakari.ai...@iki.fi

-- 
Sakari Ailus
e-mail: sakari.ai...@iki.fi XMPP: sai...@retiisi.org.uk
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH] media: omap3isp: video: Use v4l2_get_timestamp()

2015-03-12 Thread Laurent Pinchart
Replace the open-coded copy by a function call.

Signed-off-by: Laurent Pinchart laurent.pinch...@ideasonboard.com
---
 drivers/media/platform/omap3isp/ispvideo.c | 5 +
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/drivers/media/platform/omap3isp/ispvideo.c 
b/drivers/media/platform/omap3isp/ispvideo.c
index 3de8d5d..0b5967e 100644
--- a/drivers/media/platform/omap3isp/ispvideo.c
+++ b/drivers/media/platform/omap3isp/ispvideo.c
@@ -452,7 +452,6 @@ struct isp_buffer *omap3isp_video_buffer_next(struct 
isp_video *video)
enum isp_pipeline_state state;
struct isp_buffer *buf;
unsigned long flags;
-   struct timespec ts;
 
spin_lock_irqsave(video-irqlock, flags);
if (WARN_ON(list_empty(video-dmaqueue))) {
@@ -465,9 +464,7 @@ struct isp_buffer *omap3isp_video_buffer_next(struct 
isp_video *video)
list_del(buf-irqlist);
spin_unlock_irqrestore(video-irqlock, flags);
 
-   ktime_get_ts(ts);
-   buf-vb.v4l2_buf.timestamp.tv_sec = ts.tv_sec;
-   buf-vb.v4l2_buf.timestamp.tv_usec = ts.tv_nsec / NSEC_PER_USEC;
+   v4l2_get_timestamp(buf-vb.v4l2_buf.timestamp);
 
/* Do frame number propagation only if this is the output video node.
 * Frame number either comes from the CSI receivers or it gets
-- 
Regards,

Laurent Pinchart

--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html