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@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