[PATCH v4 4/5] dm: core: Modify default for OFNODE_MULTI_TREE

Simon Glass sjg at chromium.org
Sun Nov 12 21:01:05 CET 2023


Hi Sean,

On Fri, 3 Nov 2023 at 13:38, Simon Glass <sjg at chromium.org> wrote:
>
> On Fri, 3 Nov 2023 at 12:38, <seanedmond at linux.microsoft.com> wrote:
> >
> > From: Sean Edmond <seanedmond at microsoft.com>
> >
> > There is a preference to use the "ofnode" API for FDT fixups
> > moving forward.  The FDT fixup will usually be for the kernel FDT.  To
> > fixup the kernel FDT with the ofnode API, it's required to set the
> > OFNODE_MULTI_TREE option.
> >
> > To ensure existing users of kaslr fdt fixup are not impacted, Let's modify
> > the default value for OFNODE_MULTI_TREE to ensure it's always set if
> > !OF_LIVE.  This will cause a 1007 byte increase in the code size.
> >
> > Signed-off-by: Sean Edmond <seanedmond at microsoft.com>
> > ---
> >  drivers/core/Kconfig | 3 ++-
> >  1 file changed, 2 insertions(+), 1 deletion(-)
> >
>
> Reviewed-by: Simon Glass <sjg at chromium.org>

This actually disables the option for sandbox. You may as well enabled
it by default always, since if OF_LIVE is active we are already adding
a lot of code.

That explains the cedit error that Tom mentioned.

Regards,
Simon


More information about the U-Boot mailing list