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: opensbi: Use mainilne OpenSBI #292

Merged
merged 1 commit into from
Jun 2, 2021

Conversation

alistair23
Copy link
Collaborator

No description provided.

@@ -3,7 +3,7 @@ require recipes-bsp/u-boot/u-boot.inc

FILESEXTRAPATHS_prepend := "${THISDIR}/files:"

SRC_URI = "git://github.com/starfive-tech/u-boot.git;protocol=git;branch=Fedora_VIC_7100_2021.04 \
SRC_URI = "git://github.com/atishp04/u-boot.git;protocol=git;branch=beagleV/Fedora_VIC_7100_2021.04 \
Copy link
Collaborator

Choose a reason for hiding this comment

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

why should we switch to this fork ?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Apparently that's the one to use: starfive-tech/opensbi#4 (comment)

Copy link
Collaborator

Choose a reason for hiding this comment

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

OK and I hope this is short lived and eventually all these changes will flow into upstream u-boot ?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

That's my hope. It seems to only be two patches on top of the starfive branch, so hopefully we can switch to upstream or starfive soon.

@kraj kraj merged commit acc633e into riscv:master Jun 2, 2021
@alistair23 alistair23 deleted the alistair/opensbi branch June 2, 2021 03:23
@lbmeng
Copy link

lbmeng commented Jun 2, 2021

See the StarFive U-Boot changes to work with upstream OpenSBI (starfive-tech/u-boot#17)

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.

3 participants