[U-Boot] [PATCH v2b 04/10] powerpc, 8xx: Implement GLL2 ERRATA

[U-Boot] [PATCH v2b 04/10] powerpc, 8xx: Implement GLL2 ERRATA

[Christophe Leroy]

Signed-off-by: Christophe Leroy <christophe.leroy@c-s.fr>
---
 Takes into account comments received from Wolfgang and Heiko.
 Superseeds the one included in v2 of the serie 'powerpc, 8xx:
 Modernise the 8xx'. Not resending the entire serie.

 arch/powerpc/cpu/mpc8xx/cpu_init.c | 20 ++++++++++++++++++++
 1 file changed, 20 insertions(+)

diff --git a/arch/powerpc/cpu/mpc8xx/cpu_init.c b/arch/powerpc/cpu/mpc8xx/cpu_init.c
index cf1280983a..a51596a583 100644
--- a/arch/powerpc/cpu/mpc8xx/cpu_init.c
+++ b/arch/powerpc/cpu/mpc8xx/cpu_init.c
@@ -51,6 +51,26 @@ void cpu_init_f(immap_t __iomem *immr)
 	clrsetbits_be32(&immr->im_clkrst.car_sccr, ~SCCR_MASK,
 			CONFIG_SYS_SCCR);
 
+	/*
+	 * MPC866/885 ERRATA GLL2
+	 * Description:
+	 *   In 1:2:1 mode, when HRESET is detected at the positive edge of
+	 *   EXTCLK, then there will be a loss of phase between
+	 *   EXTCLK and CLKOUT.
+	 *
+	 * Workaround:
+	 *   Reprogram the SCCR:
+	 *   1.   Write 1'b00 to SCCR[EBDF].
+	 *   2.   Write 1'b01 to SCCR[EBDF].
+	 *   3.   Rewrite the desired value to the PLPRCR register.
+	 */
+	reg = in_be32(&immr->im_clkrst.car_sccr);
+	/* Are we in mode 1:2:1 ? */
+	if ((reg & SCCR_EBDF11) == SCCR_EBDF01) {
+		clrbits_be32(&immr->im_clkrst.car_sccr, SCCR_EBDF11);
+		setbits_be32(&immr->im_clkrst.car_sccr, SCCR_EBDF01);
+	}
+
 	/* PLL (CPU clock) settings (15-30) */
 
 	out_be32(&immr->im_clkrstk.cark_plprcrk, KAPWR_KEY);
-- 
2.12.0

[U-Boot] [PATCH v2b 04/10] powerpc, 8xx: Implement GLL2 ERRATA

[Wolfgang Denk]

Dear Christophe,

In message <20170706144956.31FF769745@pc13941vm.idsi0.si.c-s.fr> you wrote:
> Signed-off-by: Christophe Leroy <christophe.leroy@c-s.fr>
> ---
>  Takes into account comments received from Wolfgang and Heiko.
>  Superseeds the one included in v2 of the serie 'powerpc, 8xx:
>  Modernise the 8xx'. Not resending the entire serie.
> 
>  arch/powerpc/cpu/mpc8xx/cpu_init.c | 20 ++++++++++++++++++++
>  1 file changed, 20 insertions(+)

Thanks.

Acked-by: Wolfgang Denk <wd@denx.de>


Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
I'd rather be led to hell than managed to heaven.

[U-Boot] [PATCH v2b 04/10] powerpc, 8xx: Implement GLL2 ERRATA

[Heiko Schocher]

Hello Christophe,

Am 06.07.2017 um 16:49 schrieb Christophe Leroy:
> Signed-off-by: Christophe Leroy <christophe.leroy@c-s.fr>
> ---
>   Takes into account comments received from Wolfgang and Heiko.
>   Superseeds the one included in v2 of the serie 'powerpc, 8xx:
>   Modernise the 8xx'. Not resending the entire serie.
>
>   arch/powerpc/cpu/mpc8xx/cpu_init.c | 20 ++++++++++++++++++++
>   1 file changed, 20 insertions(+)

Thanks!

Reviewed-by: Heiko Schocher <hs@denx.de>

bye,
Heiko
-- 
-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany

[U-Boot] [U-Boot, v2b, 04/10] powerpc, 8xx: Implement GLL2 ERRATA

[Tom Rini]

On Thu, Jul 06, 2017 at 04:49:56PM +0200, Christophe Leroy wrote:

> Signed-off-by: Christophe Leroy <christophe.leroy@c-s.fr>
> Acked-by: Wolfgang Denk <wd@denx.de>
> Reviewed-by: Heiko Schocher <hs@denx.de>

Applied to u-boot/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170708/00b5dbdf/attachment.sig>