[PATCH] ARM: imx: armadillo5x0: Fix illegal register access

[PATCH] ARM: imx: armadillo5x0: Fix illegal register access

[Fabio Estevam]

From: Fabio Estevam <fabio.estevam@freescale.com>

commit 0c8bad6a(Armadillo 500 add NAND flash device support (resend).)
introduced an incorrect access to MXC_CCM_RCSR.

MXC_CCM_RCSR is an offset and in order to properly access this register, 
the mx3_ccm_base needs to be passed, otherwise an illegal access is done.

Reported-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
 arch/arm/mach-imx/mach-armadillo5x0.c |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/arch/arm/mach-imx/mach-armadillo5x0.c b/arch/arm/mach-imx/mach-armadillo5x0.c
index 2c6ab32..d22e8cc 100644
--- a/arch/arm/mach-imx/mach-armadillo5x0.c
+++ b/arch/arm/mach-imx/mach-armadillo5x0.c
@@ -526,7 +526,8 @@ static void __init armadillo5x0_init(void)
 	imx31_add_mxc_nand(&armadillo5x0_nand_board_info);
 
 	/* set NAND page size to 2k if not configured via boot mode pins */
-	__raw_writel(__raw_readl(MXC_CCM_RCSR) | (1 << 30), MXC_CCM_RCSR);
+	__raw_writel(__raw_readl(mx3_ccm_base + MXC_CCM_RCSR) |
+						(1 << 30), MXC_CCM_RCSR);
 
 	/* RTC */
 	/* Get RTC IRQ and register the chip */
-- 
1.7.9.5

[PATCH] ARM: imx: armadillo5x0: Fix illegal register access

[Arnd Bergmann]

On Saturday 15 September 2012, Fabio Estevam wrote:
> 
>         /* set NAND page size to 2k if not configured via boot mode pins */
> -       __raw_writel(__raw_readl(MXC_CCM_RCSR) | (1 << 30), MXC_CCM_RCSR);
> +       __raw_writel(__raw_readl(mx3_ccm_base + MXC_CCM_RCSR) |
> +                                               (1 << 30), MXC_CCM_RCSR);
>  
>         /* RTC */

Shouldn't you change both the read and the write here, rather than just the read?

	Arnd