Skip to content
This repository has been archived by the owner on Nov 22, 2023. It is now read-only.

occamy: Add occamy_soc as synchronous top-level submodule #305

Merged
merged 3 commits into from
Nov 1, 2021

Conversation

paulsc96
Copy link
Member

@paulsc96 paulsc96 commented Oct 28, 2021

NOTE: #303 should be merged before this.

This introduces the occamy_soc module, confining all hart-synchronous top-level logic previously in contained in occamy_top to a submodule. This simplifies synthesis by

  • Enabling strategies generally only safe in fully synchronous designs, such as global retiming and self-gating
  • Enforcing disciplined handling of IO crossing clock domains
  • Reducing constraint evaluation turnaround time.

It also moves the SPM to the system clock domain, which is much safer since the SPM is boot-critical in all scenarios. Should this cause too much timing pressure, a halved system clock could be used as a safer option here.

The MR also includes a minor synthesis fix in CVA6.

Copy link
Contributor

@GiannaP GiannaP left a comment

Choose a reason for hiding this comment

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

Looking good to me :)

@paulsc96 paulsc96 force-pushed the feature/occamy-partition branch from d0d0899 to cd8c5d9 Compare November 1, 2021 15:19
@paulsc96 paulsc96 marked this pull request as ready for review November 1, 2021 16:00
@paulsc96 paulsc96 merged commit 3955925 into master Nov 1, 2021
@paulsc96 paulsc96 deleted the feature/occamy-partition branch November 1, 2021 16:18
colluca pushed a commit to pulp-platform/occamy that referenced this pull request Aug 19, 2023
…tform/snitch#305)

* occamy: Add SoC to encapsulate synchronous top-level

* cva6: Fix AXI icache synthesis

* vendor: Add cva6 patch
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants