This is an automated email from Gerrit. "Name of user not set <de...@thom.fr.eu.org>" just uploaded a new patch set to Gerrit, which you can find at https://review.openocd.org/c/openocd/+/7563
-- gerrit commit d795777639917cbb70f60cd0161710017b1998ad Author: François LEGAL <de...@thom.fr.eu.org> Date: Wed Mar 29 14:19:48 2023 +0200 src/target/mips_m4k : fix condition on overlapping workspace data area The condition to check if the workspace area (used by actual MIPS code executed on target) and data area (sandbox to put data to be read/written to/from flash) is wrong, thus preventing the use of FAST_* commands to program/verify FLASH. Signed-off-by: François LEGAL <de...@thom.fr.eu.org> Change-Id: Ic68424b7f42d44e550433a120093db5e7980fd56 diff --git a/src/target/mips_m4k.c b/src/target/mips_m4k.c index 491b247b15..d3b07585dd 100644 --- a/src/target/mips_m4k.c +++ b/src/target/mips_m4k.c @@ -1218,8 +1218,8 @@ static int mips_m4k_bulk_write_memory(struct target *target, target_addr_t addre fast_data_area = mips32->fast_data_area; - if (address <= fast_data_area->address + fast_data_area->size && - fast_data_area->address <= address + count) { + if (address < (fast_data_area->address + fast_data_area->size) && + fast_data_area->address < (address + count)) { LOG_ERROR("fast_data (" TARGET_ADDR_FMT ") is within write area " "(" TARGET_ADDR_FMT "-" TARGET_ADDR_FMT ").", fast_data_area->address, address, address + count); --