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 <jo...@foundries.io>
> wrote:
> >
> > On the MX7ULP, OCRAM for DCD is at 0x2f010000
> >
> > Signed-off-by: Jorge Ramirez-Ortiz <jo...@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?
