All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Yann E. MORIN" <yann.morin.1998@free.fr>
To: Bin Meng <bmeng.cn@gmail.com>
Cc: Bin Meng <bmeng@tinylab.org>,
	Alistair Francis <alistair.francis@wdc.com>,
	Jan-Benedict Glaw <jbglaw@lug-owl.de>,
	buildroot@buildroot.org
Subject: Re: [Buildroot] [PATCH 06/10] configs: riscv: Drop BR2_TARGET_OPENSBI_CUSTOM_VERSION
Date: Tue, 27 Jun 2023 22:39:36 +0200	[thread overview]
Message-ID: <20230627203936.GJ646621@scaer> (raw)
In-Reply-To: <CAEUhbmW47WVOM+Zhxa4=T4tgumKU+koEcf+QfivjsUGsUZj2Dw@mail.gmail.com>

Bin, All,

On 2023-06-27 10:11 +0800, Bin Meng spake thusly:
> On Tue, Jun 27, 2023 at 1:10 AM Yann E. MORIN <yann.morin.1998@free.fr> wrote:
> > On 2023-06-27 00:00 +0800, Bin Meng spake thusly:
> > > From: Bin Meng <bmeng@tinylab.org>
> > > Since commit 64f1f82385b2 ("boot/opensbi: Bump to version 1.2"),
> > > the default OpenSBI version has been bumped up to 1.2. With that,
> > > there is no need to define a custom version in each RISC-V board's
> > > defconfig files.
> > In fact, no, because opensbi is a "critical" component part of the boot
> > chain and, like uboot or the kernel, we want to pin the version we use
> > in defconfig files. We only want the v ersion to be bumped in defconfig,
> > when someone actually tested the defconfig still generates a bootable
> > system.
> Makes sense to me.
> But I would argue at least we should 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 will always be integrated into QEMU as the
> default bios
> thoughts?

For the qemu defconfigs, I wouild indeed understand that we use the
latest version; the rationale looks sound.

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

  reply	other threads:[~2023-06-27 20:39 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-06-26 16:00 [Buildroot] [PATCH 00/10] riscv: Various fixes and small enhancements Bin Meng
2023-06-26 16:00 ` [Buildroot] [PATCH 01/10] configs/hifive_unleashed: Update to use OpenSBI 1.2 Bin Meng
2023-06-26 16:00 ` [Buildroot] [PATCH 02/10] configs/hifive_unleashed: Update to use U-Boot v2023.04 Bin Meng
2023-06-26 16:00 ` [Buildroot] [PATCH 03/10] configs/hifive_unleashed: Enable BR2_RISCV_ISA_CUSTOM_RVA explicitly Bin Meng
2023-06-26 16:00 ` [Buildroot] [PATCH 04/10] configs/beaglev_defconfig: " Bin Meng
2023-06-26 16:00 ` [Buildroot] [PATCH 05/10] arch/Config.in.riscv: Remove BR2_RISCV_ISA_CUSTOM_RVA from BR2_riscv_custom Bin Meng
2023-06-26 16:00 ` [Buildroot] [PATCH 06/10] configs: riscv: Drop BR2_TARGET_OPENSBI_CUSTOM_VERSION Bin Meng
2023-06-26 17:10   ` Yann E. MORIN
2023-06-27  2:11     ` Bin Meng
2023-06-27 20:39       ` Yann E. MORIN [this message]
2023-06-26 16:00 ` [Buildroot] [PATCH 07/10] boot/opensbi: Config.in: Cosmetic style fix Bin Meng
2023-06-26 16:00 ` [Buildroot] [PATCH 08/10] boot/opensbi: Bump to version 1.3 Bin Meng
2023-06-26 16:00 ` [Buildroot] [PATCH 09/10] board/sifive: hifive-unleashed: Update rootfs offset Bin Meng
2023-06-26 17:12   ` Yann E. MORIN
2023-08-12 21:12   ` Thomas Petazzoni via buildroot
2023-06-26 16:00 ` [Buildroot] [PATCH 10/10] board/sifive: hifive-unleashed: Include QEMU instructions in readme.txt Bin Meng
2023-06-26 17:21   ` Yann E. MORIN
2023-06-27  0:32     ` Alistair Francis
2023-06-28 15:15       ` Bin Meng
2023-08-12 21:14   ` Thomas Petazzoni via buildroot
2023-06-26 17:04 ` [Buildroot] [PATCH 00/10] riscv: Various fixes and small enhancements Yann E. MORIN

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=20230627203936.GJ646621@scaer \
    --to=yann.morin.1998@free.fr \
    --cc=alistair.francis@wdc.com \
    --cc=bmeng.cn@gmail.com \
    --cc=bmeng@tinylab.org \
    --cc=buildroot@buildroot.org \
    --cc=jbglaw@lug-owl.de \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.