search

efabless / caravel_mgmt_soc_litex

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

Added the "constant_block" in the include #56

Closed RTimothyEdwards closed 2 years ago

RTimothyEdwards commented 2 years ago

Added the "constant_block" in the include file (includes.rtl.caravel) to match the block added in the "fix_direct_power_connections" branch in the caravel repository.

RTimothyEdwards commented 2 years ago

This needs to be merged into caravel_mgmt_soc_litex caravel_redesign branch at the same time that the corresponding fix_direct_power_connections pull request is merged into the caravel repository caravel_redesign branch. That pull request creates a new verilog module that then needs to be added to the include file list in caravel_mgmt_soc_litex.

It is not clear to me why this repository has include files that reference files in another repository. Shouldn't they be more self-contained?