search

pulp-platform / pulp_soc

pulp_soc is the core building component of PULP based SoCs
Other
78 stars 81 forks source link

Update ibex #47

Closed micprog closed 3 years ago

micprog commented 3 years ago

Updates ibex core used as FC core within pulp. Adds performance counters and fixes compatibility.

Pending merge of https://github.com/pulp-platform/pulp_soc/pull/45

micprog commented 3 years ago

Commit changed to no longer depend on bender integration (as discussed in #45)

micprog commented 3 years ago

To properly build and execute code with ibex, following PRs may be required, as well as the environment variable USE_IBEX set for code build:

vogelpi commented 3 years ago

I see that the CI for pulpissimo-v6.1.1 failed. Is this ok? https://github.com/lowRISC/ibex/tree/pulpissimo-v6.1.1

Otherwise LGTM and can be merged

Yes, that's correct. The PULP-specific changes are not compatible with upstream (e.g. replacement of clock gate cell etc.) and some of the changes are not lint clean. Nothing to worry though as long as @micprog has tested the changes in PULP.

micprog commented 3 years ago

@meggiman @bluewww I have tested the changes using ibex as FC, everything I've seen that is supposed to work with ibex works with the changes previously mentioned:

To properly build and execute code with ibex, following PRs may be required, as well as the environment variable USE_IBEX set for code build:

Let me know if you'd like other changes here or want me to run specific tests for this non-standard config, otherwise I'd appreciate it if one of you merged this PR.