From 524d79ebe74e28d751c88e665aabac2ea4d04e5c Mon Sep 17 00:00:00 2001 From: Stefan Mahr Date: Mon, 30 May 2011 16:21:04 +0200 Subject: [PATCH] mips: fix some more endian madness --- src/target/mips32.c | 7 +++ src/target/mips_m4k.c | 115 +++++++++++++++++++----------------------- 2 files changed, 59 insertions(+), 63 deletions(-) diff --git a/src/target/mips32.c b/src/target/mips32.c index 79215b5f3f..4782067bf5 100644 --- a/src/target/mips32.c +++ b/src/target/mips32.c @@ -559,6 +559,13 @@ int mips32_configure_break_unit(struct target *target) return retval; } + /* check if target endianness settings matches debug control register */ + if ( ( (dcr & EJTAG_DCR_ENM) && (target->endianness == TARGET_LITTLE_ENDIAN) ) || + ( !(dcr & EJTAG_DCR_ENM) && (target->endianness == TARGET_BIG_ENDIAN) ) ) + { + LOG_WARNING("DCR endianness settings does not match target settings"); + } + LOG_DEBUG("DCR 0x%" PRIx32 " numinst %i numdata %i", dcr, mips32->num_inst_bpoints, mips32->num_data_bpoints); diff --git a/src/target/mips_m4k.c b/src/target/mips_m4k.c index 1166b8748c..03c996941e 100644 --- a/src/target/mips_m4k.c +++ b/src/target/mips_m4k.c @@ -868,25 +868,22 @@ static int mips_m4k_read_memory(struct target *target, uint32_t address, if (ERROR_OK != retval) return retval; - /* TAP data register is loaded LSB first (little endian) */ - if (target->endianness == TARGET_BIG_ENDIAN) + /* mips32_..._read_mem with size 4/2 returns uint32_t/uint16_t in host */ + /* endianness, but byte array should represent target endianness */ + uint32_t i, t32; + uint16_t t16; + for(i = 0; i < (count*size); i += size) { - uint32_t i, t32; - uint16_t t16; - - for(i = 0; i < (count*size); i += size) + switch(size) { - switch(size) - { - case 4: - t32 = le_to_h_u32(&buffer[i]); - h_u32_to_be(&buffer[i], t32); - break; - case 2: - t16 = le_to_h_u16(&buffer[i]); - h_u16_to_be(&buffer[i], t16); - break; - } + case 4: + t32 = *(uint32_t*)&buffer[i]; + target_buffer_set_u32(target,&buffer[i], t32); + break; + case 2: + t16 = *(uint16_t*)&buffer[i]; + target_buffer_set_u16(target,&buffer[i], t16); + break; } } @@ -915,36 +912,33 @@ static int mips_m4k_write_memory(struct target *target, uint32_t address, if (((size == 4) && (address & 0x3u)) || ((size == 2) && (address & 0x1u))) return ERROR_TARGET_UNALIGNED_ACCESS; + /* mips32_..._write_mem with size 4/2 requires uint32_t/uint16_t in host */ + /* endianness, but byte array represents target endianness */ uint8_t * t = NULL; - - /* TAP data register is loaded LSB first (little endian) */ - if (target->endianness == TARGET_BIG_ENDIAN) + t = malloc(count * sizeof(uint32_t)); + if (t == NULL) { - t = malloc(count * sizeof(uint32_t)); - if (t == NULL) - { - LOG_ERROR("Out of memory"); - return ERROR_FAIL; - } + LOG_ERROR("Out of memory"); + return ERROR_FAIL; + } - uint32_t i, t32, t16; - for(i = 0; i < (count*size); i += size) + uint32_t i, t32; + uint16_t t16; + for(i = 0; i < (count*size); i += size) + { + switch(size) { - switch(size) - { - case 4: - t32 = be_to_h_u32((uint8_t *) &buffer[i]); - h_u32_to_le(&t[i], t32); - break; - case 2: - t16 = be_to_h_u16((uint8_t *) &buffer[i]); - h_u16_to_le(&t[i], t16); - break; - } + case 4: + t32 = target_buffer_get_u32(target,&buffer[i]); + *(uint32_t*)&t[i] = t32; + break; + case 2: + t16 = target_buffer_get_u16(target,&buffer[i]); + *(uint16_t*)&t[i] = t16; + break; } - - buffer = t; } + buffer = t; /* if noDMA off, use DMAACC mode for memory write */ int retval; @@ -952,12 +946,13 @@ static int mips_m4k_write_memory(struct target *target, uint32_t address, retval = mips32_pracc_write_mem(ejtag_info, address, size, count, (void *)buffer); else retval = mips32_dmaacc_write_mem(ejtag_info, address, size, count, (void *)buffer); - if (ERROR_OK != retval) - return retval; if (t != NULL) free(t); + if (ERROR_OK != retval) + return retval; + return ERROR_OK; } @@ -1065,31 +1060,25 @@ static int mips_m4k_bulk_write_memory(struct target *target, uint32_t address, ejtag_info->fast_access_save = -1; } + /* mips32_pracc_fastdata_xfer requires uint32_t in host endianness, */ + /* but byte array represents target endianness */ uint8_t * t = NULL; - const uint8_t *ec_buffer = buffer; /* endian-corrected buffer */ - - /* TAP data register is loaded LSB first (little endian) */ - if (target->endianness == TARGET_BIG_ENDIAN) + t = malloc(count * sizeof(uint32_t)); + if (t == NULL) { - t = malloc(count * sizeof(uint32_t)); - if (t == NULL) - { - LOG_ERROR("Out of memory"); - return ERROR_FAIL; - } - - uint32_t i, t32; - for(i = 0; i < (count * 4); i += 4) - { - t32 = be_to_h_u32((uint8_t *) &buffer[i]); - h_u32_to_le(&t[i], t32); - } - - ec_buffer = t; + LOG_ERROR("Out of memory"); + return ERROR_FAIL; } + uint32_t i, t32; + for(i = 0; i < (count*4); i += 4) + { + t32 = target_buffer_get_u32(target,&buffer[i]); + *(uint32_t*)&t[i] = t32; + } + retval = mips32_pracc_fastdata_xfer(ejtag_info, mips32->fast_data_area, write_t, address, - count, (uint32_t*) (void *)ec_buffer); + count, (uint32_t*) (void *)t); if (t != NULL) free(t); -- 2.30.2