ibex icon indicating copy to clipboard operation
ibex copied to clipboard

[pmp] Use top-level straps for PMP reset values

Open Razer6 opened this issue 11 months ago • 3 comments

By using top-level straps for the PMP reset configuration its easier to implement different reset configurations if there are multiple Ibex cores in the system.

Razer6 avatar Mar 01 '24 16:03 Razer6

I had intentionally avoid doing this with parameters when I first wrote it to avoid adding yet more parameters to Ibex, though given we've already got loads perhaps it doesn't matter.

Ultimately this is probably what we have to do but I'll have a think about alternative methods.

GregAC avatar Mar 04 '24 10:03 GregAC

@Razer6 we intend to take this, I just need to do a couple of checks to ensure there's no issues for non PMP configs or breakages in other things we don't check in CI I hope to get this done over the next couple of days.

GregAC avatar Mar 05 '24 17:03 GregAC

@GregAC No hurry on that one. We have changed that in our environment and are not dependent on vendoring in a new version of ibex at the moment.

Razer6 avatar Mar 05 '24 23:03 Razer6

@GregAC @rswarbrick Could you take another look?

Razer6 avatar Sep 03 '24 21:09 Razer6

@Razer6 yes will do

GregAC avatar Sep 16 '24 09:09 GregAC

Thanks @GregAC for the review. I just updated the changes for the nits.

Razer6 avatar Sep 19 '24 07:09 Razer6

Thanks @Razer6

GregAC avatar Sep 23 '24 10:09 GregAC