Re: [PATCH v4 1/2] media: add new mediabus format enums for dm365

2012-07-17 Thread Laurent Pinchart
Hi Manjunath, Thank you for the patch. Just some nitpicking, please see below. On Friday 30 March 2012 10:09:13 Hadli, Manjunath wrote: > add new enum entries for supporting the media-bus formats on dm365. > These include some bayer and some non-bayer formats. > V4L2_MBUS_FMT_YDYC8_1X16 and V4L2

RE: [PATCH v4 1/2] media: add new mediabus format enums for dm365

2012-07-17 Thread Hadli, Manjunath
Hi Laurent, Thanks for the review. On Tue, Jul 17, 2012 at 16:26:24, Laurent Pinchart wrote: > Hi Manjunath, > > Thank you for the patch. > > Just some nitpicking, please see below. > > On Friday 30 March 2012 10:09:13 Hadli, Manjunath wrote: > > add new enum entries for supporting the media-b

Re: [PATCH v4 1/2] media: add new mediabus format enums for dm365

2012-07-17 Thread Laurent Pinchart
Hi Manjunath, On Tuesday 17 July 2012 11:41:11 Hadli, Manjunath wrote: > On Tue, Jul 17, 2012 at 16:26:24, Laurent Pinchart wrote: > > On Friday 30 March 2012 10:09:13 Hadli, Manjunath wrote: > > > add new enum entries for supporting the media-bus formats on dm365. > > > These include some bayer a

RE: [PATCH v4 1/2] media: add new mediabus format enums for dm365

2012-07-17 Thread Hadli, Manjunath
Hi Laurent, On Tue, Jul 17, 2012 at 17:25:42, Laurent Pinchart wrote: > Hi Manjunath, > > On Tuesday 17 July 2012 11:41:11 Hadli, Manjunath wrote: > > On Tue, Jul 17, 2012 at 16:26:24, Laurent Pinchart wrote: > > > On Friday 30 March 2012 10:09:13 Hadli, Manjunath wrote: > > > > add new enum entr

Re: [PATCH v4 1/2] media: add new mediabus format enums for dm365

2012-07-18 Thread Laurent Pinchart
Hi Manjunath, On Tuesday 17 July 2012 12:22:42 Hadli, Manjunath wrote: > On Tue, Jul 17, 2012 at 17:25:42, Laurent Pinchart wrote: > > On Tuesday 17 July 2012 11:41:11 Hadli, Manjunath wrote: > > > On Tue, Jul 17, 2012 at 16:26:24, Laurent Pinchart wrote: > > > > On Friday 30 March 2012 10:09:13 H

RE: [PATCH v4 1/2] media: add new mediabus format enums for dm365

2012-07-19 Thread Hadli, Manjunath
Hi Laurent, On Wed, Jul 18, 2012 at 16:35:18, Laurent Pinchart wrote: > Hi Manjunath, > > On Tuesday 17 July 2012 12:22:42 Hadli, Manjunath wrote: > > On Tue, Jul 17, 2012 at 17:25:42, Laurent Pinchart wrote: > > > On Tuesday 17 July 2012 11:41:11 Hadli, Manjunath wrote: > > > > On Tue, Jul 17, 2

Re: [PATCH v4 1/2] media: add new mediabus format enums for dm365

2012-07-19 Thread Laurent Pinchart
Hi Manjunath, On Thursday 19 July 2012 11:33:56 Hadli, Manjunath wrote: > On Wed, Jul 18, 2012 at 16:35:18, Laurent Pinchart wrote: > > On Tuesday 17 July 2012 12:22:42 Hadli, Manjunath wrote: > > > On Tue, Jul 17, 2012 at 17:25:42, Laurent Pinchart wrote: > > > > On Tuesday 17 July 2012 11:41:11

RE: [PATCH v4 1/2] media: add new mediabus format enums for dm365

2012-07-19 Thread Guennadi Liakhovetski
On Thu, 19 Jul 2012, Hadli, Manjunath wrote: > Hi Laurent, > > On Wed, Jul 18, 2012 at 16:35:18, Laurent Pinchart wrote: > > Hi Manjunath, > > > > On Tuesday 17 July 2012 12:22:42 Hadli, Manjunath wrote: > > > On Tue, Jul 17, 2012 at 17:25:42, Laurent Pinchart wrote: > > > > On Tuesday 17 July 2

Re: [PATCH v4 1/2] media: add new mediabus format enums for dm365

2012-07-19 Thread Hans Verkuil
On Thu 19 July 2012 13:33:56 Hadli, Manjunath wrote: > Hi Laurent, > > On Wed, Jul 18, 2012 at 16:35:18, Laurent Pinchart wrote: > > Hi Manjunath, > > > > On Tuesday 17 July 2012 12:22:42 Hadli, Manjunath wrote: > > > On Tue, Jul 17, 2012 at 17:25:42, Laurent Pinchart wrote: > > > > On Tuesday 17

Re: [PATCH v4 1/2] media: add new mediabus format enums for dm365

2012-07-19 Thread Laurent Pinchart
Hi Hans, On Thursday 19 July 2012 14:58:12 Hans Verkuil wrote: > On Thu 19 July 2012 13:33:56 Hadli, Manjunath wrote: > > On Wed, Jul 18, 2012 at 16:35:18, Laurent Pinchart wrote: > > > On Tuesday 17 July 2012 12:22:42 Hadli, Manjunath wrote: > > > > On Tue, Jul 17, 2012 at 17:25:42, Laurent Pinch