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

Tom Rini trini at konsulko.com
Fri Dec 22 13:07:00 CET 2023


On Fri, Dec 22, 2023 at 12:05:39PM +0000, Shantur Rathore wrote:
> Hi all,
> 
> On Mon, Dec 11, 2023 at 6:27 PM Tom Rini <trini at konsulko.com> wrote:
> >
> > On Mon, Dec 11, 2023 at 11:22:45AM -0700, Simon Glass wrote:
> > > Hi Tom,
> > [snip]
> > > > I think in hind-sight too much stuff is omitted without BOOTSTD_FULL.
> > > > The option itself then enables other stuff too by default, but some
> > > > parts of the bootflow command itself should be visible even without FULL
> > > > to make things easier on the user.
> > >
> > > At the time the goal was to avoid growth compared to the distro
> > > scripts. We could perhaps add some more things in with BOOTSTD_FULL
> > > but still have it as an option?
> >
> > Right. But now that we've tried this, some of the feedback has been that
> > it's just too minimal right now. Like looking at the help message for
> > bootflow, list and info should probably always be available. And maybe
> > the flags for "scan" should be re-thought? Too late to change things now
> > but "bootflow scan -b" should maybe how it's always been for "scan and
> > boot".
> 
> What would be the preferred approach for this patch?
> Is it to update the default capabilities of bootflow or we can have this patch?

Well, I don't see a problem with just adding this to the platforms which
enable BOOTSTD_FULL until we can rework what's included/excluded by that
flag, and there's an issue filed for it now.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20231222/2fee41f0/attachment.sig>


More information about the U-Boot mailing list