Skip to content

ST Synopsys port lacks SOF support #1684

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.

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

Open
1 task done
battlesnake opened this issue Oct 15, 2022 · 7 comments
Open
1 task done

ST Synopsys port lacks SOF support #1684

battlesnake opened this issue Oct 15, 2022 · 7 comments

Comments

@battlesnake
Copy link
Contributor

Related area

ST Synopsys port

Hardware specification

Cortex M7, STM32H750, USB3300, custom board

Is your feature request related to a problem?

The port currently lacks support for USB SOF frames.

Describe the solution you'd like

I will provide a PR with my proposed solution

I have checked existing issues, dicussion and documentation

  • I confirm I have checked existing issues, dicussion and documentation.
@battlesnake
Copy link
Contributor Author

PR: #1685

@PanRe
Copy link
Collaborator

PanRe commented Oct 18, 2022

Just a short question, the driver in portable/synopsis/dwc2 does not work for you?

@battlesnake
Copy link
Contributor Author

battlesnake commented Oct 19, 2022

I didn't use try it. I didn't find info in the docs about which port(s) to use, so I tried st/synopsys (since using ST part), and it worked after adding the SOF feature.

@battlesnake
Copy link
Contributor Author

Besides (presumably) supporting SOF already, what are other differences of that synopsys/dw2 port vs. st/synopsys?

@PanRe
Copy link
Collaborator

PanRe commented Oct 20, 2022

It fuses the device driver for multiple MCUs as far as i know. I don't know why the st/synopsis driver is still there, maybe due to backward compatibilities.

@battlesnake
Copy link
Contributor Author

battlesnake commented Oct 20, 2022

Thanks, I'll give it a try in an hour or so [once I've hopefully got UAC2 interrupt endpoint working and delivering control-change messages to the host].

@battlesnake
Copy link
Contributor Author

Using synopsis/dwc2, I get into a crash-loop.
Almost certainly due to configuration on my side though.

I'll go back to using st/synopsys for now.
Once I've got my UAC2 interrupt status endpoint working, then I'll try synopsys/dwc2 again and see if I can either fix the issue in my code, or file a PR if it's due to a bug in tinyusb.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: 📋 Backlog
Development

No branches or pull requests

2 participants