Messages in this thread |  | | | Date | Tue, 15 Jun 2004 21:14:18 +0200 | | From | Sam Ravnborg <> | | Subject | Re: [PATCH 0/5] kbuild |
| |
On Tue, Jun 15, 2004 at 07:09:51PM +0100, Russell King wrote: > > > > Compared to the original behaviour where the all: target picked the default > > target for a given architecture, this patch adds the following: > > This isn't the case on ARM. I've always told people 'make zImage' > or 'make Image'. I've never told people to use just 'make' on its > own - in fact, I've never used 'make' on its own with the kernel.
Why not? Letting the build system select a default target is often a better choice than some random choice by a developer.
> > > - One has to select the default kernel image only once > > when configuring the kernel. > > - There exist a possibility to add more than half a line of text > > describing individual targets. All relevant information can be > > specified in the help section in the Kconfig file > > You can't fit details for 500 platforms into half a line of text. Not discussing different platforms, only discussing kernel targets. For arm I see the following: zImage, Image bootpImage uImage And some test targets: zImg, Img, bp, i, zi
Not counting the test targets it is only 4 target of which 3 is documented in help today.
> > > If we remove the current support for for example uboot we create an > > additional step in between the make and copy image. > > uboot support on ARM was only recently added, and only happened > because I happened to misread the patch. Had I been more on the > ball, the support would NOT have been merged. However, as it did > get merged, I didn't want to create extra noise by taking it out. > > Please don't take this as acceptance that throwing the uboot crap > into the kernel for ARM was something I found agreeable. I still > find it distasteful that boot loaders have to define their own > image formats and the kernel has to conform to the boot loader > authors whims.
Maybe Wolgang can jump in here - I do not know why mkimage is needed. But I do like to have it present for convinience. It is btw called mkuboot.sh in scripts/ to better say what it does.
Sam - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
|  |