[PATCH 0/2] Remove "saveenv" functionality from am57xx evms

2025-02-07 Thread Anurag Dutta
Previously saved environment introduce discrepancies and may lead to incompatibilities without default settings. This series removes the saved environment functionality on am57xx evms so that the default configuration is always loaded Test result: https://gist.github.com/anuragdutta731/b253ddb0a5

[PATCH 2/2] configs: am57xx: Remove saved environments

2025-02-07 Thread Anurag Dutta
Saved environments lead to inconsistencies leading to conflicts with the default environment that U-boot should update during development. Remove the previously saved environment so that the default environment is always loaded. Signed-off-by: Anurag Dutta --- configs/am57xx_evm_defconfig | 4 +-

[PATCH 1/2] configs: am57xx_hs: Remove saved environments

2025-02-07 Thread Anurag Dutta
Saved environments cause inconsistencies leading to conflicts with the default environment that U-boot should update during development. Remove the previously saved environment so that the default environment is always loaded. Signed-off-by: Anurag Dutta --- configs/am57xx_hs_evm_defconfig | 4 +

Re: [PATCH 1/6] configs: add jh7110-deepcomputing-fml13v01 to VF2 defconfig

2025-02-07 Thread Sumit Garg
On Fri, 7 Feb 2025 at 22:52, Heinrich Schuchardt wrote: > > The DeepComputing Framework motherboard is a JH7110 device support by the > upstream kernel. Add its device-tree to the list of device-trees to be > included into the starfive_visionfive_defconfig. > > Signed-off-by: Heinrich Schuchardt

Re: [PATCH 3/6] riscv: dts: jh7110: add DeepComputing FML13V01 device-tree

2025-02-07 Thread Sumit Garg
On Fri, 7 Feb 2025 at 22:53, Heinrich Schuchardt wrote: > > Add the u-boot device-tree include needed to support the > DeepComputing Framework motherboard (FML13V01). > > Signed-off-by: Heinrich Schuchardt > --- > arch/riscv/dts/jh7110-deepcomputing-fml13v01-u-boot.dtsi | 7 +++ > 1 file cha

Re: Help needed: How to Specify MTD Partitions on NAND in a Device Tree

2025-02-07 Thread Weijie Gao
Hi Mauri, Here's an example: https://github.com/openwrt/openwrt/blob/main/package/boot/uboot-mediatek/patches/105-configs-add-usefull-stuff-to-mt7988-rfb.patch#L316 Weijie Mauri Sandberg 于2025年2月8日周六 03:06写道: > > Hello all, > > I have a MT7621A basec SoC, which I am trying to update the boot pr

Re: Help needed: How to Specify MTD Partitions on NAND in a Device Tree

2025-02-07 Thread Mauri Sandberg
Hi Weijie On 2/7/25 21:53, Weijie Gao wrote: Here's an example: https://github.com/openwrt/openwrt/blob/main/package/boot/uboot-mediatek/patches/105-configs-add-usefull-stuff-to-mt7988-rfb.patch#L316 Sorry, I should have been more specific. Actually defining the partitions is the simpler part

Re: [PATCH 6/6] doc: add DeepComputing FML13V01 documentation

2025-02-07 Thread Heinrich Schuchardt
E Shattow schrieb am Fr., 7. Feb. 2025, 19:10: > Can we reduce the visionfive2 multi-target docs down to one document for > the target and not each individual board? Essentially the differences are > the mmc ordering and what boot mode GPIO selection is possible. Much is > copy-paste anyhow. -E >

[PATCH v2 05/29] test: Show the average time per test

2025-02-07 Thread Simon Glass
Show the average duration of a test, so we can keep track of how it is trending. Report the suite with the longest average test to encourage people to improve it. Add a function to update the stats based on the results from a single suite and another to show the summary information. Make this opt

[PATCH v2 04/29] test: Keep track of suite duration

2025-02-07 Thread Simon Glass
Show the time taken by each test suite with 'ut all' and the total time for all suites. Take care to remove any sandbox time-offset from the values. Fix the comment-format on timer_test_add_offset() while we are here. Signed-off-by: Simon Glass --- (no changes since v1) drivers/timer/sandbox

[PATCH v2 02/29] test: Fix a stray asterisk in ut_run_list()

2025-02-07 Thread Simon Glass
Drop the unwanted asterisk in the comment. Signed-off-by: Simon Glass --- (no changes since v1) test/test-main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/test-main.c b/test/test-main.c index 22b9b46d9cd..815f54bebd5 100644 --- a/test/test-main.c +++ b/test/test-

[PATCH v2 00/29] test: Complete the suite migration

2025-02-07 Thread Simon Glass
This series completes the removal of test commands for suites. With this it is possible to declare a suite (including init and uninit functions) without needing to write a command. It also adds timing for test suites, so we can keep track of how long things take. Changes in v2: - Add new patch to

[PATCH v2 01/29] test: Drop sandbox_set_enable_memio() from mux-cmd test

2025-02-07 Thread Simon Glass
This test does not appear to use sandbox's memory-mapped I/O so there is no need to enable it. Even if there were a need, it should be disabled at the end of the test, so as not to affect other tests. Drop these lines from the test. Signed-off-by: Simon Glass --- Changes in v2: - Add new patch

[PATCH v2 03/29] test: Add up the number of tests manually

2025-02-07 Thread Simon Glass
All tests should belong to a suite, but if there is a suite we don't know about (e.g. not added to cmd_ut.c) then the totals will not add up. Add a check for this. Signed-off-by: Simon Glass --- (no changes since v1) test/cmd_ut.c | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-

Re: [PATCH 1/3] bootstd: Fix memleak on errors in bootmeth_setup_iter_order()

2025-02-07 Thread Sam Protsenko
On Mon, Jan 13, 2025 at 1:37 AM Heinrich Schuchardt wrote: > > On 1/12/25 04:42, Sam Protsenko wrote: > > Free memory allocated for 'order' (array of bootmeths) on error paths in > > bootmeth_setup_iter_order() function. > > > > Fixes: c627cfc14c08 ("bootstd: Allow scanning for global bootmeths >

[PATCH v2 27/29] test: Move code out of cmd_ut_category()

2025-02-07 Thread Simon Glass
Move the logic from this function into run_suite(), on the way to having flag parsing in the top-level 'ut' command instead of its children. Signed-off-by: Simon Glass --- (no changes since v1) test/cmd_ut.c | 62 +-- 1 file changed, 25 insertion

[PATCH v2 21/29] test: Drop the function for running optee tests

2025-02-07 Thread Simon Glass
Use the new suite-runner to run these tests instead. Signed-off-by: Simon Glass --- (no changes since v1) include/test/suites.h | 2 -- test/cmd_ut.c | 4 +--- test/optee/cmd_ut_optee.c | 14 -- 3 files changed, 1 insertion(+), 19 deletions(-) diff --git a/includ

[PATCH v2 23/29] test: Drop support for test commands

2025-02-07 Thread Simon Glass
Now that everything is using the new test-suite features, drop support for running commands. Fix a missing closing-bracket while we are here. Signed-off-by: Simon Glass --- (no changes since v1) include/test/suites.h | 5 - test/cmd_ut.c | 43 -

[PATCH v2 13/29] test: Move fdt_overlay init into a function

2025-02-07 Thread Simon Glass
Move the init code into a separate function since it is quite large. Adjust it to use unit-test functions which have become available since the test was written. Signed-off-by: Simon Glass --- (no changes since v1) test/fdt_overlay/cmd_ut_fdt_overlay.c | 113 +- 1 file

[PATCH v2 16/29] test: Drop the function for running fdt_overlay tests

2025-02-07 Thread Simon Glass
Use the new suite-runner to run these tests instead. Signed-off-by: Simon Glass --- (no changes since v1) include/test/suites.h | 2 -- test/cmd_ut.c | 4 +--- test/fdt_overlay/cmd_ut_fdt_overlay.c | 15 --- 3 files changed, 1 insertion(+),

Re: [PATCH 28/29] Revert "bloblist: Load the bloblist from the previous loader"

2025-02-07 Thread Tom Rini
On Fri, Feb 07, 2025 at 01:26:46PM -0500, Raymond Mao wrote: > Hi Simon, > > On Fri, 7 Feb 2025 at 12:51, Simon Glass wrote: > > > > Hi Raymond, > > > > On Fri, 7 Feb 2025 at 08:10, Raymond Mao wrote: > > > > > > Hi Simon, > > > > > > On Thu, 6 Feb 2025 at 19:09, Simon Glass wrote: > > > > > >

[PATCH v2 24/29] test: Make cmd_ut_category() static

2025-02-07 Thread Simon Glass
This function is not used outside the cmd_ut file anymore, so make it static. Signed-off-by: Simon Glass --- (no changes since v1) include/test/suites.h | 22 -- test/cmd_ut.c | 6 +++--- 2 files changed, 3 insertions(+), 25 deletions(-) diff --git a/include/test/

Re: [PATCH 2/6] riscv: dts: starfive: DeepComputing FML13V01 binman config

2025-02-07 Thread Heinrich Schuchardt
E Shattow schrieb am Fr., 7. Feb. 2025, 19:12: > Is this use of filename preferable compared to a cherry-pick commit > of/from dt-rebasing? > The upstream device-tree is already meged into U-Boot. The entries for the new board matches how the other boards are implemented. I do not understand how

[PATCH] Azure: Add missing "set -e" to docs job

2025-02-07 Thread Tom Rini
Without setting the shell flag to exit immediately when a command exists with a non-zero status we can have the situation where the htmldocs target fails with an error but the job will succeed due to infodocs passing and being the last build target. Signed-off-by: Tom Rini --- .azure-pipelines.y

[PATCH v2 12/29] test: Make all tests depend on UNIT_TEST

2025-02-07 Thread Simon Glass
Rather than having this condition defined separately for each suite, bracket all options with 'if UNIT_TEST'. Signed-off-by: Simon Glass --- (no changes since v1) test/Kconfig | 14 ++ test/dm/Kconfig | 2 +- test/env/Kconfig | 1 - test/fdt_overlay/K

[PATCH v2 19/29] test: Update optee to do init and uninit from tests

2025-02-07 Thread Simon Glass
Rather than having an init function and then running the tests, create a test-init function to do it. This will allow us to get rid of the command function. Fix the comment abotu 'environment' while we are here. Signed-off-by: Simon Glass --- (no changes since v1) include/test/optee.h |

[PATCH v2 15/29] test: Update fdt_overlay to do init from tests

2025-02-07 Thread Simon Glass
Rather than having an init function and then running the tests, create a test-init function to do it. This will allow us to get rid of the command function. Signed-off-by: Simon Glass --- (no changes since v1) include/test/fdt_overlay.h| 4 test/fdt_overlay/cmd_ut_fdt_overlay.

[PATCH v2 09/29] test: Move env-test rule into test/

2025-02-07 Thread Simon Glass
The Makefile rules for tests should be within test/Makefile so move the 'env' rule over. Signed-off-by: Simon Glass --- (no changes since v1) Makefile | 1 - test/Makefile | 1 + 2 files changed, 1 insertion(+), 1 deletion(-) diff --git a/Makefile b/Makefile index db663979206..74eafdeb64

[PATCH v2 28/29] test: Do flag-processing in the correct place

2025-02-07 Thread Simon Glass
At present the 'ut' command handles its flags in a strange way, in that they must come after the subcommand. So, we must use 'ut bloblist -r2' to run the bloblist tests twice. This is an artefact of the way tests were run, through subcommands. It is now possible to correct this, by doing flag-pro

[PATCH v2 29/29] test: Update documentation

2025-02-07 Thread Simon Glass
Update documentation for how to write tests and the 'ut' command. Signed-off-by: Simon Glass --- (no changes since v1) doc/develop/tests_writing.rst | 52 ++-- doc/usage/cmd/ut.rst | 145 ++ test/cmd_ut.c | 12 +-- 3 files chan

Re: [PATCH 28/29] Revert "bloblist: Load the bloblist from the previous loader"

2025-02-07 Thread Raymond Mao
Hi Simon, On Fri, 7 Feb 2025 at 12:51, Simon Glass wrote: > > Hi Raymond, > > On Fri, 7 Feb 2025 at 08:10, Raymond Mao wrote: > > > > Hi Simon, > > > > On Thu, 6 Feb 2025 at 19:09, Simon Glass wrote: > > > > > > Hi Raymond, > > > > > > On Thu, 6 Feb 2025 at 15:35, Raymond Mao wrote: > > > > >

[PATCH v2 14/29] test: Update fdt_overlay test to use fdtdec functions

2025-02-07 Thread Simon Glass
Use the helpers provided for this purpose, rather than different ones in this particular test. Leave fdt_getprop_str() alone as it seems to have more value. Signed-off-by: Simon Glass --- (no changes since v1) test/fdt_overlay/cmd_ut_fdt_overlay.c | 73 --- 1 file chan

Re: Security vulnerabilities report to Das-U-Boot

2025-02-07 Thread Jonathan Bar Or
Thank you. So, I'm attaching my findings in a md file - see attachment. All of those could be avoided by using safe math, such as __builtin_mul_overflow and __builtin_add_overflow, which are used in some modules in Das-U-Boot. There are many cases where seemingly unsafe addition and multiplication

Help needed: How to Specify MTD Partitions on NAND in a Device Tree

2025-02-07 Thread Mauri Sandberg
Hello all, I have a MT7621A basec SoC, which I am trying to update the boot prcess for. As it is now, I have v2024.10 based U-boot running and I can specify MTD partitions using env variable 'mtdparts' and use them all right. But I am looking for a way to specify them in a device tree. Linux iden

[PATCH v2 06/29] test: Leave out the prefix when printing test names

2025-02-07 Thread Simon Glass
When tests are all in the same suite it is annoying to have to read all the common text after each name. Skip this to help the user. Signed-off-by: Simon Glass --- (no changes since v1) test/cmd_ut.c | 2 +- test/py/tests/test_suite.py | 7 --- test/test-main.c|

[PATCH v2 25/29] test: Drop suites.h

2025-02-07 Thread Simon Glass
This file is empty now. Remove it and its uses. Signed-off-by: Simon Glass --- (no changes since v1) include/test/suites.h | 10 -- test/boot/bootdev.c | 1 - test/boot/bootflow.c | 1 - test/boot/bootm.c | 1 -

[PATCH v2 17/29] test: Update bootstd to do init from tests

2025-02-07 Thread Simon Glass
Rather than having an init function and then running the tests, create a test-init function to do it. This will allow us to get rid of the command function. Signed-off-by: Simon Glass --- (no changes since v1) test/boot/bootstd_common.c | 18 +- test/boot/bootstd_common.h | 10

[PATCH v2 18/29] test: Drop the function for running bootstd tests

2025-02-07 Thread Simon Glass
Use the new suite-runner to run these tests instead. Signed-off-by: Simon Glass --- (no changes since v1) include/test/suites.h | 2 -- test/boot/bootstd_common.c | 10 -- test/cmd_ut.c | 4 +--- 3 files changed, 1 insertion(+), 15 deletions(-) diff --git a/include

[PATCH v2 10/29] test: Move optee-test rule into test/

2025-02-07 Thread Simon Glass
The Makefile rules for tests should be within test/Makefile so move the 'optee' rule over. Signed-off-by: Simon Glass --- (no changes since v1) Makefile | 1 - test/Makefile | 1 + 2 files changed, 1 insertion(+), 1 deletion(-) diff --git a/Makefile b/Makefile index 74eafdeb646..e0f58ccc

[PATCH v2 11/29] test: Move fdt-overlay-test rule into test/

2025-02-07 Thread Simon Glass
The Makefile rules for tests should be within test/Makefile so move the 'fdt-overlay' rule over. Signed-off-by: Simon Glass --- (no changes since v1) Makefile | 1 - test/Makefile | 1 + 2 files changed, 1 insertion(+), 1 deletion(-) diff --git a/Makefile b/Makefile index e0f58cccd00..6f

[PATCH v2 26/29] test: Allow running a selection of suites

2025-02-07 Thread Simon Glass
Enhance the ut command to accept a comma-separated list of test suites to run. Report the summary information for these at the end. Signed-off-by: Simon Glass --- Changes in v2: - Use strsep() instead, in case tests use strtok() test/cmd_ut.c | 32

[PATCH v2 20/29] test: Split optee tests into three functions

2025-02-07 Thread Simon Glass
These tests run three different checks on the nodes, but the logic is currently all in one tests. Split the code out into three different tests, which do different setup and then run the same checks. Signed-off-by: Simon Glass --- (no changes since v1) test/optee/cmd_ut_optee.c | 57 +

[PATCH v2 22/29] test: Rename optee test-file

2025-02-07 Thread Simon Glass
This has nothing to do with commands anymore, so rename the file. Signed-off-by: Simon Glass --- (no changes since v1) test/optee/Makefile| 2 +- test/optee/{cmd_ut_optee.c => optee.c} | 0 2 files changed, 1 insertion(+), 1 deletion(-) rename test/optee/{cmd_ut_optee.c =>

Re: [PATCH v4 0/2] led: add function naming option from linux

2025-02-07 Thread Tom Rini
On Tue, 28 Jan 2025 14:52:44 +0100, Heiko Schocher wrote: > in linux we have the option to create the name of a led > optionally through the following properties: > > - function > - color > - function-enumerator > > [...] Applied to u-boot/master, thanks! -- Tom

[PATCH 6/6] doc: add DeepComputing FML13V01 documentation

2025-02-07 Thread Heinrich Schuchardt
Describe building U-Boot for the board and booting. Signed-off-by: Heinrich Schuchardt --- doc/board/starfive/deepcomputing_fml13v01.rst | 143 ++ doc/board/starfive/index.rst | 1 + 2 files changed, 144 insertions(+) create mode 100644 doc/board/starfive/deep

[PATCH 1/6] configs: add jh7110-deepcomputing-fml13v01 to VF2 defconfig

2025-02-07 Thread Heinrich Schuchardt
The DeepComputing Framework motherboard is a JH7110 device support by the upstream kernel. Add its device-tree to the list of device-trees to be included into the starfive_visionfive_defconfig. Signed-off-by: Heinrich Schuchardt --- configs/starfive_visionfive2_defconfig | 2 +- 1 file changed,

[PATCH 5/6] board: starfive: spl: support DeepComputing FML13V01

2025-02-07 Thread Heinrich Schuchardt
On the DeepComputing Framework motherboard (FML13V01) choose the matching FIT configuration. Signed-off-by: Heinrich Schuchardt --- board/starfive/visionfive2/spl.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/board/starfive/visionfive2/spl.c b/board/starfive/visionfiv

[PATCH 3/6] riscv: dts: jh7110: add DeepComputing FML13V01 device-tree

2025-02-07 Thread Heinrich Schuchardt
Add the u-boot device-tree include needed to support the DeepComputing Framework motherboard (FML13V01). Signed-off-by: Heinrich Schuchardt --- arch/riscv/dts/jh7110-deepcomputing-fml13v01-u-boot.dtsi | 7 +++ 1 file changed, 7 insertions(+) create mode 100644 arch/riscv/dts/jh7110-deepcomp

[PATCH 2/6] riscv: dts: starfive: DeepComputing FML13V01 binman config

2025-02-07 Thread Heinrich Schuchardt
Add the DeepComputing Framework motherboard (FML13V01) to the binman include for the starfive_visionfive2_defconfig. Signed-off-by: Heinrich Schuchardt --- arch/riscv/dts/starfive-visionfive2-binman.dtsi | 17 + 1 file changed, 17 insertions(+) diff --git a/arch/riscv/dts/starfi

[PATCH 0/6] board: starfive: DeepComputing FML13V01

2025-02-07 Thread Heinrich Schuchardt
We use starfive_visionfive2_defconfig for supporting JH7110 based boards. The DeepComputing Framework motherboard is a new JH7110 device with upstream kernel support since v6.13. Add support and documentation for the board. Heinrich Schuchardt (6): configs: add jh7110-deepcomputing-fml13v01 to

Re: [PATCH RFC v2 2/8] arm: dts: k3-j784s4-binman.dtsi: Clean up and templatize boot binaries

2025-02-07 Thread Andrew Davis
On 2/6/25 4:52 AM, Manorit Chawdhry wrote: Hi Andrew, On 09:35-20250205, Andrew Davis wrote: On 2/4/25 11:50 PM, Manorit Chawdhry wrote: Hi Andrew, On 09:43-20250204, Andrew Davis wrote: On 2/3/25 10:43 PM, Manorit Chawdhry wrote: Hi Andrew, On 14:44-20250203, Andrew Davis wrote: On 1/6/2

[PATCH 4/6] board: starfive: DeepComputing FML13V01 fdt selection

2025-02-07 Thread Heinrich Schuchardt
We support all JH7110 boards with starfive_visionfive2_defconfig. The relevant device-tree is selected at runtime based on EEPROM data. Support setting $fdtfile to the file name of the DeepComputing Framework motherboard (FML13V01) device-tree. Signed-off-by: Heinrich Schuchardt --- board/starf

Re: [PATCH 28/29] Revert "bloblist: Load the bloblist from the previous loader"

2025-02-07 Thread Simon Glass
Hi Raymond, On Fri, 7 Feb 2025 at 08:10, Raymond Mao wrote: > > Hi Simon, > > On Thu, 6 Feb 2025 at 19:09, Simon Glass wrote: > > > > Hi Raymond, > > > > On Thu, 6 Feb 2025 at 15:35, Raymond Mao wrote: > > > > > > Hi Simon, > > > > > > On Thu, 6 Feb 2025 at 10:41, Simon Glass wrote: > > > > >

[PATCH v2 07/29] test: Support an init/uninit functions for test suites

2025-02-07 Thread Simon Glass
Some suites need things to be set up before they can run. Add a way to declare an init function using the UNIT_TEST_INIT() macro. The init function is just like any other test, but is always placed first so that it runs before all the other test functions in the suite. Add an uninit function as we

[PATCH v2 08/29] test: Tweak FDT-overlay tests

2025-02-07 Thread Simon Glass
Use fdt_overlay consistently in the identifiers and file/dir names. Signed-off-by: Simon Glass --- (no changes since v1) Makefile | 2 +- include/test/{overlay.h => fdt_overlay.h} | 4 +- include/test/suites.h | 4 +- test/Kco

Re: [PATCH v4 6/6] configs: am6*_evm_a53_defconfig: Enable remoteproc support

2025-02-07 Thread Judith Mendez
Hi Andrew, On 2/6/25 5:07 PM, Andrew Davis wrote: On 2/5/25 7:54 PM, Judith Mendez wrote: From: Hari Nagalla Enable configs needed to enable remoteproc drivers for AM62 and AM64 devices. While we are here, cleanup some remoteproc configs, there is no DSP subsystem for am62px device. Signed-

Re: [PATCH v3 0/4] cmd/setexpr: support concatenation of direct strings

2025-02-07 Thread Tom Rini
On Mon, 03 Feb 2025 16:10:25 +0100, Heinrich Schuchardt wrote: > The setexpr.s command allows to concatenate two strings. > > According to the description in doc/usage/cmd/setexpr.rst the parameters > value1 and value2 can be either direct values or pointers to a > memory location holding the val

Re: [PATCH 1/3] bootstd: Fix memleak on errors in bootmeth_setup_iter_order()

2025-02-07 Thread Sam Protsenko
On Tue, Jan 14, 2025 at 7:14 AM Simon Glass wrote: > > On Sat, 11 Jan 2025 at 20:42, Sam Protsenko > wrote: > > > > Free memory allocated for 'order' (array of bootmeths) on error paths in > > bootmeth_setup_iter_order() function. > > > > Fixes: c627cfc14c08 ("bootstd: Allow scanning for global

Re: [PATCH 1/3] bootstd: Fix memleak on errors in bootmeth_setup_iter_order()

2025-02-07 Thread Simon Glass
Hi Sam, On Fri, 7 Feb 2025 at 18:11, Sam Protsenko wrote: > > On Tue, Jan 14, 2025 at 7:14 AM Simon Glass wrote: > > > > On Sat, 11 Jan 2025 at 20:42, Sam Protsenko > > wrote: > > > > > > Free memory allocated for 'order' (array of bootmeths) on error paths in > > > bootmeth_setup_iter_order()

Re: [PATCH 01/32] Convert build-efi script to Python

2025-02-07 Thread Simon Glass
Hi Tom, On Thu, 6 Feb 2025 at 10:10, Tom Rini wrote: > > On Thu, Feb 06, 2025 at 06:05:36AM -0700, Simon Glass wrote: > > Hi Tom, > > > > On Mon, 3 Feb 2025 at 13:07, Tom Rini wrote: > > > > > > On Mon, Feb 03, 2025 at 10:41:54AM -0700, Simon Glass wrote: > > > > > > > Before this gets any longe

Re: [PATCH v2 2/2] board: remove capsule update support in set_dfu_alt_info()

2025-02-07 Thread Neil Armstrong
On 06/02/2025 16:47, Jonathan Humphreys wrote: Now that capsule update sets the dfu_alt_info environment variable explicitly, there is no need to support it in the set_dfu_alt_info() function. Decouple SET_DFU_ALT_INFO from EFI_CAPSULE_FIRMWARE_FIT and EFI_CAPSULE_FIRMWARE_RAW. For many boards, t

Re: [PATCH] tools: mkenvimage: Add pure shell implementation of mkenvimage

2025-02-07 Thread Rasmus Villemoes
On Wed, Feb 05 2025, Marek Vasut wrote: > Add implementation of mkenvimage written purely in bourne shell. > > This is not a replacement for mkenvimage tool, but rather a simple > implementation which can be used in environments where mkenvimage > itself cannot be deployed due to various constrai

[PATCH v2] board: nuvoton: use an event to replace last_stage_init()

2025-02-07 Thread Michael Chang
Add a new event which handles this function refer commit ("91caa3bb89b1 event: Use an event to replace last_stage_init()") Signed-off-by: Michael Chang --- board/nuvoton/arbel_evb/arbel_evb.c | 7 ++- board/nuvoton/common/uart.c | 7 --- board/nuvoton/common/uart.h | 2 +-

RE: [PATCH v2] board: starfive: Update the maintainer file for VisionFive 2 board

2025-02-07 Thread Hal Feng
> On 06.02.25 20:23, Simon Glass wrote: > Hi, > > On Wed, 5 Feb 2025 at 06:58, Tom Rini wrote: > > > > On Wed, Feb 05, 2025 at 10:56:52AM +0800, Hal Feng wrote: > > > > > Update the maintainer file and mark jh7110 related files with N: pattern. > > > > > > Signed-off-by: Hal Feng > > > --- > > >

Security vulnerabilities report to Das-U-Boot

2025-02-07 Thread Jonathan Bar Or
Dear U-boot maintainers, What is the best way of reporting security vulnerabilities (memory corruption issues) to Das-U-Boot? Is there a PGP key I should be using? I have 4 issues that I think are worth fixing (with very easy fixes). Best regards, Jonathan

[PATCH v3 2/6] efi_loader: Support for file renaming

2025-02-07 Thread Daniel Venzin
File renaming is necessary for features like systemd-boot bootcounting. Signed-off-by: Daniel Venzin --- (no changes since v2) Changes in v2: - Use EFI_ACCESS_DENIED for unsuccessful renames - Avoid using of "== 0" - Move the call to set_blk_dev earlier in the sequence lib/efi_loader/efi_fil

[PATCH v3 0/6] FAT file renaming

2025-02-07 Thread Daniel Venzin
This patch series represents a continuation of version 2 [1], incorporating tests and improvements. [1] https://lore.kernel.org/u-boot/20250203211719.GX1233568@bill-the-cat/ Changes in v3: - Abort the rename if the destination is a path instead of a file - Add fatrnfile command for renaming fil

[PATCH v3 4/6] fs: fat: Support basic file rename

2025-02-07 Thread Daniel Venzin
This implementation currently does not support moving files between directories. Signed-off-by: Daniel Venzin --- Changes in v3: - Abort the rename if the destination is a path instead of a file Changes in v2: - Separate the rename implementation from changes in the filesystem layer - Eliminat

[PATCH v3 1/6] fs: Add unimplemented rename hook to FS layer

2025-02-07 Thread Daniel Venzin
The EFI API supports file renaming, which is required by systemd-stub for its bootcounting implementation. Signed-off-by: Daniel Venzin --- (no changes since v2) Changes in v2: - Adapt variable names for better consistency - Add missing rename hooks fs/fs.c | 30

Re: [PATCH 0/7] board: samsung: e850-96: Enable Standard Boot

2025-02-07 Thread Minkyu Kang
Hi, On Fri, 17 Jan 2025 at 08:01, Sam Protsenko wrote: > Make all necessary configuration and environment changes to enable > Standard Boot on E850-96 board. Also rework LDFW firmware loading code > to look for LDFW binary on EFI System Partition instead of dedicated > 'ldfw' partition. > > Pend

[PATCH v3 3/6] fs: fat: Refactor delete_dentry_long interface

2025-02-07 Thread Daniel Venzin
Directory entries cannot be deleted without also removing their corresponding FAT entries. This refactor allows for individual selection of deleting the directory entry and the FAT entry, facilitating better reuse of existing functionality. Signed-off-by: Daniel Venzin --- (no changes since v2)

[PATCH v3 6/6] test/py: fs: Add fstest/rnfile test

2025-02-07 Thread Daniel Venzin
Extend the test_fs test suite with tests to verify operations related to FAT file renaming. Signed-off-by: Daniel Venzin --- Changes in v3: - Verify FAT file rename functionality with unit tests test/py/tests/test_fs/conftest.py| 75 ++ test/py/tests/test_fs/test_rnfil

[PATCH v3 5/6] cmd: fat: Add fatrnfile command

2025-02-07 Thread Daniel Venzin
The FAT rename file command supports renaming of a file, used by fs_test test suite. Signed-off-by: Daniel Venzin --- Changes in v3: - Add fatrnfile command for renaming files in a FAT file system cmd/fat.c| 13 + fs/fs.c | 20 include/fs.h | 2 ++ 3

Re: [PATCH 01/19] mbedtls: Add SHA symbols for VPL

2025-02-07 Thread Raymond Mao
Hi Tom, On Thu, 30 Jan 2025 at 19:09, Tom Rini wrote: > > On Thu, Jan 30, 2025 at 04:49:36PM -0500, Raymond Mao wrote: > > Hi Tom, > > > > On Thu, 30 Jan 2025 at 14:55, Tom Rini wrote: > > > > > > On Mon, Jan 27, 2025 at 10:23:12AM -0500, Raymond Mao wrote: > > > > Hi Simon, > > > > > > > > > >

[PATCH] board: kobol: helios4: enable ddr odt0 on write for both chip-select

2025-02-07 Thread Josua Mayer
/* clock enable mask */ }; struct mv_ddr_topology_map *mv_ddr_topology_map_get(void) --- base-commit: 6d41f0a39d6423c8e57e92ebbe9f8c0333a63f72 change-id: 20250207-a388-sr-som-odt-2-bd250cdd68bd Best regards, -- Josua Mayer

Re: [PATCH 28/29] Revert "bloblist: Load the bloblist from the previous loader"

2025-02-07 Thread Raymond Mao
Hi Simon, On Thu, 6 Feb 2025 at 19:09, Simon Glass wrote: > > Hi Raymond, > > On Thu, 6 Feb 2025 at 15:35, Raymond Mao wrote: > > > > Hi Simon, > > > > On Thu, 6 Feb 2025 at 10:41, Simon Glass wrote: > > > > > > Hi Raymond, > > > > > > On Wed, 5 Feb 2025 at 08:25, Raymond Mao wrote: > > > > >

Re: [PATCH v2] board: nuvoton: use an event to replace last_stage_init()

2025-02-07 Thread Tom Rini
On Fri, Feb 07, 2025 at 11:03:59AM +0800, Michael Chang wrote: > Add a new event which handles this function refer commit > ("91caa3bb89b1 event: Use an event to replace last_stage_init()") > > Signed-off-by: Michael Chang Reviewed-by: Tom Rini -- Tom signature.asc Description: PGP signatu

Re: Security vulnerabilities report to Das-U-Boot

2025-02-07 Thread Tom Rini
On Thu, Feb 06, 2025 at 07:47:54PM -0800, Jonathan Bar Or wrote: > Dear U-boot maintainers, > > What is the best way of reporting security vulnerabilities (memory > corruption issues) to Das-U-Boot? Is there a PGP key I should be using? > I have 4 issues that I think are worth fixing (with very e

Re: [GIT PULL] amd/xilinx patches for v2025.04-rc2

2025-02-07 Thread Tom Rini
On Fri, Feb 07, 2025 at 07:27:43AM +0100, Michal Simek wrote: > > > On 2/7/25 00:33, Tom Rini wrote: > > On Thu, Feb 06, 2025 at 08:57:41AM +0100, Michal Simek wrote: > > > > > Hi Tom, > > > > > > please pull these patches to your tree. Nothing really significant there. > > > Some binman fixes

Re: [PATCH 1/4] board: phytec: common: k3: Introduce Configs to Sign Images

2025-02-07 Thread Andrew Davis
On 2/7/25 12:51 AM, Daniel Schultz wrote: Private keys to sign bootloader images shouldn't be commit or part of this repository. Add config entries to use keys located outside of U-Boot to sign images. The custMpk.pem was always a placeholder, we fill it with the TI Dummy key as an example. Th