[PATCH v3] arch: arm: Kconfig: Enable BOOTSTD_FULL for Rockchip SoCs

Shantur Rathore i at shantur.com
Sat Dec 9 20:49:04 CET 2023


On Sat, Dec 9, 2023 at 7:18 PM Tom Rini <trini at konsulko.com> wrote:
>
> On Fri, Dec 08, 2023 at 10:52:02AM +0000, Shantur Rathore wrote:
>
> > Hi Tom / Kever
> >
> > On Sun, Nov 19, 2023 at 5:24 PM Shantur Rathore <i at shantur.com> wrote:
> > >
> > > Rockchip SoCs can support wide range of bootflows.
> > > Without full bootflow commands, it can be difficult to
> > > figure out issues if any, hence enable by default.
> > >
> > > Reviewed-by: Simon Glass <sjg at chromium.org>
> > >
> > > Signed-off-by: Shantur Rathore <i at shantur.com>
> > > ---
> > >
> > > (no changes since v1)
> > >
> > >  arch/arm/Kconfig | 1 +
> > >  1 file changed, 1 insertion(+)
> > >
> > > diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
> > > index d812685c98..fca6ef6d7e 100644
> > > --- a/arch/arm/Kconfig
> > > +++ b/arch/arm/Kconfig
> > > @@ -1986,6 +1986,7 @@ config ARCH_ROCKCHIP
> > >         imply CMD_DM
> > >         imply DEBUG_UART_BOARD_INIT
> > >         imply BOOTSTD_DEFAULTS
> > > +       imply BOOTSTD_FULL if BOOTSTD_DEFAULTS
> > >         imply FAT_WRITE
> > >         imply SARADC_ROCKCHIP
> > >         imply SPL_SYSRESET
> >
> > Can this please be merged in ?
>
> I wonder if we shouldn't really globally default to BOOTSTD_FULL if
> BOOTSTD_DEFAULTS for everyone.
>

Its matter of ~21KB in size, unless any platform is really to its
limits it should be alright.

Regards,
Shantur


More information about the U-Boot mailing list