Re: [EXT] Re: [PATCH v3 1/2] dt-bindings: display: panel: Add support for Raydium RM67191 panel

From: Robert Chiras
Date: Fri Jun 21 2019 - 10:17:14 EST


Hi Fabio,

On Vi, 2019-06-21 at 11:00 -0300, Fabio Estevam wrote:
> Hi Robert,
>
> On Thu, Jun 20, 2019 at 10:32 AM Robert Chiras <robert.chiras@xxxxxxx
> > wrote:
> >
> >
> > Add dt-bindings documentation for Raydium RM67191 DSI panel.
> >
> > Signed-off-by: Robert Chiras <robert.chiras@xxxxxxx>
> > Reviewed-by: Sam Ravnborg <sam@xxxxxxxxxxxx>
> > ---
> > Â.../bindings/display/panel/raydium,rm67191.txtÂÂÂÂÂ| 39
> > ++++++++++++++++++++++
> > Â1 file changed, 39 insertions(+)
> > Âcreate mode 100644
> > Documentation/devicetree/bindings/display/panel/raydium,rm67191.txt
> >
> > diff --git
> > a/Documentation/devicetree/bindings/display/panel/raydium,rm67191.t
> > xt
> > b/Documentation/devicetree/bindings/display/panel/raydium,rm67191.t
> > xt
> > new file mode 100644
> > index 0000000..52af272
> > --- /dev/null
> > +++
> > b/Documentation/devicetree/bindings/display/panel/raydium,rm67191.t
> > xt
> > @@ -0,0 +1,39 @@
> > +Raydium RM67171 OLED LCD panel with MIPI-DSI protocol
> > +
> > +Required properties:
> > +- compatible:ÂÂÂÂÂÂÂÂÂÂ"raydium,rm67191"
> > +- reg:ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂvirtual channel for MIPI-DSI protocol
> > +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂmust be <0>
> > +- dsi-lanes:ÂÂÂÂÂÂÂÂÂÂÂnumber of DSI lanes to be used
> > +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂmust be <3> or <4>
> > +- port:ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂinput port node with endpoint definition as
> > +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂdefined in
> > Documentation/devicetree/bindings/graph.txt;
> > +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂthe input port should be connected to a
> > MIPI-DSI device
> > +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂdriver
> > +
> > +Optional properties:
> > +- reset-gpios:ÂÂÂÂÂÂÂÂÂa GPIO spec for the RST_B GPIO pin
> > +- width-mm:ÂÂÂÂÂÂÂÂÂÂÂÂsee panel-common.txt
> > +- height-mm:ÂÂÂÂÂÂÂÂÂÂÂsee panel-common.txt
> > +- video-mode:ÂÂÂÂÂÂÂÂÂÂ0 - burst-mode
> > +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ1 - non-burst with sync event
> > +ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ2 - non-burst with sync po ulse
> No power-supply property?
From what I've seen in the schematics, the power lines on the DSI port
on all the i.MX8 cores are coming from a PMIC providing power for all
the peripherals. Since I didn't find a way to cut the power on a single
peripheral (like DSI, for example) it doesn't make sense for power-
supply property. For now, at least.