Re: [PATCH 2/2] imx8mn/8mp: Select ENV_IS_NOWHERE

From: Fabio Estevam <festevam@gmail.com>
To: Marek Vasut <marex@denx.de>
Cc: Stefano Babic <sbabic@denx.de>, Peng Fan <peng.fan@nxp.com>,
	U-Boot-Denx <u-boot@lists.denx.de>,
	Adam Ford <aford173@gmail.com>,
	Michael Trimarchi <michael@amarulasolutions.com>,
	 Fabio Estevam <festevam@denx.de>
Subject: Re: [PATCH 2/2] imx8mn/8mp: Select ENV_IS_NOWHERE
Date: Thu, 21 Apr 2022 11:17:04 -0300	[thread overview]
Message-ID: <CAOMZO5BgQA3d3MnCC37S6WrO-Dbrf1__KSrwUuKObHSGP4VQBQ@mail.gmail.com> (raw)
In-Reply-To: <b57738f3-ff67-33d9-0af7-7d45bfa41b82@denx.de>

Hi Marek,

On Wed, Apr 20, 2022 at 7:15 PM Marek Vasut <marex@denx.de> wrote:

> Did you actually hit the USB_BOOT case or did you fall into the default:
> case ?

It does hit the USB_BOOT case.

I also tested forcing to return ENVL_UNKNOWN:

--- a/arch/arm/mach-imx/imx8m/soc.c
+++ b/arch/arm/mach-imx/imx8m/soc.c
@@ -1531,6 +1531,7 @@ void do_error(struct pt_regs *pt_regs)
 enum env_location env_get_location(enum env_operation op, int prio)
 {
        enum boot_device dev = get_boot_device();
+       return ENVL_UNKNOWN;

        if (prio)
                return ENVL_UNKNOWN;

but still, the boot does not complete.
