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

[Christophe LEROY]

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:

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
>