[RFC PATCH v3 1/5] imx8m: drop env_get_location for imx8mn and imx8mp

Teresa Remmet T.Remmet at phytec.de
Tue Jan 4 12:04:10 CET 2022


Hello Tommaso,

Am Samstag, dem 25.12.2021 um 21:25 +0100 schrieb Tommaso Merciai:
> This function defined for two architecture is not really generic
> and can generate problem when people add a new board.
> 
> Signed-off-by: Tommaso Merciai <tomm.merciai at gmail.com>
> ---
>  arch/arm/mach-imx/imx8m/soc.c | 39 -------------------------------
> ----
>  1 file changed, 39 deletions(-)
> 
> diff --git a/arch/arm/mach-imx/imx8m/soc.c b/arch/arm/mach-
> imx/imx8m/soc.c
> index 863508776d..f0030a557a 100644
> --- a/arch/arm/mach-imx/imx8m/soc.c
> +++ b/arch/arm/mach-imx/imx8m/soc.c
> @@ -1313,45 +1313,6 @@ void do_error(struct pt_regs *pt_regs,
> unsigned int esr)
>  #endif
>  
>  #if defined(CONFIG_IMX8MN) || defined(CONFIG_IMX8MP)
> -enum env_location env_get_location(enum env_operation op, int prio)
> -{
> -	enum boot_device dev = get_boot_device();
> -	enum env_location env_loc = ENVL_UNKNOWN;
> -
> -	if (prio)
> -		return env_loc;
> -
> -	switch (dev) {
> -#ifdef CONFIG_ENV_IS_IN_SPI_FLASH
> -	case QSPI_BOOT:
> -		env_loc = ENVL_SPI_FLASH;
> -		break;
> -#endif
> -#ifdef CONFIG_ENV_IS_IN_NAND
> -	case NAND_BOOT:
> -		env_loc = ENVL_NAND;
> -		break;
> -#endif
> -#ifdef CONFIG_ENV_IS_IN_MMC
> -	case SD1_BOOT:
> -	case SD2_BOOT:
> -	case SD3_BOOT:
> -	case MMC1_BOOT:
> -	case MMC2_BOOT:
> -	case MMC3_BOOT:
> -		env_loc =  ENVL_MMC;
> -		break;
> -#endif
> -	default:
> -#if defined(CONFIG_ENV_IS_NOWHERE)
> -		env_loc = ENVL_NOWHERE;
> -#endif
> -		break;
> -	}
> -
> -	return env_loc;
> -}
> -
>  #ifndef ENV_IS_EMBEDDED
>  long long env_get_offset(long long defautl_offset)

would it not make sense to move also env_get_offset() to board level?

Regards,
Teresa


>  {
-- 
PHYTEC Messtechnik GmbH | Robert-Koch-Str. 39 | 55129 Mainz, Germany

Geschäftsführer: Dipl.-Ing. Michael Mitezki, Dipl.-Ing. Bodo Huber |
Handelsregister Mainz HRB 4656 | Finanzamt Mainz | St.Nr. 266500608, DE
149059855


More information about the U-Boot mailing list