patman: Suppress empty changelog entries
[oweals/u-boot.git] / tools / imximage.c
index 0da48a733d4296f521cc3cbc8b2a0513e4bf7e69..d7edd3c52f48edc5afeab4d4be5d246db5b34676 100644 (file)
@@ -1,3 +1,4 @@
+// SPDX-License-Identifier: GPL-2.0+
 /*
  * (C) Copyright 2009
  * Stefano Babic, DENX Software Engineering, sbabic@denx.de.
@@ -5,16 +6,18 @@
  * (C) Copyright 2008
  * Marvell Semiconductor <www.marvell.com>
  * Written-by: Prafulla Wadaskar <prafulla@marvell.com>
- *
- * SPDX-License-Identifier:    GPL-2.0+
  */
 
 #include "imagetool.h"
 #include <image.h>
 #include "imximage.h"
+#include <generated/autoconf.h>
 
 #define UNDEFINED 0xFFFFFFFF
 
+#if !defined(CONFIG_IMX_DCD_ADDR)
+#define CONFIG_IMX_DCD_ADDR 0x00910000
+#endif
 /*
  * Supported commands for configuration file
  */
@@ -23,10 +26,12 @@ static table_entry_t imximage_cmds[] = {
        {CMD_BOOT_OFFSET,       "BOOT_OFFSET",          "Boot offset",    },
        {CMD_WRITE_DATA,        "DATA",                 "Reg Write Data", },
        {CMD_WRITE_CLR_BIT,     "CLR_BIT",              "Reg clear bit",  },
+       {CMD_WRITE_SET_BIT,     "SET_BIT",              "Reg set bit",  },
        {CMD_CHECK_BITS_SET,    "CHECK_BITS_SET",   "Reg Check bits set", },
        {CMD_CHECK_BITS_CLR,    "CHECK_BITS_CLR",   "Reg Check bits clr", },
        {CMD_CSF,               "CSF",           "Command Sequence File", },
        {CMD_IMAGE_VERSION,     "IMAGE_VERSION",        "image version",  },
+       {CMD_PLUGIN,            "PLUGIN",               "file plugin_addr",  },
        {-1,                    "",                     "",               },
 };
 
@@ -80,6 +85,9 @@ static uint32_t imximage_ivt_offset = UNDEFINED;
 static uint32_t imximage_csf_size = UNDEFINED;
 /* Initial Load Region Size */
 static uint32_t imximage_init_loadsize;
+static uint32_t imximage_iram_free_start;
+static uint32_t imximage_plugin_size;
+static uint32_t plugin_image;
 
 static set_dcd_val_t set_dcd_val;
 static set_dcd_param_t set_dcd_param;
@@ -118,7 +126,11 @@ static uint32_t detect_imximage_version(struct imx_header *imx_hdr)
 
        /* Try to detect V2 */
        if ((fhdr_v2->header.tag == IVT_HEADER_TAG) &&
-               (hdr_v2->dcd_table.header.tag == DCD_HEADER_TAG))
+               (hdr_v2->data.dcd_table.header.tag == DCD_HEADER_TAG))
+               return IMXIMAGE_V2;
+
+       if ((fhdr_v2->header.tag == IVT_HEADER_TAG) &&
+           hdr_v2->boot_data.plugin)
                return IMXIMAGE_V2;
 
        return IMXIMAGE_VER_INVALID;
@@ -160,54 +172,89 @@ static void set_dcd_val_v1(struct imx_header *imxhdr, char *name, int lineno,
        }
 }
 
+static struct dcd_v2_cmd *gd_last_cmd;
+
 static void set_dcd_param_v2(struct imx_header *imxhdr, uint32_t dcd_len,
                int32_t cmd)
 {
-       dcd_v2_t *dcd_v2 = &imxhdr->header.hdr_v2.dcd_table;
+       dcd_v2_t *dcd_v2 = &imxhdr->header.hdr_v2.data.dcd_table;
+       struct dcd_v2_cmd *d = gd_last_cmd;
+       struct dcd_v2_cmd *d2;
+       int len;
+
+       if (!d)
+               d = &dcd_v2->dcd_cmd;
+       d2 = d;
+       len = be16_to_cpu(d->write_dcd_command.length);
+       if (len > 4)
+               d2 = (struct dcd_v2_cmd *)(((char *)d) + len);
 
        switch (cmd) {
        case CMD_WRITE_DATA:
-               dcd_v2->write_dcd_command.tag = DCD_WRITE_DATA_COMMAND_TAG;
-               dcd_v2->write_dcd_command.length = cpu_to_be16(
-                               dcd_len * sizeof(dcd_addr_data_t) + 4);
-               dcd_v2->write_dcd_command.param = DCD_WRITE_DATA_PARAM;
+               if ((d->write_dcd_command.tag == DCD_WRITE_DATA_COMMAND_TAG) &&
+                   (d->write_dcd_command.param == DCD_WRITE_DATA_PARAM))
+                       break;
+               d = d2;
+               d->write_dcd_command.tag = DCD_WRITE_DATA_COMMAND_TAG;
+               d->write_dcd_command.length = cpu_to_be16(4);
+               d->write_dcd_command.param = DCD_WRITE_DATA_PARAM;
                break;
        case CMD_WRITE_CLR_BIT:
-               dcd_v2->write_dcd_command.tag = DCD_WRITE_DATA_COMMAND_TAG;
-               dcd_v2->write_dcd_command.length = cpu_to_be16(
-                               dcd_len * sizeof(dcd_addr_data_t) + 4);
-               dcd_v2->write_dcd_command.param = DCD_WRITE_CLR_BIT_PARAM;
+               if ((d->write_dcd_command.tag == DCD_WRITE_DATA_COMMAND_TAG) &&
+                   (d->write_dcd_command.param == DCD_WRITE_CLR_BIT_PARAM))
+                       break;
+               d = d2;
+               d->write_dcd_command.tag = DCD_WRITE_DATA_COMMAND_TAG;
+               d->write_dcd_command.length = cpu_to_be16(4);
+               d->write_dcd_command.param = DCD_WRITE_CLR_BIT_PARAM;
+               break;
+       case CMD_WRITE_SET_BIT:
+               if ((d->write_dcd_command.tag == DCD_WRITE_DATA_COMMAND_TAG) &&
+                   (d->write_dcd_command.param == DCD_WRITE_SET_BIT_PARAM))
+                       break;
+               d = d2;
+               d->write_dcd_command.tag = DCD_WRITE_DATA_COMMAND_TAG;
+               d->write_dcd_command.length = cpu_to_be16(4);
+               d->write_dcd_command.param = DCD_WRITE_SET_BIT_PARAM;
                break;
        /*
         * Check data command only supports one entry,
-        * so use 0xC = size(address + value + command).
         */
        case CMD_CHECK_BITS_SET:
-               dcd_v2->write_dcd_command.tag = DCD_CHECK_DATA_COMMAND_TAG;
-               dcd_v2->write_dcd_command.length = cpu_to_be16(0xC);
-               dcd_v2->write_dcd_command.param = DCD_CHECK_BITS_SET_PARAM;
+               d = d2;
+               d->write_dcd_command.tag = DCD_CHECK_DATA_COMMAND_TAG;
+               d->write_dcd_command.length = cpu_to_be16(4);
+               d->write_dcd_command.param = DCD_CHECK_BITS_SET_PARAM;
                break;
        case CMD_CHECK_BITS_CLR:
-               dcd_v2->write_dcd_command.tag = DCD_CHECK_DATA_COMMAND_TAG;
-               dcd_v2->write_dcd_command.length = cpu_to_be16(0xC);
-               dcd_v2->write_dcd_command.param = DCD_CHECK_BITS_SET_PARAM;
+               d = d2;
+               d->write_dcd_command.tag = DCD_CHECK_DATA_COMMAND_TAG;
+               d->write_dcd_command.length = cpu_to_be16(4);
+               d->write_dcd_command.param = DCD_CHECK_BITS_CLR_PARAM;
                break;
        default:
                break;
        }
+       gd_last_cmd = d;
 }
 
 static void set_dcd_val_v2(struct imx_header *imxhdr, char *name, int lineno,
                                        int fld, uint32_t value, uint32_t off)
 {
-       dcd_v2_t *dcd_v2 = &imxhdr->header.hdr_v2.dcd_table;
+       struct dcd_v2_cmd *d = gd_last_cmd;
+       int len;
+
+       len = be16_to_cpu(d->write_dcd_command.length);
+       off = (len - 4) >> 3;
 
        switch (fld) {
        case CFG_REG_ADDRESS:
-               dcd_v2->addr_data[off].addr = cpu_to_be32(value);
+               d->addr_data[off].addr = cpu_to_be32(value);
                break;
        case CFG_REG_VALUE:
-               dcd_v2->addr_data[off].value = cpu_to_be32(value);
+               d->addr_data[off].value = cpu_to_be32(value);
+               off++;
+               d->write_dcd_command.length = cpu_to_be16((off << 3) + 4);
                break;
        default:
                break;
@@ -235,13 +282,22 @@ static void set_dcd_rst_v1(struct imx_header *imxhdr, uint32_t dcd_len,
 static void set_dcd_rst_v2(struct imx_header *imxhdr, uint32_t dcd_len,
                                                char *name, int lineno)
 {
-       dcd_v2_t *dcd_v2 = &imxhdr->header.hdr_v2.dcd_table;
-
-       dcd_v2->header.tag = DCD_HEADER_TAG;
-       dcd_v2->header.length = cpu_to_be16(
-                       dcd_len * sizeof(dcd_addr_data_t) + 8);
-       dcd_v2->header.version = DCD_VERSION;
-       set_dcd_param_v2(imxhdr, dcd_len, CMD_WRITE_DATA);
+       if (!imxhdr->header.hdr_v2.boot_data.plugin) {
+               dcd_v2_t *dcd_v2 = &imxhdr->header.hdr_v2.data.dcd_table;
+               struct dcd_v2_cmd *d = gd_last_cmd;
+               int len;
+
+               if (!d)
+                       d = &dcd_v2->dcd_cmd;
+               len = be16_to_cpu(d->write_dcd_command.length);
+               if (len > 4)
+                       d = (struct dcd_v2_cmd *)(((char *)d) + len);
+
+               len = (char *)d - (char *)&dcd_v2->header;
+               dcd_v2->header.tag = DCD_HEADER_TAG;
+               dcd_v2->header.length = cpu_to_be16(len);
+               dcd_v2->header.version = DCD_VERSION;
+       }
 }
 
 static void set_imx_hdr_v1(struct imx_header *imxhdr, uint32_t dcd_len,
@@ -257,8 +313,7 @@ static void set_imx_hdr_v1(struct imx_header *imxhdr, uint32_t dcd_len,
        /* Set magic number */
        fhdr_v1->app_code_barker = APP_CODE_BARKER;
 
-       /* TODO: check i.MX image V1 handling, for now use 'old' style */
-       hdr_base = entry_point - 4096;
+       hdr_base = entry_point - imximage_init_loadsize + flash_offset;
        fhdr_v1->app_dest_ptr = hdr_base - flash_offset;
        fhdr_v1->app_code_jump_vector = entry_point;
 
@@ -283,24 +338,94 @@ static void set_imx_hdr_v2(struct imx_header *imxhdr, uint32_t dcd_len,
        fhdr_v2->header.length = cpu_to_be16(sizeof(flash_header_v2_t));
        fhdr_v2->header.version = IVT_VERSION; /* 0x40 */
 
-       fhdr_v2->entry = entry_point;
-       fhdr_v2->reserved1 = fhdr_v2->reserved2 = 0;
-       hdr_base = entry_point - imximage_init_loadsize +
-               flash_offset;
-       fhdr_v2->self = hdr_base;
-       if (dcd_len > 0)
-               fhdr_v2->dcd_ptr = hdr_base
-                       + offsetof(imx_header_v2_t, dcd_table);
-       else
+       if (!hdr_v2->boot_data.plugin) {
+               fhdr_v2->entry = entry_point;
+               fhdr_v2->reserved1 = 0;
+               fhdr_v2->reserved1 = 0;
+               hdr_base = entry_point - imximage_init_loadsize +
+                       flash_offset;
+               fhdr_v2->self = hdr_base;
+               if (dcd_len > 0)
+                       fhdr_v2->dcd_ptr = hdr_base +
+                               offsetof(imx_header_v2_t, data);
+               else
+                       fhdr_v2->dcd_ptr = 0;
+               fhdr_v2->boot_data_ptr = hdr_base
+                               + offsetof(imx_header_v2_t, boot_data);
+               hdr_v2->boot_data.start = entry_point - imximage_init_loadsize;
+
+               fhdr_v2->csf = 0;
+
+               header_size_ptr = &hdr_v2->boot_data.size;
+               csf_ptr = &fhdr_v2->csf;
+       } else {
+               imx_header_v2_t *next_hdr_v2;
+               flash_header_v2_t *next_fhdr_v2;
+
+               if (imximage_csf_size != 0) {
+                       fprintf(stderr, "Error: Header v2: SECURE_BOOT is only supported in DCD mode!");
+                       exit(EXIT_FAILURE);
+               }
+
+               fhdr_v2->entry = imximage_iram_free_start +
+                       flash_offset + sizeof(flash_header_v2_t) +
+                       sizeof(boot_data_t);
+
+               fhdr_v2->reserved1 = 0;
+               fhdr_v2->reserved2 = 0;
+               fhdr_v2->self = imximage_iram_free_start + flash_offset;
+
                fhdr_v2->dcd_ptr = 0;
-       fhdr_v2->boot_data_ptr = hdr_base
-                       + offsetof(imx_header_v2_t, boot_data);
-       hdr_v2->boot_data.start = entry_point - imximage_init_loadsize;
 
-       fhdr_v2->csf = 0;
+               fhdr_v2->boot_data_ptr = fhdr_v2->self +
+                               offsetof(imx_header_v2_t, boot_data);
+
+               hdr_v2->boot_data.start = imximage_iram_free_start;
+               /*
+                * The actural size of plugin image is "imximage_plugin_size +
+                * sizeof(flash_header_v2_t) + sizeof(boot_data_t)", plus the
+                * flash_offset space.The ROM code only need to copy this size
+                * to run the plugin code. However, later when copy the whole
+                * U-Boot image to DDR, the ROM code use memcpy to copy the
+                * first part of the image, and use the storage read function
+                * to get the remaining part. This requires the dividing point
+                * must be multiple of storage sector size. Here we set the
+                * first section to be MAX_PLUGIN_CODE_SIZE(64KB) for this
+                * purpose.
+                */
+               hdr_v2->boot_data.size = MAX_PLUGIN_CODE_SIZE;
+
+               /* Security feature are not supported */
+               fhdr_v2->csf = 0;
+
+               next_hdr_v2 = (imx_header_v2_t *)((char *)hdr_v2 +
+                              imximage_plugin_size);
+
+               next_fhdr_v2 = &next_hdr_v2->fhdr;
+
+               next_fhdr_v2->header.tag = IVT_HEADER_TAG; /* 0xD1 */
+               next_fhdr_v2->header.length =
+                       cpu_to_be16(sizeof(flash_header_v2_t));
+               next_fhdr_v2->header.version = IVT_VERSION; /* 0x40 */
 
-       header_size_ptr = &hdr_v2->boot_data.size;
-       csf_ptr = &fhdr_v2->csf;
+               next_fhdr_v2->entry = entry_point;
+               hdr_base = entry_point - sizeof(struct imx_header);
+               next_fhdr_v2->reserved1 = 0;
+               next_fhdr_v2->reserved2 = 0;
+               next_fhdr_v2->self = hdr_base + imximage_plugin_size;
+
+               next_fhdr_v2->dcd_ptr = 0;
+               next_fhdr_v2->boot_data_ptr = next_fhdr_v2->self +
+                               offsetof(imx_header_v2_t, boot_data);
+
+               next_hdr_v2->boot_data.start = hdr_base - flash_offset;
+
+               header_size_ptr = &next_hdr_v2->boot_data.size;
+
+               next_hdr_v2->boot_data.plugin = 0;
+
+               next_fhdr_v2->csf = 0;
+       }
 }
 
 static void set_hdr_func(void)
@@ -314,6 +439,7 @@ static void set_hdr_func(void)
                max_dcd_entries = MAX_HW_CFG_SIZE_V1;
                break;
        case IMXIMAGE_V2:
+               gd_last_cmd = NULL;
                set_dcd_val = set_dcd_val_v2;
                set_dcd_param = set_dcd_param_v2;
                set_dcd_rst = set_dcd_rst_v2;
@@ -358,16 +484,19 @@ static void print_hdr_v2(struct imx_header *imx_hdr)
 {
        imx_header_v2_t *hdr_v2 = &imx_hdr->header.hdr_v2;
        flash_header_v2_t *fhdr_v2 = &hdr_v2->fhdr;
-       dcd_v2_t *dcd_v2 = &hdr_v2->dcd_table;
-       uint32_t size, version;
+       dcd_v2_t *dcd_v2 = &hdr_v2->data.dcd_table;
+       uint32_t size, version, plugin;
 
-       size = be16_to_cpu(dcd_v2->header.length) - 8;
-       if (size > (MAX_HW_CFG_SIZE_V2 * sizeof(dcd_addr_data_t))) {
-               fprintf(stderr,
-                       "Error: Image corrupt DCD size %d exceed maximum %d\n",
-                       (uint32_t)(size / sizeof(dcd_addr_data_t)),
-                       MAX_HW_CFG_SIZE_V2);
-               exit(EXIT_FAILURE);
+       plugin = hdr_v2->boot_data.plugin;
+       if (!plugin) {
+               size = be16_to_cpu(dcd_v2->header.length);
+               if (size > (MAX_HW_CFG_SIZE_V2 * sizeof(dcd_addr_data_t))) {
+                       fprintf(stderr,
+                               "Error: Image corrupt DCD size %d exceed maximum %d\n",
+                               (uint32_t)(size / sizeof(dcd_addr_data_t)),
+                               MAX_HW_CFG_SIZE_V2);
+                       exit(EXIT_FAILURE);
+               }
        }
 
        version = detect_imximage_version(imx_hdr);
@@ -375,19 +504,102 @@ static void print_hdr_v2(struct imx_header *imx_hdr)
        printf("Image Type:   Freescale IMX Boot Image\n");
        printf("Image Ver:    %x", version);
        printf("%s\n", get_table_entry_name(imximage_versions, NULL, version));
-       printf("Data Size:    ");
-       genimg_print_size(hdr_v2->boot_data.size);
-       printf("Load Address: %08x\n", (uint32_t)fhdr_v2->boot_data_ptr);
-       printf("Entry Point:  %08x\n", (uint32_t)fhdr_v2->entry);
-       if (fhdr_v2->csf && (imximage_ivt_offset != UNDEFINED) &&
-           (imximage_csf_size != UNDEFINED)) {
-               printf("HAB Blocks:   %08x %08x %08x\n",
-                      (uint32_t)fhdr_v2->self, 0,
-                      hdr_v2->boot_data.size - imximage_ivt_offset -
-                      imximage_csf_size);
+       printf("Mode:         %s\n", plugin ? "PLUGIN" : "DCD");
+       if (!plugin) {
+               printf("Data Size:    ");
+               genimg_print_size(hdr_v2->boot_data.size);
+               printf("Load Address: %08x\n", (uint32_t)fhdr_v2->boot_data_ptr);
+               printf("Entry Point:  %08x\n", (uint32_t)fhdr_v2->entry);
+               if (fhdr_v2->csf) {
+                       uint16_t dcdlen;
+                       int offs;
+
+                       dcdlen = hdr_v2->data.dcd_table.header.length;
+                       offs = (char *)&hdr_v2->data.dcd_table
+                               - (char *)hdr_v2;
+
+                       /*
+                        * The HAB block is the first part of the image, from
+                        * start of IVT header (fhdr_v2->self) to the start of
+                        * the CSF block (fhdr_v2->csf). So HAB size is
+                        * calculated as:
+                        * HAB_size = fhdr_v2->csf - fhdr_v2->self
+                        */
+                       printf("HAB Blocks:   0x%08x 0x%08x 0x%08x\n",
+                              (uint32_t)fhdr_v2->self, 0,
+                              (uint32_t)(fhdr_v2->csf - fhdr_v2->self));
+                       printf("DCD Blocks:   0x%08x 0x%08x 0x%08x\n",
+                              offs, CONFIG_IMX_DCD_ADDR, be16_to_cpu(dcdlen));
+               }
+       } else {
+               imx_header_v2_t *next_hdr_v2;
+               flash_header_v2_t *next_fhdr_v2;
+
+               /*First Header*/
+               printf("Plugin Data Size:     ");
+               genimg_print_size(hdr_v2->boot_data.size);
+               printf("Plugin Code Size:     ");
+               genimg_print_size(imximage_plugin_size);
+               printf("Plugin Load Address:  %08x\n", hdr_v2->boot_data.start);
+               printf("Plugin Entry Point:   %08x\n", (uint32_t)fhdr_v2->entry);
+
+               /*Second Header*/
+               next_hdr_v2 = (imx_header_v2_t *)((char *)hdr_v2 +
+                               imximage_plugin_size);
+               next_fhdr_v2 = &next_hdr_v2->fhdr;
+               printf("U-Boot Data Size:     ");
+               genimg_print_size(next_hdr_v2->boot_data.size);
+               printf("U-Boot Load Address:  %08x\n",
+                      next_hdr_v2->boot_data.start);
+               printf("U-Boot Entry Point:   %08x\n",
+                      (uint32_t)next_fhdr_v2->entry);
        }
 }
 
+static void copy_plugin_code(struct imx_header *imxhdr, char *plugin_file)
+{
+       int ifd;
+       struct stat sbuf;
+       char *plugin_buf = imxhdr->header.hdr_v2.data.plugin_code;
+       char *ptr;
+
+       ifd = open(plugin_file, O_RDONLY|O_BINARY);
+       if (ifd < 0) {
+               fprintf(stderr, "Can't open %s: %s\n",
+                       plugin_file,
+                       strerror(errno));
+               exit(EXIT_FAILURE);
+       }
+
+       if (fstat(ifd, &sbuf) < 0) {
+               fprintf(stderr, "Can't stat %s: %s\n",
+                       plugin_file,
+                       strerror(errno));
+               exit(EXIT_FAILURE);
+       }
+
+       ptr = mmap(0, sbuf.st_size, PROT_READ, MAP_SHARED, ifd, 0);
+       if (ptr == MAP_FAILED) {
+               fprintf(stderr, "Can't read %s: %s\n",
+                       plugin_file,
+                       strerror(errno));
+               exit(EXIT_FAILURE);
+       }
+
+       if (sbuf.st_size > MAX_PLUGIN_CODE_SIZE) {
+               printf("plugin binary size too large\n");
+               exit(EXIT_FAILURE);
+       }
+
+       memcpy(plugin_buf, ptr, sbuf.st_size);
+       imximage_plugin_size = sbuf.st_size;
+
+       (void) munmap((void *)ptr, sbuf.st_size);
+       (void) close(ifd);
+
+       imxhdr->header.hdr_v2.boot_data.plugin = 1;
+}
+
 static void parse_cfg_cmd(struct imx_header *imxhdr, int32_t cmd, char *token,
                                char *name, int lineno, int fld, int dcd_len)
 {
@@ -442,6 +654,7 @@ static void parse_cfg_cmd(struct imx_header *imxhdr, int32_t cmd, char *token,
                break;
        case CMD_WRITE_DATA:
        case CMD_WRITE_CLR_BIT:
+       case CMD_WRITE_SET_BIT:
        case CMD_CHECK_BITS_SET:
        case CMD_CHECK_BITS_CLR:
                value = get_cfg_value(token, name, lineno);
@@ -462,6 +675,10 @@ static void parse_cfg_cmd(struct imx_header *imxhdr, int32_t cmd, char *token,
                if (unlikely(cmd_ver_first != 1))
                        cmd_ver_first = 0;
                break;
+       case CMD_PLUGIN:
+               plugin_image = 1;
+               copy_plugin_code(imxhdr, token);
+               break;
        }
 }
 
@@ -488,6 +705,7 @@ static void parse_cfg_fld(struct imx_header *imxhdr, int32_t *cmd,
                switch(*cmd) {
                case CMD_WRITE_DATA:
                case CMD_WRITE_CLR_BIT:
+               case CMD_WRITE_SET_BIT:
                case CMD_CHECK_BITS_SET:
                case CMD_CHECK_BITS_CLR:
 
@@ -507,6 +725,10 @@ static void parse_cfg_fld(struct imx_header *imxhdr, int32_t *cmd,
                                }
                        }
                        break;
+               case CMD_PLUGIN:
+                       value = get_cfg_value(token, name, lineno);
+                       imximage_iram_free_start = value;
+                       break;
                default:
                        break;
                }
@@ -614,6 +836,7 @@ static void imximage_set_header(void *ptr, struct stat *sbuf, int ifd,
 {
        struct imx_header *imxhdr = (struct imx_header *)ptr;
        uint32_t dcd_len;
+       uint32_t header_size;
 
        /*
         * In order to not change the old imx cfg file
@@ -629,13 +852,19 @@ static void imximage_set_header(void *ptr, struct stat *sbuf, int ifd,
        /* Parse dcd configuration file */
        dcd_len = parse_cfg_file(imxhdr, params->imagename);
 
-       if (imximage_version == IMXIMAGE_V2) {
-               if (imximage_init_loadsize < imximage_ivt_offset +
-                       sizeof(imx_header_v2_t))
-                               imximage_init_loadsize = imximage_ivt_offset +
-                                       sizeof(imx_header_v2_t);
+       if (imximage_version == IMXIMAGE_V1)
+               header_size = sizeof(flash_header_v1_t);
+       else {
+               header_size = sizeof(flash_header_v2_t) + sizeof(boot_data_t);
+               if (!plugin_image)
+                       header_size += sizeof(dcd_v2_t);
+               else
+                       header_size += MAX_PLUGIN_CODE_SIZE;
        }
 
+       if (imximage_init_loadsize < imximage_ivt_offset + header_size)
+                       imximage_init_loadsize = imximage_ivt_offset + header_size;
+
        /* Set the imx header */
        (*set_imx_hdr)(imxhdr, dcd_len, params->ep, imximage_ivt_offset);
 
@@ -686,7 +915,7 @@ static int imximage_generate(struct image_tool_params *params,
        size_t alloc_len;
        struct stat sbuf;
        char *datafile = params->datafile;
-       uint32_t pad_len;
+       uint32_t pad_len, header_size;
 
        memset(&imximage_header, 0, sizeof(imximage_header));
 
@@ -704,18 +933,22 @@ static int imximage_generate(struct image_tool_params *params,
        /* Parse dcd configuration file */
        parse_cfg_file(&imximage_header, params->imagename);
 
-       /* TODO: check i.MX image V1 handling, for now use 'old' style */
-       if (imximage_version == IMXIMAGE_V1) {
-               alloc_len = 4096;
-       } else {
-               if (imximage_init_loadsize < imximage_ivt_offset +
-                       sizeof(imx_header_v2_t))
-                               imximage_init_loadsize = imximage_ivt_offset +
-                                       sizeof(imx_header_v2_t);
-               alloc_len = imximage_init_loadsize - imximage_ivt_offset;
+       if (imximage_version == IMXIMAGE_V1)
+               header_size = sizeof(imx_header_v1_t);
+       else {
+               header_size = sizeof(flash_header_v2_t) + sizeof(boot_data_t);
+               if (!plugin_image)
+                       header_size += sizeof(dcd_v2_t);
+               else
+                       header_size += MAX_PLUGIN_CODE_SIZE;
        }
 
-       if (alloc_len < sizeof(struct imx_header)) {
+       if (imximage_init_loadsize < imximage_ivt_offset + header_size)
+                       imximage_init_loadsize = imximage_ivt_offset + header_size;
+
+       alloc_len = imximage_init_loadsize - imximage_ivt_offset;
+
+       if (alloc_len < header_size) {
                fprintf(stderr, "%s: header error\n",
                        params->cmdname);
                exit(EXIT_FAILURE);
@@ -744,11 +977,7 @@ static int imximage_generate(struct image_tool_params *params,
 
        pad_len = ROUND(sbuf.st_size, 4096) - sbuf.st_size;
 
-       /* TODO: check i.MX image V1 handling, for now use 'old' style */
-       if (imximage_version == IMXIMAGE_V1)
-               return 0;
-       else
-               return pad_len;
+       return pad_len;
 }