Re: [PATCH 12/16] i.MX6: Apply errata 845369 workaround

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Mar 17, 2016 at 12:31 AM, Stefan Christ <s.christ@xxxxxxxxx> wrote:
> Hi,
>
>> @@ -31,4 +31,5 @@ void imx6_cpu_lowlevel_init(void)
>>       enable_arm_errata_751472_war();
>>       enable_arm_errata_761320_war();
>>       enable_arm_errata_794072_war();
>> +     enable_arm_errata_845369_war();
>
> The errata 845369 seems to be a general errata for all ARM Cortex-A9 cores. Why
> is this fix only apply for the i.MX6 SoC and not for other Cortex-A9 devices in
> the barebox?

AFAIR, there's a bit of a caveat to that errata, since it applies only
to multi-core systems or systems with ACP, so some of the cores might
not need it and IMHO in order to properly apply that errata one would
have to consult individual SoC's reference manuals.

But the main reason why I applied it only there was that there didn't
seem to be a common place where those erratas are applied for all
platforms (I suspect it might have to do with the fact that it is
possible to build BB for multiple generations or ARM), but there was a
place where erratas were applied for i.MX6, so I followed the path of
least resistance.


>
> Mit freundlichen Grüßen / Kind regards,
>         Stefan Christ
>
> On Tue, Mar 15, 2016 at 08:33:47PM -0700, Andrey Smirnov wrote:
>> Signed-off-by: Andrey Smirnov <andrew.smirnov@xxxxxxxxx>
>> ---
>>  arch/arm/mach-imx/cpu_init.c | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/arch/arm/mach-imx/cpu_init.c b/arch/arm/mach-imx/cpu_init.c
>> index 8b10e63..7603883 100644
>> --- a/arch/arm/mach-imx/cpu_init.c
>> +++ b/arch/arm/mach-imx/cpu_init.c
>> @@ -31,4 +31,5 @@ void imx6_cpu_lowlevel_init(void)
>>       enable_arm_errata_751472_war();
>>       enable_arm_errata_761320_war();
>>       enable_arm_errata_794072_war();
>> +     enable_arm_errata_845369_war();
>>  }
>> --
>> 2.5.0
>>
>>
>> _______________________________________________
>> barebox mailing list
>> barebox@xxxxxxxxxxxxxxxxxxx
>> http://lists.infradead.org/mailman/listinfo/barebox

_______________________________________________
barebox mailing list
barebox@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/barebox




[Index of Archives]     [Linux Embedded]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux