The Linux Kernel Mailing List
 help / color / mirror / Atom feed
* sh O= builds broken
@ 2008-08-01 21:27 Adrian Bunk
  2008-08-01 21:53 ` Paul Mundt
  0 siblings, 1 reply; 4+ messages in thread
From: Adrian Bunk @ 2008-08-01 21:27 UTC (permalink / raw)
  To: sam, lethal; +Cc: linux-sh, linux-kernel

Most likely caused by the headers move:

<--  snip  -->

...
  CC      init/main.o
In file included from /home/bunk/linux/kernel-2.6/git/linux-2.6/arch/sh/include/asm/irq.h:4,
                 from /home/bunk/linux/kernel-2.6/git/linux-2.6/include/linux/irq.h:23,
                 from /home/bunk/linux/kernel-2.6/git/linux-2.6/arch/sh/include/asm/hardirq.h:5,
                 from /home/bunk/linux/kernel-2.6/git/linux-2.6/include/linux/hardirq.h:7,
                 from /home/bunk/linux/kernel-2.6/git/linux-2.6/include/asm-generic/local.h:5,
                 from /home/bunk/linux/kernel-2.6/git/linux-2.6/arch/sh/include/asm/local.h:4,
                 from /home/bunk/linux/kernel-2.6/git/linux-2.6/include/linux/module.h:19,
                 from /home/bunk/linux/kernel-2.6/git/linux-2.6/init/main.c:13:
/home/bunk/linux/kernel-2.6/git/linux-2.6/arch/sh/include/asm/machvec.h:15:27: 
error: asm/machtypes.h: No such file or directory
make[2]: *** [init/main.o] Error 1

<--  snip  -->

cu
Adrian

-- 

       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: sh O= builds broken
  2008-08-01 21:27 sh O= builds broken Adrian Bunk
@ 2008-08-01 21:53 ` Paul Mundt
  2008-08-01 22:03   ` Adrian Bunk
  2008-08-01 22:12   ` Sam Ravnborg
  0 siblings, 2 replies; 4+ messages in thread
From: Paul Mundt @ 2008-08-01 21:53 UTC (permalink / raw)
  To: Adrian Bunk; +Cc: sam, linux-sh, linux-kernel

On Sat, Aug 02, 2008 at 12:27:28AM +0300, Adrian Bunk wrote:
> Most likely caused by the headers move:
> 
> <--  snip  -->
> 
> ...
>   CC      init/main.o
> In file included from /home/bunk/linux/kernel-2.6/git/linux-2.6/arch/sh/include/asm/irq.h:4,
>                  from /home/bunk/linux/kernel-2.6/git/linux-2.6/include/linux/irq.h:23,
>                  from /home/bunk/linux/kernel-2.6/git/linux-2.6/arch/sh/include/asm/hardirq.h:5,
>                  from /home/bunk/linux/kernel-2.6/git/linux-2.6/include/linux/hardirq.h:7,
>                  from /home/bunk/linux/kernel-2.6/git/linux-2.6/include/asm-generic/local.h:5,
>                  from /home/bunk/linux/kernel-2.6/git/linux-2.6/arch/sh/include/asm/local.h:4,
>                  from /home/bunk/linux/kernel-2.6/git/linux-2.6/include/linux/module.h:19,
>                  from /home/bunk/linux/kernel-2.6/git/linux-2.6/init/main.c:13:
> /home/bunk/linux/kernel-2.6/git/linux-2.6/arch/sh/include/asm/machvec.h:15:27: 
> error: asm/machtypes.h: No such file or directory
> make[2]: *** [init/main.o] Error 1
> 
> <--  snip  -->
> 
Ok, so this is what I get for moving the auto-generated header outside of
include/asm-sh.. it seemed like a good idea at the time.

---

diff --git a/arch/sh/Makefile b/arch/sh/Makefile
index 25659ce..7b70cfd 100644
--- a/arch/sh/Makefile
+++ b/arch/sh/Makefile
@@ -164,7 +164,7 @@ KBUILD_AFLAGS		+= $(cflags-y)
 PHONY += maketools FORCE
 
 maketools:  include/linux/version.h FORCE
-	$(Q)$(MAKE) $(build)=arch/sh/tools arch/sh/include/asm/machtypes.h
+	$(Q)$(MAKE) $(build)=arch/sh/tools include/asm-sh/machtypes.h
 
 all: $(KBUILD_IMAGE)
 
@@ -215,4 +215,4 @@ arch/sh/lib64/syscalltab.h: arch/sh/kernel/syscalls_64.S
 	$(call filechk,gen-syscalltab)
 
 CLEAN_FILES += arch/sh/lib64/syscalltab.h \
-	       arch/sh/include/asm/machtypes.h
+	       include/asm-sh/machtypes.h
diff --git a/arch/sh/tools/Makefile b/arch/sh/tools/Makefile
index b5d202b..567516b 100644
--- a/arch/sh/tools/Makefile
+++ b/arch/sh/tools/Makefile
@@ -10,7 +10,7 @@
 # Shamelessly cloned from ARM.
 #
 
-arch/sh/include/asm/machtypes.h: $(src)/gen-mach-types $(src)/mach-types
+include/asm-sh/machtypes.h: $(src)/gen-mach-types $(src)/mach-types
 	@echo '  Generating $@'
-	$(Q)if [ ! -d arch/sh/include/asm ]; then mkdir -p arch/sh/include/asm; fi
+	$(Q)if [ ! -d include/asm-sh ]; then mkdir -p include/asm-sh; fi
 	$(Q)$(AWK) -f $^ > $@ || { rm -f $@; /bin/false; }

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: sh O= builds broken
  2008-08-01 21:53 ` Paul Mundt
@ 2008-08-01 22:03   ` Adrian Bunk
  2008-08-01 22:12   ` Sam Ravnborg
  1 sibling, 0 replies; 4+ messages in thread
From: Adrian Bunk @ 2008-08-01 22:03 UTC (permalink / raw)
  To: Paul Mundt, Adrian Bunk, sam, linux-sh, linux-kernel

On Sat, Aug 02, 2008 at 06:53:38AM +0900, Paul Mundt wrote:
> On Sat, Aug 02, 2008 at 12:27:28AM +0300, Adrian Bunk wrote:
> > Most likely caused by the headers move:
> > 
> > <--  snip  -->
> > 
> > ...
> >   CC      init/main.o
> > In file included from /home/bunk/linux/kernel-2.6/git/linux-2.6/arch/sh/include/asm/irq.h:4,
> >                  from /home/bunk/linux/kernel-2.6/git/linux-2.6/include/linux/irq.h:23,
> >                  from /home/bunk/linux/kernel-2.6/git/linux-2.6/arch/sh/include/asm/hardirq.h:5,
> >                  from /home/bunk/linux/kernel-2.6/git/linux-2.6/include/linux/hardirq.h:7,
> >                  from /home/bunk/linux/kernel-2.6/git/linux-2.6/include/asm-generic/local.h:5,
> >                  from /home/bunk/linux/kernel-2.6/git/linux-2.6/arch/sh/include/asm/local.h:4,
> >                  from /home/bunk/linux/kernel-2.6/git/linux-2.6/include/linux/module.h:19,
> >                  from /home/bunk/linux/kernel-2.6/git/linux-2.6/init/main.c:13:
> > /home/bunk/linux/kernel-2.6/git/linux-2.6/arch/sh/include/asm/machvec.h:15:27: 
> > error: asm/machtypes.h: No such file or directory
> > make[2]: *** [init/main.o] Error 1
> > 
> > <--  snip  -->
> > 
> Ok, so this is what I get for moving the auto-generated header outside of
> include/asm-sh.. it seemed like a good idea at the time.
> 
> ---
> 
> diff --git a/arch/sh/Makefile b/arch/sh/Makefile
> index 25659ce..7b70cfd 100644
> --- a/arch/sh/Makefile
> +++ b/arch/sh/Makefile
> @@ -164,7 +164,7 @@ KBUILD_AFLAGS		+= $(cflags-y)
>  PHONY += maketools FORCE
>  
>  maketools:  include/linux/version.h FORCE
> -	$(Q)$(MAKE) $(build)=arch/sh/tools arch/sh/include/asm/machtypes.h
> +	$(Q)$(MAKE) $(build)=arch/sh/tools include/asm-sh/machtypes.h
>  
>  all: $(KBUILD_IMAGE)
>  
> @@ -215,4 +215,4 @@ arch/sh/lib64/syscalltab.h: arch/sh/kernel/syscalls_64.S
>  	$(call filechk,gen-syscalltab)
>  
>  CLEAN_FILES += arch/sh/lib64/syscalltab.h \
> -	       arch/sh/include/asm/machtypes.h
> +	       include/asm-sh/machtypes.h
> diff --git a/arch/sh/tools/Makefile b/arch/sh/tools/Makefile
> index b5d202b..567516b 100644
> --- a/arch/sh/tools/Makefile
> +++ b/arch/sh/tools/Makefile
> @@ -10,7 +10,7 @@
>  # Shamelessly cloned from ARM.
>  #
>  
> -arch/sh/include/asm/machtypes.h: $(src)/gen-mach-types $(src)/mach-types
> +include/asm-sh/machtypes.h: $(src)/gen-mach-types $(src)/mach-types
>  	@echo '  Generating $@'
> -	$(Q)if [ ! -d arch/sh/include/asm ]; then mkdir -p arch/sh/include/asm; fi
> +	$(Q)if [ ! -d include/asm-sh ]; then mkdir -p include/asm-sh; fi
>  	$(Q)$(AWK) -f $^ > $@ || { rm -f $@; /bin/false; }


Thanks for the quick fix, it's working fine.


cu
Adrian

-- 

       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: sh O= builds broken
  2008-08-01 21:53 ` Paul Mundt
  2008-08-01 22:03   ` Adrian Bunk
@ 2008-08-01 22:12   ` Sam Ravnborg
  1 sibling, 0 replies; 4+ messages in thread
From: Sam Ravnborg @ 2008-08-01 22:12 UTC (permalink / raw)
  To: Paul Mundt, Adrian Bunk, linux-sh, linux-kernel

On Sat, Aug 02, 2008 at 06:53:38AM +0900, Paul Mundt wrote:
> > 
> Ok, so this is what I get for moving the auto-generated header outside of
> include/asm-sh.. it seemed like a good idea at the time.
Ups - I should have caught that when I reviewd your initial patch.
kbuild does on purpose not fiddle with the path in the inital
include specification.

	Sam

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2008-08-01 22:11 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-08-01 21:27 sh O= builds broken Adrian Bunk
2008-08-01 21:53 ` Paul Mundt
2008-08-01 22:03   ` Adrian Bunk
2008-08-01 22:12   ` Sam Ravnborg

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox