Differential D54421

sysutils/opensbi: update to v1.8
Needs ReviewPublic
Actions

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

Details

Reviewers

None

Group Reviewers

riscv

Summary

In this update, drop the workaround patch for FU540 as no longer
supported (?).

Diff Detail

Repository

R11 FreeBSD ports repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 69529
Build 66412: 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.