Hi Jonas,

    Thanks for your patch, and the ramboot with usb download do help bootloader developers.

On 2025/4/5 23:38, Jonas Karlman wrote:
The BootROM in Rockchip SoCs will enter maskrom mode when boot firmware
cannot be found in nand/spi/mmc storage.

In maskrom mode the USB OTG port can accept one of two custom commands.

Initially a 0x471 command to load TPL into SRAM. After TPL has been
executed and it has returned back-to-BROM, a 0x472 command to load SPL
into start of DRAM.

Add two binman images that can be used to RAM boot from maskrom mode:
- u-boot-rockchip-usb471.bin that contains TPL to init DRAM.
- u-boot-rockchip-usb472.bin that contains SPL and the normal FIT
   payload with i.e. U-Boot proper, TF-A and FDT.

This makes things quite complicated, needs so many new binaries and tools which

lead to confuse to users.

Let's make it simple for RAM boot with usb download so that user can use it easily:

use rkdeveloptool to download the loader and boot from RAM, and that's all.


Thanks,
- Kever
These images can be used with rkbin tools/boot_merger to create a loader
image to be used with rkdeveloptool or rockusb tools, e.g.:

   Create loader image:
     $ ../rkbin/tools/boot_merger ./RK3588MINIALL.ini

   Boot from maskrom:
     $ rkdeveloptool db u-boot-rockchip-rk3588-loader.bin
    or
     $ rockusb download-boot u-boot-rockchip-rk3588-loader.bin

Or directly with tools such as rkflashtool or rkusbboot:

   $ rkflashtool l < u-boot-rockchip-usb471.bin
   $ rkflashtool L < u-boot-rockchip-usb472.bin
  or
   $ rkusbboot u-boot-rockchip-usb471.bin u-boot-rockchip-usb472.bin

Signed-off-by: Jonas Karlman<jo...@kwiboo.se>
Tested-by: Arnaud Patard<arnaud.pat...@collabora.com>
---
v2: Use fit_template instead of common_part
v2: Collect t-b tag
---
  arch/arm/dts/rockchip-u-boot.dtsi       | 33 +++++++++++++++++++++++++
  arch/arm/mach-rockchip/Kconfig          |  8 ++++++
  arch/arm/mach-rockchip/spl-boot-order.c | 15 ++++++++---
  boot/Kconfig                            |  3 +++
  4 files changed, 56 insertions(+), 3 deletions(-)

diff --git a/arch/arm/dts/rockchip-u-boot.dtsi 
b/arch/arm/dts/rockchip-u-boot.dtsi
index 0dfd45bb9bed..dafae297d0cf 100644
--- a/arch/arm/dts/rockchip-u-boot.dtsi
+++ b/arch/arm/dts/rockchip-u-boot.dtsi
@@ -231,5 +231,38 @@
                };
        };
  #endif /* CONFIG_ROCKCHIP_SPI_IMAGE */
+
+#ifdef CONFIG_ROCKCHIP_MASKROM_IMAGE
+       simple-bin-usb471 {
+               filename = "u-boot-rockchip-usb471.bin";
+
+#ifdef CONFIG_ROCKCHIP_EXTERNAL_TPL
+               rockchip-tpl {
+               };
+#elif defined(CONFIG_TPL)
+               u-boot-tpl {
+                       no-write-symbols;
+               };
+#endif
+       };
+
+       simple-bin-usb472 {
+               filename = "u-boot-rockchip-usb472.bin";
+               pad-byte = <0x00>;
+
+               u-boot-spl {
+                       no-write-symbols;
+               };
+
+#ifdef HAS_FIT
+               fit {
+                       insert-template = <&fit_template>;
+#else
+               u-boot-img {
+#endif
+                       offset = <(CONFIG_SPL_LOAD_FIT_ADDRESS - 
CFG_SYS_SDRAM_BASE)>;
+               };
+       };
+#endif /* CONFIG_ROCKCHIP_MASKROM_IMAGE */
  };
  #endif /* CONFIG_SPL */
diff --git a/arch/arm/mach-rockchip/Kconfig b/arch/arm/mach-rockchip/Kconfig
index c9d8f9105083..e817bf46e565 100644
--- a/arch/arm/mach-rockchip/Kconfig
+++ b/arch/arm/mach-rockchip/Kconfig
@@ -607,6 +607,14 @@ config ROCKCHIP_SPI_IMAGE
          option to produce a SPI-flash image containing U-Boot. The image
          is built by binman. U-Boot sits near the start of the image.
+config ROCKCHIP_MASKROM_IMAGE
+       bool "Build a maskrom mode image for Rockchip"
+       depends on TPL || ROCKCHIP_EXTERNAL_TPL
+       select SPL_RAM_DEVICE
+       help
+         Rockchip SoCs support maskrom mode boot over USB. Enable this
+         option to produce maskrom mode boot images containing U-Boot.
+
  config LNX_KRNL_IMG_TEXT_OFFSET_BASE
        default TEXT_BASE
diff --git a/arch/arm/mach-rockchip/spl-boot-order.c b/arch/arm/mach-rockchip/spl-boot-order.c
index 3dce9b30898d..20cbee8c9ff4 100644
--- a/arch/arm/mach-rockchip/spl-boot-order.c
+++ b/arch/arm/mach-rockchip/spl-boot-order.c
@@ -8,7 +8,9 @@
  #include <log.h>
  #include <mmc.h>
  #include <spl.h>
+#include <asm/arch-rockchip/bootrom.h>
  #include <asm/global_data.h>
+#include <asm/io.h>
  #include <dm/uclass-internal.h>
#if CONFIG_IS_ENABLED(OF_LIBFDT)
@@ -98,15 +100,22 @@ __weak const char *board_spl_was_booted_from(void)
void board_boot_order(u32 *spl_boot_list)
  {
+       int idx = 0;
+
+       /* Add RAM boot for maskrom mode boot over USB */
+       if (BROM_BOOTSOURCE_ID_ADDR && CONFIG_IS_ENABLED(RAM_DEVICE) &&
+           readl(BROM_BOOTSOURCE_ID_ADDR) == BROM_BOOTSOURCE_USB) {
+               spl_boot_list[idx++] = BOOT_DEVICE_RAM;
+       }
+
        /* In case of no fdt (or only plat), use spl_boot_device() */
        if (!CONFIG_IS_ENABLED(OF_CONTROL) || CONFIG_IS_ENABLED(OF_PLATDATA)) {
-               spl_boot_list[0] = spl_boot_device();
+               spl_boot_list[idx++] = spl_boot_device();
                return;
        }
const void *blob = gd->fdt_blob;
        int chosen_node = fdt_path_offset(blob, "/chosen");
-       int idx = 0;
        int elem;
        int boot_device;
        int node;
@@ -115,7 +124,7 @@ void board_boot_order(u32 *spl_boot_list)
        if (chosen_node < 0) {
                debug("%s: /chosen not found, using spl_boot_device()\n",
                      __func__);
-               spl_boot_list[0] = spl_boot_device();
+               spl_boot_list[idx++] = spl_boot_device();
                return;
        }
diff --git a/boot/Kconfig b/boot/Kconfig
index f101200ba7a5..abfc9eaf325f 100644
--- a/boot/Kconfig
+++ b/boot/Kconfig
@@ -246,6 +246,9 @@ config SPL_LOAD_FIT_ADDRESS
        hex "load address of fit image"
        depends on SPL_LOAD_FIT
        default 0x44000000 if ARCH_IMX8M
+       default 0x60080000 if ARCH_ROCKCHIP && SPL_TEXT_BASE = 0x60000000
+       default 0x40200000 if ARCH_ROCKCHIP && SPL_TEXT_BASE = 0x40000000
+       default 0x00200000 if ARCH_ROCKCHIP && SPL_TEXT_BASE = 0x00000000
        default 0x0
        help
          Specify the load address of the fit image that will be loaded

Reply via email to