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.majew...@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!
--
Tom
signature.asc
Description: Digital signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
