mbox series

[00/11] bootm: Support substitions in bootargs and add tests

Message ID 20201019135602.3943835-1-sjg@chromium.org
Headers show
Series bootm: Support substitions in bootargs and add tests | expand

Message

Simon Glass Oct. 19, 2020, 1:55 p.m. UTC 
This series adds tests to the fixup_silent-linux() function and extends
the 'zimage' command to use it.

It also adds a new string-substition feature to allow bootargs to be a
template, rather than having to build it up piece by piece with
information obtained in a build script.

It also updates zimage to use the same command-line processing.

With these additions it is possible to boot Chrome OS from a U-Boot script
on most Chromebooks.


Simon Glass (11):
  env: Allow returning errors from hdelete_r()
  bootm: Add tests for fixup_silent_linux()
  bootm: Update fixup_silent_linux() to return an error
  bootm: Rename fixup_silent_linux()
  bootm: Add a bool parameter to bootm_process_cmdline_env()
  bootm: Use size rather than length for CONSOLE_ARG
  bootm: Split out bootargs environment reading / writing
  bootm: Update bootm_process_cmdline_env() to use flags
  bootm: Allow updating the bootargs in a buffer
  x86: zimage: Add silent-console processing
  bootm: Support string substitution in bootargs

 README                |  16 +++
 arch/Kconfig          |   2 +
 arch/x86/lib/zimage.c |  14 +++
 cmd/nvedit.c          |   6 +-
 common/Kconfig.boot   |  20 ++++
 common/bootm.c        | 206 +++++++++++++++++++++++++++++------
 include/bootm.h       |  40 +++++++
 include/search.h      |  11 +-
 include/test/suites.h |   1 +
 lib/hashtable.c       |  12 +--
 test/Makefile         |   1 +
 test/bootm.c          | 243 ++++++++++++++++++++++++++++++++++++++++++
 test/cmd_ut.c         |   1 +
 test/env/hashtable.c  |   2 +-
 14 files changed, 534 insertions(+), 41 deletions(-)
 create mode 100644 test/bootm.c

Comments

Heinrich Schuchardt Oct. 19, 2020, 2:39 p.m. UTC | #1 
On 19.10.20 15:55, Simon Glass wrote:
> This series adds tests to the fixup_silent-linux() function and extends
> the 'zimage' command to use it.
>
> It also adds a new string-substition feature to allow bootargs to be a
> template, rather than having to build it up piece by piece with
> information obtained in a build script.
>
> It also updates zimage to use the same command-line processing.
>
> With these additions it is possible to boot Chrome OS from a U-Boot script
> on most Chromebooks.

Hello Simon,

could you, please, add a patch documenting the usage of the new
string-substitution feature in the HTML documentation.

Best regards

Heinrich

>
>
> Simon Glass (11):
>   env: Allow returning errors from hdelete_r()
>   bootm: Add tests for fixup_silent_linux()
>   bootm: Update fixup_silent_linux() to return an error
>   bootm: Rename fixup_silent_linux()
>   bootm: Add a bool parameter to bootm_process_cmdline_env()
>   bootm: Use size rather than length for CONSOLE_ARG
>   bootm: Split out bootargs environment reading / writing
>   bootm: Update bootm_process_cmdline_env() to use flags
>   bootm: Allow updating the bootargs in a buffer
>   x86: zimage: Add silent-console processing
>   bootm: Support string substitution in bootargs
>
>  README                |  16 +++
>  arch/Kconfig          |   2 +
>  arch/x86/lib/zimage.c |  14 +++
>  cmd/nvedit.c          |   6 +-
>  common/Kconfig.boot   |  20 ++++
>  common/bootm.c        | 206 +++++++++++++++++++++++++++++------
>  include/bootm.h       |  40 +++++++
>  include/search.h      |  11 +-
>  include/test/suites.h |   1 +
>  lib/hashtable.c       |  12 +--
>  test/Makefile         |   1 +
>  test/bootm.c          | 243 ++++++++++++++++++++++++++++++++++++++++++
>  test/cmd_ut.c         |   1 +
>  test/env/hashtable.c  |   2 +-
>  14 files changed, 534 insertions(+), 41 deletions(-)
>  create mode 100644 test/bootm.c
>
Simon Glass Oct. 20, 2020, 7:12 p.m. UTC | #2 
Hi Heinrich,

On Mon, 19 Oct 2020 at 08:45, Heinrich Schuchardt <xypron.glpk@gmx.de> wrote:
>
> On 19.10.20 15:55, Simon Glass wrote:
> > This series adds tests to the fixup_silent-linux() function and extends
> > the 'zimage' command to use it.
> >
> > It also adds a new string-substition feature to allow bootargs to be a
> > template, rather than having to build it up piece by piece with
> > information obtained in a build script.
> >
> > It also updates zimage to use the same command-line processing.
> >
> > With these additions it is possible to boot Chrome OS from a U-Boot script
> > on most Chromebooks.
>
> Hello Simon,
>
> could you, please, add a patch documenting the usage of the new
> string-substitution feature in the HTML documentation.

OK will do. I meant to send this series three weeks ago and forgot about it...

Regards,
Simon