arm: mxs: fix comments in arch_cpu_init to match the code
authorRasmus Villemoes <rasmus.villemoes@prevas.dk>
Thu, 12 Sep 2019 09:17:10 +0000 (09:17 +0000)
committerStefano Babic <sbabic@denx.de>
Tue, 7 Jan 2020 09:26:57 +0000 (10:26 +0100)
The comment says to clear the bypass bit, but in fact it sets it, thus
selecting ref_xtal. And the next line of code does not set the divider
to 12, but to (the reset value of) 1.

Signed-off-by: Rasmus Villemoes <rasmus.villemoes@prevas.dk>
arch/arm/cpu/arm926ejs/mxs/mxs.c

index 85c65dc..585c53b 100644 (file)
@@ -98,11 +98,11 @@ int arch_cpu_init(void)
        /*
         * Enable NAND clock
         */
-       /* Clear bypass bit */
+       /* Set bypass bit */
        writel(CLKCTRL_CLKSEQ_BYPASS_GPMI,
                &clkctrl_regs->hw_clkctrl_clkseq_set);
 
-       /* Set GPMI clock to ref_gpmi / 12 */
+       /* Set GPMI clock to ref_xtal / 1 */
        clrsetbits_le32(&clkctrl_regs->hw_clkctrl_gpmi,
                CLKCTRL_GPMI_CLKGATE | CLKCTRL_GPMI_DIV_MASK, 1);