On Thu, Nov 30, 2023 at 01:02:25PM +0530, Sumit Garg wrote: > On Wed, 29 Nov 2023 at 22:06, Neil Armstrong <neil.armstr...@linaro.org> > wrote: > > > > On 29/11/2023 16:34, Caleb Connolly wrote: > > > > > > > > > On 23/11/2023 07:04, Sumit Garg wrote: > > >> On Wed, 22 Nov 2023 at 21:34, Caleb Connolly <caleb.conno...@linaro.org> > > >> wrote: > > >>> > > >>> > > >>> > > >>> On 22/11/2023 14:27, Tom Rini wrote: > > >>>> On Wed, Nov 22, 2023 at 07:44:09PM +0530, Sumit Garg wrote: > > >>>>> On Wed, 22 Nov 2023 at 19:31, Tom Rini <tr...@konsulko.com> wrote: > > >>>>>> > > >>>>>> On Wed, Nov 22, 2023 at 11:51:29AM +0530, Sumit Garg wrote: > > >>>>>>> Hi Caleb, > > >>>>>>> > > >>>>>>> On Tue, 21 Nov 2023 at 22:39, Caleb Connolly > > >>>>>>> <caleb.conno...@linaro.org> wrote: > > >>>>>> [snip] > > >>>>>>>> == DT loading == > > >>>>>>>> > > >>>>>>>> Previously, boards used the FDT blob embedded into U-Boot (via > > >>>>>>>> OF_SEPARATE). However, most Qualcomm boards run U-Boot as a > > >>>>>>>> secondary > > >>>>>>>> bootloader, so we can instead rely on the first-stage bootloader to > > >>>>>>>> populate some useful FDT properties for us (notably the /memory > > >>>>>>>> node and > > >>>>>>>> KASLR seed) and fetch the DTB that it provides. Combined with the > > >>>>>>>> memory > > >>>>>>>> map changes above, this let's us entirely avoid configuring the > > >>>>>>>> memory > > >>>>>>>> map explicitly. > > >>>>>>> > > >>>>>>> Since with this change, we don't need to embed FDT blob in the > > >>>>>>> u-boot > > >>>>>>> binary, so I was thinking if we really need to import DTs from Linux > > >>>>>>> for different platforms and then play a catchup game? > > >>> > > >>> For now, yes. > > >> > > >> But why? Is there any value added by larger u-boot specific DT (most > > >> of the nodes being unused by u-boot) than what currently u-boot > > >> supports? The more important part is to get alignment with Linux DT > > >> bindings. If you need to have memory/reserved-memory nodes in u-boot > > >> DT for generalization purposes then you should import those particular > > >> nodes only. > > > > > > I've been thinking about and hacking on this for the last week or so, > > > sorry for the delayed reply here. > > > > > > The value is in preventing any of the existing bindings from regressing, > > That is actually best addressed in Linux by checking the DTS against > yaml DT bindings. We don't have that testing available in u-boot and > only depend on careful reviews.
I would absolutely love for someone to make another attempt at updating our kbuild infrastucture so that we can run the validation targets. -- Tom
signature.asc
Description: PGP signature