Page MenuHomeFreeBSD
Differential D38707

sbi: support debug console replacement extension
Needs ReviewPublic
Actions

Authored by mhorne on Feb 21 2023, 2:53 PM.
Tags
None
Referenced Files
F109186601: D38707.diff
Sat, Feb 1, 9:05 PM
Unknown Object (File)
Thu, Jan 9, 12:38 AM
Unknown Object (File)
Oct 4 2024, 12:19 PM
Unknown Object (File)
Oct 3 2024, 6:26 AM
Unknown Object (File)
Oct 1 2024, 9:53 PM
Unknown Object (File)
Sep 27 2024, 2:11 PM
Unknown Object (File)
Sep 23 2024, 3:33 PM
Unknown Object (File)
Sep 6 2024, 2:07 AM
View All 26 Files
Subscribers
imp
jrtc27

Details

Reviewers
None
Group Reviewers
riscv
Summary

With this, all functionality provided by the legacy SBI interface is
covered by support for the newer replacement extensions. Eventually,
support for the legacy interface could be dropped, but there is no
reason to do so yet. BBL still provides only the legacy interface.

For reference:
https://github.com/riscv-non-isa/riscv-sbi-doc/blob/master/riscv-sbi.adoc#debug-console-extension-eid-0x4442434e-dbcn

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 49927
Build 46819: arc lint + arc unit

Event Timeline

mhorne created this revision.Feb 21 2023, 2:53 PM
Herald added a subscriber: imp. · View Herald TranscriptFeb 21 2023, 2:53 PM
mhorne requested review of this revision.Feb 21 2023, 2:53 PM
mhorne added a parent revision: D38706: sbi: prefer register_t type in sbi_call().
Harbormaster completed remote builds in B49927: Diff 117694.Feb 21 2023, 2:53 PM
jrtc27 added a subscriber: jrtc27.Feb 21 2023, 2:58 PM

Last I checked these took a physical address represented as a lo/hi pair of unsigned long?

mhorne added a comment.Feb 21 2023, 3:02 PM
In D38707#880893, @jrtc27 wrote:

Last I checked these took a physical address represented as a lo/hi pair of unsigned long?

Yes, that's right. Looks like my early-boot test of sbi_console_getchar() was bogus. Hang tight...

Revision Contents
Changeset List

PathSize
sys/
riscv/
include/
28 lines
riscv/
50 lines

Diff 117694

View Options

sys/riscv/include/sbi.h

Loading...
View Options

sys/riscv/riscv/sbi.c

Loading...