This is an automated email from Gerrit.

Adrian M Negreanu ([email protected]) just uploaded a new patch set to 
Gerrit, which you can find at http://openocd.zylin.com/5920

-- gerrit

commit 95674ec9f2c5d3c58bae2ccd76aad64b9857f543
Author: Adrian Negreanu <[email protected]>
Date:   Wed Nov 4 17:23:13 2020 +0200

    cmsis-dap, hid: set PACKET_SIZE according to the usb-hid specs
    
    The USB HID v1.11 spec is reporting bMaxPacketSize0 to be 8, 16, 32 or 64 
bytes.
    
    See pg76, hid1_11.pdf
    
    Change-Id: I0c450adbc7674d5fcd8208dd23062d5cdd209efd
    Signed-off-by: Adrian Negreanu <[email protected]>

diff --git a/src/jtag/drivers/cmsis_dap_usb.c b/src/jtag/drivers/cmsis_dap_usb.c
index 6d55392..590adc3 100644
--- a/src/jtag/drivers/cmsis_dap_usb.c
+++ b/src/jtag/drivers/cmsis_dap_usb.c
@@ -64,7 +64,7 @@ static uint16_t cmsis_dap_pid[MAX_USB_IDS + 1] = { 0 };
 static wchar_t *cmsis_dap_serial;
 static bool swd_mode;
 
-#define PACKET_SIZE       (64 + 1)     /* 64 bytes plus report id */
+#define PACKET_SIZE       64    /* 64 bytes */
 #define USB_TIMEOUT       1000
 
 /* CMSIS-DAP General Commands */
@@ -334,7 +334,7 @@ static int cmsis_dap_usb_open(void)
        /* TODO: HID report descriptor should be parsed instead of
         * hardcoding a match by VID */
        if (target_vid == 0x03eb && target_pid != 0x2145 && target_pid != 
0x2175)
-               packet_size = 512 + 1;
+               packet_size = 512;
 
        cmsis_dap_handle->packet_buffer = malloc(packet_size);
        cmsis_dap_handle->packet_size = packet_size;

-- 


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

Reply via email to