From: Thomas Petazzoni via buildroot <buildroot@buildroot.org>
To: Bin Meng <bmeng.cn@gmail.com>
Cc: Bin Meng <bmeng@tinylab.org>,
Alistair Francis <alistair.francis@wdc.com>,
"Yann E . MORIN" <yann.morin.1998@free.fr>,
buildroot@buildroot.org
Subject: Re: [Buildroot] [PATCH v2] configs: riscv: qemu: Drop BR2_TARGET_OPENSBI_CUSTOM_VERSION
Date: Fri, 30 Aug 2024 19:02:22 +0200 [thread overview]
Message-ID: <20240830190222.6e2285cc@windsurf> (raw)
In-Reply-To: <20240829154150.174934-1-bmeng.cn@gmail.com>
Hello Bin,
On Thu, 29 Aug 2024 23:41:50 +0800
Bin Meng <bmeng.cn@gmail.com> wrote:
> From: Bin Meng <bmeng@tinylab.org>
>
> Update qemu_riscv32_virt_defconfig and qemu_riscv64_virt_defconfig
> to use the latest OpenSBI due to the reason that:
>
> 1. they are the main platforms to develop and test OpenSBI
> 2. latest OpenSBI binaries are always integrated into QEMU as
> the default bios
>
> Signed-off-by: Bin Meng <bmeng@tinylab.org>
> ---
> See v1 @ https://patchwork.ozlabs.org/project/buildroot/patch/20230626160030.93063-7-bmeng.cn@gmail.com/
Sorry but no. We _do_ want our defconfigs to specify a custom and fixed
version of Linux, U-Boot, OpenSBI, TF-A, etc.
See https://buildroot.org/downloads/manual/manual.html#adding-board-support:
"""
Always use fixed versions or commit hashes for the different
components, not the "latest" version. For example, set
BR2_LINUX_KERNEL_CUSTOM_VERSION=y and
BR2_LINUX_KERNEL_CUSTOM_VERSION_VALUE to the kernel version you tested
with.
"""
Do not hesitate to send patches to update the OpenSBI version used in
those defconfigs, after testing that this update works fine for those
particular platforms.
Thanks!
Thomas
--
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
next prev parent reply other threads:[~2024-08-30 17:02 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-08-29 15:41 [Buildroot] [PATCH v2] configs: riscv: qemu: Drop BR2_TARGET_OPENSBI_CUSTOM_VERSION Bin Meng
2024-08-29 23:35 ` Alistair Francis
2024-08-30 17:02 ` Thomas Petazzoni via buildroot [this message]
2024-09-02 13:31 ` Bin Meng
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20240830190222.6e2285cc@windsurf \
--to=buildroot@buildroot.org \
--cc=alistair.francis@wdc.com \
--cc=bmeng.cn@gmail.com \
--cc=bmeng@tinylab.org \
--cc=thomas.petazzoni@bootlin.com \
--cc=yann.morin.1998@free.fr \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox