[U-Boot] [PATCH 2/4] arm: mxs: fix comments in arch_cpu_init to match the code

2020-01-04 Thread sbabic
> 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 
Applied to u-boot-imx, -next, thanks !

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: sba...@denx.de
=


[U-Boot] [PATCH 2/4] arm: mxs: fix comments in arch_cpu_init to match the code

2019-09-12 Thread Rasmus Villemoes
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 
---
 arch/arm/cpu/arm926ejs/mxs/mxs.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/cpu/arm926ejs/mxs/mxs.c b/arch/arm/cpu/arm926ejs/mxs/mxs.c
index 85c65dcb44..585c53baf6 100644
--- a/arch/arm/cpu/arm926ejs/mxs/mxs.c
+++ b/arch/arm/cpu/arm926ejs/mxs/mxs.c
@@ -98,11 +98,11 @@ int arch_cpu_init(void)
/*
 * Enable NAND clock
 */
-   /* Clear bypass bit */
+   /* Set bypass bit */
writel(CLKCTRL_CLKSEQ_BYPASS_GPMI,
_regs->hw_clkctrl_clkseq_set);
 
-   /* Set GPMI clock to ref_gpmi / 12 */
+   /* Set GPMI clock to ref_xtal / 1 */
clrsetbits_le32(_regs->hw_clkctrl_gpmi,
CLKCTRL_GPMI_CLKGATE | CLKCTRL_GPMI_DIV_MASK, 1);
 
-- 
2.20.1

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot