On Fri, 24 Jun 2022 17:12:09 +0100
Andre Przywara <andre.przyw...@arm.com> wrote:
> From: Samuel Holland <sam...@sholland.org>
>
> Commit e42dad4168fe ("sunxi: use boot source for determining environment
> location") changed our implementation of env_get_location() and enabled
> it for every board, even those without MMC support (like the C.H.I.P.
> boards). However the default fallback location of ENVL_FAT requires MMC
> support compiled in, so the board hangs when trying to initially load
> the environment.
>
> Change the algorithm to only return configured environment locations,
> and improve the fallback algorithm on the way.
>
> The env_init() routine calling this function here does not behave well
> if the return value is ENVL_UNKNOWN on the very first call: it will make
> U-Boot proper silently hang very early.
> Work around this issue by making sure we return some configured (dummy)
> environment location when prio is 0. This for instance happens when
> booting via FEL.
>
> This fixes U-Boot loading on the C.H.I.P. boards.
>
> Fixes: e42dad4168fe ("sunxi: use boot source for determining environment
> location")
> Reported-by: Chris Morgan <macroalph...@gmail.com>
> Signed-off-by: Samuel Holland <sam...@sholland.org>
> [Andre: fix FEL boot case by not returning ENVL_UNKNOWN when prio==0]
> Signed-off-by: Andre Przywara <andre.przyw...@arm.com>
Applied to sunxi/master, for v2022.07.
Cheers,
Andre
> ---
> Hi Samuel,
>
> I cheekily added your Signed-off-by:, as I made this your patch (since
> you came up with it in that email reply some weeks ago). I hope that's
> fine.
> I tested this briefly on a Pine64-LTS, booting from FEL, SD card, eMMC
> and SPI. I also simulated a CHIP board, by just defining ENV_IS_NOWHERE,
> then booting, as this was the case that broke.
> If no one complains, I would like to push this ASAP, to make it into
> the 2022.07 release still.
>
> Cheers,
> Andre
>
> board/sunxi/board.c | 49 +++++++++++++++++++++++++++------------------
> 1 file changed, 29 insertions(+), 20 deletions(-)
>
> diff --git a/board/sunxi/board.c b/board/sunxi/board.c
> index 806e3bcb69..21a2407e06 100644
> --- a/board/sunxi/board.c
> +++ b/board/sunxi/board.c
> @@ -128,26 +128,37 @@ void i2c_init_board(void)
> * Try to use the environment from the boot source first.
> * For MMC, this means a FAT partition on the boot device (SD or eMMC).
> * If the raw MMC environment is also enabled, this is tried next.
> + * When booting from NAND we try UBI first, then NAND directly.
> * SPI flash falls back to FAT (on SD card).
> */
> enum env_location env_get_location(enum env_operation op, int prio)
> {
> - enum env_location boot_loc = ENVL_FAT;
> + if (prio > 1)
> + return ENVL_UNKNOWN;
>
> - gd->env_load_prio = prio;
> + /* NOWHERE is exclusive, no other option can be defined. */
> + if (IS_ENABLED(CONFIG_ENV_IS_NOWHERE))
> + return ENVL_NOWHERE;
>
> switch (sunxi_get_boot_device()) {
> case BOOT_DEVICE_MMC1:
> case BOOT_DEVICE_MMC2:
> - boot_loc = ENVL_FAT;
> + if (prio == 0 && IS_ENABLED(CONFIG_ENV_IS_IN_FAT))
> + return ENVL_FAT;
> + if (IS_ENABLED(CONFIG_ENV_IS_IN_MMC))
> + return ENVL_MMC;
> break;
> case BOOT_DEVICE_NAND:
> + if (prio == 0 && IS_ENABLED(CONFIG_ENV_IS_IN_UBI))
> + return ENVL_UBI;
> if (IS_ENABLED(CONFIG_ENV_IS_IN_NAND))
> - boot_loc = ENVL_NAND;
> + return ENVL_NAND;
> break;
> case BOOT_DEVICE_SPI:
> - if (IS_ENABLED(CONFIG_ENV_IS_IN_SPI_FLASH))
> - boot_loc = ENVL_SPI_FLASH;
> + if (prio == 0 && IS_ENABLED(CONFIG_ENV_IS_IN_SPI_FLASH))
> + return ENVL_SPI_FLASH;
> + if (IS_ENABLED(CONFIG_ENV_IS_IN_FAT))
> + return ENVL_FAT;
> break;
> case BOOT_DEVICE_BOARD:
> break;
> @@ -155,21 +166,19 @@ enum env_location env_get_location(enum env_operation
> op, int prio)
> break;
> }
>
> - /* Always try to access the environment on the boot device first. */
> - if (prio == 0)
> - return boot_loc;
> -
> - if (prio == 1) {
> - switch (boot_loc) {
> - case ENVL_SPI_FLASH:
> + /*
> + * If we come here for the first time, we *must* return a valid
> + * environment location other than ENVL_UNKNOWN, or the setup sequence
> + * in board_f() will silently hang. This is arguably a bug in
> + * env_init(), but for now pick one environment for which we know for
> + * sure to have a driver for. For all defconfigs this is either FAT
> + * or UBI, or NOWHERE, which is already handled above.
> + */
> + if (prio == 0) {
> + if (IS_ENABLED(CONFIG_ENV_IS_IN_FAT))
> return ENVL_FAT;
> - case ENVL_FAT:
> - if (IS_ENABLED(CONFIG_ENV_IS_IN_MMC))
> - return ENVL_MMC;
> - break;
> - default:
> - break;
> - }
> + if (IS_ENABLED(CONFIG_ENV_IS_IN_UBI))
> + return ENVL_UBI;
> }
>
> return ENVL_UNKNOWN;
