[PATCH v1] board: mpfs_icicle: implement board_fdt_blob_setup()

[Conor Dooley]

On Fri, Jun 28, 2024 at 08:53:11AM +0300, Ilias Apalodimas wrote:
> On Thu, 27 Jun 2024 at 23:27, Conor Dooley <conor at kernel.org> wrote:
> > On Thu, Jun 27, 2024 at 11:50:33AM +0100, Simon Glass wrote:
> > > On Thu, 27 Jun 2024 at 10:38, Conor Dooley <conor.dooley at microchip.com> wrote:
> > > > On Thu, Jun 27, 2024 at 09:36:49AM +0100, Simon Glass wrote:

> > > > I think the only way a bloblist could work is if it was created at build
> > > > time and linked into the firmware, since the on-disk format seems pretty
> > > > minimal. Is there tooling for generating a bloblist at build time that I
> > > > could use to check whether or not a bloblist is viable?
> > > > I'd also have to investigate how that would interact with OpenSBI, since
> > > > it's integrated into the firmware and involved with loading U-Boot.
> > >
> > > There is not such a tool, but it would be easy enough to write. If you
> > > think that would help, I could give it a try.
> >
> > I mean I could also just do it myself, I just wanted to know if it
> > existed, since that'd make investigating this pretty straightforward to
> > do.
> 
> Someone is already working on it.
> https://review.trustedfirmware.org/c/TF-A/trusted-firmware-a/+/29253
> this will help!

Eww, gerrit. Thanks for the link.

> > Either way, the firmware already is capable of doing this with a
> > devicetree blob, so I figure even if we manage to get bloblist stuff
> > going, there's little harm in supporting what's already there?
> 
> There isn't, but OTOH the bloblist support is already in u-boot.

Devicetree in a1 support is already in U-Boot too, it's already been
nicely placed in gd->arch.firmware_fdt_addr by the arch code :)

> I would prefer to try that out first. Up to you

If it is up to me, I am going to say go with what I have already done,
as it's much easier to update U-Boot on a disk that you can write to
from Linux than it is to change the firmware in envm and something
functional already exists.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20240628/fca88a6e/attachment.sig>