Re: [PATCH 2/2] omap3isp: ccdc: Add support to ITU-R BT.656 video data format

2011-10-09 Thread Laurent Pinchart
Hi Javier, Thanks for the patch. On Sunday 09 October 2011 04:37:33 Javier Martinez Canillas wrote: The ITU-R BT.656 standard data format provides interlaced video data. This patch adds to the ISP CCDC driver the ability to deinterlace the video data and send progressive frames to

Re: [PATCH 2/2] omap3isp: ccdc: Add support to ITU-R BT.656 video data format

2011-10-09 Thread Javier Martinez Canillas
On Sun, Oct 9, 2011 at 12:02 PM, Laurent Pinchart laurent.pinch...@ideasonboard.com wrote: Hi Javier, Thanks for the patch. Hi Laurent, I'm so glad that you are providing feedback on this :) On Sunday 09 October 2011 04:37:33 Javier Martinez Canillas wrote: The ITU-R BT.656 standard data

Re: [PATCH 2/2] omap3isp: ccdc: Add support to ITU-R BT.656 video data format

2011-10-09 Thread Enrico
On Sun, Oct 9, 2011 at 12:02 PM, Laurent Pinchart laurent.pinch...@ideasonboard.com wrote: Hi Javier, Thanks for the patch. Laurent, apart from the specific comments on Javier code, did you have a look at Deepthy patches too? I say this because asking Javier for fixes means (to me) that you

Re: [PATCH 2/2] omap3isp: ccdc: Add support to ITU-R BT.656 video data format

2011-10-09 Thread Javier Martinez Canillas
On Sun, Oct 9, 2011 at 2:58 PM, Enrico ebut...@users.berlios.de wrote: On Sun, Oct 9, 2011 at 12:02 PM, Laurent Pinchart laurent.pinch...@ideasonboard.com wrote: Hi Javier, Thanks for the patch. Laurent, apart from the specific comments on Javier code, did you have a look at Deepthy

Re: [PATCH 2/2] omap3isp: ccdc: Add support to ITU-R BT.656 video data format

2011-10-09 Thread Javier Martinez Canillas
On Sun, Oct 9, 2011 at 2:58 PM, Enrico ebut...@users.berlios.de wrote: On Sun, Oct 9, 2011 at 12:02 PM, Laurent Pinchart laurent.pinch...@ideasonboard.com wrote: Hi Javier, Thanks for the patch. Laurent, apart from the specific comments on Javier code, did you have a look at Deepthy

[PATCH 2/2] omap3isp: ccdc: Add support to ITU-R BT.656 video data format

2011-10-08 Thread Javier Martinez Canillas
The ITU-R BT.656 standard data format provides interlaced video data. This patch adds to the ISP CCDC driver the ability to deinterlace the video data and send progressive frames to user-space applications. The changes are: - Maintain two buffers (struct isp_buffer), current and last. -