usb: wait 100ms for transfers to finish

xobs · xobs · commit 037d35011616 · 2020-04-22T16:35:45.000+08:00
The host may pipeline several requests which could possibly finish
before we've acknowledged them.  Wait 100 ms in order to acknowledge
all of these LUNs.

Signed-off-by: Sean Cross &lt;sean@xobs.io&gt;
diff --git a/src/usb/usb.c b/src/usb/usb.c
@@ -97,7 +97,13 @@ void usb_teardown(void)
 {
   if ( NRF_USBD->ENABLE )
   {
-    // Abort all transfers
+    // Wait for transfers to finish.  This works around an issue wherein we
+    // disconnect while there are still pipelined transfers going.
+    unsigned int i;
+    for (i = 0; i < 100; i++) {
+      tud_task();
+      NRFX_DELAY_MS(1);
+    }
 
     // Disable pull up
     nrf_usbd_pullup_disable(NRF_USBD);

Original file line number	Diff line number	Diff line change
`@@ -97,7 +97,13 @@ void usb_teardown(void)`
`97`	`97`	`{`
`98`	`98`	`if ( NRF_USBD->ENABLE )`
`99`	`99`	`{`
`100`		`- // Abort all transfers`
	`100`	`+ // Wait for transfers to finish. This works around an issue wherein we`
	`101`	`+ // disconnect while there are still pipelined transfers going.`
	`102`	`+ unsigned int i;`
	`103`	`+ for (i = 0; i < 100; i++) {`
	`104`	`+ tud_task();`
	`105`	`+ NRFX_DELAY_MS(1);`
	`106`	`+ }`
`101`	`107`
`102`	`108`	`// Disable pull up`
`103`	`109`	`nrf_usbd_pullup_disable(NRF_USBD);`