Hi Marek,

On 6/27/24 1:58 AM, Marek Vasut wrote:
Insert /u-boot,<dto-name> = <index> property into the U-Boot control DT
during SPL DTO application process. This can be used by user to inspect
which DTOs got applied by the SPL and in which order from running U-Boot.

Example:
```
u-boot=> fdt addr $fdtcontroladdr && fdt list /
Working FDT set to aee9aeb0
/ {
         u-boot,fdt-dto-imx8mp-dhcom-pdk3-overlay-rev100 = <0x00000005>;
         u-boot,fdt-dto-imx8mp-dhcom-som-overlay-rev100 = <0x00000004>;
         u-boot,fdt-dto-imx8mp-dhcom-pdk-overlay-eth2xfast = <0x00000003>;
         u-boot,fdt-dto-imx8mp-dhcom-som-overlay-eth2xfast = <0x00000002>;
...
```


Shouldn't this rather be in /config node?

Also, I am highly suggesting to have an additional prefix aside from u-boot to avoid unfortunate name clashes between DTO and existing properties. Or could be in its own child node of /config ?

In any case, can you please add or edit a file somewhere to define this new dt-binding.

Additional question, what is this index for?

I am wondering if we cannot simply have a string array with overlays applied left-to-right or right-to-left instead? Or u-boot,fdt-dto-N = "imx8mp-dhcom-pdk3-overlay-rev100";

Cheers,
Quentin

Reply via email to