-
Notifications
You must be signed in to change notification settings - Fork 210
os-boot/opensbi: add opensbi. #966
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
The number should change. |
defa20c
to
2b0a675
Compare
Signed-off-by: Ariel Xiong <ArielHeleneto@outlook.com>
@@ -14,10 +17,16 @@ | |||
reg = <0>; | |||
status = "okay"; | |||
compatible = "riscv"; | |||
riscv,isa = "rv64imac"; | |||
riscv,isa-base = "rv64i"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This can't boot FreeBSD, it currently only recognises riscv,isa.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, why is the FDT within opensbi/?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This can't boot FreeBSD, it currently only recognises riscv,isa.
I just test openSBI here, let FreeBSD to another pr(
@@ -5,6 +5,9 @@ | |||
#size-cells = <2>; | |||
compatible = "ucbbar,spike-bare-dev"; | |||
model = "ucbbar,spike-bare"; | |||
chosen { | |||
bootargs = "console=hvc0 earlycon=sbi"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These are Linux-specific, do they really belong here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That doesn't mater I think(
No description provided.