| Message ID | 1303778789-13423-1-git-send-email-festevam@gmail.com |
|---|---|
| State | Superseded |
| Headers | show |
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
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; default: system_rev |= CHIP_REV_1_0; break; diff --git a/arch/arm/include/asm/arch-mx5/imx-regs.h b/arch/arm/include/asm/arch-mx5/imx-regs.h index a1849f8..2d3d501 100644 --- a/arch/arm/include/asm/arch-mx5/imx-regs.h +++ b/arch/arm/include/asm/arch-mx5/imx-regs.h @@ -175,6 +175,7 @@ #define CHIP_REV_1_0 0x10 #define CHIP_REV_1_1 0x11 #define CHIP_REV_2_0 0x20 +#define CHIP_REV_2_1 0x21 #define CHIP_REV_2_5 0x25 #define CHIP_REV_3_0 0x30
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(-)