search

openhwgroup / cva6

The CORE-V CVA6 is an Application class 6-stage RISC-V CPU capable of booting Linux
https://docs.openhwgroup.org/projects/cva6-user-manual/
Other
2.14k stars 652 forks source link

Fix mstatus.mpp in relation to the possible legal values #2285

Closed JeanRochCoulon closed 1 week ago

JeanRochCoulon commented 1 week ago

The possible legal value depend on the cva6 config: RVU, RVH, RVS. When writing an illegal value, the previous mstatus.mpp value is kept

Moschn commented 1 week ago

I like your change better than mine. I will close my PR.

PS: I think some of these checks are done at different places already, but its good to have them all in one place.

JeanRochCoulon commented 1 week ago

Thanks @Moschn for your review. I will wait for the CI result to merge.

JeanRochCoulon commented 1 week ago

Fixing #2274

github-actions[bot] commented 1 week ago

:x: failed run, report available here.