Re: [PATCH v2 1/3] arm64: dts: qcom: lemans-evk: Add LVDS display overlay

[Vishnu Saini <vishnu.saini@oss.qualcomm.com>]

On Thu, Sep 03, 2026 at 03:06:10PM +0200, Konrad Dybcio wrote:
> On 8/13/26 7:33 PM, Vishnu Saini wrote:
> > The Lemans EVK IFP mezzanine supports LVDS display output using the
> > Lontium LT9211C DSI-to-LVDS bridge. The bridge is connected to the
> > DSI0 output and drives a dual-channel 1920x1080 LVDS panel in
> > VESA-24 mapping.
> > 
> > Add a DTBO overlay, lemans-evk-lvds-boe,dv215fhm-r01.dtso, that wires
> > up the LT9211C bridge on I2C1 (address 0x2d) with reset on GPIO37,
> > a panel-lvds node for the boe,dv215fhm-r01 panel (matched against the
> > panel-simple-lvds-dual-ports binding, which supplies the panel's size
> > and timing internally), and an LCD display bias regulator supplied
> > through the GPIO expander. DSI power is provided by vreg_s4a.
> > 
> > The overlay is built as a composite DTB combining the base
> > lemans-evk.dtb, the IFP mezzanine overlay, and this LVDS overlay.
> > 
> > Signed-off-by: Vishnu Saini <vishnu.saini@oss.qualcomm.com>
> > ---
> 
> [...]
> 
> > +	panel_lvds: panel-lvds {
> > +		compatible = "boe,dv215fhm-r01";
> > +		power-supply = <&lcd_disp_bias>;
> > +
> > +		status = "okay";
> 
> Nodes are enabled by default unless explicitly disabled
Yes, status = "okay"; will be removed in next revision from all such instances.
> > +
> > +		ports {
> > +			#address-cells = <1>;
> > +			#size-cells = <0>;
> > +
> > +			port@0 {
> > +				reg = <0>;
> > +				dual-lvds-odd-pixels;
> > +				panel_in_lvds_odd: endpoint {
> 
> Please keep a \n between the last property and the following subnodes
Yes, This will be fixed at all the places in next revision.
> [...]
> 
> > +	lt9211c_codec: lvds-bridge@2d {
> > +		compatible = "lontium,lt9211c";
> > +		reg = <0x2d>;
> > +		reset-gpios = <&tlmm 37 GPIO_ACTIVE_HIGH>;
> > +		vccio-supply = <&vreg_s4a>;
> > +
> > +		status = "okay";
> 
> ditto
> 
> [...]
> 
> > +&mdss0 {
> > +	status = "okay";
> > +};
> 
> The DT which you overlay this atop already enables it
Yes, will remove this from overlay.
> Konrad
> 
> > +
> > +&mdss0_dsi0 {
> > +	vdda-supply = <&vreg_l1c>;
> > +
> > +	status = "okay";
> > +};
> > +
> > +&mdss0_dsi0_out {
> > +	data-lanes = <0 1 2 3>;
> > +	remote-endpoint = <&lt9211c_in>;
> > +};
> > +
> > +&mdss0_dsi0_phy {
> > +	vdds-supply = <&vreg_l4a>;
> > +
> > +	status = "okay";
> > +};
> >