From: OpenOCD-Gerrit <ope...@us...> - 2013-07-17 14:31:22
|
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "". The branch, master has been updated via 7a3182fa42b0f4c11ccc3c667398da819cad4bd2 (commit) via f684252b2530d86f3a8330940105b86c770231f1 (commit) from 8faa419fad72592970911b7973c61a88e785bcc0 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log ----------------------------------------------------------------- commit 7a3182fa42b0f4c11ccc3c667398da819cad4bd2 Author: Martin Schmölzer <mar...@st...> Date: Thu Jul 4 19:38:04 2013 +0200 OpenULINK firmware: fix name of USB Jump Table symbol Fix a case mistake in the name of the USB Jump Table ("USB_jump_table" vs. "USB_Jump_Table") which led to an assembler error when attempting to build the firmware with recent SDCC versions, because the assembler now treats symbol names as case-sensitive. Successfully tested with ULINK probe and STM32F103 (debug, erase and write flash). Change-Id: I979667b9130efcdccc3ac73269c38f06e0590048 Signed-off-by: Martin Schmölzer <mar...@st...> Reviewed-on: http://openocd.zylin.com/1482 Tested-by: jenkins Reviewed-by: Spencer Oliver <sp...@sp...> diff --git a/src/jtag/drivers/OpenULINK/src/USBJmpTb.a51 b/src/jtag/drivers/OpenULINK/src/USBJmpTb.a51 index 244a1be..8ddd680 100644 --- a/src/jtag/drivers/OpenULINK/src/USBJmpTb.a51 +++ b/src/jtag/drivers/OpenULINK/src/USBJmpTb.a51 @@ -1,5 +1,5 @@ ;--------------------------------------------------------------------------; -; Copyright (C) 2011 by Martin Schmoelzer ; +; Copyright (C) 2011-2013 by Martin Schmoelzer ; ; <mar...@st...> ; ; ; ; This program is free software; you can redistribute it and/or modify ; @@ -28,7 +28,7 @@ .area USB_JV (ABS,OVR) ; Absolute, Overlay .org 0x43 ; USB interrupt (INT2) jumps here USB_AutoVector = #. + 2 - ljmp USB_jump_table + ljmp USB_Jump_Table ;--------------------------------------------------------------------------; ; USB Jump Table ; @@ -36,7 +36,7 @@ USB_AutoVector = #. + 2 .area USB_JT (ABS) ; Absolute placement .org 0x1B00 ; Place jump table at 0x1B00 -USB_jump_table: ; autovector jump table +USB_Jump_Table: ; autovector jump table ljmp _sudav_isr ; Setup Data Available .db 0 ljmp _sof_isr ; Start of Frame commit f684252b2530d86f3a8330940105b86c770231f1 Author: Martin Schmölzer <mar...@st...> Date: Thu Jun 27 19:26:19 2013 +0200 ULINK driver: port from libusb-0.1 to libusb-1.0 API The ULINK driver now uses libusb-1.0 calls (synchronous device I/O) directly (not the common layer provided by libusb1_common.c) Successfully tested with ULINK probe and STM32F103 (debug, erase and write flash). Change-Id: Ic037a3582db85e49a8cc1ec0dd36f629e4757929 Signed-off-by: Martin Schmölzer <mar...@st...> Reviewed-on: http://openocd.zylin.com/1459 Tested-by: jenkins Reviewed-by: Spencer Oliver <sp...@sp...> diff --git a/configure.ac b/configure.ac index 2998ec7..1895813 100644 --- a/configure.ac +++ b/configure.ac @@ -1177,7 +1177,7 @@ fi # check if some driver requires libusb-0.1 need_usb0=no if test $build_vsllink = yes -o $build_usbprog = yes -o \ - $build_rlink = yes -o $build_ulink = yes -o $build_armjtagew = yes; then + $build_rlink = yes -o $build_armjtagew = yes; then need_usb0=yes fi @@ -1191,7 +1191,7 @@ fi # check if some driver requires libusb-1.x need_usb_ng=no if test $build_ftdi = yes -o $build_hladapter_icdi = yes -o \ - $build_hladapter_stlink = yes; then + $build_hladapter_stlink = yes -o $build_ulink = yes; then need_usb_ng=yes fi diff --git a/src/jtag/drivers/ulink.c b/src/jtag/drivers/ulink.c index c195bb2..94f6c31 100644 --- a/src/jtag/drivers/ulink.c +++ b/src/jtag/drivers/ulink.c @@ -1,5 +1,5 @@ /*************************************************************************** - * Copyright (C) 2011 by Martin Schmoelzer * + * Copyright (C) 2011-2013 by Martin Schmoelzer * * <mar...@st...> * * * * This program is free software; you can redistribute it and/or modify * @@ -26,7 +26,7 @@ #include <jtag/interface.h> #include <jtag/commands.h> #include <target/image.h> -#include "usb_common.h" +#include <libusb-1.0/libusb.h> #include "OpenULINK/include/msgtypes.h" /** USB Vendor ID of ULINK device in unconfigured state (no firmware loaded @@ -146,7 +146,8 @@ struct ulink_cmd { /** Describes one driver instance */ struct ulink { - struct usb_dev_handle *usb_handle; + struct libusb_context *libusb_ctx; + struct libusb_device_handle *usb_device_handle; enum ulink_type type; int delay_scan_in; /* /< Delay value for SCAN_IN commands */ @@ -167,7 +168,7 @@ int ulink_usb_open(struct ulink **device); int ulink_usb_close(struct ulink **device); /* ULINK MCU (Cypress EZ-USB) specific functions */ -int ulink_cpu_reset(struct ulink *device, char reset_bit); +int ulink_cpu_reset(struct ulink *device, unsigned char reset_bit); int ulink_load_firmware_and_renumerate(struct ulink **device, char *filename, uint32_t delay); int ulink_load_firmware(struct ulink *device, char *filename); @@ -258,30 +259,46 @@ struct ulink *ulink_handle; /** * Opens the ULINK device and claims its USB interface. * + * Currently, only the original ULINK is supported + * * @param device pointer to struct ulink identifying ULINK driver instance. * @return on success: ERROR_OK * @return on failure: ERROR_FAIL */ int ulink_usb_open(struct ulink **device) { - int ret; - struct usb_dev_handle *usb_handle; + ssize_t num_devices, i; + bool found; + libusb_device **usb_devices; + struct libusb_device_descriptor usb_desc; + struct libusb_device_handle *usb_device_handle; - /* Currently, only original ULINK is supported */ - uint16_t vids[] = { ULINK_VID, 0 }; - uint16_t pids[] = { ULINK_PID, 0 }; + num_devices = libusb_get_device_list((*device)->libusb_ctx, &usb_devices); - ret = jtag_usb_open(vids, pids, &usb_handle); + if (num_devices <= 0) + return ERROR_FAIL; - if (ret != ERROR_OK) - return ret; + found = false; + for (i = 0; i < num_devices; i++) { + if (libusb_get_device_descriptor(usb_devices[i], &usb_desc) != 0) + continue; + else if (usb_desc.idVendor == ULINK_VID && usb_desc.idProduct == ULINK_PID) { + found = true; + break; + } + } + + if (!found) + return ERROR_FAIL; - ret = usb_claim_interface(usb_handle, 0); + if (libusb_open(usb_devices[i], &usb_device_handle) != 0) + return ERROR_FAIL; + libusb_free_device_list(usb_devices, 1); - if (ret != 0) - return ret; + if (libusb_claim_interface(usb_device_handle, 0) != 0) + return ERROR_FAIL; - (*device)->usb_handle = usb_handle; + (*device)->usb_device_handle = usb_device_handle; (*device)->type = ULINK_1; return ERROR_OK; @@ -296,13 +313,12 @@ int ulink_usb_open(struct ulink **device) */ int ulink_usb_close(struct ulink **device) { - if (usb_release_interface((*device)->usb_handle, 0) != 0) + if (libusb_release_interface((*device)->usb_device_handle, 0) != 0) return ERROR_FAIL; - if (usb_close((*device)->usb_handle) != 0) - return ERROR_FAIL; + libusb_close((*device)->usb_device_handle); - (*device)->usb_handle = NULL; + (*device)->usb_device_handle = NULL; return ERROR_OK; } @@ -318,12 +334,12 @@ int ulink_usb_close(struct ulink **device) * @return on success: ERROR_OK * @return on failure: ERROR_FAIL */ -int ulink_cpu_reset(struct ulink *device, char reset_bit) +int ulink_cpu_reset(struct ulink *device, unsigned char reset_bit) { int ret; - ret = usb_control_msg(device->usb_handle, - (USB_ENDPOINT_OUT | USB_TYPE_VENDOR | USB_RECIP_DEVICE), + ret = libusb_control_transfer(device->usb_device_handle, + (LIBUSB_ENDPOINT_OUT | LIBUSB_REQUEST_TYPE_VENDOR | LIBUSB_RECIPIENT_DEVICE), REQUEST_FIRMWARE_LOAD, CPUCS_REG, 0, &reset_bit, 1, USB_TIMEOUT); /* usb_control_msg() returns the number of bytes transferred during the @@ -467,9 +483,9 @@ int ulink_write_firmware_section(struct ulink *device, else chunk_size = bytes_remaining; - ret = usb_control_msg(device->usb_handle, - (USB_ENDPOINT_OUT | USB_TYPE_VENDOR | USB_RECIP_DEVICE), - REQUEST_FIRMWARE_LOAD, addr, FIRMWARE_ADDR, (char *)data_ptr, + ret = libusb_control_transfer(device->usb_device_handle, + (LIBUSB_ENDPOINT_OUT | LIBUSB_REQUEST_TYPE_VENDOR | LIBUSB_RECIPIENT_DEVICE), + REQUEST_FIRMWARE_LOAD, addr, FIRMWARE_ADDR, (unsigned char *)data_ptr, chunk_size, USB_TIMEOUT); if (ret != (int)chunk_size) { @@ -694,7 +710,7 @@ int ulink_append_queue(struct ulink *device, struct ulink_cmd *ulink_cmd) int ulink_execute_queued_commands(struct ulink *device, int timeout) { struct ulink_cmd *current; - int ret, i, index_out, index_in, count_out, count_in; + int ret, i, index_out, index_in, count_out, count_in, transferred; uint8_t buffer[64]; #ifdef _DEBUG_JTAG_IO_ @@ -719,20 +735,20 @@ int ulink_execute_queued_commands(struct ulink *device, int timeout) } /* Send packet to ULINK */ - ret = usb_bulk_write(device->usb_handle, (2 | USB_ENDPOINT_OUT), - (char *)buffer, count_out, timeout); - if (ret < 0) + ret = libusb_bulk_transfer(device->usb_device_handle, (2 | LIBUSB_ENDPOINT_OUT), + (unsigned char *)buffer, count_out, &transferred, timeout); + if (ret != 0) return ERROR_FAIL; - if (ret != count_out) + if (transferred != count_out) return ERROR_FAIL; /* Wait for response if commands contain IN payload data */ if (count_in > 0) { - ret = usb_bulk_read(device->usb_handle, (2 | USB_ENDPOINT_IN), - (char *)buffer, 64, timeout); - if (ret < 0) + ret = libusb_bulk_transfer(device->usb_device_handle, (2 | LIBUSB_ENDPOINT_IN), + (unsigned char *)buffer, 64, &transferred, timeout); + if (ret != 0) return ERROR_FAIL; - if (ret != count_in) + if (transferred != count_in) return ERROR_FAIL; /* Write back IN payload data */ @@ -2142,17 +2158,17 @@ static int ulink_speed_div(int speed, int *khz) */ static int ulink_init(void) { - int ret; + int ret, transferred; char str_manufacturer[20]; bool download_firmware = false; - uint8_t *dummy; + unsigned char *dummy; uint8_t input_signals, output_signals; ulink_handle = calloc(1, sizeof(struct ulink)); if (ulink_handle == NULL) return ERROR_FAIL; - usb_init(); + libusb_init(&ulink_handle->libusb_ctx); ret = ulink_usb_open(&ulink_handle); if (ret != ERROR_OK) { @@ -2163,7 +2179,7 @@ static int ulink_init(void) } /* Get String Descriptor to determine if firmware needs to be loaded */ - ret = usb_get_string_simple(ulink_handle->usb_handle, 1, str_manufacturer, 20); + ret = libusb_get_string_descriptor_ascii(ulink_handle->usb_device_handle, 1, (unsigned char *)str_manufacturer, 20); if (ret < 0) { /* Could not get descriptor -> Unconfigured or original Keil firmware */ download_firmware = true; @@ -2202,12 +2218,12 @@ static int ulink_init(void) * shut down by the user via Ctrl-C. Try to retrieve this Bulk IN packet. */ dummy = calloc(64, sizeof(uint8_t)); - ret = usb_bulk_read(ulink_handle->usb_handle, (2 | USB_ENDPOINT_IN), - (char *)dummy, 64, 200); + ret = libusb_bulk_transfer(ulink_handle->usb_device_handle, (2 | LIBUSB_ENDPOINT_IN), + dummy, 64, &transferred, 200); free(dummy); - if (ret < 0) { + if (ret != 0 || transferred == 0) { /* Bulk IN transfer failed -> unrecoverable error condition */ LOG_ERROR("Cannot communicate with ULINK device. Disconnect ULINK from " "the USB port and re-connect, then re-run OpenOCD"); ----------------------------------------------------------------------- Summary of changes: configure.ac | 4 +- src/jtag/drivers/OpenULINK/src/USBJmpTb.a51 | 6 +- src/jtag/drivers/ulink.c | 100 +++++++++++++++----------- 3 files changed, 63 insertions(+), 47 deletions(-) hooks/post-receive -- |