(EXT) Re: [PATCH 2/2] arm64: dts: freescale: add initial device tree for TQMa8MPQL with i.MX8MP

[Krzysztof Kozlowski]

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