[PATCH v2 1/2] ARM PJ4B: Add support for errata 4742

Gregory CLEMENT gregory.clement at free-electrons.com
Tue Jun 4 15:04:09 EDT 2013


On 06/04/2013 08:32 PM, Nicolas Pitre wrote:
> On Tue, 4 Jun 2013, Russell King - ARM Linux wrote:
> 
>> On Tue, Jun 04, 2013 at 02:19:32PM -0400, Nicolas Pitre wrote:
>>> On Tue, 4 Jun 2013, Russell King - ARM Linux wrote:
>>>
>>>> On Tue, Jun 04, 2013 at 02:07:29PM -0400, Nicolas Pitre wrote:
>>>>> Alternatively, the following could be done:
>>>>>
>>>>> #ifdef CONFIG_CPU_PJ4B
>>>>>
>>>>> 	.equ	cpu_pj4b_switch_mm,	cpu_v7_switch_mm
>>>>> 	.equ	cpu_pj4b_set_pte_ex,	cpu_v7_set_pte_ex
>>>>
>>>> No, that doesn't work.  I did forget a hunk from my patch though, which
>>>> needs the pj4b symbols to be global:
>>>
>>> 	.macro globl_equ x, y
>>> 	.globl	x
>>> 	.equ	x, y
>>> 	.endm
>>>
>>> 	.globl_equ	cpu_pj4b_switch_mm,     cpu_v7_switch_mm
>>> 	.globl_equ	cpu_pj4b_set_pte_ex,	cpu_v7_set_pte_ex
>>
>> Which is really no different from doing what I did.
> 
> This can be wrapped inside #ifdef CONFIG_CPU_PJ4B clearing the symbol 
> aliasing confusion I mentioned when not configured, and can be done in a 
> single location instead of being spread all over.  Other than that, this 
> is functionally equivalent to what you did indeed.
> 
> 
Russell, Nicolas,

Thanks for both of you for help. As you both agree that your variants are functionally
equivalent and as the variant of Nicolas seems to be more easy to debug, I will
adopt Nicolas' version. Unless, there was a drawback you didn't mention yet.

Regards,

Gregory



More information about the linux-arm-kernel mailing list