[PATCH] tools/imximage: fix reported DCD information for MX7ULP

Jorge Ramirez-Ortiz, Foundries jorge at foundries.io
Tue Dec 3 15:07:12 CET 2019


On 02/12/19 10:42:13, Fabio Estevam wrote:
> Hi Jorge,
> 
> Adding Breno.
> 
> On Thu, Nov 28, 2019 at 7:33 AM Jorge Ramirez-Ortiz <jorge at foundries.io> wrote:
> >
> > On the MX7ULP, OCRAM for DCD is at 0x2f010000
> >
> > Signed-off-by: Jorge Ramirez-Ortiz <jorge at foundries.io>
> > ---
> >  tools/imximage.c | 6 ++++++
> >  1 file changed, 6 insertions(+)
> >
> > diff --git a/tools/imximage.c b/tools/imximage.c
> > index d7c0b6e883..762a06d468 100644
> > --- a/tools/imximage.c
> > +++ b/tools/imximage.c
> > @@ -11,6 +11,7 @@
> >  #include "imagetool.h"
> >  #include <image.h>
> >  #include "imximage.h"
> > +#include <generated/autoconf.h>
> >
> >  #define UNDEFINED 0xFFFFFFFF
> >
> > @@ -524,8 +525,13 @@ static void print_hdr_v2(struct imx_header *imx_hdr)
> >                         printf("HAB Blocks:   0x%08x 0x%08x 0x%08x\n",
> >                                (uint32_t)fhdr_v2->self, 0,
> >                                (uint32_t)(fhdr_v2->csf - fhdr_v2->self));
> > +#ifndef CONFIG_MX7ULP
> >                         printf("DCD Blocks:   0x00910000 0x%08x 0x%08x\n",
> >                                offs, be16_to_cpu(dcdlen));
> > +#else
> > +                       printf("DCD Blocks:   0x2f010000 0x%08x 0x%08x\n",
> > +                              offs, be16_to_cpu(dcdlen));
> > +#endif
> 
> Can we make the DCD OCRAM a variable passed via Kconfig and selected
> by SoC type?
> 
> This way we don't need to keep changing this for future SoCs.
> 
> What do you think?

sure that would work. given that the default 0x00910000 is valid on
most of the cases - well all but the mx7ulp - we could apply this default and only change to the
Kconfig value if defined.

should we go for that?





More information about the U-Boot mailing list