Re: [PATCH v4 3/5] media: i2c: Add TDA1997x HDMI receiver driver

2017-12-14 Thread Hans Verkuil
On 14/12/17 00:35, Tim Harvey wrote: >>> Close. What is missing is a check of the AVI InfoFrame: if it has an >>> explicit >>> colorimetry then use that. E.g. check for HDMI_COLORIMETRY_ITU_601 or >>> ITU_709 >>> and set the colorspace accordingly. Otherwise fall back to what you have >>> here.

Re: [PATCH v4 3/5] media: i2c: Add TDA1997x HDMI receiver driver

2017-12-13 Thread Tim Harvey
On Mon, Dec 4, 2017 at 9:30 AM, Tim Harvey wrote: > On Mon, Dec 4, 2017 at 4:50 AM, Hans Verkuil wrote: >> Hi Tim, >> >> Found a few more small issues. After that's fixed and you have the Ack for >> the >> bindings this can be merged I think. > > Hans, > > Thanks. Can you weigh in on the binding

Re: [PATCH v4 3/5] media: i2c: Add TDA1997x HDMI receiver driver

2017-12-04 Thread Tim Harvey
On Mon, Dec 4, 2017 at 4:50 AM, Hans Verkuil wrote: > Hi Tim, > > Found a few more small issues. After that's fixed and you have the Ack for the > bindings this can be merged I think. Hans, Thanks. Can you weigh in on the bindings? Rob was hoping for some discussion on making some generic bus fo

Re: [PATCH v4 3/5] media: i2c: Add TDA1997x HDMI receiver driver

2017-12-04 Thread Hans Verkuil
Hi Tim, Found a few more small issues. After that's fixed and you have the Ack for the bindings this can be merged I think. On 11/29/2017 10:19 PM, Tim Harvey wrote: > Add support for the TDA1997x HDMI receivers. > > Cc: Hans Verkuil > Signed-off-by: Tim Harvey > --- > v4: > - move include/dt

[PATCH v4 3/5] media: i2c: Add TDA1997x HDMI receiver driver

2017-11-29 Thread Tim Harvey
Add support for the TDA1997x HDMI receivers. Cc: Hans Verkuil Signed-off-by: Tim Harvey --- v4: - move include/dt-bindings/media/tda1997x.h to bindings patch - fix typos - fix default quant range for VGA - fix quant range handling and conv matrix - add additional standards and capabilities