MIPS: fastdata bulk write fallback
authorSpencer Oliver <ntfreak@users.sourceforge.net>
Wed, 6 Jan 2010 20:24:31 +0000 (20:24 +0000)
committerSpencer Oliver <ntfreak@users.sourceforge.net>
Wed, 6 Jan 2010 20:24:31 +0000 (20:24 +0000)
If fastdata access fails, then fallback to default mips_m4k_write_memory
Remove unnecessary fastdata loader verify check

Signed-off-by: Spencer Oliver <ntfreak@users.sourceforge.net>
src/target/mips32_pracc.c
src/target/mips_m4k.c

index 26d5a6b7d9c2ce079ecb36caa8f4dccb56833649..ac1d1b39860504921fd9122616a037016b53ad37 100644 (file)
@@ -968,15 +968,7 @@ int mips32_pracc_fastdata_xfer(struct mips_ejtag *ejtag_info, struct working_are
        /* write program into RAM */
        mips32_pracc_write_mem32(ejtag_info, source->address, ARRAY_SIZE(handler_code), handler_code);
 
-       /* quick verify RAM is working */
-       mips32_pracc_read_u32(ejtag_info, source->address, &val);
-       if (val != handler_code[0])
-       {
-               LOG_ERROR("fastdata handler verify failed\n");
-               return ERROR_TARGET_RESOURCE_NOT_AVAILABLE;
-       }
-
-       LOG_INFO("%s using 0x%.8x for write handler\n", __func__, source->address);
+       LOG_DEBUG("%s using 0x%.8x for write handler\n", __func__, source->address);
 
        jmp_code[1] |= UPPER16(source->address);
        jmp_code[2] |= LOWER16(source->address);
index 4adc1f193c39818d91a715819afceb32cb659338..312fc09cf517376a87892fee02435e9ff77b17b8 100644 (file)
@@ -1001,6 +1001,12 @@ int mips_m4k_bulk_write_memory(struct target *target, uint32_t address, uint32_t
        }
 
        retval = mips32_pracc_fastdata_xfer(ejtag_info, source, write, address, count, (uint32_t*) buffer);
+       if (retval != ERROR_OK)
+       {
+               /* FASTDATA access failed, try normal memory write */
+               LOG_WARNING("Fastdata access Failed, falling back to non-bulk write");
+               retval = mips_m4k_write_memory(target, address, 4, count, buffer);
+       }
 
        if (source)
                target_free_working_area(target, source);