From: Andrew Abbott <andrew@mirx.dev>
To: u-boot@lists.denx.de
Cc: Andrew Abbott <andrew@mirx.dev>, Simon Glass <sjg@chromium.org>,
Philipp Tomsich <philipp.tomsich@vrull.eu>,
Kever Yang <kever.yang@rock-chips.com>,
Peter Robinson <pbrobinson@gmail.com>,
Akash Gajjar <akash@openedev.com>,
Jagan Teki <jagan@amarulasolutions.com>,
Samuel Dionne-Riel <samuel@dionne-riel.com>
Subject: [RFC PATCH 6/8] rockchip: Enable binman for ARM64
Date: Mon, 25 Apr 2022 10:03:27 +1000 [thread overview]
Message-ID: <20220425000329.28124-7-andrew@mirx.dev> (raw)
In-Reply-To: <20220425000329.28124-1-andrew@mirx.dev>
Binman is now being used to build the final flashable images for
Rockchip devices, thus enabling it for all Rockchip targets here. But
it is not yet being used to generate the FIT image (u-boot.itb),
thus we need to force it to be built.
Signed-off-by: Andrew Abbott <andrew@mirx.dev>
---
Question: Will this causes issues with eg. Chromebook gru/bob, which build
u-boot.itb with binman already?
---
Kconfig | 4 ++--
arch/arm/Kconfig | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/Kconfig b/Kconfig
index bdae59e06f..99e5491f08 100644
--- a/Kconfig
+++ b/Kconfig
@@ -406,8 +406,8 @@ config BUILD_TARGET
default "u-boot-with-spl.sfp" if TARGET_SOCFPGA_GEN5
default "u-boot-spl.kwb" if ARCH_MVEBU && SPL
default "u-boot-elf.srec" if RCAR_GEN3
- default "u-boot.itb" if !BINMAN && SPL_LOAD_FIT && (ARCH_ROCKCHIP || \
- ARCH_SUNXI || RISCV || ARCH_ZYNQMP)
+ default "u-boot.itb" if ARCH_ROCKCHIP || (!BINMAN && SPL_LOAD_FIT && \
+ (ARCH_SUNXI || RISCV || ARCH_ZYNQMP))
default "u-boot.kwb" if ARCH_KIRKWOOD
default "u-boot-with-spl.bin" if ARCH_AT91 && SPL_NAND_SUPPORT
default "u-boot-with-spl.imx" if ARCH_MX6 && SPL
diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
index 57946f61fa..7697c74edf 100644
--- a/arch/arm/Kconfig
+++ b/arch/arm/Kconfig
@@ -1961,7 +1961,7 @@ config ARCH_STM32MP
config ARCH_ROCKCHIP
bool "Support Rockchip SoCs"
select BLK
- select BINMAN if SPL_OPTEE || (SPL && !ARM64)
+ select BINMAN if SPL
select DM
select DM_GPIO
select DM_I2C
--
2.35.3
next prev parent reply other threads:[~2022-04-25 0:05 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-04-25 0:03 [RFC PATCH 0/8] Build Rockchip final images using binman Andrew Abbott
2022-04-25 0:03 ` [RFC PATCH 1/8] binman: mkimage: Support ':'-separated inputs Andrew Abbott
2022-04-25 0:03 ` [RFC PATCH 2/8] rockchip: Add binman definitions for final images Andrew Abbott
2022-04-27 9:22 ` Johan Jonker
2022-05-04 9:52 ` Andrew Abbott
2022-05-19 11:35 ` Alper Nebi Yasak
2022-04-25 0:03 ` [RFC PATCH 3/8] soc: rockchip: Include common U-Boot dtsi file Andrew Abbott
2022-04-25 0:03 ` [RFC PATCH 4/8] board: rockchip: Move SPI U-Boot offset to config Andrew Abbott
2022-04-25 0:03 ` [RFC PATCH 5/8] rockchip: Remove obsolete Makefile targets Andrew Abbott
2022-04-25 0:03 ` Andrew Abbott [this message]
2022-04-25 0:03 ` [RFC PATCH 7/8] doc: rockchip: Update for new binman image generation Andrew Abbott
2022-04-25 0:03 ` [RFC PATCH 8/8] board: rockpro64: Enable building SPI image Andrew Abbott
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=20220425000329.28124-7-andrew@mirx.dev \
--to=andrew@mirx.dev \
--cc=akash@openedev.com \
--cc=jagan@amarulasolutions.com \
--cc=kever.yang@rock-chips.com \
--cc=pbrobinson@gmail.com \
--cc=philipp.tomsich@vrull.eu \
--cc=samuel@dionne-riel.com \
--cc=sjg@chromium.org \
--cc=u-boot@lists.denx.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox