efabless / caravel_mgmt_soc_litex

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

Added an ifdef for USE_POWER_PINS around the power pins of the SRAM in RTL verilog #41

Closed RTimothyEdwards closed 1 year ago

RTimothyEdwards commented 1 year ago

Added an ifdef for USE_POWER_PINS around the power pins of the SRAM module in the RTL verilog file mgmt_core.v to match the verilog of the SRAM module itself. As far as I am aware, the VexRISC core does not need the same treatment because the VexRISC core does not define power supply pins at all. Note that this is a change to an automatically-generated file, and the change needs to be made elsewhere in the LiteX code to become permanent.

RTimothyEdwards commented 1 year ago

This PR should have been marked with the branch caravel_stanford as target. It has already been merged into that branch, so this pull request can be closed (the permanent fix is in the LiteX code, not a quick fix to the RTL verilog, so this will not be merged into branch main).