On 19/03/2023 20.29, Simon Glass wrote: > Hi Rasmus, > > On Sun, 19 Mar 2023 at 13:34, Rasmus Villemoes > <rasmus.villem...@prevas.dk> wrote: >> >> On 18/03/2023 21.20, Simon Glass wrote: >> >>>> >>>> Now, the only way to be really sure is to build the world >>>> with/without this patch and check if any .dtb file changes, but I >>>> don't have the means to do that. But I do notice that >>>> >>>> >>>> doc/develop/devicetree/control.rst | 1 + >>>> scripts/Makefile.lib | 2 ++ >>>> 2 files changed, 3 insertions(+) >>> >>> Reviewed-by: Simon Glass <s...@chromium.org> >>> >>> There is something missing at the end of your note. >> >> Ah, sorry. I meant to say something about that it seems some existing >> in-tree cases could make use of this and thus be simplified, e.g. all >> the t8103-*-u-boot.dtsi just include t8103-u-boot.dtsi, but then I >> wasn't really sure if that was a proper example, so decided to edit that >> part out (but left that small fragment in). >> >>> What U-Boot-specific things are you using? >> >> Just the u-boot,dm-spl kind of stuff. > > With the new bindings, we can put those tags (bootph-...) in the kernel DTs.
Hm, ok, but actually I was a bit too fast, there's also some binman stuff, and a few things that I want/need to do for U-Boot but not the kernel and vice versa (e.g. some gpio-hogs), so all in all, having this patch in would still be very useful to me. Rasmus