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

Add Configuration and PTM TLP support. #121

Merged
merged 14 commits into from
Jul 28, 2023
Merged

Add Configuration and PTM TLP support. #121

merged 14 commits into from
Jul 28, 2023

Conversation

enjoy-digital
Copy link
Owner

  • Rework Packetizer/Depacketizer to be more modular (and allow individual selection of capabilities: Request, Completion, Configuration, PTM TLPs).
  • Add Configuration TLP definitions/layouts.
  • Add PTM TLP definitions/layouts.
  • Implement Configuration support in Depacketizer (not required in Packetizer since returning Completion).
  • Implement PTM support in Packetizer/Depacketizer.

For now, Configuration and PTM TLPs can now just be sent/received to/from the User logic. Logic to handle this still has to be done and integrated.

Note: As a bonus, making Packetizer/Depacketizer more modular allow some optimization on Ultrascale(+) were we can only select Completion or Request support (see 33c52fc).

By default, we generally only want Request/Completion TLPs support, but for
other specific cases (ex PTM) we also want to handle Configuration TLPs.

With these change, the module is made more modular to automatically adapt itself
to the capabilities provided (and will also allow easily extending capabilities in
the future, ex for PTM TLPs).
By default, we generally only want Request/Completion TLPs support, but for
other specific cases (ex PTM) we also want to handle PTM TLPs.

With these change, the module is made more modular to automatically adapt itself
to the capabilities provided (and will also allow easily extending capabilities in
the future, ex for PTM TLPs).
@enjoy-digital enjoy-digital merged commit 9107450 into master Jul 28, 2023
2 checks passed
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.

1 participant