[PATCH v7 2/5] bootstd: select BLK rather than depending on it

[Simon Glass]

Hi Tom,

On Thu, 25 Sept 2025 at 08:18, Tom Rini <trini at konsulko.com> wrote:
>
> On Wed, Sep 24, 2025 at 06:23:01PM -0600, Simon Glass wrote:
>
> > From: Simon Glass <sjg at chromium.org>
> >
> > With recent changes to how BLK works, we must use 'select' to enable the
> > BLK feature, which is currently required by BOOTSTD.
> >
> > This fixes the following error:
> >
> >     GEN     Makefile
> >   scripts/kconfig/conf  --syncconfig Kconfig
> >   boot/Kconfig:496:error: recursive dependency detected!
> >   boot/Kconfig:496:   symbol BOOTSTD_DEFAULTS depends on BOOTSTD
> >   boot/Kconfig:441:   symbol BOOTSTD default is visible depending on BLK
> >   drivers/block/Kconfig:1:    symbol BLK is selected by USB
> >   drivers/usb/Kconfig:1:      symbol USB is selected by BOOTSTD_DEFAULTS
> >
> > Disable BOOTSTD for the boards which don't have enough space or some
> > other problem:
> >
> >   zynq_cse_nand
> >   zynq_cse_nor
> >   zynq_cse_qspi
> >   gardena-smart-gateway-mt7688
> >   10m50
> >   3c120
> >
> > Signed-off-by: Simon Glass <sjg at chromium.org>
>
> This is the wrong direction. Please instead get the dependencies correct
> in the next patch. You're likely missing some conditionals on the
> selects you add as bootstd does not depend on BLK.

OK, I found that I shouldn't need the 'select USB' so I dropped that.
So I can send v8.

Also I spoke to Heinrich and we came up with the idea of supporting
bootdev priority in global boomeths, as a less hacky way of ensuring
the bootmgr doesn't slow down the boot when it isn't being used. That
would be a separate series but I can take a look at that.

- Simon