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 new cdc #5

Merged
merged 10 commits into from
Nov 4, 2023
Merged

added new cdc #5

merged 10 commits into from
Nov 4, 2023

Conversation

Peter-Herrmann
Copy link
Owner

No description provided.

@Peter-Herrmann
Copy link
Owner Author

  • While a transaction's grant is in flight, mask the req output.
  • A transaction's grant starts when the req and gnt bits are high
    at the secondary port and no gnt is in flight, send a grant pulse.
  • The receipt of a grant by the controller port must be pulse
    synchronized.
  • Grant pulses sent by the secondary port must be separated by at least
    one clock cycle to enable edge detection on the pulse synchronizer.
  • When the primary receives a grant pulse, it asserts gnt_ack until
    gnt_in_flight is de-asserted.
  • The receipt of a grant acknowledgement by the secondary port must be
    pulse synchronized.
  • A transaction's grant is completed when the secondary port receives
    acknowledgement that the grant was received.
  • The delay between the controller port receiving a grant and the
    controller port receiving a grant ack must be at least as long as
    the pipeline delay of the req signal.
  • Resets must be synchronized across clock domains

@Peter-Herrmann Peter-Herrmann marked this pull request as ready for review October 24, 2023 19:15
@Peter-Herrmann Peter-Herrmann self-assigned this Oct 24, 2023
@Peter-Herrmann Peter-Herrmann merged commit e3bf206 into main Nov 4, 2023
1 check passed
@Peter-Herrmann Peter-Herrmann deleted the feature/cdc_fast_primary branch November 12, 2023 04:54
@Peter-Herrmann Peter-Herrmann restored the feature/cdc_fast_primary branch November 12, 2023 04:54
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