[PATCH 13/24] kbuild: refactor modversions build rules

Backport from kernel commit ee3e46b7efd2 ("kbuild: refactor modversions build rules") Signed-off-by: Ilias Apalodimas --- scripts/Makefile.build | 22 +- 1 file changed, 9 insertions(+), 13 deletions(-) diff --git a/scripts/Makefile.build b/scripts/Makefile.build index b2bf5

[PATCH 12/24] kbuild: remove redundant 'set -e' from sub_cmd_record_mcount

Back from kernel commit 4317ee3b6a5e ("kbuild: remove redundant 'set -e' from sub_cmd_record_mcount") Signed-off-by: Ilias Apalodimas --- scripts/Makefile.build | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/Makefile.build b/scripts/Makefile.build index 2f2d5d4afc25

[PATCH 15/24] kbuild: remove trailing semicolon from cmd_* passed to if_changed_rule

Backport from kernel commit e5d289100d3a ("kbuild: remove trailing semicolon from cmd_* passed to if_changed_rule") Signed-off-by: Ilias Apalodimas --- scripts/Makefile.build | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/scripts/Makefile.build b/scripts/Makefile.buil

[PATCH 06/24] kbuild: skip 'addtree' and 'flags' magic for external module build

Backport from kernel commit 58156ba4468f ("kbuild: skip 'addtree' and 'flags' magic for external module build") Signed-off-by: Ilias Apalodimas --- scripts/Makefile.host | 6 -- scripts/Makefile.lib | 10 +- 2 files changed, 9 insertions(+), 7 deletions(-) diff --git a/scripts/Ma

[PATCH 24/24] kbuild: add -Werror=strict-prototypes flag unconditionally

Backported from kernel commit eeb5687a7139649e ("kbuild: add -Werror=strict-prototypes flag unconditionally") Signed-off-by: Ilias Apalodimas --- Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Makefile b/Makefile index 66803f089d48..2890edbcc43f 100644 --- a/Makefi

[PATCH 23/24] kbuild: make -r/-R effective in top Makefile for old Make versions

Backported from kernel commit 3812b8c5c5d5 ("kbuild: make -r/-R effective in top Makefile for old Make versions") Signed-off-by: Ilias Apalodimas --- Makefile | 48 ++-- 1 file changed, 26 insertions(+), 22 deletions(-) diff --git a/Makefile b/Makefi

[PATCH 22/24] kbuild: remove unused archmrproper

Backported from kernel commit b421b8a6cb87 ("kbuild: remove unused archmrproper") Signed-off-by: Ilias Apalodimas --- Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Makefile b/Makefile index 09d36b8cbbed..00876ef54a2a 100644 --- a/Makefile +++ b/Makefile @@ -2281,7

[PATCH 21/24] kbuild: strip whitespace in cmd_record_mcount findstring

Backport from kernel commit 1a49b2fd8f58 ("kbuild: strip whitespace in cmd_record_mcount findstring") Signed-off-by: Ilias Apalodimas --- scripts/Makefile.build | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/scripts/Makefile.build b/scripts/Makefile.build index b7611c

[PATCH 20/24] kbuild: hardcode genksyms path and remove GENKSYMS variable

Back port from kernel although it's not used since it makes diffing easier. commit 88110713ca9dfb ("kbuild: hardcode genksyms path and remove GENKSYMS variable") Signed-off-by: Ilias Apalodimas --- scripts/Makefile.build | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/script

[PATCH 17/24] kbuild: remove redundant quiet_modtag for $(obj-m)

Backported from kernel commit b39a691617e4 commit ("kbuild: remove redundant quiet_modtag for $(obj-m)") Signed-off-by: Ilias Apalodimas --- scripts/Makefile.build | 2 -- 1 file changed, 2 deletions(-) diff --git a/scripts/Makefile.build b/scripts/Makefile.build index fa91445f132b..2944629804

[PATCH 19/24] kbuild: generate modules.order only when CONFIG_MODULES=y

Backport from kernel although it is unused since it makes diffing easier. commit 1d8001ef358 ("kbuild: generate modules.order only when CONFIG_MODULES=y") Signed-off-by: Ilias Apalodimas --- scripts/Makefile.build | 2 ++ 1 file changed, 2 insertions(+) diff --git a/scripts/Makefile.build b/scr

[PATCH 18/24] kbuild: refactor quiet_modtag

Backport from kernel commit 45c4372d00 ("kbuild: refactor quiet_modtag") Signed-off-by: Ilias Apalodimas --- scripts/Makefile.build | 7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/scripts/Makefile.build b/scripts/Makefile.build index 29446298041f..ed58bcab291e 100644 --

[PATCH 16/24] kbuild: remove redundant 'set -e' from cmd_* defines

Backported from kernel commit 5439f09f488f ("kbuild: remove redundant 'set -e' from cmd_* defines") Signed-off-by: Ilias Apalodimas --- scripts/Makefile.build | 1 - 1 file changed, 1 deletion(-) diff --git a/scripts/Makefile.build b/scripts/Makefile.build index 80caddde09ab..fa91445f132b 10064

[PATCH 14/24] kbuild: change if_changed_rule for multi-line recipe

Backport from kernel commit 3a2429e1faf4 ("kbuild: change if_changed_rule for multi-line recipe") Signed-off-by: Ilias Apalodimas --- scripts/Kbuild.include | 12 scripts/Makefile.build | 12 +++- 2 files changed, 11 insertions(+), 13 deletions(-) diff --git a/scripts/Kbuil

[PATCH 11/24] kbuild: let fixdep directly write to .*.cmd files

Backport from kernel commit 392885ee82d3 ("kbuild: let fixdep directly write to .*.cmd files") Signed-off-by: Ilias Apalodimas --- Makefile | 13 + scripts/Kbuild.include | 5 ++--- scripts/Makefile.build | 11 +++ 3 files changed, 14 insertions(+), 15 deletion

[PATCH 10/24] kbuild: move archive command to scripts/Makefile.lib

Backport from kernel commit 898f5a009f22 ("kbuild: move archive command to scripts/Makefile.lib") Signed-off-by: Ilias Apalodimas --- scripts/Makefile.build | 4 +--- scripts/Makefile.lib | 5 + 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/scripts/Makefile.build b/scripts

[PATCH 08/24] kbuild: add real-prereqs shorthand for $(filter-out FORCE, $^)

Backport from kernel commit afa974b77128 ("kbuild: add real-prereqs shorthand for $(filter-out FORCE,$^)") Signed-off-by: Ilias Apalodimas --- scripts/Kbuild.include | 4 scripts/Makefile.build | 2 +- scripts/Makefile.lib | 10 +- 3 files changed, 10 insertions(+), 6 deletions

[PATCH 09/24] kbuild: remove unnecessary in-subshell execution

Backport kernel changes commit b79c6aa6a1f1 ("kbuild: remove unnecessary in-subshell execution") Signed-off-by: Ilias Apalodimas --- scripts/Makefile.lib | 21 ++--- 1 file changed, 10 insertions(+), 11 deletions(-) diff --git a/scripts/Makefile.lib b/scripts/Makefile.lib index

[PATCH 07/24] kbuild: simplify rules of data compression with size appending

Backport kernel changes commit ecbd10d90e94 ("kbuild: simplify rules of data compression with size appending") Signed-off-by: Ilias Apalodimas --- scripts/Makefile.lib | 18 +++--- 1 file changed, 7 insertions(+), 11 deletions(-) diff --git a/scripts/Makefile.lib b/scripts/Makefile

[PATCH 04/24] kbuild: remove redundant 'set -e' from filechk_offsets

Backport changes from the kernel commit f3fd4a3f3a38 ("kbuild: remove redundant 'set -e' from filechk_offsets") Signed-off-by: Ilias Apalodimas --- scripts/Makefile.lib | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/Makefile.lib b/scripts/Makefile.lib index c3c6c941

[PATCH 05/24] kbuild: remove redundant target cleaning on failure

Bacport from the kernel commit 172caf1993b7 ("kbuild: remove redundant target cleaning on failure") Signed-off-by: Ilias Apalodimas --- scripts/Makefile.lib | 21 +++-- 1 file changed, 7 insertions(+), 14 deletions(-) diff --git a/scripts/Makefile.lib b/scripts/Makefile.lib ind

[PATCH 02/24] kbuild: disable dtc simple_bus_reg warnings by default

Backport changes from the kernel commit 70523a3ce5ff ("kbuild: disable dtc simple_bus_reg warnings by default") Signed-off-by: Ilias Apalodimas --- scripts/Makefile.lib | 1 + 1 file changed, 1 insertion(+) diff --git a/scripts/Makefile.lib b/scripts/Makefile.lib index 83fd5ff6c31c..2422ac980a4

[PATCH 03/24] kbuild: Enable dtc graph_port warning by default

Backport changes from the kernel commit a2237fec1e06 ("kbuild: Enable dtc graph_port warning by default") Signed-off-by: Ilias Apalodimas --- scripts/Makefile.lib | 1 - 1 file changed, 1 deletion(-) diff --git a/scripts/Makefile.lib b/scripts/Makefile.lib index 2422ac980a46..c3c6c941bef4 10064

[PATCH 01/24] kbuild: de-duplicate fixdep usage

Backported from the linux kernel commit e4aca4595005 ("kbuild: de-duplicate fixdep usage") A lot of the kernels kbuild depend on that patch. Backport it and pull in the 'rule_as_o_S'a rule as well. This might end up being unused but it doesn't break anything and makes diffing the files easier. Si

[PATCH 00/24] Update kbuild

Last time the kbuild scripts were synced with the Linux kernel was on the 4.20 release. Updating directly to 6.x makes the diff difficult to read, so let's do the changes incrementally and bump to 5.1 first. Since the number of comits is bug, I am splitting this in multiple series. This one includ

Re: [PATCH] bootm: Fix bootz, bmi->images pointer was uninitialized

On Mon, May 19, 2025 at 09:48:00PM +0200, Benjamin ROBIN wrote: > When building with only bootz command, without bootm, images pointer > inside bootm_info structure is not initialized. And since this structure > is stored in stack the generated error is kind of random, but most of > the time this

[PATCH] bootm: Fix bootz, bmi->images pointer was uninitialized

When building with only bootz command, without bootm, images pointer inside bootm_info structure is not initialized. And since this structure is stored in stack the generated error is kind of random, but most of the time this will generate: "ramdisk - allocation error" Signed-off-by: Benjamin ROBI

Re: [PATCH] lib: add NULL pointer check for %s format

Hi Tom, Thank you for your feedback. You're absolutely right — the subject and commit message were too generic and wordy. I've revised the patch accordingly and sent a v2 with the subject: [PATCH v2] tiny-printf: handle NULL pointer for %s format string The v2 includes: - Updated subject pr

Re: [PATCH] lib: add NULL pointer check for %s format

On Fri, May 16, 2025 at 09:54:47PM +0300, ant.v.morya...@gmail.com wrote: > From: Anton Moryakov > > Handle NULL pointer case in string formatting (%s) by printing '(null)' > instead of dereferencing NULL pointer. Makes behavior consistent with > standard printf implementations and prevents pote

Re: [PATCH v4 1/5] efi: stub: Define debug uart functions only if CONFIG_DEBUG_UART is enabled

On Mon, May 19, 2025 at 05:49:12PM +0200, Quentin Schulz wrote: > On 5/19/25 5:43 PM, Tom Rini wrote: > > On Mon, May 19, 2025 at 05:09:07PM +0200, Quentin Schulz wrote: > > > Hi Tom, > > > > > > On 5/19/25 4:48 PM, Tom Rini wrote: > > > > On Thu, May 15, 2025 at 05:55:44PM +0200, Lukasz Czechowsk

Re: [PATCH v3 1/2] lib/uuid.c: use unique name for PARTITION_SYSTEM_GUID

Hi Jerome, Sorry for my late review of your patch, but I see a side effect exist for this patch the modified string in list_guid[] is used in uuid_guid_get_bin() called by uuid_str_to_bin() with CONFIG_PARTITION_TYPE_GUID activated to support shortname for knwon GUID partition in gpt command

Re: [PATCH v3] stm32mp: Add tamp_nvram driver

On 5/5/25 14:38, Patrick DELAUNAY wrote: > Hi, > > minor remark to be coherent between SoC family > > STM32MP1 <=> STM32MP2 for some part of the driver code > > but STM32MP1 <=> STM32MP25 for other part of the code > > and stm32mp15 <=> stm32mp25 for compatible > > > On 4/29/25 08:53, Patr

Re: [PATCH v4 1/5] efi: stub: Define debug uart functions only if CONFIG_DEBUG_UART is enabled

On 5/19/25 5:43 PM, Tom Rini wrote: On Mon, May 19, 2025 at 05:09:07PM +0200, Quentin Schulz wrote: Hi Tom, On 5/19/25 4:48 PM, Tom Rini wrote: On Thu, May 15, 2025 at 05:55:44PM +0200, Lukasz Czechowski wrote: Initialize the debug uart only in case CONFIG_DEBUG_UART is enabled. The _debug_ua

Re: [PATCH v4 1/5] efi: stub: Define debug uart functions only if CONFIG_DEBUG_UART is enabled

On Mon, May 19, 2025 at 05:09:07PM +0200, Quentin Schulz wrote: > Hi Tom, > > On 5/19/25 4:48 PM, Tom Rini wrote: > > On Thu, May 15, 2025 at 05:55:44PM +0200, Lukasz Czechowski wrote: > > > Initialize the debug uart only in case CONFIG_DEBUG_UART is > > > enabled. The _debug_uart_putc is used int

Re: [PATCH v4 1/5] efi: stub: Define debug uart functions only if CONFIG_DEBUG_UART is enabled

Hi Tom, On 5/19/25 4:48 PM, Tom Rini wrote: On Thu, May 15, 2025 at 05:55:44PM +0200, Lukasz Czechowski wrote: Initialize the debug uart only in case CONFIG_DEBUG_UART is enabled. The _debug_uart_putc is used internally by debug uart functions, so it must be also included inside #ifdef block, o

Re: [PATCH] tools: fix handle leak in ifdtool.c

Maybe. However, the static analyser did not swear at others Пн, 19 мая 2025 г. в 11:47, Quentin Schulz : > Hi Anton, > > On 5/16/25 5:25 PM, ant.v.morya...@gmail.com wrote: > > From: Anton Moryakov > > > > Prevent file descriptor leaks by properly closing 'fd' and 'new_fd' > > when fstat() or w

Re: [PATCH] [RFC]: dtc: Add Kconfig option to pad device tree blob

On Mon, May 19, 2025 at 02:58:38PM +, Eric Schikschneit wrote: > Hello Tom, I have made the changes as discussed and rebased on master. You > can find the latest version of my commit here: > > https://github.com/emz229/u-boot/commit/739fe94e1f8096a90241f5a9348f3dcce4eb97c1 > > With your app

Re: [PATCH] [RFC]: dtc: Add Kconfig option to pad device tree blob

Hello Tom, I have made the changes as discussed and rebased on master. You can find the latest version of my commit here: https://github.com/emz229/u-boot/commit/739fe94e1f8096a90241f5a9348f3dcce4eb97c1 With your approval I can issue a PR. Eric Schikschneit Senior Embedded Linux Engineer III

Re: [PATCH] env: Fix network support when CONFIG_NET_LWIP is set

On Sat, May 17, 2025 at 10:09:30PM +0200, Benjamin ROBIN wrote: > When lwIP (CONFIG_NET_LWIP) is used instead of legacy stack (CONFIG_NET), > environment flags support associated with network was not built: restore > support of "i" and "m" environment flags. > > Signed-off-by: Benjamin ROBIN Re

Re: [PATCH v4 1/5] efi: stub: Define debug uart functions only if CONFIG_DEBUG_UART is enabled

On Thu, May 15, 2025 at 05:55:44PM +0200, Lukasz Czechowski wrote: > Initialize the debug uart only in case CONFIG_DEBUG_UART is > enabled. The _debug_uart_putc is used internally by debug uart > functions, so it must be also included inside #ifdef block, > otherwise it will cause compilation warni

Re: [PATCH v4 2/5] debug_uart: Replace debug functions with dummies if CONFIG_DEBUG_UART is not set

On Thu, May 15, 2025 at 05:55:45PM +0200, Lukasz Czechowski wrote: > In case DEBUG UART is not used, define dummy macros replacing > the actual function implementations that will not be available. > This allows to compile code and avoid linker errors. > Redefine the DEBUG_UART_FUNCS macro if DEBUG

Re: [PATCH 1/4] usb: onboard-hub: Use the ofnode to check if the peer-hub was probed

Hi Lukasz, On Fri, 25 Apr 2025 at 16:32, Lukasz Czechowski wrote: > > Currently the check in usb_onboard_hub_bind is relying on specific > compatible string for the Michrochip USB5744. Replace this with > more generic approach that will allow to add new types of devices > to the of_match table. B

Re: [PATCH] cmd: Flush destination buffer only if it is available

On Mon, May 19, 2025 at 12:19:34PM +0530, Varadarajan Narayanan wrote: > From: Vandhiadevan Karunamoorthy > > There are code paths in do_imgextract(), where 'dest' could be zero. > Hence, avoid cache flush of 'dest' doesn't point to any data buffer. > > Signed-off-by: Vandhiadevan Karunamoorthy

[PATCH v1] toradex: tdx-cfg-block: add new pid4 support

From: Vitor Soares Add the new PID4 to the ConfigBlock handling: - 0098 Aquila iMX95 Hexa 16GB WB IT - 0099 Verdin AM62P Quad 2GB WB IT - 0201 SMARC iMX95 Hexa 8GB IT - 0202 SMARC iMX95 Hexa 4GB WB IT - 0203 SMARC iMX95 Hexa 4GB ET - 0204 SMARC iMX95 Hexa 2GB WB IT - 0205 SMARC iMX95 Hexa

Re: [PATCH 1/4] usb: onboard-hub: Use the ofnode to check if the peer-hub was probed

Hi Quentin, On Mon, 19 May 2025 at 18:45, Quentin Schulz wrote: > > Hi Anand, > > On 5/19/25 2:42 PM, Anand Moon wrote: > > Hi Lukasz, > > > > On Fri, 25 Apr 2025 at 16:32, Lukasz Czechowski > > wrote: > >> > >> Currently the check in usb_onboard_hub_bind is relying on specific > >> compatible s

[PATCH] board: rzg2l: Drop myself to reviewer for RZ/G2L boards

I still intend to review patches, and will be using my own email address going forward. Signed-off-by: Paul Barker --- board/renesas/rzg2l/MAINTAINERS | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/board/renesas/rzg2l/MAINTAINERS b/board/renesas/rzg2l/MAINTAINERS index 0

Re: [PATCH 1/4] usb: onboard-hub: Use the ofnode to check if the peer-hub was probed

Hi Anand, On 5/19/25 2:42 PM, Anand Moon wrote: Hi Lukasz, On Fri, 25 Apr 2025 at 16:32, Lukasz Czechowski wrote: Currently the check in usb_onboard_hub_bind is relying on specific compatible string for the Michrochip USB5744. Replace this with more generic approach that will allow to add ne

Re: [PATCH v4 2/5] debug_uart: Replace debug functions with dummies if CONFIG_DEBUG_UART is not set

Hi Quentin, pt., 16 maj 2025 o 17:27 Quentin Schulz napisał(a): > > Hi Lukasz, > > On 5/15/25 5:55 PM, Lukasz Czechowski wrote: > > In case DEBUG UART is not used, define dummy macros replacing > > the actual function implementations that will not be available. > > This allows to compile code and

Re: [PATCH v4 1/1] Initial support for Wiznet W5500

>> Typically what happens in open-source projects is that once the code is >> merged, the contributor disappears :) So there's usually reticence to >> merging code if there's some on-going discussion. Here, the struct for >> cmd change suggestion is not a blocker as it's mostly cosmetic anyway >> a

Re: [PATCH v4 2/5] debug_uart: Replace debug functions with dummies if CONFIG_DEBUG_UART is not set

pon., 19 maj 2025 o 12:40 Quentin Schulz napisał(a): > > Hi Lukasz, > > On 5/19/25 12:32 PM, Łukasz Czechowski wrote: > > Hi Quentin, > > > > pt., 16 maj 2025 o 17:27 Quentin Schulz > > napisał(a): > >> > >> Hi Lukasz, > >> > >> On 5/15/25 5:55 PM, Lukasz Czechowski wrote: > >>> In case DEBUG UA

Re: [PATCH v4 1/1] Initial support for Wiznet W5500

Hi Jean-Marie, On 5/19/25 11:05 AM, Verdun, Jean-Marie wrote: Fair enough. You could have the same ternary operator to better match what's in the kernel. Can you also reuse the macros that are in the kernel already? W5500_SPI_BLOCK_SELECT, W5500_SPI_READ_CONTROL and W5500_SPI_WRITE_CONTROL se

Re: [PATCH 1/4] usb: onboard-hub: Use the ofnode to check if the peer-hub was probed

Hi Quentin, pt., 16 maj 2025 o 16:28 Quentin Schulz napisał(a): > > Hi Lukasz, > > On 4/25/25 12:56 PM, Lukasz Czechowski wrote: > > Currently the check in usb_onboard_hub_bind is relying on specific > > compatible string for the Michrochip USB5744. Replace this with > > more generic approach tha

Re: [PATCH 3/4] usb: onboard-hub: Add support for multiple power supplies

Hi Quentin, pt., 16 maj 2025 o 17:05 Quentin Schulz napisał(a): > > Hi Lukasz, > > On 4/25/25 12:56 PM, Lukasz Czechowski wrote: > > Some of the onboard hubs require multiple power supplies, so extend > > the driver to support them. > > The implementation is inspired by the kernel driver, as intr

[PATCH v4] stm32mp: Add tamp_nvram driver

From: Simeon Marijon TAMP backup registers will be exposed as nvmem cells. Each registers ([0..127] for STM32MP2, [0..31] for STM32MP1) could be exposed as nvmem cells under the nvram node in device tree Signed-off-by: Simeon Marijon Signed-off-by: Patrice Chotard Reviewed-by: Patrick Delauna

Re: [PATCH v4 1/1] Initial support for Wiznet W5500

Hi Jean-Marie, On 5/15/25 4:01 PM, Verdun, Jean-Marie wrote: This is dangerous too. How do I know if the returned data is an error code or actual data? Right now, xfer returns 0 if successful, -1 otherwise. I would suggest return ret only if it is < 0. Or even better, propagate the erro

Re: [RFC PATCH 1/2 v2] tools: binman: control.py: Propagate bootph-* properties to supernodes

Hi Moteen, On 5/19/25 11:33 AM, Moteen Shah wrote: Hey Simon and Quentin, Should I send one RFC for this for folks not using binman? If you already have code for it, sure. Otherwise, I guess whoever encounter a similar issues as yours for their boards can fix that themselves if you don't f

Re: [PATCH v4 2/5] debug_uart: Replace debug functions with dummies if CONFIG_DEBUG_UART is not set

Hi Lukasz, On 5/19/25 12:32 PM, Łukasz Czechowski wrote: Hi Quentin, pt., 16 maj 2025 o 17:27 Quentin Schulz napisał(a): Hi Lukasz, On 5/15/25 5:55 PM, Lukasz Czechowski wrote: In case DEBUG UART is not used, define dummy macros replacing the actual function implementations that will not b

Re: [PATCH] mkimage: do a rough estimate for the size needed for hashes/signatures

On Fri, May 16 2025, Simon Glass wrote: > Hi Rasmus, > > On Fri, 16 May 2025 at 14:54, Rasmus Villemoes wrote: >> >> >> While not perfect, we can give a reasonable estimate of an upper bound >> on the necessary extra size by simply counting the number of hash and >> signature nodes in the FIT im

Re: [PATCH v2] fdt: Allow expanding the devicetree during relocation

Hi Simon, On 5/16/25 4:12 PM, Simon Glass wrote: Some boards set fdt_high to -1 which means that the FDT is not relocated in boot_relocate_fdt(). A comment in that function says that we assume there is space after the existing fdt to use for padding, with the padding size set to CONFIG_SYS_FDT_

Re: [PATCH 1/3] lib/uuid.c: restore support of system partition type for ESP

Hi, On 5/19/25 11:23, Jerome Forissier wrote: Hi Patrick, On 5/19/25 10:04, Patrick Delaunay wrote: Add support of shortname for type parameter and restore "system" as short name for EFI System Partition (ESP) for filed "type" of the "gpt write" command. Fixes: d54e1004b8b1 ("lib/uuid.c: use

[PATCH 0/3] Restore support of short name for type UUID parameter

Fix and add documentation/tests for selection by string for known partition type GUID introduced by bcb41dcaefac ("uuid: add selection by string for known partition type GUID"): - split list_guid for short name (used also for partiton description with type parameter) and full name to display

[PATCH 3/3] test/py: tests: gpt: add test_gpt_write_part_type

Add sandbox test on gpt command with partition type for known type. Signed-off-by: Patrick Delaunay --- test/py/tests/test_gpt.py | 27 +++ 1 file changed, 27 insertions(+) diff --git a/test/py/tests/test_gpt.py b/test/py/tests/test_gpt.py index cfc8f1319a9b..e6d8792ac1

Re: [RFC PATCH 1/2 v2] tools: binman: control.py: Propagate bootph-* properties to supernodes

Hey Simon and Quentin, Should I send one RFC for this for folks not using binman? Regards, Moteen On 01/04/25 21:12, Simon Glass wrote: Hi Manorit, On Tue, 1 Apr 2025 at 18:04, Manorit Chawdhry wrote: Hi Quentin, On 15:46-20250328, Quentin Schulz wrote: Hi Moteen, On 3/27/25 9:06 AM, Mot

Re: [PATCH 1/3] lib/uuid.c: restore support of system partition type for ESP

Hi Patrick, On 5/19/25 10:04, Patrick Delaunay wrote: > Add support of shortname for type parameter and restore "system" > as short name for EFI System Partition (ESP) for filed "type" of the > "gpt write" command. > > Fixes: d54e1004b8b1 ("lib/uuid.c: use unique name for PARTITION_SYSTEM_GUID")

[PATCH 1/3] lib/uuid.c: restore support of system partition type for ESP

Add support of shortname for type parameter and restore "system" as short name for EFI System Partition (ESP) for filed "type" of the "gpt write" command. Fixes: d54e1004b8b1 ("lib/uuid.c: use unique name for PARTITION_SYSTEM_GUID") Signed-off-by: Patrick Delaunay --- lib/uuid.c | 148 +

[PATCH 2/3] doc: cmd: gpt: add information on type partition

Add information on type partition, copied from README.gpt. I also correct issue for gpt_parts variable and add example of "gpt read" usage. Signed-off-by: Patrick Delaunay --- doc/usage/cmd/gpt.rst | 93 ++- 1 file changed, 84 insertions(+), 9 deletions(

Re: [PATCH v4 1/1] Initial support for Wiznet W5500

>> Fair enough. You could have the same ternary operator to better match >> what's in the kernel. >> Can you also reuse the macros that are in the kernel already? >> W5500_SPI_BLOCK_SELECT, W5500_SPI_READ_CONTROL and >> W5500_SPI_WRITE_CONTROL seems to be of interest? I will check it >> If you d

Re: U-Boot support for wolfTPM and firmware update for SLB9672/SLB9673

On Fri, 9 May 2025 at 18:43, David Garske wrote: > > Hi Ilias, > > Thank you for the quick reply. I am happy that you will consider wolfTPM as a > submodule. We have 100’s of commercial customers and it is very actively > maintained. In fact we will continue to provide direct maintenance for any

Re: [PATCH] env: Fix network support when CONFIG_NET_LWIP is set

On 5/17/25 21:09, Benjamin ROBIN wrote: > When lwIP (CONFIG_NET_LWIP) is used instead of legacy stack (CONFIG_NET), > environment flags support associated with network was not built: restore > support of "i" and "m" environment flags. > > Signed-off-by: Benjamin ROBIN > --- > env/flags.c

Re: [PATCH] tools: fix handle leak in ifdtool.c

Hi Anton, On 5/16/25 5:25 PM, ant.v.morya...@gmail.com wrote: From: Anton Moryakov Prevent file descriptor leaks by properly closing 'fd' and 'new_fd' when fstat() or write() operations fail. - Added close(fd) before return in open_for_read() if fstat() fails. - Added close(new_fd) before ret

Re: [PATCH v3 21/22] global: Avoid indirect inclusion of from

On Fri, 16 May 2025 at 02:42, Tom Rini wrote: > > Now that env_get_ip() has been removed, the include file does > not need anything from . Furthermore, include/env.h itself > includes other headers which can lead to longer indirect inclusion > paths. To prepare to remove from fix all of the > r

Re: [PATCH v3 21/22] global: Avoid indirect inclusion of from

> From: Tom Rini > Sent: Friday, May 16, 2025 1:31 > Subject: [PATCH v3 21/22] global: Avoid indirect inclusion of from > > > Now that env_get_ip() has been removed, the include file does > not need anything from . Furthermore, include/env.h itself > includes other headers which can lead to lo

Re: [PATCH v4 2/2] cmd/Kconfig: select CMD_WGET if ARCH_QEMU

On Fri, 16 May 2025 at 11:52, Adriano Cordova wrote: > > select CMD_WGET for ARCH_QEMU. This way HTTP can be tested > on the qemu platforms in CI. > > Signed-off-by: Adriano Cordova > Reviewed-by: Tom Rini > --- Reviewed-by: Ilias Apalodimas > > (no changes since v1) > > cmd/Kconfig | 2 +- >

Re: [PATCH v4 1/2] CI testing: add http server to CI tests

On Fri, 16 May 2025 at 11:52, Adriano Cordova wrote: > > Add an http server to CI tests so that HTTP booting and > loading can be tested. > > Signed-off-by: Adriano Cordova > Reviewed-by: Tom Rini > --- Reviewed-by: Ilias Apalodimas > > changes in v2: > - Remove HTTP_SERVER argument for te

Re: [PATCH v3 1/2] CI testing: add http server to CI tests

Thanks for the review, but please use the v4 of this series. On Mon, 19 May 2025 at 03:06, Ilias Apalodimas wrote: > On Wed, 14 May 2025 at 19:17, Adriano Cordova wrote: > > > > Add an http server to CI tests so that HTTP booting and > > loading can be tested. > > > > Signed-off-by: Adriano Cor

Re: [PATCH] common/board_f: init malloc earlier

On Wed, Nov 27, 2024 at 06:08:25AM -0700, Simon Glass wrote: > Hi Caleb, > > On Tue, 26 Nov 2024 at 09:15, Caleb Connolly > wrote: > > > > Hi Simon, > > > > On 26/11/2024 16:38, Simon Glass wrote: > > > Hi Caleb, > > > > > > On Tue, 26 Nov 2024 at 05:22, Caleb Connolly > > > wrote: > > >> > > >

Re: [PATCH v3 2/2] cmd/Kconfig: select CMD_WGET if ARCH_QEMU

On Wed, 14 May 2025 at 19:17, Adriano Cordova wrote: > > select CMD_WGET for ARCH_QEMU. This way HTTP can be tested > on the qemu platforms in CI. > > Signed-off-by: Adriano Cordova > --- > > (no changes since v1) > > cmd/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff -

Re: [PATCH v3 1/2] CI testing: add http server to CI tests

On Wed, 14 May 2025 at 19:17, Adriano Cordova wrote: > > Add an http server to CI tests so that HTTP booting and > loading can be tested. > > Signed-off-by: Adriano Cordova > --- > > changes in v2: > - Remove HTTP_SERVER argument for test.py > > .azure-pipelines.yml | 11 +++ > .gitlab