From mboxrd@z Thu Jan 1 00:00:00 1970 From: Patrick Delaunay Date: Fri, 16 Dec 2016 18:41:30 +0100 Subject: [U-Boot] [PATCH 0/2] Series to clean handle_XXX() in f_dfu.c Message-ID: <1481910092-13844-1-git-send-email-patrick.delaunay73@gmail.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 1/ DFU_GETSTATE response error when the DFU device state is requested using libusb with DFU_GETSTATUS a error is generated : LIBUSB_ERROR_IO I see the issue with my programmer tools based on dfu with libusb and I don't understood the actual code in U-Boot (req->actual is updated but not req->lenght ?) With the proposed patch, I aligned the code on the GETSTATUS behavior: the correct lenght is provided in req->lenght as the other answer and the issue is solved. PS: the issue not see with dfu-util as the existing function dfu_get_state() is never called in main (with dfu-util 0.9) But I can reproduce the same issue with patched version of dfu-util to add call to this function, on the dfu-util master branch diff --git a/src/main.c b/src/main.c index 7f31d4c..d99ace9 100644 --- a/src/main.c +++ b/src/main.c @@ -535,6 +535,13 @@ dfustate: } status_again: + printf("Determining device state: "); + ret = dfu_get_state(dfu_root->dev_handle, dfu_root->interface); + if (ret < 0) { + errx(EX_IOERR, "error get_state: %s", libusb_error_name(ret)); + } + printf("state = %s\n", dfu_state_to_string(ret)); + printf("Determining device status: "); ret = dfu_get_status(dfu_root, &status ); if (ret < 0) { 2/ harmonize handle_getstatus() with other function => add return value with size of the buffer as - handle_getstate() add in part 1 or existing functions - handle_dnload() - handle_upload() Patrick Delaunay (2): usb: gadget: dfu: correct size for USB_REQ_DFU_GETSTATE result usb: gadget: dfu: add result for handle_getstatus() drivers/usb/gadget/f_dfu.c | 56 ++++++++++++++++++++-------------------------- drivers/usb/gadget/f_dfu.h | 1 - 2 files changed, 24 insertions(+), 33 deletions(-) -- 1.9.1