Re: [PATCH] roms/opensbi: Update to v1.5

[Daniel Henrique Barboza]

On 7/29/24 7:31 PM, Heinrich Schuchardt wrote:
> On 7/15/24 19:15, Daniel Henrique Barboza wrote:
> > Update OpenSBI and the pre-built opensbi32 and opensbi64 images to
> > v1.5.
> >
> > The following commits were included in v1.5:
> >
> > 455de67 include: Bump-up version to 1.5

(...)

> > Signed-off-by: Daniel Henrique Barboza<dbarboza@ventanamicro.com>
> > ---
> >   .../opensbi-riscv32-generic-fw_dynamic.bin    | Bin 267416 -> 268312 bytes
> >   .../opensbi-riscv64-generic-fw_dynamic.bin    | Bin 270808 -> 272504 bytes
> >   roms/opensbi                                  |   2 +-
> >   3 files changed, 1 insertion(+), 1 deletion(-)
>
> While OpenSBI v1.5 compiled from source allows to poweroff U-Boot 
> qemu-riscv64_smode_defconfig, poweroff fails with the binary provided by this 
> patch.
>
> For details see
> https://gitlab.com/qemu-project/qemu/-/issues/2467

Replied in the bug, but to give some context here: the problem is reproducible 
when
trying to do a 'poweroff' using the default machine 'Spike'. 'poweroff' works 
when
using the 'virt' machine.

And it has nothing to do with the binaries distributed in this patch. These are 
the
same OpenSBI v1.5 binaries that you would compile from source, as I showed in 
the
issue. They aren't 'patched'.


Thanks,

Daniel

> Best regards
>
> Heinrich