Skip to content

Commit

Permalink
Only include necessary LFSR primitive
Browse files Browse the repository at this point in the history
We previously had a dependency on all primitives in Ibex, even though we
only depend on the LFSR primitive. Now that there's a more fine-grained
dependency available, we can use that.

This has the great benefit of restricting all lint tools to only the
code we're interested in, and not linting all primitives in OpenTitan
together with Ibex. This also helps tools like yosys, which aren't able
to parse all of OpenTitan's code yet.
  • Loading branch information
imphil committed Jul 7, 2020
1 parent ee0a1cf commit c476329
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions ibex_core.core
Original file line number Diff line number Diff line change
Expand Up @@ -10,9 +10,7 @@ filesets:
depend:
- lowrisc:prim:assert
- lowrisc:prim:clock_gating
# TODO: Only lfsr is needed. Replace with a more specific dependency
# once available.
- lowrisc:prim:all
- lowrisc:prim:lfsr
- lowrisc:ibex:ibex_pkg
- lowrisc:ibex:ibex_icache
files:
Expand Down

0 comments on commit c476329

Please sign in to comment.