Re: [PATCH RFC] media: i2c: mt9p031: add OF support

2013-04-30 Thread Laurent Pinchart
Hi Sascha, On Tuesday 30 April 2013 08:16:25 Sascha Hauer wrote: > On Mon, Apr 29, 2013 at 01:30:01PM +0530, Prabhakar Lad wrote: > > From: Lad, Prabhakar > > > > add OF support for the mt9p031 sensor driver. > > > > +static struct mt9p031_platform_data > > + *mt9p031_get_pdata(struct i2c_cli

Re: [PATCH RFC] media: i2c: mt9p031: add OF support

2013-04-29 Thread Sascha Hauer
Hi, One more point for your devicetree conversions, On Mon, Apr 29, 2013 at 01:30:01PM +0530, Prabhakar Lad wrote: > From: Lad, Prabhakar > > add OF support for the mt9p031 sensor driver. > > +static struct mt9p031_platform_data > + *mt9p031_get_pdata(struct i2c_client *client) > + > +{ >

Re: [PATCH RFC] media: i2c: mt9p031: add OF support

2013-04-29 Thread Laurent Pinchart
On Monday 29 April 2013 17:18:02 Prabhakar Lad wrote: > On Mon, Apr 29, 2013 at 5:05 PM, Laurent Pinchart wrote: > > Hi Prabhakar, > > > > Thank you for the patch. Please see below for a couple of comments in > > addition to the ones I've just sent (in reply to Sascha's e-mail). > > Yep fixed the

Re: [PATCH RFC] media: i2c: mt9p031: add OF support

2013-04-29 Thread Prabhakar Lad
Hi Laurent, On Mon, Apr 29, 2013 at 5:05 PM, Laurent Pinchart wrote: > Hi Prabhakar, > > Thank you for the patch. Please see below for a couple of comments in addition > to the ones I've just sent (in reply to Sascha's e-mail). > Yep fixed them all. [snip] > >> --- >> .../devicetree/bindings/m

Re: [PATCH RFC] media: i2c: mt9p031: add OF support

2013-04-29 Thread Laurent Pinchart
Hi Prabhakar, Thank you for the patch. Please see below for a couple of comments in addition to the ones I've just sent (in reply to Sascha's e-mail). On Monday 29 April 2013 13:30:01 Prabhakar Lad wrote: > From: Lad, Prabhakar > > add OF support for the mt9p031 sensor driver. > > Signed-off-

Re: [PATCH RFC] media: i2c: mt9p031: add OF support

2013-04-29 Thread Laurent Pinchart
Hi Prabhakar, On Monday 29 April 2013 16:41:07 Prabhakar Lad wrote: > On Mon, Apr 29, 2013 at 1:51 PM, Sascha Hauer wrote: > > > >> +Required Properties : > >> +- compatible : value should be either one among the following > >> + (a) "aptina,mt9p031-sensor" for mt9p031 sensor > >> + (b)

Re: [PATCH RFC] media: i2c: mt9p031: add OF support

2013-04-29 Thread Prabhakar Lad
Hi Sascha, Thanks for the quick review. On Mon, Apr 29, 2013 at 1:51 PM, Sascha Hauer wrote: >> +Required Properties : >> +- compatible : value should be either one among the following >> + (a) "aptina,mt9p031-sensor" for mt9p031 sensor >> + (b) "aptina,mt9p031m-sensor" for mt9p031m se

Re: [PATCH RFC] media: i2c: mt9p031: add OF support

2013-04-29 Thread Sascha Hauer
On Mon, Apr 29, 2013 at 01:30:01PM +0530, Prabhakar Lad wrote: > From: Lad, Prabhakar > > add OF support for the mt9p031 sensor driver. > > Signed-off-by: Lad, Prabhakar > Cc: Hans Verkuil > Cc: Laurent Pinchart > Cc: Mauro Carvalho Chehab > Cc: Guennadi Liakhovetski > Cc: Sylwester Nawrock

[PATCH RFC] media: i2c: mt9p031: add OF support

2013-04-29 Thread Prabhakar Lad
From: Lad, Prabhakar add OF support for the mt9p031 sensor driver. Signed-off-by: Lad, Prabhakar Cc: Hans Verkuil Cc: Laurent Pinchart Cc: Mauro Carvalho Chehab Cc: Guennadi Liakhovetski Cc: Sylwester Nawrocki Cc: Sakari Ailus Cc: Grant Likely Cc: Rob Herring Cc: Rob Landley Cc: device