[Buildroot] [PATCH v6 3/5] boot/opensbi: new package
Alistair Francis
alistair23 at gmail.com
Tue Mar 26 22:21:00 UTC 2019
On Tue, Mar 26, 2019 at 6:57 AM Thomas Petazzoni
<thomas.petazzoni at bootlin.com> wrote:
>
> Hello,
>
> On Wed, 20 Mar 2019 23:05:21 +0000
> Alistair Francis <Alistair.Francis at wdc.com> wrote:
>
> > OpenSBI is a much improved alternative to BBL (riscv-pk). Add OpenSBI
> > support to buildroot.
> >
> > Signed-off-by: Alistair Francis <alistair.francis at wdc.com>
>
> I have applied, but after doing a few changes for things that were not
> completely correct. See below.
Thanks for applying this and the rest of the series.
>
> > +OPENSBI_VERSION = v0.3
> > +OPENSBI_SITE = $(call github,riscv,opensbi,$(OPENSBI_VERSION))
> > +OPENSBI_LICENSE = BSD-2-Clause
> > +OPENSBI_LICENSE_FILES = COPYING.BSD
> > +OPENSBI_INSTALL_IMAGES = YES
>
> This is only really needed when OPENSBI_PLAT != "", so I've moved this
> statement inside the appropriate condition.
>
> I've added OPENSBI_INSTALL_TARGET = NO, because we're not installing
> anything to target.
Thanks
>
> But more importantly, I've added OPENSBI_INSTALL_STAGING = YES, without
> which your OPENSBI_INSTALL_STAGING_CMDS is never called. I'm not sure
> how you tested this, but it clearly couldn't work as is.
Ah, I tested it in the previous version when I was using the Image
install, but must have missed checking it was installed in the latest
version. Sorry about that.
>
> > +# Install libsbi.a in the host lib dir so it can be linked in the future.
>
> The library is not installed in host dir. So I've changed this to
> rather explain why we install in usr/share/opensbi and not in usr/lib.
Thanks for this.
Alistair
>
> Applied with those issues fixed. Thanks!
>
> Thomas
> --
> Thomas Petazzoni, CTO, Bootlin
> Embedded Linux and Kernel engineering
> https://bootlin.com
More information about the buildroot
mailing list