[PATCH 1/3] [ARM]: U8500 core machine support

Surinder P Singh srplsnh at gmail.com
Thu Sep 24 10:56:56 EDT 2009


On Thu, Sep 24, 2009 at 8:13 PM, Surinder P Singh <srplsnh at gmail.com> wrote:
> Some review comments:
>
>> diff --git a/arch/arm/mach-ux500/include/mach/debug-macro.S b/arch/arm/mach-ux500/include/mach/debug-macro.S
>> +
>> +#include <linux/amba/serial.h>
>> +               .macro  addruart,rx
>> +               mrc     p15, 0, \rx, c1, c0
>> +               tst     \rx, #1                 @MMU enabled?
>> +               moveq   \rx, #0x80007000        @MMU off, Physical address
>> +               movne   \rx, #0xF0007000        @MMU on, Virtual address
>> +               .endm
>> +
>
> It looks like the constants used here will break compilation. You
> could instead use 0x80000000 and 0xf0000000 and add an 'orr \rx, \rx,
> #0x7000' after the 'movne' to fix this.
>

You can also remove serial.h from here since this is now included in
debug-pl01x.S



More information about the linux-arm-kernel mailing list