diff options
author | Marc Schink <dev@zapb.de> | 2021-06-17 15:47:11 +0200 |
---|---|---|
committer | Antonio Borneo <borneo.antonio@gmail.com> | 2021-06-26 14:40:18 +0100 |
commit | 56b72b33cfc7014e001fa9e209b62f4b6c009fb5 (patch) | |
tree | de5f80b61a3a7d92a7677593266d53e687fbbd18 | |
parent | fb34fd60ca955fe05c31357052aefad5fa1ef08f (diff) | |
download | riscv-openocd-56b72b33cfc7014e001fa9e209b62f4b6c009fb5.zip riscv-openocd-56b72b33cfc7014e001fa9e209b62f4b6c009fb5.tar.gz riscv-openocd-56b72b33cfc7014e001fa9e209b62f4b6c009fb5.tar.bz2 |
target/dsp563xx: Handle return values
This fixes 'dead assignment' bugs identified by the clang static analyzer.
Change-Id: I140ed55f0043e06a533f45f50a36887614585b04
Signed-off-by: Marc Schink <dev@zapb.de>
Reviewed-on: http://openocd.zylin.com/6323
Tested-by: jenkins
Reviewed-by: Antonio Borneo <borneo.antonio@gmail.com>
-rw-r--r-- | src/target/dsp563xx.c | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/src/target/dsp563xx.c b/src/target/dsp563xx.c index 81ea21e..7305733 100644 --- a/src/target/dsp563xx.c +++ b/src/target/dsp563xx.c @@ -1085,8 +1085,17 @@ static int dsp563xx_poll(struct target *target) if (!dsp563xx->hardware_breakpoints_cleared) { err = dsp563xx_once_reg_write(target->tap, 1, DSP563XX_ONCE_OBCR, 0); + if (err != ERROR_OK) + return err; + err = dsp563xx_once_reg_write(target->tap, 1, DSP563XX_ONCE_OMLR0, 0); + if (err != ERROR_OK) + return err; + err = dsp563xx_once_reg_write(target->tap, 1, DSP563XX_ONCE_OMLR1, 0); + if (err != ERROR_OK) + return err; + dsp563xx->hardware_breakpoints_cleared = 1; } |