sysutils/opensbi: update to v1.8.1
AcceptedPublic
Actions

Authored by mhorne on Tue, Dec 30, 4:29 PM.

Details

Reviewers

maciphone2_googlemail.com
lwhsu

Group Reviewers

riscv

Summary

In this update, drop the PMP workaround patch for FU540. This will be
fixed in the src tree.

As usual, bump dependent u-boot ports.

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 69842
Build 66725: arc lint + arc unit

Event Timeline

mhorne requested review of this revision.Tue, Dec 30, 4:29 PM

mhorne created this revision.

Harbormaster completed remote builds in B69529: Diff 168779.Tue, Dec 30, 4:29 PM

I think the question is more "do we want to support FU540?"

If the workaround is still working and no impact to others, it probably doesn't very hurt.
But it it cause issues, we may remove its support, along with sysutils/u-boot-sifive-fu540.

In D54421#1243662, @lwhsu wrote:

I think the question is more "do we want to support FU540?"

If the workaround is still working and no impact to others, it probably doesn't very hurt.
But it it cause issues, we may remove its support, along with sysutils/u-boot-sifive-fu540.

I've opened this discussion out-of-band.

It only causes me a small headache with each update of the port, as I have to regenerate the patches. The hardware is exceedingly rare, however.

Yes, removal of sysutils/u-boot-sifive-fu540 would be the next step.

Bump to latest v1.8.1 and bump fu540 u-boot revision

mhorne retitled this revision from sysutils/opensbi: update to v1.8 to sysutils/opensbi: update to v1.8.1.Mon, Jan 12, 9:41 PM

mhorne edited the summary of this revision. (Show Details)

Harbormaster completed remote builds in B69842: Diff 169548.Wed, Jan 14, 12:50 AM

mhorne added a subscriber: maciphone2_googlemail.com.Wed, Jan 14, 5:19 PM

👍 https://dmesgd.nycbug.org/dmesgd?do=view&id=8800

maciphone2_googlemail.com accepted this revision.Wed, Jan 14, 11:38 PM

This revision is now accepted and ready to land.Wed, Jan 14, 11:38 PM

In D54421#1249144, @mhorne wrote:

Bump to latest v1.8.1 and bump fu540 u-boot revision

Sorry for missing this one. So does fu540 u-boot work with this 1.8.1 opensbi? If no I think we should either remove fu540 u-boot port, or include the patch (or 1.8.1 can work with fu540 without patch?)