Skip to content
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

added protocol analyzer for USB #168

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

StuckNomad
Copy link

  • Added Protocol Analyzer for USB 2.0.
  • Supports both Low-Speed and Full-Speed.
Mode Preferred Sampling rate
Low Speed 15Mhz
High Speed 100Mhz
Mode Preferred triggers
Low Speed Positive edge trigger for D+ channel
High Speed Positive edge trigger for D- channel

@StuckNomad StuckNomad changed the title USB Protocol Analyzer added USB Protocol Analyzer Dec 18, 2024
@StuckNomad StuckNomad changed the title added USB Protocol Analyzer added protocol analyzer for USB Dec 18, 2024
@mengstr
Copy link

mengstr commented Dec 18, 2024

Out of curiosity - why is the recommended sample rate 100MS/s for the Full Speed? It is not evenly divisible by the bit rate. The only thing I can think of is the granularity of the available sample rates, so I guess that's it?

@StuckNomad
Copy link
Author

StuckNomad commented Dec 18, 2024

My initial plan was to calculate bits per sample by dividing the sampling rate by bit rate. But the D+ and D- channels were often misaligned and caused issues in determining J and K states. I implemented a workaround for this and even divisibility is not required anymore. Moreover, I've observed that this misalignment goes away at higher sampling rates, which is why I chose 100Mhz instead of 96Mhz.

@gusmanb
Copy link
Owner

gusmanb commented Dec 18, 2024

Thanks for the analyzer and your participation. I will test it this weekend and if it works I will accept the PR.

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants