On Sun, Sep 18, 2016 at 09:10:34PM +0200, Lukasz Majewski wrote:
Hi Tom,
On Fri, Sep 16, 2016 at 10:43:51AM +0200, Lukasz Majewski wrote:
This define gives the possibility to copy entire image (including header) from NOR parallel memory to e.g. SDRAM.
The legacy behavior is preserved, since other board don't enabled this option.
What's the use case?
This code is used on some boards (ppc [1]) in which we copy only payload of the image (i.e. we drop the package metadata).
In my particular case I would like to copy the image with its meta data (in similar way works spl_nand.c and spl_mmc.c).
Since, I wanted to avoid code duplication, I've decided to add this flag.
So another way of saying this, if I follow correctly, is that today SPL NOR doesn't work with u-boot.img but only u-boot.bin ?
Signed-off-by: Lukasz Majewski l.majewski@majess.pl
Kconfig | 10 ++++++++++
This should now get moved to common/spl/Kconfig
OK.
As a side note - I do have some other patches which move some SPL_NOR options to Kconfig.
However, I omitted them when I saw great work done by Simon to move a lot of SPL options to Kconfig. I hope them will be merged soon :-)
It's in now :)
README | 4 ++++
This can be dropped.
So, we don't add any new entries to README and now rely solely on Kconfig "help" sections?
Correct.
[snip]
diff --git a/common/spl/spl_nor.c b/common/spl/spl_nor.c index 8ea874c..1e0f739 100644 --- a/common/spl/spl_nor.c +++ b/common/spl/spl_nor.c @@ -9,13 +9,15 @@
int spl_nor_load_image(void) {
- void *img_src; int ret;
+#ifndef CONFIG_SPL_NOR_COPY_ENTIRE_IMAGE /* * Loading of the payload to SDRAM is done with skipping of * the mkimage header in this SPL NOR driver */ spl_image.flags |= SPL_COPY_PAYLOAD_ONLY;
+#endif #ifdef CONFIG_SPL_OS_BOOT if (!spl_start_uboot()) { const struct image_header *header; @@ -64,9 +66,13 @@ int spl_nor_load_image(void) if (ret) return ret;
We should have a comment here about how we're calculating the base of where to copy from.
I will be more verbose with v2 :-).
Thanks for review.
No problem, thank you!