[PATCH v2 0/7] efi: Start to chip away at the EFI workaround

Tom Rini trini at konsulko.com
Thu Dec 12 02:17:05 CET 2024


On Wed, Dec 11, 2024 at 05:58:12PM -0700, Simon Glass wrote:
> Hi Tom,
> 
> On Wed, 11 Dec 2024 at 15:50, Tom Rini <trini at konsulko.com> wrote:
> >
> > On Wed, Dec 11, 2024 at 03:38:15PM -0700, Simon Glass wrote:
> >
> > > At present there is a function, efi_set_bootdev(), which is used in
> > > various places to tell the EFI loader which device a file came from.
> > >
> > > With bootstd, this information is available in the bootflow.
> > >
> > > This little series provides a way for bootstd to provide the bootflow to
> > > the EFI loader, so that it is able to select the correct paths.
> > >
> > > For now only the EFI bootmeth is updated.
> >
> > Hi Simon,
> >
> > I think as a general rule it would help everyone if you had no more than
> > one outstanding series per subsystem per week. This is 30 efi_loader
> > patches in 24 hours and that's _a_lot_. One could easily argue that's
> > too much. Thanks.
> 
> Sure, I'm happy to send fewer patches. Could you go into a little more
> detail about what 'per week' means in this context. Can you give me an
> example?

Sure. No new patches, no re-spinnning patches, no re-posting patches
that touch lib/efi_loader/* until next Wednesday.

> This series has been sitting around since mid-October. It just didn't
> get reviewed / applied. I just hit a situation where I needed PXE
> booting, so I added another patch and sent a v2.

Per the changelog, you also made a few other changes in here since v1
other than picking up Ilias' tag.

Please also note that it's been not quite a week since you posted the 46
part "pxe" series, and that needs to be broken up in to something
reviewable-sized too.

-- 
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/20241211/52037bd3/attachment.sig>


More information about the U-Boot mailing list