Skip to content

Reworking some of the Zicsr stuff - #303

Open
TheDeepestSpace wants to merge 5 commits into
mainfrom
boris/zicsr-rearch
Open

Reworking some of the Zicsr stuff#303
TheDeepestSpace wants to merge 5 commits into
mainfrom
boris/zicsr-rearch

Conversation

@TheDeepestSpace

@TheDeepestSpace TheDeepestSpace commented Sep 26, 2026

Copy link
Copy Markdown
Member

See micro-arch doc for changes

@github-actions

github-actions Bot commented Sep 27, 2026 •
edited
Loading

Copy link
Copy Markdown

📊 Synthesis & Hardening Report Summary Diff

  • RV32I

    • DE1-SoC synthesis

      • Fitter summary

        @@ -1,4 +1,4 @@
-Fitter Status : Successful - Sat Sep 26 11:44:31 2026
+Fitter Status : Successful - Sun Sep 27 01:41:08 2026
 Quartus Prime Version : 25.1std.0 Build 1129 10/21/2025 SC Lite Edition
 Revision Name : utoss-risc-v
 Top-level Entity Name : top

      • Fitter by entity
        No changes detected

      • Timing
        No changes detected

    • Nangate45 hardening

      • Utilization
        missing

      • Gate breakdown
        missing

  • RV32IZbb_Zbkb

    • DE1-SoC synthesis

      • Fitter summary

        @@ -1,4 +1,4 @@
-Fitter Status : Successful - Sat Sep 26 11:44:23 2026
+Fitter Status : Successful - Sun Sep 27 01:40:16 2026
 Quartus Prime Version : 25.1std.0 Build 1129 10/21/2025 SC Lite Edition
 Revision Name : utoss-risc-v
 Top-level Entity Name : top

      • Fitter by entity
        No changes detected

      • Timing
        No changes detected

    • Nangate45 hardening

      • Utilization
        missing

      • Gate breakdown
        missing

  • RV32IZicsr

    • DE1-SoC synthesis

      • Fitter summary

        @@ -1,4 +1,4 @@
-Fitter Status : Successful - Sat Sep 26 11:45:33 2026
+Fitter Status : Successful - Sun Sep 27 01:41:38 2026
 Quartus Prime Version : 25.1std.0 Build 1129 10/21/2025 SC Lite Edition
 Revision Name : utoss-risc-v
 Top-level Entity Name : top

      • Fitter by entity
        No changes detected

      • Timing
        No changes detected

    • Nangate45 hardening

      • Utilization
        missing

      • Gate breakdown
        missing

Comparing synthesis & hardening results from main branch vs. this PR

@TheDeepestSpace TheDeepestSpace self-assigned this Sep 27, 2026
This was linked to issues Sep 27, 2026
Zicsr extension #290
Open
Extract decode logic for Zicsr into its own module #292
Open
@TheDeepestSpace TheDeepestSpace removed a link to an issue Sep 27, 2026
Zicsr extension #290
Open
@TheDeepestSpace TheDeepestSpace linked an issue Sep 27, 2026 that may be closed by this pull request
Aggregate CSR-related wires into a struct #291
Open
Dazhou-20383
Dazhou-20383 approved these changes Sep 27, 2026
View reviewed changes

@Dazhou-20383 Dazhou-20383 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good! I like how it's more organized while keeping the syntax for declaring a struct exactly the same.

Dazhou-20383
Dazhou-20383 reviewed Sep 27, 2026
View reviewed changes
Comment thread src/ext/zicsr/csr_data.sv
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Dazhou-20383 Dazhou-20383 Dazhou-20383 approved these changes

Assignees

@TheDeepestSpace TheDeepestSpace

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Extract decode logic for Zicsr into its own module Aggregate CSR-related wires into a struct

2 participants

@TheDeepestSpace @Dazhou-20383