Skip to content

Commit ef41652

Browse files
committed
drivers/arm-jtag-ew: switch to libusb1
Convert the driver from libusb0 to libusb1. Change-Id: Idef0b6cf10fab583bc8d13b3b4fadd5cc368c090 Signed-off-by: Antonio Borneo <[email protected]> Reviewed-on: http://openocd.zylin.com/5990 Tested-by: jenkins
1 parent 5c53e2c commit ef41652

File tree

2 files changed

+29
-21
lines changed

2 files changed

+29
-21
lines changed

Diff for: configure.ac

+2-1
Original file line numberDiff line numberDiff line change
@@ -122,12 +122,13 @@ m4_define([USB1_ADAPTERS],
122122
[[cmsis_dap_v2], [CMSIS-DAP v2 Compliant Debugger], [CMSIS_DAP_USB]],
123123
[[osbdm], [OSBDM (JTAG only) Programmer], [OSBDM]],
124124
[[opendous], [eStick/opendous JTAG Programmer], [OPENDOUS]],
125+
[[armjtagew], [Olimex ARM-JTAG-EW Programmer], [ARMJTAGEW]],
125126
[[rlink], [Raisonance RLink JTAG Programmer], [RLINK]],
126127
[[usbprog], [USBProg JTAG Programmer], [USBPROG]],
127128
[[aice], [Andes JTAG Programmer], [AICE]]])
128129

129130
m4_define([USB0_ADAPTERS],
130-
[[[armjtagew], [Olimex ARM-JTAG-EW Programmer], [ARMJTAGEW]]])
131+
[])
131132

132133
m4_define([HIDAPI_ADAPTERS],
133134
[[[cmsis_dap], [CMSIS-DAP Compliant Debugger], [CMSIS_DAP_HID]],

Diff for: src/jtag/drivers/arm-jtag-ew.c

+27-20
Original file line numberDiff line numberDiff line change
@@ -22,8 +22,7 @@
2222

2323
#include <jtag/interface.h>
2424
#include <jtag/commands.h>
25-
#include <usb.h>
26-
#include "usb_common.h"
25+
#include "libusb_helper.h"
2726

2827
#define USB_VID 0x15ba
2928
#define USB_PID 0x001e
@@ -75,7 +74,7 @@ static void armjtagew_tap_append_scan(int length, uint8_t *buffer, struct scan_c
7574

7675
/* ARM-JTAG-EW lowlevel functions */
7776
struct armjtagew {
78-
struct usb_dev_handle *usb_handle;
77+
struct libusb_device_handle *usb_handle;
7978
};
8079

8180
static struct armjtagew *armjtagew_usb_open(void);
@@ -684,35 +683,37 @@ static int armjtagew_tap_execute(void)
684683

685684
static struct armjtagew *armjtagew_usb_open(void)
686685
{
687-
usb_init();
688-
689686
const uint16_t vids[] = { USB_VID, 0 };
690687
const uint16_t pids[] = { USB_PID, 0 };
691-
struct usb_dev_handle *dev;
692-
if (jtag_usb_open(vids, pids, &dev) != ERROR_OK)
688+
struct libusb_device_handle *dev;
689+
690+
if (jtag_libusb_open(vids, pids, NULL, &dev, NULL) != ERROR_OK)
693691
return NULL;
694692

695693
struct armjtagew *result = malloc(sizeof(struct armjtagew));
696694
result->usb_handle = dev;
697695

698696
#if 0
699-
/* usb_set_configuration required under win32 */
700-
usb_set_configuration(dev, dev->config[0].bConfigurationValue);
697+
/* libusb_set_configuration required under win32 */
698+
struct libusb_config_descriptor *config;
699+
struct libusb_device *usb_dev = libusb_get_device(dev);
700+
libusb_get_config_descriptor(usb_dev, 0, &config);
701+
libusb_set_configuration(dev, config->bConfigurationValue);
701702
#endif
702-
usb_claim_interface(dev, 0);
703+
libusb_claim_interface(dev, 0);
703704
#if 0
704705
/*
705706
* This makes problems under Mac OS X. And is not needed
706707
* under Windows. Hopefully this will not break a linux build
707708
*/
708-
usb_set_altinterface(dev, 0);
709+
libusb_set_interface_alt_setting(dev, 0, 0);
709710
#endif
710711
return result;
711712
}
712713

713714
static void armjtagew_usb_close(struct armjtagew *armjtagew)
714715
{
715-
usb_close(armjtagew->usb_handle);
716+
libusb_close(armjtagew->usb_handle);
716717
free(armjtagew);
717718
}
718719

@@ -725,13 +726,13 @@ static int armjtagew_usb_message(struct armjtagew *armjtagew, int out_length, in
725726
if (result == out_length) {
726727
result = armjtagew_usb_read(armjtagew, in_length);
727728
if (result != in_length) {
728-
LOG_ERROR("usb_bulk_read failed (requested=%d, result=%d)",
729+
LOG_ERROR("jtag_libusb_bulk_read failed (requested=%d, result=%d)",
729730
in_length,
730731
result);
731732
return -1;
732733
}
733734
} else {
734-
LOG_ERROR("usb_bulk_write failed (requested=%d, result=%d)", out_length, result);
735+
LOG_ERROR("jtag_libusb_bulk_write failed (requested=%d, result=%d)", out_length, result);
735736
return -1;
736737
}
737738
return 0;
@@ -741,6 +742,7 @@ static int armjtagew_usb_message(struct armjtagew *armjtagew, int out_length, in
741742
static int armjtagew_usb_write(struct armjtagew *armjtagew, int out_length)
742743
{
743744
int result;
745+
int transferred;
744746

745747
if (out_length > ARMJTAGEW_OUT_BUFFER_SIZE) {
746748
LOG_ERROR("armjtagew_write illegal out_length=%d (max=%d)",
@@ -749,29 +751,34 @@ static int armjtagew_usb_write(struct armjtagew *armjtagew, int out_length)
749751
return -1;
750752
}
751753

752-
result = usb_bulk_write(armjtagew->usb_handle, ARMJTAGEW_EPT_BULK_OUT,
753-
(char *)usb_out_buffer, out_length, ARMJTAGEW_USB_TIMEOUT);
754+
result = jtag_libusb_bulk_write(armjtagew->usb_handle, ARMJTAGEW_EPT_BULK_OUT,
755+
(char *)usb_out_buffer, out_length, ARMJTAGEW_USB_TIMEOUT, &transferred);
754756

755757
LOG_DEBUG_IO("armjtagew_usb_write, out_length = %d, result = %d", out_length, result);
756758

757759
#ifdef _DEBUG_USB_COMMS_
758760
armjtagew_debug_buffer(usb_out_buffer, out_length);
759761
#endif
760-
return result;
762+
if (result != ERROR_OK)
763+
return -1;
764+
return transferred;
761765
}
762766

763767
/* Read data from USB into in_buffer. */
764768
static int armjtagew_usb_read(struct armjtagew *armjtagew, int exp_in_length)
765769
{
766-
int result = usb_bulk_read(armjtagew->usb_handle, ARMJTAGEW_EPT_BULK_IN,
767-
(char *)usb_in_buffer, exp_in_length, ARMJTAGEW_USB_TIMEOUT);
770+
int transferred;
771+
int result = jtag_libusb_bulk_read(armjtagew->usb_handle, ARMJTAGEW_EPT_BULK_IN,
772+
(char *)usb_in_buffer, exp_in_length, ARMJTAGEW_USB_TIMEOUT, &transferred);
768773

769774
LOG_DEBUG_IO("armjtagew_usb_read, result = %d", result);
770775

771776
#ifdef _DEBUG_USB_COMMS_
772777
armjtagew_debug_buffer(usb_in_buffer, result);
773778
#endif
774-
return result;
779+
if (result != ERROR_OK)
780+
return -1;
781+
return transferred;
775782
}
776783

777784
#ifdef _DEBUG_USB_COMMS_

0 commit comments

Comments
 (0)