Re: [PATCH 1/4] virtio: Add a little more debugging

2025-07-24 Thread Simon Glass
Hi, On Thu, 24 Jul 2025 at 04:52, Simon Glass wrote: > > From: Simon Glass > > Add some debugging to virtiofs in the directory-handling area. > > Drop a stray, blank line while here. > > Signed-off-by: Simon Glass > --- > > drivers/virtio/fs.c | 1

Away for a while

2025-06-07 Thread Simon Glass
Hi, Just a note to say that I'll be disengaging from the mailing list and irc for a while I work on some specific areas. Please send me a direct email / irc message if needed, but no promises! Regards, Simon

Re: [PATCH v3 0/2] bootstd: New bootmeth for RAUC A/B systems

2025-06-05 Thread Simon Glass
Add implementation for bootmeth rauc > doc: Add description for bootmeth rauc > > boot/Kconfig | 51 + > boot/Makefile| 1 + > boot/bootmeth_rauc.c | 432 > +++ > doc/develop/bootstd/index.rst| 1 + > doc/develop/bootstd/overview.rst | 1 + > doc/develop/bootstd/rauc.rst | 56 + > 6 files changed, 542 insertions(+) > --- > base-commit: b22a276f039f818d5564bec6637071cfc8a7e432 > change-id: 20250129-wip-bootmeth-rauc-03b5e2dd88f4 > > Best regards, > -- > Martin Schwan > Great to see this. Could you please add a test, e.g. to test/boot/bootflow.c ? Let me know if you need help. Regards, Simon

Re: [PATCH] tpm: Export the function to set up the TPM log

2025-06-05 Thread Simon Glass
Hi Ilias, On Sat, 24 May 2025 at 12:24, Ilias Apalodimas wrote: > > On Sat, 24 May 2025 at 20:59, Ilias Apalodimas > wrote: > > > > Thanks Tom > > > > On Sat, 24 May 2025 at 17:26, Tom Rini wrote: > > > > > > On Sat, May 24, 2025 at 08:13:4

Re: [PATCH] x86: Correct condition for init_cache_f_r()

2025-06-04 Thread Simon Glass
Hi Tom, On Wed, 4 Jun 2025 at 11:23, Tom Rini wrote: > > On Wed, Jun 04, 2025 at 10:17:32AM -0600, Simon Glass wrote: > > Hi Jerome, > > > > On Wed, 4 Jun 2025 at 07:15, Jerome Forissier > > wrote: > > > > > > On 6/4/25 15:09, Simon Glass wrot

Re: [PATCH] x86: Correct condition for init_cache_f_r()

2025-06-04 Thread Simon Glass
Hi Jerome, On Wed, 4 Jun 2025 at 07:15, Jerome Forissier wrote: > > On 6/4/25 15:09, Simon Glass wrote: > > The condition here is reversed, which makes link and coral very slow, > > leading to lab failures. > > > > Signed-off-by: Simon Glass > > Fixes 6c171f7

Re: [PATCH v6 2/3] common: board: make initcalls static

2025-06-04 Thread Simon Glass
Hi Tom, On Tue, 3 Jun 2025 at 10:10, Tom Rini wrote: > > On Tue, Jun 03, 2025 at 09:22:01AM -0600, Simon Glass wrote: > > Hi Jerome, > > > > On Fri, 4 Apr 2025 at 07:50, Jerome Forissier > > wrote: > > > > > > Change board_init_f(), board_init_f_r

[PATCH] x86: Correct condition for init_cache_f_r()

2025-06-04 Thread Simon Glass
The condition here is reversed, which makes link and coral very slow, leading to lab failures. Signed-off-by: Simon Glass Fixes 6c171f7a184 ("common: board: make initcalls static") --- common/board_f.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/common/b

Re: [PATCH v2] bootm: Fix bmi->images pointer not initialized in some cases

2025-06-03 Thread Simon Glass
Hi Tom, On Fri, 23 May 2025 at 08:08, Tom Rini wrote: > > On Fri, May 23, 2025 at 02:48:15PM +0100, Simon Glass wrote: > > Hi Benjamin, > > > > On Tue, 20 May 2025 at 22:35, Benjamin ROBIN wrote: > > > > > > When building with only bootz command,

Re: [PATCH v6 2/3] common: board: make initcalls static

2025-06-03 Thread Simon Glass
y x86 uses this route, but it is intended that > * all archs will move to this when generic relocation is implemented. > */ > -static const init_fnc_t init_sequence_f_r[] = { > -#if !CONFIG_IS_ENABLED(X86_64) > - init_cache_f_r, > +static void initcall_run_f_r(void) > +{ > +#if CONFIG_IS_ENABLED(X86_64) This is reversing the condition, which breaks link and coral in my lab. > + INITCALL(init_cache_f_r); > #endif > - > - NULL, > -}; > +} Regards, Simon

TPM result

2025-06-03 Thread Simon Glass
Hi Ilias, I finally picked up the TPM you recommended (LetsTrust [1]) and attached it to a rpi4 and used the instructions at [2]. It seems to work fine in my CI - see [3]. Regards, Simon [1] https://buyzero.de/products/letstrust-hardware-tpm-trusted-platform-module [2] https://github.com/joholl

[PATCH 1/2] doc: efi_loader: Tidy up the bootefi-command docs

2025-06-01 Thread Simon Glass
There are backslashes in some of the tags which seems to be unnecessary. Remove then. Change the word 'either' to 'any' since there are three options. Signed-off-by: Simon Glass --- doc/usage/cmd/bootefi.rst | 20 ++-- 1 file changed, 10 insertions(+), 1

Re: [PATCH 1/2] doc: efi_loader: Tidy up the bootefi-command docs

2025-06-01 Thread Simon Glass
Hi, On Sun, 1 Jun 2025 at 13:04, Simon Glass wrote: > > There are backslashes in some of the tags which seems to be unnecessary. > Remove then. > > Change the word 'either' to 'any' since there are three options. > > Signed-off-by: Simon Glass

Re: Azure runners

2025-05-30 Thread Simon Glass
Hi Tom, On Fri, 30 May 2025 at 17:59, Tom Rini wrote: > > On Fri, May 30, 2025 at 04:42:18PM +0100, Simon Glass wrote: > > Hi Tom, > > > > On Fri, 30 May 2025 at 15:06, Tom Rini wrote: > > > > > > On Fri, May 30, 2025 at 02:31:13PM +010

Re: [PATCH 00/24] Update kbuild

2025-05-30 Thread Simon Glass
> [21/24] kbuild: strip whitespace in cmd_record_mcount findstring > > commit: 25ab30f1c46efc8d2c11b01013449c09815a3f65 > > [22/24] kbuild: remove unused archmrproper > > commit: 0719c79e09de41aaba0ee5d86dcd578828d4aad3 > > [23/24] kbuild: make -r/-R effective in top Makefile for old Make versions > > commit: af1a9935708a30614e3dbf4a33e5fe90e846f598 > > [24/24] kbuild: add -Werror=strict-prototypes flag unconditionally > > commit: 0128b2cf4da8bd69c695d56743a666faf159d2d5 > > -- > > Tom This is fantastic, thank you Ilias! - Simon

Re: Azure runners

2025-05-30 Thread Simon Glass
Hi Tom, On Fri, 30 May 2025 at 15:06, Tom Rini wrote: > > On Fri, May 30, 2025 at 02:31:13PM +0100, Simon Glass wrote: > > > Hi Tom, > > > > I have always found the azure pipeline to be slow, but I found that it > > actually does support having your own runners

Azure runners

2025-05-30 Thread Simon Glass
5f-9710-46ad-88c1-5e99a7c3cde6_1.log:2025-05-30T13:24:34.1360950Z + ./test/py/test.py -ra -o cache_dir=/tmp/sandbox64/.pytest_cache --bd sandbox64 --build-dir /tmp/sandbox64 --report-dir /tmp/sandbox64 --junitxml=/home/azure/azure/agent3/_work/1/s/results.xml azure@ruru:~/azure$ It seems a bit strange. If you have any ideas, please let me know. Regards, Simon

Please pull u-boot-dm into -next

2025-05-30 Thread Simon Glass
f improvements. ---- Simon Glass (46): abuf: Add a helper for initing and allocating a buffer abuf: Add a function to copy a buffer abuf: Add a way to printf() into a buffer expo: Add CLI context to the expo

Re: [PATCH RFC v2 3/8] tools: binman: mark optional missing blobs as absent

2025-05-30 Thread Simon Glass
or me to comment on the series in this state. Also there's no need to put an 'RFC' tag on this series. We do want to resolve this, one way or another. > > Reported-by: Simon Glass > Signed-off-by: Yannic Moog > --- > tools/binman/binman.rst| 7 +++ > to

Re: [PATCH RFC v2 5/8] binman: rework dropping absent entries from packaged image

2025-05-30 Thread Simon Glass
git a/tools/binman/image.py b/tools/binman/image.py > index > 24ce0af7c72b5256a9a71963a6d94c080ed8bdd4..86543ad8db36b38afc1957fde7e20152067b79aa > 100644 > --- a/tools/binman/image.py > +++ b/tools/binman/image.py > @@ -15,7 +15,7 @@ import sys > from binman.entry import Entry > from binman.etype import fdtmap > from binman.etype import image_header > -from binman.etype import section > +from etype import section Why this change? I suspect it will break when installed with 'pip install binary-manager' > from dtoc import fdt > from dtoc import fdt_util > from u_boot_pylib import tools > @@ -183,6 +183,8 @@ class Image(section.Entry_section): > fname = tools.get_output_filename(self._filename) > tout.info("Writing image to '%s'" % fname) > with open(fname, 'wb') as fd: > +# For final image, don't write absent blobs to file > +self.drop_absent_optional() > data = self.GetPaddedData() > fd.write(data) > tout.info("Wrote %#x bytes" % len(data)) > > -- > 2.43.0 > Regards, Simon

Re: [PATCH RFC v2 0/8] Fix handling of optional blobs in binman

2025-05-30 Thread Simon Glass
--- > tools/binman/etype/ti_board_config.py | 8 ++--- > tools/binman/ftest.py | 43 +--- > tools/binman/image.py | 4 ++- > 17 files changed, 138 insertions(+), 82 deletions(-) > --- > base-commit: eeb5ff1a468b2303efa9deb2483b5edbebb568c6 > change-id: 20250122-binman_faked_optional-5025f7046e25 > > Best regards, > -- > Yannic Moog > Regards, Simon

Re: [PATCH RFC v2 7/8] tools: binman: ftest: fix tests that require non-faked ext blobs

2025-05-30 Thread Simon Glass
, But let's try it for now. Can you push these changes into the other commits so that the series is bisectable with 'binman test' ? > > Signed-off-by: Yannic Moog > --- > tools/binman/ftest.py | 18 +++--- > 1 file changed, 11 insertions(+), 7 deletions(-) > Regards, Simon

Re: [PATCH RFC v2 4/8] binman: replace _entries with GetEntries()

2025-05-30 Thread Simon Glass
tools/binman/etype/ti_board_config.py | 8 +++ > tools/binman/ftest.py | 2 +- > 11 files changed, 44 insertions(+), 44 deletions(-) Are you sure this is right? The classes which use _entries are subclasses, except for ftest.py which is test code where we should probably add: #pylint: disable=protected-access Regards, Simon

Re: [PATCH RFC v2 2/8] tools: binman: drop "faked" return value from check_fake_fname

2025-05-30 Thread Simon Glass
ned-off-by: Yannic Moog > --- > tools/binman/entry.py | 11 +-- > tools/binman/etype/blob.py | 7 +++ > tools/binman/etype/blob_ext_list.py | 4 ++-- > 3 files changed, 10 insertions(+), 12 deletions(-) Reviewed-by: Simon Glass For the commi

[PATCH 1/2] test/py: Use the correct fixture name in exception handler

2025-05-29 Thread Simon Glass
If a BootFail exception is thrown in a test, it is not handled correctly. Use the correct fixture variable 'ubman_fix' to resolve this. Signed-off-by: Simon Glass Fixes: d9ed4b75add ("test/py: Drop u_boot_ prefix on test files") --- test/py/conftest.py | 2 +- 1 file change

[PATCH 2/2] test/py: Correct handling of exceptions

2025-05-29 Thread Simon Glass
If an Unexpected exception is thrown in a test, an undefined variable error is reported. Fix this. Signed-off-by: Simon Glass Fixes: 85d7dae377a ("test: Detect dead connections") --- test/py/conftest.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/py/con

Re: [PATCH v5 00/25] passage: Define a standard for firmware data flow

2025-05-28 Thread Simon Glass
Hi Tom, On Wed, 28 May 2025 at 16:19, Tom Rini wrote: > > On Wed, May 28, 2025 at 03:32:12PM +0100, Simon Glass wrote: > > Hi Tom, > > > > On Wed, 28 May 2025 at 15:25, Tom Rini wrote: > > > > > > On Wed, May 28, 2025 at 06:32:02AM -0600, Simon Glass

Re: [PATCH v5 00/25] passage: Define a standard for firmware data flow

2025-05-28 Thread Simon Glass
Hi Tom, On Wed, 28 May 2025 at 15:25, Tom Rini wrote: > > On Wed, May 28, 2025 at 06:32:02AM -0600, Simon Glass wrote: > > > > This series adds a standard way of passing information between different > > firmware phases. This already exists in U-Boot at a very basic lev

Re: [PATCH 0/3] efi_loader: Move the public cerificate back to the devicetree

2025-05-28 Thread Simon Glass
Hi Tom, On Wed, 28 May 2025 at 16:05, Tom Rini wrote: > > On Wed, May 28, 2025 at 03:32:08PM +0100, Simon Glass wrote: > > Hi Tom, > > > > On Wed, 28 May 2025 at 15:12, Tom Rini wrote: > > > > > > On Wed, May 28, 2025 at 08:35:37AM +0100, Simon Glass w

Re: [PATCH 1/2] dm: core: use u32 instead of int for dev_read_u32_default

2025-05-28 Thread Simon Glass
Hi Quentin, On Wed, 28 May 2025 at 17:15, Quentin Schulz wrote: > > Hi Simon, > > On 5/28/25 4:32 PM, Simon Glass wrote: > > Hi Quentin, > > > > On Wed, 28 May 2025 at 14:06, Quentin Schulz wrote: > >> > >> From: Quentin Schulz > >> >

[PATCH 3/4] efi: Create a new CONFIG_EFI

2025-05-28 Thread Simon Glass
Create a Kconfig which indicates that EFI functionality is in use, either as a client (EFI app / stub) or provider (EFI loader). This will make it easier to share code between these two parts of U-Boot Signed-off-by: Simon Glass --- doc/develop/uefi/u-boot_on_efi.rst | 6 +++--- lib/efi

[PATCH 4/4] efi: Update maintainers for EFI_CLIENT

2025-05-28 Thread Simon Glass
The 'EFI PAYLOAD' entry really refers to the EFI loader and EFI APP refers to the app and the payload. Rename 'EFI PAYLOAD' to 'EFI LOADER' and 'EFI APP' to 'EFI CLIENT' Signed-off-by: Simon Glass --- MAINTAINERS | 4 ++-- 1 file chan

[PATCH 1/4] efi: Rename the lib/efi directory

2025-05-28 Thread Simon Glass
for U-Boot being a client of EFI, i.e. the EFI app and stub. Signed-off-by: Simon Glass --- MAINTAINERS| 2 +- Makefile | 2 +- doc/develop/uefi/u-boot_on_efi.rst | 8 lib/Makefile | 2 +- lib/{efi

[PATCH 2/4] efi: Rename CONFIG_EFI to CONFIG_EFI_CLIENT

2025-05-28 Thread Simon Glass
The generic name 'EFI' would be more useful for common EFI features. At present it just refers to the EFI app and stub, which is confusing. Rename it to EFI_CLIENT Signed-off-by: Simon Glass --- arch/x86/Kconfig| 6 +++--- arch/x86/cpu/Makefile |

Re: [PATCH 00/27] x86: efi: Various app improvements to support bootstd

2025-05-28 Thread Simon Glass
Hi Tom, On Wed, 28 May 2025 at 16:01, Tom Rini wrote: > > On Wed, May 28, 2025 at 03:25:32PM +0100, Simon Glass wrote: > > Hi Tom, > > > > On Wed, 28 May 2025 at 15:09, Tom Rini wrote: > > > > > > On Wed, May 28, 2025 at 02:24:26AM -0600, Simon Glas

Re: [PATCH 1/2] dm: core: use u32 instead of int for dev_read_u32_default

2025-05-28 Thread Simon Glass
avoid the compiler masking values before calling a function, which can increase code size. In this case I could make the argument either way. Reviewed-by: Simon Glass Regards, Simon

Re: [PATCH 1/2] bootm: Shorten BOOTM_STATE_

2025-05-28 Thread Simon Glass
Hi Tom, On Tue, 27 May 2025 at 14:32, Tom Rini wrote: > > On Tue, May 27, 2025 at 06:40:34AM -0600, Simon Glass wrote: > > > This is quite long at present. Rename it so that fixes better with the > > bootstd naming. > > > > Signed-off-by: Simon Glass > >

Re: [PATCH 2/2] dm: core: use s32 instead of int for dev_read_s32_default

2025-05-28 Thread Simon Glass
int") > Signed-off-by: Quentin Schulz > --- > drivers/core/read.c | 4 ++-- > include/dm/read.h | 8 > 2 files changed, 6 insertions(+), 6 deletions(-) Reviewed-by: Simon Glass

Re: [PATCH 0/3] efi_loader: Move the public cerificate back to the devicetree

2025-05-28 Thread Simon Glass
Hi Tom, On Wed, 28 May 2025 at 15:12, Tom Rini wrote: > > On Wed, May 28, 2025 at 08:35:37AM +0100, Simon Glass wrote: > > Hi Ilias, > > > > On Tue, 27 May 2025 at 10:20, Ilias Apalodimas > > wrote: > > > > > > Hi Simon, > > &g

Re: [PATCH 00/27] x86: efi: Various app improvements to support bootstd

2025-05-28 Thread Simon Glass
Hi Tom, On Wed, 28 May 2025 at 15:09, Tom Rini wrote: > > On Wed, May 28, 2025 at 02:24:26AM -0600, Simon Glass wrote: > > > This series makes a number of improvements to the EFI app. Mostly it > > applies to x86 but the ARM version will share some improvements. > &g

Re: [PATCH 00/10] efi: Move some efi-loader code into a new shared dir

2025-05-28 Thread Simon Glass
Hi Casey, On Wed, 28 May 2025 at 12:03, Casey Connolly wrote: > > > > On 5/23/25 16:06, Tom Rini wrote: > > On Fri, May 23, 2025 at 02:06:35PM +0100, Simon Glass wrote: > > > >> Some functions provided in lib/efi_loader are actually useful for the > >&

[PATCH v5 18/25] arm: qemu: Add a 64-bit SPL build

2025-05-28 Thread Simon Glass
, so we can obtain it from standard passage. For now this just boots and hangs in SPL as there is no bloblist. Signed-off-by: Simon Glass --- (no changes since v3) Changes in v3: - Add a build for aarch64 as well arch/arm/dts/qemu-arm64-u-boot.dtsi | 29 ++ arch/arm/dts/qemu-arm64.dts

[PATCH v5 00/25] passage: Define a standard for firmware data flow

2025-05-28 Thread Simon Glass
ches already applied) - Rework global_data for new stdpass convention - Split the jump_to_image_no_args() change into its own patch - Update the commit message to mention the long lines - Use three registers instead of two for the entry Simon Glass (25): RFC: scripts: Add scripts for running QEM

[PATCH v5 03/25] emulation: fdt: Allow using U-Boot's device tree with QEMU

2025-05-28 Thread Simon Glass
e set automatically by each platform. [1] https://docs.u-boot.org/en/latest/develop/devicetree/dt_qemu.html Link: https://patchwork.kernel.org/project/qemu-devel/patch/20210926183410.256484-1-...@chromium.org/#24481799 Signed-off-by: Simon Glass Suggested-by: Tom Rini --- (no changes since v3) Chang

[PATCH v5 25/25] CI: Add tests for gitlab and azure

2025-05-28 Thread Simon Glass
Add tests that check that SPL can generate a standard passage on ARM and that U-Boot proper can accept it successfully. - Finish the commit message Signed-off-by: Simon Glass --- (no changes since v3) Changes in v3: - Add test for aarch64 - Add tests for azure .azure-pipelines.yml | 6

[PATCH v5 24/25] passage: Add checks for pre-existing blobs

2025-05-28 Thread Simon Glass
Add checks / documentation for blobs which are already in the list. This brings U-Boot up to the standard required by the standard-passage documentation. Signed-off-by: Simon Glass --- (no changes since v1) board/sandbox/stdpass_check.c | 70 - include/stdpass/tpm2_eventlog.h | 42

[PATCH v5 23/25] passage: Add docs for spl_handoff

2025-05-28 Thread Simon Glass
This tag already exists in U-Boot. Add documentation, following the format set out. Signed-off-by: Simon Glass --- (no changes since v1) board/sandbox/stdpass_check.c | 15 --- include/handoff.h | 10 +++--- 2 files changed, 15 insertions(+), 10 deletions(-) diff

[PATCH v5 22/25] passage: Add documentation

2025-05-28 Thread Simon Glass
Add documentation about standard passage and update the maintainers. Signed-off-by: Simon Glass --- (no changes since v3) Changes in v3: - Fix 'that' typo - Update docs for the various code changes Changes in v2: - Add comments about how to pass standard passage to EFI - Add comm

[PATCH v5 20/25] passage: Add a qemu test for ARM

2025-05-28 Thread Simon Glass
Check that the standard passage works on ARM, by setting it up in SPL and making sure that it comes through correctly in U-Boot proper. Signed-off-by: Simon Glass --- Changes in v5: - Enable the test for any board which uses OF_PASSAGE Changes in v3: - Add conditions to avoid enaling the test

[PATCH v5 21/25] sandbox: Add a way of checking structs for standard passage

2025-05-28 Thread Simon Glass
Add a file which can be used to check that structs used in standard passage do compile. No other validation is done at present, but could be considered later. Signed-off-by: Simon Glass --- (no changes since v1) board/sandbox/Makefile| 3 ++- board/sandbox/stdpass_check.c | 34

[PATCH v5 19/25] xferlist: Drop old xferlist code

2025-05-28 Thread Simon Glass
This is not needed now, as the startup protocol is handled in arch-specific code early in boot. Drop BLOBLIST_PASSAGE_MANDATORY as well, as OF_BLOBLIST is enough to cover this. With standard passage the devicetree is accessed before the bloblist is inited. Signed-off-by: Simon Glass --- (no

[PATCH v5 17/25] arm: qemu: Add an SPL build

2025-05-28 Thread Simon Glass
can obtain it from standard passage. For now this just boots and hangs in SPL as there is no bloblist. Signed-off-by: Simon Glass --- (no changes since v3) Changes in v3: - Drop common.h - Refresh the U-Boot output in the documentation - Use bootph tags arch/arm/dts/qemu-arm-u-boot.dtsi

[PATCH v5 16/25] vexpress_fvp: Update to use the new Kconfig options

2025-05-28 Thread Simon Glass
We need to enable CONFIG_BLOBLIST_PASSAGE sand CONFIG_PASSAGE_IN so that U-Boot is built with support for accepting an incoming standard passage. Update the defconfig accordingly. Signed-off-by: Simon Glass --- (no changes since v4) Changes in v4: - Add new patch to update vexpress_fvp to use

[PATCH v5 15/25] passage: arm: Add the arch-specific standard passage impl

2025-05-28 Thread Simon Glass
Add implementations for 32- and 64-bit ARM. Signed-off-by: Simon Glass --- (no changes since v3) Changes in v3: - Update registers to match the Firmware Handoff protocol Changes in v2: - Add new patch with the arm-specific standard passage implementation arch/arm/cpu/armv7/cpu.c | 18

[PATCH v5 14/25] passage: spl: Support passing the passage to U-Boot

2025-05-28 Thread Simon Glass
Create a new arch-specific way of creating a standard passage for the next phase. Signed-off-by: Simon Glass --- (no changes since v4) Changes in v4: - Fix 'to' typo Changes in v2: - Make the stdpass calling standard arch-specific common/spl/spl.c | 26 +---

[PATCH v5 13/25] passage: spl: Support adding the dtb to the passage bloblist

2025-05-28 Thread Simon Glass
Add an option for SPL to add a devicetree to the passage bloblist, so SPL can provide the devicetree to U-Boot. Fix a few long lines while we are here. Signed-off-by: Simon Glass --- (no changes since v2) Changes in v2: - Rebase on -master - Update the commit message to mention the long lines

[PATCH v5 12/25] passage: arm: Accept a passage from the previous phase

2025-05-28 Thread Simon Glass
/firmware_handoff/main/index.html Signed-off-by: Simon Glass --- While some efforts were made to encourage Intel to adopt this specification, this has not yet been successful. However I do intend to use my proposed x86 handoff in U-Boot, once this series is in. (no changes since v4) Changes in v4: - Update

[PATCH v5 09/25] fdt: Support reading FDT from standard passage

2025-05-28 Thread Simon Glass
Add an option to receive the FDT using standard passage from a previous phase. Rename the FDT source to 'passage' to match the old terminology, which was only partially added. Signed-off-by: Simon Glass --- (no changes since v1) env/common.c | 2 +- include/fdtdec.h | 4

[PATCH v5 11/25] bloblist: Drop bloblist_maybe_init()

2025-05-28 Thread Simon Glass
The bloblist is not inited any earlier in the board_f sequence now, so drop this function. Just use bloblist_init() instead. Signed-off-by: Simon Glass --- Changes in v5: - Add new patch to drop bloblist_maybe_init() common/bloblist.c | 8 common/board_f.c | 4 +++- include

[PATCH v5 10/25] bloblist: Adjust how the bloblist is received from passage

2025-05-28 Thread Simon Glass
valid bloblist, since the devicetree is processed first. Signed-off-by: Simon Glass --- (no changes since v3) Changes in v3: - Add new patch to adjust how the bloblist is received from stdpass common/bloblist.c | 70 --- 1 file changed, 30

[PATCH v5 08/25] fdt: Redo devicetree setup

2025-05-28 Thread Simon Glass
The existing logic for handling receiving a devicetree from a previous phase is quite complicated. Simplify it in preparation for introducing standard passage. Add a Kconfig called OF_PASSAGE which indicates that the devicetree must come from standard passage. Signed-off-by: Simon Glass

[PATCH v5 07/25] passage: Support an incoming passage

2025-05-28 Thread Simon Glass
Plumb in the ability for U-Boot proper to accept an incoming standard passage from a previous phase, such as SPL or TF-A. This allows data to be passed from binary to binary when firmware is booting. Signed-off-by: Simon Glass --- (no changes since v3) Changes in v3: - Add passage_valid() to

[PATCH v5 06/25] spl: Rename jump_to_image_no_args()

2025-05-28 Thread Simon Glass
This is about to become a misnomer since with standard passage we need to pass arguments to U-Boot. Rename the function. Also rename the local variable to 'jumper' to avoid a conflict. Signed-off-by: Simon Glass --- (no changes since v2) Changes in v2: - Split the jump_to_ima

[PATCH v5 05/25] x86: Move Intel GNVS file into the common include directory

2025-05-28 Thread Simon Glass
Move this so we can include it from sandbox, needed since it is in a bloblist and must have a check. Signed-off-by: Simon Glass --- (no changes since v1) arch/x86/cpu/apollolake/acpi.c| 2 +- arch/x86/cpu/intel_common/acpi.c | 2 +- arch/x86/include/asm

[PATCH v5 01/25] RFC: scripts: Add scripts for running QEMU

2025-05-28 Thread Simon Glass
These is present in my tree. Provide it as a patch to simplify testing this series with QEMU. Signed-off-by: Simon Glass --- (no changes since v1) scripts/build-efi | 173 + scripts/build-qemu | 415 scripts/build_helper.py

[PATCH v5 04/25] spl: Tidy up the header includes

2025-05-28 Thread Simon Glass
These are out of order. Fix them. Signed-off-by: Simon Glass --- (no changes since v1) common/spl/spl.c | 27 +-- 1 file changed, 13 insertions(+), 14 deletions(-) diff --git a/common/spl/spl.c b/common/spl/spl.c index 76fd56dfe4b..abbca9acde0 100644 --- a/common/spl

[PATCH v5 02/25] RFC: scripts: build-qemu: Support xPL with ARM

2025-05-28 Thread Simon Glass
Add a -x flag to build/run xPL instead of the normal build. Signed-off-by: Simon Glass --- Changes in v5: - Add RFC for test script scripts/build-qemu | 15 --- 1 file changed, 12 insertions(+), 3 deletions(-) diff --git a/scripts/build-qemu b/scripts/build-qemu index 5798da3775c

Re: Bloblist request

2025-05-28 Thread Simon Glass
Hi Tom, On Mon, 26 May 2025 at 15:42, Tom Rini wrote: > > On Sun, May 25, 2025 at 09:43:14AM +0100, Simon Glass wrote: > > > Hi Tom, > > > > Bloblist is IMO a bit of a mess, with 2-3 maintainers who don't agree > > on the direction. > > > > Yo

Re: Booting distros in CI

2025-05-28 Thread Simon Glass
Hi Tom, On Mon, 26 May 2025 at 15:51, Tom Rini wrote: > > On Mon, May 26, 2025 at 03:10:49PM +0100, Simon Glass wrote: > > > Hi Tom, > > > > What do you think about having a Linux distro in the docker image so > > we could run a 'boot' test in CI witho

[PATCH 13/27] efi: Allow disabling networking in the app

2025-05-28 Thread Simon Glass
When networking is disabled, avoid trying to use networking in the app. Signed-off-by: Simon Glass --- lib/efi_client/efi_app_init.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/lib/efi_client/efi_app_init.c b/lib/efi_client/efi_app_init.c index 25a9b105d63

[PATCH 18/27] efi: Provide an easy way to debug with gdb

2025-05-28 Thread Simon Glass
Add a Kconfig option to easily enable debugging of the app using the recommended method. Provide some docs too. Signed-off-by: Simon Glass --- doc/develop/uefi/u-boot_on_efi.rst | 39 ++ lib/efi_client/Kconfig | 12 - lib/efi_client/efi.c

[PATCH 11/27] vbe: Avoid accessing TEXT_BASE if undefined

2025-05-28 Thread Simon Glass
Use a condition to ensure this symbol is not accessed in the EFI app. Signed-off-by: Simon Glass --- boot/vbe_common.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/boot/vbe_common.c b/boot/vbe_common.c index a86986d86e9..f9368a6a9ab 100644 --- a/boot/vbe_common.c +++ b

[PATCH 20/27] x86: efi: Tidy up the link scripts

2025-05-28 Thread Simon Glass
Mark important sections with KEEP so that the linker will not remove them when garbage-collection is (later) enabled. Provide a _end symbol so that this can be accessed in setup_mon_len() Signed-off-by: Simon Glass --- arch/x86/lib/elf_ia32_efi.lds | 5 +++-- arch/x86/lib/elf_x86_64_efi.lds

[PATCH 01/27] scripts: Tidy up build-qemu a little

2025-05-28 Thread Simon Glass
This provides a few clean-ups: - sort the imports correctly - sort the arguments (-v is in the wrong place) - use os_arch for the OS architecture, to match build-efi Signed-off-by: Simon Glass --- scripts/build-qemu | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions

[PATCH 05/27] scripts: Update build-efi to allow showing the QEMU cmdline

2025-05-28 Thread Simon Glass
Add a -v option for verbose output. For now this just shows the QEMU cmdline before starting it. This allows the user to start it separately, perhaps tweaking the args. Signed-off-by: Simon Glass --- scripts/build-efi | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a

[PATCH 12/27] efi: Use the correct driver for TPM

2025-05-28 Thread Simon Glass
The TPM driver used in the app here should 'efi_tpm', not 'efi_net'. Fix it. Signed-off-by: Simon Glass --- lib/efi_client/efi_app_init.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/efi_client/efi_app_init.c b/lib/efi_client/efi_app_init

[PATCH 27/27] x86: efi: Add various useful options

2025-05-28 Thread Simon Glass
Enable logging, TPM and the lsblk and cat commands. These are all useful for using and debugging the EFI app. For the 32-bit app, enable CONFIG_BOARD_EARLY_INIT_R so that the EFI devices are scanned. Signed-off-by: Simon Glass --- configs/efi-x86_app32_defconfig | 9 + configs/efi

[PATCH 26/27] x86: efi: Enable bootstd

2025-05-28 Thread Simon Glass
Use bootstd for booting. Add the -p option since we may wish to boot from an extlinux file even if a EFI partition is present. Disable the bootz command since this is not applicable for x86 and causes bootstd to attempt it anyway. Signed-off-by: Simon Glass --- configs/efi-x86_app32_defconfig

[PATCH 25/27] x86: efi: Drop some unused options

2025-05-28 Thread Simon Glass
The cache is managed by the first-stage EFI firmware, so the 'cache' command is not useful. SCSI and AHCI are not used in the app, since it relies on EFI for all block devices. Make sure all three options are not present in the 32/64-bit apps. Signed-off-by: Simon Glass --- c

[PATCH 24/27] x86: efi: Enable --gc-sections to drop unused code

2025-05-28 Thread Simon Glass
Add --gc-sections to help reduce the size of the EFI app. Add a check for undefined symbols too, since using these can hang or crash the app. Signed-off-by: Simon Glass --- arch/x86/config.mk | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/x86/config.mk b/arch/x86

[PATCH 23/27] x86: efi: Drop relocate and physmem

2025-05-28 Thread Simon Glass
These two files are not used in the EFI app and it does not make sense to compile them in. Omit relocate.c and physmem.c Signed-off-by: Simon Glass --- arch/x86/lib/Makefile | 3 +++ 1 file changed, 3 insertions(+) diff --git a/arch/x86/lib/Makefile b/arch/x86/lib/Makefile index afb6abb57d3

[PATCH 21/27] x86: efi: Add a few dummy functions

2025-05-28 Thread Simon Glass
Although GD_FLG_SKIP_RELOC is enabled for the app, the code for relocation is still present in the image. Add a few dummy functions needed on x86. Signed-off-by: Simon Glass --- lib/efi_client/efi_app.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/lib/efi_client/efi_app.c b

[PATCH 22/27] x86: efi: Avoid calling arch_setup_gd() from jump_to_copy()

2025-05-28 Thread Simon Glass
The EFI app has its own way of handling the global_data pointer, regardless of which arch is in use. Skip the jump_to_copy() function since it calls the arch-specific arch_setup_gd(). Signed-off-by: Simon Glass --- common/board_f.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

[PATCH 17/27] efi: Add a bootdev for EFI media

2025-05-28 Thread Simon Glass
Bootstd needs a driver so that it can scan EFI media within the EFI app. Add one. Signed-off-by: Simon Glass --- drivers/block/efi_blk.c | 30 ++ 1 file changed, 30 insertions(+) diff --git a/drivers/block/efi_blk.c b/drivers/block/efi_blk.c index 9766cd6f832

[PATCH 16/27] efi: doc: Use inline links for EFI app documentation

2025-05-28 Thread Simon Glass
Update the links to an embedded style. Also drop the part about needing to support ARM, as this is done now. Signed-off-by: Simon Glass --- doc/develop/uefi/u-boot_on_efi.rst | 9 +++-- 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/doc/develop/uefi/u-boot_on_efi.rst b/doc

[PATCH 19/27] x86: efi: Sync up the two link scripts somewhat

2025-05-28 Thread Simon Glass
. Signed-off-by: Simon Glass --- arch/x86/lib/elf_ia32_efi.lds | 26 ++ arch/x86/lib/elf_x86_64_efi.lds | 25 + 2 files changed, 23 insertions(+), 28 deletions(-) diff --git a/arch/x86/lib/elf_ia32_efi.lds b/arch/x86/lib/elf_ia32_efi.lds index

[PATCH 15/27] efi: Allow disabling the TPM in the app

2025-05-28 Thread Simon Glass
When the TPM is disabled, avoid trying to use it in the app. Signed-off-by: Simon Glass --- lib/efi_client/efi_app_init.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/lib/efi_client/efi_app_init.c b/lib/efi_client/efi_app_init.c index ff31266fb12..938b16d14ab

[PATCH 10/27] bootstd: Provide an option to scan any partition

2025-05-28 Thread Simon Glass
tflow scan' to allow any partition to be scanned. Signed-off-by: Simon Glass --- cmd/bootflow.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/cmd/bootflow.c b/cmd/bootflow.c index 6063d9dfa7d..a8d9cb213e6 100644 --- a/cmd/bootflow.c +++ b/cmd/bootflow.c @@ -15

[PATCH 14/27] efi: Rename the media device

2025-05-28 Thread Simon Glass
The current name of 'efi_media' is annoying in that it must be given with 'bootflow scan'. We would prefer to use 'bootflow scan efi', for example. Rename the driver to 'efi'. Signed-off-by: Simon Glass --- drivers/block/efi-media-uclass.c | 2 +- 1

[PATCH 08/27] scripts: Update build-efi to allow firmware settings

2025-05-28 Thread Simon Glass
At present the -bios option is used to specify the firmware, but this does not allow settings to be adjusted. Use the pflash method instead, with a separate variables file, so that settings can be changed. Add a -g option to tell QEMU to stop and wait for gdb to connect. Signed-off-by: Simon

[PATCH 07/27] scripts: Update build-efi to support debugging with QEMU

2025-05-28 Thread Simon Glass
Add a -g option to tell QEMU to stop and wait for gdb to connect. Signed-off-by: Simon Glass --- scripts/build-efi | 5 + 1 file changed, 5 insertions(+) diff --git a/scripts/build-efi b/scripts/build-efi index 5a6eef450d4..33702953c32 100755 --- a/scripts/build-efi +++ b/scripts/build

[PATCH 09/27] bootstd: Tidy up help for boot flags

2025-05-28 Thread Simon Glass
The help for 'bootflow scan' is missing some flags. Tidy this up and update the docs as well, keeping the flags in alpha order. Signed-off-by: Simon Glass --- cmd/bootflow.c | 4 +++- doc/usage/cmd/bootflow.rst | 15 +-- 2 files changed, 12 insertions(+), 7

[PATCH 02/27] scripts: Change -k and -o options

2025-05-28 Thread Simon Glass
These have a different meaning in build-qemu, so capitalise these options to avoid confusion. Signed-off-by: Simon Glass --- scripts/build-efi | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/scripts/build-efi b/scripts/build-efi index 2f4b916c49b..443a5ca4908 100755

[PATCH 06/27] scripts: Update build-efi to support passing a root disk

2025-05-28 Thread Simon Glass
Add a -d option to specify a root disk to boot with. Signed-off-by: Simon Glass --- scripts/build-efi | 13 - 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/scripts/build-efi b/scripts/build-efi index 4eb3a76d1a2..5a6eef450d4 100755 --- a/scripts/build-efi +++ b

[PATCH 00/27] x86: efi: Various app improvements to support bootstd

2025-05-28 Thread Simon Glass
an extlinux OS on EFI media - Make the 32/64-bit apps more similar in terms of features and link script Simon Glass (27): scripts: Tidy up build-qemu a little scripts: Change -k and -o options scripts: Update build-efi to support kvm scripts: Update build-efi to support booting an OS

[PATCH 04/27] scripts: Update build-efi to support booting an OS

2025-05-28 Thread Simon Glass
Bring this script into line with build-qemu so an OS and release can be specified. As with that script, only Ubuntu is supported for now. Signed-off-by: Simon Glass --- scripts/build-efi | 29 - 1 file changed, 28 insertions(+), 1 deletion(-) diff --git a/scripts

[PATCH 03/27] scripts: Update build-efi to support kvm

2025-05-28 Thread Simon Glass
This is a useful feature when booting the EFI app, so add this option to the build-efi script too, matching the same option in build-qemu Signed-off-by: Simon Glass --- scripts/build-efi | 6 ++ 1 file changed, 6 insertions(+) diff --git a/scripts/build-efi b/scripts/build-efi index

Re: [PATCH 00/10] efi: Move some efi-loader code into a new shared dir

2025-05-28 Thread Simon Glass
On Fri, 23 May 2025 at 14:07, Simon Glass wrote: > > Some functions provided in lib/efi_loader are actually useful for the > app as well. This series refactors the Kconfig and directories a little > so that code is easier to share. > > As a starting point, it moves some filena

Re: [PATCH v3 3/4] lib: efi_loader: efi_memory.c: add efi_realloc() for realloc memory

2025-05-28 Thread Simon Glass
Hi Paul, On Wed, 28 May 2025 at 01:24, Paul Liu wrote: > > > > On Fri, 23 May 2025 at 14:44, Simon Glass wrote: >> >> Hi, >> >> On Thu, 22 May 2025 at 03:18, Ying-Chun Liu (PaulLiu) >> wrote: >> > >> > From: "Ying-Chun Li

Re: [PATCH 0/3] efi_loader: Move the public cerificate back to the devicetree

2025-05-28 Thread Simon Glass
Hi Ilias, On Tue, 27 May 2025 at 10:20, Ilias Apalodimas wrote: > > Hi Simon, > > On Tue, 27 May 2025 at 11:20, Simon Glass wrote: > > > > Hi Ilias, > > > > On Sat, 24 May 2025 at 19:13, Ilias Apalodimas > > wrote: > > > > > > Hi

  1   2   3   4   5   6   7   8   9   10   >