Page MenuHomeFreeBSD

riscv: Include syscon_power device driver in GENERIC kernel config
ClosedPublic

Authored by jrtc27 on Jul 19 2020, 6:27 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mar 7 2025, 11:04 PM
Unknown Object (File)
Mar 4 2025, 2:39 PM
Unknown Object (File)
Feb 22 2025, 3:56 AM
Unknown Object (File)
Feb 11 2025, 6:20 PM
Unknown Object (File)
Feb 1 2025, 4:16 PM
Unknown Object (File)
Dec 19 2024, 7:12 AM
Unknown Object (File)
Dec 10 2024, 8:02 PM
Unknown Object (File)
Nov 23 2024, 11:09 AM
Subscribers

Details

Summary

QEMU's RISC-V virt machine provides syscon-power and syscon-reset
devices as the means by which to shutdown and reboot. We also need to
ensure that we have attached the syscon_generic device before attaching
any syscon_power devices, and so we introduce a new riscv_syscon device
akin to aw_syscon added in r327936. Currently the SiFive test finisher
is used as the specific implementation of such a syscon device.

Test Plan

Running reboot on QEMU causes the system to reboot, rather than falling back
on cpu_reset which on RISC-V just powers off via SBI.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

jrtc27 created this revision.
This revision is now accepted and ready to land.Jul 20 2020, 8:59 PM
sys/riscv/riscv/riscv_syscon.c
4 ↗(On Diff #74654)

this copyright is not needed

sys/riscv/riscv/riscv_syscon.c
4 ↗(On Diff #74654)

But aw_syscon is copyright kevans@, which is where most of this code comes from? As in, I copied that file, changed aw to riscv everywhere, updated compat_data, the device description and comments. But it's basically the same code.

br added inline comments.
sys/riscv/riscv/riscv_syscon.c
4 ↗(On Diff #74654)

Since you copied then yes, otherwise it looks like a basic FDT driver template