search

riscv-software-src / opensbi

RISC-V Open Source Supervisor Binary Interface
Other
1k stars 501 forks source link

support bare metal toolchains #344

Closed yogo1212 closed 1 month ago

yogo1212 commented 8 months ago

This addresses the issues that

Fixes #343.

yogo1212 commented 8 months ago

don't know what i did there just now. hunk was missing from commit, make saw the changed file, git didn't - fixed now.

avpatel commented 8 months ago

We have mailing list based patch review so it would be great if you can send these patchs to OpenSBI mailing list.

You need to join OpenSBI mailing list using following link http://lists.infradead.org/mailman/listinfo/opensbi

Make sure you use "git send-email" to send the patches.

Regards, Anup

egeKeyvan commented 3 months ago

These changes also fixed the same problem that I had on MacOS Ventura. Is it possible for this patch to be merged ?

yogo1212 commented 3 months ago

@egeKeyvan the check for the option was merged. are you talking about the dynsym removal?

egeKeyvan commented 2 months ago

@egeKeyvan the check for the option was merged. are you talking about the dynsym removal?

I was talking about the dynsym removal. Without it, I had the problems of the #343

avpatel commented 1 month ago

Most of these changes are already in latest OpenSBI. Please send patches to mailing list for any unaddressed issues.