Skip to content

Conversation

@Navaneeth-KunhiPurayil
Copy link
Contributor

@Navaneeth-KunhiPurayil Navaneeth-KunhiPurayil commented Jan 28, 2026

PR to fix the parameters in Spatz cluster to support configuration ELEN=32 by setting RVD=0.

Fixed

  1. Set RVF and RVD based on isa string in the cfg file
  2. Add input parameter NarrowAXIDataWidth to spatz_cluster to use 32b narrow interconnect
  3. TCDMSize is input parameter set using the cfg
  4. Bootrom code is modified to initialize the FP register file after reading misa read-only CSR to identify the supported RISC-V extensions

Added

  1. Configuration for ELEN=32 - spatz_cluster.32b.dram.hjson - 4 x FP32 datapaths
  2. Added ELEN=32 configuration to CI for riscvTests

DiyouS
DiyouS previously approved these changes Jan 28, 2026
Copy link
Collaborator

@DiyouS DiyouS left a comment

Choose a reason for hiding this comment

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

Looks good to me in general. One discussion point I have (not to this PR but general cluster): do you think we can find a centralized location to pass down parameters for Spatz cluster? Now there are mixture use of localparam (hardcode), input parameter, and also package. It would be much easier to maintain and modify if everything are defined in one place.

@Navaneeth-KunhiPurayil
Copy link
Contributor Author

Yeah we should be able to do that. We can define all required parameters in a configuration file and pass as a spatz_cluster module parameter. In this case, we might need to move type definitions that depend on these parameters removing from spatz_pkg into the required modules. But should be possible.

@Navaneeth-KunhiPurayil Navaneeth-KunhiPurayil merged commit 67a2be2 into main Jan 30, 2026
9 of 10 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.

3 participants