[U-Boot] [PATCH] MX53: Handle silicon revision 2.1 case

[Stefano Babic <sbabic@denx.de>]

On 04/26/2011 02:46 AM, Fabio Estevam wrote:
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> ---
>  arch/arm/cpu/armv7/mx5/soc.c             |    3 +++
>  arch/arm/include/asm/arch-mx5/imx-regs.h |    1 +
>  2 files changed, 4 insertions(+), 0 deletions(-)
> 

Hi Fabio,

> diff --git a/arch/arm/cpu/armv7/mx5/soc.c b/arch/arm/cpu/armv7/mx5/soc.c
> index 6f4e8db..fa7ee6d 100644
> --- a/arch/arm/cpu/armv7/mx5/soc.c
> +++ b/arch/arm/cpu/armv7/mx5/soc.c
> @@ -69,6 +69,9 @@ u32 get_cpu_rev(void)
>  	case 0x20:
>  		system_rev |= CHIP_REV_2_0;
>  		break;
> +	case 0x21:
> +		system_rev |= CHIP_REV_2_1;
> +		break;

At the moment it seems to me that the revision number corresponds 1:1 to
the value of the register in the i.MX53, and we do not need some mapping
as for i.MX51. What about to drop the switch and adding a simpler:

	if (reg < 0x20)
		system_rev |= CHIP_REV_1_0;
	else
  		system_rev |= reg;

Best regards,
Stefano

-- 
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: office@denx.de
=====================================================================