All of lore.kernel.org
 help / color / mirror / Atom feed
From: Yao Zi <ziyao@disroot.org>
To: Leo Liang <ycliang@andestech.com>
Cc: Rick Chen <rick@andestech.com>, Tom Rini <trini@konsulko.com>,
	Bin Meng <bmeng.cn@gmail.com>,
	Paul Walmsley <paul.walmsley@sifive.com>,
	"Palmer Dabbelt" <palmer@dabbelt.com>,
	Anup Patel <anup@brainfault.org>,
	Atish Patra <atishp@atishpatra.org>,
	Green Wan <green.wan@sifive.com>,
	Minda Chen <minda.chen@starfivetech.com>,
	Simon Glass <sjg@chromium.org>,
	"Angelo Dureghello" <angelo@kernel-space.org>,
	Ilias Apalodimas <ilias.apalodimas@linaro.org>,
	Heinrich Schuchardt <xypron.glpk@gmx.de>, <u-boot@lists.denx.de>
Subject: Re: [PATCH 5/5] riscv: select OF_HAS_PRIOR_STAGE by default if SBI is enabled
Date: Thu, 6 Mar 2025 12:40:23 +0000	[thread overview]
Message-ID: <Z8mXpFvGu-EcYkOH@pie.lan> (raw)
In-Reply-To: <Z8mCUtgAW2qujI13@swlinux02>

On Thu, Mar 06, 2025 at 07:09:06PM +0800, Leo Liang wrote:
> On Thu, Feb 27, 2025 at 02:47:33PM +0000, Yao Zi wrote:
> > Availability of RISC-V SBI service implies a prior stage exists. As SBI
> > firmware usually passes a FDT to the loaded program, let's select
> > OF_HAS_PRIOR_STAGE if SBI is enabled.
> > 
> > With previously added fallback version of board_fdt_blob_setup, S-Mode
> > RISC-V ports use the SBI-provided FDT by default. This covers the most
> > common usecase, where a SPL (probably the U-Boot one) selects proper
> > devicetree, loads SBI and U-Boot then invokes SBI with the devicetree.
> > 
> > Signed-off-by: Yao Zi <ziyao@disroot.org>
> > Reviewed-by: Leo Yu-Chi Liang <ycliang@andestech.com>
> > ---
> >  arch/riscv/Kconfig | 1 +
> >  1 file changed, 1 insertion(+)
> 
> Hi Yao,
> 
> This patch would fail some of the CI tests.
> I will take other patches from this patchset first,
> and could you take a look at the CI tests issue ?
> (https://source.denx.de/u-boot/custodians/u-boot-riscv/-/jobs/1049827)

Sure, will have a check later.

> Best regards,
> Leo

Thanks,
Yao Zi

  reply	other threads:[~2025-03-06 12:40 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-02-27 14:47 [PATCH 0/5] Simplifiy retrieving FDT from SBI in S-Mode Yao Zi
2025-02-27 14:47 ` [PATCH 1/5] riscv: lib: Add a default implementation of board_fdt_blob_setup Yao Zi
2025-03-06  4:05   ` Leo Liang
2025-02-27 14:47 ` [PATCH 2/5] board: qemu: riscv: Remove duplicated board_fdt_blob_setup Yao Zi
2025-03-06  4:05   ` Leo Liang
2025-02-27 14:47 ` [PATCH 3/5] board: starfive: " Yao Zi
2025-03-06  4:08   ` Leo Liang
2025-02-27 14:47 ` [PATCH 4/5] board: sifive: " Yao Zi
2025-03-06  4:09   ` Leo Liang
2025-02-27 14:47 ` [PATCH 5/5] riscv: select OF_HAS_PRIOR_STAGE by default if SBI is enabled Yao Zi
2025-03-06  4:10   ` Leo Liang
2025-03-06 11:09   ` Leo Liang
2025-03-06 12:40     ` Yao Zi [this message]
2025-03-07 11:15     ` Yao Zi

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=Z8mXpFvGu-EcYkOH@pie.lan \
    --to=ziyao@disroot.org \
    --cc=angelo@kernel-space.org \
    --cc=anup@brainfault.org \
    --cc=atishp@atishpatra.org \
    --cc=bmeng.cn@gmail.com \
    --cc=green.wan@sifive.com \
    --cc=ilias.apalodimas@linaro.org \
    --cc=minda.chen@starfivetech.com \
    --cc=palmer@dabbelt.com \
    --cc=paul.walmsley@sifive.com \
    --cc=rick@andestech.com \
    --cc=sjg@chromium.org \
    --cc=trini@konsulko.com \
    --cc=u-boot@lists.denx.de \
    --cc=xypron.glpk@gmx.de \
    --cc=ycliang@andestech.com \
    /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.