Hi Bin, On Wed, 12 Jul 2023 at 05:54, Bin Meng <[email protected]> wrote: > > Hi Simon, > > On Tue, Jun 20, 2023 at 4:23 PM Simon Glass <[email protected]> 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
