USBD Driver - Bug when removing ISO endpoint

RE: USBD Driver - Bug when removing ISO endpoint

Håkon Alseth — Mon, 21 Jan 2019 09:07:43 GMT

Hi,

ISO transfers will occur anyway as long as the bus is available, so aborting it may not make much sense. I still see this as an issue from our side, as the _disable() should not call _abort() on ISO EPs. I'll forward this to our USB developers so they can have a look, and see if any improvement can be made.

Cheers,

Håkon

RE: USBD Driver - Bug when removing ISO endpoint

George Beckstien — Sat, 19 Jan 2019 17:30:29 GMT

After looking more closely at the datasheet, it seems the `SIZE.ISOOUT` register is read-only. So maybe this function is correct in ASSERT-ing when given the ISOOUT endpoint as an argument. However, is it possible to remove an ISO endpoint without encountering this ASSERT since calling `nrf_drv_usbd_ep_disable` causes this?

It seems that `nrf_drv_usbd_ep_disable` should make sure not to call `usbd_ep_abort` on ISO endpoints... not sure if there is a way to abort ISO transfers.