Re: [PATCH v5 3/3] arm64: dts: imx95: Add support for Data Modul i.MX95 eDM SBC

[Frank Li]

On Wed, Sep 30, 2026 at 07:44:13PM +0200, Francesco Dolcini wrote:
> On Tue, Sep 29, 2026 at 05:20:27PM +0200, Marek Vasut wrote:
> > On 9/29/26 4:45 PM, Frank Li wrote:
> >
> > Hello Frank,
> >
> > > > +	backlight_lvds: backlight-lvds {
> > > > +		compatible = "pwm-backlight";
> > > > +		pinctrl-names = "default";
> > > > +		pinctrl-0 = <&pinctrl_panel_backlight_lvds>;
> > > > +		brightness-levels = <0 1 10 20 30 40 50 60 70 75 80 90 100>;
> > > > +		default-brightness-level = <7>;
> > > > +		enable-gpios = <&gpio2 27 GPIO_ACTIVE_HIGH>;
> > > > +		pwms = <&tpm3 3 5000000 0>;
> > > > +		/* Disabled by default, unless display board plugged in. */
> > > > +		status = "disabled";
> > > > +	};
> > >
> > > The both backlight is not used in this dts, can you remove it and
> > > add these when it real use.
> > It's the same thing we do on the other boards, so I would prefer to keep
> > this consistent:
>
> I agree with Marek, and I would not use the criteria about being used or
> not.


Others boards: some has dtso, which use it. And krzy have some concern for
similar case although he have not reply alex's answer.

https://lore.kernel.org/imx/30599533-91de-41fd-9420-3f8693ec92ac@xxxxxxxxxx/

And it is most likely go through nexus connector node to support display
panel in future.

So I suggest remove it now and add it when need.

Frank

>
> Francesco
>