On 18/03/14 12:07, Sathya Prakash M R wrote: > AM43x-epos and AM437x-gp device use external sii9022 > DPI to HDMI encoder. > Sii9022 use i2c for communication. > Sii9022 dt entries are added to corresponding dts. > > Signed-off-by: Sathya prakash M R <[email protected]> > --- > arch/arm/boot/dts/am437x-gp-evm.dts | 58 +++++++++++++++++++++++++++++++-- > arch/arm/boot/dts/am43x-epos-evm.dts | 59 > ++++++++++++++++++++++++++++++++-- > 2 files changed, 111 insertions(+), 6 deletions(-) > > diff --git a/arch/arm/boot/dts/am437x-gp-evm.dts > b/arch/arm/boot/dts/am437x-gp-evm.dts > index a178e8d..c4225b1 100644 > --- a/arch/arm/boot/dts/am437x-gp-evm.dts > +++ b/arch/arm/boot/dts/am437x-gp-evm.dts > @@ -51,6 +51,41 @@ > remote-endpoint = <&dpi_out>; > }; > }; > + > + sii9022: encoder@0 { > + compatible = "sii,sii9022";
See Documentation/devicetree/bindings/vendor-prefixes.txt
The vendor prefix is "sil".
> + reset-gpio = <&gpio5 8 GPIO_ACTIVE_LOW>;/* 'SelLCDorHDMI' Gpio,
> LOW to select HDMI */
> +
> + ports {
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> + port@0 {
> + reg = <0>;
> +
> + sii9022_in: endpoint@0 {
> + remote-endpoint = <&dpi_out>;
> + };
> + };
> +
> + port@1 {
> + reg = <1>;
> +
> + sii9022_out: endpoint@0 {
> + remote-endpoint = <&hdmi_connector_in>;
> + };
> + };
> + };
> + };
> +
> + hdmi0: connector@0 {
> + compatible = "hdmi-connector";
> + label = "hdmi";
> +
> + hdmi_connector_in: endpoint {
> + remote-endpoint = <&sil9022_out>;
> + };
> + };
> };
>
> &am43xx_pinmux {
> @@ -119,6 +154,9 @@
> status = "okay";
> pinctrl-names = "default";
> pinctrl-0 = <&i2c1_pins>;
> + &sii9022 {
> + reg = <0x3b>;
> + }
This looks very very odd... The whole sii9022 node should be here.
> };
>
> &epwmss0 {
> @@ -141,8 +179,22 @@
> pinctrl-names = "default";
> pinctrl-0 = <&dss_pinctrl>;
>
> - dpi_out: endpoint@0 {
> - remote-endpoint = <&lcd_in>;
> - data-lines = <24>;
> + ports {
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> + port@0 {
> + reg = <0>;
> + dpi_out: endpoint@0 {
> + remote-endpoint = <&lcd_in>;
> + data-lines = <24>;
> + };
> +
> + dpi_out:endpoint@1 {
> + remote-endpoint = <&sii9022_in>;
> + data-lines = <24>;
> +
> + };
You add the same label, "dpi_out" to both endpoints. Does this even compile?
As there's just one port, you can leave out the 'ports' node.
Tomi
signature.asc
Description: OpenPGP digital signature
