Re: [PATCH V2 3/5] arm64: imx: imx8mm-beacon: Enable CPU_IMX

2025-03-27 Thread Adam Ford
On Fri, Mar 21, 2025 at 7:10 AM Fabio Estevam wrote: > > On Thu, Mar 20, 2025 at 11:27 PM Marek Vasut wrote: > > > Yes, I know. But then, I can safely say it is desired on hardware I > > maintain, I think we could try and run a quick poll and see if others > > see it the same way ? > > I think it

[PATCH 3/3] configs: phycore_am62x_a53_defconfig: Enable gpio

2025-03-27 Thread Daniel Schultz
The AM62x uses the DA8XX (DaVinci) GPIO controller. Enable CONFIG_DA8XX_GPIO to support GPIO access from the Cortex-A53. Also enable the 'gpio' command to allow users to interact with GPIOs from the U-Boot shell. Signed-off-by: Daniel Schultz --- configs/phycore_am62x_a53_defconfig | 4 1

[PATCH 1/3] configs: phycore_am64x_a53_defconfig: Enable remoteproc cmd

2025-03-27 Thread Daniel Schultz
This enables the 'rproc' command, allowing users to start, stop, and manage co-processors as well as load firmware images. Useful for systems with auxiliary cores, such as M4 or R5 cores in the AM64x soc. Signed-off-by: Daniel Schultz --- configs/phycore_am64x_a53_defconfig | 1 + 1 file change

[PATCH 2/3] configs: phycore_am62x_a53_defconfig: Enable remoteproc cmd

2025-03-27 Thread Daniel Schultz
This enables the 'rproc' command, allowing users to start, stop, and manage co-processors as well as load firmware images. Useful for systems with auxiliary cores, such as the M4 core in the AM62x soc. Signed-off-by: Daniel Schultz --- configs/phycore_am62x_a53_defconfig | 1 + 1 file changed,

Re: [PATCH 12/18] mtd: ubi: annotate fallthrough

2025-03-27 Thread Heiko Schocher
Hello Andre, On 27.03.25 16:33, Andre Przywara wrote: The UBI code uses an implicit switch/case fallthrough when handling two related cases of bad header errors. Also there is a switch/case for unit prefix handling (G/M/K), which accumulates multiplications. Add our "fallthrough;" statement-lik

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

2025-03-27 Thread Caleb Connolly
On 2/15/25 04:22, 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 (Flat Image Tree). Today, sadly, there are a few efforts to re-invent FIT poorly, to cater to more primitive

Re: [PATCH] efi_loader: remove EFI_BOUNCE_BUFFER

2025-03-27 Thread Ilias Apalodimas
Hi Simon On Thu, 27 Mar 2025 at 15:33, Simon Glass wrote: > > Hi Ilias, > > On Wed, 26 Mar 2025 at 02:37, Ilias Apalodimas > wrote: > > > > Hi Heinrich, > > > > On Mon, 24 Mar 2025 at 19:50, Heinrich Schuchardt > > wrote: > > > > > > On 17.03.25 14:38, Ilias Apalodimas wrote: > > > > > > %s/EF

Re: [PATCH v2 0/4] USB-PD TCPM improvements

2025-03-27 Thread Nicolas Frattaroli
On Wednesday, 26 February 2025 19:44:43 Central European Standard Time Sebastian Reichel wrote: > Hi, > > I have a couple of fixes/improvements for the TCPM code. Three are fixing > actual problems I noticed on the Rock 5B, which prevented booting up the > system. One of them simply adds an error

[RESEND PATCH 1/2] config_distro_bootcmd: make possible to substitute 'part list' in distro_bootcmd

2025-03-27 Thread Pascal Zimmermann
Make it possible to substitute the 'part list' command inside 'scan_dev_for_boot_part' with a custom board specific implementation. For this the new define 'SCAN_DEV_FOR_BOOT_PARTS' is introduced. Signed-off-by: Pascal Zimmermann --- Cc: Tom Rini Cc: Simon Glass Cc: u-boot@lists.denx.de Cc: Ma

[RESEND PATCH 2/2] ARM: stm32: env: Enable scan and start for AB schema on STM32MP15 DHSOM

2025-03-27 Thread Pascal Zimmermann
For the STM32MP15 DHSOM, change the default environment so an AB schema on a device can be detected. For this the define "SCAN_DEV_FOR_BOOT_PARTS" is overwritten and appended. The detection works by looking for the partitions with specific lables. The name of those partitions are in the variables

[RESEND PATCH 0/2] ARM: stm32: env: Enable scan and start for AB schema on STM32MP15 DHSOM

2025-03-27 Thread Pascal Zimmermann
For the STM32MP15 DHSOM, change the default environment so an AB schema on a device can be detected. The first patch makes possible to substitute the 'part list' command inside 'scan_dev_for_boot_part' with a custom board specific implementation. The second patch uses the change of the first patc

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

2025-03-27 Thread Tom Rini
On Thu, Mar 27, 2025 at 07:36:06AM -0600, Simon Glass wrote: > Hi Tom, > > On Fri, 21 Feb 2025 at 13:45, Tom Rini wrote: > > > > On Fri, Feb 14, 2025 at 08:22:21PM -0700, Simon Glass wrote: > > > > > The original x86 EFI-app was written back in 2015, about a year before > > > the EFI-loader was i

Re: [PATCH 3/5] x86/acpi: simplify updating header checksum

2025-03-27 Thread Matthias Brugger
On 22/03/2025 00:21, Heinrich Schuchardt wrote: Use acpi_update_checksum() for updating ACPI table header checksum. Signed-off-by: Heinrich Schuchardt Reviewed-by: Matthias Brugger --- arch/x86/cpu/apollolake/hostbridge.c | 2 +- arch/x86/lib/acpi_nhlt.c | 2 +- arch/x

[PATCH 02/17] fs: boot: Update fs_load_alloc() to use abuf

2025-03-27 Thread Simon Glass
Using an abuf for this function simplifies returning the size and also makes it easier to free memory afterwards. Update the API and callers. Signed-off-by: Simon Glass --- cmd/cat.c| 13 ++--- cmd/cedit.c | 18 -- fs/fs.c | 11 --- include/fs.h | 10 ++

[PATCH 02/34] sunxi: mmc: remove usage of struct sunxi_ccm_reg

2025-03-27 Thread Andre Przywara
The Allwinner MMC code uses a complex C struct, modelling the clock device's register frame. We rely on sharing the member names across all Allwinner SoCs, which is fragile. Drop the usage of the struct in the MMC code, by using #define'd register names and their offset, and then adding those name

[PATCH v5 0/8] Add support for sam9x7 SoC and SAM9X75 Curiosity board

2025-03-27 Thread Manikandan Muralidharan
This patch series adds support for the new SoC family - sam9x7. - sam9x7 SoC is added - Clock driver for sam9x7 is added - Target board SAM9X75 Curiosity is added with its differences in DT and MMC config support changes in v5: - 4/8 - Define ID_MAIN_RC name in sam9x7 clock driver thus d

[PATCH v2 23/24] clk: clk-fixed-factor: Resolve parent clock by name

2025-03-27 Thread Marek Vasut
Use clock-names property which is accessible via parent clock OF node to look up the parent clock by name instead of depending on unreliable global clock name to perform look up. Signed-off-by: Marek Vasut --- Cc: Adam Ford Cc: Christoph Niedermaier Cc: Dong Aisheng Cc: Fabio Estevam Cc: Hou

Re: [PATCH] log: Add helpers for calling log_msg_ret() et al

2025-03-27 Thread Tom Rini
On Wed, Mar 26, 2025 at 10:46:57AM -0600, Simon Glass wrote: > Hi Quentin, > > On Tue, 25 Mar 2025 at 04:20, Quentin Schulz wrote: > > > > Hi Simon, > > > > On 3/20/25 4:26 AM, Simon Glass wrote: > > > Hi Quentin, > > > > > > On Wed, 19 Mar 2025 at 16:31, Quentin Schulz > > > wrote: > > >> > >

[PATCH 2/2] env: point fdt address to the fdt in a bloblist

2025-03-27 Thread Raymond Mao
Point fdt_addr to the fdt embedded in the bloblist since fdt_addr is a default address for bootefi, bootm and booti to look for the device tree when launching the kernel. Signed-off-by: Raymond Mao --- env/common.c | 17 + 1 file changed, 17 insertions(+) diff --git a/env/common

[PATCH 1/2] bloblist: fix the overriding of fdt from bloblist

2025-03-27 Thread Raymond Mao
When a bloblist is valid and contains fdt, it explicitly means a previous boot stage is passing transfer list compliant with Firmware Handoff specification, thus the fdt from bloblist should not be overridden with the ones from board or env variables. Fixes: 70fe23859437 ("fdt: Allow the devicetre

[PATCH] tools: add pkg-config for preload_check_sign

2025-03-27 Thread Raymond Mao
The cflags and ldflags of preload_check_sign depend on the openssl package thus pkg-config is needed to get the location where openssl is installed. This fix a potential build failure when openssl is not from the distro and installed in a varied place. Signed-off-by: Raymond Mao --- tools/Makefi

Re: [PATCH] sunxi: h616: defconfigs: enable eMMC

2025-03-27 Thread Jernej Škrabec
Dne četrtek, 27. marec 2025 ob 22:21:02 Srednjeevropski standardni čas je Andre Przywara napisal(a): > Now that eMMC is working properly on H616 devices, it became apparent > that some boards were missing the right defconfig bits to enable eMMC > access. > > Add the eMMC device number to the Tani

Re: [PATCH v1] efi_loader: Handle GD_FLG_SKIP_RELOC

2025-03-27 Thread Ilias Apalodimas
On Thu, 27 Mar 2025 at 08:12, Varadarajan Narayanan wrote: > > On Wed, Mar 26, 2025 at 09:28:04AM +0200, Ilias Apalodimas wrote: > > Hi Varadarajan > > > > On Wed, 26 Mar 2025 at 07:47, Varadarajan Narayanan > > wrote: > > > > > > If the EFI runtime services pointers are relocated even though > >

[PATCH] sunxi: h616: defconfigs: enable eMMC

2025-03-27 Thread Andre Przywara
Now that eMMC is working properly on H616 devices, it became apparent that some boards were missing the right defconfig bits to enable eMMC access. Add the eMMC device number to the Tanix TX1 and the X96 Mate defconfig, also the eMMC boot option to the TX1. Oddly enough the X96 Mate had just this

[PATCH 36/52] expo: Support rendering multiple lines of text

2025-03-27 Thread Simon Glass
Use the measurement info to write each line of text separately, thus respecting word-wrapping and newlines. Fix up the comment for scene_obj_render() while we are here. Signed-off-by: Simon Glass --- boot/bootflow_menu.c | 2 ++ boot/scene.c | 22 ++ include/expo.h

[PATCH V2 4/5] arm64: imx: imx8mn-beacon: Enable CPU_IMX

2025-03-27 Thread Adam Ford
Enable CPU_IMX to register the ARM core and identify the speed and temperature information using the driver model. Signed-off-by: Adam Ford --- configs/imx8mn_beacon_defconfig | 2 ++ 1 file changed, 2 insertions(+) V2: No Change diff --git a/configs/imx8mn_beacon_defconfig b/configs/imx8mn_be

Re: [PATCH v2 13/15] mach-snapdragon: Support booting as a coreboot payload

2025-03-27 Thread Caleb Connolly
On 2/26/25 23:15, Stephen Boyd wrote: Coreboot sets up the DRAM, CPU, and devices before booting a payload, meaning we can parse the coreboot tables at boot for memory info when running as a payload. Use the memory descriptors from coreboot if the tables are present so that Snapdragon SoCs can

Re: [PATCH] part_efi: cache last scanned GPT for next partition

2025-03-27 Thread Heinrich Schuchardt
HE_ALIGN_BUFFER_PAD(legacy_mbr, mbr, 1, desc->blksz); > > /* Read MBR Header from device */ >@@ -1081,6 +1138,9 @@ static int is_gpt_valid(struct blk_desc *desc, u64 lba, >gpt_header *pgpt_head, > return 0; > } > >+ if (cache) >+ cache_gpt_pte(desc, lba, *pgpt_pte, pgpt_head); >+ > /* We're done, all's well */ > return 1; > } >@@ -1100,13 +1160,14 @@ static int find_valid_gpt(struct blk_desc *desc, >gpt_header *gpt_head, > int r; > > r = is_gpt_valid(desc, GPT_PRIMARY_PARTITION_TABLE_LBA, gpt_head, >- pgpt_pte); >+ pgpt_pte, true); > > if (r != 1) { > if (r != 2) > log_debug("Invalid GPT\n"); > >- if (is_gpt_valid(desc, desc->lba - 1, gpt_head, pgpt_pte) >+ if (is_gpt_valid(desc, desc->lba - 1, gpt_head, pgpt_pte, >+ true) > != 1) { > log_debug("Invalid Backup GPT\n"); > return 0; > >--- >base-commit: 4adbf64ff8d8c730223fd8ae299d770bebb6fe86 >change-id: 20250327-u-boot-efi-part-cache-ddc1499c3ef6 > >Best regards,

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

2025-03-27 Thread Simon Glass
Hi Tom, On Fri, 21 Feb 2025 at 13:45, Tom Rini wrote: > > On Fri, Feb 14, 2025 at 08:22:21PM -0700, 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 (Flat Image Tree

[PATCH 2/5] doc: cmd: Add documentation for fuse command

2025-03-27 Thread Harsha Vardhan V M
Add documentation for the 'fuse' sub-system commands in doc/usage/cmd/fuse.rst file. Remove doc/README.fuse file. Signed-off-by: Harsha Vardhan V M --- doc/README.fuse| 67 -- doc/usage/cmd/fuse.rst | 156 + doc/usage/index.rst

Re: [PATCH] tiny-printf: Add support for upper case hex values

2025-03-27 Thread Tom Rini
On Thu, Mar 20, 2025 at 05:58:56PM +, Christoph Niedermaier wrote: > From: Tom Rini > Sent: Thursday, March 20, 2025 3:19 PM > > On Thu, Mar 20, 2025 at 03:14:03PM +0100, Marek Vasut wrote: > > > On 3/20/25 3:00 PM, Quentin Schulz wrote: > > > > Hi Marek, > > > > > > > > On 3/20/25 12:49 PM, M

[PATCH v1] board: starfive: Remove compatible boards Milk-V Mars CM and Mars CM Lite

2025-03-27 Thread E Shattow
Remove leftover code from Milk-V Mars CM and Mars CM Lite boards that do not exist in upstream Linux Kernel devicetree-rebasing. These will be re- introduced when submitted upstream for a future U-Boot release. Users of these boards should use the previous stable release of U-Boot until then. Sign

Re: [PATCH] arm64: zynqmp: Use CONFIG_SPL_FS_LOAD_PAYLOAD_NAME in binman

2025-03-27 Thread Michal Simek
Hi, On 3/25/25 10:48, Quentin Schulz wrote: Hi Michal, On 3/19/25 2:08 PM, Michal Simek wrote: u-boot.itb name is coming via CONFIG_SPL_FS_LOAD_PAYLOAD_NAME and it's They are unrelated as far as I can tell? Isn't CONFIG_SPL_FS_LOAD_PAYLOAD_NAME the name of the U-Boot FIT that the SPL is go

Re: [PATCH v4 29/29] efi: arm: Add an EFI app for arm64

2025-03-27 Thread Caleb Connolly
On 2/15/25 04:22, Simon Glass wrote: Introduce an EFI app for arm64 and update the documentation. Provide a value for LOAD_ADDR to avoid a link error. Signed-off-by: Simon Glass --- (no changes since v2) Changes in v2: - Use ARCH_EFI instead of VENDOR_EFI - Merge the linker-script rules i

Re: [PATCH v4 22/29] efi: arm: Drop startup code from the app

2025-03-27 Thread Caleb Connolly
On 2/15/25 04:22, Simon Glass wrote: The EFI app uses different startup and relocation code and the existing code uses symbols not present in the app. Drop it. Shouldn't this be squashed into patch 18 ("efi: arm: Omit the ARM start-up code in the EFI app")? Signed-off-by: Simon Glass Rev

Re: Unexpected RPi5 Behavior with Saved U-Boot Environment

2025-03-27 Thread Richard Weinberger
On Donnerstag, 27. März 2025 17:24 Mauro Salvini wrote: > could be related tho this [1]? > > I sent this patch some years ago but was never accepted in u-boot tree. > But meta-rpi uses it in u-boot recipe [2]. Uh, good point! In our initial experiments we have used meta-raspberrypi but now we don

Re: [PATCH v4 18/29] efi: arm: Omit the ARM start-up code in the EFI app

2025-03-27 Thread Caleb Connolly
On 2/15/25 04:22, Simon Glass wrote: This code is not used with the EFI app, since it enters through a call to efi_main(). Remove start.S and crt files from the build. Signed-off-by: Simon Glass Reviewed-by: Tom Rini Reviewed-by: Caleb Connolly --- (no changes since v1) arch/arm/Mak

Unexpected RPi5 Behavior with Saved U-Boot Environment

2025-03-27 Thread Richard Weinberger
Hi! We recently started using U-Boot on various Raspberry Pi 5 boards with different hardware configurations. While testing, Simon noticed unusual boot failures, such as the kernel failing to boot with no output or U-Boot reporting: "ERROR: Did not find a cmdline Flattened Device Tree." Today, I

Re: [PATCH v4 17/29] efi: arm: Implement access to global_data for EFI

2025-03-27 Thread Caleb Connolly
On 2/15/25 04:22, Simon Glass wrote: The EFI app likes to use a simple variable for global data. Implement this for ARM (for use only with 64-bit ARM at present). Signed-off-by: Simon Glass Reviewed-by: Tom Rini Reviewed-by: Caleb Connolly --- (no changes since v2) Changes in v2: - Me

Re: [PATCH v4 16/29] efi: arm: Avoid enabling caches when running under EFI

2025-03-27 Thread Caleb Connolly
On 2/15/25 04:22, Simon Glass wrote: The previous bootloader has already set up the cache, so don't try to do it again. This fixes a crash in QEMU when booting from EDK2 Signed-off-by: Simon Glass --- (no changes since v1) common/board_r.c | 5 +++-- 1 file changed, 3 insertions(+), 2

Re: [PATCH v4 15/29] efi: arm: Avoid allocating page tables when running under EFI

2025-03-27 Thread Caleb Connolly
On 2/15/25 04:22, Simon Glass wrote: The previous bootloader has already set up the page tables, so don't try to do it again. This fixes a crash in QEMU when booting from EDK2 Signed-off-by: Simon Glass Reviewed-by: Caleb Connolly --- (no changes since v1) arch/arm/lib/cache.c | 3 +

Re: [PATCH v4 14/29] efi: arm: Don't do the EL2 switch when running under EFI

2025-03-27 Thread Caleb Connolly
On 2/15/25 04:22, Simon Glass wrote: The previous bootloader has likely done this already, so avoid trying to do it again. This fixes a crash in QEMU when booting from EDK2 Signed-off-by: Simon Glass ll_boot_init is a pretty unintuitive name tbh. skip_lowlevel_init() might be clearer? o

Re: Unexpected RPi5 Behavior with Saved U-Boot Environment

2025-03-27 Thread Mauro Salvini
On 27/03/25 16:54, Richard Weinberger wrote: Hi! We recently started using U-Boot on various Raspberry Pi 5 boards with different hardware configurations. While testing, Simon noticed unusual boot failures, such as the kernel failing to boot with no output or U-Boot reporting: "ERROR: Did not f

Re: [PATCH v4 13/29] efi: lmb: Avoid access to _start

2025-03-27 Thread Caleb Connolly
On 2/15/25 04:22, Simon Glass wrote: The app does not have this symbol. Also the memory where the app is loaded is not under U-Boot's control. There is no sense in relocating from one alloced region to another. U-Boot is not able to relocate to the top of memory, nor decide what other parts o

[PATCH 10/18] video: annotate switch/case fall-through

2025-03-27 Thread Andre Przywara
The generic DM video code uses an implicit switch/case fallthrough to provide fallback code paths when certain colour depths are not enabled. Add our "fallthrough;" statement-like macro to the video_fill() function to avoid a warning when GCC's -Wimplicit-fallthrough warning option is enabled. Si

[PATCH 17/18] cmd: spl: annotate switch/case fallthrough

2025-03-27 Thread Andre Przywara
The argument parsing in the SPL configuration command uses an implicit switch/case fallthrough when dealing with a different number of arguments. Add our "fallthrough;" statement-like macro before the respective labels in the bootm code, to avoid a warning when GCC's -Wimplicit-fallthrough warning

[PATCH 18/18] [DO NOT MERGE] Makefile: enable switch/case fallthrough warnings

2025-03-27 Thread Andre Przywara
= This does not fully work yet, there is at least the libbz2 code left to be handled. But this patch helps to let CI find more places where we need annotations. = C's imp

RE: [PATCH] ARM: socfpga: Drop incorrect imply SYS_MMCSD_RAW_MODE_U_BOOT_USE_PARTITION*

2025-03-27 Thread Chee, Tien Fong
> -Original Message- > From: Tom Rini > Sent: Tuesday, March 18, 2025 9:58 PM > To: Chee, Tien Fong > Cc: u-boot@lists.denx.de > Subject: Re: [PATCH] ARM: socfpga: Drop incorrect imply > SYS_MMCSD_RAW_MODE_U_BOOT_USE_PARTITION* > > On Tue, Mar 18, 2025 at 03:11:44AM +, Chee, Tien

[PATCH 04/18] use proper fallthrough annotations

2025-03-27 Thread Andre Przywara
In some cases in the generic code, we were already using switch/case fallthrough annotations comments, though in a way which might not be understood by most compilers. Replace two non-standard /* no break */ comments with our fallthrough; statement-like macro, to make this visible to the compiler.

Re: [PATCH] topic: zynqmp: Remove zynqmp-xilinx-xdp platform

2025-03-27 Thread Mike Looijmans
On 27-03-2025 12:05, Andre Przywara wrote: On Thu, 27 Mar 2025 10:00:50 +0100 Michal Simek wrote: Hi Mike, I don't have this board, so don't really care, but was wondering: On 3/26/25 17:15, Mike Looijmans wrote: The "xdp" board went out of production years ago, I have the feeling that thi

[PATCH 16/18] cmd: pmic: annotate switch/case fallthrough

2025-03-27 Thread Andre Przywara
The argument parsing code in the pmic command uses an implicit switch/case fallthrough to handle the common part of having one or two arguments. Add our "fallthrough;" statement-like macro before the second branch in the parsing code, to avoid a warning when GCC's -Wimplicit-fallthrough warning op

[PATCH 14/18] mtd: spi-nor-tiny: annotate switch/case fallthrough

2025-03-27 Thread Andre Przywara
The SPI NOR code uses an implicit switch/case fallthrough when checking different vendors to determine how to deal with extended addressig modes. Add our "fallthrough;" statement-like macro before some label in the 4-byte addressing mode code, to avoid a warning when GCC's -Wimplicit-fallthrough w

[PATCH 15/18] mtd: rawnand: nand_base: annotate switch/case fallthrough

2025-03-27 Thread Andre Przywara
The raw NAND flash code uses an implicit switch/case fallthrough to share code when dealing with different ECC modes, and also when handling some read command. Add our "fallthrough;" statement-like macro before the respective labels in the NAND code, to avoid a warning when GCC's -Wimplicit-fallth

[PATCH 13/18] arm: mach-k3: am62p: annotate switch/case fallthrough

2025-03-27 Thread Andre Przywara
The MMC boot mode selection for the TI AM62P series of SoCs uses an implicit switch/case fallthrough for falling back to some default boot mode. Add our "fallthrough;" statement-like macro before the default branch in the code, to avoid a warning when GCC's -Wimplicit-fallthrough warning option is

[PATCH 11/18] net: e1000: annotate switch/case fallthrough

2025-03-27 Thread Andre Przywara
The E1000 driver uses an implicit switch/case fallthrough for sharing some code supporting different PHYs. Add our "fallthrough;" statement-like macro before the two labels in e1000_set_phy_type(), to avoid a warning when GCC's -Wimplicit-fallthrough warning option is enabled. Signed-off-by: Andr

[PATCH 12/18] mtd: ubi: annotate fallthrough

2025-03-27 Thread Andre Przywara
The UBI code uses an implicit switch/case fallthrough when handling two related cases of bad header errors. Also there is a switch/case for unit prefix handling (G/M/K), which accumulates multiplications. Add our "fallthrough;" statement-like macro before the respective labels in both cases, to av

[PATCH 08/18] usb: ohci-hcd: annotate switch/case fallthrough

2025-03-27 Thread Andre Przywara
The USB OCHI code uses an implicit switch/case fallthrough after checking for valid descriptor IDs. Add our "fallthrough;" statement-like macro before the default branch in the OHCI code, to avoid a warning when GCC's -Wimplicit-fallthrough warning option is enabled. Signed-off-by: Andre Przywara

[PATCH 09/18] usb: xhci: annotate switch/case fallthrough properly

2025-03-27 Thread Andre Przywara
The USB XHCI code uses an implicit switch/case fallthrough to share code for handling full speed and low speed transfers. Add our "fallthrough;" statement-like macro before the second label in the XHCI code, to avoid a warning when GCC's -Wimplicit-fallthrough warning option is enabled. Signed-of

[PATCH 06/18] fastboot: annotate switch/case fallthrough case

2025-03-27 Thread Andre Przywara
The fastboot command handling uses an implicit switch/case fallthrough when receiving the OEM_CONSOLE command, but when this command is not enabled in Kconfig, to report this command as unknown. Add our "fallthrough;" statement-like macro before the default branch in the fastboot code, to avoid a

[PATCH 07/18] net: sun8i-emac: annotate fallthrough

2025-03-27 Thread Andre Przywara
The Allwinner sun8i EMAC driver uses an implicit switch/case fallthrough when setting up the MAC/PHY communication protocol, to handle the case when RMII is requested, but would not be supported by the hardware. Add our "fallthrough;" statement-like macro before the default branch in sun8i_emac_se

[PATCH 05/18] net/net: fix switch/case fallthrough annotations

2025-03-27 Thread Andre Przywara
The net_check_prereq() routine in the generic network handling code mixes case: labels with #ifdef's, which makes predicting fallthrough situations tricky. We had two "fall through" comments in the code, but at the wrong places. Remove one unneeded comment (no annotations necessary between just em

[PATCH 02/18] zlib: annotate switch/case fallthrough cases

2025-03-27 Thread Andre Przywara
The inflate state machine in zlib uses switch/case fall-through's extensively, as it sometimes advances the state, and lets the conveniently placed next case statement handle the new state already. The pattern here is: state->mode = LEN; case LEN: Annotate those occasions w

[PATCH 03/18] gadget: f_thor: annotate switch/case fallthrough

2025-03-27 Thread Andre Przywara
Even though we seem to catch POWEROFF and EFSCLEAR commands in the THOR protocol request handling, we ultimately do not seem to handle them (apart from sending a response), so those commands still print an error message. Annotate the switch/case fallthrough in this case, to make this clear to the

[PATCH 00/18] Annotate switch/case fallthrough cases

2025-03-27 Thread Andre Przywara
C's implicit fallthrough behaviour in switch/case statements can lead to subtle bugs. Quite some while ago many compilers introduced warnings in those cases, requiring intentional fallthrough's to be annotated. So far we were not enabling that compiler option, so many ambiguities and some bugs

[PATCH 01/18] spl: mmc: properly annotate fallthrough

2025-03-27 Thread Andre Przywara
Depending on the various MMC boot configurations, we might end up with trying filesystem mode when a raw image boot failed. This fall-through in the switch/case statement is explained in a comment, but this is not visible to the compiler, which still will complain. Add the proper compiler-visible

Re: [PATCH v4 03/29] video: Use a local mode_info struct in the EFI driver

2025-03-27 Thread Caleb Connolly
On 2/15/25 04:22, Simon Glass wrote: There is no need to use the global struct provided by PCI. Declare a local structure instead, so this can be used on ARM without including support for PCI ROMs. Honestly this solution is so obvious I assumed it wouldn't work when I was hacking on this st

Re: [PATCH v2 5/6] dt-bindings: clock: drop NUM_CLOCKS define for EN7581

2025-03-27 Thread Krzysztof Kozlowski
On 27/03/2025 15:50, Christian Marangi wrote: > On Thu, Mar 27, 2025 at 03:43:47PM +0100, Krzysztof Kozlowski wrote: >> On 14/03/2025 19:59, Christian Marangi wrote: >>> Drop NUM_CLOCKS define for EN7581 dts/upstream/src/include. This is not a >>> binding and >>> should not be placed here. Value i

Re: [PATCH v2 5/6] dt-bindings: clock: drop NUM_CLOCKS define for EN7581

2025-03-27 Thread Krzysztof Kozlowski
On 14/03/2025 19:59, Christian Marangi wrote: > Drop NUM_CLOCKS define for EN7581 dts/upstream/src/include. This is not a > binding and > should not be placed here. Value is derived internally in the user > driver. > > Signed-off-by: Christian Marangi > Acked-by: Krzysztof Kozlowski Please drop

Re: [PATCH v2 5/6] dt-bindings: clock: drop NUM_CLOCKS define for EN7581

2025-03-27 Thread Christian Marangi
On Thu, Mar 27, 2025 at 03:43:47PM +0100, Krzysztof Kozlowski wrote: > On 14/03/2025 19:59, Christian Marangi wrote: > > Drop NUM_CLOCKS define for EN7581 dts/upstream/src/include. This is not a > > binding and > > should not be placed here. Value is derived internally in the user > > driver. > >

Re: [PATCH v2 6/6] dt-bindings: clock: add ID for eMMC for EN7581

2025-03-27 Thread Krzysztof Kozlowski
On 14/03/2025 19:59, Christian Marangi wrote: > Add ID for eMMC for EN7581. This is to control clock selection of eMMC > between 200MHz and 150MHz. > > Signed-off-by: Christian Marangi > Acked-by: Conor Dooley > Link: https://lore.kernel.org/r/20250113231030.6735-4-ansuels...@gmail.com > Signed-

Re: [PATCH 4/8] env: Rename SYS_MMC_ENV_DEV to ENV_SDMMC_DEVICE_INDEX

2025-03-27 Thread Tom Rini
On Wed, Mar 26, 2025 at 10:22:34AM +0100, Quentin Schulz wrote: > Hi Marek, > > On 3/26/25 2:37 AM, Marek Vasut wrote: > > On 3/26/25 1:15 AM, Tom Rini wrote: > > > On Wed, Mar 26, 2025 at 12:02:08AM +0100, Marek Vasut wrote: > > > > On 3/24/25 1:39 PM, Quentin Schulz wrote: > > > > > Hi Marek, >

Re: [PATCH v2 0/6] airoha: Add initial support AN7581

2025-03-27 Thread Christian Marangi
On Fri, Mar 14, 2025 at 02:26:11PM -0600, Tom Rini wrote: > On Fri, Mar 14, 2025 at 07:59:20PM +0100, Christian Marangi wrote: > ` > > This little series adds initial support for Airoha AN7581 SoC. > > > > With the help of some backport patch, this use OF_UPSTREAM > > directly. > > > > Posting th

Re: [PATCH 00/52] expo: Various features and improvements

2025-03-27 Thread Tom Rini
On Thu, Mar 27, 2025 at 07:36:02AM -0600, Simon Glass wrote: > Hi Tom, > > On Tue, 25 Mar 2025 at 10:24, Tom Rini wrote: > > > > On Fri, Mar 21, 2025 at 07:38:45PM +0100, Simon Glass wrote: > > > Hi Tom, > > > > > > On Thu, 20 Mar 2025 at 15:21, Tom Rini wrote: > > > > > > > > On Thu, Mar 20, 20

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

2025-03-27 Thread Tom Rini
On Thu, Mar 27, 2025 at 10:31:18AM -0300, Fabio Estevam wrote: > Hi Miquel, > > On Wed, Mar 26, 2025 at 12:01 PM Miquel Raynal > wrote: > > > I rebased this work on the latest next, fixed the conflicts, adapted my > > changes to use Svyatoslav's work, fixed the CI. > > v5 still fails in CI: >

Re: [PATCH] efi_loader: remove EFI_BOUNCE_BUFFER

2025-03-27 Thread Simon Glass
Hi Ilias, On Wed, 26 Mar 2025 at 02:37, Ilias Apalodimas wrote: > > Hi Heinrich, > > On Mon, 24 Mar 2025 at 19:50, Heinrich Schuchardt wrote: > > > > On 17.03.25 14:38, Ilias Apalodimas wrote: > > > > %s/EFI_BOUNCE_BUFFER/CONFIG_EFI_LOADER_BOUNCE_BUFFER/ > > > > > The EFI subsystem defines its o

does spl in falcon mode support decompressing the linux kernel?

2025-03-27 Thread Ilya Shpoliansky
Hello community, I'm doing a bring-up to a new SOC based on RISC-V and due to very strict boot time constraints we decided to try out the falcon mode. I'm using the latest 2025.01 version I found a presentation from about 4 years ago that SPL in falcon doesn't support kernel decompression Does it

Re: [PATCH 1/8] arm: dts: k3-am625-phyboard-lyra-rdk: Add boot phase tag to phy_gmii_sel

2025-03-27 Thread Daniel Schultz
On 3/25/25 04:58, Wadim Egorov wrote: Add bootph-all tag to phy_gmii_sel node. This is needed for booting via Ethernet. While at it, drop main_pktdma reg redefinitions which are already provided by the top-level SoC device tree file. Signed-off-by: Wadim Egorov For entire series, Tested-by:

Re: [PATCH V3 1/6] imx: imx9: Imply IMX_TMU

2025-03-27 Thread Marek Vasut
On 3/22/25 7:38 PM, Adam Ford wrote: If the CPU Information is displayed from imx8_cpu, it displays the cpu temperature grade and operating temperature if CONFIG_IMX9 is defined. This behavior is similar to what happens arch/arm/mach-imx/cpu.c except that the latter checks for CONFIG_IMX_THERMAL

[PATCH 45/52] expo: Adjust expo_poll() to avoid looping forever

2025-03-27 Thread Simon Glass
If the user does not quickly provide some input, exit so that other things can be done, such as searching for the next OS. Signed-off-by: Simon Glass --- boot/expo.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/boot/expo.c b/boot/expo.c index 4e855f60d84..bf2f1dfc68

Re: usb:composite: data abort on second ums launch

2025-03-27 Thread Mattijs Korpershoek
Hi Zixun, Marek, On lun., mars 24, 2025 at 18:40, Mattijs Korpershoek wrote: > Hi Zixun, > > On lun., mars 24, 2025 at 18:33, Zixun LI wrote: > > resetting ... [...] > >> >> Each time when ums is called 2*FSG_BUFLEN, 256kB buffer is allocated >> and it seems not >> freed as fsg_common_releas

Re: [PATCH 00/52] expo: Various features and improvements

2025-03-27 Thread Simon Glass
Hi Tom, On Tue, 25 Mar 2025 at 10:24, Tom Rini wrote: > > On Fri, Mar 21, 2025 at 07:38:45PM +0100, Simon Glass wrote: > > Hi Tom, > > > > On Thu, 20 Mar 2025 at 15:21, Tom Rini wrote: > > > > > > On Thu, Mar 20, 2025 at 03:39:26AM +, Simon Glass wrote: > > > > Hi Tom, > > > > > > > > On Wed

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

2025-03-27 Thread Fabio Estevam
Hi Miquel, On Wed, Mar 26, 2025 at 12:01 PM Miquel Raynal wrote: > I rebased this work on the latest next, fixed the conflicts, adapted my > changes to use Svyatoslav's work, fixed the CI. v5 still fails in CI: https://source.denx.de/u-boot/custodians/u-boot-imx/-/pipelines/25364 > Then I pus

Re: [PATCH v8 02/19] pinctrl: nxp: add a pin controller driver based on SCMI pin control protocol

2025-03-27 Thread Marek Vasut
On 3/26/25 7:42 AM, Peng Fan wrote: Subject: Re: [PATCH v8 02/19] pinctrl: nxp: add a pin controller driver based on SCMI pin control protocol On 3/26/25 2:52 AM, Peng Fan wrote: Subject: Re: [PATCH v8 02/19] pinctrl: nxp: add a pin controller driver based on SCMI pin control protocol On 3/25/

Re: [PATCH 4/4] qcom_defconfig: enable capsule update support

2025-03-27 Thread Caleb Connolly
On 3/27/25 09:46, Peng Fan wrote: On Wed, Mar 26, 2025 at 06:40:37PM +0100, Caleb Connolly wrote: We can now correctly identify which partition U-Boot is flashed to between uefi, xbl, and boot (including A/B support) so enable capsule update support for all boards. Signed-off-by: Caleb Conno

Re: [PATCH v2] ubi: fix bug creating partitions for non-existent volumes

2025-03-27 Thread Heiko Schocher
Hello Oskar, On 26.03.25 10:22, Oskar Nilsson wrote: The part_get_info_ubi() function was incorrectly returning 0 (success) when a UBI volume was not found for a given partition index. This caused the part_create_block_devices() function in blk-uclass.c to continue creating devices for non-exist

Re: undefined reference to `__stack_chk_guard' when building with sandbox_defconfig

2025-03-27 Thread Andrew Goodbody
On 26/03/2025 13:03, McLaughlin Amy wrote: At my workplace, I ran: make distclean make sandbox_defconfig make all The result: LTO u-boot /usr/bin/ld: /tmp/ccvKFEMH.ltrans0.ltrans.o: in function `pci_io_read': /home/mclauam/gitclone/u-boot/arch/sandbox/lib/pci_io.c:52:(.text+0x3d1): und

Pull request: u-boot-sunxi/next for next

2025-03-27 Thread Andre Przywara
Hi Tom, please pull these sunxi changes into next: Assorted fixes, refactorings and additions that are ready, and shave off some load from upcoming series'. Improves MMC performance on D1/T113 (missed clock divider), enables eMMC access on the H616 family (never worked, many thanks to Jernej for

Re: [PATCH] topic: zynqmp: Remove zynqmp-xilinx-xdp platform

2025-03-27 Thread Andre Przywara
On Thu, 27 Mar 2025 10:00:50 +0100 Michal Simek wrote: Hi Mike, I don't have this board, so don't really care, but was wondering: > On 3/26/25 17:15, Mike Looijmans wrote: > > The "xdp" board went out of production years ago, I have the feeling that this is true for many boards in the U-Boot t

[PATCH 3/3] xilinx: versal: remove versal loadpdi command

2025-03-27 Thread Prasad Kummari
The source code for the versal loadpdi command and the CONFIG_CMD_VERSAL configuration has been removed. It now utilizes the fpga loadcommand to load secure & non-secure pdi images. Signed-off-by: Prasad Kummari --- arch/arm/mach-versal/Kconfig | 1 - board/xilinx/versal/Kconfig | 14 ---

[PATCH 1/3] arm64: versal-net: Add PL bit stream load support

2025-03-27 Thread Prasad Kummari
Add support for loading the secure & non-secure pdi images and PL bitstream on the Versal NET platform. The FPGA driver is enabled to load the bitstream in PDI format on the AMD Versal NET device. PDI is the new programmable device image format for Versal NET, and the bitstream for the Versal NET p

[PATCH 2/3] arm64: versal2: Add PL bit stream load support

2025-03-27 Thread Prasad Kummari
Add support for loading the secure & non-secure pdi images and PL bitstream on the Versal Gen2 platform. The FPGA driver is enabled to load the bitstream in PDI format on the AMD Versal Gen2 device. PDI is the new programmable device image format for Versal Gen2, and the bitstream for the Versal Ge

[PATCH 0/3] loadpdi related code changes.

2025-03-27 Thread Prasad Kummari
Prasad Kummari (3): arm64: versal-net: Add PL bit stream load support arm64: versal2: Add PL bit stream load support xilinx: versal: remove versal loadpdi command arch/arm/mach-versal-net/Kconfig | 1 - arch/arm/mach-versal/Kconfig | 1 - arch/arm/mach-versal2/Kconfi

[PATCH 15/17] bootstd: Correct the comment for bootmeth_set_order()

2025-03-27 Thread Simon Glass
The function is used to handle the boot_targets variable so must support space-separated items. Update the comment to show this. Signed-off-by: Simon Glass --- include/bootmeth.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/include/bootmeth.h b/include/bootmeth.h inde

[PATCH] board: ti: am335x: Fix selection of BeagleBone Green

2025-03-27 Thread Kory Maincent
The BeagleBone Green board is a revision of the BoneBlack board. Having BeagleBone Black devicetree listed before BeagleBone Green will select always the BeagleBone Black devicetree following the functioning of board_fit_config_name_match(). Fix it by changing the test condition and selecting Bone

[PATCH] part_efi: cache last scanned GPT for next partition

2025-03-27 Thread Neil Armstrong
f (is_gpt_valid(desc, desc->lba - 1, gpt_head, pgpt_pte) + if (is_gpt_valid(desc, desc->lba - 1, gpt_head, pgpt_pte, + true) != 1) { log_debug("Invalid Backup GPT\n"); return 0; --- base-commit: 4adbf64ff8d8c730223fd8ae299d770bebb6fe86 change-id: 20250327-u-boot-efi-part-cache-ddc1499c3ef6 Best regards, -- Neil Armstrong

[PATCH v5 4/8] clk: at91: sam9x7: add pmc driver for sam9x7 SoC family

2025-03-27 Thread Manikandan Muralidharan
From: Varshini Rajendran Add PMC driver support for sam9x7 SoC family Signed-off-by: Varshini Rajendran [balamanikandan.gunasun...@microchip.com: Add peripheral clock id for pmecc] Signed-off-by: Balamanikandan Gunasundar --- drivers/clk/at91/Makefile |1 + drivers/clk/at91/sam9x7.c | 10

Re: [PATCH] topic: zynqmp: Remove zynqmp-xilinx-xdp platform

2025-03-27 Thread Michal Simek
On 3/26/25 17:15, Mike Looijmans wrote: The "xdp" board went out of production years ago, remove the remnants from the U-boot codebase as there is no point in maintaining them. Signed-off-by: Mike Looijmans --- arch/arm/dts/Makefile |1 - .../zynqmp-topic-miam

Re: [PATCH v5 6/8] ARM: dts: at91: sam9x75_curiosity: add tweaks for sam9x75 curiosity board

2025-03-27 Thread Eugen Hristev
On 3/27/25 09:06, Manikandan Muralidharan wrote: > Since the SoC and board DT are already available in dts/upstream, > add the difference from upstream DTS to at91-sam9x75_curiosity-u-boot.dtsi > > Signed-off-by: Manikandan Muralidharan > --- > .../dts/at91-sam9x75_curiosity-u-boot.dtsi|

Re: [PATCH v5 6/8] ARM: dts: at91: sam9x75_curiosity: add tweaks for sam9x75 curiosity board

2025-03-27 Thread Manikandan.M
On 27/03/25 1:11 pm, Eugen Hristev wrote: > EXTERNAL EMAIL: Do not click links or open attachments unless you know the > content is safe > > On 3/27/25 09:06, Manikandan Muralidharan wrote: >> Since the SoC and board DT are already available in dts/upstream, >> add the difference from upstream DT

[RFC PATCH 0/1 v2] Propagate bootph* property to all parent nodes

2025-03-27 Thread Moteen Shah
In the U-Boot pre-relocation stage, if the parent node lacks bootph* property and the driver lacks a pre-reloc flag, all of its subsequent subnodes gets skipped over from driver binding—even if they have a bootph* property. This series addresses the issue by scanning through all the nodes during b

  1   2   >