This is an automated email from Gerrit.

Tomas Vanek ([email protected]) just uploaded a new patch set to Gerrit, which you 
can find at http://openocd.zylin.com/5378

-- gerrit

commit 505b5eb8dbf203e327f8f3135f8e32d93ae9b0e8
Author: Tomas Vanek <[email protected]>
Date:   Fri Dec 20 23:46:46 2019 +0100

    target/avr32_jtag: fix error returns
    
    Discovered by clang static analyzer.
    
    Change-Id: I6c79f6248db774990ddb42c0dacdb621651ed69e
    Signed-off-by: Tomas Vanek <[email protected]>

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,

-- 


_______________________________________________
OpenOCD-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openocd-devel

Reply via email to