Re: [Upstream] [PATCH 3/3] board: phytec: imx93: Add phyCORE-i.MX93 support all SOM variants

2024-11-04 Thread Wadim Egorov
Hi Christoph, Am 04.11.24 um 12:25 schrieb Christoph Stoidner: The phyCORE-i.MX 93 is available in various variants (e.g. different ram sizes, eMMC HS400 yes/no). Add a new SOM-scoped defconfig that makes use of the hardware introspection of the phycore-imx93 board-code, to detect the SOM module

[PATCH] cadence_qspi: fix odd byte read issue in STIG mode

2024-11-04 Thread Prasad Kummari
In DDR mode, even bytes are read using DMA, while the remaining odd bytes are read using STIG mode. However, the data is not correctly transferred into the flash read data lower register because the supplementary byte of the STIG opcode is not being written to the opcode extension register, resulti

Re: [PATCH] arm: dts: e850-96: Remove TRNG node from appended dts

2024-11-04 Thread Minkyu Kang
Hello, 2024년 10월 31일 (목) 05:46, Sam Protsenko 님이 작성: > Commit 136b7b6d2e98 ("Subtree merge tag 'v6.11-dts' of dts repo [1] into > dts/upstream") updates the upstream dts for E850-96 board, bringing > upstream TRNG node. There is no need to keep TRNG node in appended dts > anymore, so remove it.

Re: [PATCH] pinctrl: exynos: fix type of count integer

2024-11-04 Thread Minkyu Kang
Hello, 2024년 9월 11일 (수) 03:46, Henrik Grimler 님이 작성: > unsigned int count is used to capture return of dev_read_string_count. > When an error occurs dev_read_string_count returns -ve, which can > hence not be handled correctly. Use int instead of unsigned int so > that errors are detected. > > F

[PATCH v3 06/13] pcie: starfive: Make the driver compatible with upstream DT

2024-11-04 Thread Hal Feng
There are difference between upstream DT and the old DT in terms of reg base, reset gpio and syscon. Make the driver compatible with upstream DT. Signed-off-by: Hal Feng --- drivers/pci/pcie_starfive_jh7110.c | 59 +++--- 1 file changed, 30 insertions(+), 29 deletions(-)

[PATCH 1/1] efi_loader: simplify efi_tcg2_hash_log_extend_event()

2024-11-04 Thread Heinrich Schuchardt
The value of variable nt is never used. Just use NULL when calling efi_check_pe(). The API function is not expected to write to the console. Such output might have unwanted side effects on the screen layout of an EFI application. Leave error handling to the caller. Signed-off-by: Heinrich Schuch

[PATCH v3 05/13] clk: starfive: jh7110: Sync clock definitions with upstream dt-bindings

2024-11-04 Thread Hal Feng
JH7110 switch to use upstream dt-bindings, so update the clock definitions in drivers accordingly. Signed-off-by: Hal Feng --- drivers/clk/starfive/clk-jh7110-pll.c | 6 ++-- drivers/clk/starfive/clk-jh7110.c | 44 +-- 2 files changed, 25 insertions(+), 25 deletions(

[PATCH v3 13/13] board: starfive: spl: Support multiple DTBs for JH7110 based boards

2024-11-04 Thread Hal Feng
Get product ID and PCB version from EEPROM, use them to select the correct DTB. Signed-off-by: Hal Feng --- board/starfive/visionfive2/spl.c | 25 +++-- 1 file changed, 23 insertions(+), 2 deletions(-) diff --git a/board/starfive/visionfive2/spl.c b/board/starfive/visionfive

[PATCH v3 09/13] board: starfive: spl: Drop the unneeded DT modification code

2024-11-04 Thread Hal Feng
As OF_UPSTREAM is implemented, these code are redundant. Signed-off-by: Hal Feng --- board/starfive/visionfive2/spl.c | 356 --- 1 file changed, 356 deletions(-) diff --git a/board/starfive/visionfive2/spl.c b/board/starfive/visionfive2/spl.c index 3fd535e7cf..38132e

[PATCH v3 00/13] Support OF_UPSTREAM for StarFive JH7110

2024-11-04 Thread Hal Feng
This patchset add OF_UPSTREAM support for StarFive JH7110 based boards. All the JH7110 based boards can use the DT from upstreaming linux kernel. The v1.3b board device tree is set as the default device tree. Support CONFIG_MULTI_DTB_FIT and the SPL selects the correct U-Boot dtb using the EEPROM i

[PATCH v3 11/13] riscv: dts: jh7110: Support multiple DTBs in a Fit image

2024-11-04 Thread Hal Feng
Support multiple DTBs for JH7110 based boards, so they can select the correct DT at runtime. Signed-off-by: Hal Feng --- arch/riscv/dts/jh7110-common-u-boot.dtsi | 60 ++-- 1 file changed, 57 insertions(+), 3 deletions(-) diff --git a/arch/riscv/dts/jh7110-common-u-boot.dtsi

[PATCH v3 12/13] board: starfive: spl: Fix the wrong use of CONFIG_IS_ENABLED()

2024-11-04 Thread Hal Feng
The prefix "SPL_" is not needed when using CONFIG_IS_ENABLED(). Fixes: 5ecf9b0b8a75 ("board: starfive: add StarFive VisionFive v2 board support") Signed-off-by: Hal Feng --- board/starfive/visionfive2/spl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/board/starfive/visio

[PATCH v3 10/13] configs: visionfive2: Enable MULTI_DTB_FIT for JH7110 based board DT

2024-11-04 Thread Hal Feng
So JH7110 based boards can select their own DT at runtime. Signed-off-by: Hal Feng --- configs/starfive_visionfive2_defconfig | 2 ++ 1 file changed, 2 insertions(+) diff --git a/configs/starfive_visionfive2_defconfig b/configs/starfive_visionfive2_defconfig index 5257e64a79..47b7122470 100644

[PATCH v3 03/13] dt-bindings: Remove StarFive JH7110 clock and reset definitions

2024-11-04 Thread Hal Feng
As JH7110 switch to use OF_UPSTREAM dt-bindings, remove the redundant clock and reset definitions. Signed-off-by: Hal Feng --- .../dt-bindings/clock/starfive,jh7110-crg.h | 258 -- .../dt-bindings/reset/starfive,jh7110-crg.h | 183 - 2 files changed, 441 deletions

[PATCH v3 08/13] riscv: dts: jh7110: Add u-boot device tree for JH7110 based boards

2024-11-04 Thread Hal Feng
To support the other JH7110 based boards, add u-boot device tree for them. Cc: Heinrich Schuchardt Cc: H Bell Signed-off-by: Hal Feng --- arch/riscv/dts/jh7110-milkv-mars-u-boot.dtsi | 10 ++ arch/riscv/dts/jh7110-pine64-star64-u-boot.dtsi| 6 ++ .../dts/jh7110-s

[PATCH v3 07/13] riscv: dts: jh7110: Move common code to the new jh7110-common-u-boot.dtsi

2024-11-04 Thread Hal Feng
To support JH7110 based boards besides v1.3B, add a common dtsi and add common code to it. Signed-off-by: Hal Feng --- arch/riscv/dts/jh7110-common-u-boot.dtsi | 150 ++ ...10-starfive-visionfive-2-v1.3b-u-boot.dtsi | 146 + 2 files changed, 151 insertions(+)

[PATCH v3 04/13] riscv: dts: jh7110: Make u-boot device trees adapting to upstream DT

2024-11-04 Thread Hal Feng
Add u-boot features to the U-Boot device tree. Signed-off-by: Hal Feng --- ...10-starfive-visionfive-2-v1.3b-u-boot.dtsi | 49 +-- arch/riscv/dts/jh7110-u-boot.dtsi | 36 +- 2 files changed, 80 insertions(+), 5 deletions(-) diff --git a/arch/riscv/dts/jh7

[PATCH v3 02/13] riscv: dts: jh7110: Drop redundant devicetree files

2024-11-04 Thread Hal Feng
JH7110 boards switch to using upstream DT, so drop redundant DT files from arch/riscv/dts/. Signed-off-by: Hal Feng --- arch/riscv/dts/Makefile | 1 - .../dts/jh7110-starfive-visionfive-2.dts | 11 - .../dts/jh7110-starfive-visionfive-2.dtsi | 380 - arc

[PATCH v3 01/13] dts: starfive: Switch to using upstream DT

2024-11-04 Thread Hal Feng
Enable OF_UPSTREAM to use upstream DT and add starfive/ prefix to the DEFAULT_DEVICE_TREE. Rename jh7110-starfive-visionfive-2-u-boot.dtsi to jh7110-starfive-visionfive-2-v1.3b-u-boot.dtsi and set the v1.3b device tree as the default device tree. Signed-off-by: Hal Feng --- arch/riscv/cpu/jh7110

[PATCH 1/1] doc: qemu-riscv: describe CONFIG_DEBUG_SBI_CONSOLE

2024-11-04 Thread Heinrich Schuchardt
In main U-Boot the SBI DBCN extension can be used to supply a debug console. Signed-off-by: Heinrich Schuchardt --- doc/board/emulation/qemu-riscv.rst | 5 + 1 file changed, 5 insertions(+) diff --git a/doc/board/emulation/qemu-riscv.rst b/doc/board/emulation/qemu-riscv.rst index 8a5eb1eda

[PATCH 1/1] doc: conf.py: correct title and author of generated pdf

2024-11-04 Thread Heinrich Schuchardt
Adjust the PDF output options for rst2pdf: * Set start document to 'index'. * Set title to 'Das U-Boot'. * Set author to 'The U-Boot development community'. Correct documentation link. Remove obsolete FIXME. Link: https://rst2pdf.org/static/manual.html Signed-off-by: Heinrich Schuchardt --- d

Re: [PATCH v5 03/11] led: implement LED boot API

2024-11-04 Thread Christian Marangi
On Sun, Nov 03, 2024 at 07:46:19AM -0700, Simon Glass wrote: > Hi Christian, > > On Sat, 2 Nov 2024 at 13:52, Christian Marangi wrote: > > > > On Sat, Nov 02, 2024 at 01:50:13PM -0600, Simon Glass wrote: > > > Hi Christian, > > > > > > On Sat, 2 Nov 2024 at 13:36, Christian Marangi > > > wrote:

Re: [PATCH] bios_emulator: fix first argument of pci_{read,write}_config_* function calls

2024-11-04 Thread Tom Rini
On Wed, 30 Oct 2024 20:38:20 +0100, Yuri Zaporozhets wrote: > When compiling for riscv64, a bunch of warning is produced for the file > drivers/bios_emulator/besys.c. This patch fixes a portion of those warnings, > caused by incorrect first argument to pci_{read,write}_config_* functions. > > A

Re: [PATCH] bios_emulator: define the comment symbol for RISC-V assembler too

2024-11-04 Thread Tom Rini
On Wed, 30 Oct 2024 14:00:51 +0100, Yuri Zaporozhets wrote: > The bios_emulator driver cannot be compiled for RISC-V because the x86emu.h > header file doesn't define the comment symbol ("#") for the assembler. > With this patch, use the same symbol as for e.g. x86. > > Applied to u-boot/master

Re: [PATCH] drivers/video/vesa: use MTRRs only on x86

2024-11-04 Thread Tom Rini
On Wed, 30 Oct 2024 14:17:33 +0100, Yuri Zaporozhets wrote: > MTRR functionality is available only on x86, so this driver cannot be compiled > on other architectures. Fix this with preprocessor directives. > > Applied to u-boot/master, thanks! -- Tom

Re: [PATCH v1 0/8] U-boot: arm: Refine the booting on Total Compute

2024-11-04 Thread Tom Rini
On Fri, 25 Oct 2024 18:18:13 +0100, Leo Yan wrote: > This patch series is to refine the booting on Arm Total Compuate > platform. > > It changes to use the info passed in DTB for initialization DRAM info, > and dynamically initializes the booting envoironment variables. > > Another big change is

Re: [PATCH v2 1/3] menu: fix the logic checking whether ESC key is pressed

2024-11-04 Thread Tom Rini
On Tue, 29 Oct 2024 17:47:10 +0800, Weijie Gao wrote: > It's observed that the bootmenu on a serial console sometimes > incorrectly quitted with superfluous characters filled to command > line input: > > > *** U-Boot Boot Menu *** > > > > 1. Startup system (Default) > > 2. Upgrade firm

Re: [PATCH 0/2] Fixes to allow 'ut bootm' to pass when run interactively

2024-11-04 Thread Tom Rini
On Fri, 01 Nov 2024 13:02:52 +, Andrew Goodbody wrote: > Starting a sandbox session and running 'ut bootm' on the command > line instead of using pytest with --verbose will result in some > test failures. This series makes the tests more deterministic so > that they will better control their e

Re: [PATCH] usb: Fix test failure with multiple partitions

2024-11-04 Thread Tom Rini
On Tue, 15 Oct 2024 13:19:16 +0100, Andrew Goodbody wrote: > When test_usb_load finds multiple partitions of the same type then > it will cause a test failure. The call to write the test file will > write a different test file to each partition but only return the > name and size of the last one w

Re: [PATCH] usb: Fix test dependency on ext4write command

2024-11-04 Thread Tom Rini
On Tue, 15 Oct 2024 15:17:37 +0100, Andrew Goodbody wrote: > The tests test_usb_ext4load_ext4write, test_usb_ext2load and > test_usb_load depend on the command ext4write being present not just > the feature of being able to write to an ext4 file system. So update > their dependencies to include th

Re: [PATCH v8 10/11] test: Support testing with two board-builds

2024-11-04 Thread Tom Rini
On Mon, Nov 04, 2024 at 03:45:37PM -0700, Simon Glass wrote: > The Beagleplay board uses two entirely separate builds to produce an > image, rather than using an SPL build for this purpose. You forgot to reword this :) -- Tom signature.asc Description: PGP signature

Re: [PATCH v12 00/13] net: tcp: improve tcp support in legacy stack

2024-11-04 Thread Tom Rini
On Mon, Oct 28, 2024 at 05:31:30PM +0300, Mikhail Kshevetskiy wrote: > Legacy TCP stack is bad. Here are some of the known issues: > * tcp packet from other connection can break a current one > * tcp send sequence always starts from zero > * bad tcp options processing > * strange assumptions o

Re: [PATCH v3 04/25] pinephone: Drop status-LED code and use LED

2024-11-04 Thread Tom Rini
On Sat, Nov 02, 2024 at 06:32:56PM -0600, Simon Glass wrote: > Drop the STATUS_LED options and enable LED and LED_BOOT instead, since > this is the new way to access the LED functionality. > > Signed-off-by: Simon Glass > --- > > Changes in v3: > - Combine removal and enable patches > > Change

[PATCH v8 05/11] test: Introduce lab mode

2024-11-04 Thread Simon Glass
There is quite a bit of code in pytest to try to start up U-Boot on a board, with timeouts, expects, etc. This is tedious to maintain and is peripheral to the test system's purpose. It seems better to put this logic in the lab itself, where is can provide such support. With Labgrid we can use the

Re: [PATCH v5 6/6] common: android_ab: fix slot suffix for abc block

2024-11-04 Thread Sam Protsenko
On Thu, Oct 17, 2024 at 9:12 AM Dmitry Rokosov wrote: > > To align with the official Android BCB (Bootloader Control Block) > specifications, it's important to note that the slot_suffix should start > with an underscore symbol. > > For a comprehensive understanding of the expected slot_suffix form

[PATCH v8 00/11] labgrid: Provide an integration with Labgrid

2024-11-04 Thread Simon Glass
Labgrid provides access to a hardware lab in an automated way. It is possible to boot U-Boot on boards in the lab without physically touching them. It relies on relays, USB UARTs and SD muxes, among other things. By way of background, about 4 years ago I wrong a thing called Labman[1] which allowe

[PATCH v8 03/11] test: Allow connecting to a running board

2024-11-04 Thread Simon Glass
Sometimes we know that the board is already running the right software, so provide an option to allow running of tests directly, without first resetting the board. This saves time when re-running a test where only the Python code is changing. Signed-off-by: Simon Glass --- (no changes since v7)

[PATCH v8 11/11] CI: Allow running tests on sjg lab

2024-11-04 Thread Simon Glass
Add a way to run tests on a real hardware lab. This is in the very early experimental stages. There are only 23 boards and 3 of those are broken! (bob, ff3399, samus). A fourth fails due to problems with the TPM tests. To try this, assuming you have gitlab access, set SJG_LAB=1, e.g.: git push

[PATCH v8 10/11] test: Support testing with two board-builds

2024-11-04 Thread Simon Glass
The Beagleplay board uses two entirely separate builds to produce an image, rather than using an SPL build for this purpose. Handle this in test.py by adding more parameters. Signed-off-by: Simon Glass --- (no changes since v5) Changes in v5: - Add a patch to support testing with two board-bui

[PATCH v8 09/11] test: Add a section for closing the connection

2024-11-04 Thread Simon Glass
This can take a while and involve multiple steps (e.g. turning the board back off). Add a section for it and show the output. Signed-off-by: Simon Glass Reviewed-by: Tom Rini --- (no changes since v1) test/py/u_boot_console_base.py | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) di

[PATCH v8 08/11] test: Try to shut down the lab console gracefully

2024-11-04 Thread Simon Glass
Send the Labgrid quit characters to ask it to exit gracefully. This typically allows it to power off the board being used. Only do this when labgrid is being used (detected with an env var). If that doesn't work, try the less graceful approach. Signed-off-by: Simon Glass --- Changes in v8: - Us

[PATCH v8 07/11] test: Avoid double echo when starting up

2024-11-04 Thread Simon Glass
There is a very annoying bug at present where the terminal echos part of the first command sent to the board. This happens because the terminal is still set to echo for a period until Labgrid starts up and can change this. Fix this by disabling echo (and other terminal features) as soon as the spa

[PATCH v8 06/11] test: Improve handling of sending commands

2024-11-04 Thread Simon Glass
We expect commands to be echoed and this should happen quite quickly, since U-Boot is sitting at the prompt waiting for a command. Reduce the timeout for this situation. Try to produce a more useful error message when something goes wrong. Also handle the case where the connection has gone away si

[PATCH v8 04/11] test: Introduce the concept of a role

2024-11-04 Thread Simon Glass
In Labgrid there is the concept of a 'role', which is similar to the U-Boot board ID in U-Boot's pytest subsystem. The role indicates both the target and information about the U-Boot build to use. It can also provide any amount of other configuration. The information is obtained using the 'labgrid

[PATCH v8 01/11] test: Allow signaling that U-Boot is ready

2024-11-04 Thread Simon Glass
When Labgrid is used, it can get U-Boot ready for running tests. It prints a message when it has done so. Add logic to detect this message and accept it. Note that this does not change pytest, which still (also) looks for the U-Boot banner. This change merely makes it possible for pytest to belie

[PATCH v8 02/11] test: Release board after tests complete

2024-11-04 Thread Simon Glass
When a board is finished with, the lab may want to power it off, or perform some other function. Add a new script which is called when tests are complete. Reviewed-by: Tom Rini Signed-off-by: Simon Glass --- (no changes since v1) test/py/u_boot_console_exec_attach.py | 10 ++ 1 file c

Re: [PATCH] efi_loader: Change efi_dp_from_mem() to use size

2024-11-04 Thread Heinrich Schuchardt
On 11/4/24 02:49, Moritz Fischer wrote: All call sites are using size rather than end addresses, so instead - as previously done - calculating an end address everywhere, just modify the function to use size and internally calculate the end address Cc: Heinrich Schuchardt Cc: Ilias Apalodimas C

Re: [PATCH v3 02/19] bootstd: Move bootflow-clearing to bootstd

2024-11-04 Thread Heinrich Schuchardt
On 11/4/24 18:50, Simon Glass wrote: This relates to more than just the bootdev, since there is a global list of bootflows. Move the function to the bootstd file and rename it. Signed-off-by: Simon Glass Acked-by: Heinrich Schuchardt --- (no changes since v1) boot/bootdev-uclass.c | 19

Re: [PATCH v3 01/19] bootstd: Move bootflow-adding to bootstd

2024-11-04 Thread Heinrich Schuchardt
On 11/4/24 18:50, Simon Glass wrote: This relates to more than just the bootdev, since there is a global list of bootflows. Move the function to the bootstd file and rename it. Signed-off-by: Simon Glass --- (no changes since v1) boot/bootdev-uclass.c | 25 - boot/b

Re: [PATCH v3 11/19] bootmeth_efi: Check the filename-allocation in the network path

2024-11-04 Thread Heinrich Schuchardt
On 11/4/24 18:51, Simon Glass wrote: If the filename cannot be set we should give up. Add the missing error check. Signed-off-by: Simon Glass --- (no changes since v2) Changes in v2: - Change the tag to bootmeth_efi boot/bootmeth_efi.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a

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

2024-11-04 Thread Simon Glass
This series provides a way to keep track of the images used in bootstd, including the type of each image. At present this is sort-of handled by struct bootflow but in quite an ad-hoc way. The structure has become quite large and is hard to query. Future work will be able to reduce its size. Ultim

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

2024-11-04 Thread Tom Rini
On Mon, Nov 04, 2024 at 09:58:50PM +0100, Heinrich Schuchardt wrote: > > > Am 4. November 2024 14:39:48 MEZ schrieb Simon Glass : > >The package names are slightly different for arm64 and we don't seem to > >have a linux-image-kvm package. Provide a different set for arm64 > > > >Signed-off-by: S

Re: [PATCH 8/9] docker: Fix LegacyKeyValueFormat warning with PYTHONPATH

2024-11-04 Thread Heinrich Schuchardt
On 11/4/24 14:39, Simon Glass wrote: 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 --- tools/docker/Doc

Re: [PATCH 6/9] docker: Drop tracing for arm64

2024-11-04 Thread Heinrich Schuchardt
On 11/4/24 14:39, Simon Glass wrote: This doesn't build on arm64 at present, so drop it for now. This can be revisited if we wish to support arm64 for the sandbox trace-test. I had no problems building the trace libraries on Ubuntu 24.04 arm64. So the problem you experienced should be easily fi

Re: [PATCH 5/9] docker: Drop grub for arm64

2024-11-04 Thread Tom Rini
On Mon, Nov 04, 2024 at 10:04:39PM +0100, Heinrich Schuchardt wrote: > > > Am 4. November 2024 14:39:49 MEZ schrieb Simon Glass : > >This doesn't build on arm64 at present, so drop it for now. We only > >expect to run sandbox tests, so perhaps it isn't needed. > > > >Signed-off-by: Simon Glass >

Re: [PATCH 2/4] lmb: cosmetic: reorder functions and global LMB variable

2024-11-04 Thread Tom Rini
On Fri, Nov 01, 2024 at 11:15:52AM +0100, Janne Grunau wrote: > Low lovel LMB functionality will be used to manage IOVA space in the > Apple dart iommu driver. This reordering ensures that those function > can not access the global LMB memory map variable. > > Signed-off-by: Janne Grunau Review

Re: [PATCH 0/4] Fix IOVA allocation in Apple dart iommu after global LMB mem map changes

2024-11-04 Thread Tom Rini
On Sun, Nov 03, 2024 at 10:56:00PM +0100, Mark Kettenis wrote: > > Date: Sun, 3 Nov 2024 18:36:33 +0100 > > From: Janne Grunau > > > > On Sun, Nov 03, 2024 at 07:53:36PM +0530, Sughosh Ganu wrote: > > > On Sat, 2 Nov 2024 at 16:00, Mark Kettenis > > > wrote: > > > > > > > > > From: Sughosh Ganu

Re: [PATCH 1/4] lmb: Do not use global LMB variable in _lmb_free()

2024-11-04 Thread Tom Rini
On Fri, Nov 01, 2024 at 11:15:51AM +0100, Janne Grunau wrote: > It will be re-used with a lmb list pointer as argument for IOVA > allocations in the apple_dart iommu driver. > > Signed-off-by: Janne Grunau Reviewed-by: Tom Rini -- Tom signature.asc Description: PGP signature

Re: [PATCH 5/9] docker: Drop grub for arm64

2024-11-04 Thread Heinrich Schuchardt
Am 4. November 2024 14:39:49 MEZ schrieb Simon Glass : >This doesn't build on arm64 at present, so drop it for now. We only >expect to run sandbox tests, so perhaps it isn't needed. > >Signed-off-by: Simon Glass >--- > > tools/docker/Dockerfile | 7 +-- > 1 file changed, 5 insertions(+), 2 d

[PATCH 2/3] rockchip: rk3288: Use rk3288-cru.h from dts/upstream

2024-11-04 Thread Jonas Karlman
clock/rk3288-cru.h in include/dt-bindings is almost identical to the version in dts/upstream, remove the copy from include/dt-bindings to only use the version from dts/upstream. One clk, SCLK_MAC_PLL, is not part of the upstream bindings, this clk is not used by any upstream or in-tree DT. For now

[PATCH 3/3] rockchip: rk3288: Use rk3288.dtsi from dts/upstream

2024-11-04 Thread Jonas Karlman
rk3288.dtsi from arch/arm/dts is almost identical to the rk3288.dtsi from dts/upstream, it differs only with a minor change in hdmi port nodes, something that does not affect U-Boot. Remove arch/arm/dts/rk3288.dtsi to use rk3288.dtsi from dts/upstream. Also drop gpio aliases from -u-boot.dtsi that

[PATCH 1/3] rockchip: rk3288: Use rk3288-power.h from dts/upstream

2024-11-04 Thread Jonas Karlman
power/rk3288-power.h in include/dt-bindings is identical to the version in dts/upstream, remove the copy from include/dt-bindings to only use the version from dts/upstream. No functional change to board DTs is intended with this removal. Signed-off-by: Jonas Karlman --- include/dt-bindings/powe

[PATCH 0/3] rockchip: rk3288: Include files from dts/upstream

2024-11-04 Thread Jonas Karlman
This series remove common rk3288 related device tree files that instead can be included directly from dts/upstream. No functional change to board DTs is intended with these removals. Jonas Karlman (3): rockchip: rk3288: Use rk3288-power.h from dts/upstream rockchip: rk3288: Use rk3288-cru.h f

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

2024-11-04 Thread Heinrich Schuchardt
Am 4. November 2024 14:39:48 MEZ schrieb Simon Glass : >The package names are slightly different for arm64 and we don't seem to >have a linux-image-kvm package. Provide a different set for arm64 > >Signed-off-by: Simon Glass >--- > > tools/docker/Dockerfile | 12 +++- > 1 file changed, 7

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

2024-11-04 Thread Heinrich Schuchardt
Am 4. November 2024 14:39:47 MEZ schrieb Simon Glass : >Add instructions on how to build the file for multiple architectures. >Add a message indicating what is happening. > >Signed-off-by: Simon Glass >--- > > tools/docker/Dockerfile | 7 +++ > 1 file changed, 7 insertions(+) > >diff --git a

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

2024-11-04 Thread Heinrich Schuchardt
Am 4. November 2024 14:39:46 MEZ schrieb 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. > >Signed-off-by: Simon Glass >--- > > test/print_ut.c | 6 +++--- > 1 file changed, 3

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

2024-11-04 Thread Heinrich Schuchardt
Am 4. November 2024 14:39:45 MEZ schrieb 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 u

[PATCH] rockchip: rk3288: Fix bootph prop for vop nodes

2024-11-04 Thread Jonas Karlman
The vop nodes does not need to be included in xPL control FDT, they only need to be included at U-Boot proper pre-reloc phase. Change to use bootph-some-ram prop to fix this. Signed-off-by: Jonas Karlman --- arch/arm/dts/rk3288-u-boot.dtsi | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(

Re: [PATCH 00/17] (no cover subject)

2024-11-04 Thread Marek Vasut
On 11/4/24 1:39 PM, Peng Fan wrote: Subject: Re: [PATCH 00/17] (no cover subject) On 11/4/24 1:20 PM, Peng Fan wrote: Subject: Re: [PATCH 00/17] (no cover subject) On 10/16/24 9:17 AM, Alice Guo wrote: This patch set adds the basic support of i.MX95 and has been tested on i.MX95 19x19 EVK.

Re: [u-boot-test-hooks][PATCH] labgrid: Add wrapper scripts around using labgrid-client for device control

2024-11-04 Thread Tom Rini
On Thu, Aug 29, 2024 at 12:56:20PM -0600, Tom Rini wrote: > From: Edi Feschiyan > > General usage for these hooks is documented in the README.md file. In > terms of implementation, To start with, we add a common file to ensure > that all required environment variables are set. Next, we add a con

Re: [u-boot-test-hooks PATCH v5 0/3] Add support for Labgrid

2024-11-04 Thread Tom Rini
On Mon, 04 Nov 2024 11:36:40 -0700, Simon Glass wrote: > This adds hooks for use with my integration for Labgrid (called > Labgrid-sjg). The only existing hook that really does anything with the > target, with this integration, is the 'console' one. There is also a new > 'getrole' hook which looks

Re: [PATCH] clk: Propagate clk_set_rate() if CLK_SET_PARENT_RATE present

2024-11-04 Thread Sam Protsenko
Hi Tom, Sean, Can we please apply this patch? I don't really understand why it's been consistently ignored for so long, with no explanation even. eMMC on E850-96 still doesn't work properly, this patch is really needed. Thanks! On Tue, Aug 13, 2024 at 8:17 PM Sam Protsenko wrote: > > Hi Sean, >

[u-boot-test-hooks PATCH v5 1/3] Create a common file for test scripts

2024-11-04 Thread Simon Glass
The top part of each of the u-boot-test-* files is common. Put it in a common script file to avoid duplication and to allow it to be replaced for the Labgrid-sjg integration. Signed-off-by: Simon Glass Reviewed-by: Tom Rini --- Changes in v5: - Use Labgrid-sjg instead of Labgrid - Use LABGRID_S

[u-boot-test-hooks PATCH v5 3/3] Provide some basic scripts for Labgrid integration

2024-11-04 Thread Simon Glass
With Labgrid we don't need to specify the various methods, except for the console, which simply calls labgrid-client. This allows supporting any boards in your lab, without adding per-board configuration to these hooks. Provide ellesmere files as an example lab-configuration. Signed-off-by: Simo

[u-boot-test-hooks PATCH v5 2/3] Add .none scripts for power off/on

2024-11-04 Thread Simon Glass
These methods don't have a 'none' method right now, but it will be needed for Labgrid. Add these scripts. Note that these are not used by the existing integration, which requires that boards can be powered off and on. Signed-off-by: Simon Glass --- (no changes since v4) Changes in v4: - Add ne

[u-boot-test-hooks PATCH v5 0/3] Add support for Labgrid

2024-11-04 Thread Simon Glass
This adds hooks for use with my integration for Labgrid (called Labgrid-sjg). The only existing hook that really does anything with the target, with this integration, is the 'console' one. There is also a new 'getrole' hook which looks up a role to find various things about it, such as the U-Boot b

[PATCH v3 18/19] bootstd: Add the concept of an ad-hoc bootflow

2024-11-04 Thread Simon Glass
The normal situation with bootstd is that a bootflow is created from a bootmeth. In some cases, a script or user-command may cause an image to be loaded. To deal with this, add the concept of an ad-hoc bootflow. This can be used to record information about manually loaded images. Signed-off-by: Si

[PATCH v3 10/19] bootmeth: Update the read_file() method to include a type

2024-11-04 Thread Simon Glass
We want to record the type of each file which is loaded. Add an new parameter for this, to the read_file() method. Update all users. Make bootmeth_common_read_file() store information about the image that is read. Signed-off-by: Simon Glass Reviewed-by: Mattijs Korpershoek --- Changes in v3: -

[PATCH v2] tools: imx8image: add possibility to skip dcd

2024-11-04 Thread Heiko Schocher
Currently U-Boot always adds DCD Image to boot container. On imx8qxp SoC it is possible to init RAM from within SCFW, and adding a DCD image type to the boot container in this case breaks booting (No debug output anymore from SCFW! Nor any output from SPL), so we need to configure a dcd skip someho

[PATCH v3 05/19] bootstd: Move the bootflow list into an alist

2024-11-04 Thread Simon Glass
Use an alist for this data structure as it is somewhat simpler to manage. This means that bootstd holds a simple list of bootflow structs and can drop it at will, without chasing down lists. Signed-off-by: Simon Glass --- Changes in v3: - Update bootstd_add_bootflow() to return the element numbe

[PATCH v3 02/19] bootstd: Move bootflow-clearing to bootstd

2024-11-04 Thread Simon Glass
This relates to more than just the bootdev, since there is a global list of bootflows. Move the function to the bootstd file and rename it. Signed-off-by: Simon Glass --- (no changes since v1) boot/bootdev-uclass.c | 19 +-- boot/bootstd-uclass.c | 15 +++ cmd/bootf

[PATCH v3 01/19] bootstd: Move bootflow-adding to bootstd

2024-11-04 Thread Simon Glass
This relates to more than just the bootdev, since there is a global list of bootflows. Move the function to the bootstd file and rename it. Signed-off-by: Simon Glass --- (no changes since v1) boot/bootdev-uclass.c | 25 - boot/bootstd-uclass.c | 25

[PATCH v3 19/19] fs: Record loaded files in an ad-hoc bootflow

2024-11-04 Thread Simon Glass
This makes a start on dealing with images loaded outside the context of bootstd. For now, it just records these images. They can be listed using the 'bootstd images' command. Often, very little is known about these images, but future work could perhaps use the filename or contents to detect the ty

[PATCH v3 17/19] bootstd: Export bootdev_get_from_blk()

2024-11-04 Thread Simon Glass
Export this function so it can be used from other files. Signed-off-by: Simon Glass --- Changes in v3: - Add new patch to export bootdev_get_from_blk() boot/bootdev-uclass.c | 2 +- include/bootdev.h | 9 + 2 files changed, 10 insertions(+), 1 deletion(-) diff --git a/boot/bootdev

[PATCH v3 16/19] bootstd: Add a simple command to list images

2024-11-04 Thread Simon Glass
Add a new 'bootstd images' command, which lists the images which have been loaded. Update some existing tests to use it. Provide some documentation about images in general and this command in particular. Use a more realistic kernel command-line to make the test easier to follow. Signed-off-by: S

[PATCH v3 15/19] bootstd: Update cros bootmeth to record images

2024-11-04 Thread Simon Glass
Record images loaded by this bootmeth. Signed-off-by: Simon Glass --- Changes in v3: - Use the helper function Changes in v2: - Update to use a new image-type enum in bootflow.h boot/bootmeth_cros.c | 14 ++ 1 file changed, 14 insertions(+) diff --git a/boot/bootmeth_cros.c b/boo

[PATCH v3 14/19] Update bootmeth_alloc_other() to record images

2024-11-04 Thread Simon Glass
Update this function to add the image to the list. Signed-off-by: Simon Glass --- Changes in v3: - Use the helper function Changes in v2: - Update to use a new image-type enum in bootflow.h boot/bootmeth-uclass.c | 6 +- boot/bootmeth_script.c | 4 ++-- include/bootmeth.h | 3 ++- 3 f

[PATCH v3 13/19] boot: Update pxe bootmeth to record images

2024-11-04 Thread Simon Glass
Record images loaded by this bootmeth. Signed-off-by: Simon Glass --- Changes in v3: - Use the helper function boot/bootmeth_pxe.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/boot/bootmeth_pxe.c b/boot/bootmeth_pxe.c index b2873075ba5..b91e61bcbc4 100644 --- a/boot/bootmeth_pxe.c ++

[PATCH v3 12/19] boot: Update extlinux pxe_getfile_func() to include type

2024-11-04 Thread Simon Glass
Add a file-type parameter to this function and update all users. Add a proper comment to the function which we are here. This will allow tracking of the file types loaded by the extlinux bootmeth. Signed-off-by: Simon Glass --- (no changes since v2) Changes in v2: - Update to use a new image-t

[PATCH v3 11/19] bootmeth_efi: Check the filename-allocation in the network path

2024-11-04 Thread Simon Glass
If the filename cannot be set we should give up. Add the missing error check. Signed-off-by: Simon Glass --- (no changes since v2) Changes in v2: - Change the tag to bootmeth_efi boot/bootmeth_efi.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/boot/bootmeth_efi.c b/boot/bootmeth_efi.

[PATCH v3 09/19] bootmeth_efi: Simplify reading files by using the common function

2024-11-04 Thread Simon Glass
The efiload_read_file() does similar things to a common function, so update it to use that instead. Signed-off-by: Simon Glass Acked-by: Ilias Apalodimas --- (no changes since v2) Changes in v2: - Change the tag to bootmeth_efi boot/bootmeth_efi.c | 11 +++ 1 file changed, 3 insertio

[PATCH v3 08/19] boot: pxe: Drop the duplicate comment on get_pxe_file()

2024-11-04 Thread Simon Glass
This function is exported, so document it in the header file. Drop the duplicate comment in the C file. Signed-off-by: Simon Glass --- (no changes since v1) boot/pxe_utils.c | 10 -- 1 file changed, 10 deletions(-) diff --git a/boot/pxe_utils.c b/boot/pxe_utils.c index d6a4b2cb859..4f

[PATCH v3 07/19] bootstd: Update bootmeth_alloc_file() to record images

2024-11-04 Thread Simon Glass
As a first step to recording images and where they came from, update this function to do so, since it is used by two bootmeths Create a helper function in the bootflow system, since recorded images are always associated with bootflows. Signed-off-by: Simon Glass --- Changes in v3: - Add a helpe

[PATCH v3 06/19] bootstd: Maintain a list of images

2024-11-04 Thread Simon Glass
We want to keep track of images which are loaded, or those which could perhaps be loaded. This will make it easier to manage memory allocation, as well as permit removal of the EFI set_efi_bootdev() feature. Add a list of these, attached to the bootflow. For now the list is empty. Signed-off-by:

[PATCH v3 04/19] bootstd: Drop the bootdev-specific list of bootflows

2024-11-04 Thread Simon Glass
This list is only used by two functions, which can be updated to iterate through the global list. Take this approach, which allows the bootdev list to be dropped. Overall this makes the code slightly more complicated, but will allow moving the bootflow list into an alist Signed-off-by: Simon Glas

[PATCH v3 03/19] bootstd: Add a function to get bootstd only if available

2024-11-04 Thread Simon Glass
Provide a function which is safe to call in the 'unbind' path, which returns the bootstd priv data if available. Signed-off-by: Simon Glass --- (no changes since v1) boot/bootstd-uclass.c | 11 +++ include/bootstd.h | 17 + 2 files changed, 28 insertions(+) diff --

Re: [PATCH 1/4] clk: imx: clk-imx8mn Fix nand and spi clock parent

2024-11-04 Thread Adam Ford
On Mon, Nov 4, 2024 at 11:22 AM Michael Nazzareno Trimarchi wrote: > > Hi > > On Mon, Nov 4, 2024 at 6:17 PM Adam Ford wrote: > > > > On Mon, Nov 4, 2024 at 11:04 AM Michael Nazzareno Trimarchi > > wrote: > > > > > > Hi Adam > > > > > > On Mon, Nov 4, 2024 at 6:01 PM Adam Ford wrote: > > > > >

Re: [PATCH 1/4] clk: imx: clk-imx8mn Fix nand and spi clock parent

2024-11-04 Thread Michael Nazzareno Trimarchi
Hi On Mon, Nov 4, 2024 at 6:17 PM Adam Ford wrote: > > On Mon, Nov 4, 2024 at 11:04 AM Michael Nazzareno Trimarchi > wrote: > > > > Hi Adam > > > > On Mon, Nov 4, 2024 at 6:01 PM Adam Ford wrote: > > > > > > On Mon, Nov 4, 2024 at 10:29 AM Michael Nazzareno Trimarchi > > > wrote: > > > > > > >

Re: [PATCH 1/4] clk: imx: clk-imx8mn Fix nand and spi clock parent

2024-11-04 Thread Adam Ford
On Mon, Nov 4, 2024 at 11:04 AM Michael Nazzareno Trimarchi wrote: > > Hi Adam > > On Mon, Nov 4, 2024 at 6:01 PM Adam Ford wrote: > > > > On Mon, Nov 4, 2024 at 10:29 AM Michael Nazzareno Trimarchi > > wrote: > > > > > > Hi Adam > > > > > > On Mon, Nov 4, 2024 at 5:11 PM Adam Ford wrote: > > >

Re: [PATCH 1/4] clk: imx: clk-imx8mn Fix nand and spi clock parent

2024-11-04 Thread Michael Nazzareno Trimarchi
Hi Adam On Mon, Nov 4, 2024 at 6:01 PM Adam Ford wrote: > > On Mon, Nov 4, 2024 at 10:29 AM Michael Nazzareno Trimarchi > wrote: > > > > Hi Adam > > > > On Mon, Nov 4, 2024 at 5:11 PM Adam Ford wrote: > > > > > > On Sun, Jul 7, 2024 at 3:45 AM Michael Trimarchi > > > wrote: > > > > > > > > The

  1   2   >