[U-Boot] [PATCH 2/2] arm: cpu: armv7: omap-common: Clear XN bit in the ARMV7_DCACHE_POLICY
Keerthy
j-keerthy at ti.com
Fri Oct 28 09:40:07 CEST 2016
On Friday 28 October 2016 12:56 PM, Alexander Graf wrote:
>
>
> On 28/10/2016 08:31, Keerthy wrote:
>> Clear the XN bit in the ARMV7_DCACHE_POLICY so as to mark
>> the regions as execute okay.
>>
>> Signed-off-by: Keerthy <j-keerthy at ti.com>
>
> How did you ever get the bit set in DCACHE_WRITEALLOC or WRITEBACK in
> the first place? Both are RAM mapping flags which shouldn't have the XN
> mask set.
Just searched through none of them have the bit sit so that is
redundant. I added to be doubly sure and can be removed. This patch is
not needed.
>
>
> Alex
>
>> ---
>> arch/arm/cpu/armv7/omap-common/omap-cache.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/arch/arm/cpu/armv7/omap-common/omap-cache.c b/arch/arm/cpu/armv7/omap-common/omap-cache.c
>> index b37163a..a71aa0d 100644
>> --- a/arch/arm/cpu/armv7/omap-common/omap-cache.c
>> +++ b/arch/arm/cpu/armv7/omap-common/omap-cache.c
>> @@ -34,7 +34,7 @@ DECLARE_GLOBAL_DATA_PTR;
>> */
>>
>> #ifdef CONFIG_ARMV7_LPAE
>> -#define ARMV7_DCACHE_POLICY DCACHE_WRITEALLOC
>> +#define ARMV7_DCACHE_POLICY DCACHE_WRITEALLOC & ~TTB_SECT_XN_MASK
>> #else
>> #define ARMV7_DCACHE_POLICY DCACHE_WRITEBACK & ~TTB_SECT_XN_MASK
>> #endif
>>
More information about the U-Boot
mailing list