Re: [PATCH RFC v5 4/8] drm/i2c: tda998x: Add support of a DT graph of ports

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 02/18/16 16:35, Rob Herring wrote:
On Wed, Feb 17, 2016 at 04:49:05PM +0200, Jyri Sarha wrote:
From: Jean-Francois Moine <moinejf@xxxxxxx>

Two kinds of ports may be declared in a DT graph of ports: video and audio.
This patch accepts the port value from a video port as an alternative
to the video-ports property.
It also accepts audio ports in the case the transmitter is not used as
a slave encoder.
The new file include/sound/tda998x.h prepares to the definition of
a tda998x CODEC.

Signed-off-by: Jean-Francois Moine <moinejf@xxxxxxx>
Signed-off-by: Jyri Sarha <jsarha@xxxxxx>
---
  .../devicetree/bindings/display/bridge/tda998x.txt | 51 ++++++++++++
  drivers/gpu/drm/i2c/tda998x_drv.c                  | 90 +++++++++++++++++++---
  include/sound/tda998x.h                            |  8 ++
  3 files changed, 140 insertions(+), 9 deletions(-)
  create mode 100644 include/sound/tda998x.h

diff --git a/Documentation/devicetree/bindings/display/bridge/tda998x.txt b/Documentation/devicetree/bindings/display/bridge/tda998x.txt
index e9e4bce..35f6a80 100644
--- a/Documentation/devicetree/bindings/display/bridge/tda998x.txt
+++ b/Documentation/devicetree/bindings/display/bridge/tda998x.txt
@@ -16,6 +16,35 @@ Optional properties:

    - video-ports: 24 bits value which defines how the video controller
  	output is wired to the TDA998x input - default: <0x230145>
+	This property is not used when ports are defined.
+
+Optional nodes:
+
+  - port: up to three ports.
+	The ports are defined according to [1].
+
+    Video port.
+	There may be only one video port.
+	This one must contain the following property:
+
+	- port-type: must be "rgb"

This should be implied from the port unit address. In other words,
port@0 is defined to be the rgb port. Now, if this is one of several
modes for the video port, then that is a different story.


Do you suggest that also the audio i2s and s/p-dif port-types should be coded in the port unit addresses? Something like: port@0 is always rgb, port@1 is i2s, and port@2 is spdif?

Having the port-type information explicitly written serves the purpose keeping the dts files human readable. Is saving couple of bytes this important or is there some other reason to not to have the port-type property?

+	and may contain the optional property:
+
+	- reg: 24 bits value which defines how the video controller
+		output is wired to the TDA998x input (video pins)
+		When absent, the default value is <0x230145>.

This is not really how reg is intended to be used. Can you explain how
this value is determined?


I never liked this unorthodox usage of reg property either. I'll replace the reg -properties with something more explicit.

+    Audio ports.
+	There may be one or two audio ports.
+	These ones must contain the following properties:
+
+	- port-type: must be "i2s" or "spdif"
+
+	- reg: 8 bits value which defines how the audio controller
+		output is wired to the TDA998x input (audio pins)

Same here.

Rob


--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux