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

[Marek Vasut <marex@nabladev.com>]

On 9/21/26 9:23 PM, Frank Li wrote:

[...]

>>>> Hence, base common stuff in the base DT, stuff that changes in the DTOs.
>>>
>>> Idea is good, but avoid hack stuff. Can you show me what's plan use
>>> reg_panel_vcc in dtso and how to share it.
>> See above, currently I cannot really use the regulator, because of the
>> missing DPU support.
> 
> I suggest add it when it will be used.

The users can already use it with additional downstream patches, so 
having the base node in the DT is helpful to the users.

> Or Remove gpios = <&i2c2_gpio_expander_20 17 0>;

Having the GPIO described here is also helpful to the users, since they 
won't have to look up which GPIO is controlling this regulator in the 
schematic, and won't make mistakes.

> Suppose it will be overwrited by dtso future.

Only the polarity.

>> The only DTO I temporarily have in tree (used with my
>> DPU patchset) is for the RPi 7" display 2, which does not use this
>> regulator.
>>
>> The plan is to use it the same way as on the MX8MM/MX8MP eDM SBC in the DTO:
>>
>> &reg_panel_vcc {
>>    gpios = <&i2c2_gpio_expander_20 17 GPIO_ACTIVE_HIGH>;
>>    enable-active-high;
>>    status = "okay";
>> };
>>
>> or
>>
>> &reg_panel_vcc {
>>    gpios = <&i2c2_gpio_expander_20 17 GPIO_ACTIVE_LOW>;
>>    status = "okay";
>> };
> 
> In future, it should be nexus node for it and put reg_panel_vcc into dtso
> files, and looks like you use gpio controller add-on boards's regulator.
There is really no need for a nexus node, only for a polarity setting.