Re: [PATCH] media: i2c: add support for omnivision's ov2659 sensor

2015-02-10 Thread Benoit Parrot
Laurent Pinchart wrote on Thu [2015-Feb-05 16:52:22 +0200]: > Hi Prabhakar, > > (CC'ing Benoit Parrot) > > On Thursday 05 February 2015 11:55:28 Lad, Prabhakar wrote: > > On Thu, Feb 5, 2015 at 11:53 AM, Laurent Pinchart wrote: > > > On Wednesday 04 February 2015 20:55:02 Lad, Prabhakar wrote:

Re: [PATCH] media: i2c: add support for omnivision's ov2659 sensor

2015-02-05 Thread Laurent Pinchart
Hi Prabhakar, (CC'ing Benoit Parrot) On Thursday 05 February 2015 11:55:28 Lad, Prabhakar wrote: > On Thu, Feb 5, 2015 at 11:53 AM, Laurent Pinchart wrote: > > On Wednesday 04 February 2015 20:55:02 Lad, Prabhakar wrote: > >> On Wed, Feb 4, 2015 at 5:03 PM, Laurent Pinchart wrote: > >> > On Thurs

Re: [PATCH] media: i2c: add support for omnivision's ov2659 sensor

2015-02-05 Thread Lad, Prabhakar
Hi Laurent, On Thu, Feb 5, 2015 at 11:53 AM, Laurent Pinchart wrote: > Hi Prabhakar, > > On Wednesday 04 February 2015 20:55:02 Lad, Prabhakar wrote: >> On Wed, Feb 4, 2015 at 5:03 PM, Laurent Pinchart wrote: >> > On Thursday 15 January 2015 23:39:23 Lad, Prabhakar wrote: >> >> From: Benoit Parro

Re: [PATCH] media: i2c: add support for omnivision's ov2659 sensor

2015-02-05 Thread Laurent Pinchart
Hi Prabhakar, On Wednesday 04 February 2015 20:55:02 Lad, Prabhakar wrote: > On Wed, Feb 4, 2015 at 5:03 PM, Laurent Pinchart wrote: > > On Thursday 15 January 2015 23:39:23 Lad, Prabhakar wrote: > >> From: Benoit Parrot > >> > >> this patch adds support for omnivision's ov2659 > >> sensor. > >>

Re: [PATCH] media: i2c: add support for omnivision's ov2659 sensor

2015-02-04 Thread Lad, Prabhakar
Hi Laurent, Thanks for the review. On Wed, Feb 4, 2015 at 5:03 PM, Laurent Pinchart wrote: > Hi Prabhakar, > > Thank you for the patch. Here's a partial review. > > On Thursday 15 January 2015 23:39:23 Lad, Prabhakar wrote: >> From: Benoit Parrot >> >> this patch adds support for omnivision's o

Re: [PATCH] media: i2c: add support for omnivision's ov2659 sensor

2015-02-04 Thread Laurent Pinchart
Hi Prabhakar, Thank you for the patch. Here's a partial review. On Thursday 15 January 2015 23:39:23 Lad, Prabhakar wrote: > From: Benoit Parrot > > this patch adds support for omnivision's ov2659 > sensor. > > Signed-off-by: Benoit Parrot > Signed-off-by: Lad, Prabhakar > --- > .../devicet

Re: [PATCH] media: i2c: add support for omnivision's ov2659 sensor

2015-02-04 Thread Rob Herring
On Wed, Feb 4, 2015 at 9:23 AM, Fabio Estevam wrote: > Rob, > > On Wed, Feb 4, 2015 at 12:55 PM, Rob Herring wrote: > >> I'm surprised there are not already compatible strings with >> OmniVision. There are some examples using "omnivision", but no dts >> files and examples don't count. >> >> The s

Re: [PATCH] media: i2c: add support for omnivision's ov2659 sensor

2015-02-04 Thread Lad, Prabhakar
On Wed, Feb 4, 2015 at 3:23 PM, Fabio Estevam wrote: > Rob, > > On Wed, Feb 4, 2015 at 12:55 PM, Rob Herring wrote: > >> I'm surprised there are not already compatible strings with >> OmniVision. There are some examples using "omnivision", but no dts >> files and examples don't count. >> >> The s

Re: [PATCH] media: i2c: add support for omnivision's ov2659 sensor

2015-02-04 Thread Fabio Estevam
Rob, On Wed, Feb 4, 2015 at 12:55 PM, Rob Herring wrote: > I'm surprised there are not already compatible strings with > OmniVision. There are some examples using "omnivision", but no dts > files and examples don't count. > > The stock ticker is ovti, so please use that. That's what I sent: htt

Re: [PATCH] media: i2c: add support for omnivision's ov2659 sensor

2015-02-04 Thread Lad, Prabhakar
Hi Rob, Thanks for the review. On Wed, Feb 4, 2015 at 2:55 PM, Rob Herring wrote: > On Thu, Jan 15, 2015 at 5:39 PM, Lad, Prabhakar > wrote: >> From: Benoit Parrot >> >> this patch adds support for omnivision's ov2659 >> sensor. >> >> Signed-off-by: Benoit Parrot >> Signed-off-by: Lad, Prabha

Re: [PATCH] media: i2c: add support for omnivision's ov2659 sensor

2015-02-04 Thread Rob Herring
On Thu, Jan 15, 2015 at 5:39 PM, Lad, Prabhakar wrote: > From: Benoit Parrot > > this patch adds support for omnivision's ov2659 > sensor. > > Signed-off-by: Benoit Parrot > Signed-off-by: Lad, Prabhakar > --- [...] > diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt > b/Doc

Re: [PATCH] media: i2c: add support for omnivision's ov2659 sensor

2015-01-30 Thread Lad, Prabhakar
Hello, On Thu, Jan 15, 2015 at 11:39 PM, Lad, Prabhakar wrote: > From: Benoit Parrot > > this patch adds support for omnivision's ov2659 > sensor. > > Signed-off-by: Benoit Parrot > Signed-off-by: Lad, Prabhakar > --- > .../devicetree/bindings/media/i2c/ov2659.txt | 33 + > .../device

[PATCH] media: i2c: add support for omnivision's ov2659 sensor

2015-01-15 Thread Lad, Prabhakar
From: Benoit Parrot this patch adds support for omnivision's ov2659 sensor. Signed-off-by: Benoit Parrot Signed-off-by: Lad, Prabhakar --- .../devicetree/bindings/media/i2c/ov2659.txt | 33 + .../devicetree/bindings/vendor-prefixes.txt|1 + MAINTAINERS