April 2023 - U-Boot

Uboot-Raspberry3b-Bare metal code
by Little Tree 29 Apr '23

29 Apr '23

Hi, I have written (copied from here https://github.com/bztsrc/raspi3-tutorial/tree/master/0F_executionlevel) a small bare metal program to switch exception level from EL2 to EL1 in Raspberry Pi 3B. The program is linked to load at the address 0x80000. The stripped version of the ELF is directly loaded through the "config.txt" file (without U-Boot) and it works. It is able to access the mini UART and print a message from EL1. Now I have modified the same program to load at the address (0x1000000) by modifying the linker (link.ld) file. Stripped it using objcopy. started the board with U-Boot. From U-boot prompt loaded the binary to the address 0x1000000 (using tftpboot). Then initiated the execution using go 0x1000000. This ends up in a board reset with the following error. U-Boot> go 0x1000000 ## Starting application at 0x01000000 ... "Synchronous Abort" handler, esr 0x86000004, far 0x13af5d330 elr: 00000000ffc92330 lr : 00000000ffc92330 (reloc) elr: 000000013af5d330 lr : 000000013af5d330 x0 : 0000000003000022 x1 : 0000000000000001 x2 : 0000000001000084 x3 : 0000000001000000 x4 : 0000000000000000 x5 : 0000000000001680 x6 : 0000000000000010 x7 : 0000000000000008 x8 : 0000000000000001 x9 : 0000000000000010 x10: 0000000000000004 x11: 000000003e96cea0 x12: 0000000000000001 x13: 000000003e96cea0 x14: 0000000000000000 x15: 0000000000000004 x16: 0000000000000000 x17: 0000000000000000 x18: 000000003af46d60 x19: 000000003af3e9c0 x20: 000000003b365f20 x21: 000000003af3eaa0 x22: 0000000000000000 x23: 000000000000000a x24: 0000000000000000 x25: 000000003af5d310 x26: 0000000000000000 x27: 0000000000000000 x28: 000000003af5d400 x29: 0000000000000060 Code: "Synchronous Abort" handler, esr 0x96000004, far 0x13af5d320 elr: 0000000000082b10 lr : 0000000000082af0 (reloc) elr: 000000003b34db10 lr : 000000003b34daf0 x0 : 000000003b3c05c5 x1 : 000000003ebe9a00 x2 : 0000000000000040 x3 : 000000000000003f x4 : 0000000000000000 x5 : 0000000000001680 x6 : 0000000000000010 x7 : 0000000000000008 x8 : 0000000000000001 x9 : 0000000000000010 x10: 0000000000000004 x11: 000000003eb2ea40 x12: 0000000000000001 x13: 000000003eb2ea40 x14: 0000000000000000 x15: 0000000000000004 x16: 0000000000000000 x17: 0000000000000000 x18: 000000003af46d60 x19: 000000013af5d320 x20: 000000003b3c5000 x21: 00000000fffffffc x22: 000000003b3c6dd5 x23: 000000003b3c05c5 x24: 0000000000000000 x25: 000000003af5d310 x26: 0000000000000000 x27: 0000000000000000 x28: 000000003af5d400 x29: 000000003af3e7f0 Code: d1004273 913756d6 aa0003f7 12800075 (b9400261) Resetting CPU ... I built U-Boot with default configuration for raspberry-pi3 and arm64. My program crashed on both of them. This error log is from BCM2835-arm64 U-Boot build. Is there any specific requirement for U-Boot which should be met by the bare metal program? I have been struggling with this for quite some time. Any help/hint is (to some documentation at lleast) greatly appreciated. Thanks a lot, Little

1 0

Please pull u-boot-dm
by Simon Glass 29 Apr '23

29 Apr '23

Hi Tom, https://source.denx.de/u-boot/custodians/u-boot-dm/-/pipelines/16143 The following changes since commit c9c2c95d4cd27fe0cd41fe13a863899d268f973c: Merge branch '2023-04-27-introduce-nvm-xip-block-storage-emulation' (2023-04-27 19:22:38 -0400) are available in the Git repository at: git://git.denx.de/u-boot-dm.git tags/dm-pull-28apr23 for you to fetch changes up to f43fc16812487289e98389f0f643d20c444f9c9c: fdt: Indicate that people should use the ofnode API (2023-04-28 11:52:38 -0600) ---------------------------------------------------------------- sandbox and fdt bug fixes / tweaks various other minor fixes ---------------------------------------------------------------- Andrew Davis (1): binman: Use unsigned long over typedef ulong Bin Meng (2): boot: vbe_simple: Fix vbe_simple_read_bootflow() dependency dm: core: Make aliases_lookup static Heinrich Schuchardt (9): sandbox: fix fall through in sandbox_flash_bulk() sandbox: fix sandbox_hub_submit_control_msg() sandbox: spi: sandbox_sf_process_cmd() missing fallthrough sandbox: mark sandbox_exit() as no return. common: static fdt_simplefb_enable_existing_node() MAINTAINERS: assign include/os.h sandbox: fix return type of os_filesize() sandbox: correct posix_types.h define patman: fix class TestFunctional Hugo Villeneuve (1): fdt_support: fix comments syntax error Jan Kiszka (1): tools: Fall back to importlib_resources on Python 3.6 Johan Jonker (1): core: fdtaddr: use map_sysmem() as cast for the return Jorge Ramirez-Ortiz (1): drivers: tee: sandbox: Fix SCP03 control emulator Marek Vasut (1): test: fdt: Fix copyright message Pavel Skripkin (1): sandbox: disable tracing before unmapping RAM Rasmus Villemoes (2): uclass: add uclass_find_device_by_phandle_id() helper dm: core: introduce uclass_get_device_by_of_path() Simon Glass (2): binman: Use expanduser instead of HOME fdt: Indicate that people should use the ofnode API Tom Rini (1): bootflow: Rework do_bootflow_menu() slightly MAINTAINERS | 1 + arch/sandbox/cpu/cpu.c | 2 +- arch/sandbox/cpu/os.c | 8 ++++++-- arch/sandbox/cpu/state.c | 5 +++++ arch/sandbox/include/asm/posix_types.h | 7 ++++--- arch/sandbox/include/asm/u-boot-sandbox.h | 2 +- boot/vbe_simple.c | 12 +++++++----- cmd/bootflow.c | 24 ++++++++++++------------ common/fdt_simplefb.c | 8 +++++++- common/fdt_support.c | 4 ++-- drivers/block/host_dev.c | 3 ++- drivers/core/fdtaddr.c | 6 +++++- drivers/core/of_access.c | 2 +- drivers/core/uclass.c | 50 +++++++++++++++++++++++++------------------------- drivers/mtd/spi/sandbox.c | 1 + drivers/sysreset/sysreset_sandbox.c | 1 - drivers/tee/sandbox.c | 15 +++++++++++---- drivers/usb/emul/sandbox_flash.c | 1 + drivers/usb/emul/sandbox_hub.c | 30 +++++++++--------------------- include/binman_sym.h | 8 ++++---- include/dm/uclass.h | 17 +++++++++++++++++ include/fdt_simplefb.h | 1 - include/os.h | 2 +- lib/fdtdec.c | 3 +++ test/cmd/fdt.c | 2 +- test/dm/test-fdt.c | 5 ++++- tools/binman/cmdline.py | 2 +- tools/binman/control.py | 6 +++++- tools/binman/test/blob_syms.c | 2 -- tools/binman/test/u_boot_binman_syms.c | 2 -- tools/binman/test/u_boot_binman_syms_size.c | 2 -- tools/buildman/control.py | 6 +++++- tools/patman/__main__.py | 6 +++++- tools/patman/func_test.py | 4 ++-- 34 files changed, 149 insertions(+), 101 deletions(-)

2 1

[PATCH v2 00/17] rockchip: eMMC fixes for RK3568 and support for RK3588
by Jonas Karlman 29 Apr '23

29 Apr '23

This series fixes eMMC HS400 modes on RK3568 and add support for RK3588. It has been tested with rock-3a-rk3568/rock5b-rk3588 defconfig and CONFIG_MMC_HS200_SUPPORT=y CONFIG_MMC_HS400_SUPPORT=y CONFIG_MMC_HS400_ES_SUPPORT=y CONFIG_MMC_SPEED_MODE_SET=y using the following command to switch mode and then read 512 MiB of data from eMMC into memory, => mmc dev 0 0 <mode> && mmc info && mmc read 10000000 2000 10000 for each of the modes below. 0 = MMC legacy 1 = MMC High Speed (26MHz) 3 = MMC High Speed (52MHz) 4 = MMC DDR52 (52MHz) 10 = HS200 (200MHz) 11 = HS400 (200MHz) 12 = HS400ES (200MHz) All reads have reported OK, prior to this some of these modes worked and others failed. Patch 1-2 fixes an issue with high-speed bit and uhs speed mode field. Patch 3-6 refactors the rk3568 driver to use set_clock and config_dll ops, so that clocks and regs are changed when output clock is disabled. Patch 7-10 continues refactoring and simplification of the driver. Patch 11-12 updates tap and delay values to fix HS400 modes on RK3568. Patch 13-15 adds support for RK3588 to driver and device tree. Patch 16-17 adds workarounds needed to use PIO mode in SPL to successfully load TF-A into SRAM when booting from eMMC on RK3588. Note that this series does not include any change to defconfigs to enable HS200/HS400/HS400ES modes. Changes in v2: - Add Signed-off-by tag and update commit message with a note from where reg-values originates from - Rename quirks to flags - Use driver data for hs200/hs400 txclk tapnum values - Change u-boot,dm-spl to bootph-pre-ram This series require working pinctrl, see [1]. A copy of this series and its dependencies can be found at [2]. [1] https://patchwork.ozlabs.org/project/uboot/patch/20230315153215.389809-1-eu… [2] https://github.com/Kwiboo/u-boot-rockchip/commits/rk35xx-emmc-v2 Jonas Karlman (16): mmc: sdhci: Fix HISPD bit handling for MMC HS 52MHz mode mmc: sdhci: Set UHS Mode Select field for UHS SDR25 mode mmc: rockchip_sdhci: Fix use of device private data mmc: rockchip_sdhci: Remove unneeded emmc_phy_init ops mmc: rockchip_sdhci: Add set_clock and config_dll sdhci_ops mmc: rockchip_sdhci: Use set_clock and config_dll sdhci_ops mmc: rockchip_sdhci: Refactor execute tuning error handling mmc: rockchip_sdhci: Update speed mode controls in set_ios_post mmc: rockchip_sdhci: Remove empty get_phy and set_enhanced_strobe ops mmc: rockchip_sdhci: Rearrange and simplify used regs and flags mmc: rockchip_sdhci: Fix HS400 and HS400ES mode on RK3568 rockchip: rk3568-rock-3a: Enable support for more eMMC modes mmc: rockchip_sdhci: Add support for RK3588 rockchip: rk3588-rock-5b: Include eMMC node in SPL dtb clk: rockchip: rk3588: Add limited TMCLK_EMMC clock support mmc: rockchip_sdhci: Limit number of blocks read in a single command Peter Geis (1): mmc: sdhci: Allow disabling of SDMA in SPL arch/arm/dts/rk3568-rock-3a-u-boot.dtsi | 8 + arch/arm/dts/rk3588-rock-5b-u-boot.dtsi | 12 +- arch/arm/dts/rk3588s-u-boot.dtsi | 4 + configs/rock5b-rk3588_defconfig | 1 + drivers/clk/rockchip/clk_rk3588.c | 2 + drivers/mmc/Kconfig | 8 + drivers/mmc/rockchip_sdhci.c | 309 +++++++++++++----------- drivers/mmc/sdhci.c | 13 +- 8 files changed, 215 insertions(+), 142 deletions(-) -- 2.40.0

4 43

[PATCH 0/1] pci: pcie_dw_rockchip: support fixed vcc3v3 regulator type
by John Clark 29 Apr '23

29 Apr '23

When trying to nvme boot rk3568 boards (tested with odroid-m1 and nanopi-r5s), it failes with error -19. This is because it is trying to set the voltage of a fixed-regulator to 3.3v. John Clark (1): pci: pcie_dw_rockchip: support fixed vcc3v3 regulator type drivers/pci/pcie_dw_rockchip.c | 7 +++++++ 1 file changed, 7 insertions(+) -- 2.39.2 (Apple Git-143)

2 2

[PATCH] dm: core: introduce uclass_get_device_by_of_path()
by Rasmus Villemoes 29 Apr '23

29 Apr '23

There's quite a few instances of board-specific code doing off = fdt_path_offset(gd->fdt_blob, ...); ... ret = uclass_get_device_by_of_offset(..., off, &dev); looking for an eeprom or a pmic via some alias. Such code can be simplified a little if we have a helper for directly getting a device via device tree path (including being given as an alias). Implement it in terms of ofnode rather than raw offsets so that this will work whether live tree is enabled or not. Signed-off-by: Rasmus Villemoes <rasmus.villemoes(a)prevas.dk> --- drivers/core/uclass.c | 6 ++++++ include/dm/uclass.h | 17 +++++++++++++++++ 2 files changed, 23 insertions(+) diff --git a/drivers/core/uclass.c b/drivers/core/uclass.c index 3a919104a6..56ea878701 100644 --- a/drivers/core/uclass.c +++ b/drivers/core/uclass.c @@ -544,6 +544,12 @@ int uclass_get_device_by_ofnode(enum uclass_id id, ofnode node, return uclass_get_device_tail(dev, ret, devp); } +int uclass_get_device_by_of_path(enum uclass_id id, const char *path, + struct udevice **devp) +{ + return uclass_get_device_by_ofnode(id, ofnode_path(path), devp); +} + #if CONFIG_IS_ENABLED(OF_CONTROL) int uclass_get_device_by_phandle_id(enum uclass_id id, uint phandle_id, struct udevice **devp) diff --git a/include/dm/uclass.h b/include/dm/uclass.h index ee15c92063..5c5fb9acac 100644 --- a/include/dm/uclass.h +++ b/include/dm/uclass.h @@ -264,6 +264,23 @@ int uclass_get_device_by_of_offset(enum uclass_id id, int node, int uclass_get_device_by_ofnode(enum uclass_id id, ofnode node, struct udevice **devp); +/** + * uclass_get_device_by_of_path() - Get a uclass device by device tree path + * + * This searches the devices in the uclass for one attached to the + * device tree node corresponding to the given path (which may also be + * an alias). + * + * The device is probed to activate it ready for use. + * + * @id: ID to look up + * @node: Device tree path to search for (if no such path then -ENODEV is returned) + * @devp: Returns pointer to device (there is only one for each node) + * Return: 0 if OK, -ve on error + */ +int uclass_get_device_by_of_path(enum uclass_id id, const char *path, + struct udevice **devp); + /** * uclass_get_device_by_phandle_id() - Get a uclass device by phandle id * -- 2.37.2

2 4

Enable SDP for Compulab IOT-GATE-iMX8 BOARD
by Parthiban 29 Apr '23

29 Apr '23

Dear All, Good day! Tried enabling SDP for "Compulab IOT-GATE-iMX8 BOARD" for booting over SDP and uuu, but fails to compile. Please find the below compilation error. SRAM overflow considers the whole data section of flash.bin or is it just SPL? .config is already minimal and only enabled required components. Also the USB dependency is tricky for me to fix as the gadget part depends on the host driver. Many thanks to narrow down. ``` aarch64-none-linux-gnu-ld.bfd: u-boot-spl section `.data' will not fit in region `.sram' aarch64-none-linux-gnu-ld.bfd: region `.sram' overflowed by 9816 bytes aarch64-none-linux-gnu-ld.bfd: drivers/usb/gadget/ci_udc.o: in function `usb_gadget_register_driver': u-boot/out/../drivers/usb/gadget/ci_udc.c:1026: undefined reference to `usb_setup_ehci_gadget' make[2]: *** [../scripts/Makefile.spl:527: spl/u-boot-spl] Error 1 make[1]: *** [u-boot/Makefile:2043: spl/u-boot-spl] Error 2 make[1]: Leaving directory 'u-boot/out' make: *** [Makefile:177: sub-make] Error 2 ``` -- Thanks, Parthiban N https://www.linumiz.com Berlin, Germany

1 0

[PATCH 1/1] doc: man-page for cp
by Heinrich Schuchardt 28 Apr '23

28 Apr '23

Add a man-page for the cp command. Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt(a)canonical.com> --- doc/usage/cmd/cp.rst | 83 ++++++++++++++++++++++++++++++++++++++++++++ doc/usage/index.rst | 1 + 2 files changed, 84 insertions(+) create mode 100644 doc/usage/cmd/cp.rst diff --git a/doc/usage/cmd/cp.rst b/doc/usage/cmd/cp.rst new file mode 100644 index 0000000000..897c0bb7df --- /dev/null +++ b/doc/usage/cmd/cp.rst @@ -0,0 +1,83 @@ +.. SPDX-License-Identifier: GPL-2.0+: + +cp command +========== + +Synopsis +-------- + +:: + + mm source target count + mm.b source target count + mm.w source target count + mm.l source target count + mm.q source target count + +Description +----------- + +The cp command is used to copy *count* words of memory from the *source* +address to the *target* address. If the *target* address points to NOR flash, +the flash is programmed. + +The word size is defined by the suffix defaulting to 32 bits: + +====== ============ +suffix word size +====== ============ +.b 8 bit words +.w 16 bit words +.l 32 bit words +.q 64 bit words +<none> 32 bit words +====== ============ + +source + source address, hexadecimal + +target + target address, hexadecimal + +count + number of words to be copied, hexadecimal + +Examples +-------- + +The example device has a NOR flash where the lower part of the flash is +protected. We first copy to RAM, then to unprotected flash. Last we try to +write to protectd flash. + +:: + + => mtd list + List of MTD devices: + * nor0 + - device: flash@0 + - parent: root_driver + - driver: cfi_flash + - path: /flash@0 + - type: NOR flash + - block size: 0x20000 bytes + - min I/O: 0x1 bytes + - 0x000000000000-0x000002000000 : "nor0" + => cp.b 4020000 5000000 200000 + => cp.b 4020000 1e00000 20000 + Copy to Flash... done + => cp.b 4020000 0 20000 + Copy to Flash... Can't write to protected Flash sectors + => + +Configuration +------------- + +The cp command is available if CONFIG_CMD_MEMORY=y. Support for 64 bit words +(cp.q) depends on CONFIG_MEM_SUPPORT_64BIT_DATA=y. Copying to flash depends on +CONFIG_MTD_NOR_FLASH=y. + +Return value +------------ + +The return value $? is set to 0 (true) if the command was successfully, +1 (false) otherwise. diff --git a/doc/usage/index.rst b/doc/usage/index.rst index cdf710919a..0fde130a54 100644 --- a/doc/usage/index.rst +++ b/doc/usage/index.rst @@ -41,6 +41,7 @@ Shell commands cmd/cmp cmd/coninfo cmd/conitrace + cmd/cp cmd/cyclic cmd/dm cmd/ebtupdate -- 2.39.2

2 3

[PATCH] sandbox: disable tracing before unmapping RAM
by Pavel Skripkin 28 Apr '23

28 Apr '23

Currently doing 'reset' command in sandbox with tracing enabled causes SIGSEV ``` Hit any key to stop autoboot: 0 => => => reset resetting ... Segmentation fault (core dumped) ``` Tracing callback uses RAM buffer for storing tracing reports, but state_uninit() function unmaps whole RAM, which causes SIGSEV on umapped memory inside tracing subsystem. Fix it by disabling tracing before unmapping memory Signed-off-by: Pavel Skripkin <paskripkin(a)gmail.com> --- arch/sandbox/cpu/state.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/arch/sandbox/cpu/state.c b/arch/sandbox/cpu/state.c index 69da378ab5..d8d5da25b1 100644 --- a/arch/sandbox/cpu/state.c +++ b/arch/sandbox/cpu/state.c @@ -10,6 +10,7 @@ #include <fdtdec.h> #include <log.h> #include <os.h> +#include <trace.h> #include <asm/malloc.h> #include <asm/state.h> #include <asm/test.h> @@ -525,6 +526,9 @@ int state_uninit(void) if (state->jumped_fname) os_unlink(state->jumped_fname); + /* Disable tracing before unmapping RAM */ + trace_set_enabled(0); + os_free(state->state_fdt); os_free(state->ram_buf); memset(state, '\0', sizeof(*state)); -- 2.39.2

3 6

[PATCH] uclass: add uclass_find_device_by_phandle_id() helper
by Rasmus Villemoes 28 Apr '23

28 Apr '23

The functions uclass_find_device_by_phandle() and uclass_get_device_by_phandle_id() both loop over a given uclass looking for a device with a given phandle. Factor that out to a common helper. For now, there are no (known potential) users of the new helper outside uclass.c, so make it static. Signed-off-by: Rasmus Villemoes <rasmus.villemoes(a)prevas.dk> --- drivers/core/uclass.c | 42 ++++++++++++++++++------------------------ 1 file changed, 18 insertions(+), 24 deletions(-) diff --git a/drivers/core/uclass.c b/drivers/core/uclass.c index 1762a0796d..3a919104a6 100644 --- a/drivers/core/uclass.c +++ b/drivers/core/uclass.c @@ -411,18 +411,14 @@ done: } #if CONFIG_IS_ENABLED(OF_REAL) -int uclass_find_device_by_phandle(enum uclass_id id, struct udevice *parent, - const char *name, struct udevice **devp) +static int +uclass_find_device_by_phandle_id(enum uclass_id id, uint find_phandle, + struct udevice **devp) { struct udevice *dev; struct uclass *uc; - int find_phandle; int ret; - *devp = NULL; - find_phandle = dev_read_u32_default(parent, name, -1); - if (find_phandle <= 0) - return -ENOENT; ret = uclass_get(id, &uc); if (ret) return ret; @@ -440,6 +436,19 @@ int uclass_find_device_by_phandle(enum uclass_id id, struct udevice *parent, return -ENODEV; } + +int uclass_find_device_by_phandle(enum uclass_id id, struct udevice *parent, + const char *name, struct udevice **devp) +{ + int find_phandle; + + *devp = NULL; + find_phandle = dev_read_u32_default(parent, name, -1); + if (find_phandle <= 0) + return -ENOENT; + + return uclass_find_device_by_phandle_id(id, find_phandle, devp); +} #endif int uclass_get_device_by_driver(enum uclass_id id, @@ -540,26 +549,11 @@ int uclass_get_device_by_phandle_id(enum uclass_id id, uint phandle_id, struct udevice **devp) { struct udevice *dev; - struct uclass *uc; int ret; *devp = NULL; - ret = uclass_get(id, &uc); - if (ret) - return ret; - - uclass_foreach_dev(dev, uc) { - uint phandle; - - phandle = dev_read_phandle(dev); - - if (phandle == phandle_id) { - *devp = dev; - return uclass_get_device_tail(dev, ret, devp); - } - } - - return -ENODEV; + ret = uclass_find_device_by_phandle_id(id, phandle_id, &dev); + return uclass_get_device_tail(dev, ret, devp); } int uclass_get_device_by_phandle(enum uclass_id id, struct udevice *parent, -- 2.37.2

2 2

[PATCH] binman: Use expanduser instead of HOME
by Simon Glass 28 Apr '23

28 Apr '23

There may not be a HOME environment variable, so use the os.expanduser() function instead. Signed-off-by: Simon Glass <sjg(a)chromium.org> --- tools/binman/bintool.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/binman/bintool.py b/tools/binman/bintool.py index 8fda13ff012..f460243e796 100644 --- a/tools/binman/bintool.py +++ b/tools/binman/bintool.py @@ -43,7 +43,7 @@ FETCH_NAMES = { # Status of tool fetching FETCHED, FAIL, PRESENT, STATUS_COUNT = range(4) -DOWNLOAD_DESTDIR = os.path.join(os.getenv('HOME'), 'bin') +DOWNLOAD_DESTDIR = os.path.expanduser('~/bin') class Bintool: """Tool which operates on binaries to help produce entry contents -- 2.40.0.634.g4ca3ef3211-goog

1 1