diff options
| author | Antonio Borneo <borneo.antonio@gmail.com> | 2021-05-05 12:27:11 +0200 |
|---|---|---|
| committer | Antonio Borneo <borneo.antonio@gmail.com> | 2021-05-22 10:04:12 +0100 |
| commit | 510df38407da25aa0fa7427d26b9253455885d9b (patch) | |
| tree | 0a8daa3246ae10b2146842e4d6ad79a755483b7c /src/target/arm_dpm.h | |
| parent | 936cff887abf116427802c4f07096fca1f2b1d88 (diff) | |
| download | riscv-openocd-510df38407da25aa0fa7427d26b9253455885d9b.zip riscv-openocd-510df38407da25aa0fa7427d26b9253455885d9b.tar.gz riscv-openocd-510df38407da25aa0fa7427d26b9253455885d9b.tar.bz2 | |
target/arm_dpm: rename 'wp_pc' as 'wp_addr'
The field 'wp_pc' was originally introduced in commit 55eeea7fceb6
("ARMv7a/Cortex-A8: report watchpoint trigger insn") in end 2009
to contain the address of the instruction which triggered a
watchpoint. Later on with commit 651b861d5d5f ("target/aarch64:
Add watchpoint support") it has been reused in to hold directly
the memory address that triggered a watchpoint.
Rename 'wp_pc' as 'wp_addr' and change its doxygen description.
While there, fix the format string to print the field.
Change-Id: I2e5ced1497e4a6fb6b38f91e881807512e8d8c47
Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
Reviewed-on: http://openocd.zylin.com/6204
Tested-by: jenkins
Reviewed-by: Liming Sun <limings@nvidia.com>
Diffstat (limited to 'src/target/arm_dpm.h')
| -rw-r--r-- | src/target/arm_dpm.h | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/src/target/arm_dpm.h b/src/target/arm_dpm.h index 8270782..80587f5 100644 --- a/src/target/arm_dpm.h +++ b/src/target/arm_dpm.h @@ -137,8 +137,12 @@ struct arm_dpm { struct dpm_bp *dbp; struct dpm_wp *dwp; - /** Address of the instruction which triggered a watchpoint. */ - target_addr_t wp_pc; + /** + * Target dependent watchpoint address. + * Either the address of the instruction which triggered a watchpoint + * or the memory address whose access triggered a watchpoint. + */ + target_addr_t wp_addr; /** Recent value of DSCR. */ uint32_t dscr; |