Harald Welte has submitted this change and it was merged. ( 
https://gerrit.osmocom.org/12514 )

Change subject: enable flashing by providing real downloaded data length
......................................................................

enable flashing by providing real downloaded data length

Change-Id: I6f16170a4b41a70d89954e2c4a841efc67a26786
---
M usb/class/dfu/device/dfudf.c
1 file changed, 1 insertion(+), 3 deletions(-)

Approvals:
  Jenkins Builder: Verified
  Harald Welte: Looks good to me, approved



diff --git a/usb/class/dfu/device/dfudf.c b/usb/class/dfu/device/dfudf.c
index aefcab4..41e235b 100644
--- a/usb/class/dfu/device/dfudf.c
+++ b/usb/class/dfu/device/dfudf.c
@@ -245,11 +245,9 @@
                } else { // there is data to be flash
                        if (USB_SETUP_STAGE == stage) { // there will be data 
to be flash
                                to_return = usbdc_xfer(ep, dfu_download_data, 
req->wLength, false); // send ack to the setup request to get the data
-                               //to_return = usbdc_xfer(ep, NULL, 0, false); 
// send ACK
                        } else { // now there is data to be flashed
                                dfu_download_progress = req->wValue * 
sizeof(dfu_download_data); // remember which block to flash
-                               dfu_download_length = 0;
-                               //dfu_download_length = req->wLength; // 
remember the data size to be flash
+                               dfu_download_length = req->wLength; // remember 
the data size to be flash
                                dfu_state = USB_DFU_STATE_DFU_DNLOAD_SYNC; // 
go to sync state
                                to_return = usbdc_xfer(ep, NULL, 0, false); // 
ACK the data
                                // we let the main application flash the data 
because this can be long and would stall the USB ISR

--
To view, visit https://gerrit.osmocom.org/12514
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-asf4-dfu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I6f16170a4b41a70d89954e2c4a841efc67a26786
Gerrit-Change-Number: 12514
Gerrit-PatchSet: 3
Gerrit-Owner: Kévin Redon <[email protected]>
Gerrit-Reviewer: Harald Welte <[email protected]>
Gerrit-Reviewer: Jenkins Builder (1000002)

Reply via email to