UEFI update files missing on sandbox?

[Sughosh Ganu]

hi Simon,

On Wed, 13 Sept 2023 at 23:37, Simon Glass <sjg at chromium.org> wrote:
>
> Hi Sughosh,
>
> On Sun, 10 Sept 2023 at 13:13, Simon Glass <sjg at chromium.org> wrote:
> >
> > Hi Sughosh,
> >
> > On Sun, 10 Sept 2023 at 04:54, Sughosh Ganu <sughosh.ganu at linaro.org> wrote:
> > >
> > > hi Simon,
> > >
> > > On Sun, 10 Sept 2023 at 04:10, Simon Glass <sjg at chromium.org> wrote:
> > > >
> > > > Hi Sughosh,
> > > >
> > > > I thought that all your patches were merged but now when I build
> > > > sandbox I don't see the capsule-update files generated. Is there
> > > > something missing?
> > >
> > > The building of the EFI capsules was moved to the corresponding test
> > > setup. This was based on feedback from Tom Rini [1], where he had
> > > suggested that instead of having the binman nodes for the capsules as
> > > part of the sandbox dtb, it should rather be highlighted in the
> > > document.
> >
> > Hmmm that was not how I was hoping it would work.
> >
> > Is there a way to make the build create the files?
> >
> > ...I just found efi_capsule_data()
> >
> > I really don't like this at all...it is running a part of the build
> > system in the test. We have to do this in some cases, e.g. for tracing
> > we must run with a special build of sandbox, but it should not be
> > necessary for EFI capsules.
> >
> > I would really like the sandbox build to produce the capsules needed
> > for tests, without any extra effort. As it is, there is no way to try
> > this out.
>
> How about we add a single capsule update to the sandbox build, so that
> it produces an update file containing U-Boot? Is that easy to do?

Not a difficult thing at all. But is Tom okay with this? I moved the
capsule generation to the tests based on his feedback. So I would like
to hear what he has to say on this. I don't want to keep doing this
again and again.

-sughosh