[PATCH] arm: add support to generate uImage

[Sascha Hauer]

On Sat, Apr 23, 2011 at 07:05:18AM +0200, Jean-Christophe PLAGNIOL-VILLARD wrote:
> so barebox could be loaded as kernel by itself
> 
> add LOAD_TEXT_BASE to specify the load address
> 
> Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj at jcrosoft.com>
> ---
>  arch/arm/Makefile |   15 +++++++++++++++
>  common/Kconfig    |    7 +++++++
>  2 files changed, 22 insertions(+), 0 deletions(-)
> 
> diff --git a/arch/arm/Makefile b/arch/arm/Makefile
> index 108bd5e..f0ec667 100644
> --- a/arch/arm/Makefile
> +++ b/arch/arm/Makefile
> @@ -134,6 +134,21 @@ ifeq ($(machine-y),netx)
>  KBUILD_IMAGE := barebox.netx
>  endif
>  
> +MKIMAGE		:= $(objtree)/scripts/mkimage
> +
> +quiet_cmd_uimage = UIMAGE  $@
> +      cmd_uimage = $(MKIMAGE) -A arm -O linux -T kernel \
> +		   -C none -a $(BAREBOX_LOAD) -e $(BAREBOX_ENTRY) \
> +		   -n 'Barebox-$(KERNELRELEASE)' -d $< $@
> +
> +uImage: $(MKIMAGE)
> +uImage: BAREBOX_LOAD=$(CONFIG_LOAD_TEXT_BASE)
> +uImage: BAREBOX_ENTRY=$(CONFIG_TEXT_BASE)
> +
> +uImage: barebox.bin
> +	$(call if_changed,uimage,none)
> +	@echo '  Image $@ is ready'
> +
>  all: $(KBUILD_IMAGE)
>  
>  archprepare: maketools
> diff --git a/common/Kconfig b/common/Kconfig
> index 9e30579..50995d3 100644
> --- a/common/Kconfig
> +++ b/common/Kconfig
> @@ -80,6 +80,13 @@ config TEXT_BASE
>  	help
>  	  The Address barebox gets linked at.
>  
> +config LOAD_TEXT_BASE
> +	prompt "LOAD_TEXT_BASE"
> +	hex
> +	default TEXT_BASE
> +	help
> +	  The Address barebox gets load at as a uImage
> +

No, please don't. When you load barebox with itself you end up
overwriting the running barebox with the just loaded image if using
TEXT_BASE here.
barebox can be started from any address in SDRAM. Isn't there a
mechanism to support this with the uImage format? The U-Boot guys have
the same problem with ARM_PATCH_PHYS_VIRT in the kernel aswell, so I
assume this is already solved in some way, no?

Sascha

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |