Re: [PATCH v3 4/4] riscv: dts: starfive: remove duplicate itb entries

2025-02-22 Thread E Shattow
On 2/21/25 01:58, Heinrich Schuchardt wrote: > As binman already creates nodes based on CONFIG_OF_LIST we don't need to > add extra nodes. > > Signed-off-by: Heinrich Schuchardt > Reviewed-by: Simon Glass > Tested-by: Simon Glass # StarFIve VisionFive 2 > --- > v3: > no change > v2: >

Re: [PATCH v3 3/4] board: starfive: spl: strip off 'starfive/' prefix

2025-02-22 Thread E Shattow
On 2/21/25 01:58, Heinrich Schuchardt wrote: > The configuration descriptions generated by binman contain the vendor > device-tree directory. Instead of adding it to all match strings just strip > it off. > > Signed-off-by: Heinrich Schuchardt > Reviewed-by: Leo Yu-Chi Liang > --- > v3: >

Re: [PATCH] Add 'bootsource' /chosen property

2025-02-22 Thread Simon Glass
Hi Quentin, On Thu, 20 Feb 2025 at 08:55, Quentin Schulz wrote: > > On 2/20/25 4:22 PM, Daniel Golle wrote: > > On Thu, Feb 20, 2025 at 04:04:27PM +0100, Quentin Schulz wrote: > >> Hi Daniel, > >> > >> On 2/15/25 4:13 PM, Daniel Golle wrote: > >>> Hi Quentin, > >>> > >>> On 10 February 2025 16:25

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

2025-02-22 Thread Simon Glass
Hi Tom, On Sat, 22 Feb 2025 at 14:37, Tom Rini wrote: > > On Sat, Feb 22, 2025 at 10:23:59AM -0700, Simon Glass wrote: > > Hi Tom, > > > > On Fri, 21 Feb 2025 at 17:08, Tom Rini wrote: > > > > > > On Fri, Feb 21, 2025 at 04:42:09PM -0700, Simon Glass wrote: > > > > Hi Tom, > > > > > > > > On Mon

Re: Rate of change in the project (Was: Re: xPL Proposal)

2025-02-22 Thread Simon Glass
Hi Tom, (I think you chopped off too much context, so I've added it below) On Fri, 21 Feb 2025 at 11:11, Tom Rini wrote: > > On Fri, Feb 21, 2025 at 11:53:20AM -0600, Tom Rini wrote: > > On Fri, Feb 21, 2025 at 08:03:12AM -0700, Simon Glass wrote: > > > Hi Tom, > [snip] [unsnip] > > > > > > > >

Re: [PATCH v3 2/4] riscv: dts: no default configuration for MULTI_DTB_FIT

2025-02-22 Thread E Shattow
On 2/21/25 01:58, Heinrich Schuchardt wrote: > JH7110 boards are currently the only use case for multi DTB FIT images > on RISC-V. > > Booting JH7110 systems with a VisionFive 2 device-tree used to kind of > work without causing harm to the hardware. But there is no guarantee > that this will h

Re: [PATCH v3 1/4] riscv: dts: add OF_LIST handling to binman.dtsi

2025-02-22 Thread E Shattow
On 2/21/25 01:58, Heinrich Schuchardt wrote: > Binman can automatically generate device-tree and configuration entries in > the FIT image based on CONFIG_MULTI_DTB_FIT if the binman node includes the > right sub-nodes. > > Signed-off-by: Heinrich Schuchardt > --- > v3: > Use CONFIG_MULTI

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

2025-02-22 Thread Tom Rini
On Sat, Feb 22, 2025 at 10:23:59AM -0700, Simon Glass wrote: > Hi Tom, > > On Fri, 21 Feb 2025 at 17:08, Tom Rini wrote: > > > > On Fri, Feb 21, 2025 at 04:42:09PM -0700, Simon Glass wrote: > > > Hi Tom, > > > > > > On Mon, 17 Feb 2025 at 07:14, Tom Rini wrote: > > > > > > > > On Mon, Feb 17, 20

[PATCH v3 14/23] arm: mvebu: a38x: Allocate bb_miiphy using bb_miiphy_alloc() and fill in callbacks

2025-02-22 Thread Marek Vasut
Allocate bb_miiphy using bb_miiphy_alloc() and fill in callbacks currently listed in bb_miiphy_buses[] array. This is a temporary duplication of assignment to avoid breakage, which will be removed in follow up patches. At this point, the bb_miiphy callbacks can reach these accessors by doing contai

Re: [PATCH v2 28/28] test: Add a test for booting Ubuntu 24.04

2025-02-22 Thread Heiko Schocher
Hello Tom, On 21.02.25 17:06, Tom Rini wrote: On Fri, Feb 21, 2025 at 06:57:34AM -0700, Simon Glass wrote: Hi Tom, On Thu, 20 Feb 2025 at 07:53, Tom Rini wrote: On Thu, Feb 20, 2025 at 06:49:49AM -0700, Simon Glass wrote: Hi Tom, On Tue, 18 Feb 2025 at 17:55, Tom Rini wrote: On Tue, Fe

[PATCH v3 11/23] net: designware: Extract bbmiiphy initialization into dedicated function

2025-02-22 Thread Marek Vasut
Pull the bbmiiphy initialization code from designware_eth_probe() into dedicated function, dw_bb_mdio_init(), just like all the other MDIO initialization functions. Keep check for "snps,bitbang-mii" in the designware_eth_probe(), so the driver can initialize this MDIO only in case the property is

[PATCH v3 16/23] net: sh_eth: Allocate bb_miiphy using bb_miiphy_alloc() and fill in callbacks

2025-02-22 Thread Marek Vasut
Allocate bb_miiphy using bb_miiphy_alloc() and fill in callbacks currently listed in bb_miiphy_buses[] array. This is a temporary duplication of assignment to avoid breakage, which will be removed in follow up patches. At this point, the bb_miiphy callbacks can reach these accessors by doing contai

[PATCH v3 10/23] net: designware: Drop bus index

2025-02-22 Thread Marek Vasut
There is literally one single bbmiiphy bus in this driver, remove the bus index handling. Signed-off-by: Marek Vasut --- Cc: Christian Marangi Cc: Ilias Apalodimas Cc: Jerome Forissier Cc: Jim Liu Cc: Joe Hershberger Cc: Mario Six Cc: Michal Simek Cc: Nobuhiro Iwamatsu Cc: Paul Barker Cc

[PATCH v3 23/23] net: miiphybb: Drop bb_miiphy_buses and bb_miiphy_buses_num

2025-02-22 Thread Marek Vasut
Neither bb_miiphy_buses nor bb_miiphy_buses_num are used anymore. Drop both of them. Reviewed-by: Paul Barker Signed-off-by: Marek Vasut --- Cc: Christian Marangi Cc: Ilias Apalodimas Cc: Jerome Forissier Cc: Jim Liu Cc: Joe Hershberger Cc: Mario Six Cc: Michal Simek Cc: Nobuhiro Iwamatsu

[PATCH v3 22/23] net: sh_eth: Drop use of miiphy_get_dev_by_name()

2025-02-22 Thread Marek Vasut
Instead of doing another lookup, trivially access the struct mii_dev embedded in struct bb_miiphy_bus . No functional change. Signed-off-by: Marek Vasut --- Cc: Christian Marangi Cc: Ilias Apalodimas Cc: Jerome Forissier Cc: Jim Liu Cc: Joe Hershberger Cc: Mario Six Cc: Michal Simek Cc: No

[PATCH v3 21/23] net: ravb: Drop use of miiphy_get_dev_by_name()

2025-02-22 Thread Marek Vasut
Instead of doing another lookup, trivially access the struct mii_dev embedded in struct bb_miiphy_bus . No functional change. Reviewed-by: Paul Barker Signed-off-by: Marek Vasut --- Cc: Christian Marangi Cc: Ilias Apalodimas Cc: Jerome Forissier Cc: Jim Liu Cc: Joe Hershberger Cc: Mario Six

[PATCH v3 20/23] arm: mvebu: a38x: Drop use of miiphy_get_dev_by_name()

2025-02-22 Thread Marek Vasut
Instead of doing another lookup, trivially access the struct mii_dev embedded in struct bb_miiphy_bus . No functional change. Signed-off-by: Marek Vasut --- Cc: Christian Marangi Cc: Ilias Apalodimas Cc: Jerome Forissier Cc: Jim Liu Cc: Joe Hershberger Cc: Mario Six Cc: Michal Simek Cc: No

[PATCH v3 18/23] net: miiphybb: Use container_of() in bb_miiphy_getbus()

2025-02-22 Thread Marek Vasut
Replace the name based look up in bb_miiphy_getbus() with trivial container_of() call. This works because the struct bb_miiphy_bus always embeds the matching struct mii_dev . This also makes the code much simpler and more efficient. Reviewed-by: Paul Barker Signed-off-by: Marek Vasut --- Cc: Chr

[PATCH v3 19/23] net: miiphybb: Drop name field from struct bb_miiphy_bus

2025-02-22 Thread Marek Vasut
The struct bb_miiphy_bus embeds struct struct mii_dev, which already contains one copy of name field. Drop the duplicate top level copy of name field. The a38x code does static assignment of disparate names, use snprintf(...) to fill in matching name in probe to avoid any breakage. Reviewed-by: P

[PATCH v3 17/23] net: designware: Allocate bb_miiphy using bb_miiphy_alloc() and fill in callbacks

2025-02-22 Thread Marek Vasut
Allocate bb_miiphy using bb_miiphy_alloc() and fill in callbacks currently listed in bb_miiphy_buses[] array. This is a temporary duplication of assignment to avoid breakage, which will be removed in follow up patches. At this point, the bb_miiphy callbacks can reach these accessors by doing contai

[PATCH v3 15/23] net: ravb: Allocate bb_miiphy using bb_miiphy_alloc() and fill in callbacks

2025-02-22 Thread Marek Vasut
Allocate bb_miiphy using bb_miiphy_alloc() and fill in callbacks currently listed in bb_miiphy_buses[] array. This is a temporary duplication of assignment to avoid breakage, which will be removed in follow up patches. At this point, the bb_miiphy callbacks can reach these accessors by doing contai

[PATCH v3 08/23] arm: mvebu: a38x: Call bb_miiphy init directly in driver probe

2025-02-22 Thread Marek Vasut
All the resources needed by this .init callback should already be available by the time probe function runs, simply call the init callback directly and set the bb_miiphy init callback to NULL. This shouldn't break anything on this hardware, but would be nice if someone could double-check and test t

[PATCH v3 13/23] net: miiphybb: Introduce bb_miiphy_alloc()/bb_miiphy_free() wrappers

2025-02-22 Thread Marek Vasut
Introduce bb_miiphy_alloc()/bb_miiphy_free() wrappers to allocate and free struct bb_miiphy_bus. Make struct bb_miiphy_bus wrap struct mii_dev, which will become useful later in bb_miiphy_bus accessors, which would be able to access struct bb_miiphy_bus using container_of, even if the PHY stack onl

[PATCH v3 12/23] net: miiphy: Introduce mdio_init()

2025-02-22 Thread Marek Vasut
Introduce mdio_init() split off from mdio_alloc(), which is used to initialize already allocated struct mii_dev. Reviewed-by: Paul Barker Signed-off-by: Marek Vasut --- Cc: Christian Marangi Cc: Ilias Apalodimas Cc: Jerome Forissier Cc: Jim Liu Cc: Joe Hershberger Cc: Mario Six Cc: Michal

[PATCH v3 09/23] net: miiphybb: Drop bb_miiphy_init() and .init callback

2025-02-22 Thread Marek Vasut
The .init callback is not called by any function, drop it. There are no more users of the init callback, drop the entire mechanism. Reviewed-by: Paul Barker Signed-off-by: Marek Vasut --- Cc: Christian Marangi Cc: Ilias Apalodimas Cc: Jerome Forissier Cc: Jim Liu Cc: Joe Hershberger Cc: Mar

[PATCH v3 07/23] net: designware: Reorder bb_miiphy functions

2025-02-22 Thread Marek Vasut
Move the bb_miiphy functions before MDIO registration. This is a preparatory patch, the functions will be referenced around the MDIO registration in the follow up patches. No functional change. Signed-off-by: Marek Vasut --- Cc: Christian Marangi Cc: Ilias Apalodimas Cc: Jerome Forissier Cc: J

[PATCH v3 06/23] arm: mvebu: a38x: Reorder bb_miiphy functions

2025-02-22 Thread Marek Vasut
Move the bb_miiphy functions before MDIO registration. This is a preparatory patch, the functions will be referenced around the MDIO registration in the follow up patches. No functional change. Signed-off-by: Marek Vasut --- Cc: Christian Marangi Cc: Ilias Apalodimas Cc: Jerome Forissier Cc: J

[PATCH v3 05/23] net: sh_eth: Reorder bb_miiphy functions

2025-02-22 Thread Marek Vasut
Move the bb_miiphy functions before MDIO registration. This is a preparatory patch, the functions will be referenced around the MDIO registration in the follow up patches. No functional change. Signed-off-by: Marek Vasut --- Cc: Christian Marangi Cc: Ilias Apalodimas Cc: Jerome Forissier Cc: J

[PATCH v3 04/23] net: ravb: Reorder bb_miiphy functions

2025-02-22 Thread Marek Vasut
Move the bb_miiphy functions before MDIO registration. This is a preparatory patch, the functions will be referenced around the MDIO registration in the follow up patches. No functional change. Reviewed-by: Paul Barker Signed-off-by: Marek Vasut --- Cc: Christian Marangi Cc: Ilias Apalodimas C

[PATCH v3 03/23] net: designware: Drop NULL priv assignment

2025-02-22 Thread Marek Vasut
This is unnecessary, the unset structure member is initialized to NULL by default, drop the assignment. Reviewed-by: Paul Barker Signed-off-by: Marek Vasut --- Cc: Christian Marangi Cc: Ilias Apalodimas Cc: Jerome Forissier Cc: Jim Liu Cc: Joe Hershberger Cc: Mario Six Cc: Michal Simek Cc

[PATCH v3 02/23] net: sh_eth: Drop empty init callback

2025-02-22 Thread Marek Vasut
The init function does nothing, the bb_miiphy_init() already checks whether the .init callback is assigned, and if not, skips calling it. Remove the empty init function. The entire init callback will be removed in follow up patches. Reviewed-by: Paul Barker Signed-off-by: Marek Vasut --- Cc: Chr

[PATCH v3 01/23] net: ravb: Drop empty init callback

2025-02-22 Thread Marek Vasut
The init function does nothing, the bb_miiphy_init() already checks whether the .init callback is assigned, and if not, skips calling it. Remove the empty init function. The entire init callback will be removed in follow up patches. Reviewed-by: Paul Barker Signed-off-by: Marek Vasut --- Cc: Chr

[PATCH v3 00/23] net: miiphybb: Get rid of global bb_miiphy_buses[]

2025-02-22 Thread Marek Vasut
Get rid of global bb_miiphy_buses[] which does not fit well with U-Boot DM in favor of usual per-ethernet-MAC-driver-instance MDIO bus accessors. Introduce the ability to allocate struct bb_miiphy_bus {} and make it wrap struct mii_dev {}, so the struct mii_dev can be passed into MDIO bus access i

Re: [PATCH] common: board_r: Initialize interrupts before watchdog

2025-02-22 Thread J . Neuschäfer
(CC'ing Mario Six regarding MPC83xx timer driver in U-Boot) On Thu, Feb 20, 2025 at 06:49:58AM -0700, Simon Glass wrote: > Hi J, > > On Tue, 18 Feb 2025 at 08:55, J. Neuschäfer via B4 Relay > wrote: > > > > From: "J. Neuschäfer" > > > > On some platforms, initializing the watchdog driver enable

Re: [PATCH v2 16/24] net: ravb: Allocate bb_miiphy using bb_miiphy_alloc() and fill in callbacks

2025-02-22 Thread Marek Vasut
On 2/17/25 12:10 PM, Paul Barker wrote: On 09/02/2025 12:01, Marek Vasut wrote: Allocate bb_miiphy using bb_miiphy_alloc() and fill in callbacks currently listed in bb_miiphy_buses[] array. This is a temporary duplication of assignment to avoid breakage, which will be removed in follow up patche

Re: Please pull u-boot-tegra picasso

2025-02-22 Thread Tom Rini
On Sat, 22 Feb 2025 09:39:41 +0200, Svyatoslav Ryhel wrote: > The following changes since commit 274dc5291cfbcb54cf54c337d2123adea075e299: > > Merge branch 'master' of https://source.denx.de/u-boot/custodians/u-boot-sh > (2025-02-15 15:29:20 -0600) > > are available in the Git repository at:

Re: [PATCH] buildman: Update tests for newer filelock module

2025-02-22 Thread Tom Rini
On Tue, 04 Feb 2025 16:33:53 -0700, Simon Glass wrote: > Recent versions of this module call time.perf_counter() so add a patch > for this also. > > Applied to u-boot/master, thanks! -- Tom

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

2025-02-22 Thread Simon Glass
Hi Tom, On Fri, 21 Feb 2025 at 17:08, Tom Rini wrote: > > On Fri, Feb 21, 2025 at 04:42:09PM -0700, Simon Glass wrote: > > Hi Tom, > > > > On Mon, 17 Feb 2025 at 07:14, Tom Rini wrote: > > > > > > On Mon, Feb 17, 2025 at 06:14:06AM -0700, Simon Glass wrote: > > > > Hi Tom, > > > > > > > > On Sun

[PATCH] ARM: stm32: Introduce DH STM32MP15xx DHSOM board specific defconfigs

2025-02-22 Thread Marek Vasut
Move stm32mp15_dhcom_basic_defconfig into stm32mp15_dhcom_basic.config. Retain legacy stm32mp15_dhcom_basic_defconfig as multi-config for all DH STM32MP15xx DHCOM based boards. Move stm32mp15_dhsor_basic_defconfig into stm32mp15_dhsor_basic.config. Retain stm32mp15_dhsor_basic_defconfig as multi-co

[PATCH] ARM: imx: Introduce DH i.MX6 DHSOM board specific defconfigs

2025-02-22 Thread Marek Vasut
Move content of dh_imx6_defconfig into dh_imx6.config. Retain legacy dh_imx6_defconfig as multi-config for all DH i.MX6 DHSOM based boards. Introduce separate imx6_dhcom_drc02_defconfig, imx6_dhcom_pdk2_defconfig and imx6_dhcom_picoitx_defconfig for each i.MX6 DHSOM based board, to make build for t

Re: [PATCH v2 28/28] test: Add a test for booting Ubuntu 24.04

2025-02-22 Thread Heiko Schocher
Hello Tom, just a fast update, testing with tbot running @gitlab works now too: https://source.denx.de/u-boot/custodians/u-boot-i2c/-/jobs/1034474#L2199 problem was: https://source.denx.de/u-boot/custodians/u-boot-i2c/-/commit/e32624a1320a5e75a7360903f6de75b14a61 So, I try to clean this u

[PATCH] net: dwc: xgmac: Allow DMA buffers above 4GB

2025-02-22 Thread Nikunj Kela
Currently, Synopsis xgmac driver only works if DMA region is under 4GB. This change enables the DMA buffers allocations above 4GB memory regions. Signed-off-by: Nikunj Kela --- drivers/net/dwc_eth_xgmac.c | 35 +++ 1 file changed, 23 insertions(+), 12 deletions(-)

Incorrect include path in cmd/setexpr.c

2025-02-22 Thread Ahmed Kilany
Hello U-Boot developers, I have noticed an issue in the file `cmd/setexpr.c` at line 13. The include path currently has:`#include ` It should be changed to: `#include ` Thank you. Best regards Ahmed Kilany

[PATCH 2/2] imx8mm: imx8mm_evk: fix BOOTSTD boot

2025-02-22 Thread Peng Fan (OSS)
From: Peng Fan Select BOOTSTD_FULL and BOOTSTD_BOOTCOMMAND Correct DEFAULT_FDT_FILE Correct env file for imx8mm_evk_fspi_defconfig Fixes: 364ba68ed1a ("imx: imx8mm_evk: Switch to BOOTSTD") Reported-by: Ludwig Nussel Signed-off-by: Peng Fan --- arch/arm/mach-imx/imx8m/Kconfig | 2 ++ configs

[PATCH 1/2] imx8mq: imx8mq_evk: fix DEFAULT_FDT_FILE

2025-02-22 Thread Peng Fan (OSS)
From: Peng Fan The CONFIG_DEFAULT_FDT_FILE should be imx8mq_evk.dtb for this board Fixes: 7050bd925f7 ("imx: imx8mq_evk: Switch to BOOTSTD") Reported-by: Ludwig Nussel Signed-off-by: Peng Fan --- configs/imx8mq_evk_defconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/

Re: [EXTERNAL] Re: [PATCH] spl: return kernel image header size in os boot

2025-02-22 Thread Anshul Dalal
On Sat Feb 22, 2025 at 1:34 AM IST, Tom Rini wrote: > On Fri, Feb 21, 2025 at 08:47:51PM +0530, Anshul Dalal wrote: > > > During kernel build process the header size is computed including the > > BSS whereas it's removed when creating the uncompressed image. Therefore > > the size of the uncompress