-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 On 04/19/2013 07:51 AM, Marek Vasut wrote: > Dear Benoît Thébaudeau, > >> Dear Marek Vasut, >> >> On Friday, April 19, 2013 1:16:31 PM, Marek Vasut wrote: >>> Dear Benoît Thébaudeau, >>> >>>> Dear Marek Vasut, >>>> >>>> On Friday, April 19, 2013 6:10:52 AM, Marek Vasut wrote: >>>>> This target is currently concatenating u-boot SPL in >>>>> imximage format with u-boot.bin. The NAND SPL can load a >>>>> raw binary, but the preffered format with much less >>>>> limitations is uImage format. Fix the target so u-boot.bin >>>>> is first converted into uImage format and only after that >>>>> is concatenated. >>>> >>>> Please reword: This is not a fix, but an enhancement. >>>> >>>>> Signed-off-by: Marek Vasut <ma...@denx.de> Cc: Albert >>>>> ARIBAUD <albert.u.b...@aribaud.net> Cc: Benoît Thébaudeau >>>>> <benoit.thebaud...@advansee.com> Cc: Fabio Estevam >>>>> <fabio.este...@freescale.com> Cc: Scott Wood >>>>> <scottw...@freescale.com> Cc: Stefano Babic >>>>> <sba...@denx.de> Cc: Tom Rini <tr...@ti.com> --- >>>>> >>>>> arch/arm/imx-common/Makefile | 7 +++++-- 1 file changed, 5 >>>>> insertions(+), 2 deletions(-) >>>>> >>>>> diff --git a/arch/arm/imx-common/Makefile >>>>> b/arch/arm/imx-common/Makefile index 44b6822..ba31d3e >>>>> 100644 --- a/arch/arm/imx-common/Makefile +++ >>>>> b/arch/arm/imx-common/Makefile @@ -69,8 +69,11 @@ >>>>> $(OBJTREE)/u-boot-with-nand-spl.imx: $(OBJTREE)/SPL >>>>> $(OBJTREE)/u-boot.bin >>>>> >>>>> -I binary -O binary $(OBJTREE)/spl/u-boot-nand-spl.imx \ >>>>> $(OBJTREE)/spl/u-boot-nand-spl-pad.imx rm >>>>> $(OBJTREE)/spl/u-boot-nand-spl.imx >>>>> >>>>> - cat $(OBJTREE)/spl/u-boot-nand-spl-pad.imx >>>>> $(OBJTREE)/u-boot.bin >> >>>>> $@ - rm $(OBJTREE)/spl/u-boot-nand-spl-pad.imx + >>>>> $(OBJTREE)/tools/mkimage -A arm -O U-Boot -a >>>>> $(CONFIG_SYS_TEXT_BASE) \ + -e $(CONFIG_SYS_TEXT_BASE) -C >>>>> > none -d >>>>> $(OBJTREE)/u-boot.bin \ + $(OBJTREE)/u-boot.uim + cat >>>>> $(OBJTREE)/spl/u-boot-nand-spl-pad.imx >>>>> $(OBJTREE)/u-boot.uim >> >>>>> $@ + rm $(OBJTREE)/spl/u-boot-nand-spl-pad.imx >>>>> $(OBJTREE)/u-boot.uim >>>>> >>>>> #################################################################### >>>>> >>>>> #### # >>>>> >>>>> -- 1.7.11.7 >>>> >>>> This change is correct. >>>> >>>> I don't know the uImage format very well. Can you give >>>> examples of what this patch can bring compared to the raw >>>> binary format? >>>> >>>> If this change is useful, it should probably be made for >>>> u-boot-with-spl.imx too. >>> >>> Please see common/spl/spl_nand.c which calls >>> spl_parse_image_header() on the payload loaded from NAND. This >>> is implemented in common/spl/spl.c and if proper header is not >>> found, the payload is assumed to be of 200kB of size, which is >>> far less than fully configured U-Boot. >> >> Or you could #define CONFIG_SYS_MONITOR_LEN to what you need. > > YUCK! > >>> So using the uimage format is much more flexible. >> >> I agree. >> >> So I'd see the following changes for this patch: - Reword "fix" >> to show enhancement, and detail the flexibility rationale in the >> description. - Extend to u-boot-with-spl.imx. > > Ok. > > Tom, shall we merge this one into current release after it's > fixed?
This is an enhancement not a bug fix, right? If so, next release. - -- Tom -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.11 (GNU/Linux) Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/ iQIcBAEBAgAGBQJRcTCWAAoJENk4IS6UOR1WFx8P/1CXwEcPqrvZthAJPqMYOjvU r1FVOX0MgXCkRQ1FXnGKaYdlJbc+1GS09KgsnbQ+AOwMEPPPAju62npHzP1/CfhH FkRInGmb/cqSuIt3shXg/Q4URkGEyMW6QhQpaskMgLNAzjGX1Q12Wnz2O1BLnIzr 1y9hMxNaqA8aIW9apXEtC/U67vHFVU74jT5IHzv/F6rzcAayoCz2Az0oG4bvRR90 HTPJMLH/z2lis1JgG5GSiH+604z+KOprNz1Uid2aZr7os/WmDsPkO4rJ0l8kd2HE Us7Au7/b0h1TfXrJv6oYNEsYn4jECPZ8HeHxjcRTi102JBAiiAK914PSuKVm+Kys ILffCZXcOHS6Q3zUTp4EkrcrxVzJxX+yIv2SJipy0dXx7mmrM559J55DUyFLsrKh o8Z4MMidjqulmTdQNeBhLLIo80URynouRQ2ZpiwpR7iKdObFuAPr0HVzg65tNpbo vr5WOquk4fooK88s3HXKjDvIyBJcq08rtn5sgmJCvba57WRrlnYwS1D3SF6NT/Me isF6jeL7nsZF2mckeCHJ3Wduw6kbg9n9mbfDWWqpYyqh/srvWvN6A7gCtwrlK2eR vN8DvDkBpWWulqfRzq6PLs8nH1x3WOajiA9iICc1O7+YyXzKGF+sAEBcBgTcH3fX lz4Hs0VAu3RA/kdeJiVJ =zMqB -----END PGP SIGNATURE----- _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot