[PATCH v4] fdt: Allow the devicetree to come from a bloblist

[Tom Rini]

On Wed, Dec 27, 2023 at 05:48:42PM +0000, Simon Glass wrote:
> Hi Tom,
> 
> On Tue, Dec 26, 2023 at 12:07 PM Tom Rini <trini at konsulko.com> wrote:
> >
> > On Tue, Dec 26, 2023 at 09:46:25AM +0000, Simon Glass wrote:
> >
> > > Standard passage provides for a bloblist to be passed from one firmware
> > > phase to the next. That can be used to pass the devicetree along as well.
> > > Add an option to support this.
> > >
> > > Tests for this will be added as part of the Universal Payload work.
> > >
> > > Signed-off-by: Simon Glass <sjg at chromium.org>
> > > ---
> > > The discussion on this was not resolved and is now important due to the
> > > bloblist series from Raymond. So I am sending it again since I believe
> > > this is a better starting point than building on OF_BOARD
> >
> > I really don't like adding another option for "DT is given to us". Why
> > isn't adding another enum to fdt_source_t sufficient
> 
> That is added by this patch, but...
> 
> >, and if we have
> > bloblist enabled, that will look for and use if found?
> 
> ...this is the question. I would like to be able to enable bloblist
> without *requiring* the DT to come from there, hence the separate
> Kconfig option.
> 
> > Maybe some other
> > code needs to be restructured and cleaned up too?
> 
> Possibly, but I really am not keen on this board-specific solution. I
> believe Ilias & I agreed that OF_BOARD should fade away, so I don't
> like adding another thing onto it.

I think you're missing something then. It should not be board specific
to just find the dtb via bloblist, without a new CONFIG. Whatever is
stopping that is what needs to be refactored and then you can just
extend fdt_source_t without a new CONFIG option.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20231227/f4b32af6/attachment.sig>