1
0
mirror of git://projects.qi-hardware.com/openwrt-xburst.git synced 2024-11-20 20:01:33 +02:00

mkcsysimg cleanups

git-svn-id: svn://svn.openwrt.org/openwrt/trunk@7392 3c298f89-4303-0410-b956-a3cf2f4a3e73
This commit is contained in:
florian 2007-05-30 10:05:27 +00:00
parent f203cd4d6d
commit 59583de014

View File

@ -135,7 +135,8 @@ struct board_info {
char *progname; char *progname;
char *ofname = NULL; char *ofname = NULL;
int verblevel = 0; int verblevel = 0;
int size_check = 1; int invalid_causes_error = 1;
int keep_invalid_images = 0;
struct board_info *board = NULL; struct board_info *board = NULL;
@ -147,7 +148,6 @@ struct csys_block *code_block = NULL;
struct csys_block blocks[MAX_NUM_BLOCKS]; struct csys_block blocks[MAX_NUM_BLOCKS];
int num_blocks = 0; int num_blocks = 0;
static struct board_info boards[] = { static struct board_info boards[] = {
/* The original Edimax products */ /* The original Edimax products */
BOARD_ADM("BR-6104K", "Edimax BR-6104K", 2, SIG_BR6104K), BOARD_ADM("BR-6104K", "Edimax BR-6104K", 2, SIG_BR6104K),
@ -177,44 +177,32 @@ static struct board_info boards[] = {
}; };
/* /*
* Helper routines * Message macros
*/ */
void #define ERR(fmt, ...) do { \
errmsgv(int syserr, const char *fmt, va_list arg_ptr) fflush(0); \
{ fprintf(stderr, "[%s] *** error: " fmt "\n", progname, ## __VA_ARGS__ ); \
int save = errno; } while (0)
fflush(0); #define ERRS(fmt, ...) do { \
fprintf(stderr, "%s: error, ", progname); int save = errno; \
vfprintf(stderr, fmt, arg_ptr); fflush(0); \
if (syserr != 0) { fprintf(stderr, "[%s] *** error: " fmt "\n", progname, ## __VA_ARGS__ \
fprintf(stderr, " (%s)", strerror(save)); , strerror(save)); \
} } while (0)
fprintf(stderr, "\n");
}
void #define WARN(fmt, ...) do { \
errmsg(int syserr, const char *fmt, ...) fprintf(stderr, "[%s] *** warning: " fmt "\n", progname, ## __VA_ARGS__ ); \
{ } while (0)
va_list arg_ptr;
va_start(arg_ptr, fmt);
errmsgv(syserr, fmt, arg_ptr);
va_end(arg_ptr);
}
void #define DBG(lev, fmt, ...) do { \
dbgmsg(int level, const char *fmt, ...) if (verblevel < lev) \
{ break;\
va_list arg_ptr; fprintf(stderr, "[%s] " fmt "\n", progname, ## __VA_ARGS__ ); \
if (verblevel >= level) { } while (0)
fflush(0);
va_start(arg_ptr, fmt);
vfprintf(stderr, fmt, arg_ptr);
fprintf(stderr, "\n");
va_end(arg_ptr);
}
}
#define ERR_FATAL -1
#define ERR_INVALID_IMAGE -2
void void
usage(int status) usage(int status)
@ -235,8 +223,8 @@ usage(int status)
board->model, board->name); board->model, board->name);
}; };
fprintf(stream, fprintf(stream,
" -d disable check for available space\n" " -d don't throw error on invalid images\n"
" add boot code to the image\n" " -k keep invalid images\n"
" -b <file>[:<len>[:<padc>]]\n" " -b <file>[:<len>[:<padc>]]\n"
" add boot code to the image\n" " add boot code to the image\n"
" -c <file>[:<len>[:<padc>]]\n" " -c <file>[:<len>[:<padc>]]\n"
@ -374,8 +362,8 @@ required_arg(char c, char *arg)
if (arg == NULL || *arg != '-') if (arg == NULL || *arg != '-')
return 0; return 0;
errmsg(0,"option -%c requires an argument\n", c); ERR("option -%c requires an argument\n", c);
return -1; return ERR_FATAL;
} }
@ -497,8 +485,8 @@ write_out_data(FILE *outfile, uint8_t *data, size_t len,
fwrite(data, len, 1, outfile); fwrite(data, len, 1, outfile);
if (errno) { if (errno) {
errmsg(1,"unable to write output file"); ERRS("unable to write output file");
return -1; return ERR_FATAL;
} }
if (css) { if (css) {
@ -515,14 +503,16 @@ write_out_padding(FILE *outfile, size_t len, uint8_t padc,
{ {
uint8_t buf[512]; uint8_t buf[512];
size_t buflen = sizeof(buf); size_t buflen = sizeof(buf);
int err;
memset(buf, padc, buflen); memset(buf, padc, buflen);
while (len > 0) { while (len > 0) {
if (len < buflen) if (len < buflen)
buflen = len; buflen = len;
if (write_out_data(outfile, buf, buflen, css)) err = write_out_data(outfile, buf, buflen, css);
return -1; if (err)
return err;
len -= buflen; len -= buflen;
} }
@ -535,15 +525,15 @@ int
block_stat_file(struct csys_block *block) block_stat_file(struct csys_block *block)
{ {
struct stat st; struct stat st;
int res; int err;
if (block->file_name == NULL) if (block->file_name == NULL)
return 0; return 0;
res = stat(block->file_name, &st); err = stat(block->file_name, &st);
if (res){ if (err){
errmsg(1, "stat failed on %s", block->file_name); ERRS("stat failed on %s", block->file_name);
return res; return ERR_FATAL;
} }
block->file_size = st.st_size; block->file_size = st.st_size;
@ -565,7 +555,7 @@ block_writeout_hdr(FILE *outfile, struct csys_block *block)
hdr.addr = HOST_TO_LE32(block->addr); hdr.addr = HOST_TO_LE32(block->addr);
hdr.size = HOST_TO_LE32(block->size-block->size_hdr); hdr.size = HOST_TO_LE32(block->size-block->size_hdr);
dbgmsg(1,"writing header for block"); DBG(1,"writing header for block");
res = write_out_data(outfile, (uint8_t *)&hdr, sizeof(hdr),NULL); res = write_out_data(outfile, (uint8_t *)&hdr, sizeof(hdr),NULL);
return res; return res;
@ -590,8 +580,8 @@ block_writeout_file(FILE *outfile, struct csys_block *block)
errno = 0; errno = 0;
f = fopen(block->file_name,"r"); f = fopen(block->file_name,"r");
if (errno) { if (errno) {
errmsg(1,"unable to open file: %s", block->file_name); ERRS("unable to open file: %s", block->file_name);
return -1; return ERR_FATAL;
} }
len = block->file_size; len = block->file_size;
@ -603,9 +593,8 @@ block_writeout_file(FILE *outfile, struct csys_block *block)
errno = 0; errno = 0;
fread(buf, buflen, 1, f); fread(buf, buflen, 1, f);
if (errno != 0) { if (errno != 0) {
errmsg(1,"unable to read from file: %s", ERRS("unable to read from file: %s", block->file_name);
block->file_name); res = ERR_FATAL;
res = -1;
break; break;
} }
@ -633,7 +622,7 @@ block_writeout_data(FILE *outfile, struct csys_block *block)
/* write padding data if neccesary */ /* write padding data if neccesary */
padlen = block->size_avail - block->file_size; padlen = block->size_avail - block->file_size;
dbgmsg(1,"padding block, length=%d", padlen); DBG(1,"padding block, length=%d", padlen);
res = write_out_padding(outfile, padlen, block->padc, block->css); res = write_out_padding(outfile, padlen, block->padc, block->css);
return res; return res;
@ -649,7 +638,7 @@ block_writeout_csum(FILE *outfile, struct csys_block *block)
if (block->size_csum == 0) if (block->size_csum == 0)
return 0; return 0;
dbgmsg(1,"writing checksum for block"); DBG(1,"writing checksum for block");
csum = HOST_TO_LE16(csum_get(block->css)); csum = HOST_TO_LE16(csum_get(block->css));
res = write_out_data(outfile, (uint8_t *)&csum, block->size_csum, NULL); res = write_out_data(outfile, (uint8_t *)&csum, block->size_csum, NULL);
@ -670,7 +659,7 @@ block_writeout(FILE *outfile, struct csys_block *block)
block->css = NULL; block->css = NULL;
dbgmsg(2, "writing block, file=%s, file_size=%d, space=%d", DBG(2, "writing block, file=%s, file_size=%d, space=%d",
block->file_name, block->file_size, block->size_avail); block->file_name, block->file_size, block->size_avail);
res = block_writeout_hdr(outfile, block); res = block_writeout_hdr(outfile, block);
if (res) if (res)
@ -753,20 +742,20 @@ int
parse_opt_board(char ch, char *arg) parse_opt_board(char ch, char *arg)
{ {
dbgmsg(1,"parsing board option: -%c %s", ch, arg); DBG(1,"parsing board option: -%c %s", ch, arg);
if (board != NULL) { if (board != NULL) {
errmsg(0,"only one board option allowed"); ERR("only one board option allowed");
return -1; return ERR_FATAL;
} }
if (required_arg(ch, arg)) if (required_arg(ch, arg))
return -1; return ERR_FATAL;
board = find_board(arg); board = find_board(arg);
if (board == NULL){ if (board == NULL){
errmsg(0,"invalid/unknown board specified: %s", arg); ERR("invalid/unknown board specified: %s", arg);
return -1; return ERR_FATAL;
} }
return 0; return 0;
@ -784,8 +773,8 @@ parse_opt_block(char ch, char *arg)
int i; int i;
if ( num_blocks > MAX_NUM_BLOCKS ) { if ( num_blocks > MAX_NUM_BLOCKS ) {
errmsg(0,"too many blocks specified"); ERR("too many blocks specified");
return -1; return ERR_FATAL;
} }
block = &blocks[num_blocks]; block = &blocks[num_blocks];
@ -797,7 +786,7 @@ parse_opt_block(char ch, char *arg)
switch (ch) { switch (ch) {
case 'b': case 'b':
if (boot_block) { if (boot_block) {
errmsg(0,"only one boot block allowed"); WARN("only one boot block allowed");
break; break;
} }
block->type = BLOCK_TYPE_BOOT; block->type = BLOCK_TYPE_BOOT;
@ -805,7 +794,7 @@ parse_opt_block(char ch, char *arg)
break; break;
case 'c': case 'c':
if (conf_block) { if (conf_block) {
errmsg(0,"only one config block allowed"); WARN("only one config block allowed");
break; break;
} }
block->type = BLOCK_TYPE_CONF; block->type = BLOCK_TYPE_CONF;
@ -813,7 +802,7 @@ parse_opt_block(char ch, char *arg)
break; break;
case 'w': case 'w':
if (webp_block) { if (webp_block) {
errmsg(0,"only one web block allowed"); WARN("only one web block allowed");
break; break;
} }
block->type = BLOCK_TYPE_WEBP; block->type = BLOCK_TYPE_WEBP;
@ -824,7 +813,7 @@ parse_opt_block(char ch, char *arg)
break; break;
case 'r': case 'r':
if (code_block) { if (code_block) {
errmsg(0,"only one runtime block allowed"); WARN("only one runtime block allowed");
break; break;
} }
block->type = BLOCK_TYPE_CODE; block->type = BLOCK_TYPE_CODE;
@ -836,8 +825,8 @@ parse_opt_block(char ch, char *arg)
block->type = BLOCK_TYPE_XTRA; block->type = BLOCK_TYPE_XTRA;
break; break;
default: default:
errmsg(0,"unknown block type \"%c\"", ch); ERR("unknown block type \"%c\"", ch);
return -1; return ERR_FATAL;
} }
argc = parse_arg(arg, buf, argv); argc = parse_arg(arg, buf, argv);
@ -847,20 +836,20 @@ parse_opt_block(char ch, char *arg)
if (!is_empty_arg(p)) { if (!is_empty_arg(p)) {
block->file_name = strdup(p); block->file_name = strdup(p);
if (block->file_name == NULL) { if (block->file_name == NULL) {
errmsg(0,"not enough memory"); ERR("not enough memory");
return -1; return ERR_FATAL;
} }
} else if (block->need_file){ } else if (block->need_file){
errmsg(0,"no file specified in %s", arg); ERR("no file specified in %s", arg);
return -1; return ERR_FATAL;
} }
if (block->size_hdr) { if (block->size_hdr) {
p = argv[i++]; p = argv[i++];
if (!is_empty_arg(p)) { if (!is_empty_arg(p)) {
if (str2u32(p, &block->addr) != 0) { if (str2u32(p, &block->addr) != 0) {
errmsg(0,"invalid start address in %s", arg); ERR("invalid start address in %s", arg);
return -1; return ERR_FATAL;
} }
block->addr_set = 1; block->addr_set = 1;
} }
@ -869,16 +858,16 @@ parse_opt_block(char ch, char *arg)
p = argv[i++]; p = argv[i++];
if (!is_empty_arg(p)) { if (!is_empty_arg(p)) {
if (str2u32(p, &block->size) != 0) { if (str2u32(p, &block->size) != 0) {
errmsg(0,"invalid block size in %s", arg); ERR("invalid block size in %s", arg);
return -1; return ERR_FATAL;
} }
block->size_set = 1; block->size_set = 1;
} }
p = argv[i++]; p = argv[i++];
if (!is_empty_arg(p) && (str2u8(p, &block->padc) != 0)) { if (!is_empty_arg(p) && (str2u8(p, &block->padc) != 0)) {
errmsg(0,"invalid paddig character in %s", arg); ERR("invalid paddig character in %s", arg);
return -1; return ERR_FATAL;
} }
num_blocks++; num_blocks++;
@ -895,6 +884,7 @@ process_blocks(void)
int i; int i;
int res; int res;
res = 0;
/* collecting stats */ /* collecting stats */
for (i = 0; i < num_blocks; i++) { for (i = 0; i < num_blocks; i++) {
block = &blocks[i]; block = &blocks[i];
@ -914,8 +904,8 @@ process_blocks(void)
block->size = board->boot_size; block->size = board->boot_size;
} }
if (block->size > size_avail) { if (block->size > size_avail) {
errmsg(0,"boot block is too big"); WARN("boot block is too big");
return -1; res = ERR_INVALID_IMAGE;
} }
} }
size_avail -= board->boot_size; size_avail -= board->boot_size;
@ -929,8 +919,8 @@ process_blocks(void)
block->size = board->conf_size; block->size = board->conf_size;
} }
if (block->size > size_avail) { if (block->size > size_avail) {
errmsg(0,"config block is too big"); WARN("config block is too big");
return -1; res = ERR_INVALID_IMAGE;
} }
} }
@ -943,8 +933,8 @@ process_blocks(void)
block->size = board->webp_size; block->size = board->webp_size;
board->webp_size = block->size; board->webp_size = block->size;
if (block->size > board->webp_size_max) { if (block->size > board->webp_size_max) {
errmsg(0,"webpages block is too big"); WARN("webpages block is too big");
return -1; res = ERR_INVALID_IMAGE;
} }
memcpy(block->sig, board->sig_webp, 4); memcpy(block->sig, board->sig_webp, 4);
if (block->addr_set == 0) if (block->addr_set == 0)
@ -961,9 +951,8 @@ process_blocks(void)
} }
board->code_size = block->size; board->code_size = block->size;
if (board->code_size > size_avail) { if (board->code_size > size_avail) {
errmsg(0,"code block is too big"); WARN("code block is too big");
if (size_check) res = ERR_INVALID_IMAGE;
return -1;
} }
memcpy(code_block->sig, SIG_CSYS, 4); memcpy(code_block->sig, SIG_CSYS, 4);
if (code_block->addr_set == 0) if (code_block->addr_set == 0)
@ -981,10 +970,10 @@ process_blocks(void)
block->size = ALIGN(block->file_size, 0x10000); block->size = ALIGN(block->file_size, 0x10000);
if (block->size > size_avail) { if (block->size > size_avail) {
errmsg(0,"no space for file %s", WARN("file %s is too big, size=%d, avail=%d",
block->file_name); block->file_name, block->file_size,
if (size_check) size_avail);
return -1; res = ERR_INVALID_IMAGE;
} }
size_avail -= block->size; size_avail -= block->size;
@ -997,11 +986,10 @@ process_blocks(void)
block->size_csum; block->size_csum;
if (block->size_avail < block->file_size) { if (block->size_avail < block->file_size) {
errmsg(0,"file %s is too big, size=%d, avail=%d", WARN("file %s is too big, size=%d, avail=%d",
block->file_name, block->file_size, block->file_name, block->file_size,
block->size_avail); block->size_avail);
res = -1; res = ERR_INVALID_IMAGE;
break;
} }
} }
@ -1014,7 +1002,7 @@ main(int argc, char *argv[])
{ {
int optinvalid = 0; /* flag for invalid option */ int optinvalid = 0; /* flag for invalid option */
int c; int c;
int res = EXIT_FAILURE; int res = ERR_FATAL;
FILE *outfile; FILE *outfile;
@ -1024,7 +1012,7 @@ main(int argc, char *argv[])
while ( 1 ) { while ( 1 ) {
optinvalid = 0; optinvalid = 0;
c = getopt(argc, argv, "b:c:dB:hr:vw:x:"); c = getopt(argc, argv, "b:B:c:dhkr:vw:x:");
if (c == -1) if (c == -1)
break; break;
@ -1044,7 +1032,10 @@ main(int argc, char *argv[])
optinvalid = parse_opt_block(c,optarg); optinvalid = parse_opt_block(c,optarg);
break; break;
case 'd': case 'd':
size_check = 0; invalid_causes_error = 0;
break;
case 'k':
keep_invalid_images = 1;
break; break;
case 'B': case 'B':
optinvalid = parse_opt_board(c,optarg); optinvalid = parse_opt_board(c,optarg);
@ -1060,52 +1051,67 @@ main(int argc, char *argv[])
break; break;
} }
if (optinvalid != 0 ){ if (optinvalid != 0 ){
errmsg(0, "invalid option: -%c", optopt); ERR("invalid option: -%c", optopt);
goto out; goto out;
} }
} }
if (board == NULL) {
ERR("no board specified");
goto out;
}
if (optind == argc) { if (optind == argc) {
errmsg(0, "no output file specified"); ERR("no output file specified");
goto out; goto out;
} }
ofname = argv[optind++]; ofname = argv[optind++];
if (optind < argc) { if (optind < argc) {
errmsg(0, "invalid option: %s", argv[optind]); ERR("invalid option: %s", argv[optind]);
goto out; goto out;
} }
res = process_blocks();
if (res == ERR_FATAL)
goto out;
if (board == NULL) { if (res == ERR_INVALID_IMAGE) {
errmsg(0, "no board specified"); if (invalid_causes_error)
res = ERR_FATAL;
if (keep_invalid_images == 0) {
WARN("generation of invalid images disabled", ofname);
goto out; goto out;
} }
if (process_blocks() != 0) { WARN("generating invalid image", ofname);
goto out;
} }
outfile = fopen(ofname, "w"); outfile = fopen(ofname, "w");
if (outfile == NULL) { if (outfile == NULL) {
errmsg(1, "could not open \"%s\" for writing", ofname); ERRS("could not open \"%s\" for writing", ofname);
res = ERR_FATAL;
goto out; goto out;
} }
if (write_out_blocks(outfile) != 0) if (write_out_blocks(outfile) != 0) {
res = ERR_FATAL;
goto out_flush; goto out_flush;
}
dbgmsg(1,"Image file %s completed.", ofname); DBG(1,"Image file %s completed.", ofname);
res = EXIT_SUCCESS;
out_flush: out_flush:
fflush(outfile); fflush(outfile);
fclose(outfile); fclose(outfile);
if (res != EXIT_SUCCESS) { if (res == ERR_FATAL) {
unlink(ofname); unlink(ofname);
} }
out: out:
return res; if (res == ERR_FATAL)
return EXIT_FAILURE;
return EXIT_SUCCESS;
} }