[U-Boot] [PATCH v2] powerpc/t2080: CPU erratum A-007907
york sun
york.sun at nxp.com
Wed Oct 26 18:25:26 CEST 2016
On 10/24/2016 02:16 PM, york.sun at nxp.com wrote:
>> +
>> +#ifdef CONFIG_SYS_FSL_ERRATUM_A007907
>> + flush_dcache();
>> + mtspr(L1CSR2, (mfspr(L1CSR2) & ~L1CSR2_DCSTASHID));
>> + sync();
>> +#endif
>> +
>
> My erratum document shows a second step is to insert sync instruction
> before each dcbtls CT=0 or dcbtstls CT=0. I only see dcbtls used in
> start.S and no referendce to dcbtstls at all. I will consult the design
> team to confirm if the second step.
>
I got feedback from our design team. The second step is a must if these
instructions are used. However, we don't hit either in U-Boot. Please be
advised, if you have other code using dcbtls or dcbtstls with CT=0, you
need to implement the 2nd step, i.e. proceed with a sync instruction.
York
More information about the U-Boot
mailing list