[U-Boot] [PATCH v2] arm, imx6: fix NOR/OneNAND boot mode mix-up

From: Stefano Babic <sbabic@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2] arm, imx6: fix NOR/OneNAND boot mode mix-up
Date: Mon, 23 Jul 2018 10:48:57 +0200	[thread overview]
Message-ID: <1638b3f0-5c92-bfeb-0821-0e5b384351b0@denx.de> (raw)
In-Reply-To: <20180622051852.21840-1-jay.d.carlson@gmail.com>

Hi Jay,

On 22/06/2018 07:18, Jay Carlson wrote:
> This patch fixes the ordering of the EMI enum to match Table 8-8 in 
> the i.MX6ULL Reference Manual, and has been spot-checked in two other 
> i.MX reference manuals for accuracy.
> 
> Signed-off-by: Jay Carlson <jay.d.carlson@gmail.com>
> ---
>  arch/arm/include/asm/mach-imx/sys_proto.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/arm/include/asm/mach-imx/sys_proto.h b/arch/arm/include/asm/mach-imx/sys_proto.h
> index d1d6cbc462..0acc3640f9 100644
> --- a/arch/arm/include/asm/mach-imx/sys_proto.h
> +++ b/arch/arm/include/asm/mach-imx/sys_proto.h
> @@ -63,8 +63,8 @@ enum imx6_bmode_serial_rom {
>  };
>  
>  enum imx6_bmode_emi {
> -	IMX6_BMODE_ONENAND,
>  	IMX6_BMODE_NOR,
> +	IMX6_BMODE_ONENAND
>  };
>  

I am not understanding which is the issue. The enum are just used in
switch() section as far as I know, and then the order is not so
important. Which is the issue / error you found ?

Best regards,
Stefano babic

-- 
=====================================================================
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================