On 17/06/2022 04:21, Alexander Stein wrote:
>> What's this node about? There is no compatible.
>
> As mentioned in the cover letter, some nodes are not used yet. In this case
> the intention is to apply an overlay for enabling the display later on. As the
> MBa8MPxL is a starter kit mainboard there is no fixed display attached. So a
> user might use our supported display or chose his own. But the backlight and
> display are fixed, so anything related to that can be part of the general
> mainboad DTS.
> The display specific overlay will then have (among other things) the following
> snippet:
>
Sure. Could you add such explanation to the comment in DTS next to this
unusual node? This would answer people's question if they wonder about
the display node.
>> &display0 {
>> compatible = "tianma,tm070jvhg33";
>> status = "okay";
>> };
>
> Power supply and enable GPIO are fixed, so there the can be part of the
> mainboard DTS while the actual compatible for the display might change.
>
> I hope this makes this approach a bit more clear and this approach is
> acceptable.
Best regards,
Krzysztof
