This is an automated email from Gerrit.

"Mark O'Donovan <[email protected]>" just uploaded a new patch set to Gerrit, 
which you can find at https://review.openocd.org/c/openocd/+/9549

-- gerrit

commit fd91fcea116042d4fe15ed0dcaf108cba310b15a
Author: Mark O'Donovan <[email protected]>
Date:   Sun Mar 29 14:43:11 2026 +0100

    target/dsp5680xx: add missing retval assignment
    
    retval was being checked without being assigned.
    
    Change-Id: I3a87abb00159414c869e7ad558566f5c062252b3
    Signed-off-by: Mark O'Donovan <[email protected]>

diff --git a/src/target/dsp5680xx.c b/src/target/dsp5680xx.c
index 66ffabdb24..7d06ce9fef 100644
--- a/src/target/dsp5680xx.c
+++ b/src/target/dsp5680xx.c
@@ -2070,7 +2070,7 @@ int dsp5680xx_f_wr(struct target *t, const uint8_t *b, 
uint32_t a, uint32_t coun
        retval = core_move_long_to_r3(target, address); /* Destination address 
to r3 */
        if (retval != ERROR_OK)
                return retval;
-       core_load_tx_rx_high_addr_to_r0(target); /* TX/RX reg address to r0 */
+       retval = core_load_tx_rx_high_addr_to_r0(target); /* TX/RX reg address 
to r0 */
        if (retval != ERROR_OK)
                return retval;
        retval = core_move_long_to_r2(target, HFM_BASE_ADDR); /* FM base 
address to r2 */

-- 

Reply via email to