diff options
| author | Øyvind Harboe <oyvind.harboe@zylin.com> | 2009-10-12 11:57:59 +0200 |
|---|---|---|
| committer | Øyvind Harboe <oyvind.harboe@zylin.com> | 2009-10-12 11:59:18 +0200 |
| commit | b5b4fee811009a2febc3fe4ccc979489b5f910df (patch) | |
| tree | b78b8180e7233f6b40a3d2a531a03c9490d25437 /src | |
| parent | 5e837387aa7260518cf43f53b66ee917d8660802 (diff) | |
| download | riscv-openocd-b5b4fee811009a2febc3fe4ccc979489b5f910df.zip riscv-openocd-b5b4fee811009a2febc3fe4ccc979489b5f910df.tar.gz riscv-openocd-b5b4fee811009a2febc3fe4ccc979489b5f910df.tar.bz2 | |
Fix bogus 'transfer errors' with arm11 'memwrite burst enable'. A regression introduced in b8103660fa36a77158bd77379572c09913d85c00
Diffstat (limited to 'src')
| -rw-r--r-- | src/target/arm11_dbgtap.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/src/target/arm11_dbgtap.c b/src/target/arm11_dbgtap.c index b08d300..9aa7a30 100644 --- a/src/target/arm11_dbgtap.c +++ b/src/target/arm11_dbgtap.c @@ -585,7 +585,8 @@ int arm11_run_instr_data_to_core_noack(arm11_common_t * arm11, uint32_t opcode, arm11_setup_field(arm11, 1, NULL, NULL, chain5_fields + 2); uint8_t *Readies; - int bytes = sizeof(*Readies)*(count + 1); + int readiesNum = (count + 1); + int bytes = sizeof(*Readies)*readiesNum; Readies = (uint8_t *) malloc(bytes); if (Readies == NULL) { @@ -622,10 +623,9 @@ int arm11_run_instr_data_to_core_noack(arm11_common_t * arm11, uint32_t opcode, int retval = jtag_execute_queue(); if (retval == ERROR_OK) { - size_t error_count = 0; - for (size_t i = 0; i < asizeof(Readies); i++) + for (size_t i = 0; i < readiesNum; i++) { if (Readies[i] != 1) { @@ -633,8 +633,8 @@ int arm11_run_instr_data_to_core_noack(arm11_common_t * arm11, uint32_t opcode, } } - if (error_count) - LOG_ERROR("Transfer errors " ZU, error_count); + if (error_count > 0 ) + LOG_ERROR(ZU " words not transferred", error_count); } |