From 7beed534039f364b064e2eb87432d3f1fec95e37 Mon Sep 17 00:00:00 2001 From: Gabor Juhos Date: Sat, 3 Nov 2007 14:53:24 +0000 Subject: [PATCH] image generation fixes (closes: #2643, #2644) * rewrite mkcsysimg to use an align parameter instead of a fixed size * Edimax images contains the jffs2 end-of-filesystem marker at the right position from now on SVN-Revision: 9490 --- target/linux/adm5120/image/Makefile | 14 +- target/linux/adm5120/image/router_le.mk | 37 +++- .../adm5120/router_le/profiles/Edimax.mk | 11 +- tools/firmware-utils/src/mkcsysimg.c | 159 +++++++++++------- 4 files changed, 145 insertions(+), 76 deletions(-) diff --git a/target/linux/adm5120/image/Makefile b/target/linux/adm5120/image/Makefile index 710e82be5c..ee8cf42ea8 100644 --- a/target/linux/adm5120/image/Makefile +++ b/target/linux/adm5120/image/Makefile @@ -9,7 +9,8 @@ include $(INCLUDE_DIR)/image.mk LOADER_MAKE := $(NO_TRACE_MAKE) -C lzma-loader KDIR=$(KDIR) IMGNAME := $(BIN_DIR)/openwrt-$(BOARD)-$(KERNEL) -JFFS2EOF := $(KDIR)/jffs2.eof +JFFS2BLOCK := $(KDIR)/jffs2.block +JFFS2MARK := $(KDIR)/jffs2.mark define imgname $(IMGNAME)-$(2)-$(patsubst jffs2-%,jffs2,$(1)) @@ -21,9 +22,12 @@ endef define Image/Prepare cat $(KDIR)/vmlinux | $(STAGING_DIR_HOST)/bin/lzma e -si -so -eos -lc1 -lp2 -pb2 > $(KDIR)/vmlinux.lzma - rm -f $(JFFS2EOF) - touch $(JFFS2EOF) - $(call prepare_generic_squashfs,$(JFFS2EOF)) + rm -f $(JFFS2BLOCK) + touch $(JFFS2BLOCK) + $(call prepare_generic_squashfs,$(JFFS2BLOCK)) + rm -f $(JFFS2MARK) + touch $(JFFS2MARK) + $(call add_jffs2_mark,$(JFFS2MARK)) endef define Image/Build/Loader @@ -63,7 +67,7 @@ define trxalign/jffs2-64k endef define trxalign/squashfs --a 1024 -f $(KDIR)/root.squashfs -a 0x10000 -A $(JFFS2EOF) +-a 1024 -f $(KDIR)/root.squashfs -a 0x10000 -A $(JFFS2BLOCK) endef define Image/Build/TRX diff --git a/target/linux/adm5120/image/router_le.mk b/target/linux/adm5120/image/router_le.mk index 135b8dff53..f08ed564dd 100644 --- a/target/linux/adm5120/image/router_le.mk +++ b/target/linux/adm5120/image/router_le.mk @@ -5,6 +5,23 @@ # See /LICENSE for more information. # +define trxedimax/jffs2-128k +-a 0x20000 -f $(KDIR)/root.jffs2-128k +endef + +define trxedimax/jffs2-64k +-a 0x10000 -f $(KDIR)/root.jffs2-64k +endef + +define trxedimax/squashfs +-a 1024 -f $(KDIR)/root.squashfs +endef + +define Image/Build/TRXEdimax + $(STAGING_DIR_HOST)/bin/trx -o $(1) -f $(KDIR)/vmlinux.lzma \ + $(call trxedimax/$(2)) +endef + define Image/Build/Compex $(call Image/Build/Loader,$(2),gz,0x80500000,0,y,$(2)) $(call Image/Build/TRX,$(call imgname,$(1),$(2)).trx,$(1),$(KDIR)/loader-$(2).gz) @@ -12,14 +29,16 @@ endef define Image/Build/Edimax $(call Image/Build/Loader,$(2),gz,0x80500000,0x6D8,y,$(2)) - $(call Image/Build/TRXNoloader,$(call imgname,$(1),$(2)).trx,$(1)) - $(STAGING_DIR_HOST)/bin/mkcsysimg -B $(2) -d -w \ - -r $(KDIR)/loader-$(2).gz::8192 \ - -x $(call imgname,$(1),$(2)).trx \ + $(call Image/Build/TRXEdimax,$(call imgname,$(1),$(2)).trx,$(1)) + $(STAGING_DIR_HOST)/bin/mkcsysimg -B $(2) -v -v -d -w \ + -r $(KDIR)/loader-$(2).gz::0x1000 \ + -x $(call imgname,$(1),$(2)).trx:0x10000 \ + -x $(JFFS2MARK):0x10000 \ $(call imgname,$(1),$(2))-webui.bin - $(STAGING_DIR_HOST)/bin/mkcsysimg -B $(2) -d \ - -r $(KDIR)/loader-$(2).gz::8192 \ - -x $(call imgname,$(1),$(2)).trx \ + $(STAGING_DIR_HOST)/bin/mkcsysimg -B $(2) -v -v -d \ + -r $(KDIR)/loader-$(2).gz::0x1000 \ + -x $(call imgname,$(1),$(2)).trx:0x10000 \ + -x $(JFFS2MARK):0x10000 \ $(call imgname,$(1),$(2))-xmodem.bin rm -f $(call imgname,$(1),$(2)).trx endef @@ -340,6 +359,10 @@ define Image/Build/Profile/CAS771W $(call Image/Build/Board/CAS771W/$(1)) endef +define Image/Build/Profile/BR6104K + $(call Image/Build/Board/BR6104K/$(1)) +endef + define Image/Build/Profile/BR6104KP $(call Image/Build/Board/BR6104KP/$(1)) endef diff --git a/target/linux/adm5120/router_le/profiles/Edimax.mk b/target/linux/adm5120/router_le/profiles/Edimax.mk index 34798b2381..9b0ca65c2b 100644 --- a/target/linux/adm5120/router_le/profiles/Edimax.mk +++ b/target/linux/adm5120/router_le/profiles/Edimax.mk @@ -5,8 +5,16 @@ # See /LICENSE for more information. # +define Profile/BR6104K + NAME:=Edimax BR-6104K (Unofficial) +endef + +define Profile/BR6104K/Description + Package set optimized for the Edimax BR-6104K +endef + define Profile/BR6104KP - NAME:=Edimax BR-6104KP + NAME:=Edimax BR-6104KP (Unofficial) PACKAGES:=kmod-usb-core kmod-usb-adm5120 endef @@ -14,4 +22,5 @@ define Profile/BR6104KP/Description Package set optimized for the Edimax BR-6104KP endef +$(eval $(call Profile,BR6104K)) $(eval $(call Profile,BR6104KP)) diff --git a/tools/firmware-utils/src/mkcsysimg.c b/tools/firmware-utils/src/mkcsysimg.c index 7df27c48eb..be53147f1b 100644 --- a/tools/firmware-utils/src/mkcsysimg.c +++ b/tools/firmware-utils/src/mkcsysimg.c @@ -52,8 +52,6 @@ # define LE32_TO_HOST(x) bswap_32(x) #endif -#define ALIGN(x,y) ((x)+((y)-1)) & ~((y)-1) - #define MAX_NUM_BLOCKS 8 #define MAX_ARG_COUNT 32 #define MAX_ARG_LEN 1024 @@ -66,6 +64,12 @@ #define BLOCK_TYPE_CODE 3 #define BLOCK_TYPE_XTRA 4 +#define DEFAULT_BLOCK_ALIGN 0x10000U + +#define CSUM_SIZE_NONE 0 +#define CSUM_SIZE_8 1 +#define CSUM_SIZE_16 2 + struct csum_state{ int size; @@ -82,17 +86,18 @@ struct csys_block { char *file_name; /* name of the file */ uint32_t file_size; /* length of the file */ - uint32_t size; - int size_set; + unsigned char sig[SIG_LEN]; + uint32_t addr; + int addr_set; + uint32_t align; + int align_set; uint8_t padc; + uint32_t size; uint32_t size_hdr; uint32_t size_csum; uint32_t size_avail; - unsigned char sig[SIG_LEN]; - uint32_t addr; - int addr_set; struct csum_state *css; }; @@ -226,15 +231,15 @@ usage(int status) fprintf(stream, " -d don't throw error on invalid images\n" " -k keep invalid images\n" -" -b [:[:]]\n" +" -b [:[:]]\n" " add boot code to the image\n" -" -c [:[:]]\n" +" -c [:[:]]\n" " add configuration settings to the image\n" -" -r :[][:[:]]\n" +" -r :[][:[:]]\n" " add runtime code to the image\n" -" -w [:[][:[:]]]\n" +" -w [:[][:[:]]]\n" " add webpages to the image\n" -" -x [:[:]]\n" +" -x [:[:]]\n" " add extra data at the end of the image\n" " -h show this screen\n" "Parameters:\n" @@ -244,6 +249,19 @@ usage(int status) exit(status); } +static inline uint32_t align(uint32_t base, uint32_t alignment) +{ + uint32_t ret; + + if (alignment) { + ret = (base + alignment - 1); + ret &= ~(alignment-1); + } else { + ret = base; + } + + return ret; +} /* * argument parsing @@ -447,10 +465,10 @@ void csum_update(uint8_t *p, uint32_t len, struct csum_state *css) { switch (css->size) { - case 1: + case CSUM_SIZE_8: csum8_update(p,len,css); break; - case 2: + case CSUM_SIZE_16: csum16_update(p,len,css); break; } @@ -463,10 +481,10 @@ csum_get(struct csum_state *css) uint16_t ret; switch (css->size) { - case 1: + case CSUM_SIZE_8: ret = csum8_get(css); break; - case 2: + case CSUM_SIZE_16: ret = csum16_get(css); break; } @@ -554,7 +572,7 @@ block_writeout_hdr(FILE *outfile, struct csys_block *block) /* setup header fields */ memcpy(hdr.sig, block->sig, 4); hdr.addr = HOST_TO_LE32(block->addr); - hdr.size = HOST_TO_LE32(block->size-block->size_hdr); + hdr.size = HOST_TO_LE32(block->align-block->size_hdr); DBG(1,"writing header for block"); res = write_out_data(outfile, (uint8_t *)&hdr, sizeof(hdr),NULL); @@ -808,7 +826,7 @@ parse_opt_block(char ch, char *arg) } block->type = BLOCK_TYPE_WEBP; block->size_hdr = sizeof(struct csys_header); - block->size_csum = 1; + block->size_csum = CSUM_SIZE_8; block->need_file = 0; webp_block = block; break; @@ -819,7 +837,7 @@ parse_opt_block(char ch, char *arg) } block->type = BLOCK_TYPE_CODE; block->size_hdr = sizeof(struct csys_header); - block->size_csum = 2; + block->size_csum = CSUM_SIZE_16; code_block = block; break; case 'x': @@ -858,11 +876,11 @@ parse_opt_block(char ch, char *arg) p = argv[i++]; if (!is_empty_arg(p)) { - if (str2u32(p, &block->size) != 0) { - ERR("invalid block size in %s", arg); + if (str2u32(p, &block->align) != 0) { + ERR("invalid alignment value in %s", arg); return ERR_FATAL; } - block->size_set = 1; + block->align_set = 1; } p = argv[i++]; @@ -881,7 +899,7 @@ int process_blocks(void) { struct csys_block *block; - uint32_t size_avail; + uint32_t offs = 0; int i; int res; @@ -894,72 +912,82 @@ process_blocks(void) return res; } - size_avail = board->flash_size; - /* bootloader */ block = boot_block; if (block) { - if (block->size_set) { - board->boot_size= block->size; - } else { - block->size = board->boot_size; - } - if (block->size > size_avail) { + block->size = board->boot_size; + if (block->file_size > board->boot_size) { WARN("boot block is too big"); res = ERR_INVALID_IMAGE; } } - size_avail -= board->boot_size; + offs += board->boot_size; /* configuration data */ block = conf_block; if (block) { - if (block->size_set) { - board->conf_size = block->size; - } else { - block->size = board->conf_size; - } - if (block->size > size_avail) { + block->size = board->conf_size; + if (block->file_size > board->conf_size) { WARN("config block is too big"); res = ERR_INVALID_IMAGE; } - } - size_avail -= board->conf_size; + offs += board->conf_size; /* webpages */ block = webp_block; if (block) { - if (block->size_set == 0) - block->size = board->webp_size; - board->webp_size = block->size; + + memcpy(block->sig, board->sig_webp, 4); + + if (block->addr_set == 0) + block->addr = board->addr_webp; + + if (block->align_set == 0) + block->align = DEFAULT_BLOCK_ALIGN; + + block->size = align(offs + block->file_size + block->size_hdr + + block->size_csum, block->align) - offs; + if (block->size > board->webp_size_max) { WARN("webpages block is too big"); res = ERR_INVALID_IMAGE; } - memcpy(block->sig, board->sig_webp, 4); - if (block->addr_set == 0) - block->addr = board->addr_webp; + + DBG(2,"webpages start at %08x, size=%08x", offs, + block->size); + + offs += block->size; + if (offs > board->flash_size) { + WARN("webp block is too big"); + res = ERR_INVALID_IMAGE; + } } - size_avail -= board->webp_size; /* runtime code */ block = code_block; if (block) { - if (block->size_set == 0) { - block->size =ALIGN(block->file_size+ block->size_hdr + - block->size_csum, 0x10000); - } - board->code_size = block->size; - if (board->code_size > size_avail) { + memcpy(code_block->sig, SIG_CSYS, 4); + + if (block->addr_set == 0) + block->addr = board->addr_code; + + if (block->align_set == 0) + block->align = DEFAULT_BLOCK_ALIGN; + + block->size = align(offs + block->file_size + + block->size_hdr + block->size_csum, + block->align) - offs; + + DBG(2,"code block start at %08x, size=%08x", offs, + block->size); + + offs += block->size; + if (offs > board->flash_size) { WARN("code block is too big"); res = ERR_INVALID_IMAGE; } - memcpy(code_block->sig, SIG_CSYS, 4); - if (code_block->addr_set == 0) - code_block->addr = board->addr_code; } - size_avail -= board->code_size; for (i = 0; i < num_blocks; i++) { block = &blocks[i]; @@ -967,17 +995,22 @@ process_blocks(void) if (block->type != BLOCK_TYPE_XTRA) continue; - if (block->size_set == 0) - block->size = ALIGN(block->file_size, 0x10000); + if (block->align_set == 0) + block->align = DEFAULT_BLOCK_ALIGN; - if (block->size > size_avail) { + block->size = align(offs + block->file_size, + block->align) - offs; + + DBG(2,"file %s start at %08x, size=%08x, align=%08x", + block->file_name, offs, block->size, block->align); + + offs += block->size; + if (offs > board->flash_size) { WARN("file %s is too big, size=%d, avail=%d", block->file_name, block->file_size, - size_avail); + board->flash_size - offs); res = ERR_INVALID_IMAGE; } - - size_avail -= block->size; } for (i = 0; i < num_blocks; i++) { -- 2.25.1