[PATCH v3 31/32] bootstd: Introduce programmatic boot

[Tom Rini]

On Tue, Nov 28, 2023 at 03:14:30PM -0600, Simon Glass wrote:
> Hi Tom,
> 
> 
> On Tue, Nov 28, 2023, 12:21 Tom Rini <trini at konsulko.com> wrote:
> 
> > On Sat, Nov 18, 2023 at 02:05:19PM -0700, Simon Glass wrote:
> >
> > > At present bootstd requires CONFIG_CMDLINE to operate. Add a new
> > > 'programmatic' boot which can be used when no command line is available.
> > > For now it does almost nothing, since most bootmeths require the
> > > command line.
> > >
> > > Signed-off-by: Simon Glass <sjg at chromium.org>
> >
> > Overall, this seems fine. The only ask I have really is, can we think
> > about how to handle both this case and the case handled by
> > cmd/bootmenu.c in a common fashion? To me, one is "show a menu of boot
> > options, based on env variables" and one is "show a menu of boot
> > options, based on C code" and the "show a menu of boot options" should
> > have a lot in common.
> 
> I had not thought about providing a menu with prog boot.

I guess I missed how there wasn't a menu with this series and conflated
that with one of your other series.

> Could we drop this patch from the series for now and apply the others? I
> would like to finish and send the follow-on

I didn't see any other feedback so yes, I'll be applying this soon'ish.

-- 
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/20231128/6528a1c5/attachment.sig>