Hi,

On 12/12/16 15:05, Bartosz Golaszewski wrote:

> +&lcdc {
> +     status = "okay";
> +     pinctrl-names = "default";
> +     pinctrl-0 = <&lcd_pins>;
> +
> +     ports {
> +             #address-cells = <1>;
> +             #size-cells = <0>;
> +
> +             lcdc_out: port at 1 {
> +                     #address-cells = <1>;
> +                     #size-cells = <0>;
> +                     reg = <1>;
> +
> +                     lcdc_out_vga: endpoint {
> +                             reg = <0>;
> +                             remote-endpoint = <&vga_bridge_in>;
> +                     };
> +             };
> +     };
> +};
> 

This is not correct. LCDC has just one output, so port at 1 doesn't make
sense. It's port at 0. But with just one port, you can leave "ports" away.
And you don't need the port's label for anything, if I'm not mistaken. So:

&lcdc {
        status = "okay";
        pinctrl-names = "default";
        pinctrl-0 = <&lcd_pins>;

        port {
                lcdc_out_vga: endpoint {
                        remote-endpoint = <&vga_bridge_in>;
                };
        };
};

 Tomi

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL: 
<https://lists.freedesktop.org/archives/dri-devel/attachments/20161213/5ebeac17/attachment.sig>

Reply via email to