Re: [PATCH v1 01/22] wdt: imx8qxp: add option to control external PMIC wdt via IMX8 SCU

2024-11-07 Thread Stefan Roese
Hi Heiko, On 11/8/24 06:21, Heiko Schocher wrote: control the watchdog in the PMIC through SCU. A few more details would be good here (please see below). Signed-off-by: Andrej Valek Signed-off-by: Heiko Schocher --- drivers/misc/imx8/scu_api.c| 21 ++ drivers/watchdog/K

Re: [PATCH v10 02/37] acpi: x86: Write FADT in common code

2024-11-07 Thread Patrick Rudolph
Hi Heinrich, On Thu, Nov 7, 2024 at 10:19 AM Heinrich Schuchardt wrote: > > On 10/23/24 15:19, Patrick Rudolph wrote: > > From: Maximilian Brune > > > > Write the FADT in common code since it's used on all architectures. > > Since the FADT is mandatory all SoCs or mainboards must implement the >

Re: [PATCH 02/15] net: wget: let wget_with_dns work with dns disabled

2024-11-07 Thread Heinrich Schuchardt
Am 8. November 2024 04:45:01 MEZ schrieb Adriano Cordova : >This was marked as TODO in the code: >-Enable use of wget_with_dns even if CMD_DNS is disabled if >the given uri has the ip address for the http server. >-Check for port in the uri when transforming to legacy wget >syntax inside wget_with_

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

2024-11-07 Thread Heinrich Schuchardt
Am 8. November 2024 04:44:44 MEZ schrieb Adriano Cordova : >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 ip_to_string in net_utils.c and removes it fro

[PATCH v2] cadence_qspi: Refactor the flash reset functionality

2024-11-07 Thread Venkatesh Yadav Abbarapu
As the flash reset is handled in spi nor core, removing the flash reset functionality. As the configuration like tristate and hysterisis need to be enabled by the cdo. Handle the flash reset only for mini u-boot case. Rename the "cadence_qspi_versal_flash_reset" to generic name "cadence_qspi_flash

[PATCH v1 13/22] siemens: configs/capricorn_cxg3_defconfig: updates

2024-11-07 Thread Heiko Schocher
make savedefconfig and add SCU_WDT and fix environment offsets, as since silicium c0 the boot container takes place at offset 0 and so the u-boot-env must be moved outside of the boot container area. Signed-off-by: Heiko Schocher --- configs/capricorn_cxg3_defconfig | 16 +++- 1 fil

[PATCH v1 10/22] siemens: imximage.cfg: sync image names

2024-11-07 Thread Heiko Schocher
sync the image names in imximage.cfg with the ones used in arch/arm/dts/imx8qxp-u-boot.dtsi Signed-off-by: Heiko Schocher --- board/siemens/capricorn/imximage.cfg | 14 ++ 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/board/siemens/capricorn/imximage.cfg b/board/si

[PATCH v1 08/22] siemens: capricorn: use DCD_SKIP entry

2024-11-07 Thread Heiko Schocher
Boards which use DCD data in SCFW can drop SPL. We tried in our mainline rework to use this approach too as other imx8qxp boards do in mainline. But we failed ... it was a hard way to understand the reason! We cannot use DCD image in container as the SCFW from siemens, does the RAM init on boot i

[PATCH v1 22/22] siemens: capricorn: update maintainers

2024-11-07 Thread Heiko Schocher
From: Enrico Leto update MAINTAINERS file, add some more board maintainers. Signed-off-by: Enrico Leto Signed-off-by: Heiko Schocher --- board/siemens/capricorn/MAINTAINERS | 12 +++- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/board/siemens/capricorn/MAINTAINERS b

[PATCH v1 21/22] siemens: add ddr signal integrity test

2024-11-07 Thread Heiko Schocher
From: Enrico Leto The signal integrity test generates pattern on DDR lines for certification. The signals must be as fast as possible and unidirectional. The test is required from our HW team. The available u-boot memory test doesn't full fill the our requirements. The test is planed to be used

[PATCH v1 20/22] siemens: add ddr full memory test

2024-11-07 Thread Heiko Schocher
Add siemens specific memory test. Enable it through Kconfig option SPL_CMT. The test is required from our HW team. It runs over temperature during many days: * must run indefinitively through the whole DDR area * must write/read/check all values Signed-off-by: Enrico Leto Signed-off-by: Heiko Sch

[PATCH v1 19/22] siemens: capricorn: get module name from eeprom

2024-11-07 Thread Heiko Schocher
From: Enrico Leto The eeprom contains the information on which module we are running, so read it from the eeprom and print it on the console. Signed-off-by: Enrico Leto Signed-off-by: Heiko Schocher --- board/siemens/capricorn/board.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a

[PATCH v1 18/22] siemens: capricorn: get ram size from system controller

2024-11-07 Thread Heiko Schocher
From: Enrico Leto Get the memory region information from system controller to reduce the number of platform specific headers. We were aligned on NXP mek board implementation. This need at least 1 header per memory configuration. Signed-off-by: Enrico Leto Signed-off-by: Heiko Schocher --- bo

[PATCH v1 17/22] siemens: capricorn: add HW version information to boot log

2024-11-07 Thread Heiko Schocher
Add the HW version read directly from EEPROM. EEPROM chip data structure is now in a .h file common to draco and capricorn. Therefore move out the definitions in drace board to siemens common place. Signed-off-by: Alessandro Zini Signed-off-by: Heiko Schocher --- board/siemens/capricorn/boar

[PATCH v1 15/22] siemens: imx8-capricorn.dtsi: small adaptions

2024-11-07 Thread Heiko Schocher
- remove DMA for console - add wdt device Signed-off-by: Heiko Schocher --- arch/arm/dts/imx8-capricorn.dtsi | 16 1 file changed, 16 insertions(+) diff --git a/arch/arm/dts/imx8-capricorn.dtsi b/arch/arm/dts/imx8-capricorn.dtsi index 4918bf8f567..f6f9dc7df19 100644 --- a/arch

[PATCH v1 16/22] siemens: capricorn: board.c fixes

2024-11-07 Thread Heiko Schocher
show build_info and only init uart2. Signed-off-by: Heiko Schocher --- board/siemens/capricorn/board.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/board/siemens/capricorn/board.c b/board/siemens/capricorn/board.c index ad474d9baa0..99681d68c3f 100644 --- a/board/sie

[PATCH v1 14/22] siemens: capricorn: sync spl code with 8qxp-mek

2024-11-07 Thread Heiko Schocher
sync spl code with 8qxp-mek board. Signed-off-by: Heiko Schocher --- board/siemens/capricorn/spl.c | 38 +++ 1 file changed, 38 insertions(+) diff --git a/board/siemens/capricorn/spl.c b/board/siemens/capricorn/spl.c index 696b5ebd340..7ee2895b6d4 100644 --- a/b

[PATCH v1 12/22] siemens: capricorn: add missing ARCH_MISC_INIT

2024-11-07 Thread Heiko Schocher
add ARCH_MISC_INIT to capricorn configuration. Add it in arch/arm/mach-imx/imx8/Kconfig Signed-off-by: Heiko Schocher --- arch/arm/mach-imx/imx8/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm/mach-imx/imx8/Kconfig b/arch/arm/mach-imx/imx8/Kconfig index 9a43beda6fa..bfe30be

[PATCH v1 11/22] siemens: imx8-capricorn-u-boot.dtsi: add fec2

2024-11-07 Thread Heiko Schocher
we use also fec2, so add it. Signed-off-by: Heiko Schocher --- arch/arm/dts/imx8-capricorn-u-boot.dtsi | 4 1 file changed, 4 insertions(+) diff --git a/arch/arm/dts/imx8-capricorn-u-boot.dtsi b/arch/arm/dts/imx8-capricorn-u-boot.dtsi index a1058a09a9a..365cd51ed2f 100644 --- a/arch/arm/

[PATCH v1 09/22] siemens: imximage.cfg: correct comment

2024-11-07 Thread Heiko Schocher
fix wrong comment. Signed-off-by: Heiko Schocher --- board/siemens/capricorn/imximage.cfg | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/board/siemens/capricorn/imximage.cfg b/board/siemens/capricorn/imximage.cfg index e45f2c9589e..f73c5a7a9b0 100644 --- a/board/siemens/

[PATCH v1 06/22] siemens: capricorn: move to cxg3 reference project with deneb board

2024-11-07 Thread Heiko Schocher
From: Enrico Leto We have many HW with capricorn i.MX8X boards. The difference in u-boot is at all by the display of the LEDs. * put upstream a reference project & board for DT and defconfig * use the capricorn prefix outside the board/siemens/capricorn folder Signed-off-by: Enrico Leto Signed

[PATCH v1 07/22] siemens: imx8qxp-capricorn-u-boot.dtsi: fix boot

2024-11-07 Thread Heiko Schocher
current generated flash.bin image does not longer boot on cxg3 board. Rename bootph-pre-ram to bootph-all so flash.bin boots again! Signed-off-by: Heiko Schocher --- arch/arm/dts/imx8-capricorn-u-boot.dtsi | 65 - 1 file changed, 32 insertions(+), 33 deletions(-) diff

[PATCH v1 05/22] imx8qxp: Fix build when using SPL

2024-11-07 Thread Heiko Schocher
imx8qxp based boards which use SPL drop error when calling make all: """ Writing image to './flash.bin' Node '/binman/imx-boot/spl': GetData: size 0x0 Node '/binman/imx-boot': GetPaddedDataForEntry: size 0x0 Node '/binman/imx-boot': GetData: 1 entries, total size 0x0 Node '/

[PATCH v1 02/22] net: fec_mxc: fix probing for imx8qxp

2024-11-07 Thread Heiko Schocher
probing on capricorn board (imx8qxp based) brings: Can't find FEC0 clk rate: -19 Cause is that when probing fec_mxc driver, fec_mii_setspeed() is called which calls fec_get_clk_rate(). fec_mii_setspeed() calls fec_get_clk_rate with NULL pointer for udev and so as in IMX8QXP case CLK_CCF is enabl

[PATCH v1 04/22] imx: imx_cntr_image.sh: prevent warning for missing spl

2024-11-07 Thread Heiko Schocher
when building U-Boot on imx8qxp and the board port uses SPL, U-boot build shows WARNING '.../spl/u-boot-spl.bin' not found, resulting binary is not-functional This is because U-Boot binary is build first and Makefile calls script imx_cntr_image.sh which checks if files exists... but of course as

[PATCH v1 03/22] tools: imx8image: Improve error message

2024-11-07 Thread Heiko Schocher
Improve error message "header tag mismatched" Add filename to error message to see, which file is wrong. Signed-off-by: Heiko Schocher --- tools/imx8image.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/imx8image.c b/tools/imx8image.c index 96ece28bd6c..b22e8b21c47 1

[PATCH v1 01/22] wdt: imx8qxp: add option to control external PMIC wdt via IMX8 SCU

2024-11-07 Thread Heiko Schocher
control the watchdog in the PMIC through SCU. Signed-off-by: Andrej Valek Signed-off-by: Heiko Schocher --- drivers/misc/imx8/scu_api.c| 21 ++ drivers/watchdog/Kconfig | 7 + drivers/watchdog/Makefile | 1 + drivers/watchdog/scu_wdt.c | 52

[PATCH v10 3/7] usb: onboard-hub: add support for Microchip USB5744

2024-11-07 Thread Venkatesh Yadav Abbarapu
Add support for the Microchip USB5744 USB3.0 and USB2.0 Hub. The usb5744 driver trigger hub reset signal after soft reset. The usb5744 hub need to reset after the phy initialization, which toggles the gpio. Signed-off-by: Venkatesh Yadav Abbarapu Reviewed-by: Marek Vasut --- common/usb_onboard_

[PATCH v10 7/7] arm64: zynqmp: Update the usb5744 hub node as per binding

2024-11-07 Thread Venkatesh Yadav Abbarapu
Updating the usb5744 hub node as per the latest upstream DT binding https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/ tree/Documentation/devicetree/bindings/usb/microchip,usb5744.yaml?h=v6.8.8 Signed-off-by: Venkatesh Yadav Abbarapu Acked-by: Michal Simek --- arch/arm/dts/zynqmp

[PATCH v10 1/7] usb: onboard-hub: Add reset-gpio support

2024-11-07 Thread Venkatesh Yadav Abbarapu
As part of the reset, sets the direction of the pin to output before toggling the pin. Delay of millisecond is added in between low and high to meet the setup and hold time requirement of the reset. Signed-off-by: Venkatesh Yadav Abbarapu Reviewed-by: Marek Vasut --- common/usb_onboard_hub.c |

[PATCH v10 6/7] configs: zynqmp_kria: Enable the USB onboard hub

2024-11-07 Thread Venkatesh Yadav Abbarapu
USB host support on ZYNQMP KRIA SOM needs onboard USB hub driver for handling reset GPIO and for i2c initialization sequence. Signed-off-by: Venkatesh Yadav Abbarapu Acked-by: Michal Simek --- configs/xilinx_zynqmp_kria_defconfig | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --g

[PATCH v10 0/7] Add the USB5744 hub driver as per new DT binding

2024-11-07 Thread Venkatesh Yadav Abbarapu
Add the usb5744/usb2744 hub driver which does the reset gpio toggling and the i2c initialization sequence. Tested the USB5744/USB2744 usb hub for usb0, usb1 with the DT nodes on KR260 board. Changes in v2: - Added the power_on_reset_us variable, for post-reset time. - Removed the DM_REGULATOR ifd

[PATCH v10 4/7] usb: onboard-hub: Add i2c initialization for usb5744 hub

2024-11-07 Thread Venkatesh Yadav Abbarapu
Add i2c initialization hook and set usb5744 platform data with function having required i2c initialization sequence. Apart from the USB command attach, prevent the hub from suspend. when the “USB Attach with SMBUS (0xAA56)” command is issued to the hub, the hub is getting enumerated and then it pu

[PATCH v10 2/7] usb: onboard-hub: Fix the return values of regulator APIs

2024-11-07 Thread Venkatesh Yadav Abbarapu
Don't error out if there is no vdd regulator supply, as these are optional properties. Signed-off-by: Venkatesh Yadav Abbarapu Reviewed-by: Marek Vasut --- common/usb_onboard_hub.c | 12 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/common/usb_onboard_hub.c b/common

[PATCH v10 5/7] usb: onboard-hub: Bail out if peer hub is already probed

2024-11-07 Thread Venkatesh Yadav Abbarapu
The .bind function is implemented to bind the correct "half" of the hub that the driver wants to bind, and returning -ENODEV for the other "half". Signed-off-by: Venkatesh Yadav Abbarapu Reviewed-by: Marek Vasut --- common/usb_onboard_hub.c | 21 + 1 file changed, 21 inserti

Re: [PATCH v6 3/3] board: rockchip: Add support for rk3588 GenBook

2024-11-07 Thread Kever Yang
On 2024/11/4 20:23, Andy Yan wrote: Add support for Cool Pi GenBook, it works as a carrier board connect with CM5 SOM. Specification: - Rockchip RK3588 - LPDDR5X 8/32 GB - eMMC 64 GB - HDMI Type A out x 1 - USB 3.0 Host x 1 - USB-C 3.0 with DisplayPort AltMode - PCIE M.2 E Key for RTL8852BE Wi

Re: [PATCH v6 2/3] rockchip: Make it possible to define per board boot_targets

2024-11-07 Thread Kever Yang
On 2024/11/4 20:22, Andy Yan wrote: Some board may want to have a different boot priority(a laptop may want to usb has the highest boot priority for third-part os installation). So let the board can define it's own boot_targets. Signed-off-by: Andy Yan Reviewed-by: Kever Yang Thanks, - Ke

Re: [PATCH v6 1/3] arm64: dts: rockchip: Add support for rk3588 based Cool Pi CM5 GenBook

2024-11-07 Thread Kever Yang
On 2024/11/4 20:22, Andy Yan wrote: Cool Pi CM5 GenBook works as a carrier board connect with CM5 [0]. Specification: - Rockchip RK3588 - LPDDR5X 8/32 GB - eMMC 64 GB - HDMI Type A out x 1 - USB 3.0 Host x 1 - USB-C 3.0 with DisplayPort AltMode - PCIE M.2 E Key for RTL8852BE Wireless connectio

Re: [PATCH v2 2/3] rockchip: rk3588: Implement checkboard() to print SoC variant

2024-11-07 Thread Kever Yang
Hi Jonas, On 2024/11/7 22:29, Jonas Karlman wrote: Hi Quentin, On 2024-11-07 13:01, Quentin Schulz wrote: Hi Jonas, On 11/2/24 9:37 PM, Jonas Karlman wrote: Implement checkboard() to print current SoC model used by a board, e.g. one of: SoC: RK3582 v1 SoC: RK3588 v0 SoC: R

Re: [PATCH next v2 4/4] rockchip: rk3399: move sysreset-gpio logic to TPL

2024-11-07 Thread Kever Yang
On 2024/11/6 19:29, Quentin Schulz wrote: From: Quentin Schulz If TPL_GPIO and TPL_PINCTRL_ROCKCHIP are enabled and a sysreset-gpio is provided in the TPL Device Tree, this will trigger a system reset similar to what's currently been done in SPL whenever the RK3399 "warm" boots. Because there

Re: [PATCH next v2 3/4] rockchip: tpl: allow to call board/SoC-specific code before DRAM init

2024-11-07 Thread Kever Yang
On 2024/11/6 19:29, Quentin Schulz wrote: From: Quentin Schulz This defines a weak tpl_board_init function that can be used for running board/SoC-specific code before the DRAM init happens, similarly to spl_board_init() for SPL. Reviewed-by: Paul Kocialkowski Signed-off-by: Quentin Schulz

Re: [PATCH next v2 2/4] rockchip: rk3399: merge CRU check within rk3399_force_power_on_reset

2024-11-07 Thread Kever Yang
On 2024/11/6 19:29, Quentin Schulz wrote: From: Quentin Schulz To prepare to support forcing power on reset from TPL which would have the exact same logic, just in an earlier stage, let's merge the CRU check that triggers the power on reset with the rest of the logic. Reviewed-by: Paul Kocia

Re: [PATCH next v2 1/4] pinctrl: rockchip: allow to build for TPL

2024-11-07 Thread Kever Yang
On 2024/11/6 19:29, Quentin Schulz wrote: From: Quentin Schulz A later commit will make use of the pinctrl driver in TPL so let's add the ability to build the Rockchip pinctrl driver in TPL. Reviewed-by: Paul Kocialkowski Signed-off-by: Quentin Schulz Reviewed-by: Kever Yang Thanks, - K

Re: [PATCH next RFC 5/5] ram: rk3399: fail DRAM init when pctl channel init fails instead of hanging

2024-11-07 Thread Kever Yang
Hi Quentin, On 2024/11/6 01:21, Quentin Schulz wrote: From: Quentin Schulz Instead of hanging via an infinite while loop, propagate the fail to the caller and let it handle the fail. For RK3399, this means that panic() will be called, (by default) resetting the CPU and giving another chance at

RE: [PATCH 3/4] mtd: rawnand: brcmnand: Add BCMBCA read data bus interface

2024-11-07 Thread William Zhang
Hi Linus, > -Original Message- > From: Linus Walleij > Sent: Thursday, November 7, 2024 6:16 AM > To: david regan > Cc: u-boot ; Dario Binacchi > ; Michael Trimarchi > ; Anand Gore > ; William Zhang > ; Kursad Oney > ; Philippe Reynes > ; Florian Fainelli > ; Miquel Raynal > ; Kamal Dasu

Re: [PATCH v2 0/2] Fix boot failure due to misaligned DMA buffer

2024-11-07 Thread Tony Dinh
Hi Nam, On Thu, Nov 7, 2024 at 1:53 PM Nam Cao wrote: > > Hi, > > We observed the following sporadic boot failure while booting from MMC > device: > > => boot > CACHE: Misaligned operation at range [9efa25f8, 9efa27f8] > CACHE: Misaligned operation at range [9efa25f8, 9efa27f8] > CACHE: Misalign

Re: [PATCH next v2 4/4] rockchip: rk3399: move sysreset-gpio logic to TPL

2024-11-07 Thread Kever Yang
Hi Paul, On 2024/11/7 20:27, Paul Kocialkowski wrote: Hi, Le Thu 07 Nov 24, 12:04, Quentin Schulz a écrit : On 11/7/24 8:14 AM, Kever Yang wrote: Yes, this movement is reasonable and needed for this workaround, although still not understand why puma board need this. Me neither, this predate

[PATCH v10 00/10] efi: Add a test for EFI bootmeth

2024-11-07 Thread Simon Glass
The test coverage for the EFI bootmeth is incomplete since it does not actually boot the application. This series creates a simple test for this purpose. It includes a few patches to make this work: - Revert of an unreviewed patch to change the sandbox efi filename - Hang in sandbox virtio due to

Re: [PATCH v3 00/25] led: Remove old status-LED code

2024-11-07 Thread Peter Robinson
Hi Simon, > On Tue, 5 Nov 2024 at 07:18, Peter Robinson wrote: > > > > On Sun, 3 Nov 2024 at 00:34, Simon Glass wrote: > > > > > > There has been an LED framework in U-Boot which uses driver model for > > > about 9 years now. Recent work is underway to improve it and provide > > > more features.

Zeroing Unused Memory in U-Boot

2024-11-07 Thread Friedrich Doku
Hi, I hope this email finds you well. I am currently working on a project involving U-Boot, and I have a question regarding memory management. Specifically, I would like to understand the best approach to zero all memory that is not currently in use. I am considering using memset to achieve this,

[PATCH v2 2/2] fs: Use ARCH_DMA_MINALIGN as default alignment for fs_read_alloc()

2024-11-07 Thread Nam Cao
The comment above fs_read_alloc() explains: @align: Alignment to use for memory allocation (0 for default) However, in the actual implementation, there is no alignment when @align is zero. This current default is probably fine for most cases. But for some block devices which transfer data vi

[PATCH v2 1/2] boot: extlinux: Fix unaligned buffer for reading data from file system

2024-11-07 Thread Nam Cao
extlinux_read_bootflow() allocates a buffer to read from file system without any alignment. But for some block devices which transfer data via DMA, ARCH_DMA_MINALIGN alignment is required. For example, due to misaligned buffer, the below boot failure is observed. => boot CACHE: Misaligned operati

[PATCH v2 0/2] Fix boot failure due to misaligned DMA buffer

2024-11-07 Thread Nam Cao
Hi, We observed the following sporadic boot failure while booting from MMC device: => boot CACHE: Misaligned operation at range [9efa25f8, 9efa27f8] CACHE: Misaligned operation at range [9efa25f8, 9efa27f8] CACHE: Misaligned operation at range [9efa25f8, 9efa27f8] CACHE: Misaligned operation at r

[PATCH v10 10/10] test: efi: boot: Add a test for the efi bootmeth

2024-11-07 Thread Simon Glass
Add a simple test of booting with the EFI bootmeth, which runs the app and checks that it can call 'exit boot-services' (to check that all the device-removal code doesn't break anything) and then exit back to U-Boot. This uses a disk image containing the testapp, ready for execution by sandbox whe

[PATCH v10 09/10] test: efi: boot: Set up an image suitable for EFI testing

2024-11-07 Thread Simon Glass
Create a new disk for use with tests, which contains the new 'testapp' EFI app specifically intended for testing the EFI loader. Attach it to the USB device, since most testing is currently done with mmc. Initially this image will be used to test the EFI bootmeth. Fix a stale comment in prep_mmc

[PATCH v10 08/10] sandbox: virtio: Disable the sandbox virtio blk device

2024-11-07 Thread Simon Glass
This is not implemented so cannot actually be used to read blocks. Disable it until it is implemented, to avoid causing a hang with EFI, which probes every available BLK device. Signed-off-by: Simon Glass Reviewed-by: Tom Rini Issue: https://source.denx.de/u-boot/u-boot/-/issues/37 --- (no chan

[PATCH v10 07/10] efi_loader: Drop sandbox PXE architecture

2024-11-07 Thread Simon Glass
Rather than returning 0, just return an error, since sandbox is not used with PXE at present. Signed-off-by: Simon Glass --- (no changes since v9) Changes in v9: - Add new patch to drop sandbox PXE architecture lib/efi_loader/efi_helper.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/l

[PATCH v10 06/10] sandbox: Report host default-filename in native mode

2024-11-07 Thread Simon Glass
When the --native flag is given, pretend to be running the host architecture rather than sandbox. Allow the same control for PXE too. Signed-off-by: Simon Glass --- (no changes since v9) Changes in v9: - Separate into separate patches Changes in v8: - Add new patch to report host default-file

[PATCH v10 04/10] efi_loader: Move get_efi_pxe_arch() to efi_helper

2024-11-07 Thread Simon Glass
Move this function from the EFI bootmeth to the common efi_helper file. No functional change is intended. Signed-off-by: Simon Glass --- (no changes since v9) Changes in v9: - Split out into a separate change boot/bootmeth_efi.c | 25 ++--- include/efi.h

[PATCH v10 05/10] efi_loader: Allow reporting the host defaults

2024-11-07 Thread Simon Glass
Add an 'efidebug filename' command to report the default filename and PXE architecture. Signed-off-by: Simon Glass --- (no changes since v9) Changes in v9: - Split out into its own patch Changes in v8: - Add new patch to report host default-filename in native mode cmd/efidebug.c | 25 +++

[PATCH v10 02/10] sandbox: Add a -N flag to control on-host behaviour

2024-11-07 Thread Simon Glass
Sandbox is its own architecture, but sometimes we want to mimic the host architecture, e.g. when running an EFI app not built by U-Boot. Add a -N/--native flag which tells sandbox to reflect the architecture of the host. Signed-off-by: Simon Glass --- (no changes since v8) Changes in v8: - Add

[PATCH v10 03/10] efi: Move default filename to a function

2024-11-07 Thread Simon Glass
Use a function to obtain the device EFI filename, so that we can control how sandbox behaves. Signed-off-by: Simon Glass --- (no changes since v9) Changes in v9: - Move the function into efi_helper.c Changes in v8: - Add new patch to move default filename to a function boot/bootmeth_efi.c

[PATCH v10 01/10] efi_loader: Add a test app

2024-11-07 Thread Simon Glass
Add a simple app to use for testing. This is intended to do whatever it needs to for testing purposes. For now it just prints a message and exits boot services. There was a considerable amount of discussion about whether it is OK to call exit-boot-services and then return to U-Boot. This is not no

[PATCH v10 00/10] efi: Add a test for EFI bootmeth

2024-11-07 Thread Simon Glass
The test coverage for the EFI bootmeth is incomplete since it does not actually boot the application. This series creates a simple test for this purpose. It includes a few patches to make this work: - Revert of an unreviewed patch to change the sandbox efi filename - Hang in sandbox virtio due to

Re: [PATCH 2/2] boot: extlinux: Fix unaligned buffer for reading data from file system

2024-11-07 Thread Simon Glass
Hi Nam, On Wed, 6 Nov 2024 at 09:34, Nam Cao wrote: > > extlinux_read_bootflow() allocates a buffer to read from file system > without any alignment. > > But for some block devices which transfer data via DMA, ARCH_DMA_MINALIGN > alignment is required. For example, due to misaligned buffer, the b

Re: [PATCH] test: bootm: Ensure tests can be run twice

2024-11-07 Thread Simon Glass
Hi Andrew, On Wed, 6 Nov 2024 at 06:50, Andrew Goodbody wrote: > > Some of the bootm tests rely on state that is assumed to be > correct but is changed by the tests. This means that running > 'ut bootm' twice will result in failures on the second run > as the state left by the first run is not wh

Re: [PATCH v2 2/3] rockchip: rk3588: Implement checkboard() to print SoC variant

2024-11-07 Thread Jonas Karlman
Hi Quentin, On 2024-11-07 13:01, Quentin Schulz wrote: > Hi Jonas, > > On 11/2/24 9:37 PM, Jonas Karlman wrote: >> Implement checkboard() to print current SoC model used by a board, >> e.g. one of: >> >>SoC: RK3582 v1 >>SoC: RK3588 v0 >>SoC: RK3588 v1 >>SoC: RK3588S v0 >>

Re: [PATCH v2 2/3] rockchip: rk3588: Implement checkboard() to print SoC variant

2024-11-07 Thread Quentin Schulz
Hi Jonas, On 11/7/24 3:29 PM, Jonas Karlman wrote: Hi Quentin, On 2024-11-07 13:01, Quentin Schulz wrote: Hi Jonas, On 11/2/24 9:37 PM, Jonas Karlman wrote: Implement checkboard() to print current SoC model used by a board, e.g. one of: SoC: RK3582 v1 SoC: RK3588 v0 SoC: R

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

2024-11-07 Thread Tim Harvey
On Thu, Nov 7, 2024 at 12:24 AM Yannic Moog wrote: > > Add tee node in SoC u-boot device trees. Use a kconfig entry to specify > load and entry addresses for the op-tee image in the respective > defconfig. > > Signed-off-by: Yannic Moog > --- > arch/arm/dts/imx8mm-u-boot.dtsi | 17 ++

Re: [PATCH v2 2/3] rockchip: rk3588: Implement checkboard() to print SoC variant

2024-11-07 Thread Quentin Schulz
Hi Jonas, On 11/2/24 9:37 PM, Jonas Karlman wrote: Implement checkboard() to print current SoC model used by a board, e.g. one of: SoC: RK3582 v1 SoC: RK3588 v0 SoC: RK3588 v1 SoC: RK3588S v0 SoC: RK3588S v1 SoC: RK3588S2 v1 when U-Boot proper is running. U-Bo

[PATCH] lwip: fix code style issues

2024-11-07 Thread Jerome Forissier
Fix various code style issues in the lwIP code. Reported-by: Tom Rini Signed-off-by: Jerome Forissier --- cmd/net-common.c | 9 +++- cmd/net-lwip.c| 37 +++- include/net-common.h | 22 ++-- include/net-legacy.h | 44

Re: [PATCH 3/7] net: wget: Add a global struct wget_http_info

2024-11-07 Thread Jerome Forissier
On 11/6/24 13:03, Adriano Cordova wrote: > Declare and define a global struct wget_http_info. The default > values are so that wget works as it used to, but they can be > changed by an application before issuing an http request using > wget, and restored afterwards. This struct is common to legacy

[PATCH 0/2] indirect write time out related changes

2024-11-07 Thread Prasad Kummari
Need to rebase top of this commit: Link: https://lore.kernel.org/r/b3b678603f76b9f6490c0a46ed0cec06652d1aa5.1701853668.git.tejas.arvind.bhum...@amd.com Prasad Kummari (1): spi: cadence_qspi: Fix Versal NET ospi indirect write timed out issue Tejas Bhumkar (1): spi: cadence_qspi: Fix versal os

[PATCH 2/2] spi: cadence_qspi: Fix Versal NET ospi indirect write timed out issue

2024-11-07 Thread Prasad Kummari
The readl_poll_timeout() function is currently polling for the indirect operation completion bit with a 10ms timeout, which triggers an "Indirect write timed out" error. The indirect done status is also using a 10ms timeout, leading to "Indirect write clear completion" errors. To resolve these time

Re: [PATCH 0/8] arm_ffa: Add FFA_MEM_SHARE and FFA_MEM_RECLAIM support

2024-11-07 Thread Abdellatif El Khlifi
Hi Simon, Tom, > Add to the FF-A bus FFA_MEM_SHARE and FFA_MEM_RECLAIM ABIs. > > These FF-A ABIs are for memory management. They are used for sharing > memory between U-Boot and secure world. > > FFA_MEM_SHARE starts a transaction to grant access to a memory region > to one or more Borrowers (ak

Re: [PATCH 3/4] mtd: rawnand: brcmnand: Add BCMBCA read data bus interface

2024-11-07 Thread Linus Walleij
On Wed, Nov 6, 2024 at 10:04 PM david regan wrote: > The BCMBCA broadband SoC integrates the NAND controller differently than > STB, iProc and other SoCs. It has different endianness for NAND cache > data. > > Add a SoC read data bus shim for BCMBCA to meet the specific SoC need > and performanc

Re: [PATCH v9 1/7] usb: onboard-hub: Add reset-gpio support

2024-11-07 Thread Marek Vasut
On 11/7/24 7:05 AM, Abbarapu, Venkatesh wrote: Hi, -Original Message- From: Marek Vasut Sent: Wednesday, November 6, 2024 6:04 AM To: Abbarapu, Venkatesh ; u-boot@lists.denx.de Cc: Simek, Michal ; fabrice.gasn...@foss.st.com; git (AMD-Xilinx) Subject: Re: [PATCH v9 1/7] usb: onboard-h

[PATCH 1/2] spi: cadence_qspi: Fix versal ospi indirect write timed out issue

2024-11-07 Thread Prasad Kummari
From: Tejas Bhumkar To reduce the CPU load in waiting for the OSPI internal SRAM to clear in indirect mode, it's better to use the CQSPI_REG_IRQSTATUS register to check for indirect operation to complete. Enabled interrupt for Indirect Complete and Transfer Watermark Breach interrupt status regis

Re: [PATCH 1/1] cmd: upl: initialize unit test state

2024-11-07 Thread Simon Glass
Hi Heinrich, On Wed, 6 Nov 2024 at 09:00, Heinrich Schuchardt wrote: > > On 05.11.24 16:12, Simon Glass wrote: > > Hi Heinrich, > > > > On Sat, 2 Nov 2024 at 08:30, Heinrich Schuchardt > > wrote: > >> > >> do_upl_write() calls upl_get_test_data() which may increment the fail > >> count in the un

Re: [PATCH 1/2] fs: Use ARCH_DMA_MINALIGN as default alignment for fs_read_alloc()

2024-11-07 Thread Simon Glass
Hi Nam, On Wed, 6 Nov 2024 at 09:34, Nam Cao wrote: > > The comment above fs_read_alloc() explains: > > @align: Alignment to use for memory allocation (0 for default) > > However, in the actual implementation, there is no alignment when @align is > zero. > > This current default is probably f

Re: [PATCH 1/1] test: fix regular expression in test_spi_negative()

2024-11-07 Thread Simon Glass
On Thu, 7 Nov 2024 at 03:39, Heinrich Schuchardt wrote: > > Escape sequence \s is only allowable in raw strings. > > Signed-off-by: Heinrich Schuchardt > --- > test/py/tests/test_spi.py | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Reviewed-by: Simon Glass Thanks, just did the same

Re: [PATCH v9 00/11] efi: Add a test for EFI bootmeth

2024-11-07 Thread Simon Glass
Hi Heinrich, On Thu, 7 Nov 2024 at 04:29, Heinrich Schuchardt wrote: > > On 11/1/24 16:32, Simon Glass wrote: > > Hi all, > > > > On Fri, 1 Nov 2024 at 16:26, Simon Glass wrote: > >> > >> The test coverage for the EFI bootmeth is incomplete since it does not > >> actually boot the application. >

Re: [PATCH 1/6] Revert "spi: zynq_qspi: Add parallel memories support in QSPI driver"

2024-11-07 Thread Marek Vasut
On 11/7/24 4:49 PM, Jon Humphreys wrote: Marek Vasut writes: On 11/6/24 10:58 PM, Jon Humphreys wrote: Marek Vasut writes: On 11/6/24 8:18 PM, Jon Humphreys wrote: Marek Vasut writes: On 10/23/24 10:17 AM, Michal Simek wrote: On 10/22/24 23:06, Marek Vasut wrote: This reverts commi

Re: [PATCH 1/6] Revert "spi: zynq_qspi: Add parallel memories support in QSPI driver"

2024-11-07 Thread Jon Humphreys
Marek Vasut writes: > On 11/6/24 10:58 PM, Jon Humphreys wrote: >> Marek Vasut writes: >> >>> On 11/6/24 8:18 PM, Jon Humphreys wrote: Marek Vasut writes: > On 10/23/24 10:17 AM, Michal Simek wrote: >> >> >> On 10/22/24 23:06, Marek Vasut wrote: >>> This reverts c

Re: [PATCH 2/2] boot: extlinux: Fix unaligned buffer for reading data from file system

2024-11-07 Thread Nam Cao
Hi Simon, On Thu, Nov 07, 2024 at 06:44:30AM -0700, Simon Glass wrote: > On Wed, 6 Nov 2024 at 09:34, Nam Cao wrote: > > > > extlinux_read_bootflow() allocates a buffer to read from file system > > without any alignment. > > > > But for some block devices which transfer data via DMA, ARCH_DMA_MIN

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

2024-11-07 Thread Sumit Garg
On Thu, 7 Nov 2024 at 00:53, david regan wrote: > > Make use of OF_UPSTREAM which uses Linux dts. > > Signed-off-by: david regan > --- > arch/arm/dts/bcm47622.dtsi| 126 > arch/arm/dts/bcm4912.dtsi | 128 > arch/arm/dts/bcm63146.dtsi|

Re: [PATCH v4 5/8] sunxi: Move to bootstd

2024-11-07 Thread Mark Kettenis
> From: Simon Glass > Date: Tue, 5 Nov 2024 08:14:53 -0700 Heinrich, did you see this discussion concerning the EFI boot manager? Simon, > H Mark, Peter, > > On Mon, 4 Nov 2024 at 08:29, Mark Kettenis wrote: > > > > > Date: Mon, 4 Nov 2024 13:34:29 + > > > From: Andre Przywara > > > > >

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

2024-11-07 Thread Linus Walleij
On Wed, Nov 6, 2024 at 10:04 PM david regan wrote: > 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.

Re: [PATCH next v2 4/4] rockchip: rk3399: move sysreset-gpio logic to TPL

2024-11-07 Thread Paul Kocialkowski
Hi, Le Wed 06 Nov 24, 12:29, Quentin Schulz a écrit : > From: Quentin Schulz > > If TPL_GPIO and TPL_PINCTRL_ROCKCHIP are enabled and a sysreset-gpio is > provided in the TPL Device Tree, this will trigger a system reset > similar to what's currently been done in SPL whenever the RK3399 "warm" >

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

2024-11-07 Thread Linus Walleij
On Wed, Nov 6, 2024 at 10:04 PM david regan wrote: > Make use of OF_UPSTREAM which uses Linux dts. > > Signed-off-by: david regan Reviewed-by: Linus Walleij Yours, Linus Walleij

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

2024-11-07 Thread Linus Walleij
On Wed, Nov 6, 2024 at 10:03 PM david regan wrote: > 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 I believe you! Reviewed-by: Linus Walleij Yours, Linus Walleij

Re: [PATCH 1/2] fs: Use ARCH_DMA_MINALIGN as default alignment for fs_read_alloc()

2024-11-07 Thread Nam Cao
Hi Simon, On Thu, Nov 07, 2024 at 06:44:32AM -0700, Simon Glass wrote: > Hi Nam, > > On Wed, 6 Nov 2024 at 09:34, Nam Cao wrote: > > > > The comment above fs_read_alloc() explains: > > > > @align: Alignment to use for memory allocation (0 for default) > > > > However, in the actual implement

[PATCH] arm: imx: Remove arch_env_get_location to fix env lockdown

2024-11-07 Thread Abdelrahman Ibrahem
This patch resolves an issue encountered with Layerscape SoCs, where attempts to restrict write access to environment variables to enforce secureboot are ineffective when using the CONFIG_ENV_IS_NOWHERE configuration in combination with CONFIG_ENV_IS_IN_xxx. Signed-off-by: Abdelrahman Ibrahem ---

Re: [PATCH] cadence_qspi: Refactor the flash reset functionality

2024-11-07 Thread Michal Simek
On 11/7/24 05:57, Abbarapu, Venkatesh wrote: Hi Michal, -Original Message- From: Simek, Michal Sent: Wednesday, November 6, 2024 5:17 PM To: Abbarapu, Venkatesh ; u-boot@lists.denx.de Cc: ja...@amarulasolutions.com; vigne...@ti.com; u-kum...@ti.com; tr...@konsulko.com; sean...@gmail

Re: [PATCH next v2 4/4] rockchip: rk3399: move sysreset-gpio logic to TPL

2024-11-07 Thread Paul Kocialkowski
Hi, Le Thu 07 Nov 24, 12:04, Quentin Schulz a écrit : > On 11/7/24 8:14 AM, Kever Yang wrote: > > Yes, this movement is reasonable and needed for this workaround, > > although still not > > > > understand why puma board need this. > > Me neither, this predates me joining the company, c.f.: > htt

Re: [PATCH v9 1/7] usb: onboard-hub: Add reset-gpio support

2024-11-07 Thread Fabio Estevam
On Thu, Nov 7, 2024 at 8:35 AM Marek Vasut wrote: > I can very much predict there will show up a hub which expects the reset > to be active HIGH and this will have to be removed, so better remove it now. Agreed. The onboard_usb_dev.c driver in Linux does not assume that the USB hub reset is alw

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

2024-11-07 Thread Marek Vasut
On 11/7/24 10:45 AM, Ilias Apalodimas wrote: On Sun, 3 Nov 2024 at 22: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:

Re: [PATCH 7/7] net/lwip: wget: integrate struct wget_info into wget code

2024-11-07 Thread Jerome Forissier
On 11/6/24 13:04, Adriano Cordova wrote: > Each wget request now fills the struct wget_info. Also, the > efi bootdevice is now set conditionally to the set_bootdevice > variable in wget_info and a buffer size check is performed if > check_buffer_size is set. > > Signed-off-by: Adriano Cordova > -

Re: [PATCH 6/7] net/lwip: wget: put server_name and port into wget_ctx

2024-11-07 Thread Jerome Forissier
On 11/6/24 13:04, Adriano Cordova wrote: > Currently server_name and port are local variables in wget_loop. > This commit puts them inside ctx, so that they are accessible > from the http callbacks. > > Signed-off-by: Adriano Cordova > --- > net/lwip/wget.c | 8 > 1 file changed, 4 inse

Re: [PATCH 1/7] net: Kconfig: add CONFIG_WGET symbol

2024-11-07 Thread Jerome Forissier
On 11/6/24 13:02, Adriano Cordova wrote: > Let net/wget.c and net/lwip/wget.c depend on CONFIG_WGET, and > cmd/wget.c depend on CONFIG_CMD_WGET. This way, the wget code > can be used regardless of whether the wget command is available. > > Signed-off-by: Adriano Cordova > --- > cmd/Kconfig

  1   2   >