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

BeagleV: Fixes for mainline using OpenSBI #294

Merged
merged 3 commits into from
Jun 7, 2021

Conversation

alistair23
Copy link
Collaborator

Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
@kraj
Copy link
Collaborator

kraj commented Jun 2, 2021

overall this looks ok. I just tested this series + oe-core changes to build openSBI and updated beagleV to it and its working ok now.

@alistair23 alistair23 force-pushed the alistair/opensbi-fix branch from 9ef98ff to 4a86ade Compare June 3, 2021 00:13
@alistair23
Copy link
Collaborator Author

Addresses your comments, this should be ready to go once the OE patches go in

Copy link
Collaborator

@kraj kraj left a comment

Choose a reason for hiding this comment

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

lgtm

@kraj
Copy link
Collaborator

kraj commented Jun 3, 2021

I ran reboot cycle test for 4+ hrs with 252 reboots it never got stuck. So lets declare victory !

@alistair23 alistair23 marked this pull request as ready for review June 7, 2021 22:22
@alistair23
Copy link
Collaborator Author

Changes are in OE-Core

@kraj kraj merged commit f93cce5 into riscv:master Jun 7, 2021
@alistair23 alistair23 deleted the alistair/opensbi-fix branch June 7, 2021 23:51
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