diff options
author | David Brownell <dbrownell@users.sourceforge.net> | 2009-11-08 08:52:40 -0800 |
---|---|---|
committer | David Brownell <dbrownell@users.sourceforge.net> | 2009-11-08 08:52:40 -0800 |
commit | 3e6f9e8d1e65c4da505ff78481d54a115520d3d1 (patch) | |
tree | 71dba7d3285700e802edab7147ea0896d6e149ae /tcl/target/at91sam3uXX.cfg | |
parent | 2b1bd97508ebaf33c76d4f36ec4bb85592801055 (diff) | |
download | riscv-openocd-3e6f9e8d1e65c4da505ff78481d54a115520d3d1.zip riscv-openocd-3e6f9e8d1e65c4da505ff78481d54a115520d3d1.tar.gz riscv-openocd-3e6f9e8d1e65c4da505ff78481d54a115520d3d1.tar.bz2 |
target.cfg: remove "-work-area-virt 0"
The semantics of "-work-area-virt 0" (or phys) changed with
the patch to require specifying physical or virtrual work
area addresses. Specifying zero was previously a NOP. Now
it means that address zero is valid.
This patch addresses three related issues:
- MMU-less processors should never specify work-area-virt;
remove those specifications. Such processors include
ARM7TDMI, Cortex-M3, and ARM966.
- MMU-equipped processors *can* specify work-area-virt...
but zero won't be appropriate, except in mischievous
contexts (which hide null pointer exceptions).
Remove those specs from those processors too. If any of
those mappings is valid, someone will need to submit a
patch adding it ... along with a comment saying what OS
provides the mapping, and in which context. Example,
say "works with Linux 2.6.30+, in kernel mode". (Note
that ARM Linux doesn't map kernel memory to zero ...)
- Clarify docs on that "-virt" and other work area stuff.
Seems to me work-area-virt is quite problematic; not every
operating system provides such static mappings; if they do,
they're not in every MMU context...
Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
Diffstat (limited to 'tcl/target/at91sam3uXX.cfg')
-rw-r--r-- | tcl/target/at91sam3uXX.cfg | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tcl/target/at91sam3uXX.cfg b/tcl/target/at91sam3uXX.cfg index d54d3a4..985e052 100644 --- a/tcl/target/at91sam3uXX.cfg +++ b/tcl/target/at91sam3uXX.cfg @@ -33,7 +33,7 @@ set _TARGETNAME $_CHIPNAME.cpu target create $_TARGETNAME cortex_m3 -endian $_ENDIAN -chain-position $_TARGETNAME # 16K is plenty, the smallest chip has this much -$_TARGETNAME configure -work-area-virt 0 -work-area-phys 0x20000000 -work-area-size 16384 -work-area-backup 0 +$_TARGETNAME configure -work-area-phys 0x20000000 -work-area-size 16384 -work-area-backup 0 reset_config srst_only |