* [PATCH][2.6] arch/arm/mm/tlb-v4wb.S needs to lose a header
@ 2003-09-08 4:40 Zwane Mwaikambo
2003-09-08 7:42 ` Russell King
0 siblings, 1 reply; 3+ messages in thread
From: Zwane Mwaikambo @ 2003-09-08 4:40 UTC (permalink / raw)
To: Linux Kernel; +Cc: Russell King
Whilst building for an SA1100 i got the following snippets;
AS arch/arm/mm/tlb-v4wb.o
In file included from include/linux/spinlock.h:13,
from include/linux/capability.h:45,
from include/linux/sched.h:7,
from include/linux/mm.h:4,
from include/asm/tlbflush.h:14,
from arch/arm/mm/tlb-v4wb.S:18:
include/linux/kernel.h:32: warning: `ALIGN' redefined
include/linux/linkage.h:24: warning: this is the location of the previous definition
In file included from include/asm/tlbflush.h:14,
In file included from include/asm/tlbflush.h:14,
from arch/arm/mm/tlb-v4wb.S:18:
include/linux/mm.h:87: warning: `VM_EXEC' redefined
include/asm/constants.h:16: warning: this is the location of the previous
definition
Index: linux-2.6.0-test4-mm6-arm/arch/arm/mm/tlb-v4wb.S
===================================================================
RCS file: /build/cvsroot/linux-2.6.0-test4-mm6/arch/arm/mm/tlb-v4wb.S,v
retrieving revision 1.1.1.1
diff -u -p -B -r1.1.1.1 tlb-v4wb.S
--- linux-2.6.0-test4-mm6-arm/arch/arm/mm/tlb-v4wb.S 7 Sep 2003 20:26:24 -0000 1.1.1.1
+++ linux-2.6.0-test4-mm6-arm/arch/arm/mm/tlb-v4wb.S 8 Sep 2003 03:56:45 -0000
@@ -15,7 +15,6 @@
#include <linux/linkage.h>
#include <linux/init.h>
#include <asm/constants.h>
-#include <asm/tlbflush.h>
#include "proc-macros.S"
.align 5
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH][2.6] arch/arm/mm/tlb-v4wb.S needs to lose a header
2003-09-08 4:40 [PATCH][2.6] arch/arm/mm/tlb-v4wb.S needs to lose a header Zwane Mwaikambo
@ 2003-09-08 7:42 ` Russell King
2003-09-08 7:47 ` Russell King
0 siblings, 1 reply; 3+ messages in thread
From: Russell King @ 2003-09-08 7:42 UTC (permalink / raw)
To: Zwane Mwaikambo; +Cc: Linux Kernel
On Mon, Sep 08, 2003 at 12:40:10AM -0400, Zwane Mwaikambo wrote:
> Whilst building for an SA1100 i got the following snippets;
>
> AS arch/arm/mm/tlb-v4wb.o
> In file included from include/linux/spinlock.h:13,
> from include/linux/capability.h:45,
> from include/linux/sched.h:7,
> from include/linux/mm.h:4,
> from include/asm/tlbflush.h:14,
> from arch/arm/mm/tlb-v4wb.S:18:
> include/linux/kernel.h:32: warning: `ALIGN' redefined
> include/linux/linkage.h:24: warning: this is the location of the previous definition
> In file included from include/asm/tlbflush.h:14,
>
> In file included from include/asm/tlbflush.h:14,
> from arch/arm/mm/tlb-v4wb.S:18:
> include/linux/mm.h:87: warning: `VM_EXEC' redefined
> include/asm/constants.h:16: warning: this is the location of the previous
> definition
This is caused by including linux/mm.h into asm/tlbflush.h, but it does
show that we don't need asm/tlbflush.h included here.
Same change applied to the other tlb-*.S files, thanks.
--
Russell King (rmk@arm.linux.org.uk) http://www.arm.linux.org.uk/personal/
Linux kernel maintainer of:
2.6 ARM Linux - http://www.arm.linux.org.uk/
2.6 PCMCIA - http://pcmcia.arm.linux.org.uk/
2.6 Serial core
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH][2.6] arch/arm/mm/tlb-v4wb.S needs to lose a header
2003-09-08 7:42 ` Russell King
@ 2003-09-08 7:47 ` Russell King
0 siblings, 0 replies; 3+ messages in thread
From: Russell King @ 2003-09-08 7:47 UTC (permalink / raw)
To: Zwane Mwaikambo, Linux Kernel
On Mon, Sep 08, 2003 at 08:42:58AM +0100, Russell King wrote:
> > In file included from include/asm/tlbflush.h:14,
> > from arch/arm/mm/tlb-v4wb.S:18:
> > include/linux/mm.h:87: warning: `VM_EXEC' redefined
> > include/asm/constants.h:16: warning: this is the location of the previous
> > definition
>
> This is caused by including linux/mm.h into asm/tlbflush.h, but it does
> show that we don't need asm/tlbflush.h included here.
Sorry, you do need asm/tlbflush.h here - we use the various tlb flag
definitions from there in the assembly files to avoid duplicating them.
The solution is not to include linux/mm.h in the first place.
--
Russell King (rmk@arm.linux.org.uk) http://www.arm.linux.org.uk/personal/
Linux kernel maintainer of:
2.6 ARM Linux - http://www.arm.linux.org.uk/
2.6 PCMCIA - http://pcmcia.arm.linux.org.uk/
2.6 Serial core
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2003-09-08 7:47 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2003-09-08 4:40 [PATCH][2.6] arch/arm/mm/tlb-v4wb.S needs to lose a header Zwane Mwaikambo
2003-09-08 7:42 ` Russell King
2003-09-08 7:47 ` Russell King
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).