diff options
author | Alexander Kurz <akurz@blala.de> | 2016-07-17 17:53:15 +0200 |
---|---|---|
committer | Sascha Hauer <s.hauer@pengutronix.de> | 2016-07-18 07:53:33 +0200 |
commit | e0916fe82f3511d7487760401d0192a6641d94a0 (patch) | |
tree | 231dd22dd38c6cbb070f315a1700ce3b935e0e1a /scripts/imx | |
parent | 9e53ae2c388cc22f511c85df0aea4d561dcfffdd (diff) | |
download | barebox-e0916fe82f3511d7487760401d0192a6641d94a0.tar.gz barebox-e0916fe82f3511d7487760401d0192a6641d94a0.tar.xz |
scripts: imx-usb-loader: remove useless code
The configuration interface for struct usb_work is not implemented here
leaving the options set on fixed settings or even uninitialized.
Do some cleanup and remove those half-cooked dead code passages.
Signed-off-by: Alexander Kurz <akurz@blala.de>
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Diffstat (limited to 'scripts/imx')
-rw-r--r-- | scripts/imx/imx-usb-loader.c | 49 |
1 files changed, 3 insertions, 46 deletions
diff --git a/scripts/imx/imx-usb-loader.c b/scripts/imx/imx-usb-loader.c index b0973d3445..92b828f0b0 100644 --- a/scripts/imx/imx-usb-loader.c +++ b/scripts/imx/imx-usb-loader.c @@ -67,13 +67,7 @@ struct mach_id { struct usb_work { char filename[256]; unsigned char dcd; - unsigned char clear_dcd; unsigned char plug; -#define J_ADDR 1 -#define J_HEADER 2 -#define J_HEADER2 3 - unsigned char jump_mode; - unsigned jump_addr; }; struct usb_id { @@ -925,24 +919,6 @@ static int perform_dcd(unsigned char *p, const unsigned char *file_start, return ret; } -static int clear_dcd_ptr(unsigned char *p, unsigned char *file_start, unsigned cnt) -{ - struct imx_flash_header *ohdr = (struct imx_flash_header *)p; - struct imx_flash_header_v2 *hdr = (struct imx_flash_header_v2 *)p; - - switch (usb_id->mach_id->header_type) { - case HDR_MX51: - printf("clear dcd_ptr=0x%08x\n", ohdr->dcd); - ohdr->dcd = 0; - break; - case HDR_MX53: - printf("clear dcd_ptr=0x%08x\n", hdr->dcd_ptr); - hdr->dcd_ptr = 0; - break; - } - return 0; -} - static int get_dl_start(const unsigned char *p, const unsigned char *file_start, unsigned cnt, unsigned *dladdr, unsigned *max_length, unsigned *plugin, unsigned *header_addr) @@ -1017,18 +993,6 @@ static int process_header(struct usb_work *curr, unsigned char *buf, int cnt, return ret; } curr->dcd = 0; - if ((!curr->jump_mode) && (!curr->plug)) { - printf("!!dcd done, nothing else requested\n"); - return 0; - } - } - - if (curr->clear_dcd) { - ret = clear_dcd_ptr(p, buf, cnt); - if (ret < 0) { - printf("!!clear_dcd returned %i\n", ret); - return ret; - } } if (*p_plugin && (!curr->plug) && (!header_cnt)) { @@ -1110,12 +1074,6 @@ static int do_irom_download(struct usb_work *curr, int verify) header_offset = ret; - if ((!curr->jump_mode) && (!curr->plug)) { - /* nothing else requested */ - ret = 0; - goto cleanup; - } - if (plugin && (!curr->plug)) { printf("Only plugin header found\n"); ret = -1; @@ -1130,9 +1088,7 @@ static int do_irom_download(struct usb_work *curr, int verify) file_base = header_addr - header_offset; - type = (curr->plug || curr->jump_mode) ? FT_APP : FT_LOAD_ONLY; - - if (usb_id->mach_id->mode == MODE_BULK && type == FT_APP) { + if (usb_id->mach_id->mode == MODE_BULK) { /* No jump command, dladdr should point to header */ dladdr = header_addr; } @@ -1153,6 +1109,8 @@ static int do_irom_download(struct usb_work *curr, int verify) if (fsize > max_length) fsize = max_length; + type = FT_APP; + if (verify) { verify_buffer = malloc(64); @@ -1314,7 +1272,6 @@ int main(int argc, char *argv[]) w.plug = 1; w.dcd = 1; - w.jump_mode = J_HEADER; strncpy(w.filename, argv[optind], sizeof(w.filename) - 1); r = libusb_init(NULL); |