[U-Boot] [PATCH 1/2] configs: mx6_common: Restrict CONFIG_ARM_ERRATA_742230 for multi-core

[Stefano Babic]

Hi Fabio,

On 08/01/2014 13:10, Shawn Guo wrote:
> On Wed, Jan 08, 2014 at 09:55:10AM +0100, Stefano Babic wrote:
>> Hi Fabio,
>>
>> On 06/01/2014 14:14, Fabio Estevam wrote:
>>> According to e9fd66defd (ARM: mx6: define CONFIG_ARM_ERRATA_742230), the 
>>> CONFIG_ARM_ERRATA_742230 option should only be applied to multi-core
>>> variants, so restrict its usage for quad and dual-lite only.
>>>
>>> Signed-off-by: Fabio Estevam <fabio.estevam at freescale.com>
>>> ---
>>>  include/configs/mx6_common.h | 2 ++
>>>  1 file changed, 2 insertions(+)
>>>
>>> diff --git a/include/configs/mx6_common.h b/include/configs/mx6_common.h
>>> index 514d634..0b8db85 100644
>>> --- a/include/configs/mx6_common.h
>>> +++ b/include/configs/mx6_common.h
>>> @@ -17,7 +17,9 @@
>>>  #ifndef __MX6_COMMON_H
>>>  #define __MX6_COMMON_H
>>>  
>>> +#if defined(CONFIG_MX6Q) ||  defined(CONFIG_MX6DL)
>>>  #define CONFIG_ARM_ERRATA_742230
>>> +#endif
>>>  #define CONFIG_ARM_ERRATA_743622
>>>  #define CONFIG_ARM_ERRATA_751472
>>>  #define CONFIG_BOARD_POSTCLK_INIT
>>>
>>
>> In Kernel I cannot read that the errata is related to a multicore
>> processor. Indeed, it is related to the Cortex-A9 version.
>>
>> "This option enables the workaround for the 742230 Cortex-A9
>>  (r1p0..r2p2) erratum. Under rare circumstances, a DMB instruction
>>   between two write operations may not ensure the correct visibility
>>   ordering of the two writes. This workaround sets a specific bit in
>>   the diagnostic register of the Cortex-A9 which causes the DMB
>>   instruction to behave as a DSB, ensuring the correct behaviour of
>>   the two writes.
>> "
>>
>> The fix was merged in kernel since a lot of time, before having
>> multicore processors for ARM. At least the commit message does not seem
>> coherent with the explanations in kernel. Maybe has solo a upgraded core
>> version as dual/quad ?
> 
> The sololite has the same core version as dual/quad - r2p10.  The
> help text of erratum 742230 in kernel suggests that only version
> r1p0..r2p2 are affected.  So it sounds like the erratum 742230 should be
> irrelevant to i.MX6 SoCs.  However we were running into a reboot issue
> on multi-core i.MX6 SoCs.  There was a quite long discussion [1] about
> it.  Though we did not reach a conclusion in the thread, one ARM people
> sent me a private message, suggesting this should be an ARM core issue
> and workaround for erratum 742230 might help.  And it turns out what he
> said is true.  And that's why I came up with the commit e9fd66defd (ARM:
> mx6: define CONFIG_ARM_ERRATA_742230) to turn on the erratum for imx6
> dual/quad.

Shawn, thanks for explanations. Fabio, can you extend the commit
message, pointing out that the version itself of the ARM core is not the
real cause. Maybe improving with part of Russel's answer to the thread,
that catches the point.

http://lists.infradead.org/pipermail/linux-arm-kernel/2012-August/113353.html

This is a fix that I will merge for release.
I am quite astonished what can still happens in User Space, if the
compiler issues a dmb instruction...

Best regards,
Stefano Babic

-- 
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================