On 10/08/2018 11:11 PM, Vladimir Zapolskiy wrote:
> From: Sandeep Jain <Sandeep_Jain@xxxxxxxxxx>
>
> The change adds device tree binding description of TI DS90Ux9xx
> series of serializer and deserializer controllers which support video,
> audio and control data transmission over FPD-III Link connection.
>
> Signed-off-by: Sandeep Jain <Sandeep_Jain@xxxxxxxxxx>
> [vzapolskiy: various updates and corrections of secondary importance]
> Signed-off-by: Vladimir Zapolskiy <vladimir_zapolskiy@xxxxxxxxxx>
> ---
> .../devicetree/bindings/mfd/ti,ds90ux9xx.txt | 66 +++++++++++++++++++
> 1 file changed, 66 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/mfd/ti,ds90ux9xx.txt
>
> diff --git a/Documentation/devicetree/bindings/mfd/ti,ds90ux9xx.txt b/Documentation/devicetree/bindings/mfd/ti,ds90ux9xx.txt
> new file mode 100644
> index 000000000000..0733da88f7ef
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/mfd/ti,ds90ux9xx.txt
> @@ -0,0 +1,66 @@
> +Texas Instruments DS90Ux9xx de-/serializer controllers
> +
> +Required properties:
> +- compatible: Must contain a generic "ti,ds90ux9xx" value and
> + may contain one more specific value from the list:
> + "ti,ds90ub925q",
> + "ti,ds90uh925q",
> + "ti,ds90ub927q",
> + "ti,ds90uh927q",
> + "ti,ds90ub926q",
> + "ti,ds90uh926q",
Keep the list sorted.
> + "ti,ds90ub928q",
> + "ti,ds90uh928q",
> + "ti,ds90ub940q",
> + "ti,ds90uh940q".
> +
> +Optional properties:
> +- reg : Specifies the I2C slave address of a local de-/serializer.
> +- power-gpios : GPIO line to control supplied power to the device.
Shouldn't this be regulator phandle ?
> +- ti,backward-compatible-mode : Overrides backward compatibility mode.
> + Possible values are "<1>" or "<0>".
Make this bool , ie. present or not.
> + If "ti,backward-compatible-mode" is not mentioned, the backward
> + compatibility mode is not touched and given by hardware pin strapping.
> +- ti,low-frequency-mode : Overrides low frequency mode.
> + Possible values are "<1>" or "<0>".
> + If "ti,low-frequency-mode" is not mentioned, the low frequency mode
> + is not touched and given by hardware pin strapping.
> +- ti,video-map-select-msb: Sets video bridge pins to MSB mode, if it is set
> + MAPSEL pin value is ignored.
> +- ti,video-map-select-lsb: Sets video bridge pins to LSB mode, if it is set
> + MAPSEL pin value is ignored.
This needs some additional explanation, what's this about ?
> +- ti,pixel-clock-edge : Selects Pixel Clock Edge.
> + Possible values are "<1>" or "<0>".
> + If "ti,pixel-clock-edge" is High <1>, output data is strobed on the
> + Rising edge of the PCLK. If ti,pixel-clock-edge is Low <0>, data is
> + strobed on the Falling edge of the PCLK.
> + If "ti,pixel-clock-edge" is not mentioned, the pixel clock edge
> + value is not touched and given by hardware pin strapping.
> +- ti,spread-spectrum-clock-generation : Spread Sprectrum Clock Generation.
> + Possible values are from "<0>" to "<7>". The same value will be
> + written to SSC register. If "ti,spread-spectrum-clock-gen" is not
> + found, then SSCG will be disabled.
> +
> +TI DS90Ux9xx serializers and deserializer device nodes may contain a number
> +of children device nodes to describe and enable particular subcomponents
> +found on ICs.
> +
> +Example:
> +
> +serializer: serializer@c {
> + compatible = "ti,ds90ub927q", "ti,ds90ux9xx";
> + reg = <0xc>;
> + power-gpios = <&gpio5 12 GPIO_ACTIVE_HIGH>;
> + ti,backward-compatible-mode = <0>;
> + ti,low-frequency-mode = <0>;
> + ti,pixel-clock-edge = <0>;
> + ...
> +}
> +
> +deserializer: deserializer@3c {
> + compatible = "ti,ds90ub940q", "ti,ds90ux9xx";
> + reg = <0x3c>;
> + power-gpios = <&gpio6 31 GPIO_ACTIVE_HIGH>;
> + ...
> +}
> +
>
--
Best regards,
Marek Vasut
