Skip to content

Add DMI Bridge to FireSim#1852

Merged
abejgonzalez merged 3 commits intomainfrom
dmi-hacking-shcho
Jul 10, 2024
Merged

Add DMI Bridge to FireSim#1852
abejgonzalez merged 3 commits intomainfrom
dmi-hacking-shcho

Conversation

@soohyuk-cho
Copy link
Copy Markdown

@soohyuk-cho soohyuk-cho commented Apr 18, 2024

Adds a FireSim DMI Bridge for an alternative bringup interface other than TSI.

  • Adds a new DMI target config for FireSim for quick testing
  • Adds bridge binder for DMI
  • Enables DebugModule for all FireSim designs (now that clock gating can be disabled). This matches CY's behavior of always having it.

Related PRs / Issues:

Type of change:

  • Bug fix
  • New feature
  • Other enhancement

Impact:

  • RTL change
  • Software change (RISC-V software)
  • Build system change
  • Other

Contributor Checklist:

  • Did you set main as the base branch?
  • Is this PR's title suitable for inclusion in the changelog and have you added a changelog:<topic> label?
  • Did you state the type-of-change/impact?
  • Did you delete any extraneous prints/debugging code?
  • Did you mark the PR with a changelog: label?
  • (If applicable) Did you update the conda .conda-lock.yml file if you updated the conda requirements file?
  • (If applicable) Did you add documentation for the feature?
  • (If applicable) Did you add a test demonstrating the PR?
  • (If applicable) Did you mark the PR as Please Backport?

@abejgonzalez abejgonzalez marked this pull request as ready for review July 5, 2024 20:28
@abejgonzalez abejgonzalez requested a review from jerryz123 July 5, 2024 20:28
@abejgonzalez abejgonzalez self-assigned this Jul 5, 2024
@abejgonzalez abejgonzalez changed the title [WIP] Dmi Bridge hacking Add DMI Bridge to FireSim Jul 5, 2024
@abejgonzalez abejgonzalez requested a review from jerryz123 July 9, 2024 06:05
@abejgonzalez abejgonzalez merged commit bbd4dbe into main Jul 10, 2024
@abejgonzalez abejgonzalez deleted the dmi-hacking-shcho branch August 6, 2024 21:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants