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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



From: Sam Ravnborg <sam@xxxxxxxxxxxx>
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? :-)

But don't worry I fixed that up in my tree when I applied
these patches and I dealt with the conflict too already.
--
To unsubscribe from this list: send the line "unsubscribe sparclinux" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Kernel Development]     [DCCP]     [Linux ARM Development]     [Linux]     [Photo]     [Yosemite Help]     [Linux ARM Kernel]     [Linux SCSI]     [Linux x86_64]     [Linux Hams]

  Powered by Linux