refactor(hcd_dwc): Apply delay for all targets

roma-jam · roma-jam · commit 898f881226e5 · 2025-05-07T11:14:30.000+02:00
diff --git a/components/usb/hcd_dwc.c b/components/usb/hcd_dwc.c
@@ -390,13 +390,11 @@ static inline bool _buffer_check_done(pipe_t *pipe)
     if (pipe->ep_char.type != USB_DWC_XFER_TYPE_CTRL) {
         return true;
     }
-#if (CONFIG_IDF_TARGET_ESP32S2 || CONFIG_IDF_TARGET_ESP32S3)
     // The HW can't handle two transactions with preamble in one frame.
     // TODO: IDF-12986
     if (pipe->ep_char.ls_via_fs_hub) {
         esp_rom_delay_us(1000);
     }
-#endif // CONFIG_IDF_TARGET_ESP32S2 || CONFIG_IDF_TARGET_ESP32S3
     dma_buffer_block_t *buffer_inflight = pipe->buffers[pipe->multi_buffer_control.rd_idx];
     return (buffer_inflight->flags.ctrl.cur_stg == 2);
 }

Original file line number	Diff line number	Diff line change
`@@ -390,13 +390,11 @@ static inline bool _buffer_check_done(pipe_t *pipe)`
`390`	`390`	`if (pipe->ep_char.type != USB_DWC_XFER_TYPE_CTRL) {`
`391`	`391`	`return true;`
`392`	`392`	`}`
`393`		`-#if (CONFIG_IDF_TARGET_ESP32S2 \|\| CONFIG_IDF_TARGET_ESP32S3)`
`394`	`393`	`// The HW can't handle two transactions with preamble in one frame.`
`395`	`394`	`// TODO: IDF-12986`
`396`	`395`	`if (pipe->ep_char.ls_via_fs_hub) {`
`397`	`396`	`esp_rom_delay_us(1000);`
`398`	`397`	`}`
`399`		`-#endif // CONFIG_IDF_TARGET_ESP32S2 \|\| CONFIG_IDF_TARGET_ESP32S3`
`400`	`398`	`dma_buffer_block_t *buffer_inflight = pipe->buffers[pipe->multi_buffer_control.rd_idx];`
`401`	`399`	`return (buffer_inflight->flags.ctrl.cur_stg == 2);`
`402`	`400`	`}`