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, 2012 at

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 Hadli,

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 2012 11:41:11

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 July 2012

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 Pinchart

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 Hadli,

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

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-bus

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 and some

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 entries for