This is an automated email from Gerrit.

"Jacek Wuwer <jacek...@gmail.com>" just uploaded a new patch set to Gerrit, 
which you can find at https://review.openocd.org/c/openocd/+/7721

-- gerrit

commit 5822dade91c2e356cb916a63de0689dcf3f24c61
Author: Jacek Wuwer <jacek...@gmail.com>
Date:   Mon May 29 17:46:08 2023 +0200

    jtag/vdebug: fix endianness support
    
    This change fixes endianness support in the driver.
    
    Change-Id: Ida360bb58e988cea0a66fdc79e1610b528846fc4
    Signed-off-by: Jacek Wuwer <jacek...@gmail.com>

diff --git a/src/jtag/drivers/vdebug.c b/src/jtag/drivers/vdebug.c
index 9223be23f2..4187c9876b 100644
--- a/src/jtag/drivers/vdebug.c
+++ b/src/jtag/drivers/vdebug.c
@@ -272,7 +272,7 @@ static int vdebug_socket_open(char *server_addr, uint32_t 
port)
                LOG_ERROR("socket_open: cannot resolve address %s, error %d", 
server_addr, vdebug_socket_error());
                rc = VD_ERR_SOC_ADDR;
        } else {
-               buf_set_u32((uint8_t *)ainfo->ai_addr->sa_data, 0, 16, 
htons(port));
+               h_u16_to_be((uint8_t *)ainfo->ai_addr->sa_data, port);
                if (connect(hsock, ainfo->ai_addr, sizeof(struct sockaddr)) < 
0) {
                        LOG_ERROR("socket_open: cannot connect to %s:%d, error 
%d", server_addr, port, vdebug_socket_error());
                        rc = VD_ERR_SOC_CONN;
@@ -1125,7 +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 (pbuf->waddr)
+       if (le_to_h_u16(pbuf->waddr))
                return vdebug_run_reg_queue(vdc.hsocket, pbuf, 
le_to_h_u16(pbuf->waddr));
 
        return ERROR_OK;

-- 

Reply via email to