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

Jump to bottom

Update vendor_device.c fifo mutex config similar to cdc_device.c. #2443

Merged

HiFiPhile merged 1 commit into hathach:master from mamiral:tud_vendor_mutex

Feb 1, 2024

Contributor

mamiral commented Jan 31, 2024

Describe the PR
Modified vendor_device.c fifo mutex creation/configuration in the spirit of cdc_device.c.
This hopefully fixes #900.

Works on my machine (device STM32G474-DPOW1). ;)


          Update vendor_device.c fifo mutex config similar to cdc_device.c.

82776aa

Collaborator

HiFiPhile commented Feb 1, 2024

Thanks for the PR, but it only does simplify the code.
Since 876f49f the locking mechanism is already introduced.

HiFiPhile self-requested a review

February 1, 2024 09:56

HiFiPhile approved these changes

View reviewed changes

HiFiPhile merged commit f505cc2 into hathach:master

47 of 49 checks passed

mamiral deleted the tud_vendor_mutex branch

February 1, 2024 10:19

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet