On Thu, Apr 21, 2022 at 11:47 AM Fabio Estevam festevam@denx.de wrote:
Hi Michael,
On 21/04/2022 13:34, Michael Nazzareno Trimarchi wrote:
That function should drop. There is not other architecture that does it. What about:
I implemented your suggestion like this:
--- a/arch/arm/mach-imx/imx8m/soc.c +++ b/arch/arm/mach-imx/imx8m/soc.c @@ -1536,6 +1536,14 @@ enum env_location arch_env_get_location(enum env_operation op, int prio) return ENVL_UNKNOWN;
switch (dev) {
case USB_BOOT:if (IS_ENABLED(CONFIG_ENV_IS_IN_SPI_FLASH))return ENVL_SPI_FLASH;if (IS_ENABLED(CONFIG_ENV_IS_IN_NAND))return ENVL_NAND;if (IS_ENABLED(CONFIG_ENV_IS_IN_MMC))return ENVL_MMC;return ENVL_NOWHERE; case QSPI_BOOT: case SPI_NOR_BOOT: if (IS_ENABLED(CONFIG_ENV_IS_IN_SPI_FLASH))This still allows me to save the env into the eMMC when U-Boot is loaded from USB.
register_enviroment_hook() deregister_enviroment_hook()
This can replace the one default with board/arch etc
Marek submitted these patches:
https://source.denx.de/u-boot/u-boot/-/commit/de70e8879bb253f4d2a9ba9149cd41...
https://source.denx.de/u-boot/u-boot/-/commit/e4dc2d0620851d6e0e784d4ef0a50f...
I believe those were applied today. I haven't verified it yet, but I think I saw the e-mail go by.
Is this the mechanism that you are looking for?