September 2025 - Concept

ARM64 EFI_STUB on SM6115
by Philippe Simons 01 Oct '25

01 Oct '25

I'm currently trying to bring a SM6115 device on Linux but... without much success... I have replaced my ABL with qcomm/abl2eps which allows me to boot an efi application. I've tried to use ARM64 EFI STUB for that but the resulting u-boot-efi-payload.efi doesn't look like an PE32+ binary.... file u-boot-payload.efi u-boot-payload.efi: data Could you help me Thanks

2 13

[PATCH v2 00/15] boot: Support priority for global bootmeths
by Simon Glass 30 Sep '25

30 Sep '25

From: Simon Glass <sjg(a)chromium.org> At present global bootmeths always run first, before all other bootmeths. Optimisations in the code take advantage of this, putting them at the end, so they can be used once and then forgotten. In some cases it is useful to run global bootmeths later in the boot. For example, the EFI-bootmgr bootmeth may itself scan devices and the network, so running it first can hold up the boot significantly for boards not actually relying on EFI-bootmgr to boot. This series introduces a new field in global bootmeths which indicates the priority, using the same scheme as is used with bootdev hunters. Thus it is possible to insert the EFI-bootmgr bootmeth just before the hunter for network bootdevs is invoked. Despite the simplicity of the concept and the relatively small series, this is a fairly significant enhancement. It is also quite tricky to implement, largely due to the way the original code was written, with global bootmeths being a small, size-optimised add-on to the original bootstd implementation. For now we only allow each global bootmeth to run at most once, but this implementation is written in a way that we could relax that if needed. Then the bootmeth itself could decide whether to run at any particular point in the bootflow iteration. Size growth is about 390 bytes on Thumb2 (e.g. firefly-rk3288) if CONFIG_BOOTMETH_GLOBAL is enabled, which it normally is. With that disabled (which saves about 4K on the same platform), there is no growth. Changes in v2: - Add a log_err() for an invalid state - Update commit message to indicate the bootmeth will always run - Document how the priority was chosen - Add a bit more detail suggested by Heinrich Simon Glass (15): boot: Improve comments related to global bootmeths boot: Add more debugging to iter_incr() boot: Move showing of bootflows out of the command boot: Add a new test for global bootmeths boot: Update first_glob_method when dropping a bootmeth boot: Add a flag for whether there are global bootmeths boot: Keep track of which bootmeths have been used boot: Move preparing bootdev into a function boot: Support rescanning the global bootmeths boot: Only run global bootmeths once each boot: Implement a priority for global bootmeths boot: Don't change the method count after global bootmeths boot: Run global bootmeths after all bootdevs are exhausted boot: Run the EFI bootmgr just before network devices boot: doc: Update for new global-bootmeth features boot/bootflow.c | 277 +++++++++++++++++++++++++++++-- boot/bootmeth-uclass.c | 15 +- boot/bootmeth_efi_mgr.c | 9 + cmd/bootflow.c | 68 +------- doc/develop/bootstd/overview.rst | 40 ++++- include/bootflow.h | 29 +++- include/bootmeth.h | 4 + test/boot/bootflow.c | 73 +++++++- 8 files changed, 419 insertions(+), 96 deletions(-) -- 2.43.0 base-commit: 2617d1179c27ba51e9eb7996b7062a0205c587a9 branch: glob2

1 15

[PATCH 00/15] boot: Support priority for global bootmeths
by Simon Glass 30 Sep '25

30 Sep '25

From: Simon Glass <sjg(a)chromium.org> At present global bootmeths always run first, before all other bootmeths. Optimisations in the code take advantage of this, putting them at the end, so they can be used once and then forgotten. In some cases it is useful to run global bootmeths later in the boot. For example, the EFI-bootmgr bootmeth may itself scan devices and the network, so running it first can hold up the boot significantly for boards not actually relying on EFI-bootmgr to boot. This series introduces a new field in global bootmeths which indicates the priority, using the same scheme as is used with bootdev hunters. Thus it is possible to insert the EFI-bootmgr bootmeth just before the hunter for network bootdevs is invoked. Despite the simplicity of the concept and the relatively small series, this is a fairly significant enhancement. It is also quite tricky to implement, largely due to the way the original code was written, with global bootmeths being a small, size-optimised add-on to the original bootstd implementation. For now we only allow each global bootmeth to run at most once, but this implementation is written in a way that we could relax that if needed. Then the bootmeth itself could decide whether to run at any particular point in the bootflow iteration. Size growth is about 390 bytes on Thumb2 (e.g. firefly-rk3288) if CONFIG_BOOTMETH_GLOBAL is enabled, which it normally is. With that disabled (which saves about 4K on the same platform), there is no growth. Simon Glass (15): boot: Improve comments related to global bootmeths boot: Add more debugging to iter_incr() boot: Move showing of bootflows out of the command boot: Add a new test for global bootmeths boot: Update first_glob_method when dropping a bootmeth boot: Add a flag for whether there are global bootmeths boot: Keep track of which bootmeths have been used boot: Move preparing bootdev into a function boot: Support rescanning the global bootmeths boot: Only run global bootmeths once each boot: Implement a priority for global bootmeths boot: Don't change the method count after global bootmeths boot: Run global bootmeths after all bootdevs are exhausted boot: Run the EFI bootmgr just before network devices boot: doc: Update for new global-bootmeth features boot/bootflow.c | 275 +++++++++++++++++++++++++++++-- boot/bootmeth-uclass.c | 15 +- boot/bootmeth_efi_mgr.c | 1 + cmd/bootflow.c | 68 +------- doc/develop/bootstd/overview.rst | 26 ++- include/bootflow.h | 29 +++- include/bootmeth.h | 4 + test/boot/bootflow.c | 73 +++++++- 8 files changed, 397 insertions(+), 94 deletions(-) -- 2.43.0 base-commit: 2617d1179c27ba51e9eb7996b7062a0205c587a9 branch: glob

3 23

subscribe
by Heinrich Schuchardt 30 Sep '25

30 Sep '25

1 0

[PATCH 1/2] WIP: Revert "efi: arm: Simplify the crt0 file and update link script"
by Simon Glass 26 Sep '25

26 Sep '25

From: Simon Glass <sjg(a)chromium.org> Unfortunately the arm64 toolchain does not correctly add a header when buiding the payload. Restore the manual header for now. This change cannot be applied since it will break the app, but it should allow for experimentation. This reverts commit 7fa6a68131d154c5ab1ee79c5a50fdb10fdd63a6. Signed-off-by: Simon Glass <sjg(a)chromium.org> --- arch/arm/config.mk | 10 -- arch/arm/cpu/armv8/config.mk | 6 +- arch/arm/lib/crt0_aarch64_efi.S | 156 ++++++++++++++++++++++++++----- arch/arm/lib/elf_aarch64_efi.lds | 45 +++------ board/efi/efi-arm_app/config.mk | 4 - 5 files changed, 151 insertions(+), 70 deletions(-) delete mode 100644 board/efi/efi-arm_app/config.mk diff --git a/arch/arm/config.mk b/arch/arm/config.mk index 38c49c877c4..c8b5bcda8e4 100644 --- a/arch/arm/config.mk +++ b/arch/arm/config.mk @@ -146,9 +146,6 @@ OBJCOPYFLAGS += -j .text -j .secure_text -j .secure_data -j .rodata -j .hash \ -j .binman_sym_table -j .text_rest endif -OBJCOPYFLAGS_EFI += $(OBJCOPYFLAGS) -j .dynamic -j .rela -j .reloc \ - -j .embedded_dtb - # if a dtb section exists we always have to include it # there are only two cases where it is generated # 1) OF_EMBEDED is turned on @@ -178,13 +175,6 @@ endif endif endif -ifdef CONFIG_ARM64 -EFI_LDS := elf_aarch64_efi.lds -EFI_CRT0 := crt0_aarch64_efi.o -EFI_RELOC := reloc_aarch64_efi.o -EFI_TARGET := --target=efi-app-aarch64 -else EFI_LDS := elf_arm_efi.lds EFI_CRT0 := crt0_arm_efi.o EFI_RELOC := reloc_arm_efi.o -endif diff --git a/arch/arm/cpu/armv8/config.mk b/arch/arm/cpu/armv8/config.mk index 61ba3143969..0316af306c4 100644 --- a/arch/arm/cpu/armv8/config.mk +++ b/arch/arm/cpu/armv8/config.mk @@ -13,7 +13,7 @@ EFI_RELOC := reloc_aarch64_efi.o LDSCRIPT_EFI := $(srctree)/arch/arm/lib/elf_aarch64_efi.lds EFISTUB := crt0_aarch64_efi.o reloc_aarch64_efi.o -OBJCOPYFLAGS_EFI += -O binary --target=efi-app-aarch64 +OBJCOPYFLAGS_EFI += --target=pei-aarch64-little EFIPAYLOAD_BFDTARGET := pei-aarch64-little EFIPAYLOAD_BFDARCH := aarch64 LDFLAGS_EFI_PAYLOAD := -Bsymbolic -Bsymbolic-functions -shared --no-undefined \ @@ -21,3 +21,7 @@ LDFLAGS_EFI_PAYLOAD := -Bsymbolic -Bsymbolic-functions -shared --no-undefined \ CPPFLAGS_REMOVE_crt0-efi-aarch64.o += $(CFLAGS_NON_EFI) CPPFLAGS_crt0-efi-aarch64.o += $(CFLAGS_EFI) + +ifeq ($(CONFIG_EFI_APP),y) +PLATFORM_CPPFLAGS += $(CFLAGS_EFI) +endif diff --git a/arch/arm/lib/crt0_aarch64_efi.S b/arch/arm/lib/crt0_aarch64_efi.S index 88f8449bb03..c1030e5392a 100644 --- a/arch/arm/lib/crt0_aarch64_efi.S +++ b/arch/arm/lib/crt0_aarch64_efi.S @@ -10,11 +10,140 @@ #include <asm-generic/pe.h> - .text - .align 12 + .section .text.head + + /* + * Magic "MZ" signature for PE/COFF + */ + .globl image_base +image_base: + .short IMAGE_DOS_SIGNATURE /* 'MZ' */ + .skip 54 /* 'MZ' + pad + offset == 64 */ + .long LINUX_ARM64_MAGIC /* For GRUB's linux command */ + .long pe_header - image_base /* Offset to the PE header */ +pe_header: + .long IMAGE_NT_SIGNATURE /* 'PE' */ +coff_header: + .short IMAGE_FILE_MACHINE_ARM64 /* AArch64 */ + .short 3 /* nr_sections */ + .long 0 /* TimeDateStamp */ + .long 0 /* PointerToSymbolTable */ + .long 0 /* NumberOfSymbols */ + .short section_table - optional_header /* SizeOfOptionalHeader */ + /* Characteristics */ + .short (IMAGE_FILE_EXECUTABLE_IMAGE | \ + IMAGE_FILE_LINE_NUMS_STRIPPED | \ + IMAGE_FILE_LOCAL_SYMS_STRIPPED | \ + IMAGE_FILE_LARGE_ADDRESS_AWARE | \ + IMAGE_FILE_DEBUG_STRIPPED) +optional_header: + .short IMAGE_NT_OPTIONAL_HDR64_MAGIC /* PE32+ format */ + .byte 0x02 /* MajorLinkerVersion */ + .byte 0x14 /* MinorLinkerVersion */ + .long _etext - _start /* SizeOfCode */ + .long 0 /* SizeOfInitializedData */ + .long 0 /* SizeOfUninitializedData */ + .long _start - image_base /* AddressOfEntryPoint */ + .long _start - image_base /* BaseOfCode */ + +extra_header_fields: + .quad 0 /* image_base */ + .long 0x1000 /* SectionAlignment */ + .long 0x200 /* FileAlignment */ + .short 0 /* MajorOperatingSystemVersion */ + .short 0 /* MinorOperatingSystemVersion */ + .short 0 /* MajorImageVersion */ + .short 0 /* MinorImageVersion */ + .short 0 /* MajorSubsystemVersion */ + .short 0 /* MinorSubsystemVersion */ + .long 0 /* Win32VersionValue */ + + .long _edata - image_base /* SizeOfImage */ + + /* + * Everything before the kernel image is considered part of the header + */ + .long _start - image_base /* SizeOfHeaders */ + .long 0 /* CheckSum */ + .short IMAGE_SUBSYSTEM_EFI_APPLICATION /* Subsystem */ +#if CONFIG_ARCH_EFI + .short 0 /* DllCharacteristics */ +#else + .short IMAGE_DLLCHARACTERISTICS_NX_COMPAT +#endif + .quad 0 /* SizeOfStackReserve */ + .quad 0 /* SizeOfStackCommit */ + .quad 0 /* SizeOfHeapReserve */ + .quad 0 /* SizeOfHeapCommit */ + .long 0 /* LoaderFlags */ + .long 0x6 /* NumberOfRvaAndSizes */ + + .quad 0 /* ExportTable */ + .quad 0 /* ImportTable */ + .quad 0 /* ResourceTable */ + .quad 0 /* ExceptionTable */ + .quad 0 /* CertificationTable */ + .quad 0 /* BaseRelocationTable */ - .globl _start - .type _start,%function + /* Section table */ +section_table: + + /* + * The EFI application loader requires a relocation section + * because EFI applications must be relocatable. This is a + * dummy section as far as we are concerned. + */ + .ascii ".reloc" + .byte 0 + .byte 0 /* end of 0 padding of section name */ + .long 0 + .long 0 + .long 0 /* SizeOfRawData */ + .long 0 /* PointerToRawData */ + .long 0 /* PointerToRelocations */ + .long 0 /* PointerToLineNumbers */ + .short 0 /* NumberOfRelocations */ + .short 0 /* NumberOfLineNumbers */ + /* Characteristics (section flags) */ + .long (IMAGE_SCN_MEM_READ | \ + IMAGE_SCN_MEM_DISCARDABLE | \ + IMAGE_SCN_CNT_INITIALIZED_DATA) + + .ascii ".text" + .byte 0 + .byte 0 + .byte 0 /* end of 0 padding of section name */ + .long _etext - _start /* VirtualSize */ + .long _start - image_base /* VirtualAddress */ + .long _etext - _start /* SizeOfRawData */ + .long _start - image_base /* PointerToRawData */ + .long 0 /* PointerToRelocations */ + .long 0 /* PointerToLineNumbers */ + .short 0 /* NumberOfRelocations */ + .short 0 /* NumberOfLineNumbers */ + /* Characteristics (section flags) */ + .long (IMAGE_SCN_MEM_READ | \ + IMAGE_SCN_MEM_EXECUTE | \ + IMAGE_SCN_CNT_CODE) + + .ascii ".data" + .byte 0 + .byte 0 + .byte 0 /* end of 0 padding of section name */ + .long _data_size /* VirtualSize */ + .long _data - image_base /* VirtualAddress */ + .long _data_size /* SizeOfRawData */ + .long _data - image_base /* PointerToRawData */ + .long 0 /* PointerToRelocations */ + .long 0 /* PointerToLineNumbers */ + .short 0 /* NumberOfRelocations */ + .short 0 /* NumberOfLineNumbers */ + /* Characteristics (section flags) */ + .long (IMAGE_SCN_MEM_WRITE | \ + IMAGE_SCN_MEM_READ | \ + IMAGE_SCN_CNT_INITIALIZED_DATA) + + .align 12 _start: stp x29, x30, [sp, #-32]! mov x29, sp @@ -31,22 +160,3 @@ _start: 0: ldp x29, x30, [sp], #32 ret - - /* - * hand-craft a dummy .reloc section so EFI knows it's a relocatable - * executable: - */ - .data -dummy: .4byte 0 - -#define IMAGE_REL_ABSOLUTE 0 - .section .reloc, "a" -label1: - .4byte dummy-label1 // Page RVA - .4byte 12 // Block Size (2*4+2*2), must be aligned by 32 Bits - .2byte (IMAGE_REL_ABSOLUTE<<12) + 0 // reloc for dummy - .2byte (IMAGE_REL_ABSOLUTE<<12) + 0 // reloc for dummy - -#if defined(__ELF__) && defined(__linux__) - .section .note.GNU-stack,"",%progbits -#endif diff --git a/arch/arm/lib/elf_aarch64_efi.lds b/arch/arm/lib/elf_aarch64_efi.lds index cb7081abef5..f872624dd42 100644 --- a/arch/arm/lib/elf_aarch64_efi.lds +++ b/arch/arm/lib/elf_aarch64_efi.lds @@ -7,19 +7,18 @@ OUTPUT_FORMAT("elf64-littleaarch64", "elf64-littleaarch64", "elf64-littleaarch64") OUTPUT_ARCH(aarch64) + +PHDRS +{ + data PT_LOAD FLAGS(3); /* SHF_WRITE | SHF_ALLOC */ +} + ENTRY(_start) SECTIONS { - . = 0; - image_base = .; - /* .hash and/or .gnu.hash MUST come first! */ - .hash : { *(.hash) } - .gnu.hash : { *(.gnu.hash) } - . = ALIGN(4096); - .eh_frame : { *(.eh_frame) } - . = ALIGN(4096); - .text : { + .text 0x0 : { _text = .; + *(.text.head) *(.text) *(.text.*) *(.gnu.linkonce.t.*) @@ -29,12 +28,12 @@ SECTIONS *(.dynamic); . = ALIGN(512); } + .rela.dyn : { *(.rela.dyn) } + .rela.plt : { *(.rela.plt) } + .rela.got : { *(.rela.got) } + .rela.data : { *(.rela.data) *(.rela.data*) } _etext = .; _text_size = . - _text; - . = ALIGN(65536); - .reloc : { - KEEP (*(.reloc)) - } . = ALIGN(4096); .data : { _data = .; @@ -64,24 +63,9 @@ SECTIONS . = ALIGN(512); _bss_end = .; _edata = .; - _end = .; - } + } :data _data_size = _edata - _data; - . = ALIGN(4096); - .rela : { - *(.rela.text*) - *(.rela.data*) - *(.rela.got) - *(.rela.dyn) - *(.rela.stab) - *(.rela.init_array*) - *(.rela.fini_array*) - *(.rela.ctors*) - *(.rela.dtors*) - *(.rela__u_boot_list*) - } - . = ALIGN(4096); .dynsym : { *(.dynsym) } . = ALIGN(4096); @@ -93,8 +77,5 @@ SECTIONS *(.eh_frame) *(.note.GNU-stack) } - .embedded_dtb : { - *(.embedded_dtb) - } .comment 0 : { *(.comment) } } diff --git a/board/efi/efi-arm_app/config.mk b/board/efi/efi-arm_app/config.mk deleted file mode 100644 index a576a1da539..00000000000 --- a/board/efi/efi-arm_app/config.mk +++ /dev/null @@ -1,4 +0,0 @@ -# SPDX-License-Identifier: GPL-2.0+ - -BUILD_CFLAGS += -shared -PLATFORM_CPPFLAGS += $(CFLAGS_EFI) -- 2.43.0 base-commit: 8bae99245bf59fe2054f6217e90801a84cc993d1 branch: efiu

1 1

[PATCH 00/22] efi: Tidy up some commands and provide a keyboard driver
by Simon Glass 25 Sep '25

25 Sep '25

From: Simon Glass <sjg(a)chromium.org> This series collect various odds and ends to make the ARM EFI app show a menu that looks reasonable, including truetype fonts and a new keyboard driver, selectable by setting stdin to 'efi-kbd'. Some highlights: - fix some bugs in addr_find and part_find and adds docs / tests - reset pager when clearing the console - use at least 1G of memory with EFI since the app allocates 512M - allow NEON registers so floating point can work (truetype) - show the global_data flags with bdinfo Simon Glass (22): bdinfo: Show the flags script: Use a minimum of 1G of memory for EFI console: Reset the pager when clearing the console doc: Fix up the booti examples efi: Avoid a memory leak in efi_bind_block() on error path efi: Drop config.h test: Drop an unwanted blank line in dm_test_video_box() test: Allow creating disks with a different Ubuntu version cmd: part_find: Correct radix and calls cmd: sandbox: efi: Enable part_find for sandbox and EFI app cmd: Refactor part_find() into separate functions doc: test: Add docs and test for part_find cmd: Update addr_find to use a simple lmb allocation cmd: Update addr_find to ignore the devicetree doc: test: Add docs and test for addr_find boot: Tidy up positioning in bootflow_menu_new() expo: Allow manual positioning of menu items Revert "ARM: Prevent the compiler from using NEON registers" efi: Enable truetype in the ARM app efi: arm: Drop setting of fdt_addr efi: Move key decoding into a shared file efi: app: Provide a keyboard driver arch/arm/config.mk | 1 - arch/arm/dts/efi-arm_app.dts | 4 + board/efi/efi-arm_app/efi-arm_app.env | 1 - boot/bootflow_menu.c | 8 +- boot/scene.c | 14 +++ cmd/Kconfig | 5 +- cmd/addr_find.c | 23 ++-- cmd/bdinfo.c | 1 + cmd/part_find.c | 154 ++++++++++++++--------- common/console.c | 1 + configs/efi-arm_app64_defconfig | 1 + doc/usage/cmd/addr_find.rst | 63 ++++++++++ doc/usage/cmd/booti.rst | 4 +- doc/usage/cmd/part_find.rst | 119 ++++++++++++++++++ doc/usage/index.rst | 2 + drivers/input/Kconfig | 14 +++ drivers/input/Makefile | 2 + drivers/input/efi_keyb.c | 173 ++++++++++++++++++++++++++ drivers/net/efi_net.c | 1 - drivers/serial/serial_efi.c | 15 +-- drivers/tpm/tpm2_efi.c | 1 - include/efi.h | 32 +++++ include/expo.h | 27 +++- lib/efi/Makefile | 1 + lib/efi/input.c | 40 ++++++ lib/efi_client/efi_app_init.c | 4 +- scripts/build-efi | 2 +- test/cmd/Makefile | 2 + test/cmd/addr_find.c | 27 ++++ test/cmd/bdinfo.c | 1 + test/cmd/part_find.c | 42 +++++++ test/dm/video.c | 1 - test/py/img/ubuntu.py | 8 +- 33 files changed, 683 insertions(+), 111 deletions(-) create mode 100644 doc/usage/cmd/addr_find.rst create mode 100644 doc/usage/cmd/part_find.rst create mode 100644 drivers/input/efi_keyb.c create mode 100644 lib/efi/input.c create mode 100644 test/cmd/addr_find.c create mode 100644 test/cmd/part_find.c -- 2.43.0 base-commit: 271239736374b0ed1e90311ac07f2346ff858414 branch: api

1 22

[PATCH 1/4] colibri-imx8x: Increase the boot delay
by Simon Glass 23 Sep '25

23 Sep '25

From: Simon Glass <sjg(a)chromium.org> A delay of one second makes it hard for the lab to interrupt boot, thus causing test failures. Increase it to 5s Signed-off-by: Simon Glass <sjg(a)chromium.org> --- configs/colibri-imx8x_defconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configs/colibri-imx8x_defconfig b/configs/colibri-imx8x_defconfig index 3d576da05b3..210bb83d5f8 100644 --- a/configs/colibri-imx8x_defconfig +++ b/configs/colibri-imx8x_defconfig @@ -24,7 +24,7 @@ CONFIG_REMAKE_ELF=y CONFIG_FIT=y CONFIG_FIT_VERBOSE=y CONFIG_DISTRO_DEFAULTS=y -CONFIG_BOOTDELAY=1 +CONFIG_BOOTDELAY=5 CONFIG_OF_SYSTEM_SETUP=y CONFIG_USE_PREBOOT=y CONFIG_PREBOOT="test -n ${fdtfile} || setenv fdtfile ${soc}-colibri-${fdt_board}.dtb" -- 2.43.0 base-commit: 136c959dd2d089710fb131956081d09b9da3bea4 branch: labh

1 3

[PATCH 00/24] boot: efi: Various improvements to booting with the EFI app
by Simon Glass 22 Sep '25

22 Sep '25

From: Simon Glass <sjg(a)chromium.org> This series provides a number of tweaks which make it easier to test booting with the EFI app, particularly on ARM: - 'fake go' in bootstd - more debugging in various places - tidy up the generally incorrect ATAGs error - drop a panic() in the bowels of boot - ensure booti does not overwrite memory when relocating It also includes a way to quit or disable the pager, which is very helpful when running the app on a machine with no serial console. Finally, a bug in the release script is causing CI to fail, so a few patches deal with that. Simon Glass (24): doc: Resolve a formatting error in upcoming releases scripts: Fix RST formatting in release_version.py console: Restore old pager test-bypass value console: Correct condition for setting enabled pager bypass console: Add bypass keypress to disable paging console: Add a way to quit pager with no further output boot: Provide a Kconfig to enable faking the boot boot: Select the fake-go state explicitly boot: Tidy up the bootflow flags boot: Provide a bootflow option to fake a boot boot: Drop hex prefix from the booti image-moving message boot: Show the ramdisk node when debugging boot: Support a fake go with pxe boot: Add more debugging around devicetree selection boot: Fix code formatting in fit_image_load() boot: Add more debugging in FIT-handling arm: Update boot_prep_linux() to return an error arm: bootm: Fix a confusing message about ATAGs lmb: Provide a static inline for lmb_alloc() arm: Use lmb when relocating the image efi: Include efi.h in the efi_variable.h header efi: app: Call efi_exit() if board_init_r() returns efi: app: Move efi_main() to the bottom of the file efi: app: arm: Add various options to the build arch/arm/lib/bootm.c | 22 +++- arch/arm/lib/image.c | 15 ++- boot/Kconfig | 8 ++ boot/bootm.c | 16 ++- boot/ext_pxe_common.c | 4 + boot/image-board.c | 7 +- boot/image-fdt.c | 3 + boot/image-fit.c | 12 +- boot/pxe_utils.c | 21 +++- cmd/bootflow.c | 16 ++- cmd/booti.c | 2 +- common/cli_readline.c | 1 + common/console.c | 2 +- common/pager.c | 56 +++++++-- configs/efi-arm_app64_defconfig | 5 + doc/develop/concept_releases.rst | 3 +- doc/usage/cmd/bootflow.rst | 3 + doc/usage/cmd/bootm.rst | 3 +- doc/usage/console.rst | 6 +- include/bootflow.h | 14 ++- include/efi_variable.h | 1 + include/image.h | 3 +- include/lmb.h | 9 ++ include/pager.h | 22 ++++ include/pxe_utils.h | 2 + lib/efi_client/efi_app.c | 113 +++++++++--------- scripts/release_version.py | 3 +- test/common/pager.c | 93 ++++++++++++++- test/scripts/test_release_version.py | 164 +++++++++++++++++++++++++++ test/test-main.c | 6 +- 30 files changed, 519 insertions(+), 116 deletions(-) -- 2.43.0 base-commit: 05cebf0bdd955fa267b1533bbf145499f0af63b9 branch: loadw

1 24

[PATCH 00/42] video: Support a cursor more generally
by Simon Glass 19 Sep '25

19 Sep '25

From: Simon Glass <sjg(a)chromium.org> At present U-Boot does not support displaying a cursor on the console, except with expo. It is quite useful to have a cursor, particularly when using the command-line editor. This series adds tracking of the cursor position for both truetype and normal consoles. The cursor is shown when U-Boot is idle and is removed before writing characters to the display. A save-buffer ensures that the old display contents are provided. Some related improvements in this series include: - expo lineedit support with normal console - arrow keys now render correctly when editing commands with truetype - truetype console now also supports bitmap fonts The cursor is not currently supported for rotated consoles. A README for claude is provided, although so far I have not had too much success with it. A fix for an intermittent build is added as well, along with silencing some build messages noticed from Rust ulib. Simon Glass (42): Provide a README for Claude cyclic: Avoid showing messages when debugging video: Correct ops check for nominal, entry save/restore Makefile: Silence the example Rust build scripts: Fix conditional syntax for capsule ESL generation test: video: Split long strings video: Provide an option to enable the cursor snow: link: Disable the cursor video: console: Put fixed-font rendering into a shared file video: truetype: Rename the FONT_ENTRY() macro video: Move setting of the bitmap font into uclass video: Allow selection of bitmap fonts in truetype video: truetype: Handle rendering of bitmap fonts video: truetype: Use common cursor-drawing code video: Correct cursor handling when the the left side video: Convert to use uclass_id_foreach_dev() video: Provide an idle function for the console video: Provide state for the cursor video: Rename draw_cursor_vertically() to cursor_show() video: Update cursor_show() to take a simple pointer video: Move cursor drawing into the uclass video: Track the current CLI index video: Provide a way to indicate the start of CLI entry console: Add debugging for kerning video: truetype: Record the position where CLI entry starts video: truetype: Track characters and their widths expo: Enable the cursor when editing a textline video: Track whether the cursor is enabled video: Drop vidconsole_set_cursor_visible() video: Update normal console to use tracking information video: Add save and restore drivers for normal console video: Drop extra parameters from get_cursor_info() video: Support a cursor in multi-line text video: Drop extra parameters from vidconsole_show_cursor() video: Provide a buffer to hold pixels behind the cursor video: Support reading the framebuffer when writing video: Save overwritten pixels when drawing the cursor video: Show the cursor when idle video: truetype: Clear after the current char on insert video: truetype: Avoid clearing on backspace video: Add some notes about the cursor console: Add some cursor tests .claude/.gitignore | 1 + .claude/README.md | 69 ++++++ .gitignore | 1 + boot/scene_textline.c | 9 +- cmd/font.c | 3 - common/Kconfig | 9 + common/cli_readline.c | 5 + common/cyclic.c | 5 +- configs/chromebook_link_defconfig | 1 + configs/snow_defconfig | 1 + doc/usage/cmd/font.rst | 16 +- drivers/video/Kconfig | 8 + drivers/video/console_core.c | 195 ++++++++++++++--- drivers/video/console_normal.c | 140 ++++++++---- drivers/video/console_truetype.c | 326 ++++++++++++++++++---------- drivers/video/vidconsole-uclass.c | 176 ++++++++++++++- drivers/video/vidconsole_internal.h | 77 ++++++- drivers/video/video-uclass.c | 30 ++- examples/rust/Makefile | 3 - include/video_console.h | 164 ++++++++++++-- scripts/Makefile.lib | 5 +- test/cmd/font.c | 15 +- test/dm/video.c | 162 +++++++++++++- 23 files changed, 1173 insertions(+), 248 deletions(-) create mode 100644 .claude/.gitignore create mode 100644 .claude/README.md -- 2.43.0 base-commit: 0c2531998ee8fc2da099cb3e20cbabc5221c0b3d branch: curs

1 42

[PATCH 1/3] virtio: Add input-device driver
by Simon Glass 15 Sep '25

15 Sep '25

From: Simon Glass <sjg(a)chromium.org> Add a virtio input device driver that supports tablets and mice using the existing mouse uclass. The driver handles absolute and relative coordinates. Mouse buttons are supported (left, right, middle). EV_SYN events are skipped. Co-developed-by: Claude <noreply(a)anthropic.com> Signed-off-by: Simon Glass <sjg(a)chromium.org> --- drivers/virtio/Kconfig | 13 ++ drivers/virtio/Makefile | 1 + drivers/virtio/virtio_input.c | 369 ++++++++++++++++++++++++++++++++++ 3 files changed, 383 insertions(+) create mode 100644 drivers/virtio/virtio_input.c diff --git a/drivers/virtio/Kconfig b/drivers/virtio/Kconfig index eb4ff368f12..471f40b85aa 100644 --- a/drivers/virtio/Kconfig +++ b/drivers/virtio/Kconfig @@ -110,4 +110,17 @@ config VIRTIO_SCSI A specification for the protocol is available at https://docs.oasis-open.org/virtio/virtio/v1.3/virtio-v1.3.html +config VIRTIO_INPUT + bool "Input device driver for virtio devices" + depends on VIRTIO + select MOUSE + default y + help + This driver provides support for virtio-based input devices such as + tablets, mice and keyboards. It implements the mouse uclass for + tablet and pointer devices. + + A specification for the protocol is available at + https://docs.oasis-open.org/virtio/virtio/v1.3/virtio-v1.3.html + endmenu diff --git a/drivers/virtio/Makefile b/drivers/virtio/Makefile index f101536a863..9183dd56a79 100644 --- a/drivers/virtio/Makefile +++ b/drivers/virtio/Makefile @@ -14,3 +14,4 @@ obj-$(CONFIG_VIRTIO_BLK) += virtio_blk.o obj-$(CONFIG_VIRTIO_RNG) += virtio_rng.o obj-$(CONFIG_VIRTIO_FS) += fs.o fs_dir.o fs_file.o fs_compat.o obj-$(CONFIG_VIRTIO_SCSI) += virtio_scsi.o +obj-$(CONFIG_VIRTIO_INPUT) += virtio_input.o diff --git a/drivers/virtio/virtio_input.c b/drivers/virtio/virtio_input.c new file mode 100644 index 00000000000..5edc59b6037 --- /dev/null +++ b/drivers/virtio/virtio_input.c @@ -0,0 +1,369 @@ +// SPDX-License-Identifier: GPL-2.0+ +/* + * VirtIO Input (tablet) driver for U-Boot + * + * Copyright 2025 Simon Glass <sjg(a)chromium.org> + */ + +#define LOG_CATEGORY UCLASS_VIRTIO + +#include <dm.h> +#include <errno.h> +#include <mouse.h> +#include <virtio_types.h> +#include <virtio.h> +#include <virtio_ring.h> +#include <linux/byteorder/little_endian.h> +#include <dt-bindings/input/linux-event-codes.h> + +#define VIRTIO_INPUT_CFG_UNSET 0x00 +#define VIRTIO_INPUT_CFG_ID_NAME 0x01 +#define VIRTIO_INPUT_CFG_ID_SERIAL 0x02 +#define VIRTIO_INPUT_CFG_ID_DEVIDS 0x03 +#define VIRTIO_INPUT_CFG_PROP_BITS 0x10 +#define VIRTIO_INPUT_CFG_EV_BITS 0x11 +#define VIRTIO_INPUT_CFG_ABS_INFO 0x12 + +/* absolute axis information for input devices */ +struct virtio_input_absinfo { + __le32 min; /* minimum value */ + __le32 max; /* maximum value */ + __le32 fuzz; /* fuzz value for noise filtering */ + __le32 flat; /* flat area around center */ + __le32 res; /* resolution in units per mm */ +}; + +/* device identification information */ +struct virtio_input_devids { + __le16 bustype; /* bus type identifier */ + __le16 vendor; /* vendor identifier */ + __le16 product; /* product identifier */ + __le16 version; /* version number */ +}; + +/* configuration space for querying device capabilities */ +struct vinp_config { + __u8 select; /* configuration item to select */ + __u8 subsel; /* sub-selection within item */ + __u8 size; /* size of returned data */ + __u8 reserved[5]; /* padding */ + union { + char string[128]; /* for name/serial strings */ + __u8 bitmap[128]; /* for capability bitmaps */ + struct virtio_input_absinfo abs; /* for absolute axis info */ + struct virtio_input_devids ids; /* for device IDs */ + } u; +}; + +/* input event structure (follows Linux input_event format) */ +struct virtio_input_event { + __le16 type; /* event type (EV_KEY, EV_ABS, EV_SYN, etc.) */ + __le16 code; /* event code (button/axis identifier) */ + __le32 value; /* event value */ +}; + +#define BUF_COUNT 8 +#define BUF_SIZE (16 * sizeof(struct virtio_input_event)) +#define COORD_MAX 0xffff + +/* private data for virtio input device */ +struct virtio_input_priv { + struct virtqueue *event_vq; /* event virtqueue */ + struct virtqueue *status_vq; /* status virtqueue */ + char event_bufs[BUF_COUNT][BUF_SIZE]; /* event buffers */ + bool rx_running; /* true if buffers are set up */ + int abs_x_max; /* maximum X coordinate */ + int abs_y_max; /* maximum Y coordinate */ + int button_state; /* current button state */ + int last_x; /* last X coordinate */ + int last_y; /* last Y coordinate */ +}; + +static int virtio_input_free_buffer(struct udevice *dev, void *buf) +{ + struct virtio_input_priv *priv = dev_get_priv(dev); + struct virtio_sg sg = { buf, BUF_SIZE }; + struct virtio_sg *sgs[] = { &sg }; + + /* put the buffer back to the event ring */ + virtqueue_add(priv->event_vq, sgs, 0, 1); + + return 0; +} + +static int process_event(struct virtio_input_priv *priv, + struct virtio_input_event *vio_event, + struct mouse_event *evt) +{ + u16 type = le16_to_cpu(vio_event->type); + u16 code = le16_to_cpu(vio_event->code); + u32 value = le32_to_cpu(vio_event->value); + + /* skip EV_SYN events immediately */ + if (type == EV_SYN) + return -EAGAIN; + + log_debug("processing event: type=%d code=%d value=%d\n", type, code, + value); + + switch (type) { + case EV_ABS: + if (code == ABS_X) + priv->last_x = value; + else if (code == ABS_Y) + priv->last_y = value; + + /* report motion event */ + evt->type = MOUSE_EV_MOTION; + evt->motion.state = priv->button_state; + evt->motion.x = (priv->last_x * COORD_MAX) / priv->abs_x_max; + evt->motion.y = (priv->last_y * COORD_MAX) / priv->abs_y_max; + evt->motion.xrel = 0; /* Absolute mode */ + evt->motion.yrel = 0; + return 0; + + case EV_KEY: + switch (code) { + case BTN_LEFT: + evt->type = MOUSE_EV_BUTTON; + evt->button.button = BUTTON_LEFT; + evt->button.press_state = value ? BUTTON_PRESSED : + BUTTON_RELEASED; + evt->button.clicks = 1; + evt->button.x = (priv->last_x * COORD_MAX) / + priv->abs_x_max; + evt->button.y = (priv->last_y * COORD_MAX) / + priv->abs_y_max; + + if (evt->button.press_state == BUTTON_PRESSED) + priv->button_state |= BUTTON_LEFT; + else + priv->button_state &= ~BUTTON_LEFT; + return 0; + case BTN_RIGHT: + evt->type = MOUSE_EV_BUTTON; + evt->button.button = BUTTON_RIGHT; + evt->button.press_state = value ? BUTTON_PRESSED : + BUTTON_RELEASED; + evt->button.clicks = 1; + evt->button.x = (priv->last_x * COORD_MAX) / + priv->abs_x_max; + evt->button.y = (priv->last_y * COORD_MAX) / + priv->abs_y_max; + + if (evt->button.press_state == BUTTON_PRESSED) + priv->button_state |= BUTTON_RIGHT; + else + priv->button_state &= ~BUTTON_RIGHT; + return 0; + case BTN_MIDDLE: + evt->type = MOUSE_EV_BUTTON; + evt->button.button = BUTTON_MIDDLE; + evt->button.press_state = value ? BUTTON_PRESSED : + BUTTON_RELEASED; + evt->button.clicks = 1; + evt->button.x = (priv->last_x * COORD_MAX) / + priv->abs_x_max; + evt->button.y = (priv->last_y * COORD_MAX) / + priv->abs_y_max; + + if (evt->button.press_state == BUTTON_PRESSED) + priv->button_state |= BUTTON_MIDDLE; + else + priv->button_state &= ~BUTTON_MIDDLE; + return 0; + } + break; + } + + return -EAGAIN; +} + +static int virtio_input_start(struct udevice *dev) +{ + struct virtio_input_priv *priv = dev_get_priv(dev); + + if (!priv->rx_running) { + struct virtio_sg sg; + struct virtio_sg *sgs[] = { &sg }; + int i; + + /* setup event buffers */ + sg.length = BUF_SIZE; + + /* add all buffers to the event queue */ + for (i = 0; i < BUF_COUNT; i++) { + sg.addr = priv->event_bufs[i]; + virtqueue_add(priv->event_vq, sgs, 0, 1); + } + + virtqueue_kick(priv->event_vq); + + /* setup the event queue only once */ + priv->rx_running = true; + log_debug("VirtIO input buffers initialized\n"); + } + + return 0; +} + +static int virtio_input_get_event(struct udevice *dev, struct mouse_event *evt) +{ + struct virtio_input_priv *priv = dev_get_priv(dev); + struct virtio_input_event *vio_event; + int i, num_events, ret; + void *buf; + uint len; + + /* ensure buffers are setup */ + ret = virtio_input_start(dev); + if (ret) + return ret; + + log_debug("starting\n"); + + /* check for ready buffer */ + buf = virtqueue_get_buf(priv->event_vq, &len); + if (!buf) { + log_debug("No events available\n"); + return -EAGAIN; + } + + log_debug("got buffer back: addr=%p len=%d\n", buf, len); + if (len < sizeof(struct virtio_input_event)) { + log_debug("Invalid event length: %d\n", len); + /* put buffer back */ + virtio_input_free_buffer(dev, buf); + return -EAGAIN; + } + + /* process all events in the buffer */ + num_events = len / sizeof(struct virtio_input_event); + vio_event = (struct virtio_input_event *)buf; + + for (i = 0; i < num_events; i++) { + ret = process_event(priv, &vio_event[i], evt); + if (!ret) { + /* + * found a valid event to return, put buffer back + * Note: this loses any remaining events in the buffer, + * but for input devices this is acceptable + */ + virtio_input_free_buffer(dev, buf); + return 0; + } + /* -EAGAIN means continue to next event */ + } + + /* no useful events found, put buffer back */ + virtio_input_free_buffer(dev, buf); + evt->type = MOUSE_EV_NULL; + + return -EAGAIN; +} + +static int virtio_input_probe(struct udevice *dev) +{ + struct virtio_input_priv *priv = dev_get_priv(dev); + struct virtqueue *vqs[2]; + struct vinp_config cfg; + int ret; + + ret = virtio_find_vqs(dev, 2, vqs); + if (ret) { + log_debug("Failed to find virtqueues: %d\n", ret); + return ret; + } + + priv->event_vq = vqs[0]; + priv->status_vq = vqs[1]; + + /* check what event types this device supports */ + cfg.select = VIRTIO_INPUT_CFG_EV_BITS; + cfg.subsel = EV_KEY; + virtio_cwrite8(dev, offsetof(struct vinp_config, select), cfg.select); + virtio_cwrite8(dev, offsetof(struct vinp_config, subsel), cfg.subsel); + virtio_cread_bytes(dev, offsetof(struct vinp_config, u.bitmap), + cfg.u.bitmap, 16); + log_debug("EV_KEY bitmap: %02x %02x %02x %02x\n", cfg.u.bitmap[0], + cfg.u.bitmap[1], cfg.u.bitmap[2], cfg.u.bitmap[3]); + + cfg.select = VIRTIO_INPUT_CFG_EV_BITS; + cfg.subsel = EV_REL; + virtio_cwrite8(dev, offsetof(struct vinp_config, select), cfg.select); + virtio_cwrite8(dev, offsetof(struct vinp_config, subsel), cfg.subsel); + virtio_cread_bytes(dev, offsetof(struct vinp_config, u.bitmap), + cfg.u.bitmap, 16); + log_debug("EV_REL bitmap: %02x %02x %02x %02x\n", cfg.u.bitmap[0], + cfg.u.bitmap[1], cfg.u.bitmap[2], cfg.u.bitmap[3]); + + /* check if this device supports absolute coordinates (tablet) */ + cfg.select = VIRTIO_INPUT_CFG_EV_BITS; + cfg.subsel = EV_ABS; + virtio_cwrite8(dev, offsetof(struct vinp_config, select), cfg.select); + virtio_cwrite8(dev, offsetof(struct vinp_config, subsel), cfg.subsel); + + /* read the bitmap to see if ABS_X and ABS_Y are supported */ + virtio_cread_bytes(dev, offsetof(struct vinp_config, u.bitmap), + cfg.u.bitmap, 16); + + log_debug("EV_ABS bitmap: %02x %02x %02x %02x\n", cfg.u.bitmap[0], + cfg.u.bitmap[1], cfg.u.bitmap[2], cfg.u.bitmap[3]); + + /* check if ABS_X (0) and ABS_Y (1) bits are set */ + if ((cfg.u.bitmap[0] & (1 << 0)) && + (cfg.u.bitmap[0] & (1 << 1))) { + /* get ABS_X range */ + cfg.select = VIRTIO_INPUT_CFG_ABS_INFO; + cfg.subsel = ABS_X; + virtio_cwrite8(dev, offsetof(struct vinp_config, select), + cfg.select); + virtio_cwrite8(dev, offsetof(struct vinp_config, subsel), + cfg.subsel); + virtio_cread_bytes(dev, offsetof(struct vinp_config, u.abs), + &cfg.u.abs, sizeof(cfg.u.abs)); + priv->abs_x_max = le32_to_cpu(cfg.u.abs.max); + + /* get ABS_Y range */ + cfg.subsel = ABS_Y; + virtio_cwrite8(dev, offsetof(struct vinp_config, subsel), + cfg.subsel); + virtio_cread_bytes(dev, offsetof(struct vinp_config, u.abs), + &cfg.u.abs, sizeof(cfg.u.abs)); + priv->abs_y_max = le32_to_cpu(cfg.u.abs.max); + + log_debug("tablet: X range 0-%d, Y range 0-%d\n", + priv->abs_x_max, priv->abs_y_max); + } else { + /* no absolute coordinates, use default range */ + priv->abs_x_max = 32767; + priv->abs_y_max = 32767; + log_debug("table: No absolute coords, using relative mode\n"); + } + + return 0; +} + +static int virtio_input_bind(struct udevice *dev) +{ + struct virtio_dev_priv *uc_priv = dev_get_uclass_priv(dev->parent); + + /* indicate what driver features we support */ + virtio_driver_features_init(uc_priv, NULL, 0, NULL, 0); + + return 0; +} + +static const struct mouse_ops virtio_input_ops = { + .get_event = virtio_input_get_event, +}; + +U_BOOT_DRIVER(virtio_input) = { + .name = VIRTIO_INPUT_DRV_NAME, + .id = UCLASS_MOUSE, + .bind = virtio_input_bind, + .probe = virtio_input_probe, + .ops = &virtio_input_ops, + .priv_auto = sizeof(struct virtio_input_priv), + .flags = DM_FLAG_ACTIVE_DMA, +}; -- 2.43.0 base-commit: 203f498cf558fb86f9e66cafdb7cfee8f0724139 branch: tab

2 3