On 14/05/14 00:32, Tony Lindgren wrote:

> +&dss {
> +     status = "ok";
> +     vdds_dsi-supply = <&vpll2>;
> +     port {
> +             dpi_out: endpoint {
> +                     remote-endpoint = <&lcd_in>;
> +                     data-lines = <18>;
> +             };
> +     };
> +};

I just noticed the vdds_dsi-supply there. While the driver currently
uses that if available, I think it should be removed, and done the same
way Florian did with his overo patches:

/* Needed to power the DPI pins */
&vpll2 {
        regulator-always-on;
};

That supply is not DSS's supply, but it's used to power up the pins. If
the pins were used as GPIOs, that power should be enabled.

 Tomi


Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to