Re: [PATCH 1/2] config_distro_bootcmd: make possible to substitute 'part list' in distro_bootcmd

[Pascal Zimmermann <pzimmermann@dh-electronics.com>]

Hello Simon,

On Wed, 27 Nov 2024 at 14:08, Simon Glass <sjg@chromium.org> wrote:
> 
> Hi Pascal,
> 
> On Wed, 27 Nov 2024 at 03:30, Pascal Zimmermann <pzimmermann@dh-
> electronics.com> wrote:
> >
> > Make it possible to substitute the 'part list' command inside
> > 'scan_dev_for_boot_part' with a custom board specific implementation.
> >
> > For this the new define 'SCAN_DEV_FOR_BOOT_PARTS' is introduced.
> >
> > Signed-off-by: Pascal Zimmermann <pzimmermann@dh-electronics.com>
> > ---
> > Cc: Tom Rini <trini@konsulko.com>
> > Cc: Simon Glass <sjg@chromium.org>
> > Cc: u-boot@lists.denx.de
> > ---
> >  include/config_distro_bootcmd.h | 7 ++++++-
> >  1 file changed, 6 insertions(+), 1 deletion(-)
> 
> Have you looked at converting the board to standard boot?

We are at internal discussion about the switch to standard boot
for our boards, but this will take some time.

> 
> >
> > diff --git a/include/config_distro_bootcmd.h
> > b/include/config_distro_bootcmd.h index 0a4e4b8ff8..df0dc8cfde 100644
> > --- a/include/config_distro_bootcmd.h
> > +++ b/include/config_distro_bootcmd.h
> > @@ -194,6 +194,11 @@
> >  #define SCAN_DEV_FOR_EFI
> >  #endif
> >
> > +#ifndef SCAN_DEV_FOR_BOOT_PARTS
> > +#define SCAN_DEV_FOR_BOOT_PARTS \
> > +       "part list ${devtype} ${devnum} -bootable devplist; "
> > +#endif
> > +
> >  #ifdef CONFIG_SATA
> >  #define BOOTENV_SHARED_SATA    BOOTENV_SHARED_BLKDEV(sata)
> >  #define BOOTENV_DEV_SATA       BOOTENV_DEV_BLKDEV
> > @@ -538,7 +543,7 @@
> >                 "\0"                                                      \
> >         \
> >         "scan_dev_for_boot_part="                                         \
> > -               "part list ${devtype} ${devnum} -bootable devplist; "     \
> > +               SCAN_DEV_FOR_BOOT_PARTS                                   \
> >                 "env exists devplist || setenv devplist 1; "              \
> >                 "for distro_bootpart in ${devplist}; do "                 \
> >                         "if fstype ${devtype} "                           \
> > --
> > 2.30.2
> >
> 
> Regards,
> Simon

Best regards,
Pascal