This is an automated email from Gerrit.

"Kirill Radkin <kirill.rad...@syntacore.com>" just uploaded a new patch set to 
Gerrit, which you can find at https://review.openocd.org/c/openocd/+/7933

-- gerrit

commit a6714a49e526fa0a20ba309d91ef65ad5d31e9be
Author: Kirill Radkin <kirill.rad...@syntacore.com>
Date:   Wed Oct 4 17:06:44 2023 +0300

    drivers/vdebug.c: Remove unnecessary check for pbuf->waddr
    
    pbuf->waddr is fixed-length array, it can't be NULL
    
    Change-Id: Ic89ff9ebee3a6f85574e7febd7fb8fc6014f6fe4
    Signed-off-by: Kirill Radkin <kirill.rad...@syntacore.com>

diff --git a/src/jtag/drivers/vdebug.c b/src/jtag/drivers/vdebug.c
index d2311b2ea8..8da3655e0e 100644
--- a/src/jtag/drivers/vdebug.c
+++ b/src/jtag/drivers/vdebug.c
@@ -1125,10 +1125,7 @@ static int vdebug_dap_queue_ap_abort(struct adiv5_dap 
*dap, uint8_t *ack)
 
 static int vdebug_dap_run(struct adiv5_dap *dap)
 {
-       if (le_to_h_u16(pbuf->waddr))
-               return vdebug_run_reg_queue(vdc.hsocket, pbuf, 
le_to_h_u16(pbuf->waddr));
-
-       return ERROR_OK;
+       return vdebug_run_reg_queue(vdc.hsocket, pbuf, 
le_to_h_u16(pbuf->waddr));
 }
 
 COMMAND_HANDLER(vdebug_set_server)

-- 

Reply via email to