Re: [GIT PULL] unify sparc header files

[Sam Ravnborg <sam@ravnborg.org>]

On Thu, Jul 03, 2008 at 11:02:07PM +0300, Adrian Bunk wrote:
> On Thu, Jul 03, 2008 at 01:12:53AM -0700, David Miller wrote:
> > From: David Miller <davem@davemloft.net>
> > Date: Thu, 03 Jul 2008 00:50:52 -0700 (PDT)
> > 
> > > From: Sam Ravnborg <sam@ravnborg.org>
> > > Date: Sun, 29 Jun 2008 00:02:35 +0200
> > > 
> > > > The patches can be pulled here:
> > > > 
> > > >    ssh://master.kernel.org/pub/scm/linux/kernel/git/sam/sparc.git master
> > > > 
> > > > Note that the commit: 878e7a6e498bec55f10a163197c41554fb078416
> > > > ("sparc: remove CONFIG_SUN4") is not included.
> > > > I did not include it as this was independent of my work
> > > > and I did not like to Sign-off something I really had not looked at.
> > > > So please apply this commit by other means.
> > > 
> > > Ok, I've pulled all of this into my sparc-next-2.6 tree.
> > 
> > Unfortunately, this explodes.  I wonder what your build
> > environment was when testing this with on sparc64, because
> > I can't see how this could possibly work.
> > 
> > In file included from include/asm-sparc/page.h:6,
> >                  from include/asm/page.h:1,
> >                  from arch/sparc64/kernel/vmlinux.lds.S:3:
> > include/asm-sparc/page_32.h:24:25: error: asm/btfixup.h: No such file or directory
> > 
> > When we run vmlinux.lds.S through CPP, it undefines things like
> > "sparc" with the '-Usparc -Usparc64' options that get passed in.
> > So the ifdef tests in asm-sparc/page.h do not work.
> >...
> 
> It also builds for me (gcc 4.3.1) since -Usparc is not -U__sparc__
> (if I add the latter I get your build error).
> 
> Did gcc's behavior change in this area?

Thanks for testing Adrian.

	Sam