| Message ID | 20140119195844.0c975b90@armhf (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
Hi, On Sun, Jan 19, 2014 at 10:58 AM, Jean-Francois Moine <moinejf@free.fr> wrote: > Signed-off-by: Jean-Francois Moine <moinejf@free.fr> > --- > .../devicetree/bindings/drm/i2c/tda998x.txt | 24 ++++++++++++++++++++++ > 1 file changed, 24 insertions(+) > create mode 100644 Documentation/devicetree/bindings/drm/i2c/tda998x.txt Please cc bindings for review to devicetree@vger.kernel.org (cc:d here now) > diff --git a/Documentation/devicetree/bindings/drm/i2c/tda998x.txt b/Documentation/devicetree/bindings/drm/i2c/tda998x.txt > new file mode 100644 > index 0000000..72da71d > --- /dev/null > +++ b/Documentation/devicetree/bindings/drm/i2c/tda998x.txt > @@ -0,0 +1,24 @@ > +Device-Tree bindings for the NXP TDA998x HDMI transmitter > + > +Required properties; > + - compatible: must be "nxp,tda998x" > + > +Optional properties: > + - interrupts: interrupt number for HDMI exchanges - default: by polling What are HDMI exchanges, and how do they differ from other interrupts? > + > + - pinctrl-0: pin control group to be used for this controller (IRQ). > + > + - pinctrl-names: must contain a "default" entry. > + > + - video-ports: 24 bits value - default: <0x230145> What is this? -Olof
On Sun, 19 Jan 2014 20:06:09 -0800 Olof Johansson <olof@lixom.net> wrote: > Hi, > > On Sun, Jan 19, 2014 at 10:58 AM, Jean-Francois Moine <moinejf@free.fr> wrote: > > Signed-off-by: Jean-Francois Moine <moinejf@free.fr> > > --- > > .../devicetree/bindings/drm/i2c/tda998x.txt | 24 ++++++++++++++++++++++ > > 1 file changed, 24 insertions(+) > > create mode 100644 Documentation/devicetree/bindings/drm/i2c/tda998x.txt > > Please cc bindings for review to devicetree@vger.kernel.org (cc:d here now) > > > diff --git a/Documentation/devicetree/bindings/drm/i2c/tda998x.txt b/Documentation/devicetree/bindings/drm/i2c/tda998x.txt > > new file mode 100644 > > index 0000000..72da71d > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/drm/i2c/tda998x.txt > > @@ -0,0 +1,24 @@ > > +Device-Tree bindings for the NXP TDA998x HDMI transmitter > > + > > +Required properties; > > + - compatible: must be "nxp,tda998x" > > + > > +Optional properties: > > + - interrupts: interrupt number for HDMI exchanges - default: by polling > > What are HDMI exchanges, and how do they differ from other interrupts? The used HDMI interrupt events are screen plug/unplug and EDID read. There are also CEC read/write which are not yet implemented in the tda998x driver. There is no difference from normal interrupts, except that the events may be get by polling, so, the interrupt number is optional. > > + > > + - pinctrl-0: pin control group to be used for this controller (IRQ). > > + > > + - pinctrl-names: must contain a "default" entry. > > + > > + - video-ports: 24 bits value - default: <0x230145> > > What is this? The video-ports value defines how the video controller is connected to the tda998x chip. Each 4 bits value tells from which input pins comes the video data and if there is any bit inversion. Each byte of this video-ports is used to load the VIP_CNTRL_{0,1,2} registers. These ones are described in the TDA9983B documentation which is the closer available document about the TDA998x family. The default value is the one defined for TI boards. A known other value is <0x234501> which is used for Russell's Armada DRM driver in the Cubox (Marvell A510), but this driver has no DT support.
diff --git a/Documentation/devicetree/bindings/drm/i2c/tda998x.txt b/Documentation/devicetree/bindings/drm/i2c/tda998x.txt new file mode 100644 index 0000000..72da71d --- /dev/null +++ b/Documentation/devicetree/bindings/drm/i2c/tda998x.txt @@ -0,0 +1,24 @@ +Device-Tree bindings for the NXP TDA998x HDMI transmitter + +Required properties; + - compatible: must be "nxp,tda998x" + +Optional properties: + - interrupts: interrupt number for HDMI exchanges - default: by polling + + - pinctrl-0: pin control group to be used for this controller (IRQ). + + - pinctrl-names: must contain a "default" entry. + + - video-ports: 24 bits value - default: <0x230145> + +Example: + + tda998x: hdmi-encoder { + compatible = "nxp,tda998x"; + reg = <0x70>; + interrupt-parent = <&gpio0>; + interrupts = <27 2>; /* falling edge */ + pinctrl-0 = <&pmx_camera>; + pinctrl-names = "default"; + };
Signed-off-by: Jean-Francois Moine <moinejf@free.fr> --- .../devicetree/bindings/drm/i2c/tda998x.txt | 24 ++++++++++++++++++++++ 1 file changed, 24 insertions(+) create mode 100644 Documentation/devicetree/bindings/drm/i2c/tda998x.txt