2014-06-19 08:46:35

by Jean-Francois Moine

[permalink] [raw]
Subject: [PATCH v3 RESEND] drm/i2c: tda998x: Fix lack of required reg in DT documentation

The I2C address (reg) is required for the TDA998x driver to be loaded
and initialized.

Signed-off-by: Jean-Francois Moine <[email protected]>
---
- v3
- change subject to drm/i2c
- v2
- don't force the I2C address to be 0x70
This patch applies to linux-next.
---
Documentation/devicetree/bindings/drm/i2c/tda998x.txt | 2 ++
1 file changed, 2 insertions(+)

diff --git a/Documentation/devicetree/bindings/drm/i2c/tda998x.txt b/Documentation/devicetree/bindings/drm/i2c/tda998x.txt
index d7df01c..fc7effa 100644
--- a/Documentation/devicetree/bindings/drm/i2c/tda998x.txt
+++ b/Documentation/devicetree/bindings/drm/i2c/tda998x.txt
@@ -3,6 +3,8 @@ Device-Tree bindings for the NXP TDA998x HDMI transmitter
Required properties;
- compatible: must be "nxp,tda998x"

+ - reg: I2C address
+
Optional properties:
- interrupts: interrupt number and trigger type
default: polling
--
1.9.1


2014-07-02 15:27:12

by Russell King - ARM Linux

[permalink] [raw]
Subject: Re: [PATCH v3 RESEND] drm/i2c: tda998x: Fix lack of required reg in DT documentation

On Thu, Jun 19, 2014 at 10:47:32AM +0200, Jean-Francois Moine wrote:
> The I2C address (reg) is required for the TDA998x driver to be loaded
> and initialized.
>
> Signed-off-by: Jean-Francois Moine <[email protected]>
> ---
> - v3
> - change subject to drm/i2c
> - v2
> - don't force the I2C address to be 0x70
> This patch applies to linux-next.

I had hoped that DT people would pick this up, as it's touching their
documentation. However, it seems not, so I've added it to my fixes
queue, which I'll send RSN to David.

Thanks.

--
FTTC broadband for 0.8mile line: now at 9.7Mbps down 460kbps up... slowly
improving, and getting towards what was expected from it.