[PATCH v2 0/3] rpi: Convert to standard boot

[Simon Glass]

Hi Peter,

On Thu, 20 Apr 2023 at 20:20, Peter Robinson <pbrobinson at gmail.com> wrote:
>
> On Wed, Apr 19, 2023 at 11:40 PM Simon Glass <sjg at chromium.org> wrote:
> >
> > Hi Peter,
> >
> > On Wed, 19 Apr 2023 at 19:42, Peter Robinson <pbrobinson at gmail.com> wrote:
> > >
> > > Hi Simon,
> > >
> > > On Wed, Apr 19, 2023 at 2:46 AM Simon Glass <sjg at chromium.org> wrote:
> > > >
> > > > Hi Peter,
> > > >
> > > > On Sun, 2 Apr 2023 at 14:29, Simon Glass <sjg at chromium.org> wrote:
> > > > >
> > > > > This series moves Raspberry Pi boards over to use standard boot.
> > > > >
> > > > > It also moves rpi over to use a text-based environment. Unfortunately it
> > > > > is not possible to empty the header file due to several CFG options.
> > > > >
> > > > > Fix the repeated "and and" while we are here.
> > > > >
> > > > > Note that this reduces rodata size by about 4.5KB. We could get another
> > > > >
> > > > > for a total image-size saving of about 15KB. This is mostly because
> > > > > HUSH_PARSER is not enabled anymore and the environment shrinks down by
> > > > > about 3.5K. Hush is not actually needed anymore, since standard boot does
> > > > > not use it. Also CMD_SYSBOOT is dropped since standard boot calls the
> > > > > pxe_utils code directly in that case.
> > > > >
> > > > > Changes in v2:
> > > > > - Rebase to -next
> > > > > - Add new patch to disable DISTRO_DEFAULTS
> > > > >
> > > > > Simon Glass (3):
> > > > >   arm: rpi: Switch to standard boot
> > > > >   rpi: Disable DISTRO_DEFAULTS
> > > > >   arm: rpi: Switch to a text environment
> > > > >
> > > > >  board/raspberrypi/rpi/rpi.env  |  77 +++++++++++++++++++
> > > > >  configs/rpi_0_w_defconfig      |   2 +-
> > > > >  configs/rpi_2_defconfig        |   2 +-
> > > > >  configs/rpi_3_32b_defconfig    |   2 +-
> > > > >  configs/rpi_3_b_plus_defconfig |   2 +-
> > > > >  configs/rpi_3_defconfig        |   2 +-
> > > > >  configs/rpi_4_32b_defconfig    |   2 +-
> > > > >  configs/rpi_4_defconfig        |   2 +-
> > > > >  configs/rpi_arm64_defconfig    |   2 +-
> > > > >  configs/rpi_defconfig          |   2 +-
> > > > >  include/configs/rpi.h          | 134 ---------------------------------
> > > > >  11 files changed, 86 insertions(+), 143 deletions(-)
> > > > >  create mode 100644 board/raspberrypi/rpi/rpi.env
> > > > >
> > > > > --
> > > > > 2.40.0.348.gf938b09366-goog
> > > > >
> > > >
> > > > Any thoughts on this series, please?
> > >
> > > It's in my backlog to review, I've been focused on other things so my
> > > time is limited ATM but I'm also a little hesitant given the issues
> > > with the rockchip conversion and still outstanding issues there.
> >
> > Yes, there is an outstanding series on that side also. I hope you can
> > find the time soon, as the merge window closes in a week or so
> > (getting it in before then gives us the most test time before the
> > release).
>
> This isn't my day job, I do it when I get time, I'm in the midst of
> renovations and recovery from a broken wrist. My time currently is
> limited.

Thanks for the response. Yes I understand. A lot of people are in the
same boat. This isn't my day job either and I have a few things going
on.

The initial series was sent in February. One option is to apply it and
let people test it and find any remaining problems. But we should do
it at the same time as the rockchip, since they are now linked.

In the meantime I'll see if I can pick up the rockchip fixes, in case
they relate to any testing you might be able to do.

Regards,
Simon