July 2023 - U-Boot

[PATCH] efi_driver: fix duplicate efiblk#0 issue
by Masahisa Kojima 03 Jul '23

03 Jul '23

The devnum value of the blk_desc structure starts from 0, current efi_bl_create_block_device() function creates two "efiblk#0" devices for the cases that blk_find_max_devnum() returns -ENODEV and blk_find_max_devnum() returns 0(one device found in this case). The devnum value for the "efiblk" name needs to be incremented. Signed-off-by: Masahisa Kojima <masahisa.kojima(a)linaro.org> --- lib/efi_driver/efi_block_device.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/lib/efi_driver/efi_block_device.c b/lib/efi_driver/efi_block_device.c index add00eeebb..e37bfe6e80 100644 --- a/lib/efi_driver/efi_block_device.c +++ b/lib/efi_driver/efi_block_device.c @@ -129,6 +129,8 @@ efi_bl_create_block_device(efi_handle_t handle, void *interface) devnum = 0; else if (devnum < 0) return EFI_OUT_OF_RESOURCES; + else + devnum++; /* device found, note that devnum starts from 0 */ name = calloc(1, 18); /* strlen("efiblk#2147483648") + 1 */ if (!name) -- 2.34.1

3 4

[PATCH 0/6] Support for AC5X NAND and AT-x240 board
by Chris Packham 03 Jul '23

03 Jul '23

This series adds support for the NAND flash controller on the AC5X SoC and adds support for the Allied Telesis x240 board. I've also included 2 unrelated changes. "arm: mvebu: Remove unused alias from RC AC5X dts" removes an unused alias from the dts. This was in the neighborhood of the x240 so I included it. "mtd: nand: pxa3xx: Enable devbus/nand arbiter on Armada 8K" allows using the NFC and device bus at the same time. This is needed for another board (CN9130 based) that I'll hopefully get upstream in the near future. I figured I'd include it now since I was touching the NAND driver. Chris Packham (6): arm: mvebu: ac5: Add nand-controller node arm: mvebu: ac5: Define mvebu_get_nand_clock() mtd: nand: pxa3xx: Add support for the Marvell AC5 SoC mtd: nand: pxa3xx: Enable devbus/nand arbiter on Armada 8K arm: mvebu: Add Allied Telesis x240 board arm: mvebu: Remove unused alias from RC AC5X dts arch/arm/dts/Makefile | 3 +- arch/arm/dts/ac5-98dx25xx.dtsi | 9 ++ arch/arm/dts/ac5-98dx35xx-atl-x240.dts | 212 +++++++++++++++++++++++++ arch/arm/dts/ac5-98dx35xx-rd.dts | 1 - arch/arm/mach-mvebu/Kconfig | 7 + arch/arm/mach-mvebu/alleycat5/soc.c | 6 + board/alliedtelesis/x240/MAINTAINERS | 7 + board/alliedtelesis/x240/Makefile | 6 + board/alliedtelesis/x240/x240.c | 13 ++ configs/x240_defconfig | 87 ++++++++++ drivers/mtd/nand/raw/pxa3xx_nand.c | 20 ++- include/configs/x240.h | 37 +++++ 12 files changed, 402 insertions(+), 6 deletions(-) create mode 100644 arch/arm/dts/ac5-98dx35xx-atl-x240.dts create mode 100644 board/alliedtelesis/x240/MAINTAINERS create mode 100644 board/alliedtelesis/x240/Makefile create mode 100644 board/alliedtelesis/x240/x240.c create mode 100644 configs/x240_defconfig create mode 100644 include/configs/x240.h -- 2.41.0

1 7

Pull request efi-2023-07-rc6
by Heinrich Schuchardt 03 Jul '23

03 Jul '23

Dear Tom, The following changes since commit 5fa30f2351ac3c0458069896bc868eae927df410: smegw01: Fix wrong symbol override (2023-06-29 09:55:50 -0400) are available in the Git repository at: https://source.denx.de/u-boot/custodians/u-boot-efi.git tags/efi-2023-07-rc6 for you to fetch changes up to 2b17dd1d9d8ca2cfc7f5e964384d4d8418904c35: ARM: arm11: Add C wrapper for allow_unaligned() (2023-07-01 17:29:15 +0200) Gitlab CI showed no issues: https://source.denx.de/u-boot/custodians/u-boot-efi/-/pipelines/16741 ---------------------------------------------------------------- Pull request efi-2023-07-rc6 Documentation: * man-pages for the loads and saves commands UEFI: * fix implementation of allow_unaligned() for armv7 and arm11 ---------------------------------------------------------------- Heinrich Schuchardt (3): doc: loads man-page doc: saves man-page doc: fix typo loady in loadb man-page Marek Vasut (2): ARM: armv7: Add C wrapper for allow_unaligned() ARM: arm11: Add C wrapper for allow_unaligned() arch/arm/cpu/arm11/cpu.c | 6 +++ arch/arm/cpu/arm11/sctlr.S | 6 +-- arch/arm/cpu/armv7/cpu.c | 5 +++ arch/arm/cpu/armv7/sctlr.S | 6 +-- arch/arm/include/asm/arm11.h | 12 ++++++ arch/arm/include/asm/armv7.h | 1 + doc/usage/cmd/loadb.rst | 2 +- doc/usage/cmd/loads.rst | 96 ++++++++++++++++++++++++++++++++++++++++++++ doc/usage/cmd/saves.rst | 88 ++++++++++++++++++++++++++++++++++++++++ doc/usage/index.rst | 2 + 10 files changed, 217 insertions(+), 7 deletions(-) create mode 100644 arch/arm/include/asm/arm11.h create mode 100644 doc/usage/cmd/loads.rst create mode 100644 doc/usage/cmd/saves.rst

2 1

Current way of using pinctrl-mvebu
by Chris Packham 03 Jul '23

03 Jul '23

Hi, I'm looking to upstream support for a new board with the Marvell AC5X SoC and some NAND driver changes to support the SoC/board. I've got things working when chain loading vendor based u-boot -> upstream u-boot but when I boot directly the NAND controller reports "pxa3xx-nand nand-controller@805b0000: Ready timeout!!!". I think this is because the multi purpose pins are not in NAND/DEV mode. When chainloading the vendor u-boot has already done this so the driver works. I notice that pinctrl-mvebu.c just expects a single fixed pin-func property the covers all the functions required. How is the probe for this supposed to be configured? Thanks, Chris

1 1

[PATCH v2 0/3] Enable U-Boot at Virtualbox to boot images
by thomas.mittelstaedt＠de.bosch.com 02 Jul '23

02 Jul '23

From: Thomas Mittelstaedt <thomas.mittelstaedt(a)de.bosch.com> The changes are needed to get U-Boot to be started at VirtualBox images supporting distro boot capability. Atm the patch "pci: coreboot: Don't read regions when booting" from Simon Glass is not integrated yet. So AHCI and IDE devices are not handled correctly without this patch. (https://patchwork.ozlabs.org/project/uboot/patch/20230220194927.476708-8-sj…) Changes since v1: - Remove prompt change commit - Add details in commit messages - defconfig now handled with savedefconfig option at make - Unneccessary option CONFIG_CMD_SETEXPR no more set - First patch was set with mtt2hi and thomas.mittelstaedt(a)bosch.com. Now set with my usual id Thomas Mittelstaedt <thomas.mittelstaedt(a)de.bosch.com> Thomas Mittelstaedt (3): X86: Add support for SCSI devices X86: Add support for distro boot X86: pxeboot: bugfix: Set variable for size of initrd boot/pxe_utils.c | 2 +- configs/efi-x86_payload64_defconfig | 18 +++++------------- include/configs/efi-x86_payload.h | 11 +++++++++++ 3 files changed, 17 insertions(+), 14 deletions(-) -- 2.30.2

4 11

modified UBoot to include I2C communication
by Andy Goh 02 Jul '23

02 Jul '23

Dear Sir, I am working on a project based on NXP's iMX8M mini board, to include I2C library in the UBoot , I had browse thru UBoot website below, could anyone kindly advise which source file to amend and what are the code to include for I2C? like CONFIG_SYS_NUM_I2C_BUSES etc... is the modification to be done on Yocto project? Any help on source code or documentation are welcome, thanks. https://source.denx.de/u-boot/u-boot/tree/master/ Cheers, Andy andy.goh(a)stengg.com<mailto:andy.goh@stengg.com> This email is confidential and may also be privileged. If this email has been sent to you in error, please delete it immediately and notify us. Please do not copy, distribute, or disseminate part or whole of this email if you are not the intended recipient or if you have not been authorized to do so. We reserve the right, to the extent and under circumstances permitted by applicable laws, to retain, monitor, and intercept email messages to and from our systems. Thank you.

2 1

[PATCH 00/12] Sign Xilinx ZynqMP SPL/FSBL boot images using binman
by lukas.funke-oss＠weidmueller.com 02 Jul '23

02 Jul '23

From: Lukas Funke <lukas.funke(a)weidmueller.com> This series adds two etypes to create a verified boot chain for Xilinx ZynqMP devices. The first etype 'xilinx_fsbl_auth' is used to create a bootable, signed image for ZynqMP boards using the Xilinx Bootgen tool. The second etype 'u_boot_spl_pubkey_dtb' is used to add a '/signature' node to the SPL. The public key in the signature is read from a certificate file and added using the 'fdt_add_pubkey' tool. The series also contains the corresponding btool for calling 'bootgen' and 'fdt_add_pubkey' The following block shows an example on how to use this functionality: spl { filename = "boot.signed.bin"; xilinx_fsbl_auth { psk-filename = "psk0.pem"; ssk-filename = "ssk0.pem"; auth-params = "ppk_select=0", "spk_id=0x00000000"; u_boot_spl_nodtb { }; u_boot_spl_pubkey_dtb { algo = "sha384,rsa4096"; required = "conf"; key-name = "dev"; }; }; }; Lukas Funke (12): binman: elf: Check for ELF_TOOLS availability and remove extra semicolon binman: mkimage: Remove extra colon binman: Don't decompress data while signing binman: blob_dtb: Add fake_size argument to ObtainContents() binman: doc: Add documentation for fdt_add_pubkey bintool binman: ftest: Add test for u_boot_spl_pubkey_dtb binman: btool: Add fdt_add_pubkey as btool binman: etype: Add u_boot_spl_pubkey_dtb etype binman: doc: Add documentation for Xilinx Bootgen bintool binman: btool: Add Xilinx Bootgen btool binman: ftest: Add test for xilinx_fsbl_auth etype binman: etype: Add xilinx_fsbl_auth etype tools/binman/bintools.rst | 22 +++ tools/binman/btool/bootgen.py | 82 +++++++++ tools/binman/btool/fdt_add_pubkey.py | 67 +++++++ tools/binman/control.py | 2 +- tools/binman/elf.py | 10 +- tools/binman/etype/blob_dtb.py | 2 +- tools/binman/etype/mkimage.py | 2 +- tools/binman/etype/u_boot_spl_pubkey_dtb.py | 105 +++++++++++ tools/binman/etype/xilinx_fsbl_auth.py | 186 ++++++++++++++++++++ tools/binman/ftest.py | 42 ++++- tools/binman/test/280_xilinx_fsb_auth.dts | 22 +++ tools/binman/test/281_spl_pubkey_dtb.dts | 16 ++ 12 files changed, 550 insertions(+), 8 deletions(-) create mode 100644 tools/binman/btool/bootgen.py create mode 100644 tools/binman/btool/fdt_add_pubkey.py create mode 100644 tools/binman/etype/u_boot_spl_pubkey_dtb.py create mode 100644 tools/binman/etype/xilinx_fsbl_auth.py create mode 100644 tools/binman/test/280_xilinx_fsb_auth.dts create mode 100644 tools/binman/test/281_spl_pubkey_dtb.dts -- 2.30.2

4 25

[PATCH 00/58] buildman: Refactor code and correct some pylint warnings
by Simon Glass 02 Jul '23

02 Jul '23

The buildman code has grown considerable since it was originally written. In places it can be quite hard to understanding and work with. This series improves things, particularly in the control and builder parts, splitting functions into smaller sizes. More work remains in buildthead._write_result() and other places. Simon Glass (58): buildman: Tidy up pylint warnings in main buildman: Convert camel case in control.py buildman: Fix most pylint warnings in control buildman: Move full-help processing to main buildman: Move series calculations into a separate function buildman: Move fetch-arch code into a separate function buildman: Move logic for -A up a little buildman: Drop use of builder in show_actions() buildman: Move dry-run handling higher in do_buildman() buildman: Move board-selection code into a function bulidman: Move more code to determine_series() buildman: Move Boards-object code into a function bulidman: Move toolchain handling to a functoin bulldman: Set up output_dir earlier buildman: Move output-file setup into one place buildman: Pass option values to get_action_summary() buildman: Pass option values to show_actions() buildman: Build option-adjusting into a function buildman: Move counting of commits into a function buildman: Move setting up the output dir into a function buildman: Move commit numbering into determine_series() buildman: Avoid too many returns in do_buildman() buildman: Move remaining builder properties to constructor buildman: Tweak commits and show_bloat buildman: Moving running of the builder into a function buildman: Drop some unnecessary variables buildman: Adjust show_toolchain_prefix() to not return buildman: Move checking for make into run_builder() buildman: Move getting the adjust_cfg into run_builder() buildman: Use get_alow_missing() directly to avoid var buildman: Create a function to get number of built commits buildman: Convert camel case in cmdline.py buildman: Correct most pylint warnings in cmdline buildman: Add a test for --boards buildman: Convert to argparse buildman: Convert camel case in bsettings.py buildman: Convert camel case in builder.py buildman: Split parser creation in two buildman: Convert camel case in builderthread.py buildman: Correct most pylint warnings in builderthread buildman: Export _get_output_dir() to avoid warnings buildman: Correct invalid use of out_dir variable buildman: Drop unnecessary assignment of config_out buildman: Start a function to set up the make arguments buildman: Move setting of toolchain arguments to _build_args() buildman: Move more things into _build_args() buildman: Convert config_out to string IO buildman: Move reconfigure code into its own function buildman: Move bulid code into its own function buildman: Move reading of the done file into a function buildman: Move code to remove old outputs buildman: Move code to decide output dirs buildman: Move checkout code to a separate function buildman: Create a function to handle config and build buildman: Avoid passing result into _read_done_file() buildman: Tidy up reporting of a toolchain error buildman: Tidy up some comments in builderthread buildman: Move copy_files() out ot BuilderThread class tools/buildman/bsettings.py | 14 +- tools/buildman/builder.py | 262 +++++------ tools/buildman/builderthread.py | 652 ++++++++++++++++----------- tools/buildman/buildman.rst | 2 +- tools/buildman/cmdline.py | 168 ++++--- tools/buildman/control.py | 756 +++++++++++++++++++++----------- tools/buildman/func_test.py | 42 +- tools/buildman/main.py | 44 +- tools/buildman/test.py | 28 +- tools/buildman/toolchain.py | 14 +- tools/moveconfig.py | 2 +- 11 files changed, 1213 insertions(+), 771 deletions(-) -- 2.41.0.255.g8b1d071c50-goog

1 58

[RFC] rockchip: spl: Enable caches to speed up checksum validation
by Jonas Karlman 02 Jul '23

02 Jul '23

FIT checksum validation is very slow in SPL due to D-cache not being enabled. Enable caches in SPL to speed up FIT checksum validation, from seconds to milliseconds. Signed-off-by: Jonas Karlman <jonas(a)kwiboo.se> --- I am not sure if it is even a good idea to enable caches in SPL, if this is the best way to enable caches and maybe this should be behind a Kconfig option? Have only been able to runtime test this on RK3328, RK3399 and RK3568. Any advice on this would be grateful. Regards, Jonas arch/arm/mach-rockchip/spl.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/arch/arm/mach-rockchip/spl.c b/arch/arm/mach-rockchip/spl.c index 30be64042521..29fe3353105b 100644 --- a/arch/arm/mach-rockchip/spl.c +++ b/arch/arm/mach-rockchip/spl.c @@ -149,6 +149,10 @@ void board_init_f(ulong dummy) } gd->ram_top = gd->ram_base + get_effective_memsize(); gd->ram_top = board_get_usable_ram_top(gd->ram_size); + gd->relocaddr = gd->ram_top; + + arch_reserve_mmu(); + enable_caches(); #endif preloader_console_init(); } -- 2.41.0

2 2

[PATCH] usb: dwc3-generic: Ensure reset GPIO is configured as an output
by Peter Korsgaard 02 Jul '23

02 Jul '23

GPIOD_ACTIVE_LOW is not enough to configure a GPIO as an output, we need GPIOD_IS_OUT as well. Fixes: b252d79b0936d60b ("usb: dwc3: Add support to reset usb ULPI phy") Signed-off-by: Peter Korsgaard <peter(a)korsgaard.com> --- drivers/usb/dwc3/dwc3-generic.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/dwc3/dwc3-generic.c b/drivers/usb/dwc3/dwc3-generic.c index 66da5a8d6f..35e4b36a69 100644 --- a/drivers/usb/dwc3/dwc3-generic.c +++ b/drivers/usb/dwc3/dwc3-generic.c @@ -105,7 +105,7 @@ static int dwc3_generic_probe(struct udevice *dev, if (CONFIG_IS_ENABLED(DM_GPIO) && device_is_compatible(dev->parent, "xlnx,zynqmp-dwc3")) { priv->ulpi_reset = devm_gpiod_get_optional(dev->parent, "reset", - GPIOD_ACTIVE_LOW); + GPIOD_IS_OUT | GPIOD_ACTIVE_LOW); /* property is optional, don't return error! */ if (priv->ulpi_reset) { /* Toggle ulpi to reset the phy. */ -- 2.30.2

2 1