diff mbox

ARM: mm: pull asm/memory.h explicitly

Message ID 1471450634-45566-1-git-send-email-vladimir.murzin@arm.com (mailing list archive)
State New, archived
Headers show

Commit Message

Vladimir Murzin Aug. 17, 2016, 4:17 p.m. UTC
Commit d78114554939a (""ARM: 8512/1: proc-v7.S: Adjust stack address when
XIP_KERNEL"") introduced a macro which lives under asm/memory.h.
Unfortunately, for MMU-less systems (like R-class) it leads to build failure:

arch/arm/mm/proc-v7.S: Assembler messages:
arch/arm/mm/proc-v7.S:538: Error: unrecognised relocation suffix
make[1]: *** [arch/arm/mm/proc-v7.o] Error 1
make: *** [arch/arm/mm] Error 2

since it is implicitly pulled via asm/pgtable.h for MMU capable systems only.

To fix it include asm/memory.h explicitly.

Signed-off-by: Vladimir Murzin <vladimir.murzin@arm.com>
---
 arch/arm/mm/proc-v7.S |    1 +
 1 file changed, 1 insertion(+)

Comments

Russell King (Oracle) Aug. 18, 2016, 2:22 p.m. UTC | #1
On Wed, Aug 17, 2016 at 05:17:14PM +0100, Vladimir Murzin wrote:
> Commit d78114554939a (""ARM: 8512/1: proc-v7.S: Adjust stack address when
> XIP_KERNEL"") introduced a macro which lives under asm/memory.h.
> Unfortunately, for MMU-less systems (like R-class) it leads to build failure:
> 
> arch/arm/mm/proc-v7.S: Assembler messages:
> arch/arm/mm/proc-v7.S:538: Error: unrecognised relocation suffix
> make[1]: *** [arch/arm/mm/proc-v7.o] Error 1
> make: *** [arch/arm/mm] Error 2
> 
> since it is implicitly pulled via asm/pgtable.h for MMU capable systems only.
> 
> To fix it include asm/memory.h explicitly.
> 
> Signed-off-by: Vladimir Murzin <vladimir.murzin@arm.com>

Looks fine, thanks.
Vladimir Murzin Aug. 18, 2016, 2:32 p.m. UTC | #2
On 18/08/16 15:22, Russell King - ARM Linux wrote:
> On Wed, Aug 17, 2016 at 05:17:14PM +0100, Vladimir Murzin wrote:
>> Commit d78114554939a (""ARM: 8512/1: proc-v7.S: Adjust stack address when
>> XIP_KERNEL"") introduced a macro which lives under asm/memory.h.
>> Unfortunately, for MMU-less systems (like R-class) it leads to build failure:
>>
>> arch/arm/mm/proc-v7.S: Assembler messages:
>> arch/arm/mm/proc-v7.S:538: Error: unrecognised relocation suffix
>> make[1]: *** [arch/arm/mm/proc-v7.o] Error 1
>> make: *** [arch/arm/mm] Error 2
>>
>> since it is implicitly pulled via asm/pgtable.h for MMU capable systems only.
>>
>> To fix it include asm/memory.h explicitly.
>>
>> Signed-off-by: Vladimir Murzin <vladimir.murzin@arm.com>
> 
> Looks fine, thanks.
> 

Should I drop it into patch system?

Cheers
Vladimir
Russell King (Oracle) Aug. 18, 2016, 3:11 p.m. UTC | #3
On Thu, Aug 18, 2016 at 03:32:29PM +0100, Vladimir Murzin wrote:
> On 18/08/16 15:22, Russell King - ARM Linux wrote:
> > On Wed, Aug 17, 2016 at 05:17:14PM +0100, Vladimir Murzin wrote:
> >> Commit d78114554939a (""ARM: 8512/1: proc-v7.S: Adjust stack address when
> >> XIP_KERNEL"") introduced a macro which lives under asm/memory.h.
> >> Unfortunately, for MMU-less systems (like R-class) it leads to build failure:
> >>
> >> arch/arm/mm/proc-v7.S: Assembler messages:
> >> arch/arm/mm/proc-v7.S:538: Error: unrecognised relocation suffix
> >> make[1]: *** [arch/arm/mm/proc-v7.o] Error 1
> >> make: *** [arch/arm/mm] Error 2
> >>
> >> since it is implicitly pulled via asm/pgtable.h for MMU capable systems only.
> >>
> >> To fix it include asm/memory.h explicitly.
> >>
> >> Signed-off-by: Vladimir Murzin <vladimir.murzin@arm.com>
> > 
> > Looks fine, thanks.
> > 
> 
> Should I drop it into patch system?

Yes please.
diff mbox

Patch

diff --git a/arch/arm/mm/proc-v7.S b/arch/arm/mm/proc-v7.S
index d0b116c..8cb105b 100644
--- a/arch/arm/mm/proc-v7.S
+++ b/arch/arm/mm/proc-v7.S
@@ -16,6 +16,7 @@ 
 #include <asm/hwcap.h>
 #include <asm/pgtable-hwdef.h>
 #include <asm/pgtable.h>
+#include <asm/memory.h>
 
 #include "proc-macros.S"