From mboxrd@z Thu Jan 1 00:00:00 1970 From: Guan Xuetao Subject: Re: [microblaze-linux] [PATCH 2/3] Kbuild: Implement CONFIG_UIMAGE_KERNEL_NOLOAD Date: Wed, 7 Mar 2012 14:52:43 +0800 Message-ID: <1331103163.2048.33.camel@epip-laptop> References: <1331080238-1524-1-git-send-email-swarren@wwwdotorg.org> <1331080238-1524-2-git-send-email-swarren@wwwdotorg.org> Reply-To: gxt-TG0Ac1+ktVePQbnJrJN+5g@public.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1331080238-1524-2-git-send-email-swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: microblaze-linux-bounces-FR6EJeJVuqfA6Z3fQjNZrN9u6TNh0Fb7@public.gmane.org To: Stephen Warren Cc: Michal Marek , linux-arch-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Mike Frysinger , nico-vtqb6HGKxmzR7s880joybQ@public.gmane.org, linux-sh-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, microblaze-uclinux-rVRm/Wmeqae7NGdpmJTKYQ@public.gmane.org, linux-kbuild-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Paul Mundt , uclinux-dist-devel-ZG0+EudsQA8dtHy/vicBwGD2FQJk+8+b@public.gmane.org, sparclinux-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Russell King , Haavard Skinnemoen , "David S. Miller" , linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, Hans-Christian Egtvedt List-Id: linux-arch.vger.kernel.org On Tue, 2012-03-06 at 17:30 -0700, Stephen Warren wrote: > This allows the user to use U-Boot's mkimage's -T kernel_noload option > if their arch Kconfig allows it, and they desire. > > Signed-off-by: Stephen Warren > --- > The next patch enables this new CONFIG_ALLOW_ option for ARM. I assume > that some other architectures will also be able to enable it, but I'm > not familiar enough with any to know which. > > init/Kconfig | 15 +++++++++++++++ > scripts/Makefile.lib | 15 ++++++++++++--- > 2 files changed, 27 insertions(+), 3 deletions(-) > > diff --git a/init/Kconfig b/init/Kconfig > index 6cfd71d..2035562 100644 > --- a/init/Kconfig > +++ b/init/Kconfig > @@ -203,6 +203,21 @@ config KERNEL_LZO > > endchoice > > +config ALLOW_UIMAGE_KERNEL_NOLOAD > + bool > + > +config UIMAGE_KERNEL_NOLOAD > + bool "Use mkimage's -T kernel_noload instead of -T kernel" > + depends on ALLOW_UIMAGE_KERNEL_NOLOAD > + help > + uImage files of type "kernel" embed an absolute load and entry point > + address. If the uImage is loaded to a different address, then the > + image must be copied the uImage file to the specified load address. > + This copy is a waste of time if the uImage is in fact completely > + relocatable. If your kernel and bootloader support it, you can build > + a "kernel_noload" uImage rather than a "kernel" uImage, which > + prevents any such copying. If unsure say N. > + > config DEFAULT_HOSTNAME > string "Default hostname" > default "(none)" > diff --git a/scripts/Makefile.lib b/scripts/Makefile.lib > index 7b0be18..d9c1c87 100644 > --- a/scripts/Makefile.lib > +++ b/scripts/Makefile.lib > @@ -314,9 +314,18 @@ MKIMAGE := $(srctree)/scripts/mkuboot.sh > UIMAGE_ARCH = $(SRCARCH) > UIMAGE_COMPRESSION = $(if $(2),$(2),none) > UIMAGE_OPTS-y = > -UIMAGE_TYPE = kernel > -UIMAGE_LOADADDR=arch_must_set_this > -UIMAGE_ENTRYADDR=$(UIMAGE_LOADADDR) > +ifeq ($(CONFIG_UIMAGE_KERNEL_NOLOAD),y) > + UIMAGE_TYPE = kernel_noload > + # Unused by bootloader: > + UIMAGE_LOADADDR=0 > + # Entry point relative to actual load address: > + UIMAGE_ENTRYADDR=0 > +else > + UIMAGE_TYPE = kernel > + # Absolute addresses: > + UIMAGE_LOADADDR=arch_must_set_this > + UIMAGE_ENTRYADDR=$(UIMAGE_LOADADDR) > +endif > UIMAGE_NAME = 'Linux-$(KERNELRELEASE)' > UIMAGE_IN = $< > UIMAGE_OUT = $@ Hi, Stephen, It seems that UIMAGE_ARCH and UIMAGE_LOADADDR can't be modified from arch-Makefiles. Regards, Guan Xuetao _______________________________________________ microblaze-linux mailing list microblaze-linux-FR6EJeJVuqfA6Z3fQjNZrN9u6TNh0Fb7@public.gmane.org https://lists.eait.uq.edu.au/mailman/listinfo/microblaze-linux From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mprc.pku.edu.cn ([162.105.203.9]:55173 "EHLO mprc.pku.edu.cn" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750906Ab2CGHCn (ORCPT ); Wed, 7 Mar 2012 02:02:43 -0500 Message-ID: <1331103163.2048.33.camel@epip-laptop> Subject: Re: [PATCH 2/3] Kbuild: Implement CONFIG_UIMAGE_KERNEL_NOLOAD From: Guan Xuetao Reply-To: gxt@mprc.pku.edu.cn Date: Wed, 07 Mar 2012 14:52:43 +0800 In-Reply-To: <1331080238-1524-2-git-send-email-swarren@wwwdotorg.org> References: <1331080238-1524-1-git-send-email-swarren@wwwdotorg.org> <1331080238-1524-2-git-send-email-swarren@wwwdotorg.org> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Mime-Version: 1.0 Sender: linux-arch-owner@vger.kernel.org List-ID: To: Stephen Warren Cc: Michal Marek , Russell King , nico@fluxnic.net, Haavard Skinnemoen , Hans-Christian Egtvedt , Mike Frysinger , Michal Simek , Paul Mundt , "David S. Miller" , linux-kernel@vger.kernel.org, linux-kbuild@vger.kernel.org, linux-arm-kernel@lists.infradead.org, uclinux-dist-devel@blackfin.uclinux.org, microblaze-uclinux@itee.uq.edu.au, linux-sh@vger.kernel.org, sparclinux@vger.kernel.org, linux-arch@vger.kernel.org Message-ID: <20120307065243.iGL4emtq4eUyvJA00fEvsIOgjwo0970RXf3Z59FEuHY@z> On Tue, 2012-03-06 at 17:30 -0700, Stephen Warren wrote: > This allows the user to use U-Boot's mkimage's -T kernel_noload option > if their arch Kconfig allows it, and they desire. > > Signed-off-by: Stephen Warren > --- > The next patch enables this new CONFIG_ALLOW_ option for ARM. I assume > that some other architectures will also be able to enable it, but I'm > not familiar enough with any to know which. > > init/Kconfig | 15 +++++++++++++++ > scripts/Makefile.lib | 15 ++++++++++++--- > 2 files changed, 27 insertions(+), 3 deletions(-) > > diff --git a/init/Kconfig b/init/Kconfig > index 6cfd71d..2035562 100644 > --- a/init/Kconfig > +++ b/init/Kconfig > @@ -203,6 +203,21 @@ config KERNEL_LZO > > endchoice > > +config ALLOW_UIMAGE_KERNEL_NOLOAD > + bool > + > +config UIMAGE_KERNEL_NOLOAD > + bool "Use mkimage's -T kernel_noload instead of -T kernel" > + depends on ALLOW_UIMAGE_KERNEL_NOLOAD > + help > + uImage files of type "kernel" embed an absolute load and entry point > + address. If the uImage is loaded to a different address, then the > + image must be copied the uImage file to the specified load address. > + This copy is a waste of time if the uImage is in fact completely > + relocatable. If your kernel and bootloader support it, you can build > + a "kernel_noload" uImage rather than a "kernel" uImage, which > + prevents any such copying. If unsure say N. > + > config DEFAULT_HOSTNAME > string "Default hostname" > default "(none)" > diff --git a/scripts/Makefile.lib b/scripts/Makefile.lib > index 7b0be18..d9c1c87 100644 > --- a/scripts/Makefile.lib > +++ b/scripts/Makefile.lib > @@ -314,9 +314,18 @@ MKIMAGE := $(srctree)/scripts/mkuboot.sh > UIMAGE_ARCH = $(SRCARCH) > UIMAGE_COMPRESSION = $(if $(2),$(2),none) > UIMAGE_OPTS-y = > -UIMAGE_TYPE = kernel > -UIMAGE_LOADADDR=arch_must_set_this > -UIMAGE_ENTRYADDR=$(UIMAGE_LOADADDR) > +ifeq ($(CONFIG_UIMAGE_KERNEL_NOLOAD),y) > + UIMAGE_TYPE = kernel_noload > + # Unused by bootloader: > + UIMAGE_LOADADDR=0 > + # Entry point relative to actual load address: > + UIMAGE_ENTRYADDR=0 > +else > + UIMAGE_TYPE = kernel > + # Absolute addresses: > + UIMAGE_LOADADDR=arch_must_set_this > + UIMAGE_ENTRYADDR=$(UIMAGE_LOADADDR) > +endif > UIMAGE_NAME = 'Linux-$(KERNELRELEASE)' > UIMAGE_IN = $< > UIMAGE_OUT = $@ Hi, Stephen, It seems that UIMAGE_ARCH and UIMAGE_LOADADDR can't be modified from arch-Makefiles. Regards, Guan Xuetao