[U-Boot] [PATCH 1/4] common: imx: Implement generic u-boot.nand target

[Tom Rini <trini@ti.com>]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 02/27/2013 05:18 PM, Beno?t Th?baudeau wrote:
> Hi Marek,
> 
> On Monday, February 25, 2013 7:19:54 PM, Marek Vasut wrote:
>> Implement u-boot.nand target that can be reused with a small
>> amount of churn across all CPU models. The idea is to delegate
>> the u-boot.nand target out of the main Makefile and into the
>> CPU's Makefile (very similar to what u-boot.imx does now). The
>> main Makefile shall only contain path to which the u-boot.nand
>> target is delegated. Hopefully this will not produce too much 
>> bloat in the main Makefile.
>> 
>> To demonstrate this implementation, add u-boot.nand target for
>> i.MX53.
>> 
>> Signed-off-by: Marek Vasut <marex@denx.de> Cc: Beno?t Th?baudeau
>> <benoit.thebaudeau@advansee.com> Cc: Fabio Estevam
>> <fabio.estevam@freescale.com> Cc: Stefano Babic <sbabic@denx.de> 
>> --- Makefile                     |   18 ++++++++++++++++++ 
>> arch/arm/imx-common/Makefile |    6 ++++++ 2 files changed, 24
>> insertions(+)
>> 
>> diff --git a/Makefile b/Makefile index 41054b7..8b1010a 100644 
>> --- a/Makefile +++ b/Makefile @@ -470,6 +470,23 @@
>> $(obj)u-boot.img:	$(obj)u-boot.bin $(obj)u-boot.imx:
>> $(obj)u-boot.bin depend $(MAKE) -C $(SRCTREE)/arch/arm/imx-common
>> $(obj)u-boot.imx
>> 
>> +# +# Generic u-boot.nand target. +# +# Every CPU that needs
>> u-boot.nand must add a path to an implementation of +# the actual
>> u-boot.nand generator below. +# +ifdef CONFIG_MX53 
>> +CONFIG_NAND_TRG_PATH := $(SRCTREE)/arch/arm/imx-common +endif + 
>> +$(obj)u-boot.nand: $(obj)u-boot.bin depend +		if [
>> "X$(CONFIG_NAND_TRG_PATH)X" = "XX" ] ; then		\ +			echo "This CPU
>> does not support u-boot.nand target!" ;	\ +			exit 1 ;						\ +
>> fi +		$(MAKE) -C $(CONFIG_NAND_TRG_PATH) $(obj)u-boot.nand + 
>> $(obj)u-boot.kwb:       $(obj)u-boot.bin $(obj)tools/mkimage -n
>> $(CONFIG_SYS_KWD_CONFIG) -T kwbimage \ -a $(CONFIG_SYS_TEXT_BASE)
>> -e $(CONFIG_SYS_TEXT_BASE) -d $< $@ @@ -857,6 +874,7 @@ clobber:
>> tidy @rm -f $(obj)u-boot.kwb @rm -f $(obj)u-boot.pbl @rm -f
>> $(obj)u-boot.imx +	@rm -f $(obj)u-boot.nand @rm -f
>> $(obj)u-boot.ubl @rm -f $(obj)u-boot.ais @rm -f $(obj)u-boot.dtb 
>> diff --git a/arch/arm/imx-common/Makefile
>> b/arch/arm/imx-common/Makefile index 5d5c5b2..71ea36f 100644 ---
>> a/arch/arm/imx-common/Makefile +++
>> b/arch/arm/imx-common/Makefile @@ -50,6 +50,12 @@
>> $(obj)u-boot.imx: $(OBJTREE)/u-boot.bin $(OBJTREE)/$(patsubst
>> "%",%,$(CONFIG_IMX $(OBJTREE)/tools/mkimage -n $(filter-out
>> %.bin,$^) -T imximage \ -e $(CONFIG_SYS_TEXT_BASE) -d $< $@
>> 
>> +$(obj)u-boot.nand: $(obj)u-boot.imx +	(								\ +		echo -ne
>> '\x00\x00\x00\x00\x46\x43\x42\x20\x01' ;	\
> ^ It does not work in my environment (Ubuntu 12.10). -ne is
> interpreted as text, so the FCB is broken. This is because /bin/sh
> (set to dash) is invoked by default on my machine here. It would
> work with the /bin/bash set by the main Makefile for SHELL, but
> this is not passed to the sub-make.
> 
> So what do you think we should do: 1) Add "export SHELL" to the
> main Makefile? 2) Move the SHELL assignment from the main Makefile
> to the top-level config.mk? 3) Set "SHELL=$(SHELL)" on the command
> line when invoking the sub-make? 4) Use printf instead of echo? 5)
> Something else?
> 
> I like 1). Do you see possible side effects?

It looks like in these cases the kernel does $(shell echo -...), or is
that also not working on your setup?

- -- 
Tom
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iQIcBAEBAgAGBQJRLppcAAoJENk4IS6UOR1WiXoP/1wy8CU/bEgreOP/aPu2xsEA
7oh6CRRVLqGhioTo766IjaaDWD21uNmUpI3VK8wF8FBrPmmgS+IU9icYjd39ZsoL
m0riocPX3TS6SJFqs/xJpgsCFSCnw7MAjz8Eo/RphV8pohMnsqUrKX4TiPiZKxeS
2wky4E9PcMDdf07ahJEUC3Y73BNBXrml+MGh59sKrAKpvES7OutbUHCHD+ZeX3WC
Tk3nbFis3Sb+VY8vnMllyqVyCZQAI+MhJxm9zHTW2GzkX3/FsQ8tUND+DT8JmKv9
RQMkVCY92cdO5+0y+8RGO7WUwY0YghMdzfIkkr3ZQGlu9ccghcwvm21d2zVt7zCa
UfRyl1NK3xgFIfAsyBWVF8q75WkQUVT06YJo2SJTfcvIa/hlGH6FvN9fODcxeris
WAIan5Z3haZ7EiIGjn/KHq4dHYhS7c3iKqsi4FIQcoGsbwI9oFVqIQ98JW1UbWpS
174DvASSobba487fuAYTKgCwa2Qm9LHmy7XvJfb7CQTjrJuzsuRBWuhy3DD70AOC
Mo/Ewq4b1juvS/vTwT+d/eJcnBklDYvvJ31FMuf+P+EBwyz3AmIAmzBKRhRP37SV
EIKjB7GKHSuxrvJV6SjPAJW6suY4TiDSUh/myeXnGs/1+KDeGj91I5Dja4Cidl0o
TpP52BmyQrPVs2hS1PpC
=UQTu
-----END PGP SIGNATURE-----