Re: [PATCH v2 2/4] media: i2c: ov5640: Enable data pins on poweron for DVP mode

2020-08-10 Thread Lad, Prabhakar
Hi Jacopo, On Mon, Aug 10, 2020 at 9:39 AM Jacopo Mondi wrote: > > Hi Prabhakar, > > On Mon, Aug 10, 2020 at 08:40:29AM +0100, Lad, Prabhakar wrote: > > Hi Jacopo, > > > > Thank you for the review. > > > > On Fri, Aug 7, 2020 at 9:42 AM Jacopo Mondi wrote: > > > > > > Hi Prabhakar, > > >+

Re: [PATCH v2 2/4] media: i2c: ov5640: Enable data pins on poweron for DVP mode

2020-08-10 Thread Jacopo Mondi
Hi Prabhakar, On Mon, Aug 10, 2020 at 08:40:29AM +0100, Lad, Prabhakar wrote: > Hi Jacopo, > > Thank you for the review. > > On Fri, Aug 7, 2020 at 9:42 AM Jacopo Mondi wrote: > > > > Hi Prabhakar, > >+ Paul who is working with this chip on a parallel setup > > > > On Thu, Aug 06, 2020 at

Re: [PATCH v2 2/4] media: i2c: ov5640: Enable data pins on poweron for DVP mode

2020-08-10 Thread Lad, Prabhakar
Hi Jacopo, Thank you for the review. On Fri, Aug 7, 2020 at 9:42 AM Jacopo Mondi wrote: > > Hi Prabhakar, >+ Paul who is working with this chip on a parallel setup > > On Thu, Aug 06, 2020 at 05:38:57PM +0100, Lad, Prabhakar wrote: > > Hi Jacopo, > > > > Thank you for the review. > > > > On

Re: [PATCH v2 2/4] media: i2c: ov5640: Enable data pins on poweron for DVP mode

2020-08-07 Thread Jacopo Mondi
Hi Prabhakar, + Paul who is working with this chip on a parallel setup On Thu, Aug 06, 2020 at 05:38:57PM +0100, Lad, Prabhakar wrote: > Hi Jacopo, > > Thank you for the review. > > On Thu, Aug 6, 2020 at 5:27 PM Jacopo Mondi wrote: > > > > Hi Prabhakar, > > > > On Mon, Aug 03, 2020 at

Re: [PATCH v2 2/4] media: i2c: ov5640: Enable data pins on poweron for DVP mode

2020-08-06 Thread Jacopo Mondi
Hi Prabhakar, On Mon, Aug 03, 2020 at 03:31:45PM +0100, Lad Prabhakar wrote: > During testing this sensor on iW-RainboW-G21D-Qseven platform in 8-bit DVP > mode with rcar-vin bridge noticed the capture worked fine for the first run > (with yavta), but for subsequent runs the bridge driver waited

Re: [PATCH v2 2/4] media: i2c: ov5640: Enable data pins on poweron for DVP mode

2020-08-06 Thread Lad, Prabhakar
Hi Jacopo, Thank you for the review. On Thu, Aug 6, 2020 at 5:27 PM Jacopo Mondi wrote: > > Hi Prabhakar, > > On Mon, Aug 03, 2020 at 03:31:45PM +0100, Lad Prabhakar wrote: > > During testing this sensor on iW-RainboW-G21D-Qseven platform in 8-bit DVP > > mode with rcar-vin bridge noticed the

[PATCH v2 2/4] media: i2c: ov5640: Enable data pins on poweron for DVP mode

2020-08-03 Thread Lad Prabhakar
During testing this sensor on iW-RainboW-G21D-Qseven platform in 8-bit DVP mode with rcar-vin bridge noticed the capture worked fine for the first run (with yavta), but for subsequent runs the bridge driver waited for the frame to be captured. Debugging further noticed the data lines were