Re: [revised v2 PATCH 5/5] sparc,sparc64: unify boot/

[Sam Ravnborg <sam@ravnborg.org>]

On Wed, Dec 03, 2008 at 12:46:30PM -0800, David Miller wrote:
> From: Sam Ravnborg <sam@ravnborg.org>
> Date: Wed, 3 Dec 2008 21:00:44 +0100
> 
> BTW:
> 
> > +ifeq ($(CONFIG_SPARC64),y)
> > +quiet_cmd_piggy     = PIGGY   $@
> > +      cmd_piggy     = $(obj)/piggyback_64 $@ System.map $(ROOT_IMG)
> > +quiet_cmd_strip     = STRIP   $@
> > +      cmd_strip     = $(STRIP) -R .comment -R .note -K sun4u_init -K _end -K _start vmlinux -o $@
> > +
> > +
> > +# Actual linking
> > +$(obj)/image: vmlinux FORCE
> > +        $(call if_changed,strip)
> > +        @echo '  kernel: $@ is ready'
> > +
> > +$(obj)/tftpboot.img: vmlinux $(obj)/piggyback_64 System.map $(ROOT_IMG) FORCE
> > +        $(call if_changed,elftoaout)
> > +        $(call if_changed,piggy)
> > +        @echo '  kernel: $@ is ready'
> > +
> > +$(obj)/vmlinux.aout: vmlinux FORCE
> > +        $(call if_changed,elftoaout)
> > +        @echo '  kernel: $@ is ready'
> > +endif
> 
> This is one of the things I had to fix up when I applied these
> patches.  Those linking rules need to have initial tabs not spaces
> before the commands to be run, or else the build breaks.
> 
> I guess you didn't do any "make image" testing, or did you? :-)
Right - I just did "make".

> 
> But don't worry I fixed that up in my tree when I applied
> these patches and I dealt with the conflict too already.
Thanks!

	Sam