Re: [PATCH 3/4] omap3isp: ccdc: Add crop support on output formatter source pad

2012-04-24 Thread Sakari Ailus
Laurent Pinchart wrote: Hi Sakari, On Tuesday 24 April 2012 18:14:20 Sakari Ailus wrote: On Tue, Apr 24, 2012 at 11:08:12AM +0200, Laurent Pinchart wrote: On Tuesday 24 April 2012 01:23:06 Sakari Ailus wrote: Hi Laurent, The patch looks good as such on the first glance, but I have another qu

Re: [PATCH 3/4] omap3isp: ccdc: Add crop support on output formatter source pad

2012-04-24 Thread Laurent Pinchart
Hi Sakari, On Tuesday 24 April 2012 18:14:20 Sakari Ailus wrote: > On Tue, Apr 24, 2012 at 11:08:12AM +0200, Laurent Pinchart wrote: > > On Tuesday 24 April 2012 01:23:06 Sakari Ailus wrote: > > > Hi Laurent, > > > > > > The patch looks good as such on the first glance, but I have another > > > q

Re: [PATCH 3/4] omap3isp: ccdc: Add crop support on output formatter source pad

2012-04-24 Thread Sakari Ailus
Hi Laurent, On Tue, Apr 24, 2012 at 11:08:12AM +0200, Laurent Pinchart wrote: > Hi Sakari, > > On Tuesday 24 April 2012 01:23:06 Sakari Ailus wrote: > > Hi Laurent, > > > > The patch looks good as such on the first glance, but I have another > > question: why are you not using the selections API

Re: [PATCH 3/4] omap3isp: ccdc: Add crop support on output formatter source pad

2012-04-24 Thread Laurent Pinchart
Hi Sakari, On Tuesday 24 April 2012 01:23:06 Sakari Ailus wrote: > Hi Laurent, > > The patch looks good as such on the first glance, but I have another > question: why are you not using the selections API instead? It's in > Mauro's tree already. You're totally right, we need to convert the selec

Re: [PATCH 3/4] omap3isp: ccdc: Add crop support on output formatter source pad

2012-04-23 Thread Sakari Ailus
Hi Laurent, The patch looks good as such on the first glance, but I have another question: why are you not using the selections API instead? It's in Mauro's tree already. Also, the old S_CROP IOCTL only has been defined for sink pads, not source. Cheers, -- Sakari Ailus sakari.ai...@iki.fi

[PATCH 3/4] omap3isp: ccdc: Add crop support on output formatter source pad

2012-04-23 Thread Laurent Pinchart
Signed-off-by: Laurent Pinchart --- drivers/media/video/omap3isp/ispccdc.c | 147 +--- drivers/media/video/omap3isp/ispccdc.h |2 + 2 files changed, 136 insertions(+), 13 deletions(-) diff --git a/drivers/media/video/omap3isp/ispccdc.c b/drivers/media/video/omap