Hi Peng, On 20/11/18 11:19, Peng Fan wrote: > i.MX8M bootable image type is like i.MX6/7, but there is signed HDMI > firmware image in front of A53 bootable image, which is also has an IVT > header. > > Here we also include fit image to generate a bootable image. > > Signed-off-by: Peng Fan <peng....@nxp.com> > --- > > V2: > None > > common/image.c | 1 + > include/image.h | 1 + > include/imximage.h | 17 +- > tools/Makefile | 1 + > tools/imagetool.h | 1 + > tools/imx8mimage.c | 623 > +++++++++++++++++++++++++++++++++++++++++++++++++++++ > tools/mkimage.c | 7 + > 7 files changed, 650 insertions(+), 1 deletion(-) > create mode 100644 tools/imx8mimage.c > > diff --git a/common/image.c b/common/image.c > index 1c3a7720cb..d1cc1d11bb 100644 > --- a/common/image.c > +++ b/common/image.c > @@ -140,6 +140,7 @@ static const table_entry_t uimage_type[] = { > { IH_TYPE_KWBIMAGE, "kwbimage", "Kirkwood Boot Image",}, > { IH_TYPE_IMXIMAGE, "imximage", "Freescale i.MX Boot Image",}, > { IH_TYPE_IMX8IMAGE, "imx8image", "NXP i.MX8 Boot Image",}, > + { IH_TYPE_IMX8MIMAGE, "imx8mimage", "NXP i.MX8M Boot Image",}, > { IH_TYPE_INVALID, "invalid", "Invalid Image", }, > { IH_TYPE_MULTI, "multi", "Multi-File Image", }, > { IH_TYPE_OMAPIMAGE, "omapimage", "TI OMAP SPL With GP CH",}, > diff --git a/include/image.h b/include/image.h > index 031c355b48..d1e0d6dfbe 100644 > --- a/include/image.h > +++ b/include/image.h > @@ -252,6 +252,7 @@ enum { > IH_TYPE_KWBIMAGE, /* Kirkwood Boot Image */ > IH_TYPE_IMXIMAGE, /* Freescale IMXBoot Image */ > IH_TYPE_IMX8IMAGE, /* Freescale IMX8Boot Image */ > + IH_TYPE_IMX8MIMAGE, /* Freescale IMX8MBoot Image */
I applied after moving this at the end of the list, see comment in the structure. I send a patch to move IH_TYPE_IMXIMAGE, too, because it breaks compatibility. I have applied all imx8m patches, build is fine - I will send my PR now ;-). Best regards, Stefano > IH_TYPE_UBLIMAGE, /* Davinci UBL Image */ > IH_TYPE_OMAPIMAGE, /* TI OMAP Config Header Image */ > IH_TYPE_AISIMAGE, /* TI Davinci AIS Image */ > diff --git a/include/imximage.h b/include/imximage.h > index 6f7ca7f5e3..544babb53a 100644 > --- a/include/imximage.h > +++ b/include/imximage.h > @@ -33,6 +33,7 @@ > #define FLASH_OFFSET_NOR 0x1000 > #define FLASH_OFFSET_SATA FLASH_OFFSET_STANDARD > #define FLASH_OFFSET_QSPI 0x1000 > +#define FLASH_OFFSET_FLEXSPI 0x1000 > > /* Initial Load Region Size */ > #define FLASH_LOADSIZE_UNDEFINED 0xFFFFFFFF > @@ -48,6 +49,7 @@ > /* Command tags and parameters */ > #define IVT_HEADER_TAG 0xD1 > #define IVT_VERSION 0x40 > +#define IVT_VERSION_V3 0x41 > #define DCD_HEADER_TAG 0xD2 > #define DCD_VERSION 0x40 > #define DCD_WRITE_DATA_COMMAND_TAG 0xCC > @@ -71,6 +73,12 @@ enum imximage_cmd { > CMD_CHECK_BITS_CLR, > CMD_CSF, > CMD_PLUGIN, > + /* Follwoing on i.MX8MQ/MM */ > + CMD_FIT, > + CMD_SIGNED_HDMI, > + CMD_LOADER, > + CMD_SECOND_LOADER, > + CMD_DDR_FW, > }; > > enum imximage_fld_types { > @@ -84,7 +92,8 @@ enum imximage_fld_types { > enum imximage_version { > IMXIMAGE_VER_INVALID = -1, > IMXIMAGE_V1 = 1, > - IMXIMAGE_V2 > + IMXIMAGE_V2, > + IMXIMAGE_V3 > }; > > typedef struct { > @@ -177,6 +186,12 @@ typedef struct { > } data; > } imx_header_v2_t; > > +typedef struct { > + flash_header_v2_t fhdr; > + boot_data_t boot_data; > + uint32_t padding[5]; > +} imx_header_v3_t; > + > /* The header must be aligned to 4k on MX53 for NAND boot */ > struct imx_header { > union { > diff --git a/tools/Makefile b/tools/Makefile > index 3c0521f655..e9b4398039 100644 > --- a/tools/Makefile > +++ b/tools/Makefile > @@ -94,6 +94,7 @@ dumpimage-mkimage-objs := aisimage.o \ > imagetool.o \ > imximage.o \ > imx8image.o \ > + imx8mimage.o \ > kwbimage.o \ > lib/md5.o \ > lpc32xximage.o \ > diff --git a/tools/imagetool.h b/tools/imagetool.h > index 3fcfb4468d..71471420f9 100644 > --- a/tools/imagetool.h > +++ b/tools/imagetool.h > @@ -233,6 +233,7 @@ time_t imagetool_get_source_date( > void pbl_load_uboot(int fd, struct image_tool_params *mparams); > int zynqmpbif_copy_image(int fd, struct image_tool_params *mparams); > int imx8image_copy_image(int fd, struct image_tool_params *mparams); > +int imx8mimage_copy_image(int fd, struct image_tool_params *mparams); > > #define ___cat(a, b) a ## b > #define __cat(a, b) ___cat(a, b) > diff --git a/tools/imx8mimage.c b/tools/imx8mimage.c > new file mode 100644 > index 0000000000..50a256cbac > --- /dev/null > +++ b/tools/imx8mimage.c > @@ -0,0 +1,623 @@ > +// SPDX-License-Identifier: GPL-2.0+ > +/* > + * Copyright 2018 NXP > + * > + * Peng Fan <peng....@nxp.com> > + */ > + > + > +#include "imagetool.h" > +#include <image.h> > +#include "imximage.h" > +#include "compiler.h" > + > +static uint32_t ap_start_addr, sld_start_addr, sld_src_off; > +static char *ap_img, *sld_img, *signed_hdmi; > +static imx_header_v3_t imx_header[2]; /* At most there are 3 IVT headers */ > +static uint32_t rom_image_offset; > +static uint32_t sector_size = 0x200; > +static uint32_t image_off; > +static uint32_t sld_header_off; > +static uint32_t ivt_offset; > +static uint32_t using_fit; > + > +#define CSF_SIZE 0x2000 > +#define HDMI_IVT_ID 0 > +#define IMAGE_IVT_ID 1 > + > +#define HDMI_FW_SIZE 0x17000 /* Use Last 0x1000 for IVT and CSF */ > +#define ALIGN_SIZE 0x1000 > +#define ALIGN(x,a) __ALIGN_MASK((x), (__typeof__(x))(a) - 1, a) > +#define __ALIGN_MASK(x,mask,mask2) (((x) + (mask)) / (mask2) * (mask2)) > + > +static uint32_t get_cfg_value(char *token, char *name, int linenr) > +{ > + char *endptr; > + uint32_t value; > + > + errno = 0; > + value = strtoul(token, &endptr, 16); > + if (errno || token == endptr) { > + fprintf(stderr, "Error: %s[%d] - Invalid hex data(%s)\n", > + name, linenr, token); > + exit(EXIT_FAILURE); > + } > + return value; > +} > + > +int imx8mimage_check_params(struct image_tool_params *params) > +{ > + return 0; > +} > + > +static void imx8mimage_set_header(void *ptr, struct stat *sbuf, int ifd, > + struct image_tool_params *params) > +{ > +} > + > +static void imx8mimage_print_header(const void *ptr) > +{ > +} > + > +static int imx8mimage_check_image_types(uint8_t type) > +{ > + return (type == IH_TYPE_IMX8MIMAGE) ? EXIT_SUCCESS : EXIT_FAILURE; > +} > + > +static table_entry_t imx8mimage_cmds[] = { > + {CMD_BOOT_FROM, "BOOT_FROM", "boot command", }, > + {CMD_FIT, "FIT", "fit image", }, > + {CMD_SIGNED_HDMI, "SIGNED_HDMI", "signed hdmi image", }, > + {CMD_LOADER, "LOADER", "loader image", }, > + {CMD_SECOND_LOADER, "SECOND_LOADER", "2nd loader image", }, > + {CMD_DDR_FW, "DDR_FW", "ddr firmware", }, > + {-1, "", "", }, > +}; > + > +static table_entry_t imx8mimage_ivt_offset[] = { > + {0x400, "sd", "sd/emmc",}, > + {0x400, "emmc_fastboot", "emmc fastboot",}, > + {0x1000, "fspi", "flexspi", }, > + {-1, "", "Invalid", }, > +}; > + > +static void parse_cfg_cmd(int32_t cmd, char *token, char *name, int lineno) > +{ > + switch (cmd) { > + case CMD_BOOT_FROM: > + ivt_offset = get_table_entry_id(imx8mimage_ivt_offset, > + "imx8mimage ivt offset", > + token); > + if (!strncmp(token, "sd", 2)) > + rom_image_offset = 0x8000; > + break; > + case CMD_LOADER: > + ap_img = token; > + break; > + case CMD_SECOND_LOADER: > + sld_img = token; > + break; > + case CMD_SIGNED_HDMI: > + signed_hdmi = token; > + case CMD_FIT: > + using_fit = 1; > + break; > + case CMD_DDR_FW: > + /* Do nothing */ > + break; > + } > +} > + > +static void parse_cfg_fld(int32_t *cmd, char *token, > + char *name, int lineno, int fld) > +{ > + switch (fld) { > + case CFG_COMMAND: > + *cmd = get_table_entry_id(imx8mimage_cmds, > + "imx8mimage commands", token); > + if (*cmd < 0) { > + fprintf(stderr, "Error: %s[%d] - Invalid command" > "(%s)\n", > + name, lineno, token); > + exit(EXIT_FAILURE); > + } > + break; > + case CFG_REG_SIZE: > + parse_cfg_cmd(*cmd, token, name, lineno); > + break; > + case CFG_REG_ADDRESS: > + switch (*cmd) { > + case CMD_LOADER: > + ap_start_addr = get_cfg_value(token, name, lineno); > + break; > + case CMD_SECOND_LOADER: > + sld_start_addr = get_cfg_value(token, name, lineno); > + break; > + } > + break; > + case CFG_REG_VALUE: > + switch (*cmd) { > + case CMD_SECOND_LOADER: > + sld_src_off = get_cfg_value(token, name, lineno); > + break; > + } > + default: > + break; > + } > +} > + > +static uint32_t parse_cfg_file(char *name) > +{ > + FILE *fd = NULL; > + char *line = NULL; > + char *token, *saveptr1, *saveptr2; > + int lineno = 0; > + int fld; > + size_t len; > + int32_t cmd; > + > + fd = fopen(name, "r"); > + if (fd == 0) { > + fprintf(stderr, "Error: %s - Can't open cfg file\n", name); > + exit(EXIT_FAILURE); > + } > + > + /* > + * Very simple parsing, line starting with # are comments > + * and are dropped > + */ > + while ((getline(&line, &len, fd)) > 0) { > + lineno++; > + > + token = strtok_r(line, "\r\n", &saveptr1); > + if (!token) > + continue; > + > + /* Check inside the single line */ > + for (fld = CFG_COMMAND, cmd = CFG_INVALID, > + line = token; ; line = NULL, fld++) { > + token = strtok_r(line, " \t", &saveptr2); > + if (!token) > + break; > + > + /* Drop all text starting with '#' as comments */ > + if (token[0] == '#') > + break; > + > + parse_cfg_fld(&cmd, token, name, lineno, fld); > + } > + } > + > + return 0; > +} > + > +static void fill_zero(int ifd, int size, int offset) > +{ > + int fill_size; > + uint8_t zeros[4096]; > + int ret; > + > + memset(zeros, 0, sizeof(zeros)); > + > + ret = lseek(ifd, offset, SEEK_SET); > + if (ret < 0) { > + fprintf(stderr, "%s seek: %s\n", __func__, strerror(errno)); > + exit(EXIT_FAILURE); > + } > + > + while (size) { > + if (size > 4096) > + fill_size = 4096; > + else > + fill_size = size; > + > + if (write(ifd, (char *)&zeros, fill_size) != fill_size) { > + fprintf(stderr, "Write error: %s\n", > + strerror(errno)); > + exit(EXIT_FAILURE); > + } > + > + size -= fill_size; > + }; > +} > + > +static void copy_file(int ifd, const char *datafile, int pad, int offset, > + int datafile_offset) > +{ > + int dfd; > + struct stat sbuf; > + unsigned char *ptr; > + int tail; > + int zero = 0; > + uint8_t zeros[4096]; > + int size, ret; > + > + memset(zeros, 0, sizeof(zeros)); > + > + dfd = open(datafile, O_RDONLY | O_BINARY); > + if (dfd < 0) { > + fprintf(stderr, "Can't open %s: %s\n", > + datafile, strerror(errno)); > + exit(EXIT_FAILURE); > + } > + > + if (fstat(dfd, &sbuf) < 0) { > + fprintf(stderr, "Can't stat %s: %s\n", > + datafile, strerror(errno)); > + exit(EXIT_FAILURE); > + } > + > + ptr = mmap(0, sbuf.st_size, PROT_READ, MAP_SHARED, dfd, 0); > + if (ptr == MAP_FAILED) { > + fprintf(stderr, "Can't read %s: %s\n", > + datafile, strerror(errno)); > + exit(EXIT_FAILURE); > + } > + > + size = sbuf.st_size - datafile_offset; > + ret = lseek(ifd, offset, SEEK_SET); > + if (ret < 0) { > + fprintf(stderr, "lseek ifd fail\n"); > + exit(EXIT_FAILURE); > + } > + > + if (write(ifd, ptr + datafile_offset, size) != size) { > + fprintf(stderr, "Write error %s\n", > + strerror(errno)); > + exit(EXIT_FAILURE); > + } > + > + tail = size % 4; > + pad = pad - size; > + if (pad == 1 && tail != 0) { > + if (write(ifd, (char *)&zero, 4 - tail) != 4 - tail) { > + fprintf(stderr, "Write error on %s\n", > + strerror(errno)); > + exit(EXIT_FAILURE); > + } > + } else if (pad > 1) { > + while (pad > 0) { > + int todo = sizeof(zeros); > + > + if (todo > pad) > + todo = pad; > + if (write(ifd, (char *)&zeros, todo) != todo) { > + fprintf(stderr, "Write error: %s\n", > + strerror(errno)); > + exit(EXIT_FAILURE); > + } > + pad -= todo; > + } > + } > + > + munmap((void *)ptr, sbuf.st_size); > + close(dfd); > +} > + > +/* Return this IVT offset in the final output file */ > +static int generate_ivt_for_fit(int fd, int fit_offset, uint32_t ep, > + uint32_t *fit_load_addr) > +{ > + image_header_t image_header; > + int ret; > + > + uint32_t fit_size, load_addr; > + int align_len = 64 - 1; /* 64 is cacheline size */ > + > + ret = lseek(fd, fit_offset, SEEK_SET); > + if (ret < 0) { > + fprintf(stderr, "lseek fd fail for fit\n"); > + exit(EXIT_FAILURE); > + } > + > + if (read(fd, (char *)&image_header, sizeof(image_header_t)) != > + sizeof(image_header_t)) { > + fprintf(stderr, "generate_ivt_for_fit read failed: %s\n", > + strerror(errno)); > + exit(EXIT_FAILURE); > + } > + > + if (be32_to_cpu(image_header.ih_magic) != FDT_MAGIC) { > + fprintf(stderr, "%s error: not a FIT file\n", __func__); > + exit(EXIT_FAILURE); > + } > + > + fit_size = fdt_totalsize(&image_header); > + fit_size = (fit_size + 3) & ~3; > + > + fit_size = ALIGN(fit_size, ALIGN_SIZE); > + > + ret = lseek(fd, fit_offset + fit_size, SEEK_SET); > + if (ret < 0) { > + fprintf(stderr, "lseek fd fail for fit\n"); > + exit(EXIT_FAILURE); > + } > + > + /* > + * ep is the u-boot entry. SPL loads the FIT before the u-boot > + * address. 0x2000 is for CSF_SIZE > + */ > + load_addr = (ep - (fit_size + CSF_SIZE) - 512 - align_len) & > + ~align_len; > + > + flash_header_v2_t ivt_header = { { 0xd1, 0x2000, 0x40 }, > + load_addr, 0, 0, 0, > + (load_addr + fit_size), > + (load_addr + fit_size + 0x20), > + 0 }; > + > + if (write(fd, &ivt_header, sizeof(flash_header_v2_t)) != > + sizeof(flash_header_v2_t)) { > + fprintf(stderr, "IVT writing error on fit image\n"); > + exit(EXIT_FAILURE); > + } > + > + *fit_load_addr = load_addr; > + > + return fit_offset + fit_size; > +} > + > +static void dump_header_v2(imx_header_v3_t *imx_header, int index) > +{ > + const char *ivt_name[2] = {"HDMI FW", "LOADER IMAGE"}; > + > + fprintf(stdout, "========= IVT HEADER [%s] =========\n", > + ivt_name[index]); > + fprintf(stdout, "header.tag: \t\t0x%x\n", > + imx_header[index].fhdr.header.tag); > + fprintf(stdout, "header.length: \t\t0x%x\n", > + imx_header[index].fhdr.header.length); > + fprintf(stdout, "header.version: \t0x%x\n", > + imx_header[index].fhdr.header.version); > + fprintf(stdout, "entry: \t\t\t0x%x\n", > + imx_header[index].fhdr.entry); > + fprintf(stdout, "reserved1: \t\t0x%x\n", > + imx_header[index].fhdr.reserved1); > + fprintf(stdout, "dcd_ptr: \t\t0x%x\n", > + imx_header[index].fhdr.dcd_ptr); > + fprintf(stdout, "boot_data_ptr: \t\t0x%x\n", > + imx_header[index].fhdr.boot_data_ptr); > + fprintf(stdout, "self: \t\t\t0x%x\n", > + imx_header[index].fhdr.self); > + fprintf(stdout, "csf: \t\t\t0x%x\n", > + imx_header[index].fhdr.csf); > + fprintf(stdout, "reserved2: \t\t0x%x\n", > + imx_header[index].fhdr.reserved2); > + > + fprintf(stdout, "boot_data.start: \t0x%x\n", > + imx_header[index].boot_data.start); > + fprintf(stdout, "boot_data.size: \t0x%x\n", > + imx_header[index].boot_data.size); > + fprintf(stdout, "boot_data.plugin: \t0x%x\n", > + imx_header[index].boot_data.plugin); > +} > + > +void build_image(int ofd) > +{ > + int file_off, header_hdmi_off = 0, header_image_off; > + int hdmi_fd, ap_fd, sld_fd; > + uint32_t sld_load_addr = 0; > + uint32_t csf_off, sld_csf_off = 0; > + int ret; > + struct stat sbuf; > + > + if (!ap_img) { > + fprintf(stderr, "No LOADER image specificed\n"); > + exit(EXIT_FAILURE); > + } > + > + file_off = 0; > + > + if (signed_hdmi) { > + header_hdmi_off = file_off + ivt_offset; > + > + hdmi_fd = open(signed_hdmi, O_RDONLY | O_BINARY); > + if (hdmi_fd < 0) { > + fprintf(stderr, "%s: Can't open: %s\n", > + signed_hdmi, strerror(errno)); > + exit(EXIT_FAILURE); > + } > + > + if (fstat(hdmi_fd, &sbuf) < 0) { > + fprintf(stderr, "%s: Can't stat: %s\n", > + signed_hdmi, strerror(errno)); > + exit(EXIT_FAILURE); > + } > + close(hdmi_fd); > + > + /* > + * Aligned to 104KB = 92KB FW image + 0x8000 > + * (IVT and alignment) + 0x4000 (second IVT + CSF) > + */ > + file_off += ALIGN(sbuf.st_size, > + HDMI_FW_SIZE + 0x2000 + 0x1000); > + } > + > + header_image_off = file_off + ivt_offset; > + > + ap_fd = open(ap_img, O_RDONLY | O_BINARY); > + if (ap_fd < 0) { > + fprintf(stderr, "%s: Can't open: %s\n", > + ap_img, strerror(errno)); > + exit(EXIT_FAILURE); > + } > + if (fstat(ap_fd, &sbuf) < 0) { > + fprintf(stderr, "%s: Can't stat: %s\n", > + ap_img, strerror(errno)); > + exit(EXIT_FAILURE); > + } > + close(ap_fd); > + > + imx_header[IMAGE_IVT_ID].fhdr.header.tag = IVT_HEADER_TAG; /* 0xD1 */ > + imx_header[IMAGE_IVT_ID].fhdr.header.length = > + cpu_to_be16(sizeof(flash_header_v2_t)); > + imx_header[IMAGE_IVT_ID].fhdr.header.version = IVT_VERSION_V3; /* 0x41 > */ > + imx_header[IMAGE_IVT_ID].fhdr.entry = ap_start_addr; > + imx_header[IMAGE_IVT_ID].fhdr.self = ap_start_addr - > + sizeof(imx_header_v3_t); > + imx_header[IMAGE_IVT_ID].fhdr.dcd_ptr = 0; > + imx_header[IMAGE_IVT_ID].fhdr.boot_data_ptr = > + imx_header[IMAGE_IVT_ID].fhdr.self + > + offsetof(imx_header_v3_t, boot_data); > + imx_header[IMAGE_IVT_ID].boot_data.start = > + imx_header[IMAGE_IVT_ID].fhdr.self - ivt_offset; > + imx_header[IMAGE_IVT_ID].boot_data.size = > + ALIGN(sbuf.st_size + sizeof(imx_header_v3_t) + ivt_offset, > + sector_size); > + > + image_off = header_image_off + sizeof(imx_header_v3_t); > + file_off += imx_header[IMAGE_IVT_ID].boot_data.size; > + > + imx_header[IMAGE_IVT_ID].boot_data.plugin = 0; > + imx_header[IMAGE_IVT_ID].fhdr.csf = > + imx_header[IMAGE_IVT_ID].boot_data.start + > + imx_header[IMAGE_IVT_ID].boot_data.size; > + > + imx_header[IMAGE_IVT_ID].boot_data.size += CSF_SIZE; /* 8K region dummy > CSF */ > + > + csf_off = file_off; > + file_off += CSF_SIZE; > + > + /* Second boot loader image */ > + if (sld_img) { > + if (!using_fit) { > + fprintf(stderr, "Not support no fit\n"); > + exit(EXIT_FAILURE); > + } else { > + sld_header_off = sld_src_off - rom_image_offset; > + /* > + * Record the second bootloader relative offset in > + * image's IVT reserved1 > + */ > + imx_header[IMAGE_IVT_ID].fhdr.reserved1 = > + sld_header_off - header_image_off; > + sld_fd = open(sld_img, O_RDONLY | O_BINARY); > + if (sld_fd < 0) { > + fprintf(stderr, "%s: Can't open: %s\n", > + sld_img, strerror(errno)); > + exit(EXIT_FAILURE); > + } > + > + if (fstat(sld_fd, &sbuf) < 0) { > + fprintf(stderr, "%s: Can't stat: %s\n", > + sld_img, strerror(errno)); > + exit(EXIT_FAILURE); > + } > + > + close(sld_fd); > + > + file_off = sld_header_off; > + file_off += sbuf.st_size + sizeof(image_header_t); > + } > + } > + > + if (signed_hdmi) { > + header_hdmi_off -= ivt_offset; > + ret = lseek(ofd, header_hdmi_off, SEEK_SET); > + if (ret < 0) { > + fprintf(stderr, "lseek ofd fail for hdmi\n"); > + exit(EXIT_FAILURE); > + } > + > + /* The signed HDMI FW has 0x400 IVT offset, need remove it */ > + copy_file(ofd, signed_hdmi, 0, header_hdmi_off, 0x400); > + } > + > + /* Main Image */ > + header_image_off -= ivt_offset; > + image_off -= ivt_offset; > + ret = lseek(ofd, header_image_off, SEEK_SET); > + if (ret < 0) { > + fprintf(stderr, "lseek ofd fail\n"); > + exit(EXIT_FAILURE); > + } > + > + /* Write image header */ > + if (write(ofd, &imx_header[IMAGE_IVT_ID], sizeof(imx_header_v3_t)) != > + sizeof(imx_header_v3_t)) { > + fprintf(stderr, "error writing image hdr\n"); > + exit(1); > + } > + > + copy_file(ofd, ap_img, 0, image_off, 0); > + > + csf_off -= ivt_offset; > + fill_zero(ofd, CSF_SIZE, csf_off); > + > + if (sld_img) { > + sld_header_off -= ivt_offset; > + ret = lseek(ofd, sld_header_off, SEEK_SET); > + if (ret < 0) { > + fprintf(stderr, "lseek ofd fail for sld_img\n"); > + exit(EXIT_FAILURE); > + } > + > + /* Write image header */ > + if (!using_fit) { > + /* TODO */ > + } else { > + copy_file(ofd, sld_img, 0, sld_header_off, 0); > + sld_csf_off = > + generate_ivt_for_fit(ofd, sld_header_off, > + sld_start_addr, > + &sld_load_addr) + 0x20; > + } > + } > + > + if (!signed_hdmi) > + dump_header_v2(imx_header, 0); > + dump_header_v2(imx_header, 1); > + > + fprintf(stdout, "========= OFFSET dump ========="); > + if (signed_hdmi) { > + fprintf(stdout, "\nSIGNED HDMI FW:\n"); > + fprintf(stdout, " header_hdmi_off \t0x%x\n", > + header_hdmi_off); > + } > + > + fprintf(stdout, "\nLoader IMAGE:\n"); > + fprintf(stdout, " header_image_off \t0x%x\n image_off \t\t0x%x\n > csf_off \t\t0x%x\n", > + header_image_off, image_off, csf_off); > + fprintf(stdout, " spl hab block: \t0x%x 0x%x 0x%x\n", > + imx_header[IMAGE_IVT_ID].fhdr.self, header_image_off, > + csf_off - header_image_off); > + > + fprintf(stdout, "\nSecond Loader IMAGE:\n"); > + fprintf(stdout, " sld_header_off \t0x%x\n", > + sld_header_off); > + fprintf(stdout, " sld_csf_off \t\t0x%x\n", > + sld_csf_off); > + fprintf(stdout, " sld hab block: \t0x%x 0x%x 0x%x\n", > + sld_load_addr, sld_header_off, sld_csf_off - sld_header_off); > +} > + > +int imx8mimage_copy_image(int outfd, struct image_tool_params *mparams) > +{ > + /* > + * SECO FW is a container image, this is to calculate the > + * 2nd container offset. > + */ > + fprintf(stdout, "parsing %s\n", mparams->imagename); > + parse_cfg_file(mparams->imagename); > + > + build_image(outfd); > + > + return 0; > +} > + > +/* > + * imx8mimage parameters > + */ > +U_BOOT_IMAGE_TYPE( > + imx8mimage, > + "NXP i.MX8M Boot Image support", > + 0, > + NULL, > + imx8mimage_check_params, > + NULL, > + imx8mimage_print_header, > + imx8mimage_set_header, > + NULL, > + imx8mimage_check_image_types, > + NULL, > + NULL > +); > diff --git a/tools/mkimage.c b/tools/mkimage.c > index 38805f0c92..ea5ed542ab 100644 > --- a/tools/mkimage.c > +++ b/tools/mkimage.c > @@ -530,6 +530,13 @@ int main(int argc, char **argv) > ret = imx8image_copy_image(ifd, ¶ms); > if (ret) > return ret; > + } else if (params.type == IH_TYPE_IMX8MIMAGE) { > + /* i.MX8M has special Image format */ > + int ret; > + > + ret = imx8mimage_copy_image(ifd, ¶ms); > + if (ret) > + return ret; > } else { > copy_file(ifd, params.datafile, pad_len); > } > -- ===================================================================== DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de ===================================================================== _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot