[PATCH v2 00/19] bootstd: Add a bootmeth for ChromiumOS on x86

[Simon Glass]

Hi Bin,

On Wed, 12 Jul 2023 at 05:54, Bin Meng <bmeng.cn at gmail.com> wrote:
>
> Hi Simon,
>
> On Tue, Jun 20, 2023 at 4:23 PM Simon Glass <sjg at chromium.org> wrote:
> >
> > This series adds a simple bootmeth for ChromiumOS on x86. It uses zimage
> > to boot the kernel.
> >
> > Full verified boot is not included at this stage - that is still a
> > separate chunk of code to be brought into standard boot at some point.
> > For now it just obtains the kernel and command line and boots. This should
> > be enough to boot Chrome OS from coreboot on all x86 machines in
> > circulation, although only Brya (2022) and Coral (2017) have been tested.
> >
> > ChromiumOS needs quite large kernel parameters, to hold the DM verity
> > settings and other pieces. This makes it painful to modify just one
> > parameter, since the whole cmdline must be adjusted at once. To cope with
> > this, a new cmdline-editing feature is provided: the 'bootflow cmdline'
> > command allows individual parameters to be added, modified and deleted.
> >
> > To deal with enabling debug console, a variant supports setting 'earlycon'
> > and 'console' automatically. The 'bdinfo' command is updated to show
> > serial-port info also.
> >
> > Booting the zimage is now done programmatically, rather than running
> > through the command-line interface. Minor tweaks are made to the coreboot
> > and coral config so that booting works correctly.
> >
> > Note that the ACPI tables are not updated with the required firmware
> > information in this series, so a warning is shown on boot. This will be
> > addressed later since it requires quite a bit of configuration.
> >
> > Finally, this fixes a recently introduced bug in unit testing and updates
> > the algorithm to avoid running flat-tree tests which don't actually use
> > the devicetree.
> >
> > Changes in v2:
> > - Add a comment that this is not a full revert
> > - Update host_bootdev.c too
> > - Correct 'it not known' typo
> > - Explain why the setup information is not behind an #ifdef
> > - Add comment for copy_in()
> > - Avoid multiplication on a boolean
> > - Add a comment as to why @len is needed
> > - Show error messages when something goes wrong
> > - Add new patch to create a little more room for U-Boot on qemu-x86_64
> > - Add new patch to switch x86 qemu to standard boot
> > - Drop mention of NVMe support
>
> v2 does not apply on top of u-boot/master, please rebase. Thanks!
>

It is based on top of the other series. I'll rebase this series on
us/master and resend.

Regards,
Simon