Re: [PATCH 0/2] Print Reset Reason for K3 based devices

2025-05-15 Thread Neha Malcom Francis
Hi Wadim On 15/05/25 16:45, Wadim Egorov wrote: > Although AM62x, AM62Ax, AM64x, and AM62Px share the same register for > reset reason, not all TRMs document this consistently. To avoid relying > on undocumented behavior, introduce SoC-specific implementations for > now. If TI confirms all SoCs sh

Re: [PATCH v3 16/22] liebherr: Include env.h to permit reading the environment

2025-05-15 Thread Lukasz Majewski
On Thu, 15 May 2025 17:31:45 -0600 Tom Rini wrote: > From: Simon Glass > > This file reads from the environment but does not include the correct > header. Update it. > > Signed-off-by: Simon Glass > Reviewed-by: Lukasz Majewski > --- > Changes in v3: > - None > --- > board/liebherr/xea/xea.

[PATCH 3/4] riscv: cpu: th1520: Select clock driver

2025-05-15 Thread Yao Zi
The clock driver is essential for TH1520 SoCs to operate. Select the driver in SoC Kconfig entry. Signed-off-by: Yao Zi --- arch/riscv/cpu/th1520/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/riscv/cpu/th1520/Kconfig b/arch/riscv/cpu/th1520/Kconfig index a916d364e6c..4d44191bd

[PATCH 4/4] riscv: dts: th1520: Complete clock tree

2025-05-15 Thread Yao Zi
Describe the newly-supported clock controller of TH1520 in SoC devicetree, replace dummy clocks with the controller-supplied ones and add correct clocks for GPIO controllers. Signed-off-by: Yao Zi --- arch/riscv/dts/th1520-lichee-module-4a.dtsi | 8 --- arch/riscv/dts/th1520.dtsi

[PATCH 2/4] clk: thead: Port clock controller driver of TH1520 SoC

2025-05-15 Thread Yao Zi
The driver is adapted from Linux kernel's version of clk-th1520-ap.c, with only output clocks for external sensors, which are barely useful in bootloaders, removed. Same as the mainline driver, it currently lacks of ability to enable and reconfigure PLLs, which could be implemented later. Signed-

[PATCH 1/4] riscv: cpu: th1520: Initialize IOPMPs in SPL

2025-05-15 Thread Yao Zi
TH1520 SoC ships several IOPMPs protecting various on-chip peripherals. They must be configured before accessing the peripherals. Let's initialize them in SPL harts_early_init(). Signed-off-by: Yao Zi --- arch/riscv/cpu/th1520/spl.c| 65 ++ arch/riscv/include/

[PATCH 0/4] Improve usability of TH1520 with mainline SPL

2025-05-15 Thread Yao Zi
Depending on the previous SPL support series of TH1520[1], this series further improves usability of TH1520 SoC when booting with mainline SPL. The T-Head customized IOPMPs are initialized in SPL and a clock driver is ported from Linux kernel to support operation of various peripherals. With this

RE: [PATCH v3 2/2] arm: imx: imx8m: soc: replace ifdef by IS_ENABLED()

2025-05-15 Thread Peng Fan
> Subject: [PATCH v3 2/2] arm: imx: imx8m: soc: replace ifdef by > IS_ENABLED() > > Standardize on using the IS_ENABLED macro. > > Signed-off-by: Dario Binacchi > Reviewed-by: Peng Fan

RE: [PATCH v3 1/2] arm: imx: imx8m: soc: fix the macro name

2025-05-15 Thread Peng Fan
> Subject: [PATCH v3 1/2] arm: imx: imx8m: soc: fix the macro name > > From: Michael Trimarchi > > The function arch_spl_mmc_get_uboot_raw_sector() was never > compiled, even when the option > CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_USE_PARTITION was > enabled. So rename the macro > SYS_MMCSD_RAW_MODE_

[PATCH v3 13/22] amlogic: Include env.h to permit reading the environment

2025-05-15 Thread Tom Rini
From: Simon Glass This file reads from the environment but does not include the correct header. Update it. Reviewed-by: Viacheslav Bocharov Signed-off-by: Simon Glass --- Changes in v3: - None --- board/amlogic/jethub-j100/jethub-j100.c | 1 + 1 file changed, 1 insertion(+) diff --git a/boar

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

2025-05-15 Thread Tom Rini
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 remaining places which had relied on this indirect inclusion

[PATCH v3 22/22] include/net.h: Drop

2025-05-15 Thread Tom Rini
Now that all of the cases of code that relied on to provide (or one of the headers that it includes, and so forth) have been fixed, we can drop the include from here. Signed-off-by: Tom Rini --- Changes in v3: - New patch --- include/net-common.h | 1 - 1 file changed, 1 deletion(-) diff --gi

[PATCH v3 18/22] net: Include byteorder in net6.h

2025-05-15 Thread Tom Rini
From: Simon Glass This file uses byteorder but does not include the header file. Update it. Signed-off-by: Simon Glass --- Changes in v3: - None --- include/net6.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/net6.h b/include/net6.h index 2ceeaba06399..39573e490a66 100644 --- a/

[PATCH v3 20/22] net: dc2114x: Include env.h to permit reading the environment

2025-05-15 Thread Tom Rini
From: Simon Glass This file uses the environment but does not include the header file. Update it. Signed-off-by: Simon Glass --- Changes in v3: - None --- drivers/net/dc2114x.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/net/dc2114x.c b/drivers/net/dc2114x.c index 7c0665faa8e1.

[PATCH v3 19/22] net: Include env.h in pcap.c

2025-05-15 Thread Tom Rini
From: Simon Glass This file uses the environment but does not include the header file. Update it. Signed-off-by: Simon Glass --- Changes in v3: - None --- net/pcap.c | 1 + 1 file changed, 1 insertion(+) diff --git a/net/pcap.c b/net/pcap.c index c959e3e4e511..d1d6f705cda0 100644 --- a/net/pc

[PATCH v3 17/22] technexion: Include env.h to permit reading the environment

2025-05-15 Thread Tom Rini
From: Simon Glass This file reads from the environment but does not include the correct header. Update it. Signed-off-by: Simon Glass --- Changes in v3: - None --- board/technexion/pico-imx7d/pico-imx7d.c | 1 + 1 file changed, 1 insertion(+) diff --git a/board/technexion/pico-imx7d/pico-imx7

[PATCH v3 16/22] liebherr: Include env.h to permit reading the environment

2025-05-15 Thread Tom Rini
From: Simon Glass This file reads from the environment but does not include the correct header. Update it. Signed-off-by: Simon Glass Reviewed-by: Lukasz Majewski --- Changes in v3: - None --- board/liebherr/xea/xea.c | 1 + 1 file changed, 1 insertion(+) diff --git a/board/liebherr/xea/xea.

[PATCH v3 15/22] google: Include env.h to permit reading the environment

2025-05-15 Thread Tom Rini
From: Simon Glass This file reads from the environment but does not include the correct header. Update it. Signed-off-by: Simon Glass --- Changes in v3: - None --- board/google/chameleonv3/board.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/board/google/chameleonv3/board.c b/board/go

[PATCH v3 14/22] freescale: Include env.h to permit reading the environment

2025-05-15 Thread Tom Rini
From: Simon Glass This file reads from the environment but does not include the correct header. Update it. Signed-off-by: Simon Glass --- Changes in v3: - None --- board/freescale/mx7dsabresd/mx7dsabresd.c | 1 + 1 file changed, 1 insertion(+) diff --git a/board/freescale/mx7dsabresd/mx7dsabr

[PATCH v3 12/22] synology: Include env.h to permit reading the environment

2025-05-15 Thread Tom Rini
From: Simon Glass This file reads from the environment but does not include the correct header. Update it. Reviewed-by: Stefan Roese Reviewed-by: Tony Dinh Signed-off-by: Simon Glass --- Changes in v3: - None --- board/Synology/ds414/ds414.c | 1 + 1 file changed, 1 insertion(+) diff --git

[PATCH v3 09/22] toradex: Include env.h to permit reading the environment

2025-05-15 Thread Tom Rini
From: Simon Glass Two files read from the environment but don't not include the correct header. Update them. Signed-off-by: Simon Glass --- Changes in v3: - None --- board/toradex/colibri_imx7/colibri_imx7.c | 1 + board/toradex/verdin-imx8mm/verdin-imx8mm.c | 2 +- 2 files changed, 2 insert

[PATCH v3 11/22] tegra: Include env.h to permit reading the environment

2025-05-15 Thread Tom Rini
From: Simon Glass This file reads from the environment but does not include the correct header. Update it. Signed-off-by: Simon Glass --- Changes in v3: - None --- arch/arm/mach-tegra/dt-setup.c | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm/mach-tegra/dt-setup.c b/arch/arm/mach-

[PATCH v3 10/22] advantech: Include env.h in imx8qm_dmsse20_a1

2025-05-15 Thread Tom Rini
From: Simon Glass This file uses the environment but does not include the header file. Update it. Acked-by: Oliver Graute Signed-off-by: Simon Glass --- Changes in v3: - None --- board/advantech/imx8qm_dmsse20_a1/imx8qm_dmsse20_a1.c | 1 + 1 file changed, 1 insertion(+) diff --git a/board/ad

[PATCH v3 08/22] ronetix: Include env.h to permit reading the environment

2025-05-15 Thread Tom Rini
From: Simon Glass This file reads from the environment but does not include the correct header. Update it. Signed-off-by: Simon Glass --- Changes in v3: - None --- board/ronetix/imx8mq-cm/imx8mq_cm.c | 1 + 1 file changed, 1 insertion(+) diff --git a/board/ronetix/imx8mq-cm/imx8mq_cm.c b/boa

[PATCH v3 07/22] phytec: Include env.h to permit reading the environment

2025-05-15 Thread Tom Rini
From: Simon Glass This file reads from the environment but does not include the correct header. Update it. Signed-off-by: Simon Glass --- Changes in v3: - None --- board/phytec/common/phytec_som_detection_blocks.c | 1 + 1 file changed, 1 insertion(+) diff --git a/board/phytec/common/phytec_s

[PATCH v3 06/22] venice: Include env.h to permit reading the environment

2025-05-15 Thread Tom Rini
From: Simon Glass This file reads from the environment but does not include the correct header. Update it. Signed-off-by: Simon Glass --- Changes in v3: - None --- board/gateworks/venice/venice.c | 1 + 1 file changed, 1 insertion(+) diff --git a/board/gateworks/venice/venice.c b/board/gatewo

[PATCH v3 05/22] net: Include string.h in net-legacy.h

2025-05-15 Thread Tom Rini
From: Simon Glass This file uses memcpy() but does not include the header file. Update it. Reviewed-by: Jerome Forissier Signed-off-by: Simon Glass --- Changes in v3: - None --- include/net-legacy.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/net-legacy.h b/include/net-legacy.

[PATCH v3 01/22] net: Remove env_get_ip helper() function

2025-05-15 Thread Tom Rini
Currently, we have the function env_get_ip which takes an IP address in string form and returns a struct in_addr representation of that address. It is however little used and means that a number of places indirectly (and unclearly) get via . To clean this up start by replacing env_get_ip() calls w

[PATCH v3 04/22] imx8ulp_evk: Include env.h to permit reading the environment

2025-05-15 Thread Tom Rini
From: Simon Glass This file reads from the environment but does not include the correct header. Update it. Signed-off-by: Simon Glass --- Changes in v3: - None --- board/freescale/imx8ulp_evk/imx8ulp_evk.c | 1 + 1 file changed, 1 insertion(+) diff --git a/board/freescale/imx8ulp_evk/imx8ulp_

[PATCH v3 03/22] dhelectronics: Include env.h to permit reading the environment

2025-05-15 Thread Tom Rini
From: Simon Glass This file reads from the environment but does not include the correct header. Update it. Signed-off-by: Simon Glass --- Changes in v3: - None --- board/dhelectronics/common/dh_common.c | 1 + 1 file changed, 1 insertion(+) diff --git a/board/dhelectronics/common/dh_common.c

[PATCH v3 02/22] congatec: Include env.h to permit reading the environment

2025-05-15 Thread Tom Rini
From: Simon Glass This file reads from the environment but does not include the correct header. Update it. Signed-off-by: Simon Glass Signed-off-by: Tom Rini --- Changes in v3: - Keep config.h, update commit message - trini --- board/congatec/cgtqmx8/cgtqmx8.c | 1 + 1 file changed, 1 inserti

[PATCH v3 00/22] Remove from

2025-05-15 Thread Tom Rini
Hey all, This is a v3 of Simon's series[1] and depends on the series[2] I posted the other day that removes from . With this series done, I believe we've tackled all of the current cases of headers which include without directly needing it. Much of this series is in fact Simon's v2 with the main

Re: [PATCH] ARM: Align image end to 8 bytes to fit DT alignment

2025-05-15 Thread Tom Rini
On Mon, May 12, 2025 at 06:10:28PM +0200, Marek Vasut wrote: > Align U-Boot image end to 8 bytes to make sure DT alignment requirement > is fulfilled. This fixes a possible failure in fdt_find_separate() in > case the U-Boot image is aligned to 4 Bytes and DT is appended at the > end at already 8

Re: [PATCH 1/1] tools: mkimage: validate image references in FIT configurations

2025-05-15 Thread Aristo Chen
Hi all, Alexander Dahl 於 2025年5月15日 週四 下午12:44寫道: > > Hei hei, > > Am Thu, May 15, 2025 at 09:35:34AM +0200 schrieb Quentin Schulz: > > Hi Simon, > > > > On 5/14/25 9:44 PM, Simon Glass wrote: > > > Hi, > > > > > > On Tue, 13 May 2025 at 17:28, Quentin Schulz > > > wrote: > > > > > > > > Hi Ari

Re: [PATCH v4 09/22] passage: arm: Accept a passage from the previous phase

2025-05-15 Thread Tom Rini
On Thu, May 15, 2025 at 09:26:42PM +0200, Simon Glass wrote: > Hi Tom, > > On Wed, 14 May 2025 at 23:58, Tom Rini wrote: > > > > On Sat, May 10, 2025 at 03:42:07PM +0200, Simon Glass wrote: > > > > > Accept a bloblist and control devicetree from a previous phase in > > > registers 0 to 3, as docu

Re: [PATCH v4 17/22] passage: Add a qemu test for ARM

2025-05-15 Thread Tom Rini
On Thu, May 15, 2025 at 09:26:54PM +0200, Simon Glass wrote: > Hi Tom, > > On Wed, 14 May 2025 at 23:58, Tom Rini wrote: > > > > On Sat, May 10, 2025 at 03:42:15PM +0200, Simon Glass wrote: > > > > > Check that the standard passage works on ARM, by setting it up in SPL > > > and making sure that

Re: [PATCH 1/6] test: Update fs_helper to support passing in the image

2025-05-15 Thread Tom Rini
On Thu, May 15, 2025 at 09:33:18PM +0200, Simon Glass wrote: > Hi Tom, > > On Thu, 15 May 2025 at 15:55, Tom Rini wrote: > > > > On Thu, May 15, 2025 at 06:51:17AM -0700, Simon Glass wrote: > > > This function is useful for other scripts, so add a parameter to specify > > > the image name, if des

Re: [RFC PATCH 3/3] serial: mxc: restore booting for imx8mn_bsh_smm_s2

2025-05-15 Thread Michael Nazzareno Trimarchi
Hi Fabio On Thu, May 15, 2025 at 9:45 PM Fabio Estevam wrote: > On Thu, May 15, 2025 at 10:12 AM Dario Binacchi > wrote: > > > > The commit dda454e933c6 ("serial: mxc: Support bulk enabling clocks") > > breaks the booting of the BSH SMM S2 board. > > Restore proper booting of the board even in

Re: [RFC PATCH 3/3] serial: mxc: restore booting for imx8mn_bsh_smm_s2

2025-05-15 Thread Fabio Estevam
On Thu, May 15, 2025 at 10:12 AM Dario Binacchi wrote: > > The commit dda454e933c6 ("serial: mxc: Support bulk enabling clocks") > breaks the booting of the BSH SMM S2 board. > Restore proper booting of the board even in case of failure of either > clk_get_bulk() or clk_enable_bulk(). We need a b

Re: [PATCH v3 00/10] ext4: Overflow fixes

2025-05-15 Thread Tom Rini
On Thu, May 15, 2025 at 09:26:59PM +0200, Simon Glass wrote: > Hi Tom, > > On Wed, 14 May 2025 at 23:50, Tom Rini wrote: > > > > On Sat, May 10, 2025 at 03:06:06PM +0200, Simon Glass wrote: > > > > > This series fixes some potential overflows in the ext4 code with invalid > > > or malicious files

Re: [PATCH v4 06/22] fdt: Redo devicetree setup

2025-05-15 Thread Simon Glass
Hi Jonas, On Sat, 10 May 2025 at 17:31, Jonas Karlman wrote: > > Hi Simon, > > On 2025-05-10 15:42, Simon Glass wrote: > > The existing logic for handling receiving a devicetree from a previous > > phase is quite complicated. Simplify it in preparation for introducing > > standard passage. > > >

Re: [PATCH 1/6] test: Update fs_helper to support passing in the image

2025-05-15 Thread Simon Glass
Hi Tom, On Thu, 15 May 2025 at 15:55, Tom Rini wrote: > > On Thu, May 15, 2025 at 06:51:17AM -0700, Simon Glass wrote: > > This function is useful for other scripts, so add a parameter to specify > > the image name, if desired. > > > > Also add a flag to quieten the output of some of the tools, s

Re: [PATCH v3 00/10] ext4: Overflow fixes

2025-05-15 Thread Simon Glass
Hi Tom, On Wed, 14 May 2025 at 23:50, Tom Rini wrote: > > On Sat, May 10, 2025 at 03:06:06PM +0200, Simon Glass wrote: > > > This series fixes some potential overflows in the ext4 code with invalid > > or malicious filesystems. > > > > To assist with this and future work, it brings in overflow.h

Re: [PATCH] ARM: Align image end to 8 bytes to fit DT alignment

2025-05-15 Thread Simon Glass
Hi Marek, On Mon, 12 May 2025 at 18:11, Marek Vasut wrote: > > Align U-Boot image end to 8 bytes to make sure DT alignment requirement > is fulfilled. This fixes a possible failure in fdt_find_separate() in > case the U-Boot image is aligned to 4 Bytes and DT is appended at the > end at already 8

Re: [PATCH v4 17/22] passage: Add a qemu test for ARM

2025-05-15 Thread Simon Glass
Hi Tom, On Wed, 14 May 2025 at 23:58, Tom Rini wrote: > > On Sat, May 10, 2025 at 03:42:15PM +0200, Simon Glass wrote: > > > 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 Gla

Re: [PATCH v4 09/22] passage: arm: Accept a passage from the previous phase

2025-05-15 Thread Simon Glass
Hi Tom, On Wed, 14 May 2025 at 23:58, Tom Rini wrote: > > On Sat, May 10, 2025 at 03:42:07PM +0200, Simon Glass wrote: > > > Accept a bloblist and control devicetree from a previous phase in > > registers 0 to 3, as documented in the Firmware Handoff > > specification[1]. > > > > Note that there

[PATCH v4 3/5] ram: rockchip: Fix dependency of RAM_ROCKCHIP_DEBUG

2025-05-15 Thread Lukasz Czechowski
The RAM_ROCKCHIP_DEBUG can be used only if DEBUG_UART is available, otherwise it won't have any effect. Signed-off-by: Lukasz Czechowski Reviewed-by: Kever Yang --- drivers/ram/rockchip/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/ram/rockchip/Kconfig b/drivers/ram/rockch

[PATCH v4 5/5] rockchip: px30: Fix hard dependency to DEBUG_UART_BOARD_INIT

2025-05-15 Thread Lukasz Czechowski
Because DEBUG_UART_BOARD_INIT depends on DEBUG_UART, hard dependency to DEBUG_UART_BOARD_INIT in ROCKCHIP_PX30 can cause warnings if DEBUG_UART is disabled. The DEBUG_UART_BOARD_INIT is already implied by ARCH_ROCKCHIP entry. Remove hard dependency from ROCKCHIP_PX30, so that it will be consistent

[PATCH v4 4/5] rockchip: px30: Weaken dependency TPL/SPL serial

2025-05-15 Thread Lukasz Czechowski
Allow to disable serial console in TPL and SPL. Weak dependency to SPL_SERIAL and TPL_SERIAL is also used in other Rockchip boards. Signed-off-by: Lukasz Czechowski Reviewed-by: Quentin Schulz Reviewed-by: Kever Yang --- arch/arm/mach-rockchip/Kconfig | 4 ++-- 1 file changed, 2 insertions(+)

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

2025-05-15 Thread Lukasz Czechowski
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 UART is not available, to avoid compilation errors. Signed-off-by: Lukasz

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

2025-05-15 Thread Lukasz Czechowski
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 warnings. Signed-off-by: Lukasz Czechowski --- lib/efi/efi_stub.c | 2 ++ 1 f

[PATCH v4 0/5] Rockchip: Allow to silent TPL/SPL debug console

2025-05-15 Thread Lukasz Czechowski
: 1b5e435102aa29a665119430196cb366ce36a01b change-id: 20250515-silent_console-af979569e24a Best regards, -- Lukasz Czechowski

[PATCH v3 2/2] arm: imx: imx8m: soc: replace ifdef by IS_ENABLED()

2025-05-15 Thread Dario Binacchi
Standardize on using the IS_ENABLED macro. Signed-off-by: Dario Binacchi --- Changes in v3: - Replaces all replaceable macros within the file arch/arm/mach-imx/imx8m/soc.c | 40 +-- 1 file changed, 20 insertions(+), 20 deletions(-) diff --git a/arch/arm/mach-i

[PATCH] binman: add atf-bl1 to etypes

2025-05-15 Thread Bryan Brattlof
Some SoCs require a Trusted Firmware-A (TF-A) AP Trusted ROM (BL1) to initialize the SoC before U-Boot can run properly. Add an atf-bl1 etype so we can properly package BL1 into a final binary Signed-off-by: Bryan Brattlof --- Makefile | 1 + tools/binman/entries.rst |

Re: [PATCH v2 2/3] sunxi: add "fake" FEL pin support

2025-05-15 Thread Quentin Schulz
Hi Andre, On 5/12/25 2:39 PM, Andre Przywara wrote: On Tue, 22 Apr 2025 16:30:04 +0200 Quentin Schulz wrote: Hi Quentin, just found this in my draft folder. It's not really related to this patch anymore, but you seemed to be interested, and I am happy to explain some of the specialities for s

[PATCH v3 1/2] arm: imx: imx8m: soc: fix the macro name

2025-05-15 Thread Dario Binacchi
From: Michael Trimarchi The function arch_spl_mmc_get_uboot_raw_sector() was never compiled, even when the option CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_USE_PARTITION was enabled. So rename the macro SYS_MMCSD_RAW_MODE_U_BOOT_USE_PARTITION to CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_USE_PARTITION. Fixes: 2a00

Re: [PATCH RFC] fit: allow to specify RSAPSS padding salt length magic value

2025-05-15 Thread Quentin Schulz
Hi all, Gentle ping, anything to say/suggest? Cheers, Quentin On 4/11/25 6:32 PM, Quentin Schulz wrote: From: Quentin Schulz Before commit 1b99c15d73c1 ("lib: rsa: Set conventional salt length RSA-PSS parameter"), the salt length for PSS padding was openSSL's default. Prior to openSSL 3.1, t

[RFC PATCH v2 2/4] imx: spl_imx_romapi: support raw NAND sector

2025-05-15 Thread Dario Binacchi
Commit 2a00d73d081a ("spl: mmc: Try to clean up raw-mode options") breaks the boot of the BSH SMM S2 board. As stated in the dropped comment, "Some boards use this value even though MMC is not enabled in SPL, for example imx8mn_bsh_smm_s2". Support load of the U-Boot image from raw NAND sector. F

[RFC PATCH v2 4/4] serial: mxc: restore booting for imx8mn_bsh_smm_s2

2025-05-15 Thread Dario Binacchi
The commit dda454e933c6 ("serial: mxc: Support bulk enabling clocks") breaks the booting of the BSH SMM S2 board. Restore proper booting of the board even in case of failure of either clk_get_bulk() or clk_enable_bulk(). Fixes: dda454e933c6 ("serial: mxc: Support bulk enabling clocks") Signed-off-

[RFC PATCH v2 3/4] configs: imx8mn_bsh_smm_s2: load U-Boot from raw NAND

2025-05-15 Thread Dario Binacchi
Commit 2a00d73d081a ("spl: mmc: Try to clean up raw-mode options") breaks the boot of the BSH SMM S2 board. Add options to load U-Boot from raw NAND sector. Fixes: 2a00d73d081a ("spl: mmc: Try to clean up raw-mode options") Signed-off-by: Dario Binacchi --- Changes in v2: - Drop CONFIG_SYS_MMCS

[RFC PATCH v2 1/4] spl: Kconfig: support U-Boot load from raw NAND

2025-05-15 Thread Dario Binacchi
Commit 2a00d73d081a ("spl: mmc: Try to clean up raw-mode options") breaks the boot of the BSH SMM S2 board. As stated in the commit itself, "Some boards use this value even though MMC is not enabled in SPL, for example imx8mn_bsh_smm_s2". Support load of the U-Boot image from raw NAND sector. This

[RFC PATCH v2 0/4] Restore imx8mn_bsh_smm_s2 properly booting

2025-05-15 Thread Dario Binacchi
Commits [1] and [2] broke the booting of the BSH SMM S2 board. The patches in this series restore proper booting. [1] 2a00d73d081a ("spl: mmc: Try to clean up raw-mode options") [2] dda454e933c6 ("serial: mxc: Support bulk enabling clocks") (no changes since v1) Dario Binacchi (4): spl: Kconfi

Re: [RFC PATCH 1/3] spl: Kconfig: allow to enable SYS_MMCSD_RAW_MODE_U_BOOT_USE_SECTOR for i.MX8M

2025-05-15 Thread Quentin Schulz
On 5/15/25 4:30 PM, Tom Rini wrote: On Thu, May 15, 2025 at 03:26:18PM +0200, Quentin Schulz wrote: Hi Dario, On 5/15/25 3:12 PM, Dario Binacchi wrote: Commit 2a00d73d081a ("spl: mmc: Try to clean up raw-mode options") breaks the boot of the BSH SMM S2 board. As stated in the commit itself, "S

Re: [RFC PATCH 1/3] spl: Kconfig: allow to enable SYS_MMCSD_RAW_MODE_U_BOOT_USE_SECTOR for i.MX8M

2025-05-15 Thread Tom Rini
On Thu, May 15, 2025 at 03:26:18PM +0200, Quentin Schulz wrote: > Hi Dario, > > On 5/15/25 3:12 PM, Dario Binacchi wrote: > > Commit 2a00d73d081a ("spl: mmc: Try to clean up raw-mode options") breaks > > the boot of the BSH SMM S2 board. As stated in the commit itself, "Some > > boards use this va

Re: [PATCH] arm: imx: imx8m: soc: fix the macro name

2025-05-15 Thread Tom Rini
On Thu, May 15, 2025 at 01:37:44PM +0200, Quentin Schulz wrote: > Hi Dario, > > On 5/15/25 12:02 PM, Dario Binacchi wrote: > > From: Michael Trimarchi > > > > The function arch_spl_mmc_get_uboot_raw_sector() was never compiled, > > even when the option CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_USE_PARTIT

Re: [PATCH v2 2/2] arm: imx: imx8m: soc: replace ifdef by IS_ENABLED()

2025-05-15 Thread Tom Rini
On Thu, May 15, 2025 at 03:40:15PM +0200, Quentin Schulz wrote: > Hi Dario, > > On 5/15/25 3:30 PM, Dario Binacchi wrote: > > Standardize on using the IS_ENABLED macro. > > > > Signed-off-by: Dario Binacchi > > > > --- > > > > (no changes since v1) > > > > arch/arm/mach-imx/imx8m/soc.c | 2

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

2025-05-15 Thread Verdun, Jean-Marie
>> 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 error code and return data through a >> pointer as argume

[RFC PATCH 2/3] configs: imx8mn_bsh_smm_s2: restore CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR

2025-05-15 Thread Dario Binacchi
The commit d892702080d4 ("configs: Resync with savedefconfig") drop the CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR=0x300 option due to the previous commit 2a00d73d081a ("spl: mmc: Try to clean up raw-mode options"). Restore the option, which is necessary for the board to boot correctly. Fixes: 2a00d

[PATCH] ifwitool: fix memory leak in ifwi_dir_extract

2025-05-15 Thread ant . v . moryakov
From: Anton Moryakov Static analyzer triggering: Dynamic memory, referenced by 'subpart_dir_buff.name', is allocated at ifwitool.c:1132 by calling function 'parse_subpart_dir' at ifwitool.c:1878 and lost at ifwitool.c:1904. Correct explained: Added a missing call to buffer_delete() in the fun

Re: [PATCH 1/6] test: Update fs_helper to support passing in the image

2025-05-15 Thread Tom Rini
On Thu, May 15, 2025 at 06:51:17AM -0700, Simon Glass wrote: > This function is useful for other scripts, so add a parameter to specify > the image name, if desired. > > Also add a flag to quieten the output of some of the tools, since in > many cases we only want to see warnings / errors. > > Si

Re: [PATCH 1/6] test: Update fs_helper to support passing in the image

2025-05-15 Thread Simon Glass
This function is useful for other scripts, so add a parameter to specify the image name, if desired. Also add a flag to quieten the output of some of the tools, since in many cases we only want to see warnings / errors. Signed-off-by: Simon Glass --- test/py/tests/fs_helper.py | 18 +++

Re: [PATCH 5/6] u_boot_pylib: Support a fatal level in tout

2025-05-15 Thread Simon Glass
It is convenient to be able to print a message and exit. Add a new 'fatal' level to support this. Update some assumptions about the level, so that the tools continue to work as now. Signed-off-by: Simon Glass --- tools/binman/control.py| 6 +++--- tools/u_boot_pylib/tout.py | 22 +

Re: [PATCH 4/6] scripts: Create a common settings file for QEMU scripts

2025-05-15 Thread Simon Glass
Move the settings into a common file so they can be used by all tools. Signed-off-by: Simon Glass --- scripts/build-efi | 53 - scripts/build_helper.py | 65 + 2 files changed, 71 insertions(+), 47 deletions(-) Appli

Re: [PATCH v2 2/2] arm: imx: imx8m: soc: replace ifdef by IS_ENABLED()

2025-05-15 Thread Quentin Schulz
Hi Dario, On 5/15/25 3:30 PM, Dario Binacchi wrote: Standardize on using the IS_ENABLED macro. Signed-off-by: Dario Binacchi --- (no changes since v1) arch/arm/mach-imx/imx8m/soc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/mach-imx/imx8m/soc.c b/arch/ar

[PATCH v2 2/2] arm: imx: imx8m: soc: replace ifdef by IS_ENABLED()

2025-05-15 Thread Dario Binacchi
Standardize on using the IS_ENABLED macro. Signed-off-by: Dario Binacchi --- (no changes since v1) arch/arm/mach-imx/imx8m/soc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/mach-imx/imx8m/soc.c b/arch/arm/mach-imx/imx8m/soc.c index 806adcf145fa..6c53555d22bf 1

[PATCH v2 1/2] arm: imx: imx8m: soc: fix the macro name

2025-05-15 Thread Dario Binacchi
From: Michael Trimarchi The function arch_spl_mmc_get_uboot_raw_sector() was never compiled, even when the option CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_USE_PARTITION was enabled. So rename the macro SYS_MMCSD_RAW_MODE_U_BOOT_USE_PARTITION to CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_USE_PARTITION. Fixes: 2a00

Re: [RFC PATCH 1/3] spl: Kconfig: allow to enable SYS_MMCSD_RAW_MODE_U_BOOT_USE_SECTOR for i.MX8M

2025-05-15 Thread Quentin Schulz
Hi Dario, On 5/15/25 3:12 PM, Dario Binacchi wrote: Commit 2a00d73d081a ("spl: mmc: Try to clean up raw-mode options") breaks the boot of the BSH SMM S2 board. As stated in the commit itself, "Some boards use this value even though MMC is not enabled in SPL, for example imx8mn_bsh_smm_s2". The s

[RFC PATCH 3/3] serial: mxc: restore booting for imx8mn_bsh_smm_s2

2025-05-15 Thread Dario Binacchi
The commit dda454e933c6 ("serial: mxc: Support bulk enabling clocks") breaks the booting of the BSH SMM S2 board. Restore proper booting of the board even in case of failure of either clk_get_bulk() or clk_enable_bulk(). Fixes: dda454e933c6 ("serial: mxc: Support bulk enabling clocks") Signed-off-

[RFC PATCH 1/3] spl: Kconfig: allow to enable SYS_MMCSD_RAW_MODE_U_BOOT_USE_SECTOR for i.MX8M

2025-05-15 Thread Dario Binacchi
Commit 2a00d73d081a ("spl: mmc: Try to clean up raw-mode options") breaks the boot of the BSH SMM S2 board. As stated in the commit itself, "Some boards use this value even though MMC is not enabled in SPL, for example imx8mn_bsh_smm_s2". The same commit makes SPL_SYS_MMCSD_RAW_MODE depend on SPL_D

[RFC PATCH 0/3] Restore imx8mn_bsh_smm_s2 properly booting

2025-05-15 Thread Dario Binacchi
Commits [1] and [2] broke the booting of the BSH SMM S2 board. The patches in this series restore proper booting. [1] 2a00d73d081a ("spl: mmc: Try to clean up raw-mode options") [2] dda454e933c6 ("serial: mxc: Support bulk enabling clocks") Dario Binacchi (3): spl: Kconfig: allow to enable SYS

Re: [PATCH 3/4] arm: dts: k3-am625-verdin-r5*: Remove clocks from main_timer0

2025-05-15 Thread Francesco Dolcini
On Thu, May 15, 2025 at 05:08:36PM +0530, Manorit Chawdhry wrote: > The commit 79d91e77f4c2 ("clk: ti: clk-k3-pll: Add additional robustness > steps to the PLL sequence") introduced a change which requires the > main_timer0 to not rely on it's own clocks which anyways was wrong. > > Fix it by remo

Re: [PATCH 4/5] global: Avoid indirect inclusion of from

2025-05-15 Thread Jerome Forissier
On 5/14/25 23:46, Tom Rini wrote: > 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 places which had relied on this indirect > inclusion to

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

2025-05-15 Thread Quentin Schulz
Hi Moteen, On 5/15/25 11:01 AM, Moteen Shah wrote: Hey Quentin, On 14/05/25 21:42, Quentin Schulz wrote: Hi Moteen, On 5/14/25 8:05 AM, Moteen Shah wrote: Hey Quentin, Thanks for the review. On 13/05/25 16:10, Quentin Schulz wrote: Hi Moteen, On 5/12/25 1:50 PM, Moteen Shah wrote: Add a

[PATCH 2/4] arm: dts: k3-am625-phycore-r5*: Remove clocks from main_timer0

2025-05-15 Thread Manorit Chawdhry
The commit 79d91e77f4c2 ("clk: ti: clk-k3-pll: Add additional robustness steps to the PLL sequence") introduced a change which requires the main_timer0 to not rely on it's own clocks which anyways was wrong. Fix it by removing the clock dependency for it, also while at it, move it from u-boot.dtsi

[PATCH 3/4] arm: dts: k3-am625-verdin-r5*: Remove clocks from main_timer0

2025-05-15 Thread Manorit Chawdhry
The commit 79d91e77f4c2 ("clk: ti: clk-k3-pll: Add additional robustness steps to the PLL sequence") introduced a change which requires the main_timer0 to not rely on it's own clocks which anyways was wrong. Fix it by removing the clock dependency for it, also while at it, move it from u-boot.dtsi

[PATCH 4/4] arm: dts: k3-am62a7-phycore-r5*: Remove clocks from main_timer0

2025-05-15 Thread Manorit Chawdhry
The commit 79d91e77f4c2 ("clk: ti: clk-k3-pll: Add additional robustness steps to the PLL sequence") introduced a change which requires the main_timer0 to not rely on it's own clocks which anyways was wrong. Fix it by removing the clock dependency for it, also while at it, move it from u-boot.dtsi

[PATCH 0/4] Remove clocks from R5 dtsi for missed boards.

2025-05-15 Thread Manorit Chawdhry
ommit: 4c26de2eea6bcf5f27b13da0372d27d767cd38e3 change-id: 20250515-b4-upstream-k3-pll-miss-c31817b19be9 Best regards, -- Manorit Chawdhry

[PATCH 1/4] arm: dts: k3-am625-beagleplay-r5*: Remove clocks from main_timer0

2025-05-15 Thread Manorit Chawdhry
The commit 79d91e77f4c2 ("clk: ti: clk-k3-pll: Add additional robustness steps to the PLL sequence") introduced a change which requires the main_timer0 to not rely on it's own clocks which anyways was wrong. Fix it by removing the clock dependency for it, also while at it, move it from u-boot.dtsi

Re: [PATCH] arm: imx: imx8m: soc: fix the macro name

2025-05-15 Thread Quentin Schulz
Hi Dario, On 5/15/25 12:02 PM, Dario Binacchi wrote: From: Michael Trimarchi The function arch_spl_mmc_get_uboot_raw_sector() was never compiled, even when the option CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_USE_PARTITION was enabled. So rename the macro SYS_MMCSD_RAW_MODE_U_BOOT_USE_PARTITION to CONF

Re: [PATCH 1/3] arm: dts: k3-*-r5: Remove clocks from mcu_timer0

2025-05-15 Thread Manorit Chawdhry
Hi Francesco, On 12:18-20250515, Francesco Dolcini wrote: > Hello Manorit, > > On Thu, Nov 21, 2024 at 05:32:51PM +0530, Manorit Chawdhry wrote: > > Updated PLL driver sequencing requires us to use udelay in the PLL > > driver as there is no poll bit to get the status of

[PATCH 1/2] arm: mach-k3: Detect and print reset reason

2025-05-15 Thread Wadim Egorov
Call get_reset_reason() during CPU info output and display the result if the SoC provides the implementation. This helps in debugging by providing context on the last system reset reason. Signed-off-by: Wadim Egorov --- arch/arm/mach-k3/common.c| 10 ++ arch/arm/mach-k3/i

[PATCH 2/2] arm: mach-k3: am62x: Implement get_reset_reason()

2025-05-15 Thread Wadim Egorov
Implement get_reset_reason() for AM62x to enable reporting of the reset cause in the cpuinfo output. Signed-off-by: Wadim Egorov --- arch/arm/mach-k3/am62x/boot.c | 40 +++ arch/arm/mach-k3/include/mach/am62_hardware.h | 19 + 2 files changed, 59 insertion

[PATCH 0/2] Print Reset Reason for K3 based devices

2025-05-15 Thread Wadim Egorov
Although AM62x, AM62Ax, AM64x, and AM62Px share the same register for reset reason, not all TRMs document this consistently. To avoid relying on undocumented behavior, introduce SoC-specific implementations for now. If TI confirms all SoCs share the same register, this can be moved to common.c for

Re: [PATCH] qcom_defconfig: Enable AUTOBOOT_KEYED support

2025-05-15 Thread Sumit Garg
On Tue, May 13, 2025 at 04:54:22PM +0100, Casey Connolly wrote: > Hi Sumit, > > On 5/12/25 08:01, Sumit Garg wrote: > > On Fri, May 09, 2025 at 04:54:43PM +0200, Stephan Gerhold wrote: > > > On Fri, May 09, 2025 at 12:45:20PM +0200, Casey Connolly wrote: > > > > On 5/8/25 12:32, Sumit Garg wrote:

Re: [PATCH 1/1] tools: mkimage: validate image references in FIT configurations

2025-05-15 Thread Alexander Dahl
Hei hei, Am Thu, May 15, 2025 at 09:35:34AM +0200 schrieb Quentin Schulz: > Hi Simon, > > On 5/14/25 9:44 PM, Simon Glass wrote: > > Hi, > > > > On Tue, 13 May 2025 at 17:28, Quentin Schulz > > wrote: > > > > > > Hi Aristo, > > > > > > On 5/5/25 12:08 PM, Aristo Chen wrote: > > > > When pars

Re: [PATCH 1/3] arm: dts: k3-*-r5: Remove clocks from mcu_timer0

2025-05-15 Thread Francesco Dolcini
Hello Manorit, On Thu, Nov 21, 2024 at 05:32:51PM +0530, Manorit Chawdhry wrote: > Updated PLL driver sequencing requires us to use udelay in the PLL > driver as there is no poll bit to get the status of operations. > tick-timer(mcu_timer0/main_timer0) setting up the clocks for itself is > somethi

[PATCH] arm: imx: imx8m: soc: fix the macro name

2025-05-15 Thread Dario Binacchi
From: Michael Trimarchi The function arch_spl_mmc_get_uboot_raw_sector() was never compiled, even when the option CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_USE_PARTITION was enabled. So rename the macro SYS_MMCSD_RAW_MODE_U_BOOT_USE_PARTITION to CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_USE_PARTITION. Fixes: 2a00

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

2025-05-15 Thread Moteen Shah
Hey Quentin, On 14/05/25 21:42, Quentin Schulz wrote: Hi Moteen, On 5/14/25 8:05 AM, Moteen Shah wrote: Hey Quentin, Thanks for the review. On 13/05/25 16:10, Quentin Schulz wrote: Hi Moteen, On 5/12/25 1:50 PM, Moteen Shah wrote: Add a function to scan through all the nodes in the device-

Re: [PATCH 1/1] tools: mkimage: validate image references in FIT configurations

2025-05-15 Thread Quentin Schulz
Hi Simon, On 5/14/25 9:44 PM, Simon Glass wrote: Hi, On Tue, 13 May 2025 at 17:28, Quentin Schulz wrote: Hi Aristo, On 5/5/25 12:08 PM, Aristo Chen wrote: When parsing a FIT image source (ITS), mkimage does not currently check whether the image names referenced in the /configurations secti

Re: [PATCH 4/5] global: Avoid indirect inclusion of from

2025-05-15 Thread Mattijs Korpershoek
Hi Tom, Thank you for the patch. On Wed, May 14, 2025 at 16:46, Tom Rini wrote: > 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 places w

  1   2   >