Re: [PATCH v3 1/8] dt-bindings: clock: rename at91.h to at91-pmc-status.h

2025-03-05 Thread Sumit Garg
Hi Manikandan, On Wed, Mar 05, 2025 at 11:37:04AM +0530, Manikandan Muralidharan wrote: > Rename the include at91.h to at91-pmc-status.h to avoid conflicts with > the upstream bindings that has the same file and update the > relevant legacy SoC Device Trees to reflect this change. > This is useful

[PATCH v7 3/8] phy: starfive: Add Starfive JH7110 PCIe 2.0 PHY driver

2025-03-05 Thread Minda Chen
Add Starfive JH7110 PCIe 2.0 PHY driver, which is generic PHY driver and can be used as USB 3.0 driver. Signed-off-by: Minda Chen Tested-by: E Shattow --- drivers/phy/starfive/Kconfig | 7 + drivers/phy/starfive/Makefile | 1 + drivers/phy/starfive/phy-jh7110-pcie.c | 239

[PATCH v7 6/8] spl: starfive: visionfive2: Disable USB overcurrent pin by default.

2025-03-05 Thread Minda Chen
For some JH7110 boards, USB host overcurent pin is not reserved, To make USB host work, overcurrent pin must be disabled. So set the pin default disabled in spl stage. Signed-off-by: Minda Chen Tested-by: E Shattow --- arch/riscv/include/asm/arch-jh7110/gpio.h | 5 + board/starfive/visionfi

[PATCH v7 7/8] configs: starfive: Add visionfive2 cadence USB configuration

2025-03-05 Thread Minda Chen
Add cadence USB confiuration. Signed-off-by: Minda Chen Tested-by: E Shattow --- configs/starfive_visionfive2_defconfig | 10 ++ 1 file changed, 10 insertions(+) diff --git a/configs/starfive_visionfive2_defconfig b/configs/starfive_visionfive2_defconfig index c3f2142ae1b..c2be9ffae27

[PATCH v7 5/8] usb: cdns: starfive: Add cdns USB driver

2025-03-05 Thread Minda Chen
Add Starfive cdns USB3 wrapper driver. Signed-off-by: Minda Chen Reviewed-by: Marek Vasut Tested-by: E Shattow --- drivers/usb/cdns3/Kconfig | 7 ++ drivers/usb/cdns3/Makefile | 1 + drivers/usb/cdns3/cdns3-starfive.c | 182 + 3 files changed, 1

[PATCH v7 0/8] Add Starfive JH7110 Cadence USB driver

2025-03-05 Thread Minda Chen
Add Starfive JH7110 Cadence USB driver and related PHY driver. So the codes can be used in visionfive2 and star64 7110 board. The driver is almost the same with kernel driver. Test with Star64 JH7110 board USB 3.0 + USB 2.0 host. The code can work. The Starfive JH7110 has enable CONFIG_OF_UPSTRE

[PATCH v7 1/8] usb: cdns3: Set USB PHY mode in cdns3_drd_update_mode()

2025-03-05 Thread Minda Chen
USB PHY maybe need to set PHY mode in different USB dr mode. So translate USB PHY mode to generic PHY mode and call generic_phy_set_mode(). Signed-off-by: Minda Chen Reviewed-by: Marek Vasut --- drivers/usb/cdns3/drd.c | 14 ++ 1 file changed, 14 insertions(+) diff --git a/drivers/

Re: [PATCH v7 18/19] imx95_evk: add i.MX95 19x19 EVK board basic support

2025-03-05 Thread Lothar Waßmann
Hi, On Wed, 05 Mar 2025 21:28:30 +0800 Alice Guo (OSS) wrote: > From: Ye Li > > This patch adds i.MX95 19x19 EVK board basic support. > > Messaging unit for EdgeLock Secure Enclave, messaging unit for System > Manager, uSDHC for SD Card, gpio, lpuart are supported now. > > Signed-off-by: Ye Li

Re: [PATCH RFC/RFT 1/3] i2c: omap24xx_i2c: Use new function __omap24_i2c_xfer_msg()

2025-03-05 Thread Heiko Schocher
Hello Aniket, On 04.03.25 23:04, Aniket Limaye wrote: Remove __omap24_i2c_read/write() usage from omap_i2c_xfer() in favour of the more flexible __omap24_i2c_xfer_msg(). Consequently, these are also no longer needed when DM_I2C is enabled. New function __omap24_i2c_xfer_msg() will take care of

Re: [PATCH v3 06/12] i2c: Add support for ADI SC5XX-family I2C peripheral

2025-03-05 Thread Heiko Schocher
Hello Greg, On 26.02.25 18:30, Greg Malysa wrote: From: Nathan Barrett-Morrison Co-developed-by: Greg Malysa Signed-off-by: Greg Malysa Co-developed-by: Ian Roberts Signed-off-by: Ian Roberts Co-developed-by: Angelo Dureghello Signed-off-by: Angelo Dureghello Signed-off-by: Vasileios Bim

Re: [PATCH RFC/RFT 2/3] i2c: omap24xx_i2c: support CONFIG_I2C_REPEATED_START in DM_I2C xfer

2025-03-05 Thread Heiko Schocher
Hello Aniket, On 04.03.25 23:04, Aniket Limaye wrote: Repeated Start Condition (Sr) can be used to transfer multiple i2c msgs without sending a Stop condition (P). So far, the driver default was to always send a Stop condition after every i2c msg. Add support for a config option (CONFIG_I2C_REP

Re: [PATCH v7 15/19] tools: imx8image: add i.MX95 support

2025-03-05 Thread Lothar Waßmann
Hi, On Wed, 05 Mar 2025 21:28:27 +0800 Alice Guo (OSS) wrote: > From: Alice Guo > > i.MX95 uses binman to invoke mkimage to create image container. 2 image > containers are needed currently. The first one is composed of > ahab-container.img, LPDDR firmware images, OEI images, System Manager > im

Re: [PATCH RFC/RFT 0/3] omap24xx_i2c: Enable Repeated Start functionality

2025-03-05 Thread Heiko Schocher
Hello Tom, Aniket, On 05.03.25 16:50, Tom Rini wrote: On Wed, Mar 05, 2025 at 03:34:08AM +0530, Aniket Limaye wrote: This series implements Repeated Start functionality for the DM_I2C xfer API (omap_i2c_xfer()) First, split out the actual implementation for each individual i2c msg transfer int

Re: [PATCH v7 10/19] imx9: scmi: add i.MX95 SoC and clock related code

2025-03-05 Thread Lothar Waßmann
Hi, On Wed, 05 Mar 2025 21:28:22 +0800 Alice Guo (OSS) wrote: > From: Peng Fan > > This patch adds i.MX95 SoC and clock related code. Because they are > based on SCMI, put them in the scmi subfolder. > > Signed-off-by: Ye Li > Signed-off-by: Peng Fan > Signed-off-by: Alice Guo > Reviewed-by:

Re: [PATCH v7 07/19] clk: scmi: check the clock state/parent/rate control permissions

2025-03-05 Thread Lothar Waßmann
Hi, On Wed, 05 Mar 2025 21:28:19 +0800 Alice Guo (OSS) wrote: > From: Alice Guo > > Clock driver based on SCMI clock management protocol in Linux checks > clock state, parent and rate control permissions. To be consistent with > the kernel driver, add this check here. > > When using common cloc

Re: [PATCH v7 06/19] clk: scmi: add the command CLOCK_PARENT_SET

2025-03-05 Thread Lothar Waßmann
Hi, On Wed, 05 Mar 2025 21:28:18 +0800 Alice Guo (OSS) wrote: > From: Peng Fan > > This patch adds the command CLOCK_PARENT_SET that can be used to set the > parent of a clock. ARM SCMI Version 3.2 supports to change the parent of > a clock device. > > Signed-off-by: Peng Fan > Signed-off-by:

Re: [PATCH 2/4] riscv: reset: k1: Add reset driver

2025-03-05 Thread Leo Liang
Hi Huan, Please run scripts/checkpatch.pl against your patchset and respin a v2 after all the warnings & errors are attended to. Thanks, Leo On Tue, Mar 04, 2025 at 03:03:28PM +0800, Huan Zhou wrote: > Add spacemit reset driver. > --- > drivers/reset/Kconfig | 7 + > drivers/rese

Re: [resend v3 00/19] Add Cadence NAND Driver support

2025-03-05 Thread Michael Nazzareno Trimarchi
Hi Michael Nazzareno Trimarchi Co-Founder & Chief Executive Officer M. +39 347 913 2170 mich...@amarulasolutions.com __ Amarula Solutions BV Joop Geesinkweg 125, 1114 AB, Amsterdam, NL T. +31 (0)85 111 9172 i...@amarulasolutions.com www.amarulasolutions.com Il mer

Re: [PATCH 5/5] riscv: select OF_HAS_PRIOR_STAGE by default if SBI is enabled

2025-03-05 Thread Leo Liang
On Thu, Feb 27, 2025 at 02:47:33PM +, Yao Zi wrote: > Availability of RISC-V SBI service implies a prior stage exists. As SBI > firmware usually passes a FDT to the loaded program, let's select > OF_HAS_PRIOR_STAGE if SBI is enabled. > > With previously added fallback version of board_fdt_blob

Re: [PATCH 4/5] board: sifive: Remove duplicated board_fdt_blob_setup

2025-03-05 Thread Leo Liang
On Thu, Feb 27, 2025 at 02:47:32PM +, Yao Zi wrote: > The default version should work for both SiFive Unmatched and Unleashed. > > Signed-off-by: Yao Zi > --- > board/sifive/unleashed/unleashed.c | 11 --- > board/sifive/unmatched/unmatched.c | 10 -- > 2 files changed, 21 de

Re: [PATCH 3/5] board: starfive: Remove duplicated board_fdt_blob_setup

2025-03-05 Thread Leo Liang
On Thu, Feb 27, 2025 at 02:47:31PM +, Yao Zi wrote: > The default version should work for Starfive VisionFive 2. > > Signed-off-by: Yao Zi > --- > board/starfive/visionfive2/starfive_visionfive2.c | 10 -- > 1 file changed, 10 deletions(-) Reviewed-by: Leo Yu-Chi Liang

Re: [PATCH 2/5] board: qemu: riscv: Remove duplicated board_fdt_blob_setup

2025-03-05 Thread Leo Liang
On Thu, Feb 27, 2025 at 02:47:30PM +, Yao Zi wrote: > The default version should work for RISC-V QEMU. > > Signed-off-by: Yao Zi > --- > board/emulation/qemu-riscv/qemu-riscv.c | 8 > 1 file changed, 8 deletions(-) Reviewed-by: Leo Yu-Chi Liang

Re: [PATCH 1/5] riscv: lib: Add a default implementation of board_fdt_blob_setup

2025-03-05 Thread Leo Liang
On Thu, Feb 27, 2025 at 02:47:29PM +, Yao Zi wrote: > It's common for S-Mode U-Boot to retrieve a FDT blob along with taking > control from SBI firmware. Add a weak version of board_fdt_blob_setup to > make use of it by default and avoid copy-pasting similar functions among > boards. > > Signe

Re: [PATCH 00/17] Various toolchain compatibility fixes/improvements

2025-03-05 Thread Sam Edwards
On Sun, Feb 23, 2025 at 9:55 PM Sam Edwards wrote: > > Long time no see, U-Boot folks! > > This patchset consists of various bug fixes and correctness improvements that > I discovered while attempting to add first-class LLVM support to the build > system. These patches are NOT related to LLVM supp

Re: [PATCH v2 1/1] configs: SiFive Unmatched: add 'nvme scan' to preboot

2025-03-05 Thread Leo Liang
On Tue, Nov 12, 2024 at 11:26:44AM +0100, Heinrich Schuchardt wrote: > Without 'nvme scan' the ESP on the NVMe drive is not found early. > EFI variables cannot be persisted. > > Hit any key to stop autoboot: 0 > Cannot persist EFI variables without system partition > ** Booting bootfl

Re: [PATCH 1/1] riscv: qemu: imply CONFIG_RNG_RISCV_ZKR

2025-03-05 Thread Leo Liang
On Sun, Mar 02, 2025 at 09:50:17AM +0100, Heinrich Schuchardt wrote: > The zkr ISA extension can be used to generate random numbers. Since RVA22 > zkr is an optional ISA extension. It can be emulated by QEMU. Our RNG > driver detects if the extension is usable during driver binding. Let's > enable

[PATCH v5 02/29] x86: Rename zboot_run() to zboot_run_args()

2025-03-05 Thread Simon Glass
Rename this function so we can (later) create a zboot_run() function which looks the same as bootm_run() Signed-off-by: Simon Glass --- (no changes since v1) arch/x86/lib/zimage.c | 4 ++-- boot/bootmeth_cros.c | 6 +++--- include/bootm.h | 6 +++--- 3 files changed, 8 insertions(+), 8

[PATCH v5 15/29] boot: arm: riscv: sandbox: Add a format for the booti file

2025-03-05 Thread Simon Glass
Arm invented a new format for arm64 and something similar is also used with RISC-V. Add this to the list of supported formats and provide a way for the format to be detected on both architectures. Update the genimg_get_format() function to support this. Fix up switch() statements which don't curr

[PATCH v5 06/29] x86: Move the bootm state for zimage into cmd/

2025-03-05 Thread Simon Glass
Rather than holding the state in the implementation code, move it to the command code. The state is now passed to the implementation functions and can there (with future work) be pass in from bootstd, without going through the commands. Signed-off-by: Simon Glass --- (no changes since v1) arch

[PATCH v5 17/29] boot: Support compressed booti images in bootm

2025-03-05 Thread Simon Glass
A compressed booti image relies on the compression-format's header at the start to indicate which compression algorithm is used. We don't support this elsewhere in U-Boot, so assume that a compressed file is always a booti file. Once it is compressed, a check is made to make sure that it actually

[PATCH v5 10/29] boot: Split pxe label_boot() into two parts

2025-03-05 Thread Simon Glass
This function is far too long. Split out the part which builds and runs the bootm/i/z commands into its own function. Add a function comment for the new label_run_boot() function. Signed-off-by: Simon Glass Reviewed-by: Quentin Schulz --- (no changes since v2) Changes in v2: - Move strlcpy()

[PATCH v5 14/29] boot: Convert IMAGE_FORMAT into an enum

2025-03-05 Thread Simon Glass
Use an enum so it is clearer that these options are related. Update genimg_get_format(), tidy up the function comment and move it to the header file, since it is exported. Signed-off-by: Simon Glass --- (no changes since v3) Changes in v3: - Add new patch to convert IMAGE_FORMAT into an enum

[PATCH v5 01/29] x86: Make do_zboot_states() static

2025-03-05 Thread Simon Glass
This function is only called within zboot.c so make the function private. Signed-off-by: Simon Glass --- (no changes since v1) cmd/x86/zboot.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/cmd/x86/zboot.c b/cmd/x86/zboot.c index 94e602b8a5b..3035172352a 100644 --- a/c

[PATCH v5 22/29] net: Tidy up the comments to parse_args()

2025-03-05 Thread Simon Glass
This function is a bit vague as to what it does. Expand the comment a little, to specify which args are provided and which variables are updated. Signed-off-by: Simon Glass Acked-by: Ilias Apalodimas --- (no changes since v1) cmd/net.c | 13 +++-- 1 file changed, 11 insertions(+), 2 d

[PATCH v5 20/29] boot: pxe: Refactor label_run_boot() to avoid cmdline

2025-03-05 Thread Simon Glass
Adjust the remaining call in this function to use the bootm API. This will allow PXE to work without the command line. Signed-off-by: Simon Glass --- Changes in v5: - Use ramdisk address for the initrd_addr field, not kernel address arch/x86/lib/zimage.c | 27 +++ boot/

[PATCH v5 28/29] net: Drop #ifdef in parse_args()

2025-03-05 Thread Simon Glass
Use IS_ENABLED() to avoid an extra build path. Signed-off-by: Simon Glass Acked-by: Ilias Apalodimas --- (no changes since v1) cmd/net.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/cmd/net.c b/cmd/net.c index f980448e0ef..6d1c6374f76 100644 --- a/cmd/net.c

[PATCH v5 21/29] net: Keep the bootstage functions together

2025-03-05 Thread Simon Glass
Move the bootstage_mark() function just before net_loop(), so that the IPv6 code is not in the way. Signed-off-by: Simon Glass --- (no changes since v1) cmd/net.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/cmd/net.c b/cmd/net.c index 79525f73a51..deebd5b710f 100644

[PATCH v5 27/29] net: Refactor part of netboot_common() into a function

2025-03-05 Thread Simon Glass
Move the core code for starting an netboot operation into a separate function, so that we can (with additional work) move towards calling it from outside the file. Signed-off-by: Simon Glass --- (no changes since v1) cmd/net.c | 62 +-- 1 fil

[PATCH v5 25/29] net: Return the address and size from parse_addr_size()

2025-03-05 Thread Simon Glass
Rather than updating the global, update the value of some parameters, so the action of the function is simpler. Signed-off-by: Simon Glass --- (no changes since v1) cmd/net.c | 15 +-- 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/cmd/net.c b/cmd/net.c index d15d344

[PATCH v5 19/29] pxe_utils: Simplify default fdt in label_run_boot()

2025-03-05 Thread Simon Glass
Tidy up this code a little to avoid two calls to env_get() for both fdt_addr and fdtcontroladdr Signed-off-by: Simon Glass Suggested-by: Quentin Schulz --- (no changes since v2) Changes in v2: - Add new patch to simplify the code reading fdtcontroladdr and fdt_addr boot/pxe_utils.c | 14

[PATCH v5 29/29] net: Provide a function to run network operations

2025-03-05 Thread Simon Glass
Add a new netboot_run() function which can be used for simple network operations, such as loading a file. Put the implementation in an internal function, used by the existing code. Place this function into the net/ code, so that it does not need the command line to be available. Document which ne

[PATCH v5 24/29] net: Return the load address from parse_args()

2025-03-05 Thread Simon Glass
Rather than updating the global, update the value of a parameter, so the action of the function is simpler. Signed-off-by: Simon Glass --- (no changes since v1) cmd/net.c | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/cmd/net.c b/cmd/net.c index cc7d14eb082..d1

[PATCH v5 23/29] net: Simplify parse_args()

2025-03-05 Thread Simon Glass
This function repeats the same code in a few places, namely setting net_boot_file_name_explicit and copying of the filename to net_boot_file_name Move these two operations to the caller, with just the filename (or NULL) returned by parse_args() This makes things a little easier to follow. Signed

[PATCH v5 18/29] boot: pxe: Use bootm_...() functions where possible

2025-03-05 Thread Simon Glass
Rather than building a command line for each operation, use the functions provided by the bootm API. Make sure that the bootm functions are available if pxe_utils is used. Since SYS_BOOTM_LEN is not present for the tools-only build, adjust the code to handle that. Signed-off-by: Simon Glass Rev

[PATCH v5 26/29] net: Return the size from parse_args()

2025-03-05 Thread Simon Glass
Rather than setting global variables, return the size, if provided. For tftput, use the addr argument to store the save address, to avoid adding yet another parameter. Signed-off-by: Simon Glass --- (no changes since v1) cmd/net.c | 22 ++ 1 file changed, 14 insertions(+),

[PATCH v5 16/29] boot: Support booti format in bootm

2025-03-05 Thread Simon Glass
At present the booti format is handled separately, in its own command. Provide a way to boot uncompressed booti images within the bootm code, so that eventually we can boot these images without CONFIG_CMDLINE Update bootm_init() to attach the images for all formats which use them. Add some debugg

[PATCH v5 13/29] bootm: Allow building bootm.c without CONFIG_SYS_BOOTM_LEN

2025-03-05 Thread Simon Glass
This code cannot be compiled by boards which don't have this option. Add an accessor in the header file to avoid another #ifdef Signed-off-by: Simon Glass Reviewed-by: Quentin Schulz --- (no changes since v1) boot/bootm.c| 8 include/bootm.h | 8 2 files changed, 12 inse

[PATCH v5 09/29] boot: Use strlcpy() in label_boot()

2025-03-05 Thread Simon Glass
This function is recommended instead of strncpy() since it always terminates the string. Signed-off-by: Simon Glass --- (no changes since v2) Changes in v2: - Split out strlcpy() change into new patch boot/pxe_utils.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/boot/px

[PATCH v5 12/29] boot: Pass just the FDT argument to label_process_fdt()

2025-03-05 Thread Simon Glass
Since this function only adjusts one element of the bootm command, pass just that. This will make it easier to refactor things to remove the bootm command. Signed-off-by: Simon Glass Reviewed-by: Quentin Schulz --- (no changes since v1) boot/pxe_utils.c | 14 +++--- 1 file changed, 7

[PATCH v5 11/29] boot: Split pxe label_run_boot() into two parts

2025-03-05 Thread Simon Glass
This function is quite long. Split out the FDT processing into its own function. Add a function comment for the new label_process_fdt() function. Signed-off-by: Simon Glass Reviewed-by: Quentin Schulz --- (no changes since v2) Changes in v2: - Rebase on earlier change boot/pxe_utils.c | 100

[PATCH v5 07/29] bootstd: Correct display of kernel version

2025-03-05 Thread Simon Glass
The address of the bzImage is not recorded in the bootflow, so we cannot actually locate the version at present. Handle this case, to avoid showing invalid data. Signed-off-by: Simon Glass --- (no changes since v1) arch/x86/lib/zimage.c | 13 - cmd/bootflow.c| 2 ++ 2 file

[PATCH v5 08/29] x86: Drop the unnecessary base_ptr argument to zboot_dump()

2025-03-05 Thread Simon Glass
This value is include the bootm_info, so drop the unnecessary parameter. Signed-off-by: Simon Glass --- (no changes since v1) arch/x86/lib/zimage.c | 5 +++-- cmd/bootflow.c| 3 ++- cmd/x86/zboot.c | 8 +++- include/bootm.h | 7 ++- 4 files changed, 10 insertions(+)

[PATCH v5 04/29] x86: Move x86 zboot state into struct bootm_info

2025-03-05 Thread Simon Glass
This structure is supposed to handle any type of booting programmatically, i.e. without needing a command to be executed. Move the x86-specific members into it and use it instead of struct zboot_state. Provide a macro so access is possible without adding lots of #ifdefs to the code. This will allo

[PATCH v5 05/29] x86: Rename state to bmi

2025-03-05 Thread Simon Glass
Use the common name for the struct, in preparation for passing it around between functions. Signed-off-by: Simon Glass --- (no changes since v1) arch/x86/include/asm/zimage.h | 2 +- arch/x86/lib/zimage.c | 48 +-- cmd/x86/zboot.c | 4 +--

[PATCH v5 03/29] x86: Drop duplicate definition of zimage_dump()

2025-03-05 Thread Simon Glass
This is now defined in bootm.h so drop the duplicate in the x86 code. Signed-off-by: Simon Glass --- (no changes since v1) arch/x86/include/asm/zimage.h | 8 cmd/x86/zboot.c | 1 + 2 files changed, 1 insertion(+), 8 deletions(-) diff --git a/arch/x86/include/asm/zimage.

[PATCH v5 00/29] pxe: Precursor series for supporting read_all() in extlinux / PXE

2025-03-05 Thread Simon Glass
This series includes some patches related to allowing read_all() to be used with the extlinux / PXE bootmeths. These patches were split out from the stb4 series, since it will need to have additional patches for LWIP, to avoid breaking PXE booting when LWIP is used. Changes in v5: - Use ramdisk a

Re: [PATCH 2/3] net: ravb: Fix RX frame size limit

2025-03-05 Thread Marek Vasut
On 3/5/25 9:40 PM, Paul Barker wrote: Hi Marek, On 05/03/2025 19:53, Marek Vasut wrote: On 3/4/25 9:07 PM, Paul Barker wrote: The value written to the RFLR register includes the length of the CRC data at the end of each Ethernet frame. So we need to increase the value written to this register

Re: [PATCH 3/5] reset: rzg2l-usbphy-ctrl: Connect up vbus regulator

2025-03-05 Thread Marek Vasut
On 3/4/25 5:37 PM, Paul Barker wrote: Bind the USB VBUS regulator driver under the USB PHY reset driver for the Renesas RZ/G2L and related SoCs. This additional bind is needed as the corresponding device tree node does not contain a compatible string. Signed-off-by: Paul Barker Reviewed-by: Ma

Re: [PATCH 2/3] net: ravb: Fix RX frame size limit

2025-03-05 Thread Paul Barker
Hi Marek, On 05/03/2025 19:53, Marek Vasut wrote: > On 3/4/25 9:07 PM, Paul Barker wrote: >> The value written to the RFLR register includes the length of the CRC >> data at the end of each Ethernet frame. So we need to increase the value >> written to this register to ensure that we can receive f

Re: [PATCH 3/3] net: ravb: Fix error handling in ravb_probe

2025-03-05 Thread Paul Barker
Hi Marek, On 05/03/2025 19:55, Marek Vasut wrote: > On 3/4/25 9:07 PM, Paul Barker wrote: >> In ravb_probe(), we were missing a couple of things in the error >> handling path: >> >>* We must unregister the MDIO bus before freeing the corresponding >> struct mii_dev instance to avoid the p

Re: [PATCH 4/5] phy: rcar: Support RZ/G2L USB PHY

2025-03-05 Thread Marek Vasut
On 3/4/25 5:37 PM, Paul Barker wrote: Extend the existing Renesas R-Car gen3 Gen3 (uppercase G) USB 2.0 PHY driver to support the RZ/G2L and related SoCs. Also enable this driver by default for the RZ/G2L SoC family. Signed-off-by: Paul Barker Reviewed-by: Marek Vasut

Re: [PATCH 2/5] regulator: rzg2l-usbphy: Add new driver

2025-03-05 Thread Marek Vasut
On 3/4/25 5:37 PM, Paul Barker wrote: Add a new regulator driver to control the USB VBUS supply on the Renesas RZ/G2L and related SoCs. Signed-off-by: Paul Barker Reviewed-by: Marek Vasut

Re: [PATCH 1/5] reset: rzg2l-usbphy-ctrl: Add new driver

2025-03-05 Thread Marek Vasut
On 3/4/25 5:37 PM, Paul Barker wrote: [...] +static int rzg2l_usbphy_ctrl_assert(struct reset_ctl *reset_ctl) +{ + struct rzg2l_usbphy_ctrl_priv *priv = dev_get_priv(reset_ctl->dev); + u32 val; + + val = readl(priv->regs + RESET); + val |= reset_ctl->id ? PHY_RESET_PORT2

Re: [PATCH 3/3] net: ravb: Fix error handling in ravb_probe

2025-03-05 Thread Marek Vasut
On 3/4/25 9:07 PM, Paul Barker wrote: In ravb_probe(), we were missing a couple of things in the error handling path: * We must unregister the MDIO bus before freeing the corresponding struct mii_dev instance to avoid the potential for use-after-free bugs. * We must free the res

Re: [PATCH 2/3] net: ravb: Fix RX frame size limit

2025-03-05 Thread Marek Vasut
On 3/4/25 9:07 PM, Paul Barker wrote: The value written to the RFLR register includes the length of the CRC data at the end of each Ethernet frame. So we need to increase the value written to this register to ensure that we can receive full size frames. While we're here we can also copy the impr

Re: [PATCH 1/3] net: ravb: Add dependency on CONFIG_BITBANGMII

2025-03-05 Thread Marek Vasut
On 3/4/25 9:07 PM, Paul Barker wrote: The Renesas RAVB driver always requires bitbang MDIO bus support. Fixes: 8ae51b6f324e ("net: ravb: Add Renesas Ethernet RAVB driver") Signed-off-by: Paul Barker Reviewed-by: Marek Vasut

Re: [PATCH] clk: rzg2l: Ignore disable for core clocks

2025-03-05 Thread Marek Vasut
On 3/4/25 8:44 PM, Paul Barker wrote: Following on from commit 9a699a0a0d62 ("clk: rzg2l: Ignore enable for core clocks"), we also need to ignore attempts to disable core clocks to avoid the need for conditionals around clk_disable_bulk() calls in drivers which support both RZ/G2L and other Renes

Re: [PATCH v3] arm64: renesas: Add TFA BL31 handoff support on Renesas R-Car Gen4

2025-03-05 Thread Marek Vasut
On 3/3/25 10:44 AM, Quentin Schulz wrote: Hi Marek, Hi, [...]   board/renesas/common/gen4-common.c | 126 +   1 file changed, 126 insertions(+) diff --git a/board/renesas/common/gen4-common.c b/board/renesas/ common/gen4-common.c index 52a0639073b..f7d129be4c8 1

[PATCH 2/2] Makefile.lib: Set xPL build related defines for DTB build

2025-03-05 Thread Hendrik Donner
The CONFIG_*PL_BUILD defines are currently not defined when preprocessing the dts files, leading to build problems with binman. Set the defines based on the related CONFIG_*PL values. Tested-by: Oliver Graute Signed-off-by: Hendrik Donner --- scripts/Makefile.lib | 10 ++ 1 file changed

[PATCH 1/2] imx8qm: Fix build when using SPL

2025-03-05 Thread Hendrik Donner
Analog to 645a42d7ee88dc8c3848c15d178b0f111c636933, imx8qm based boards which use SPL currently fail during make all: Image 'imx-boot' is missing external blobs and is non-functional: spl /binman/imx-boot/spl (spl.bin): Missing blob Image 'imx-boot' has faked external blobs and is non-functiona

[PATCH 0/2] Build fixes for CONFIG_XPL_BUILD in dts files

2025-03-05 Thread Hendrik Donner
Just 2 fixes related to CONFIG_XPL_BUILD usage in dts files with binman. The second patch works for me, not sure if it is exactly the correct approach. Hendrik Donner (2): imx8qm: Fix build when using SPL Makefile.lib: Set xPL build related defines for DTB build arch/arm/dts/imx8qm-u-boot.dt

Re: [PATCH v4 00/12] Add imx8mp video support

2025-03-05 Thread Tom Rini
On Wed, Mar 05, 2025 at 03:12:09PM -0300, Fabio Estevam wrote: > On Mon, Mar 3, 2025 at 2:06 PM Tom Rini wrote: > > > It's a tangential failure. Svyatoslav reported this with a series he's > > working on and Simon wonders if bloblist support got broken but hasn't > > had time to look at it. > >

Re: [PATCH v4 00/12] Add imx8mp video support

2025-03-05 Thread Fabio Estevam
On Mon, Mar 3, 2025 at 2:06 PM Tom Rini wrote: > It's a tangential failure. Svyatoslav reported this with a series he's > working on and Simon wonders if bloblist support got broken but hasn't > had time to look at it. Please let me know how to proceed with this error. Because of this error, I

Re: [PATCH] Add cross compile dependecy info for ARMv7

2025-03-05 Thread Peter Polgar
Hi Heinrich, Sorry for the late reply, other things were going on so I dropped my U-boot project for a while. What you have written totally makes sense. I wasn't aware what the "hf" stands for in the package name. Just saw that my architecture was "armhf", so I assumed I needed the build essentia

Problems with test_net_pxe_get test

2025-03-05 Thread Tom Rini
Hi Love, I've been trying to setup and use the test_net_pxe_get test now and I've found a few problems. - An easy one is that we call test_net_setup_static after verifying the network is up, which I believe is a legacy of when reviewing the test initially and not removing the line. - A harder

Re: [PATCH v3 08/14] lib: time: hook uthread_schedule() into udelay()

2025-03-05 Thread Jerome Forissier
On 3/4/25 16:24, Jerome Forissier wrote: > Introduce a uthread scheduling loop into udelay() when CONFIG_UTHREAD > is enabled. This means that any uthread calling into udelay() may yield > to uthread and be scheduled again later. The granunlarity of the > scheduling loop is defined by CONFIG_UTH

Re: xPL Proposal

2025-03-05 Thread Simon Glass
Hi Tom, On Wed, 5 Mar 2025 at 08:49, Tom Rini wrote: > > On Wed, Mar 05, 2025 at 07:17:51AM -0700, Simon Glass wrote: > > Hi Tom, > > > > On Tue, 4 Mar 2025 at 09:25, Tom Rini wrote: > > > > > > On Tue, Mar 04, 2025 at 08:35:29AM -0700, Simon Glass wrote: > > > > Hi Tom, > > > > > > > > On Thu,

[PATCH v2 3/6] lwip: tls: warn when no CA exists amd log certificate validation errors

2025-03-05 Thread Jerome Forissier
Using HTTPS without root (CA) certificates is a security issue. Print a warning in this case. Also, when certificate verification fail, print an additional message because "HTTP client error 4" is not very informative (4 is HTTPC_RESULT_ERR_CLOSED). Signed-off-by: Jerome Forissier Reviewed-by: Il

[GIT PULL] amd/xilinx patches for v2025.04-rc4

2025-03-05 Thread Michal Simek
Hi Tom, please pull these fixes to your tree. Especially patches from Padmarao are must have. One is fixing issue around using BSS section before relocation and second is fix connected to switching to LWIP. 4k MTU size never really worked with our MACB driver but old ethernet stack fallback to

[PATCH 2/2] vepxress64: add guide for running FVP with TF-A

2025-03-05 Thread Harrison Mutai
Add documentation on how to run FVP with U-Boot and TF-A. This helps users configure and run U-Boot correctly on Arm models. Signed-off-by: Harrison Mutai --- doc/board/armltd/vexpress64.rst | 54 +++-- 1 file changed, 52 insertions(+), 2 deletions(-) diff --git a/do

Re: [PATCH v4 00/29] arm: Support building as an EFI app

2025-03-05 Thread Simon Glass
Hi Igor, On Tue, 4 Mar 2025 at 07:50, Igor Opaniuk wrote: > > Hello Simon, > > On Sat, Feb 15, 2025 at 2:08 PM Simon Glass wrote: > > > > The original x86 EFI-app was written back in 2015, about a year before > > the EFI-loader was introduced. At that point, ARM mostly used U-Boot and > > FIT (F

Re: Rate of innovation in the project (Was: Re: Rate of change in the project)

2025-03-05 Thread Tom Rini
On Wed, Mar 05, 2025 at 07:19:25AM -0700, Simon Glass wrote: > Hi Tom, > > On Tue, 4 Mar 2025 at 08:29, Tom Rini wrote: > > > > On Tue, Mar 04, 2025 at 06:13:36AM -0700, Simon Glass wrote: > > > Hi Tom, > > > > > > (I changed the subject as 'change' sounds like change for change's sake) > > > > F

RE: [resend v2 00/19] Add Cadence NAND Driver support

2025-03-05 Thread Maniyam, Dinesh
From: Michael Nazzareno Trimarchi Sent: Wednesday, 5 March 2025 5:27 pm To: Maniyam, Dinesh Cc: u-boot@lists.denx.de; Marek ; Simon ; Tom Rini ; Dario Binacchi ; Johan Jonker ; Michal Simek ; Arseniy Krasnov ; Alexander Dahl ; William Zhang ; Igor Prusov ; Meng, Tingting ; Chee, Tien Fong

Re: [PATCH v3 1/2] CI: Move default image under global defaults

2025-03-05 Thread Tom Rini
On Wed, Mar 05, 2025 at 07:18:40AM -0700, Simon Glass wrote: > Hi Tom, > > On Tue, 4 Mar 2025 at 09:12, Tom Rini wrote: > > > > On Tue, Mar 04, 2025 at 08:35:56AM -0700, Simon Glass wrote: > > > Hi Tom, > > > > > > On Thu, 27 Feb 2025 at 10:03, Tom Rini wrote: > > > > > > > > On Thu, Feb 27, 202

Re: [PATCH RFC/RFT 0/3] omap24xx_i2c: Enable Repeated Start functionality

2025-03-05 Thread Tom Rini
On Wed, Mar 05, 2025 at 03:34:08AM +0530, Aniket Limaye wrote: > This series implements Repeated Start functionality for the DM_I2C xfer > API (omap_i2c_xfer()) > > First, split out the actual implementation for each individual i2c msg > transfer into a new function __omap24_i2c_xfer_msg() and use

Re: [RFC PATCH] Kbuild: Always use $(PHASE_)

2025-03-05 Thread Tom Rini
On Wed, Mar 05, 2025 at 07:15:50AM -0700, Simon Glass wrote: > Hi Tom, > > On Tue, 4 Mar 2025 at 13:41, Tom Rini wrote: > > > > It is confusing to have both "$(PHASE_)" and "$(XPL_)" be used in our > > Makefiles as part of the macros to determine when to do something in our > > Makefiles based on

Re: xPL Proposal

2025-03-05 Thread Tom Rini
On Wed, Mar 05, 2025 at 07:17:51AM -0700, Simon Glass wrote: > Hi Tom, > > On Tue, 4 Mar 2025 at 09:25, Tom Rini wrote: > > > > On Tue, Mar 04, 2025 at 08:35:29AM -0700, Simon Glass wrote: > > > Hi Tom, > > > > > > On Thu, 27 Feb 2025 at 13:30, Tom Rini wrote: > > > > > > > > On Thu, Feb 27, 202

Re: [PATCH v2 1/6] net: lwip: extend wget to support CA (root) certificates

2025-03-05 Thread Jerome Forissier
Hi Heinrich, On 3/5/25 16:07, Heinrich Schuchardt wrote: > On 05.03.25 15:26, Jerome Forissier wrote: >> Add the "cacert" (Certification Authority certificates) subcommand to >> wget to pass root certificates to the code handling the HTTPS protocol. >> The subcommand is enabled by the WGET_CACERT

Re: [PATCH v2 1/6] net: lwip: extend wget to support CA (root) certificates

2025-03-05 Thread Heinrich Schuchardt
On 05.03.25 15:26, Jerome Forissier wrote: Add the "cacert" (Certification Authority certificates) subcommand to wget to pass root certificates to the code handling the HTTPS protocol. The subcommand is enabled by the WGET_CACERT Kconfig symbol. Usage example: => dhcp # Download some root c

Re: [PATCH 4/5] net: lwip: add support for built-in root certificates

2025-03-05 Thread Jerome Forissier
Hi Ilias, On 3/1/25 07:59, Ilias Apalodimas wrote: > Hi Jerome, > > On Thu, 27 Feb 2025 at 18:38, Jerome Forissier > wrote: >> >> Sorry for replying to myself, I spotted a small mistake. >> >> On 2/27/25 17:09, Jerome Forissier wrote: >>> Introduce Kconfig symbols WGET_BUILTIN_CACERT and >>> WGE

Re: (subset) [u-boot-test-hooks PATCH 1/3] labgrid: Deal with strange Python error

2025-03-05 Thread Tom Rini
On Wed, Mar 05, 2025 at 07:19:05AM -0700, Simon Glass wrote: > Hi Tom, > > On Mon, 3 Mar 2025 at 13:19, Tom Rini wrote: > > > > On Fri, 21 Feb 2025 13:32:40 -0700, Simon Glass wrote: > > > > > Add a way to work around an unexplained Python message which appears > > > sometimes with Python 3.12.3

Re: [PATCH v1 1/2] ARM: tegra: clock: fix PLLD/PLLD2 related clock calculations

2025-03-05 Thread Svyatoslav Ryhel
ср, 5 бер. 2025 р. о 16:02 Thierry Reding пише: > > On Wed, Mar 05, 2025 at 03:52:36PM +0200, Svyatoslav Ryhel wrote: > > ср, 5 бер. 2025 р. о 15:47 Thierry Reding пише: > > > > > > On Wed, Mar 05, 2025 at 01:12:12PM +0200, Svyatoslav Ryhel wrote: > > > > From: Jonas Schwöbel > > > > > > > > Whi

[PATCH v2 4/6] net: lwip: add support for built-in root certificates

2025-03-05 Thread Jerome Forissier
Introduce Kconfig symbols WGET_BUILTIN_CACERT and WGET_BUILTIN_CACERT_PATH to provide root certificates at build time. Usage example: wget -O cacert.crt https://cacerts.digicert.com/DigiCertTLSECCP384RootG5.crt make qemu_arm64_lwip_defconfig echo CONFIG_WGET_BUILTIN_CACERT=y >>.config echo CO

[PATCH v2 5/6] doc: cmd: wget: document cacert subcommand

2025-03-05 Thread Jerome Forissier
Document the 'wget cacert' subcommand which allows to configure root (CA) certificates for HTTPS. Signed-off-by: Jerome Forissier --- doc/usage/cmd/wget.rst | 82 -- 1 file changed, 80 insertions(+), 2 deletions(-) diff --git a/doc/usage/cmd/wget.rst b/do

[PATCH v2 6/6] configs: qemu_arm64_lwip_defconfig: enable WGET_CACERT

2025-03-05 Thread Jerome Forissier
Enable the "wget cacert" command. Signed-off-by: Jerome Forissier Reviewed-by: Ilias Apalodimas --- configs/qemu_arm64_lwip_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/configs/qemu_arm64_lwip_defconfig b/configs/qemu_arm64_lwip_defconfig index 754c770c33f..814e98729a3 100644

[PATCH v2 1/6] net: lwip: extend wget to support CA (root) certificates

2025-03-05 Thread Jerome Forissier
Add the "cacert" (Certification Authority certificates) subcommand to wget to pass root certificates to the code handling the HTTPS protocol. The subcommand is enabled by the WGET_CACERT Kconfig symbol. Usage example: => dhcp # Download some root certificates (note: not authenticated!) => wget

[PATCH v2 2/6] lwip: tls: enforce checking of server certificates based on CA availability

2025-03-05 Thread Jerome Forissier
Instead of relying on some build time configuration to determine if server certificates need to be checked against CA certificates, do it based on the availability of such certificates. If no CA is configured then no check can succeed; on the other hand if we have CA certs then we should not ignore

[PATCH v2 0/6] net: lwip: root certificates

2025-03-05 Thread Jerome Forissier
This series adds support for HTTP server authentication using root (CA) certificates. As a first step, the wget command is extended to support a sub-command: cacert . The memory region shall contain the CA certificates. With this, it is possible to load the certificates from storage or get them f

Re: (subset) [u-boot-test-hooks PATCH 1/3] labgrid: Deal with strange Python error

2025-03-05 Thread Simon Glass
Hi Tom, On Mon, 3 Mar 2025 at 13:19, Tom Rini wrote: > > On Fri, 21 Feb 2025 13:32:40 -0700, Simon Glass wrote: > > > Add a way to work around an unexplained Python message which appears > > sometimes with Python 3.12.3 > > > > Make the file executable while we are here. > > > > > > Applied to u-

Re: Rate of innovation in the project (Was: Re: Rate of change in the project)

2025-03-05 Thread Simon Glass
Hi Tom, On Tue, 4 Mar 2025 at 08:29, Tom Rini wrote: > > On Tue, Mar 04, 2025 at 06:13:36AM -0700, Simon Glass wrote: > > Hi Tom, > > > > (I changed the subject as 'change' sounds like change for change's sake) > > Frankly, you have a lot of "tidy up" patches where nothing was > functionally wron

  1   2   >