The default implementation of target_read/write_phys_memory
is to call target_read/write_memory, but I'm thinking the default
implementation should return an error and
LOG_ERROR("Not implemented").

Users can then discover the missing target support, stop searching
for it elsewhere, report it or even pitch in to get it added.

The optional [phys] argument to mww/mdw variants is still there for
*all* targets though, even if they can't support it in principle
(there aren't any Cortex M0/M3's w/MMU are there?).

Which brings me to another thing...

Should commands that *could* be supported by targets be added
even if the OpenOCD target code does not support it them?

If users are searching for the right way to do mrc/mcr operations
for, say, XScale, would it not be better if the command was there
but returned "not implemented"?

-- 
Øyvind Harboe
http://www.zylin.com/zy1000.html
ARM7 ARM9 ARM11 XScale Cortex
JTAG debugger and flash programmer
_______________________________________________
Openocd-development mailing list
[email protected]
https://lists.berlios.de/mailman/listinfo/openocd-development

Reply via email to