Re: [PATCH] [media] v4l2-dv-timings: Introduce v4l2_calc_fps()

2017-03-15 Thread Hans Verkuil
On 03/14/2017 08:14 PM, Jose Abreu wrote: > Hi Hans, > > > On 14-03-2017 07:24, Hans Verkuil wrote: >>> Right, I was forgetting about this ... >>> >>> So: >>> 1) Most of HDMI receivers do not have the expected precision in >>> measuring pixel clock value; >> s/Most/Some/ >> >> Newer HDMI

Re: [PATCH] [media] v4l2-dv-timings: Introduce v4l2_calc_fps()

2017-03-15 Thread Hans Verkuil
On 03/14/2017 08:14 PM, Jose Abreu wrote: > Hi Hans, > > > On 14-03-2017 07:24, Hans Verkuil wrote: >>> Right, I was forgetting about this ... >>> >>> So: >>> 1) Most of HDMI receivers do not have the expected precision in >>> measuring pixel clock value; >> s/Most/Some/ >> >> Newer HDMI

Re: [PATCH] [media] v4l2-dv-timings: Introduce v4l2_calc_fps()

2017-03-14 Thread Jose Abreu
Hi Hans, On 14-03-2017 07:24, Hans Verkuil wrote: >> Right, I was forgetting about this ... >> >> So: >> 1) Most of HDMI receivers do not have the expected precision in >> measuring pixel clock value; > s/Most/Some/ > > Newer HDMI receivers tend to have better precision. > > However, the

Re: [PATCH] [media] v4l2-dv-timings: Introduce v4l2_calc_fps()

2017-03-14 Thread Jose Abreu
Hi Hans, On 14-03-2017 07:24, Hans Verkuil wrote: >> Right, I was forgetting about this ... >> >> So: >> 1) Most of HDMI receivers do not have the expected precision in >> measuring pixel clock value; > s/Most/Some/ > > Newer HDMI receivers tend to have better precision. > > However, the

Re: [PATCH] [media] v4l2-dv-timings: Introduce v4l2_calc_fps()

2017-03-14 Thread Hans Verkuil
On 03/13/2017 08:03 PM, Jose Abreu wrote: > Hi Hans, > > > On 09-03-2017 15:40, Hans Verkuil wrote: >> On 09/03/17 16:15, Jose Abreu wrote: >>> Hi Hans, >>> >>> >>> Thanks for the review! >>> >>> >>> On 09-03-2017 12:29, Hans Verkuil wrote: On 07/03/17 17:48, Jose Abreu wrote: > HDMI

Re: [PATCH] [media] v4l2-dv-timings: Introduce v4l2_calc_fps()

2017-03-14 Thread Hans Verkuil
On 03/13/2017 08:03 PM, Jose Abreu wrote: > Hi Hans, > > > On 09-03-2017 15:40, Hans Verkuil wrote: >> On 09/03/17 16:15, Jose Abreu wrote: >>> Hi Hans, >>> >>> >>> Thanks for the review! >>> >>> >>> On 09-03-2017 12:29, Hans Verkuil wrote: On 07/03/17 17:48, Jose Abreu wrote: > HDMI

Re: [PATCH] [media] v4l2-dv-timings: Introduce v4l2_calc_fps()

2017-03-13 Thread Jose Abreu
Hi Hans, On 09-03-2017 15:40, Hans Verkuil wrote: > On 09/03/17 16:15, Jose Abreu wrote: >> Hi Hans, >> >> >> Thanks for the review! >> >> >> On 09-03-2017 12:29, Hans Verkuil wrote: >>> On 07/03/17 17:48, Jose Abreu wrote: HDMI Receivers receive video modes which, according to CEA

Re: [PATCH] [media] v4l2-dv-timings: Introduce v4l2_calc_fps()

2017-03-13 Thread Jose Abreu
Hi Hans, On 09-03-2017 15:40, Hans Verkuil wrote: > On 09/03/17 16:15, Jose Abreu wrote: >> Hi Hans, >> >> >> Thanks for the review! >> >> >> On 09-03-2017 12:29, Hans Verkuil wrote: >>> On 07/03/17 17:48, Jose Abreu wrote: HDMI Receivers receive video modes which, according to CEA

Re: [PATCH] [media] v4l2-dv-timings: Introduce v4l2_calc_fps()

2017-03-09 Thread Hans Verkuil
On 09/03/17 16:15, Jose Abreu wrote: > Hi Hans, > > > Thanks for the review! > > > On 09-03-2017 12:29, Hans Verkuil wrote: >> On 07/03/17 17:48, Jose Abreu wrote: >>> HDMI Receivers receive video modes which, according to >>> CEA specification, can have different frames per second >>> (fps)

Re: [PATCH] [media] v4l2-dv-timings: Introduce v4l2_calc_fps()

2017-03-09 Thread Hans Verkuil
On 09/03/17 16:15, Jose Abreu wrote: > Hi Hans, > > > Thanks for the review! > > > On 09-03-2017 12:29, Hans Verkuil wrote: >> On 07/03/17 17:48, Jose Abreu wrote: >>> HDMI Receivers receive video modes which, according to >>> CEA specification, can have different frames per second >>> (fps)

Re: [PATCH] [media] v4l2-dv-timings: Introduce v4l2_calc_fps()

2017-03-09 Thread Jose Abreu
Hi Hans, Thanks for the review! On 09-03-2017 12:29, Hans Verkuil wrote: > On 07/03/17 17:48, Jose Abreu wrote: >> HDMI Receivers receive video modes which, according to >> CEA specification, can have different frames per second >> (fps) values. >> >> This patch introduces a helper function in

Re: [PATCH] [media] v4l2-dv-timings: Introduce v4l2_calc_fps()

2017-03-09 Thread Jose Abreu
Hi Hans, Thanks for the review! On 09-03-2017 12:29, Hans Verkuil wrote: > On 07/03/17 17:48, Jose Abreu wrote: >> HDMI Receivers receive video modes which, according to >> CEA specification, can have different frames per second >> (fps) values. >> >> This patch introduces a helper function in

Re: [PATCH] [media] v4l2-dv-timings: Introduce v4l2_calc_fps()

2017-03-09 Thread Hans Verkuil
On 07/03/17 17:48, Jose Abreu wrote: > HDMI Receivers receive video modes which, according to > CEA specification, can have different frames per second > (fps) values. > > This patch introduces a helper function in the media core > which can calculate the expected video mode fps given the > pixel

Re: [PATCH] [media] v4l2-dv-timings: Introduce v4l2_calc_fps()

2017-03-09 Thread Hans Verkuil
On 07/03/17 17:48, Jose Abreu wrote: > HDMI Receivers receive video modes which, according to > CEA specification, can have different frames per second > (fps) values. > > This patch introduces a helper function in the media core > which can calculate the expected video mode fps given the > pixel