[PATCH v2 08/13] ARM: dts: sun8i: a23/a33: Fix Display Engine DTC warnings

Maxime Ripard maxime.ripard at bootlin.com
Fri Mar 15 08:57:10 UTC 2019


Hi,

On Fri, Mar 15, 2019 at 10:28:16AM +0800, Chen-Yu Tsai wrote:
> > diff --git a/arch/arm/boot/dts/sun8i-a33.dtsi b/arch/arm/boot/dts/sun8i-a33.dtsi
> > index 1111a6498102..4484d76c88b5 100644
> > --- a/arch/arm/boot/dts/sun8i-a33.dtsi
> > +++ b/arch/arm/boot/dts/sun8i-a33.dtsi
> > @@ -266,18 +266,9 @@
> >                         phy-names = "dphy";
> >                         status = "disabled";
> >
> > -                       ports {
> > -                               #address-cells = <1>;
> > -                               #size-cells = <0>;
> > -
> > -                               port at 0 {
> > -                                       #address-cells = <1>;
> > -                                       #size-cells = <0>;
> > -                                       reg = <0>;
> > -
> > -                                       dsi_in_tcon0: endpoint {
> > -                                               remote-endpoint = <&tcon0_out_dsi>;
> > -                                       };
> > +                       port {
> > +                               dsi_in_tcon0: endpoint {
> > +                                       remote-endpoint = <&tcon0_out_dsi>;
> 
> Shouldn't we expect to see an output port as well?

DSI panels are not using the OF Graph, but are subnodes of the DSI
controller itself. It's kind of odd when we have to mix the two, but
that means we'll never get an output endpoint for the DSI node.

Maxime

-- 
Maxime Ripard, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20190315/da4cd0f8/attachment.sig>


More information about the dri-devel mailing list