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

[Christophe LEROY]

Oops, I copied wrong alternative of the ERRATA. Correct one this time.

Le 06/07/2017 à 13:12, Christophe LEROY a écrit :
>
>
> Le 06/07/2017 à 12:56, Wolfgang Denk a écrit :
>> Dear Christophe,
>>
>> In message 
>> <466a431b5430548a018c21222080ed4040596147.1499329461.git.christophe.leroy at c-s.fr> 
>> you wrote:
>>> Signed-off-by: Christophe Leroy <christophe.leroy at c-s.fr>
>>> ---
>>>   arch/powerpc/cpu/mpc8xx/cpu_init.c | 8 ++++++++
>>>   1 file changed, 8 insertions(+)
>>>
>>> diff --git a/arch/powerpc/cpu/mpc8xx/cpu_init.c 
>>> b/arch/powerpc/cpu/mpc8xx/cpu_init.c
>>> index cf1280983a..52406e8483 100644
>>> --- a/arch/powerpc/cpu/mpc8xx/cpu_init.c
>>> +++ b/arch/powerpc/cpu/mpc8xx/cpu_init.c
>>> @@ -51,6 +51,14 @@ void cpu_init_f(immap_t __iomem *immr)
>>>       clrsetbits_be32(&immr->im_clkrst.car_sccr, ~SCCR_MASK,
>>>               CONFIG_SYS_SCCR);
>>>   +    /* BUG MPC866 GLL2 consideration */
>>> +    reg = in_be32(&immr->im_clkrst.car_sccr);
>>> +    /* probably we use the mode 1:2:1 */
>>> +    if ((reg & 0x00060000) == 0x00020000) {
>>> +        clrbits_be32(&immr->im_clkrst.car_sccr, 0x00060000);
>>> +        setbits_be32(&immr->im_clkrst.car_sccr, 0x00020000);
>>> +    }
>>
>> Like a few lines above, you could/should use a single call to
>> clrsetbits_be32() here.  And as Heiko already commented, please use
>> readable names istead of the magic numbers.
>
> I shall not use clrsetbits_be32(), because the ERRATA says:

The ERRATA says:
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

Christophe

>
> Program the PLPRCR such that the PLL clock will change, then reprogram 
> the PLPRCR value back to the desired value
>
> Christophe
>
>>
>> Reviewed-by: Wolfgang Denk <wd at denx.de>
>>
>>
>> Best regards,
>>
>> Wolfgang Denk
>>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> https://lists.denx.de/listinfo/u-boot