blackbeard420/esp-idf
1
0
Fork 0
mirror of https://github.com/espressif/esp-idf synced 2025-03-09 17:19:09 -04:00
Code Issues Projects Releases Wiki Activity

fix(usb/hal/dwc): Correct host channel number calculation

Browse Source 
The hardware field `ghwcfg2.numhstchnl` is zero-based, meaning the actual
number of available host channels is `numhstchnl + 1`. This off-by-one
error caused the USB Host controller to report N-1 channels instead of N,
leading to premature "No more HCD channels available" errors when
connecting multiple devices.

This issue affects ESP32-S2, ESP32-S3, and ESP32-P4.
This commit is contained in:
igor.masar 2025-02-20 13:41:34 +01:00 committed by BOT
parent aa424235e2
commit dfd8098716
3 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
components/hal/esp32p4/include/hal/usb_dwc_ll.h
View File

@ -362,7 +362,7 @@ static inline unsigned usb_dwc_ll_ghwcfg_get_hsphy_type(usb_dwc_dev_t *hw)
static inline unsigned usb_dwc_ll_ghwcfg_get_channel_num(usb_dwc_dev_t *hw)
{
return hw->ghwcfg2_reg.numhstchnl;
return hw->ghwcfg2_reg.numhstchnl + 1;
}
// --------------------------- HPTXFSIZ Register -------------------------------

2
components/hal/esp32s2/include/hal/usb_dwc_ll.h
View File

@ -361,7 +361,7 @@ static inline unsigned usb_dwc_ll_ghwcfg_get_hsphy_type(usb_dwc_dev_t *hw)
static inline unsigned usb_dwc_ll_ghwcfg_get_channel_num(usb_dwc_dev_t *hw)
{
return hw->ghwcfg2_reg.numhstchnl;
return hw->ghwcfg2_reg.numhstchnl + 1;
}
// --------------------------- HPTXFSIZ Register -------------------------------

2
components/hal/esp32s3/include/hal/usb_dwc_ll.h
View File

@ -361,7 +361,7 @@ static inline unsigned usb_dwc_ll_ghwcfg_get_hsphy_type(usb_dwc_dev_t *hw)
static inline unsigned usb_dwc_ll_ghwcfg_get_channel_num(usb_dwc_dev_t *hw)
{
return hw->ghwcfg2_reg.numhstchnl;
return hw->ghwcfg2_reg.numhstchnl + 1;
}
// --------------------------- HPTXFSIZ Register -------------------------------