From: E Shattow <e@freeshell.de>
To: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>,
Rick Chen <rick@andestech.com>, Leo <ycliang@andestech.com>
Cc: Minda Chen <minda.chen@starfivetech.com>,
Hal Feng <hal.feng@starfivetech.com>,
Simon Glass <sjg@chromium.org>,
Yu-Chien Peter Lin <peter.lin@sifive.com>,
Sumit Garg <sumit.garg@linaro.org>,
"Leon M . Busch-George" <leon@georgemail.eu>,
Lukas Funke <lukas.funke@weidmueller.com>,
H Bell <dmoo_dv@protonmail.com>,
u-boot@lists.denx.de
Subject: Re: [PATCH v3 4/4] riscv: dts: starfive: remove duplicate itb entries
Date: Sat, 22 Feb 2025 17:36:26 -0800 [thread overview]
Message-ID: <7baa4904-16cc-4bde-9b7b-72a1f2899645@freeshell.de> (raw)
In-Reply-To: <20250221095856.36530-5-heinrich.schuchardt@canonical.com>
On 2/21/25 01:58, Heinrich Schuchardt wrote:
> As binman already creates nodes based on CONFIG_OF_LIST we don't need to
> add extra nodes.
>
> Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
> Reviewed-by: Simon Glass <sjg@chromium.org>
> Tested-by: Simon Glass <sjg@chromium.org> # StarFIve VisionFive 2
> ---
> v3:
> no change
> v2:
> no change
> ---
> .../dts/starfive-visionfive2-binman.dtsi | 76 -------------------
> 1 file changed, 76 deletions(-)
>
> diff --git a/arch/riscv/dts/starfive-visionfive2-binman.dtsi b/arch/riscv/dts/starfive-visionfive2-binman.dtsi
> index 4cce001e80d..05787bdb92d 100644
> --- a/arch/riscv/dts/starfive-visionfive2-binman.dtsi
> +++ b/arch/riscv/dts/starfive-visionfive2-binman.dtsi
> @@ -13,82 +13,6 @@
> };
>
> &binman {
> - itb {
> - fit {
> - images {
> - fdt-jh7110-milkv-mars {
> - description = "jh7110-milkv-mars";
> - load = <0x40400000>;
> - compression = "none";
> -
> - blob-ext {
> - filename = "dts/upstream/src/riscv/starfive/jh7110-milkv-mars.dtb";
> - };
> - };
> -
> - fdt-jh7110-pine64-star64 {
> - description = "jh7110-pine64-star64";
> - load = <0x40400000>;
> - compression = "none";
> -
> - blob-ext {
> - filename = "dts/upstream/src/riscv/starfive/jh7110-pine64-star64.dtb";
> - };
> - };
> -
> - fdt-jh7110-starfive-visionfive-2-v1.2a {
> - description = "jh7110-starfive-visionfive-2-v1.2a";
> - load = <0x40400000>;
> - compression = "none";
> -
> - blob-ext {
> - filename = "dts/upstream/src/riscv/starfive/jh7110-starfive-visionfive-2-v1.2a.dtb";
> - };
> - };
> -
> - fdt-jh7110-starfive-visionfive-2-v1.3b {
> - description = "jh7110-starfive-visionfive-2-v1.3b";
> - load = <0x40400000>;
> - compression = "none";
> -
> - blob-ext {
> - filename = "dts/upstream/src/riscv/starfive/jh7110-starfive-visionfive-2-v1.3b.dtb";
> - };
> - };
> - };
> -
> - configurations {
> - conf-jh7110-milkv-mars {
> - description = "jh7110-milkv-mars";
> - firmware = "opensbi";
> - loadables = "uboot";
> - fdt = "fdt-jh7110-milkv-mars";
> - };
> -
> - conf-jh7110-pine64-star64 {
> - description = "jh7110-pine64-star64";
> - firmware = "opensbi";
> - loadables = "uboot";
> - fdt = "fdt-jh7110-pine64-star64";
> - };
> -
> - conf-jh7110-starfive-visionfive-2-v1.2a {
> - description = "jh7110-starfive-visionfive-2-v1.2a";
> - firmware = "opensbi";
> - loadables = "uboot";
> - fdt = "fdt-jh7110-starfive-visionfive-2-v1.2a";
> - };
> -
> - conf-jh7110-starfive-visionfive-2-v1.3b {
> - description = "jh7110-starfive-visionfive-2-v1.3b";
> - firmware = "opensbi";
> - loadables = "uboot";
> - fdt = "fdt-jh7110-starfive-visionfive-2-v1.3b";
> - };
> - };
> - };
> - };
> -
> spl-img {
> filename = "spl/u-boot-spl.bin.normal.out";
>
Reviewed-by: E Shattow <e@freeshell.de>
prev parent reply other threads:[~2025-02-23 1:36 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-02-21 9:58 [PATCH v3 0/4] riscv: dts: starfive: simplify binman config Heinrich Schuchardt
2025-02-21 9:58 ` [PATCH v3 1/4] riscv: dts: add OF_LIST handling to binman.dtsi Heinrich Schuchardt
2025-02-22 22:05 ` E Shattow
2025-02-21 9:58 ` [PATCH v3 2/4] riscv: dts: no default configuration for MULTI_DTB_FIT Heinrich Schuchardt
2025-02-22 22:07 ` E Shattow
2025-02-21 9:58 ` [PATCH v3 3/4] board: starfive: spl: strip off 'starfive/' prefix Heinrich Schuchardt
2025-02-23 1:33 ` E Shattow
2025-02-23 12:55 ` Heinrich Schuchardt
2025-02-23 14:18 ` E Shattow
2025-02-23 15:39 ` Heinrich Schuchardt
2025-02-24 10:00 ` E Shattow
2025-02-24 10:13 ` Heinrich Schuchardt
2025-02-25 13:43 ` E Shattow
2025-02-21 9:58 ` [PATCH v3 4/4] riscv: dts: starfive: remove duplicate itb entries Heinrich Schuchardt
2025-02-23 1:36 ` E Shattow [this message]
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=7baa4904-16cc-4bde-9b7b-72a1f2899645@freeshell.de \
--to=e@freeshell.de \
--cc=dmoo_dv@protonmail.com \
--cc=hal.feng@starfivetech.com \
--cc=heinrich.schuchardt@canonical.com \
--cc=leon@georgemail.eu \
--cc=lukas.funke@weidmueller.com \
--cc=minda.chen@starfivetech.com \
--cc=peter.lin@sifive.com \
--cc=rick@andestech.com \
--cc=sjg@chromium.org \
--cc=sumit.garg@linaro.org \
--cc=u-boot@lists.denx.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox