diff options
| author | Tomas Vanek <vanekt@fbl.cz> | 2019-12-20 23:46:46 +0100 |
|---|---|---|
| committer | Tomas Vanek <vanekt@fbl.cz> | 2020-01-16 09:45:37 +0000 |
| commit | b04d9c05f0eadc7161465755c05a1f3c1bd6a85b (patch) | |
| tree | 9c5a739f78962e12ac9584e4fd65b4433fdab2fc | |
| parent | 8bb1998e6afe8488c44828e69b5b0d5092f39545 (diff) | |
| download | riscv-openocd-b04d9c05f0eadc7161465755c05a1f3c1bd6a85b.zip riscv-openocd-b04d9c05f0eadc7161465755c05a1f3c1bd6a85b.tar.gz riscv-openocd-b04d9c05f0eadc7161465755c05a1f3c1bd6a85b.tar.bz2 | |
target/avr32_jtag: fix error returns
Fixed only 2 error returns discovered by clang static analyzer.
There are obviously many more missing error tests in avr32_jtag.c
These was not fixed to keep this change minimal.
Not tested with hw.
Change-Id: I6c79f6248db774990ddb42c0dacdb621651ed69e
Signed-off-by: Tomas Vanek <vanekt@fbl.cz>
Reviewed-on: http://openocd.zylin.com/5378
Tested-by: jenkins
Reviewed-by: Oleksij Rempel <linux@rempel-privat.de>
| -rw-r--r-- | src/target/avr32_jtag.c | 10 |
1 files changed, 3 insertions, 7 deletions
diff --git a/src/target/avr32_jtag.c b/src/target/avr32_jtag.c index 6526810..c17fbe7 100644 --- a/src/target/avr32_jtag.c +++ b/src/target/avr32_jtag.c @@ -173,19 +173,15 @@ int avr32_jtag_nexus_read(struct avr32_jtag *jtag_info, { avr32_jtag_set_instr(jtag_info, AVR32_INST_NEXUS_ACCESS); avr32_jtag_nexus_set_address(jtag_info, addr, MODE_READ); - avr32_jtag_nexus_read_data(jtag_info, value); - - return ERROR_OK; - + return avr32_jtag_nexus_read_data(jtag_info, value); } + int avr32_jtag_nexus_write(struct avr32_jtag *jtag_info, uint32_t addr, uint32_t value) { avr32_jtag_set_instr(jtag_info, AVR32_INST_NEXUS_ACCESS); avr32_jtag_nexus_set_address(jtag_info, addr, MODE_WRITE); - avr32_jtag_nexus_write_data(jtag_info, value); - - return ERROR_OK; + return avr32_jtag_nexus_write_data(jtag_info, value); } int avr32_jtag_mwa_set_address(struct avr32_jtag *jtag_info, int slave, |