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

Fabio Estevam festevam at gmail.com
Mon Dec 2 14:42:13 CET 2019


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?


More information about the U-Boot mailing list