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.

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

risc-v/mpfs: Add error handling for PMP conf #263

Merged
merged 1 commit into from
Jun 5, 2024
Merged

Conversation

jpaali
Copy link

@jpaali jpaali commented Jun 5, 2024

Check return value of mpfs_board_pmp_setup() and
jump to mpfs_board_pmp_error() in case of error.

Summary

Impact

Testing

@jpaali jpaali requested a review from pussuw June 5, 2024 09:41
@jpaali jpaali force-pushed the pmp_error_handler branch 2 times, most recently from e302cf4 to 344fb45 Compare June 5, 2024 09:58
Copy link

@pussuw pussuw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Check return value of mpfs_board_pmp_setup() and
jump to mpfs_board_pmp_error() in case of error.

Signed-off-by: Jani Paalijarvi <[email protected]>
@jpaali jpaali force-pushed the pmp_error_handler branch from 344fb45 to 0a86755 Compare June 5, 2024 17:37
@jpaali
Copy link
Author

jpaali commented Jun 5, 2024

This is now also in upstream.

@jpaali jpaali marked this pull request as ready for review June 5, 2024 17:38
@jpaali jpaali merged commit 4139cf5 into master Jun 5, 2024
11 checks passed
@jpaali jpaali deleted the pmp_error_handler branch June 5, 2024 18:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants