Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

riscv: pmp: disable all entries #2019

Merged
merged 1 commit into from Jul 17, 2020
Merged

riscv: pmp: disable all entries #2019

merged 1 commit into from Jul 17, 2020

Conversation

bradjc
Copy link
Contributor

@bradjc bradjc commented Jul 13, 2020

Pull Request Overview

With the PMP, the number of entries is double the number of regions. So for disable we want to clear all of the PMP entries.

Testing Strategy

travis

TODO or Help Wanted

n/a

Documentation Updated

  • Updated the relevant files in /docs, or no updates are required.

Formatting

  • Ran make prepush.

@bradjc
riscv: pmp: disable all entries
150b064 
Number of entries is double the number of regions.
@bradjc bradjc added the risc-v RISC-V architecture label Jul 13, 2020
@bradjc bradjc requested a review from alistair23 July 13, 2020 22:04
@bradjc bradjc added the last-call Final review period for a pull request. label Jul 16, 2020
@bradjc bradjc mentioned this pull request Jul 16, 2020
Merged
2 tasks
@bradjc
Copy link
Contributor Author

bradjc commented Jul 17, 2020

bors r+

@bors
Copy link
Contributor

bors bot commented Jul 17, 2020

Build succeeded:

@bors bors bot merged commit ce30c9d into master Jul 17, 2020
@bors bors bot deleted the pmp-disable-double branch July 17, 2020 17:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alistair23 alistair23 alistair23 approved these changes

Assignees
No one assigned
Labels
last-call Final review period for a pull request. risc-v RISC-V architecture
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@bradjc @alistair23