Skip to content

note potential issues using ep_desc in hcd_edpt_open() #3084

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Apr 18, 2025
Merged

note potential issues using ep_desc in hcd_edpt_open() #3084

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
0d2c08e
note potential issues using ep_desc in hcd_edpt_open()
joelpmichael Apr 16, 2025
7134109
Update hcd_edpt_open() note.
HiFiPhile Apr 18, 2025
b6abc90
Merge remote-tracking branch 'upstream/master' into hcd-template-comm…
HiFiPhile Apr 18, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions src/portable/template/hcd_template.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -106,6 +106,11 @@ void hcd_device_close(uint8_t rhport, uint8_t dev_addr) {
// Open an endpoint
bool hcd_edpt_open(uint8_t rhport, uint8_t dev_addr, tusb_desc_endpoint_t const * ep_desc) {
(void) rhport; (void) dev_addr; (void) ep_desc;

// NOTE: ep_desc is allocated on the stack when called from usbh_edpt_control_open()
// You need to copy the data into a local variable who maintains the state of the endpoint and transfer.
// Check _hcd_data in hcd_dwc2.c for example.

return false;
}

Expand Down
Loading