lowRISC / ibex

Ibex is a small 32 bit RISC-V CPU core, previously known as zero-riscy.
https://www.lowrisc.org
Apache License 2.0
1.29k stars 495 forks source link

[icache,dv] Fix types in ibex_icache_sim_cfg.hjson #2151

Closed rswarbrick closed 3 months ago

rswarbrick commented 3 months ago

I actually made this change in OpenTitan (editing the vendored Ibex code by accident). Oops! The easiest path to getting everything in sync again is probably to manually make the identical change here and then re-vendor the result into OpenTitan.

For reviewers who also happen to be on the OT side: this was a silly mistake from me, and it seems that a bug in the CI code in OpenTitan means we can't currently see the mistake. But I've just fixed the CI bug... which means CI checks in OpenTitan are currently failing, for silly reasons. This is the simplest path I can see to getting everything green again.

rswarbrick commented 3 months ago

I take it back! There's an easier way, that I can do on the Ibex side for now. :-)