Skip to content

Conversation

@Scheremo
Copy link

@Scheremo Scheremo commented May 21, 2025

This PR adds support for using multi-cycle response / pipeline-mode memory macros. Such macros typically have better C -> Q timing arcs at the cost of one cycle of additional reponse latency.

This PR depends on this PR, at least for testing of the feature on my end.

Added

  • Parameter BankAccessLatency which configures how many clock cycles of latency each memory macro requires

Changed

  • Updated verible config to enforce 100 character column limit

@Scheremo Scheremo force-pushed the pr/bankaccesslatency branch from db4f279 to 000a185 Compare May 21, 2025 14:43
@micprog micprog self-requested a review May 21, 2025 14:46
@Scheremo Scheremo force-pushed the pr/bankaccesslatency branch from f45884b to 1b4ed2f Compare May 21, 2025 15:07
Copy link
Member

@micprog micprog left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@micprog micprog merged commit d072ba0 into pulp-platform:main May 21, 2025
3 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.

2 participants