Re: [PATCH] media: i2c: adv748x: Fix pixel rate values

2018-05-05 Thread Niklas Söderlund
Hi Laurent, On 2018-05-05 12:48:58 +0300, Laurent Pinchart wrote: > Hi Niklas, > > On Saturday, 5 May 2018 01:58:10 EEST Niklas Söderlund wrote: > > On 2018-04-25 01:36:42 +0200, Niklas Söderlund wrote: > > > On 2018-04-21 15:44:44 +0300, Laurent Pinchart wrote: > > >> The pixel rate, as reported

Re: [PATCH] media: i2c: adv748x: Fix pixel rate values

2018-05-05 Thread Laurent Pinchart
Hi Niklas, On Saturday, 5 May 2018 01:58:10 EEST Niklas Söderlund wrote: > On 2018-04-25 01:36:42 +0200, Niklas Söderlund wrote: > > On 2018-04-21 15:44:44 +0300, Laurent Pinchart wrote: > >> The pixel rate, as reported by the V4L2_CID_PIXEL_RATE control, must > >> include both horizontal and vert

Re: [PATCH] media: i2c: adv748x: Fix pixel rate values

2018-05-04 Thread Niklas Söderlund
Hi Laurent, On 2018-04-25 01:36:42 +0200, Niklas Söderlund wrote: > Hi Laurent, > > Thanks for your patch. > > On 2018-04-21 15:44:44 +0300, Laurent Pinchart wrote: > > The pixel rate, as reported by the V4L2_CID_PIXEL_RATE control, must > > include both horizontal and vertical blanking. Both th

Re: [PATCH] media: i2c: adv748x: Fix pixel rate values

2018-04-24 Thread Niklas Söderlund
Hi Laurent, Thanks for your patch. On 2018-04-21 15:44:44 +0300, Laurent Pinchart wrote: > The pixel rate, as reported by the V4L2_CID_PIXEL_RATE control, must > include both horizontal and vertical blanking. Both the AFE and HDMI > receiver program it incorrectly: > > - The HDMI receiver goes t

Re: [PATCH] media: i2c: adv748x: Fix pixel rate values

2018-04-23 Thread Kieran Bingham
Hi Laurent, On 23/04/18 10:08, Laurent Pinchart wrote: > Hi Kieran, > > On Monday, 23 April 2018 11:59:04 EEST Kieran Bingham wrote: >> On 21/04/18 13:44, Laurent Pinchart wrote: >>> The pixel rate, as reported by the V4L2_CID_PIXEL_RATE control, must >>> include both horizontal and vertical blan

Re: [PATCH] media: i2c: adv748x: Fix pixel rate values

2018-04-23 Thread Laurent Pinchart
Hi Kieran, On Monday, 23 April 2018 11:59:04 EEST Kieran Bingham wrote: > On 21/04/18 13:44, Laurent Pinchart wrote: > > The pixel rate, as reported by the V4L2_CID_PIXEL_RATE control, must > > include both horizontal and vertical blanking. Both the AFE and HDMI > > receiver program it incorrectly

Re: [PATCH] media: i2c: adv748x: Fix pixel rate values

2018-04-23 Thread Kieran Bingham
Hi Laurent, On 21/04/18 13:44, Laurent Pinchart wrote: > The pixel rate, as reported by the V4L2_CID_PIXEL_RATE control, must > include both horizontal and vertical blanking. Both the AFE and HDMI > receiver program it incorrectly: > > - The HDMI receiver goes to the trouble of removing blanking

[PATCH] media: i2c: adv748x: Fix pixel rate values

2018-04-21 Thread Laurent Pinchart
The pixel rate, as reported by the V4L2_CID_PIXEL_RATE control, must include both horizontal and vertical blanking. Both the AFE and HDMI receiver program it incorrectly: - The HDMI receiver goes to the trouble of removing blanking to compute the rate of active pixels. This is easy to fix by remov