From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Petazzoni Date: Sat, 16 Mar 2019 08:43:19 +0100 Subject: [Buildroot] [PATCH 3/3] boot: riscv: Remove riscv-pk In-Reply-To: <20190315230540.4311-3-alistair.francis@wdc.com> References: <20190315230540.4311-1-alistair.francis@wdc.com> <20190315230540.4311-3-alistair.francis@wdc.com> Message-ID: <20190316084319.15b569e0@windsurf> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: buildroot@busybox.net Hello, On Fri, 15 Mar 2019 23:06:39 +0000 Alistair Francis wrote: > Now that OpenSBI is supported remove BBL (riscv-pk). > > Signed-off-by: Alistair Francis This commit is not enough: you do not update the qemu_riscv64_virt_defconfig and qemu_riscv32_virt_defconfig, which use riscv-pk. They should be converted to use opensbi instead, and of course be runtime tested under Qemu. And of course, it'd be nice if Mark Corbin, who added riscv-pk, could give his opinion about the replacement of riscv-pk by opensbi. Thanks! Thomas -- Thomas Petazzoni, CTO, Bootlin Embedded Linux and Kernel engineering https://bootlin.com