lynxis lazus has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/42435?usp=email )


Change subject: dfu: set poll timeout to 128 ms to reduce EPIPE errors
......................................................................

dfu: set poll timeout to 128 ms to reduce EPIPE errors

The previous 10 ms poll timeout creates ERROR_PIPE errors when the
octsim is connected to a more busy bus.
E.g. when a network device is connected to the same bus or when flashing
multiple OCTSIMs at the same time.

>From usb captures 70 ms should be already enough, but adding an additional
safety margin.
It is unclear why the EPIPE errors happens.

Related: SYS#7910
Change-Id: I4abe812e17194f37ce5d97fd1d866aeebd1d4239
---
M usb/class/dfu/device/dfudf.c
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-asf4-dfu refs/changes/35/42435/1

diff --git a/usb/class/dfu/device/dfudf.c b/usb/class/dfu/device/dfudf.c
index 4b010bd..87a97b8 100644
--- a/usb/class/dfu/device/dfudf.c
+++ b/usb/class/dfu/device/dfudf.c
@@ -187,7 +187,7 @@
                        break;
                }
                response[0] = dfu_status; // set status
-               response[1] = 10; // set poll timeout (24 bits, in 
milliseconds) to small value for periodical poll
+               response[1] = 128; // set poll timeout (24 bits, in 
milliseconds) to small value for periodical poll
                response[2] = 0; // set poll timeout (24 bits, in milliseconds) 
to small value for periodical poll
                response[3] = 0; // set poll timeout (24 bits, in milliseconds) 
to small value for periodical poll
                response[4] = dfu_state; // set state (post-transition)

--
To view, visit https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/42435?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings?usp=email

Gerrit-MessageType: newchange
Gerrit-Project: osmo-asf4-dfu
Gerrit-Branch: master
Gerrit-Change-Id: I4abe812e17194f37ce5d97fd1d866aeebd1d4239
Gerrit-Change-Number: 42435
Gerrit-PatchSet: 1
Gerrit-Owner: lynxis lazus <[email protected]>

Reply via email to