Re: [PATCH 8/8] sunxi: add support for the Allwinner A100/A133 SoC

2025-01-17 Thread Jernej Škrabec
Dne petek, 17. januar 2025 ob 02:45:37 Srednjeevropski standardni čas je Andre Przywara napisal(a): > The Allwinner A100 SoC has been around for a while, mostly on cheap > tablets, but didn't generate much interest in the community so far. > There were some efforts by two Allwinner employees in 20

Re: [PATCH 6/8] power: pmic: sunxi: add SPL support for the AXP803

2025-01-17 Thread Jernej Škrabec
Dne petek, 17. januar 2025 ob 02:45:35 Srednjeevropski standardni čas je Andre Przywara napisal(a): > The AXP803 has been around for about a decade now, but so far we didn't > need SPL support, since the DRAM rail was wired up correctly at reset. > > Now some boards using the A133 SoC use the (co

Re: [PATCH 5/8] pinctrl: sunxi: add Allwinner A100/A133 pinctrl description

2025-01-17 Thread Jernej Škrabec
Dne petek, 17. januar 2025 ob 02:45:34 Srednjeevropski standardni čas je Andre Przywara napisal(a): > The Allwinner A100 SoC has been around for a while, and has now seemingly > been replaced with its close sibling A133. > > Add the required mapping between the pinmux group strings and their > re

Re: [PATCH 4/8] clk: sunxi: Add support for the A100/A133 CCU

2025-01-17 Thread Jernej Škrabec
Dne petek, 17. januar 2025 ob 02:45:33 Srednjeevropski standardni čas je Andre Przywara napisal(a): > The Allwinner A100 SoC has been around for a while, and has now seemingly > been replaced with its close sibling A133. > > Add support for the CCU, as far as used by U-Boot proper. Linux has some

Re: [PATCH 3/8] sunxi: H616: DRAM: rename Kconfig parameters to be more generic

2025-01-17 Thread Jernej Škrabec
Dne petek, 17. januar 2025 ob 02:45:32 Srednjeevropski standardni čas je Andre Przywara napisal(a): > The H616 DRAM controller requires some board specific parameters, which > we declare in Kconfig, let each board specify in their defconfig, and > then use in the DRAM init code. > > Other DRAM co

Re: [PATCH 2/8] sunxi: pmic_bus: support alternative I2C address

2025-01-17 Thread Jernej Škrabec
Dne petek, 17. januar 2025 ob 02:45:31 Srednjeevropski standardni čas je Andre Przywara napisal(a): > Some of the X-Power AXP PMICs can be ordered with an alternative I2C > address, for instance an AXP717 could be shipped with address 0x34 or > with address 0x35. > The datasheets for the AXP717 an

Re: [PATCH 1/8] sunxi: clock: improve grouping of default clock register values

2025-01-17 Thread Jernej Škrabec
Dne petek, 17. januar 2025 ob 02:45:30 Srednjeevropski standardni čas je Andre Przywara napisal(a): > With each new SoC added to the clock_sun50i_h6.h header file, we add a > list of default values for the bus clock registers. This list gets a bit > hard to read, as the spacing between the lines l

[PATCH v3] arm64: dts: renesas: Deduplicate extalr_clk bootph-all

2025-01-17 Thread Marek Vasut
Most R-Car Gen3 and Gen4 systems require extalr_clk very early in the boot process. Currently the extalr_clk { bootph-all } DT property to indicate this clock should be available early is set in each SoC U-Boot DT extras file. Deduplicate this assignment to new common r8a779x-rcar64-u-boot.dtsi fil

Re: [PATCH 09/14] net: ravb: Support up to two instances

2025-01-17 Thread Marek Vasut
On 12/5/24 8:12 PM, Paul Barker wrote: Hi Marek, On 28/10/2024 09:50, Paul Barker wrote: On 27/10/2024 16:25, Marek Vasut wrote: On 10/24/24 5:24 PM, Paul Barker wrote: int ravb_of_to_plat(struct udevice *dev) { struct eth_pdata *pdata = dev_get_plat(dev); - const fdt32_t

Re: [PATCH 09/14] net: ravb: Support up to two instances

2025-01-17 Thread Marek Vasut
On 10/28/24 10:50 AM, Paul Barker wrote: On 27/10/2024 16:25, Marek Vasut wrote: On 10/24/24 5:24 PM, Paul Barker wrote: Several Renesas SoCs in the RZ/G2L family have two Ethernet interfaces. To support this second interface, we extend the bb_miiphy_buses[] array and keep track of the current

Re: [PATCH] Makefile: Make sure all linker input objects exist

2025-01-17 Thread Marek Vasut
On 1/15/25 11:38 AM, Quentin Schulz wrote: Hi Marek, Hi, On 1/12/25 11:32 PM, Marek Vasut wrote: In case the build system builds a directory with empty Makefile, one which does not contain any obj-y entries, the build fails to link due to missing built-in.o . This happens because of this pa

[PATCH 01/17] miiphy: Introduce mdio_init()

2025-01-17 Thread Marek Vasut
Introduce mdio_init() split off from mdio_alloc(), which is used to initialize already allocated struct mii_dev. Signed-off-by: Marek Vasut --- Cc: Christian Marangi Cc: Evgeny Bachinin Cc: Ilias Apalodimas Cc: Jerome Forissier Cc: Joe Hershberger Cc: Mario Six Cc: Michal Simek Cc: Nobuhir

[PATCH 17/17] net: miiphybb: Drop .init callback

2025-01-17 Thread Marek Vasut
The .init callback is not called by any function, drop it. Signed-off-by: Marek Vasut --- Cc: Christian Marangi Cc: Evgeny Bachinin Cc: Ilias Apalodimas Cc: Jerome Forissier Cc: Joe Hershberger Cc: Mario Six Cc: Michal Simek Cc: Nobuhiro Iwamatsu Cc: Paul Barker Cc: Ramon Fried Cc: Simo

[PATCH 16/17] net: miiphybb: Drop bb_miiphy_buses and bb_miiphy_buses_num

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

[PATCH 15/17] net: sh_eth: Drop use of miiphy_get_dev_by_name()

2025-01-17 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: Evgeny Bachinin Cc: Ilias Apalodimas Cc: Jerome Forissier Cc: Joe Hershberger Cc: Mario Six Cc: Michal Simek

[PATCH 14/17] net: ravb: Drop use of miiphy_get_dev_by_name()

2025-01-17 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: Evgeny Bachinin Cc: Ilias Apalodimas Cc: Jerome Forissier Cc: Joe Hershberger Cc: Mario Six Cc: Michal Simek

[PATCH 13/17] arm: mvebu: a38x: Drop use of miiphy_get_dev_by_name()

2025-01-17 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: Evgeny Bachinin Cc: Ilias Apalodimas Cc: Jerome Forissier Cc: Joe Hershberger Cc: Mario Six Cc: Michal Simek

[PATCH 12/17] net: miiphybb: Drop bb_miiphy_init()

2025-01-17 Thread Marek Vasut
There are no more users of the init callback, drop the entire mechanism. Signed-off-by: Marek Vasut --- Cc: Christian Marangi Cc: Evgeny Bachinin Cc: Ilias Apalodimas Cc: Jerome Forissier Cc: Joe Hershberger Cc: Mario Six Cc: Michal Simek Cc: Nobuhiro Iwamatsu Cc: Paul Barker Cc: Ramon F

[PATCH 11/17] arm: mvebu: a38x: Call bb_miiphy init directly in driver probe

2025-01-17 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 10/17] net: miiphybb: Drop name field from struct bb_miiphy_bus

2025-01-17 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. Signed-off-by:

[PATCH 07/17] net: ravb: Allocate bb_miiphy using bb_miiphy_alloc() and fill in callbacks

2025-01-17 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 09/17] net: miiphybb: Use container_of() in bb_miiphy_getbus()

2025-01-17 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. Signed-off-by: Marek Vasut --- Cc: Christian Marangi Cc: Evgeny

[PATCH 08/17] net: sh_eth: Allocate bb_miiphy using bb_miiphy_alloc() and fill in callbacks

2025-01-17 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 06/17] arm: mvebu: a38x: Allocate bb_miiphy using bb_miiphy_alloc() and fill in callbacks

2025-01-17 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 05/17] arm: mvebu: a38x: Reorder bb_miiphy functions

2025-01-17 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: Evgeny Bachinin Cc: Ilias Apalodimas Cc: Je

[PATCH 04/17] net: sh_eth: Reorder bb_miiphy functions

2025-01-17 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: Evgeny Bachinin Cc: Ilias Apalodimas Cc: Je

[PATCH 02/17] net: miiphybb: Introduce bb_miiphy_free()/bb_miiphy_free() wrappers

2025-01-17 Thread Marek Vasut
Introduce bb_miiphy_free()/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 only

[PATCH 03/17] net: ravb: Reorder bb_miiphy functions

2025-01-17 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: Evgeny Bachinin Cc: Ilias Apalodimas Cc: Je

[PATCH 2/2] net: sh_eth: Drop empty init callback

2025-01-17 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. Signed-off-by: Marek Vasut --- Cc: Christian Marangi Cc: Evgeny

[PATCH 1/2] net: ravb: Drop empty init callback

2025-01-17 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. Signed-off-by: Marek Vasut --- Cc: Christian Marangi Cc: Evgeny

[PATCH] net: miiphybb: Convert ifdef DEBUG to debug()

2025-01-17 Thread Marek Vasut
Replace ifdeffery with plain debug() function call. No functional change. Signed-off-by: Marek Vasut --- Cc: Adam Ford Cc: Hai Pham Cc: Ilias Apalodimas Cc: Jerome Forissier Cc: Joe Hershberger Cc: Mario Six Cc: Nobuhiro Iwamatsu Cc: Paul Barker Cc: Peter Robinson Cc: Ramon Fried Cc: Su

[PATCH] net: miiphybb: Fix short name length

2025-01-17 Thread Marek Vasut
Multiple drivers copy MDIO name into struct bb_miiphy_bus name, but these two name fields have different length, the bb_miiphy_bus one is 16 bytes, the source one is 32 bytes. Make sure these two lengths are always synchronized, use MDIO_NAME_LEN for both. Signed-off-by: Marek Vasut --- Cc: Adam

[PATCH] net: miiphybb: configs: Drop CONFIG_BITBANGMII_MULTI

2025-01-17 Thread Marek Vasut
It seems that every remaining system which enables BITBANGMII also enables BITBANGMII_MULTI . Remove the BITBANGMII_MULTI symbol and assume it is always enabled. This allows removal of a bit of legacy code. No functional change intended. Signed-off-by: Marek Vasut --- Cc: Adam Ford Cc: Hai Pham

[PATCH] net: ravb: Staticize bb_miiphy functions

2025-01-17 Thread Marek Vasut
These functions can be static as they are referenced only in this file. Make them static. No functional change. Signed-off-by: Marek Vasut --- Cc: Adam Ford Cc: Hai Pham Cc: Ilias Apalodimas Cc: Jerome Forissier Cc: Joe Hershberger Cc: Mario Six Cc: Nobuhiro Iwamatsu Cc: Paul Barker Cc: P

Re: [PATCH v3 00/19] bootstd: Support recording images

2025-01-17 Thread Tony Dinh
On Fri, Jan 17, 2025 at 8:32 PM Simon Glass wrote: > > Hi Tom, > > On Thu, 16 Jan 2025 at 10:22, Tom Rini wrote: > > > > On Thu, Jan 16, 2025 at 08:52:38AM -0700, Simon Glass wrote: > > > Hi Tom, > > > > > > On Wed, 15 Jan 2025 at 16:32, Tom Rini wrote: > > [snip] > > > > I'm referring to the wh

Re: [PATCH v3 00/19] bootstd: Support recording images

2025-01-17 Thread Simon Glass
Hi Tom, On Thu, 16 Jan 2025 at 10:22, Tom Rini wrote: > > On Thu, Jan 16, 2025 at 08:52:38AM -0700, Simon Glass wrote: > > Hi Tom, > > > > On Wed, 15 Jan 2025 at 16:32, Tom Rini wrote: > [snip] > > > I'm referring to the whole bootmeth/bootflow/etc thing. Is this going to > > > be put to use by

Re: [PATCH 0/8] efi_loader: Complete the bootflow_efi() test

2025-01-17 Thread Simon Glass
Hi Tom, On Mon, 13 Jan 2025 at 13:20, Tom Rini wrote: > > On Mon, Jan 13, 2025 at 12:01:36PM -0700, Simon Glass wrote: > > Hi Tom, > > > > On Fri, 10 Jan 2025 at 09:48, Tom Rini wrote: > > > > > > On Fri, Jan 10, 2025 at 06:40:37AM -0700, Simon Glass wrote: > > > > Hi Tom, > > > > > > > > On Thu

Re: [PATCH 4/8] efi_loader: Correct bounce-buffer setup

2025-01-17 Thread Simon Glass
Hi Ilias, On Fri, 17 Jan 2025 at 03:34, Ilias Apalodimas wrote: > > Hi Simon, > > Is this patch for -master of your tree? Yes, but I haven't applied it yet. The updated bootflow_efi() test ([1] which this is part of) is throwing up problems with how testing is done. We cannot today reinit EFI so

Re: [PATCH v2 00/22] vbe: Series part F

2025-01-17 Thread Simon Glass
Hi Tom, On Thu, 16 Jan 2025 at 17:00, Tom Rini wrote: > > On Wed, Jan 15, 2025 at 06:27:01PM -0700, Simon Glass wrote: > > > This includes various patches towards implementing the VBE abrec > > bootmeth in U-Boot. It mostly focuses on introducing a relocating > > SPL-loader so that VBE can run in

Re: [PATCH 1/2] dm: clarify DM_FLAG_PROBE_AFTER_BIND behaviour

2025-01-17 Thread Simon Glass
Hi Caleb, On Fri, 17 Jan 2025 at 00:29, Caleb Connolly wrote: > > The DM_FLAG_PROBE_AFTER_BIND flag only makes sense on a per-device > basis, however recently added documentation as well as some confused > drivers imply that it might be added to a driver definition, this does > nothing. > > Clari

Re: [PATCH 2/2] drivers: remove bogus DM_FLAG_PROBE_AFTER_BIND flags

2025-01-17 Thread Simon Glass
On Fri, 17 Jan 2025 at 00:29, Caleb Connolly wrote: > > Some drivers set DM_FLAG_PROBE_AFTER_BIND, this does nothing since it's > only every applied on a per-device basis. > > Remove the flags. > > Signed-off-by: Caleb Connolly > --- > drivers/mailbox/zynqmp-ipi.c | 1 - > drivers/watchdog/da90

Re: [PATCH 2/6] tmp: add TPM2_PCR_Allocate command

2025-01-17 Thread Simon Glass
Hi Raymond, On Wed, 15 Jan 2025 at 13:02, Raymond Mao wrote: > > TPM2_PCR_Allocate command is required to re-configurate a TPM device > to enable or disable algorithms in run-time, thus this patch introduces > the implementation of PCR allocate APIs and adds related cmd functions > for testing. >

Re: [PATCH v5 3/3] tpm: get tpm event log from bloblist

2025-01-17 Thread Simon Glass
Hi Ilias, On Thu, 16 Jan 2025 at 07:35, Ilias Apalodimas wrote: > > On Wed, 15 Jan 2025 at 03:17, Simon Glass wrote: > > > > Hi Raymond, > > > > On Tue, 14 Jan 2025 at 08:20, Raymond Mao wrote: > > > > > > Get tpm event log from bloblist instead of FDT when bloblist is > > > enabled and valid f

[PATCH 6/8] mmc: Handle return value from cyclic_register()

2025-01-17 Thread Marek Vasut
Handle the error code returned by cyclic_register() and propagate it, except in case the return code is EALREADY. The cyclic_register() in mmc.c can be called multiple times with the same parameters, and that is not an error, so depend on the cyclic_register() to skip such a repeated registration a

[PATCH 8/8] wdt: Handle return value from cyclic_register()

2025-01-17 Thread Marek Vasut
Handle the error code returned by cyclic_register() and propagate it. Signed-off-by: Marek Vasut --- Cc: Aaron Williams Cc: Anatolij Gustschin Cc: Angelo Dureghello Cc: Christian Marangi Cc: Devarsh Thakkar Cc: Heinrich Schuchardt Cc: Jaehoon Chung Cc: Michael Polyntsov Cc: Michael Trimar

[PATCH 7/8] video: Handle return value from cyclic_register()

2025-01-17 Thread Marek Vasut
Handle the error code returned by cyclic_register() and propagate it. Signed-off-by: Marek Vasut --- Cc: Aaron Williams Cc: Anatolij Gustschin Cc: Angelo Dureghello Cc: Christian Marangi Cc: Devarsh Thakkar Cc: Heinrich Schuchardt Cc: Jaehoon Chung Cc: Michael Polyntsov Cc: Michael Trimar

[PATCH 5/8] led: Handle return value from cyclic_register()

2025-01-17 Thread Marek Vasut
Handle the error code returned by cyclic_register() and propagate it. Signed-off-by: Marek Vasut --- Cc: Aaron Williams Cc: Anatolij Gustschin Cc: Angelo Dureghello Cc: Christian Marangi Cc: Devarsh Thakkar Cc: Heinrich Schuchardt Cc: Jaehoon Chung Cc: Michael Polyntsov Cc: Michael Trimar

[PATCH 4/8] mips: octeon: nic23: Handle return value from cyclic_register()

2025-01-17 Thread Marek Vasut
Handle the error code returned by cyclic_register(). Report the error, but do not return it from board_late_init() as that would make the board not boot, which is undesired. Let the board boot somehow, so the user can fix the problem, even if the cyclic callback is not usable. Signed-off-by: Marek

[PATCH 3/8] cyclic: Test return code from cyclic_register() in test

2025-01-17 Thread Marek Vasut
Update the cyclic test code to verify return code from cyclic_register() works. Signed-off-by: Marek Vasut --- Cc: Aaron Williams Cc: Anatolij Gustschin Cc: Angelo Dureghello Cc: Christian Marangi Cc: Devarsh Thakkar Cc: Heinrich Schuchardt Cc: Jaehoon Chung Cc: Michael Polyntsov Cc: Mich

[PATCH 2/8] cyclic: Handle return code from cyclic_register() in demo and docs

2025-01-17 Thread Marek Vasut
Update the cyclic demo code and documentation to indicate there is now a return code from cyclic_register(). Signed-off-by: Marek Vasut --- Cc: Aaron Williams Cc: Anatolij Gustschin Cc: Angelo Dureghello Cc: Christian Marangi Cc: Devarsh Thakkar Cc: Heinrich Schuchardt Cc: Jaehoon Chung Cc

[PATCH 1/8] cyclic: Prevent corruption of cyclic list on reassignment

2025-01-17 Thread Marek Vasut
Make cyclic_register() return error code, 0 in case of success, -EALREADY in case the called attempts to re-register already registered struct cyclic_info. The re-registration would lead to corruption of gd->cyclic_list because the re-registration would memset() one of its nodes, prevent that. Unre

[PATCH v1] arm: Remove redundant loading of image copy start offse

2025-01-17 Thread 1425075683
From: Liya Huang <1425075...@qq.com> Remove redundant loading of image copy start offset. Because the beginning is already computed Reviewed-by: Pali Rohár Reviewed-by: Simon Glass Signed-off-by: Liya Huang <1425075...@qq.com> --- Changes in v1: - Due to an error, the patch was resubmitted, le

[PATCH] mmc: Exit from mmc_init() if mmc_complete_init() fails

2025-01-17 Thread Marek Vasut
In case mmc_complete_init() returns error, exit from mmc_init() without possibly calling cyclic_register(), which at that point would be undesired. Signed-off-by: Marek Vasut --- Cc: Heinrich Schuchardt Cc: Jaehoon Chung Cc: Jonas Karlman Cc: Kever Yang Cc: Peng Fan Cc: Quentin Schulz Cc: R

[PATCH v2] mmc: Simplify poll CD logic in case cyclic framework is enabled

2025-01-17 Thread Marek Vasut
Simplify 90cc07fd786d ("mmc: Poll CD in case cyclic framework is enabled") according to suggestions by Rasmus. The struct cyclic_info is zero-size in case CONFIG_CYCLIC is not enabled and does not add any size to struct mmc, so it can unconditionally be part of that structure. This allows clean up

[PATCH v1] linker_lists:Update the alignment using CONFIG_LINKER_LIST_ALIGN

2025-01-17 Thread 1425075683
From: Liya Huang <1425075...@qq.com> This patch updates the alignment of linker lists to use the CONFIG_LINKER_LIST_ALIGN macro instead of a hardcoded value. This ensures that the alignment is consistent with the configuration. Replace __attribute__((unused)) with __maybe_unused and __always_unuse

Re: [PATCH] mmc: Simplify poll CD logic in case cyclic framework is enabled

2025-01-17 Thread Marek Vasut
On 1/18/25 4:08 AM, Marek Vasut wrote: Simplify 90cc07fd786d ("mmc: Poll CD in case cyclic framework is enabled") according to suggestions by Rasmus. The struct cyclic_info is zero-size in case CONFIG_CYCLIC is not enabled and does not add any size to struct mmc, so it can unconditionally be part

[PATCH] mmc: Simplify poll CD logic in case cyclic framework is enabled

2025-01-17 Thread Marek Vasut
Simplify 90cc07fd786d ("mmc: Poll CD in case cyclic framework is enabled") according to suggestions by Rasmus. The struct cyclic_info is zero-size in case CONFIG_CYCLIC is not enabled and does not add any size to struct mmc, so it can unconditionally be part of that structure. This allows clean up

Re: [PATCH v4 0/2] Static initcalls

2025-01-17 Thread Tom Rini
On Mon, Jan 06, 2025 at 03:40:12PM +0100, Jerome Forissier wrote: > This series replaces the dynamic initcalls (with function pointers) with > static calls, and gets rid of initcall_run_list(), init_sequence_f, > init_sequence_f_r and init_sequence_r. This makes the code simpler and the > binary s

Environment CRC mismatch

2025-01-17 Thread Floyd Brown
The CRC generated by mkenvimage is not valid when written to the storage device where uboot wants it. I get a "Loading Environment from MMC... *** Warning - bad CRC, using default environment" when I insert a SD card with U-boot and the environment I built. The environment is stored at the correct

[PATCH 1/1] efi_loader: reduce UEFI size if HAS_BOARD_SIZE_LIMIT=y

2025-01-17 Thread Heinrich Schuchardt
If a board has a strict size limit, disable * Unicode capitalization and * HII protocols by default to reduce the image size. Signed-off-by: Heinrich Schuchardt --- lib/efi_loader/Kconfig | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/efi_loader/Kconfig b/lib/efi_l

Re: [PATCH v4 3/7] efi_loader: Update efi_run_image() to accept image and device path

2025-01-17 Thread Heinrich Schuchardt
On 1/9/25 16:02, Simon Glass wrote: Provide these globals as parameters to this function, on the way to making it possible to start an image without relying on the globals. Signed-off-by: Simon Glass Reviewed-by: Ilias Apalodimas --- (no changes since v2) Changes in v2: - Adjust argument ord

Re: [PATCH v4 4/7] efi_loader: Add a version of efi_binary_run() with more parameters

2025-01-17 Thread Heinrich Schuchardt
On 1/9/25 16:02, Simon Glass wrote: This uses a few global variables at present. With the bootflow we have the required parameters, so add a function which accepts these. Update the existing function to call the new one with the globals. Signed-off-by: Simon Glass Reviewed-by: Ilias Apalodimas

[PATCH] lib: uuid: support more efi protocols in uuid_guid_get_str()

2025-01-17 Thread Vincent Stehlé
Add more EFI protocols GUIDs to the translation table used by uuid_guid_get_str(). Signed-off-by: Vincent Stehlé Cc: Tom Rini --- Hi, While the EFI_LOADED_IMAGE_DEVICE_PATH_PROTOCOL_GUID, the EFI_HII_CONFIG_ACCESS_PROTOCOL_GUID and the EFI_LOAD_FILE_PROTOCOL_GUID are actually used by the efi_l

Re: [XGMAC dt bindings v1 1/1] dt: net: add DWC XGMAC binding

2025-01-17 Thread Tom Rini
On Fri, Jan 17, 2025 at 11:15:34AM +0800, Boon Khai Ng wrote: > The Synopsys DWC XGMAC is a configurable Ethernet MAC/DMA IP block which > supports multiple options for bus type, clocking and reset structure, and > feature list. > > Signed-off-by: Boon Khai Ng > --- > .../net/snps,dwc-xgmac-eth

Re: [dt xgmac binding 0/1] Add Synopsys DWC XGMAC Binding

2025-01-17 Thread Tom Rini
On Fri, Jan 17, 2025 at 10:54:04AM +0800, Boon Khai Ng wrote: > The proposed patch introduces a new binding for the > Synopsys DWC 10G Ethernet MAC IP block(XGMAC). This block is > configurable and supports a wide range of options, including > various bus types, clocking and reset structures, and

Re: [GIT PULL] u-boot-riscv/master

2025-01-17 Thread Tom Rini
On Fri, Jan 17, 2025 at 09:53:45AM +0800, Leo Liang wrote: > Hi Tom, > > The following changes since commit 178f6ecb21fe12ada74a9a1a08093c812b15eea5: > > Merge patch series "bootstd: Support recording images" (2025-01-15 19:27:14 > -0600) > > are available in the Git repository at: > > ht

Re: [RFC] Adding bootsource and reset-source* to /chosen

2025-01-17 Thread Quentin Schulz
Hi Rob, On 1/14/25 9:36 PM, Rob Herring wrote: On Fri, Dec 13, 2024 at 12:08 PM Quentin Schulz wrote: Hi all, In U-Boot, there are multiple stages which can be loaded from different media each. On Rockchip SoCs, it mostly goes this way, SoC BootROM loads from a medium (storage or over USB) a

[PATCH v2 6/6] board: qemu-arm: select TPM_PCR_ALLOCATE

2025-01-17 Thread Raymond Mao
Enable PCR allocate and system reboot when algorithms don't match among the values from TPM device, U-Boot support and evenlog. Signed-off-by: Raymond Mao Reviewed-by: Ilias Apalodimas --- changes in v2 - None. configs/qemu_arm64_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/co

[PATCH v2 5/6] tpm: PCR allocate during PCR extend to disable the unsupported algorithms

2025-01-17 Thread Raymond Mao
During PCR extend process, if any unsupported algorithms are active, try to use PCR allocate to inactivate them. Signed-off-by: Raymond Mao --- changes in v2 - None. lib/tpm-v2.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/lib/tpm-v2.c b/lib/tpm-v2.c index 0e497b2c2a

[PATCH v2 4/6] tpm: add PCR allocate into the eventlog handling

2025-01-17 Thread Raymond Mao
Get eventlog bank mask when parsing the eventlog from previous boot stage and invoke PCR allocate function based on it. PCR allocate will be proceeded if any dismatches observed among the active, supportted and eventlog bank masks to re-configurate the TPM with the proper algorithms and reboot the

[PATCH v2 3/6] tpm: add wrapper and helper APIs for PCR allocate

2025-01-17 Thread Raymond Mao
Add PCR allocate wrapper APIs for using in tcg2 protocol. The wrapper proceeds a PCR allocate command, followed by a shutdown command. A system boot is required after two commands since TPM device needs a HW reset to activate the new algorithms config. Also, a helper function is included to determi

[PATCH v2 2/6] tpm: add TPM2_PCR_Allocate command

2025-01-17 Thread Raymond Mao
TPM2_PCR_Allocate command is required to reconfigure a TPM device to enable or disable algorithms in run-time, thus this patch introduces the implementation of PCR allocate APIs and adds related cmd functions for testing. To test the feature, ensure that TPM is started up. Run pcr_allocate command

[PATCH v2 1/6] tpm: add TPM2_Shutdown command

2025-01-17 Thread Raymond Mao
TPM2_shutdown command is sharing same structure and logics with TPM2_startup, thus this patch extends the existing startup APIs and cmd functions to support shutdown instead of created new ones. Signed-off-by: Raymond Mao Reviewed-by: Ilias Apalodimas --- changes in v2 - None. cmd/tpm-v2.c

[PATCH v2 0/6] Reconfigure TPM when active hash algorithms dismatch

2025-01-17 Thread Raymond Mao
This patch set implements PCR allocate command to handle the algorithm dismatches among TPM device, eventlog from previous boot stage and what U-Boot supports. It re-configurates TPM device if any active algorithms are not supported by U-Boot or does not exist in the eventlog passed in. To re-con

[PATCH v2 2/2] ARM: dts: imxrt1050: Migrate to OF_UPSTREAM

2025-01-17 Thread Jesse Taube
The device tree for imxrt1050 is now available in the /dts/upstream directory. Migrate board to use OF_UPSTREAM. Signed-off-by: Jesse Taube --- V1 -> V2: - No change --- arch/arm/dts/imxrt1050-evk.dts | 72 -- arch/arm/dts/imxrt1050-pinfunc.h | 993 --- arch/a

[PATCH v2 1/2] ARM: dts: imxrt1050: Fix clocks for mmc

2025-01-17 Thread Jesse Taube
One of the usdhc1 controller's clocks should be IMXRT1050_CLK_AHB_PODF not IMXRT1050_CLK_OSC. Fixes: 1c4f01be3490 ("ARM: dts: imx: Add i.MXRT1050-EVK support") Signed-off-by: Jesse Taube Signed-off-by: Shawn Guo [ upstream commit: 5f122030061db3e5d2bddd9cf5c583deaa6c54ff ] (cherry picked from

[PATCH v6 3/3] tpm: get tpm event log from bloblist

2025-01-17 Thread Raymond Mao
Get tpm event log from bloblist instead of FDT when bloblist is enabled and valid from previous boot stage. Signed-off-by: Raymond Mao --- Changes in v2 - Remove patch dependency. - Remove the fallback to FDT when BLOBLIST is selected. Changes in v3 - Malloc an 8KB buffer when user eventlog buffe

[PATCH v6 2/3] tcg2: decouple eventlog size from efi

2025-01-17 Thread Raymond Mao
Move default eventlog size from efi to tpm for using in both efi and measured boot. Signed-off-by: Raymond Mao Reviewed-by: Ilias Apalodimas --- Changes in v5 - Move eventlog size kconfig from EFI to TPM2, so that it can be used by both EFI and MEASURED_BOOT. Changes in v6 - None. doc/usage/

[PATCH v6 1/3] bloblist: add api to get blob with size

2025-01-17 Thread Raymond Mao
bloblist_find function only returns the pointer of blob data, which is fine for those self-describing data like FDT. But as a common scenario, an interface is needed to retrieve both the pointer and the size of the blob data. Add a few ut test cases for the new api. Signed-off-by: Raymond Mao Re

Re: [PATCH v3 1/4] debug_uart: Replace debug functions with dummies if CONFIG_DEBUG_UART is not set

2025-01-17 Thread Quentin Schulz
Hi Łukasz, Kever, On 11/27/24 11:21 PM, Łukasz Czechowski wrote: Hi Kever, On 2024/11/27 03:42 Kever Yang wrote: Hi Lukasz, I got a new error base on patch [1], see full log here [2]. Looking at the file efi_stub.c that is used in the failing configuration it looks to me that some f

Re: [PATCH next RFC 0/5] rockchip: panic() when DRAM init fails

2025-01-17 Thread Quentin Schulz
Hi Kever, On 11/5/24 6:21 PM, Quentin Schulz wrote: I am the unfortunate owner of an RK3399 Puma that is failing DRAM init every now and then with the following cryptic error message: """ pctl_start: Failed to init pctl channel 0 """ When it fails, the current logic is implemented in such a way

Re: [PATCH] ARM: dts: imxrt1050: Migrate to OF_UPSTREAM

2025-01-17 Thread Tom Rini
On Fri, Jan 17, 2025 at 03:22:08AM +, Peng Fan wrote: > > Subject: [PATCH] ARM: dts: imxrt1050: Migrate to OF_UPSTREAM > > > > The device tree for imxrt1050 is now > > available in the /dts/upstream directory. > > Migrate board to use OF_UPSTREAM. > > > > Signed-off-by: Jesse Taube > > --- >

[PATCH v4 2/3] ARM: configs: nuvoton: add bitbang feature for npcm8xx.

2025-01-17 Thread Michael Chang
Enable bitbang and multiple bitbang feature for npcm8xx platform. Signed-off-by: Jim Liu Signed-off-by: Michael Chang --- configs/arbel_evb_defconfig | 4 1 file changed, 4 insertions(+) diff --git a/configs/arbel_evb_defconfig b/configs/arbel_evb_defconfig index 4a58146e61..c93fa34eb5 10

[PATCH v4 3/3] net: designware: Add bitbang feature for designware driver.

2025-01-17 Thread Michael Chang
Add bb_miiphy_bus function for designware bitbang feature. Signed-off-by: Jim Liu Signed-off-by: Michael Chang --- drivers/net/designware.c | 113 +++ drivers/net/designware.h | 6 ++- 2 files changed, 118 insertions(+), 1 deletion(-) diff --git a/drivers/

[PATCH v4 1/3] ARM: dts: nuvoton: Add bitbang delay through dts properties.

2025-01-17 Thread Michael Chang
Add bitbang delay through dts properties. Signed-off-by: Jim Liu Signed-off-by: Michael Chang --- arch/arm/dts/nuvoton-npcm845-evb.dts | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm/dts/nuvoton-npcm845-evb.dts b/arch/arm/dts/nuvoton-npcm845-evb.dts index 0d3aaa0fff..1535defe38 10

[PATCH v4 0/3] Add bitbang feature for npcm8xx and driver

2025-01-17 Thread Michael Chang
I am resubmitting the patch titled "Add bitbang feature for npcm8xx and driver" for review and inclusion in the upstream project. Driver didn't support bitbang feature. Add bb_miiphy_bus function for driver and open feature for npcm8xx the log is as below: ---

[U-Boot][PATCH v3 4/4] fpga: xilinx exclusive commands

2025-01-17 Thread Ibai Erkiaga
Ensure all Xilinx exclusive fpga commands have a KConfig symbol and dependency to FPGA_XILINX listed. Remove (Xilinx only) text from the help command. Signed-off-by: Ibai Erkiaga --- (no changes since v1) cmd/Kconfig | 16 +++- cmd/fpga.c | 9 + 2 files changed, 16 insert

[U-Boot][PATCH v3 3/4] fpga: resort fpga commands

2025-01-17 Thread Ibai Erkiaga
Resort the fpga commands both in the Kconfig and in the source code to list Signed-off-by: Ibai Erkiaga --- (no changes since v1) cmd/Kconfig | 14 +++--- cmd/fpga.c | 4 ++-- 2 files changed, 9 insertions(+), 9 deletions(-) diff --git a/cmd/Kconfig b/cmd/Kconfig index 1d7ddb4ed36..

[U-Boot][PATCH v3 0/4] Improve fpga cmd usage information

2025-01-17 Thread Ibai Erkiaga
Current usage information is not read friendly specially when all the options are selected. This series fix some of the issues in the info, aligment and overlapping issues, excess of verbosity on the loads... Additionally Xilinx exclusive command dependencies are included to remove tex

[U-Boot][PATCH v3 2/4] fpga: improve loads usage information

2025-01-17 Thread Ibai Erkiaga
Current usage information for loads command is too verbose and long for a command usage prompt. This flag simplifies the text for readability purposes. Signed-off-by: Ibai Erkiaga --- (no changes since v1) cmd/fpga.c | 21 - 1 file changed, 8 insertions(+), 13 deletions(-)

[U-Boot][PATCH v3 1/4] fpga: fix alignment on fpga cmd usage info

2025-01-17 Thread Ibai Erkiaga
The current implementation generates some alignment issues as well as some overlapping when all the fpga command options are enabled. The fix is intended to improve readability of the usage info. Signed-off-by: Ibai Erkiaga --- Changes in v3: - Split 3rd patch into two, resorting commands and Kc

[U-Boot][PATCH v2 3/3] fpga: xilinx exclusive commands info

2025-01-17 Thread Ibai Erkiaga
Remove statements of Xilinx Only from the fpga command information and ensure Kconfig dependencies to FPGA_XILINX are implemented instead. Signed-off-by: Ibai Erkiaga --- (no changes since v1) Due to an issue with patman I had to add an empty line on the CONFIG_FPGA_LOADB description --- ---

[U-Boot][PATCH v2 1/3] fpga: fix alignment on fpga cmd usage info

2025-01-17 Thread Ibai Erkiaga
The current implementation generates some alignment issues as well as some overlapping when all the fpga command options are enabled. The fix is intended to improve readability of the usage info. Signed-off-by: Ibai Erkiaga --- Changes in v2: - Tabs replaced by spaces - fpga command added on eac

[U-Boot][PATCH v2 2/3] fpga: improve loads usage information

2025-01-17 Thread Ibai Erkiaga
Current usage information for loads command is too verbose and long for a command usage prompt. This flag simplifies the text for readability purposes. Signed-off-by: Ibai Erkiaga --- (no changes since v1) cmd/fpga.c | 21 - 1 file changed, 8 insertions(+), 13 deletions(-)

[U-Boot][PATCH v2 0/3] Improve fpga cmd usage information

2025-01-17 Thread Ibai Erkiaga
Current usage information is not read friendly specially when all the options are selected. This series fix some of the issues in the info, aligment and overlapping issues, excess of verbosity on the loads... Additionally Xilinx exclusive command dependencies are included to remove tex

[XGMAC RGMII-ID fix v1 1/1] net: dwc_eth_xgmac_socfpga: Add support for rgmii-id mode.

2025-01-17 Thread Boon Khai Ng
An issue was identified where selecting the phy-mode as rgmii-id in the device tree source (DTS) would cause the `dwc_eth_xgmac_socfpga` driver to raise an unsupported phy mode error. >From the MAC controller's perspective, the rgmii and rgmii-id phy modes are effectively identical. To address thi

[XGMAC RGMII-ID fix v1 0/1] Add support for rgmii-id phy mode

2025-01-17 Thread Boon Khai Ng
An issue was identified where selecting the phy-mode as rgmii-id in the device tree source (DTS) would cause the `dwc_eth_xgmac_socfpga` driver to raise an unsupported phy mode error. >From the MAC controller's perspective, the rgmii and rgmii-id phy modes are effectively identical. To address thi

[mailbox error message v1 1/1] fpga: intel_sdm_mb: add support for query SDM config error and status

2025-01-17 Thread Boon Khai Ng
Currently the FPGA reconfig status only return a single error status which make the debugging of FPGA reconfiguration hard. This patch is to expose the error status, major error code and minor error code, for the FPGA reconfig to upper layer app. Signed-off-by: Boon Khai Ng --- .../mach-socfpga

[XGMAC error message v1 1/1] net: dwc_eth_xgmac: Add device name for the error message.

2025-01-17 Thread Boon Khai Ng
Agilex5 having several ethernet instance, adding the device name at the error message to differentiate between which instance is having issue. Signed-off-by: Boon Khai Ng --- drivers/net/dwc_eth_xgmac.c | 82 +++-- 1 file changed, 52 insertions(+), 30 deletions(-)

[XGMAC error message v1 0/1] Adding device name when printing error.

2025-01-17 Thread Boon Khai Ng
In Intel SoC FPGA Agilex5, it support several ethernet instances, when there is an error occured at multiple interface it hardly tell which instances that is associated to that error. Hence, adding the device name at the error message to differentiate between which instance is having issue. Boon

  1   2   >