On Thu, Mar 25, 2021 at 01:46 AM, Abner Chang wrote:<br />
<blockquote>Hi Loic,<br />The current edk2 RISC-V is incorporated with opensbi v0.8, please check below link<br /><a href="https://github.com/riscv/riscv-uefi-edk2-docs" target="_blank" rel="noopener">https://github.com/riscv/riscv-uefi-edk2-docs</a></blockquote>
Yes, I think this is the version I used. In fact I followed&nbsp;<a href="https://github.com/tianocore/edk2-platforms#submodules" target="_blank" rel="noopener">https://github.com/tianocore/edk2-platforms#submodules</a>&nbsp;to add the needed submodule.<br /><br />
<blockquote>We have to update below library with the latest opensbi with your changes,<br /><a href="https://github.com/tianocore/edk2-platforms/tree/master/Silicon/RISC-V/ProcessorPkg/Library/RiscVOpensbiLib" target="_blank" rel="noopener">https://github.com/tianocore/edk2-platforms/tree/master/Silicon/RISC-V/ProcessorPkg/Library/RiscVOpensbiLib</a> <br /><br />I believe your changes don't impact the build and the functionalities because we don't use those parameters in edk2.</blockquote>
Sounds like yes. I tested in qemu without any issue, but that was not a full regression test ;-)<br /><br />
<blockquote>However, this change should be adopted with Edk2OpenSbiLib upgrade.<br />Daniel is currently working on edk2 RISC-V OVMF and also updated opensbi to the latest version. However there is a compatible issue when switching mode and we are fixing it now.<br />We can share more information with you if you would like to know.</blockquote>
So if this fix will be included in Daniel's changes that fine. I just wanted to share the fix if anyone else had issue and also check if the issue was not on my side, I'm a newbie on edk2 and on RISC-V :D<br /><br />
<blockquote><br />Thanks<br />Abner</blockquote>