Page MenuHomeFreeBSD

sysutils/u-boot-starfive-visionfive2: new port
ClosedPublic

Authored by mhorne on Jan 19 2024, 8:21 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Apr 26, 3:04 AM
Unknown Object (File)
Wed, Apr 17, 3:59 PM
Unknown Object (File)
Tue, Apr 16, 12:37 PM
Unknown Object (File)
Sat, Apr 13, 7:00 AM
Unknown Object (File)
Thu, Apr 4, 1:28 PM
Unknown Object (File)
Thu, Apr 4, 1:28 PM
Unknown Object (File)
Thu, Apr 4, 1:28 PM
Unknown Object (File)
Thu, Apr 4, 1:27 PM

Details

Summary

Create a u-boot port for the VisionFive 2 RISC-V SBC.

A small tweak to the default configuration is required to work with the
OpenSBI binary provided by sysutils/opensbi.

Sponsored by: The FreeBSD Foundation

Test Plan

Tested on real hardware using the exact commands described in the description.

Diff Detail

Repository
R11 FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

mhorne created this revision.

Hm, OpenSBI is supposed to be built as a PIE by default if the toolchain supports it, which LLVM should, and thus it shouldn't matter where the SPL loads OpenSBI. Debian doesn't override this as far as I can tell, yet also uses its generic OpenSBI binary package.

manu requested changes to this revision.Jan 20 2024, 7:06 AM

You need to add the port to sysutils/Makefile
Otherwise lgtm

This revision now requires changes to proceed.Jan 20 2024, 7:06 AM

Hm, OpenSBI is supposed to be built as a PIE by default if the toolchain supports it, which LLVM should, and thus it shouldn't matter where the SPL loads OpenSBI. Debian doesn't override this as far as I can tell, yet also uses its generic OpenSBI binary package.

If I had simply tried this I would have saved myself a few hours... I think I was thrown by the emphasis of setting FW_TEXT_START when building opensbi in the u-boot documentation. The PIE binary does work; update incoming.

u-boot seems to always demand a sacrifice from me, in one way or another.

Remove the config fragment. Add the port to SUBDIR list.

This revision is now accepted and ready to land.Jan 22 2024, 4:48 PM