Hi Maciej,

Thank you for the patch.

On Thursday, 31 August 2017 15:27:13 EEST Maciej Purski wrote:
> SiI9234 transmitter converts eTMDS/HDMI signal to MHL 1.0.
> It is controlled via I2C bus. Its interaction with other
> devices in video pipeline is performed mainly on HW level.
> The only interaction it does on device driver level is
> filtering-out unsupported video modes, it exposes drm_bridge
> interface to perform this operation.
> 
> This patch is based on the code refactored by Tomasz Stanislawski
> <t.stanisl...@samsung.com>, which was initially developed by:
> Adam Hampson <ahamp...@sta.samsung.com>
> Erik Gilling <konk...@android.com>
> Shankar Bandal <shanka...@samsung.com>
> Dharam Kumar <dharam...@samsung.com>
> 
> Signed-off-by: Maciej Purski <m.pur...@samsung.com>
> ---
> Changes in v2:
> - use bulk_requlators instead of single one
> - substitute some of the magic values with macros
> - improve coding style
> - improved error handling in sii9234_probe()
> ---
>  .../devicetree/bindings/display/bridge/sii9234.txt |  34 +
>  drivers/gpu/drm/bridge/Kconfig                     |   8 +
>  drivers/gpu/drm/bridge/Makefile                    |   1 +
>  drivers/gpu/drm/bridge/sii9234.c                   | 993 ++++++++++++++++++
>  4 files changed, 1036 insertions(+)
>  create mode 100644
> Documentation/devicetree/bindings/display/bridge/sii9234.txt create mode
> 100644 drivers/gpu/drm/bridge/sii9234.c
> 
> diff --git a/Documentation/devicetree/bindings/display/bridge/sii9234.txt
> b/Documentation/devicetree/bindings/display/bridge/sii9234.txt new file
> mode 100644
> index 0000000..3ce7413
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/display/bridge/sii9234.txt
> @@ -0,0 +1,34 @@
> +Silicon Image SiI9234 HDMI/MHL bridge bindings
> +
> +Required properties:
> +     - compatible : "sil,sii9234".
> +     - reg : I2C address for TPI interface, use 0x39
> +     - avcc33-supply : MHL/USB Switch Supply Voltage (3.3V)
> +     - iovcc18-supply : I/O Supply Voltage (1.8V)
> +     - avcc12-supply : TMDS Analog Supply Voltage (1.2V)
> +     - cvcc12-supply : Digital Core Supply Voltage (1.2V)
> +     - interrupts, interrupt-parent: interrupt specifier of INT pin
> +     - reset-gpios: gpio specifier of RESET pin (active low)
> +     - video interfaces: Device node can contain video interface port
> +                         node for HDMI encoder according to [1].
> +
> +[1]: Documentation/devicetree/bindings/media/video-interfaces.txt

Doesn't this chip have two ports, one input connected to the SoC and one 
output connected to an HDMI connector ? If so there should be two ports in DT 
too.

> +Example:
> +     sii9234@39 {
> +             compatible = "sil,sii9234";
> +             reg = <0x39>;
> +             avcc33-supply = <&vcc33mhl>;
> +             iovcc18-supply = <&vcc18mhl>;
> +             avcc12-supply = <&vsil12>;
> +             cvcc12-supply = <&vsil12>;
> +             reset-gpios = <&gpf3 4 GPIO_ACTIVE_LOW>;
> +             interrupt-parent = <&gpf3>;
> +             interrupts = <5 IRQ_TYPE_LEVEL_HIGH>;
> +
> +             port {
> +                     mhl_to_hdmi: endpoint {
> +                             remote-endpoint = <&hdmi_to_mhl>;

It would be useful to include the remote DT nodes in the example too.

> +                     };
> +             };
> +     };

-- 
Regards,

Laurent Pinchart

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to