Re: [PATCH v2 1/2] armv8: Fix get_sctlr() return type

2024-11-13 Thread Ilias Apalodimas
On Wed, 13 Nov 2024 at 04:30, Sam Protsenko wrote: > > On Mon, Nov 11, 2024 at 8:48 AM Ilias Apalodimas > wrote: > > > > On Thu, 7 Nov 2024 at 04:58, Sam Protsenko > > wrote: > > > > > > SCTLR_EL2 is a 64-bit register [1]. Return its value as long (64 bit) > > > instead of int (32 bit) in get_s

Re: [PATCH v3 08/15] efi_loader: device_path: add support for HTTP device path

2024-11-13 Thread Ilias Apalodimas
On Mon, 11 Nov 2024 at 23:10, Adriano Cordova wrote: > > Add efi_dp_from_http to form a device path from HTTP. The > device path is the concatenation of the device path returned > by efi_dp_from_ipv4 together with an URI node and an END node. > > Signed-off-by: Adriano Cordova > --- > > Changes i

[PATCH v2] mtd: spi-nor: Add support for flash reset

2024-11-13 Thread Venkatesh Yadav Abbarapu
Add support for spi-nor flash reset via GPIO controller by reading the reset-gpios property. [Ported from Linux kernel commit 8f1ee9ef71d0 ("mtd: spi-nor: Add support for flash reset") ] Signed-off-by: Venkatesh Yadav Abbarapu --- Changes in v2: - Removed the code from spi_nor_remove() as gpio d

[PATCH] treewide: remove Qualcomm dt-binding headers that are available upstream

2024-11-13 Thread Caleb Connolly
Some dt-binding headers mask the upstream ones which can lead to build failures, or worse: super weird bugs, if they get out of sync. Remove these headers so our devicetree and binding headers will both be in sync with upstream. Signed-off-by: Caleb Connolly --- Build tested with qcom_defconfig,

[PATCH v2 2/4] arm: dts: Use upstream dts

2024-11-13 Thread david regan
Make use of OF_UPSTREAM which uses Linux dts. Signed-off-by: david regan Reviewed-by: Linus Walleij Reviewed-by: Sumit Garg Reviewed-by: William Zhang Reviewed-by: Anand Gore --- arch/arm/dts/bcm47622.dtsi| 126 arch/arm/dts/bcm4912.dtsi | 128 ---

[PATCH v2 1/4] mtd: nand: brcmnand: remove device specific nand driver files

2024-11-13 Thread david regan
These device specific nand driver files can be removed because they are now replaced by a common driver bcmbca_nand.c Signed-off-by: david regan Reviewed-by: Linus Walleij Reviewed-by: William Zhang Reviewed-by: Anand Gore --- drivers/mtd/nand/raw/Kconfig | 24 drivers/

[PATCH v2 3/4] mtd: rawnand: brcmnand: Default bcmbca parameter_page_big_endian to zero

2024-11-13 Thread david regan
Set parameter_page_big_endian to zero for bcmbca Signed-off-by: david regan Reviewed-by: William Zhang Reviewed-by: Anand Gore --- drivers/mtd/nand/raw/brcmnand/brcmnand.c | 8 1 file changed, 8 insertions(+) diff --git a/drivers/mtd/nand/raw/brcmnand/brcmnand.c b/drivers/mtd/nand/r

[PATCH v2 0/4] Broadcom BCMBCA nand updates

2024-11-13 Thread david regan
These are ports of upstream Linux patches to U-Boot. Remove device specific nand driver files as they are now replaced by a common driver bcmbca_nand.c Make use of OF_UPSTREAM which uses Linux dts. Implement a minimal set of defconfig options to enable basic NAND functionality. Tested on: 47622

[PATCH v2 1/3] gpio: at91: Implement GPIOF_FUNC in get_function()

2024-11-13 Thread Zixun LI
This patch adds support for determining whether a gpio pin is mapped as peripheral function. Signed-off-by: Zixun LI --- Changes in v2: - Rename function --- drivers/gpio/at91_gpio.c | 13 - 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/drivers/gpio/at91_gpio.c b/driv

[PATCH v2 4/4] mtd: rawnand: brcmnand: update log level messages

2024-11-13 Thread david regan
Update log level messages so that more critical messages can be logged to console and help the troubleshooting with field devices. This is a port of the upstream Linux patch to U-Boot. https://lore.kernel.org/linux-mtd/20240223034758.13753-4-william.zh...@broadcom.com/ Signed-off-by: david regan

[PATCH v2 2/3] gpio: at91: Implement ops set_flags

2024-11-13 Thread Zixun LI
Support GPIO configuration with following flags: - in, out, out_active - open_drain, pull_up Signed-off-by: Zixun LI --- Changes in v2: - Return ENOTSUPP if flag is unsupported. - Inline port output variable. --- drivers/gpio/at91_gpio.c | 39 +++ 1 file chang

[PATCH v3 3/3] gpio: at91: Implement ops get_flags

2024-11-13 Thread Zixun LI
Add ops get_dir_flags() to read status from GPIO registers. Signed-off-by: Zixun LI --- Changes in v2: - Fix pullup read polarity Changes in v3: - Use clamp (!!) to turn return value into a bool --- drivers/gpio/at91_gpio.c | 45 1 file changed, 45 inser

Re: [PATCH] Revert "efi_memory: do not add U-Boot memory to the memory map"

2024-11-13 Thread Sughosh Ganu
On Wed, 13 Nov 2024 at 18:03, Heinrich Schuchardt wrote: > > On 11/13/24 12:28, Ilias Apalodimas wrote: > > Hi Sughosh > > > > On Tue, 12 Nov 2024 at 20:46, Sughosh Ganu wrote: > >> > >> On Tue, 12 Nov 2024 at 18:48, Simon Glass wrote: > >>> > >>> A bisect of Ubuntu 2022.04 boot-failure on qemu-

Re: [PATCH 2/2] efi_loader: remove non vital devices first

2024-11-13 Thread Heinrich Schuchardt
On 11/1/24 21:29, Mark Kettenis wrote: From: Janne Grunau Date: Thu, 31 Oct 2024 23:48:02 +0100 DM_FLAG_VITAL marks devices which are essential for the operation of other devices. Removing these devices before their users can result in hangs or crashes. This potentially fixes EFI boot of Renesa

Re: [PATCH] Revert "efi_memory: do not add U-Boot memory to the memory map"

2024-11-13 Thread Heinrich Schuchardt
On 11/13/24 12:28, Ilias Apalodimas wrote: Hi Sughosh On Tue, 12 Nov 2024 at 20:46, Sughosh Ganu wrote: On Tue, 12 Nov 2024 at 18:48, Simon Glass wrote: A bisect of Ubuntu 2022.04 boot-failure on qemu-x86_64 resulted in this patch. I am not sure how to investigate it. The boot hangs at so

Re: [PATCH v3 08/15] efi_loader: device_path: add support for HTTP device path

2024-11-13 Thread Adriano Córdova
El mié, 13 nov 2024 a las 5:16, Ilias Apalodimas (< ilias.apalodi...@linaro.org>) escribió: > On Mon, 11 Nov 2024 at 23:10, Adriano Cordova wrote: > > > > Add efi_dp_from_http to form a device path from HTTP. The > > device path is the concatenation of the device path returned > > by efi_dp_from_

Re: [PATCH] mmc: don't print 'MMC:' if there are no MMC devices

2024-11-13 Thread Caleb Connolly
On 13/11/2024 10:48, Dragan Simic wrote: > Hello Caleb, > > Thanks for the patch.  Please, see a comment below. > > On 2024-11-13 06:30, Caleb Connolly wrote: >> It may be the case that MMC support is enabled even though the board >> we're booting on doesn't have any MMC devices. Move the prin

Re: [PATCH] mmc: don't print 'MMC:' if there are no MMC devices

2024-11-13 Thread Dragan Simic
Hello Caleb, On 2024-11-13 14:08, Caleb Connolly wrote: On 13/11/2024 10:48, Dragan Simic wrote: Thanks for the patch.  Please, see a comment below. On 2024-11-13 06:30, Caleb Connolly wrote: It may be the case that MMC support is enabled even though the board we're booting on doesn't have an

Re: [PATCH v3 13/15] lib: uuid: display HTTP and IPV4 Config II protocols

2024-11-13 Thread Ilias Apalodimas
Hi Adriano, You forgot to add reviewed tags on this one. I am reviewing the rest of the patches, so please go through the tags and add them when you send a v4. Thanks /Ilias On Mon, 11 Nov 2024 at 23:10, Adriano Cordova wrote: > > From: Heinrich Schuchardt > > Add long texts for > > * EFI HTTP

Re: [PATCH v3 01/15] net: net_utils: Move ip_to_string to lib/net_utils.c

2024-11-13 Thread Ilias Apalodimas
Reviewed-by tags missing for me * Heinrich On Mon, 11 Nov 2024 at 23:10, Adriano Cordova wrote: > > The function string_to_ip is already in net_utils, which is > compiled unconditionally, but ip_to_string is currently only > accessible if the legacy network stack is selected. This > commit puts i

Re: [PATCH v3 8/9] CI: Add platform variable

2024-11-13 Thread Ilias Apalodimas
On Tue, 12 Nov 2024 at 16:11, Simon Glass wrote: > > Add a list of possible platforms that can be used by gitlab runners. > > Signed-off-by: Simon Glass > Reviewed-by: Tom Rini > --- > > Changes in v3: > - Drop patches which remove grub and tracing for arm64 > > .gitlab-ci.yml | 1 + > 1 file c

Re: [PATCH v3 03/15] efi_loader: device_path: add definition of DEVICE_PATH_SUB_TYPE_MSG_IPV4

2024-11-13 Thread Ilias Apalodimas
On Mon, 11 Nov 2024 at 23:10, Adriano Cordova wrote: > > Add definition for DEVICE_PATH_SUB_TYPE_MSG_IPV4 device path > subtype. > > Signed-off-by: Adriano Cordova > --- > > Changes in v3: > - Removed alignment in struct efi_ipv4_address > > include/efi_api.h | 17 + > 1 file cha

Re: [PATCH v3 04/15] efi_loader: device_path: add efi_dp_from_ipv4

2024-11-13 Thread Ilias Apalodimas
On Mon, 11 Nov 2024 at 23:10, Adriano Cordova wrote: > > Add efi_dp_from_ipv4 to form a device path from an ipv4 address. > > Signed-off-by: Adriano Cordova > --- > > Changes in v3: > - Removed some unnecessary void* casts. > - Changed sizeof(struct efi_device_path_ipv4) to sizeof(*ipv4dp) > in

Re: [u-boot PATCH] fs: btrfs: hide 'Cannot lookup file' errors on 'load'

2024-11-13 Thread Tom Rini
On Wed, Nov 13, 2024 at 02:58:42PM +0900, Dominique Martinet wrote: > Dominique Martinet wrote on Wed, Nov 06, 2024 at 09:56:42AM +0900: > > Running commands such as 'load mmc 2:1 $addr $path' when path does not > > exists historically do not print any error on filesystems such as ext4 > > or fat.

Re: [PATCH] nwt: wget: drop Content-Length processing

2024-11-13 Thread Tom Rini
On Tue, 05 Nov 2024 12:08:49 +0100, Jerome Forissier wrote: > We don't do anything with Content-Length except a debug print, and the > strict_strtoul() call is incorrect (it always returns -EINVAL and leaves > content_length to zero due to the presence of trailing characters after > the decimal va

Re: [PATCH v2] cmd: add a fetch utility

2024-11-13 Thread Neil Armstrong
On 13/11/2024 05:22, Caleb Connolly wrote: Add a small utility for displaying some information about U-Boot and the hardware it's running on in a similar fashion to the popular neofetch tool for Linux [1]. While the output is meant to be useful, it should also be pleasing to look at and perhaps

Re: [PATCH] mmc: don't print 'MMC:' if there are no MMC devices

2024-11-13 Thread Tom Rini
On Wed, Nov 13, 2024 at 06:30:08AM +0100, Caleb Connolly wrote: > It may be the case that MMC support is enabled even though the board > we're booting on doesn't have any MMC devices. Move the print over to > the print_mmc_devices() function where we can only print it if we > actually have MMC dev

Re: [PATCH v2] cmd: add a fetch utility

2024-11-13 Thread Tom Rini
On Wed, Nov 13, 2024 at 03:40:01PM +0100, Heinrich Schuchardt wrote: > Am 13. November 2024 14:32:57 MEZ schrieb Neil Armstrong > : > >On 13/11/2024 05:22, Caleb Connolly wrote: > >> Add a small utility for displaying some information about U-Boot and the > >> hardware it's running on in a similar

[PATCH v5 3/8] efi_loader: bootstd: Drop bootmgr for sunxi

2024-11-13 Thread Simon Glass
This causes problems with the boot order, so drop it until we can figure out a better way to know when bootmgr should be used. Link: https://lore.kernel.org/u-boot/20241112171205.4e805...@donnerap.manchester.arm.com/ Signed-off-by: Simon Glass --- Changes in v5: - Add new patch to drop bootmgr

Re: [PATCH v3 4/6] net: lwip: Enable https:// support for wget

2024-11-13 Thread Ilias Apalodimas
On Wed, 13 Nov 2024 at 18:04, Simon Glass wrote: > > Hi Ilias, > > On Wed, 13 Nov 2024 at 08:11, Ilias Apalodimas > wrote: > > > > On Wed, 13 Nov 2024 at 17:09, Simon Glass wrote: > > > > > > Hi Ilias, > > > > > > On Wed, 13 Nov 2024 at 07:46, Ilias Apalodimas > > > wrote: > > > > > > > > On W

Re: [PATCH v3 3/8] bootstd: Avoid depending on BLK

2024-11-13 Thread Tom Rini
On Wed, Nov 13, 2024 at 09:03:22AM -0700, Simon Glass wrote: > Hi Tom, > > On Wed, 13 Nov 2024 at 08:24, Tom Rini wrote: [snip] > > We're not going to get the sunxi series in for v2025.01, so we have time > > to think about this still and try and get the code to match up with the > > use cases. >

Re: [PATCH v1 0/4] Remove legacy Amlogic dtsi and bindings

2024-11-13 Thread neil . armstrong
From: Neil Armstrong Hi, On Tue, 12 Nov 2024 15:58:32 +0300, Alexey Romanov wrote: > We should remove bindings and dtsi from include/ folder for A1 > and AXG Amlogic devices. We have exactly the same, or with minor > differences in dts/upstream folder. > > I tested this changes on devices close

Re: [PATCH 2/2] efi_loader: remove non vital devices first

2024-11-13 Thread Simon Glass
Hi Heinrich, On Wed, 13 Nov 2024 at 08:17, Heinrich Schuchardt wrote: > > Am 13. November 2024 15:39:22 MEZ schrieb Simon Glass : > >Hi, > > > >On Wed, 13 Nov 2024 at 05:52, Heinrich Schuchardt wrote: > >> > >> On 11/1/24 21:29, Mark Kettenis wrote: > >> >> From: Janne Grunau > >> >> Date: Thu,

[PATCH v2 2/3] board: phytec: imx93: Add eeprom-based hardware introspection

2024-11-13 Thread Christoph Stoidner
The phyCORE-i.MX 93 is available in various variants. Relevant variant options for the spl/u-boot are: - with or without HS400 support for the eMMC - with 1GB ram chip, or 2GB ram chip The phyCORE's eeprom contains all information about the existing variant options. Add evaluation of the eeprom da

[PATCH v2 0/3] Add support for all variants of the phyCORE-i.MX93 SOM

2024-11-13 Thread Christoph Stoidner
The phyCORE-i.MX 93 is available in various variants (e.g. different ram sizes, eMMC HS400 yes/no). It's eeprom contains information which features the existing module supports. Extend the existing board-code to support all the SOM variants. The spl/u-boot evaluates the eeprom information and enab

[PATCH v2 3/3] board: phytec: imx93: Add phyCORE-i.MX 93 support for all SOM variants

2024-11-13 Thread Christoph Stoidner
The phyCORE-i.MX 93 is available in various variants (e.g. different ram sizes, eMMC HS400 yes/no). Enable hardware introspection for the imx93-phyboard-segin_defconfig, so that during startup the SOM module variant can be detected, and the hardware can be configured accordingly. The resulting SPL

Re: [PATCH 1/1] lib: vsprintf: fix API build

2024-11-13 Thread Tom Rini
On Sun, 03 Nov 2024 04:50:15 +0100, Heinrich Schuchardt wrote: > Avoid a build failure when building with CONFIG_API=y, CONFIG_EXAMPLES=y: > > lib/vsprintf.c:312:14: warning: > ‘device_path_string’ defined but not used [-Wunused-function] > 312 | static char *device_path_string(char

Re: [PATCH 1/1] test: run longjmp() test only on supported architectures

2024-11-13 Thread Tom Rini
On Sun, 03 Nov 2024 18:54:00 +0100, Heinrich Schuchardt wrote: > We have only implemented longjmp() on the EFI architectures. > > Define a symbol CONFIG_HAVE_SETJMP and have it selected by the relevant > architectures. > > Use CONFIG_HAVE_SETJMP to decide if the longjmp test shall be built. > >

[PATCH v2 1/3] board: phytec: phycore-imx93: Add 2GB LPDDR4X RAM timings

2024-11-13 Thread Christoph Stoidner
The phyCORE-i.MX 93 is available with a 1GB ram chip or a 2GB ram chip. Add the ram timings for the 2GB chip, in form of a diff compared to the existing LPDDR4X 1GB timings. With that, the SPL can select the appropriate timings at startup. Update also the 1GB ram timings with new version of the DDR

Re: [PATCH v2] Makefile: Fake external blob with BINMAN_ALLOW_MISSING=1

2024-11-13 Thread Tom Rini
On Wed, 06 Nov 2024 08:35:43 -0700, Simon Glass wrote: > This flag was lost by a previous change and has never been restored. > Without it, binman cannot fully handle missing blobs which are > themselves inputs to mkimage. > > Discussion on this at [1] indicated that this was necessary but the >

Re: [PATCH 1/1] fs: ext4: correct error handling

2024-11-13 Thread Tom Rini
On Thu, 07 Nov 2024 10:38:53 +0100, Heinrich Schuchardt wrote: > After calling strdup() check the returned pointer. > > Avoid a memory leak if the directory is not found. > > Applied to u-boot/master, thanks! -- Tom

[GIT PULL] Please pull u-boot-amlogic-next-20241113

2024-11-13 Thread Neil Armstrong
write}_config_* function calls (2024-11-04 18:01:58 -0600) are available in the Git repository at: https://source.denx.de/u-boot/custodians/u-boot-amlogic.git tags/u-boot-amlogic-next-20241113 for you to fetch changes up to 1cdb4e03e684d94456dee0cc30c935d7e25b263d: dt-bindings: interrupt-contr

Re: [PATCH v5 1/8] bootstd: Avoid depending on BLK

2024-11-13 Thread Tom Rini
On Wed, Nov 13, 2024 at 08:09:31AM -0700, Simon Glass wrote: > In principle bootstd can work without block devices, even if it does > require driver model to be enabled in that case. > > The use of a 'depends on BLK' for BOOTSTD conflicts with the way 'BLK' > is now defined, producing recursive e

Re: [PATCH 1/1] board: qemu-arm: select CONFIG_HAS_CUSTOM_SYS_INIT_SP_ADDR

2024-11-13 Thread Tom Rini
On Tue, 05 Nov 2024 20:31:29 +0100, Heinrich Schuchardt wrote: > qemu_arm64_defconfig with CONFIG_HAS_CUSTOM_SYS_INIT_SP_ADDR=n leads to a > build error: > > arch/arm/lib/crt0_64.S:85: > Error: constant expression expected at operand 2 -- > `ldr x0,=((CFG_SYS_INIT_RAM_ADDR+CFG_SYS_INI

Re: [PATCH v2 0/4] lib: uuid: fix uuid_str_to_le_bin() on 32-bit

2024-11-13 Thread Tom Rini
On Sun, 03 Nov 2024 23:42:19 +0100, Heinrich Schuchardt wrote: > The lib_test_uuid_to_le and lib lib_test_dynamic_uuid tests fail on > 32-bit systems. But we never caught this in our CI because we never > ran any of our C unit tests on 32-bit. > > Enable CONFIG_UNIT_TEST on qemu_arm_defconfig. >

Re: [PATCH v5 3/8] efi_loader: bootstd: Drop bootmgr for sunxi

2024-11-13 Thread Tom Rini
On Wed, Nov 13, 2024 at 08:09:33AM -0700, Simon Glass wrote: > This causes problems with the boot order, so drop it until we can figure > out a better way to know when bootmgr should be used. > > Link: > https://lore.kernel.org/u-boot/20241112171205.4e805...@donnerap.manchester.arm.com/ > > Sig

Re: [PATCH v5 4/8] RFC: Revert "bootstd: Make efi_mgr bootmeth work for non-sandbox setups"

2024-11-13 Thread Tom Rini
On Wed, Nov 13, 2024 at 08:09:34AM -0700, Simon Glass wrote: > This is another option to fix sunxi booting with bootstd, which may be > better since it will work for all boards. We can then figure out how to > automatically and deterministicaly decide when bootmgr should be used. > > This reverts

2025.01-rc2 fails to boot on VisionFive2

2024-11-13 Thread Andreas Schwab
"Error reading cluster" is the new error. U-Boot SPL 2025.01-rc2 (Nov 08 2024 - 07:13:26 +) DDR version: dc2e84f0. Trying to boot from MMC2 U-Boot 2025.01-rc2 (Nov 08 2024 - 07:13:26 +) CPU: sifive,u74-mc Model: StarFive VisionFive 2 v1.2A DRAM: 4 GiB Core: 136 devices, 26 uclasses,

Re: [PATCH v3 05/15] efi_loader: add IPv4() to device path to text protocol

2024-11-13 Thread Ilias Apalodimas
Hi Heinrich, On Mon, 11 Nov 2024 at 23:10, Adriano Cordova wrote: > > From: Heinrich Schuchardt > > Implement Ipv4() node support in the device path to text protocol. > > Signed-off-by: Heinrich Schuchardt > Signed-off-by: Adriano Cordova > --- > > (no changes since v2) > > lib/efi_loader/efi

Re: [PATCH v1] imx: Fix critical thermal threshold

2024-11-13 Thread Francesco Dolcini
Hello Peng, On Wed, Nov 13, 2024 at 05:27:23AM +, Peng Fan wrote: > > Subject: [PATCH v1] imx: Fix critical thermal threshold > > > > From: Francesco Dolcini > > > > Fix the critical thermal threshold for i.MX processors, this was changed > > while moving the code from imx8m/imx9 directorie

Re: [PATCH] mmc: don't print 'MMC:' if there are no MMC devices

2024-11-13 Thread Dragan Simic
Hello Caleb, Thanks for the patch. Please, see a comment below. On 2024-11-13 06:30, Caleb Connolly wrote: It may be the case that MMC support is enabled even though the board we're booting on doesn't have any MMC devices. Move the print over to the print_mmc_devices() function where we can on

Re: [PATCH v2 1/6] arm: imx8m: add OP-TEE node

2024-11-13 Thread Yannic Moog
On Mon, 2024-11-11 at 04:49 -0600, Adam Ford wrote: > On Mon, Nov 11, 2024 at 1:48 AM Yannic Moog wrote: > > > > Hi, > > > > On Fri, 2024-11-08 at 10:05 -0800, Tim Harvey wrote: > > > On Fri, Nov 8, 2024 at 5:49 AM Adam Ford wrote: > > > > > > > > On Thu, Nov 7, 2024 at 2:42 AM Yannic Moog wr

Re: [PATCH v2 0/4] Broadcom BCMBCA nand updates

2024-11-13 Thread Michael Nazzareno Trimarchi
HI On Wed, Nov 13, 2024 at 7:27 AM david regan wrote: > > These are ports of upstream Linux patches to U-Boot. > > Remove device specific nand driver files as they are now replaced > by a common driver bcmbca_nand.c > > Make use of OF_UPSTREAM which uses Linux dts. > > Implement a minimal set of

RE: [PATCH v1] imx: Fix critical thermal threshold

2024-11-13 Thread Peng Fan
> Subject: Re: [PATCH v1] imx: Fix critical thermal threshold > > Hello Peng, > > On Wed, Nov 13, 2024 at 05:27:23AM +, Peng Fan wrote: > > > Subject: [PATCH v1] imx: Fix critical thermal threshold > > > > > > From: Francesco Dolcini > > > > > > Fix the critical thermal threshold for i.MX pr

Re: [PATCH] Revert "efi_memory: do not add U-Boot memory to the memory map"

2024-11-13 Thread Ilias Apalodimas
Hi Sughosh On Tue, 12 Nov 2024 at 20:46, Sughosh Ganu wrote: > > On Tue, 12 Nov 2024 at 18:48, Simon Glass wrote: > > > > A bisect of Ubuntu 2022.04 boot-failure on qemu-x86_64 resulted in this > > patch. I am not sure how to investigate it. > > > > The boot hangs at some point during booting of

Re: [PATCH v2 0/8] led: update LED boot/activity to new property implementation

2024-11-13 Thread Tom Rini
On Sun, Nov 10, 2024 at 12:50:19PM +0100, Christian Marangi wrote: > This series is split in 2 part. > > While adapting the LED boot and activity code to the new property > accepted by Rob in dt-schema repository, a big BUG was discovered. > > The reason wasn't clear at start and took me some da

[PATCH v4 8/9] CI: Add platform variable

2024-11-13 Thread Simon Glass
Add a list of possible platforms that can be used by gitlab runners. Signed-off-by: Simon Glass Reviewed-by: Tom Rini Reviewed-by: Ilias Apalodimas --- (no changes since v3) Changes in v3: - Drop patches which remove grub and tracing for arm64 .gitlab-ci.yml | 1 + 1 file changed, 1 inserti

[PATCH v4 7/9] docker: Install toolchains on arm64 host

2024-11-13 Thread Simon Glass
Refactor the code to support downloading toolchains for arm64 as well as x86_64 There doesn't seem to be an xtensa toolchain for arm64 at the same location, so download that only on x86 Signed-off-by: Simon Glass Reviewed-by: Tom Rini --- (no changes since v3) Changes in v3: - Add patch to in

[PATCH v4 4/9] docker: Adjust packages for arm64

2024-11-13 Thread Simon Glass
The package names are slightly different for arm64 and we don't seem to have a linux-image-kvm package. Provide some extra ones for x86 Signed-off-by: Simon Glass Reviewed-by: Tom Rini --- (no changes since v3) Changes in v3: - Use grub-efi and linux-image-generic for both archs Changes in v2

[PATCH v4 5/9] docker: Drop coreboot for arm64

2024-11-13 Thread Simon Glass
This isn't needed for sandbox builds and coreboot isn't much used on arm64, nor does U-Boot have a build that supports running from coreboot on any ARM target. So drop it. Signed-off-by: Simon Glass Reviewed-by: Tom Rini --- (no changes since v1) tools/docker/Dockerfile | 6 -- 1 file ch

[PATCH v4 6/9] docker: Fix LegacyKeyValueFormat warning with PYTHONPATH

2024-11-13 Thread Simon Glass
Fix a warning due to the syntax used for PYTHONPATH: LegacyKeyValueFormat: "ENV key=value" should be used instead of legacy "ENV key value" format (line 304) Signed-off-by: Simon Glass Reviewed-by: Heinrich Schuchardt Reviewed-by: Tom Rini --- (no changes since v1) tools/docker/Doc

[PATCH v4 9/9] WIP: CI: Update for arm64

2024-11-13 Thread Simon Glass
Do not apply. Switch over to the new multiarch image so we can try a run. Signed-off-by: Simon Glass --- (no changes since v1) .gitlab-ci.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index e152060b046..3275d9ffa45 100644 --- a/.gitlab

Re: [PATCH v5 7/8] env: Provide a work-around for unquoting fdtfile

2024-11-13 Thread Tom Rini
On Wed, Nov 13, 2024 at 08:09:37AM -0700, Simon Glass wrote: > Some boards use a CONFIG option to specify the value of this variable. > This is normally handled by efi_get_distro_fdt_name() but in the case > of sunxi this does not work, since 'soc' is sunxi, but the files are > in the allwinner di

Re: [PATCH 2/2] efi_loader: remove non vital devices first

2024-11-13 Thread Heinrich Schuchardt
Am 13. November 2024 17:03:03 MEZ schrieb Simon Glass : >Hi Heinrich, > >On Wed, 13 Nov 2024 at 08:17, Heinrich Schuchardt >wrote: >> >> Am 13. November 2024 15:39:22 MEZ schrieb Simon Glass : >> >Hi, >> > >> >On Wed, 13 Nov 2024 at 05:52, Heinrich Schuchardt >wrote: >> >> >> >> On 11/1/24 21:29,

[PATCH v4 1/9] sandbox: efi_loader: Correct use of addresses as pointers

2024-11-13 Thread Simon Glass
The cache-flush function is incorrect which causes a crash in the remoteproc tests with arm64. Fix both problems by using map_sysmem() to convert an address to a pointer and map_to_sysmem() to convert a pointer to an address. Also update the image-loader's cache-flushing logic. Signed-off-by: Si

[PATCH v4 2/9] test: Adjust print_ut test to use unsigned char

2024-11-13 Thread Simon Glass
Since char is unsigned on arm64, this test currently fails. It seems better to use unsigned anyway, since 0xff is written into the string at the start. Update the terminator-assert to use a character instead of a byte. Signed-off-by: Simon Glass Reviewed-by: Heinrich Schuchardt --- (no changes

[PATCH v4 0/9] CI: Set up for an arm64 runner

2024-11-13 Thread Simon Glass
All gitlab runners are currently amd64 machines. This series attempts to create a docker image which can also support arm64 so that sandbox tests can be run on it. The TARGET_... environment variables for grub could perhaps be adjusted, using the new variables, but I have not done that for now. C

[PATCH v4 3/9] docker: Support building for multiple architectures

2024-11-13 Thread Simon Glass
Add instructions on how to build the file for multiple architectures. Add a message indicating what is happening. Update the documentation as well. Signed-off-by: Simon Glass --- Changes in v4: - Leave the 'sudo' in Changes in v3: - Move ARGs to the top - Revise documentation to explain a comm

Re: [PATCH v2 3/3] board: phytec: imx93: Add phyCORE-i.MX 93 support for all SOM variants

2024-11-13 Thread Wadim Egorov
Am 13.11.24 um 17:00 schrieb Christoph Stoidner: The phyCORE-i.MX 93 is available in various variants (e.g. different ram sizes, eMMC HS400 yes/no). Enable hardware introspection for the imx93-phyboard-segin_defconfig, so that during startup the SOM module variant can be detected, and the hard

[PATCH 1/1] configs: disable UEFI on rcar3_salvator-x_defconfig

2024-11-13 Thread Heinrich Schuchardt
rcar3_salvator-x_defconfig fails to build with UEFI support on current Ubuntu as u-boot-nodtb.bin becomes too big: u-boot-nodtb.bin exceeds file size limit: limit: 0x10 bytes actual: 0x1010f8 bytes excess: 0x10f8 bytes Reported-by: Adriano Cordova Signed-off-by: Heinri

Re: [PATCH v2 2/3] board: phytec: imx93: Add eeprom-based hardware introspection

2024-11-13 Thread Wadim Egorov
Am 13.11.24 um 17:00 schrieb Christoph Stoidner: The phyCORE-i.MX 93 is available in various variants. Relevant variant options for the spl/u-boot are: - with or without HS400 support for the eMMC - with 1GB ram chip, or 2GB ram chip The phyCORE's eeprom contains all information about the exi

Re: [Upstream] [PATCH v2 0/3] Add support for all variants of the phyCORE-i.MX93 SOM

2024-11-13 Thread Wadim Egorov
Am 13.11.24 um 17:00 schrieb Christoph Stoidner: The phyCORE-i.MX 93 is available in various variants (e.g. different ram sizes, eMMC HS400 yes/no). It's eeprom contains information which features the existing module supports. Extend the existing board-code to support all the SOM variants. Th

Re: [PATCH v3 4/6] net: lwip: Enable https:// support for wget

2024-11-13 Thread Simon Glass
HI Ilias, On Mon, 11 Nov 2024 at 07:20, Ilias Apalodimas wrote: > > Thanks Simon, > > On Mon, 11 Nov 2024 at 15:03, Simon Glass wrote: > > > > On Sun, 10 Nov 2024 at 01:32, Ilias Apalodimas > > wrote: > > > > > > With the recent changes of lwip & mbedTLS we can now download from > > > https://

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

2024-11-13 Thread Simon Glass
Hi Heinrich, On Tue, 12 Nov 2024 at 03:26, Heinrich Schuchardt wrote: > > Without 'nvme scan' the ESP on the NVMe drive is not found early. > EFI variables cannot be persisted. > > Hit any key to stop autoboot: 0 > Cannot persist EFI variables without system partition > ** Booting bo

Re: [PATCH v2 1/4] lib: provide function hextoull()

2024-11-13 Thread Simon Glass
Hi Heinrich, On Sun, 3 Nov 2024 at 15:42, Heinrich Schuchardt wrote: > > We often convert hexadecimal strings to hextoull(). Provide a wrapper > function to simple_strtoull() that does not require specifying the radix. > > Signed-off-by: Heinrich Schuchardt > --- > v2: > no change > ---

Re: [PATCH] fs: fat: allow FS and device to have mismatching sector sizes

2024-11-13 Thread Simon Glass
Hi Caleb, On Tue, 12 Nov 2024 at 21:50, Caleb Connolly wrote: > > The Qualcomm Robotics RB1 and RB2 boards feature MMC storage provisioned > with a 4k sector size GPT, with factory partitions formatted with > filesystems using 512 byte sector sizes. > > Whilst certainly unusual, and really not id

[PATCH v5 0/8] bootstd: sunxi: Migrate to standard boot

2024-11-13 Thread Simon Glass
This series attempts to migrate all sunxi boards to use standard boot, along with a text environment. Unfortunately a late-breaking problem has come up with bootmgr. There are two ways to fix this, both included in this series: - Drop bootmgr for sunxi - Change efi_mgr bootmeth back to sand

[PATCH v5 5/8] sunxi: Move to bootstd

2024-11-13 Thread Simon Glass
Drop support for distroboot and move to using bootstd instead. Signed-off-by: Simon Glass --- (no changes since v2) Changes in v2: - Convert the other DISTRO_DEFAULTS in the Kconfig too arch/arm/Kconfig | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/arch/arm/Kc

[PATCH v5 2/8] sunxi: Add a bootmeth for FEL

2024-11-13 Thread Simon Glass
Add support for booting from a script loaded over FEL. This mirrors the bootcmd_fel provided by distro boot. Signed-off-by: Simon Glass Reviewed-by: Mattijs Korpershoek --- (no changes since v4) Changes in v4: - Reword Kconfig - Add a name for the bootflow Changes in v2: - Put the FEL bootmet

Re: [PATCH v3 4/6] net: lwip: Enable https:// support for wget

2024-11-13 Thread Simon Glass
Hi Ilias, On Wed, 13 Nov 2024 at 07:46, Ilias Apalodimas wrote: > > On Wed, 13 Nov 2024 at 15:39, Simon Glass wrote: > > > > HI Ilias, > > > > On Mon, 11 Nov 2024 at 07:20, Ilias Apalodimas > > wrote: > > > > > > Thanks Simon, > > > > > > On Mon, 11 Nov 2024 at 15:03, Simon Glass wrote: > > >

[PATCH v5 1/8] bootstd: Avoid depending on BLK

2024-11-13 Thread Simon Glass
In principle bootstd can work without block devices, even if it does require driver model to be enabled in that case. The use of a 'depends on BLK' for BOOTSTD conflicts with the way 'BLK' is now defined, producing recursive errors through multiple different paths, one of which is this (with Links

Re: [PATCH v3 4/6] net: lwip: Enable https:// support for wget

2024-11-13 Thread Ilias Apalodimas
On Wed, 13 Nov 2024 at 17:09, Simon Glass wrote: > > Hi Ilias, > > On Wed, 13 Nov 2024 at 07:46, Ilias Apalodimas > wrote: > > > > On Wed, 13 Nov 2024 at 15:39, Simon Glass wrote: > > > > > > HI Ilias, > > > > > > On Mon, 11 Nov 2024 at 07:20, Ilias Apalodimas > > > wrote: > > > > > > > > Thank

[PATCH v5 7/8] env: Provide a work-around for unquoting fdtfile

2024-11-13 Thread Simon Glass
Some boards use a CONFIG option to specify the value of this variable. This is normally handled by efi_get_distro_fdt_name() but in the case of sunxi this does not work, since 'soc' is sunxi, but the files are in the allwinner directory. Provide a work-around for this particular case. Signed-off-

[PATCH v5 8/8] sunxi: Move to text environment

2024-11-13 Thread Simon Glass
Convert these boards to use a text environment. For the boards check, the only differences are extra spaces after the semicolons in 'dfu_alt_info_ram' and 'partitions', both of which are permitted. Add in the special boot command for old kernels, dropping the unnecessary and confusing hex prefixe

[PATCH v5 4/8] RFC: Revert "bootstd: Make efi_mgr bootmeth work for non-sandbox setups"

2024-11-13 Thread Simon Glass
This is another option to fix sunxi booting with bootstd, which may be better since it will work for all boards. We can then figure out how to automatically and deterministicaly decide when bootmgr should be used. This reverts commit f2bfa0cb17948aa4a0fa20fdf9014296b9c4d9c7. Signed-off-by: Simon

[PATCH v5 6/8] sunxi: Drop old distro boot variables

2024-11-13 Thread Simon Glass
These are not needed as bootstd handles the boot now. Drop them. Keep BOOTCMD_SUNXI_COMPAT for now since it does not relate to distro boot. Signed-off-by: Simon Glass --- (no changes since v2) Changes in v2: - Keep BOOTCMD_SUNXI_COMPAT include/configs/sunxi-common.h | 75 +---

Re: Using secondary UART for board initialization

2024-11-13 Thread Alexander Koch
> All `serial_putc()`-like functions that I could find are either static to the > actual driver and thus not accessible from `board_init()` or do not accept a > `struct udevice *` as a parameter and seem to work on the main U-Boot serial > console device only. Reading other board code I think I ha

Re: [PATCH 2/2] efi_loader: remove non vital devices first

2024-11-13 Thread Simon Glass
Hi, On Wed, 13 Nov 2024 at 05:52, Heinrich Schuchardt wrote: > > On 11/1/24 21:29, Mark Kettenis wrote: > >> From: Janne Grunau > >> Date: Thu, 31 Oct 2024 23:48:02 +0100 > >> > >> DM_FLAG_VITAL marks devices which are essential for the operation of > >> other devices. Removing these devices bef

Re: [PATCH] nwt: wget: drop Content-Length processing

2024-11-13 Thread Jerome Forissier
On 11/11/24 13:47, Ilias Apalodimas wrote: > Hi Jerome, > > On Tue, 5 Nov 2024 at 13:09, Jerome Forissier > wrote: >> >> We don't do anything with Content-Length except a debug print, and the >> strict_strtoul() call is incorrect (it always returns -EINVAL and leaves >> content_length to zero

Re: [PATCH 2/2] efi_loader: remove non vital devices first

2024-11-13 Thread Heinrich Schuchardt
Am 13. November 2024 15:39:22 MEZ schrieb Simon Glass : >Hi, > >On Wed, 13 Nov 2024 at 05:52, Heinrich Schuchardt wrote: >> >> On 11/1/24 21:29, Mark Kettenis wrote: >> >> From: Janne Grunau >> >> Date: Thu, 31 Oct 2024 23:48:02 +0100 >> >> >> >> DM_FLAG_VITAL marks devices which are essential fo

Re: [PATCH v2 1/4] lib: provide function hextoull()

2024-11-13 Thread Tom Rini
On Wed, Nov 13, 2024 at 07:42:57AM -0700, Simon Glass wrote: > Hi Tom, > > On Wed, 13 Nov 2024 at 07:12, Tom Rini wrote: > > > > On Wed, Nov 13, 2024 at 06:39:43AM -0700, Simon Glass wrote: > > > Hi Heinrich, > > > > > > On Sun, 3 Nov 2024 at 15:42, Heinrich Schuchardt > > > wrote: > > > > > > >

Re: [PATCH v3 3/8] bootstd: Avoid depending on BLK

2024-11-13 Thread Tom Rini
On Wed, Nov 13, 2024 at 07:39:53AM -0700, Simon Glass wrote: > Hi Tom, > > On Fri, 18 Oct 2024 at 12:48, Simon Glass wrote: > > > > Hi Tom, > > > > On Fri, 18 Oct 2024 at 12:29, Tom Rini wrote: > > > > > > On Fri, Oct 18, 2024 at 12:06:46PM -0600, Simon Glass wrote: > > > > Hi Tom, > > > > > > >

Re: [PATCH v2 1/4] lib: provide function hextoull()

2024-11-13 Thread Simon Glass
Hi Tom, On Wed, 13 Nov 2024 at 07:12, Tom Rini wrote: > > On Wed, Nov 13, 2024 at 06:39:43AM -0700, Simon Glass wrote: > > Hi Heinrich, > > > > On Sun, 3 Nov 2024 at 15:42, Heinrich Schuchardt > > wrote: > > > > > > We often convert hexadecimal strings to hextoull(). Provide a wrapper > > > func

Re: [PATCH v3 3/8] bootstd: Avoid depending on BLK

2024-11-13 Thread Simon Glass
Hi Tom, On Fri, 18 Oct 2024 at 12:48, Simon Glass wrote: > > Hi Tom, > > On Fri, 18 Oct 2024 at 12:29, Tom Rini wrote: > > > > On Fri, Oct 18, 2024 at 12:06:46PM -0600, Simon Glass wrote: > > > Hi Tom, > > > > > > On Fri, 18 Oct 2024 at 11:52, Tom Rini wrote: > > > > > > > > On Fri, Oct 18, 202

Re: [PATCH 3/9] buildman: Support #include files in defconfigs

2024-11-13 Thread Simon Glass
Hi Tom, On Tue, 12 Nov 2024 at 19:40, Tom Rini wrote: > > On Fri, Nov 08, 2024 at 08:23:44AM -0700, Simon Glass wrote: > > > This is used by some boards in U-Boot and is a convenient way to deal > > with common settings where using a Kconfig files is not desirable. > > > > Detect #include files a

Re: [PATCH v2] cmd: add a fetch utility

2024-11-13 Thread Heinrich Schuchardt
Am 13. November 2024 14:32:57 MEZ schrieb Neil Armstrong : >On 13/11/2024 05:22, Caleb Connolly wrote: >> Add a small utility for displaying some information about U-Boot and the >> hardware it's running on in a similar fashion to the popular neofetch >> tool for Linux [1]. >> >> While the output

Re: [PATCH v3 4/6] net: lwip: Enable https:// support for wget

2024-11-13 Thread Ilias Apalodimas
On Wed, 13 Nov 2024 at 15:39, Simon Glass wrote: > > HI Ilias, > > On Mon, 11 Nov 2024 at 07:20, Ilias Apalodimas > wrote: > > > > Thanks Simon, > > > > On Mon, 11 Nov 2024 at 15:03, Simon Glass wrote: > > > > > > On Sun, 10 Nov 2024 at 01:32, Ilias Apalodimas > > > wrote: > > > > > > > > With

Re: [PATCH] mmc: don't print 'MMC:' if there are no MMC devices

2024-11-13 Thread Caleb Connolly
On 13/11/2024 15:24, Tom Rini wrote: > On Wed, Nov 13, 2024 at 06:30:08AM +0100, Caleb Connolly wrote: > >> It may be the case that MMC support is enabled even though the board >> we're booting on doesn't have any MMC devices. Move the print over to >> the print_mmc_devices() function where we

Re: [PATCH v2] cmd: add a fetch utility

2024-11-13 Thread Neil Armstrong
On 13/11/2024 15:40, Heinrich Schuchardt wrote: Am 13. November 2024 14:32:57 MEZ schrieb Neil Armstrong : On 13/11/2024 05:22, Caleb Connolly wrote: Add a small utility for displaying some information about U-Boot and the hardware it's running on in a similar fashion to the popular neofetch t

Re: [PATCH v2] cmd: add a fetch utility

2024-11-13 Thread Simon Glass
On Tue, 12 Nov 2024 at 21:26, Caleb Connolly wrote: > > Add a small utility for displaying some information about U-Boot and the > hardware it's running on in a similar fashion to the popular neofetch > tool for Linux [1]. > > While the output is meant to be useful, it should also be pleasing to >

  1   2   >