search

efabless / caravel_mgmt_soc_litex

https://caravel-mgmt-soc-litex.readthedocs.io/en/latest/
Apache License 2.0
25 stars 15 forks source link

Correct for missing power supply pins in SRAM and VexRISC core #45

Closed RTimothyEdwards closed 1 year ago

RTimothyEdwards commented 1 year ago

The two modules for the SRAM and the VexRISC core do not have power supply pin connections matching all the other modules. These should be in #fidef USE_POWER_PINS ... #endif blocks.

Short-term fix: Correct the mgmt_core verilog to add the ifdef block

Long-term fix: Correct the upstream LiteX file that generated the code in the first place.

jeffdi commented 1 year ago

We need to confirm gl simulation or LVS for the mgmt core wrapper to confirm verification of this change. @shalan