Re: [PATCH 0/4] Generate capsules from config files

2024-03-12 Thread Sughosh Ganu
hi Bence, On Tue, 12 Mar 2024 at 20:28, Bence Balogh wrote: > > Hello Sughosh, > > We are planning to use these changes in the Corstone-1000 platform. > Do you have a timeline when these are going to get merged? I will prioritize this task since you want to use these changes. I need to send

[PATCH] mtd: nand: raw: mt7621-nand: allow writing ecc region in raw mode

2024-03-12 Thread Weijie Gao
Allow writing ecc parity region in raw mode. This makes sure the nand write.raw command can write the flash data as-is. Change-Id: Ibed3bdf13c9cf81e54041c5ac7a78192b97dcedc Signed-off-by: Weijie Gao CR-Id: WCNCR00180092 --- drivers/mtd/nand/raw/mt7621_nand.c | 6 +++--- 1 file changed, 3

RE: [PATCH v4 1/1] arch: arm: Agilex5 enablement

2024-03-12 Thread Chee, Tien Fong
Hi, > -Original Message- > From: Lim, Jit Loon > Sent: Tuesday, March 12, 2024 10:01 PM > To: u-boot@lists.denx.de > Cc: Jagan Teki ; Marek ; > Simon ; Chee, Tien Fong > ; Hea, Kok Kiang ; > Maniyam, Dinesh ; Ng, Boon Khai > ; Yuslaimi, Alif Zakuan > ; Chong, Teik Heng > ; Zamri,

RE: [PATCH] apalis-imx8: Fix sc_misc_otp_fuse_read() error check

2024-03-12 Thread Peng Fan
> -Original Message- > From: Fabio Estevam > Sent: Wednesday, March 13, 2024 9:00 AM > To: feste...@gmail.com > Cc: Marcel Ziswiler ; > hiago.fra...@toradex.com; Francesco Dolcini > ; u-boot@lists.denx.de; Peng Fan > ; joao.goncal...@toradex.com > Subject: [PATCH] apalis-imx8: Fix

RE: [PATCH] colibri-imx8x: Fix sc_misc_otp_fuse_read() error check

2024-03-12 Thread Peng Fan
> Subject: [PATCH] colibri-imx8x: Fix sc_misc_otp_fuse_read() error check > > Commit aa6e698a7acd ("imx: toradex/colibri-imx8x: correct SCU API usage") > made an incorrect logic change in the error code check of > sc_misc_otp_fuse_read(): > > - if (sc_err == SC_ERR_NONE) { > + if

Re: [PATCH] imx8m*_venice: move venice to OF_UPSTREAM

2024-03-12 Thread Fabio Estevam
Hi Tim, On Tue, Mar 12, 2024 at 4:05 PM Tim Harvey wrote: > > Move to imx8m{m,n,p}-venice to OF_UPSTREAM: > - replace the non-upstream generic imx8m{m,n,p}-venice dt with one of the >dt's from the OF_LIST > - handle the fact that dtbs now have a 'freescale/' prefix > - imply OF_UPSTREAM >

[PATCH] apalis-imx8: Fix sc_misc_otp_fuse_read() error check

2024-03-12 Thread Fabio Estevam
Commit bfb3409d676f ("imx: toradex/apalis-imx8: correct SCU API usage") made an incorrect logic change in the error code check of sc_misc_otp_fuse_read(): - if (scierr == SC_ERR_NONE) { + if (scierr) { /* QP has one A72 core disabled */ is_quadplus =

Re: [PATCH] ARM: imx: stm32: Test whether ethernet node is enabled before reading MAC EEPROM on DHSOM

2024-03-12 Thread Fabio Estevam
Hi Marek, On Tue, Mar 12, 2024 at 6:16 PM Marek Vasut wrote: > NOTE: It is probably best if this goes in via either imx or stm32 tree, > I can break the patch up, but that would introduce dependency > between two PRs in different trees. Let me know what you prefer. I can apply it

[PATCH] colibri-imx8x: Fix sc_misc_otp_fuse_read() error check

2024-03-12 Thread Fabio Estevam
Commit aa6e698a7acd ("imx: toradex/colibri-imx8x: correct SCU API usage") made an incorrect logic change in the error code check of sc_misc_otp_fuse_read(): - if (sc_err == SC_ERR_NONE) { + if (sc_err) { /* DX has two A35 cores disabled */ return (val &

[PATCH v2 12/12] rockchip: board: Move gpt_capsule_update_setup() call

2024-03-12 Thread Jonas Karlman
Move the call to gpt_capsule_update_setup() from the weak function rk_board_late_init() into the main board_late_init() function. Also change to use IS_ENABLED() instead for defined(). Signed-off-by: Jonas Karlman --- v2: - New patch, split from "board: rockchip: Add a common ROCK Pi 4 target"

[PATCH v2 10/12] rockchip: theobroma-systems: puma: Migrate to use IO-domain driver

2024-03-12 Thread Jonas Karlman
Switch to use the IO-domain driver to configure IO-domain based on device tree instead of a setup_iodomain() function. Signed-off-by: Jonas Karlman Reviewed-by: Kever Yang --- v2: - Collect r-b tag --- .../puma_rk3399/puma-rk3399.c | 21 ---

[PATCH v2 11/12] rockchip: google: gru: Migrate to use IO-domain driver

2024-03-12 Thread Jonas Karlman
Switch to use the IO-domain driver to configure IO-domain based on device tree instead of a setup_iodomain() function. Signed-off-by: Jonas Karlman Reviewed-by: Kever Yang --- v2: - Collect r-b tag --- board/google/gru/gru.c | 35 --

[PATCH v2 09/12] rockchip: vamrs: rock960: Migrate to use IO-domain driver

2024-03-12 Thread Jonas Karlman
Switch to use the IO-domain driver to configure IO-domain based on device tree instead of a setup_iodomain() function. Signed-off-by: Jonas Karlman Reviewed-by: Kever Yang --- v2: - Collect r-b tag --- board/vamrs/rock960_rk3399/Makefile | 6 -

[PATCH v2 08/12] rockchip: pine64: pinephone-pro: Migrate to use IO-domain driver

2024-03-12 Thread Jonas Karlman
Switch to use the IO-domain driver to configure IO-domain based on device tree instead of a setup_iodomain() function. Signed-off-by: Jonas Karlman Reviewed-by: Kever Yang --- v2: - Collect r-b tag --- board/pine64/pinephone-pro-rk3399/Makefile| 1 - .../pinephone-pro-rk3399.c

[PATCH v2 06/12] rockchip: pine64: rockpro64: Migrate to use IO-domain driver

2024-03-12 Thread Jonas Karlman
Switch to use the IO-domain driver to configure IO-domain based on device tree instead of a setup_iodomain() function. Signed-off-by: Jonas Karlman Reviewed-by: Kever Yang --- v2: - Collect r-b tag --- board/pine64/rockpro64_rk3399/Makefile| 7

[PATCH v2 07/12] rockchip: pine64: pinebook-pro: Migrate to use IO-domain driver

2024-03-12 Thread Jonas Karlman
Switch to use the IO-domain driver to configure IO-domain based on device tree instead of a setup_iodomain() function. Signed-off-by: Jonas Karlman Reviewed-by: Kever Yang --- v2: - Collect r-b tag --- board/pine64/pinebook-pro-rk3399/Makefile | 1 -

[PATCH v2 04/12] board: rockchip: Add a common ROCK Pi 4 target

2024-03-12 Thread Jonas Karlman
Move ROCK Pi 4 specific board code from the shared evb_rk3399 target into its own board target and update related defconfigs to use the new TARGET_ROCKPI4_RK3399 option. Signed-off-by: Jonas Karlman Reviewed-by: Christopher Obbard Reviewed-by: Peter Robinson --- v2: - Use glob pattern -

[PATCH v2 05/12] rockchip: io-domain: Add support for RK3399

2024-03-12 Thread Jonas Karlman
Port the RK3399 part of the Rockchip IO-domain driver from linux. This differs from linux version in that pmu io iodomain bit is enabled in the write ops instead of in an init ops as in linux, this way we can avoid keeping a full state of all supply that have been configured. Signed-off-by:

[PATCH v2 03/12] board: rockchip: rk3399: Remove unused board_early_init_f functions

2024-03-12 Thread Jonas Karlman
These functions is excluded from SPL build and BOARD_EARLY_INIT_F is not enabled for any of the affected boards, so this legacy code is not used. Rockchip common board code already enable all regulators flagged as always-on or boot-on in device tree, and fixed/gpio regulators now have basic

[PATCH v2 02/12] board: rockchip: rk3399: Add myself as reviewer to MAINTAINERS

2024-03-12 Thread Jonas Karlman
Add myself as a reviewer for RK3399 boards that I have and can help with review and testing of defconfig and device tree changes. Signed-off-by: Jonas Karlman Reviewed-by: Kever Yang --- v2: - Collect r-b tag --- board/pine64/pinebook-pro-rk3399/MAINTAINERS | 1 +

[PATCH v2 01/12] board: rockchip: rk3399: Add device tree files to MAINTAINERS

2024-03-12 Thread Jonas Karlman
Update MAINTAINERS files for RK3399 boards to include related device tree files. Also correct a few filenames. Signed-off-by: Jonas Karlman --- v2: - Use glob pattern where appropriate --- board/firefly/roc-pc-rk3399/MAINTAINERS | 1 + board/pine64/pinebook-pro-rk3399/MAINTAINERS | 5

[PATCH v2 00/12] rockchip: rk3399: Migrate to use IO-domain driver

2024-03-12 Thread Jonas Karlman
This series remove old unused board_early_init_f() functions, adds support for RK3399 to the IO-domain driver and migrates all boards that used setup_iodomain() to now use the IO-domain driver instead. Also add device tree files and myself as reviewer to MAINTAINERS. Changes in v2: - Use glob

Re: [PATCH v6 1/3] dt-bindings: mtd: partitions: Add binman compatible

2024-03-12 Thread Simon Glass
Hi Miquel, On Fri, 8 Mar 2024 at 20:42, Miquel Raynal wrote: > > Hi Simon, > > s...@chromium.org wrote on Fri, 8 Mar 2024 15:44:25 +1300: > > > Hi Miquel, > > > > On Tue, 6 Feb 2024 at 01:17, Miquel Raynal > > wrote: > > > > > > Hi Simon, > > > > > > > > > > > > > > > > > > +description: | > >

Re: [PATCH 04/11] board: rockchip: Add a common ROCK Pi 4 target

2024-03-12 Thread Jonas Karlman
Hi Kever, On 2024-03-11 10:29, Kever Yang wrote: > Hi Jonas, > > On 2024/2/19 17:13, Quentin Schulz wrote: >> Hi Jonas, >> >> On 2/17/24 19:35, Jonas Karlman wrote: >>> Move ROCK Pi 4 specific board code from the shared evb_rk3399 target >>> into its own board target and update related

Re: [PATCH 04/11] board: rockchip: Add a common ROCK Pi 4 target

2024-03-12 Thread Jonas Karlman
Hi Kever, On 2024-03-11 10:29, Kever Yang wrote: > Hi Jonas, > > On 2024/2/18 02:35, Jonas Karlman wrote: >> Move ROCK Pi 4 specific board code from the shared evb_rk3399 target >> into its own board target and update related defconfigs to use the new >> TARGET_ROCKPI4_RK3399 option. >> >> Also

Re: [PATCH 1/4] rockchip: Update the default USB Product ID value

2024-03-12 Thread Jonas Karlman
Hi Kever, On 2024-03-11 04:20, Kever Yang wrote: > Hi Jonas, > > On 2024/3/8 18:42, Jonas Karlman wrote: >> Hi Kever, >> >> On 2024-03-08 10:22, Kever Yang wrote: >>> Hi Jonas, >>> >>>     Thanks for your patch. >>> >>> On 2024/2/27 07:08, Jonas Karlman wrote: RK3036 is using the USB

[PATCH] ARM: imx: stm32: Test whether ethernet node is enabled before reading MAC EEPROM on DHSOM

2024-03-12 Thread Marek Vasut
Check whether the ethernet interface is enabled at all before reading MAC EEPROM. As a cost saving measure, it can happen that the MAC EEPROM is not populated on SoMs which do not use ethernet. Signed-off-by: Marek Vasut --- Cc: "NXP i.MX U-Boot Team" Cc: Andreas Geisreiter Cc: Christoph

[PATCH v4 10/13] arm: dts: introduce am62p5 U-Boot dts files

2024-03-12 Thread Bryan Brattlof
Include the U-Boot device tree files needed to boot the board. Signed-off-by: Bryan Brattlof --- arch/arm/dts/Makefile |2 + arch/arm/dts/k3-am62p-ddr-lp4-50-1600.dtsi | 2800 arch/arm/dts/k3-am62p-sk-binman.dtsi | 173 ++

[PATCH v4 03/13] arm: mach-k3: am62px: introduce clock and device files for wkup spl

2024-03-12 Thread Bryan Brattlof
Include the clock and lpsc tree files needed for the wkup spl to initialize the proper PLLs and power domains to boot the SoC. Reviewed-by: Neha Malcom Francis Signed-off-by: Bryan Brattlof --- arch/arm/mach-k3/r5/Makefile | 1 + arch/arm/mach-k3/r5/am62px/Makefile| 6 +

[PATCH v4 08/13] firmware: ti_sci_static_data: add static DMA channel data

2024-03-12 Thread Bryan Brattlof
From: Hari Nagalla Include the static DMA channel data for ti_sci Signed-off-by: Hari Nagalla Signed-off-by: Bryan Brattlof --- drivers/firmware/ti_sci_static_data.h | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/firmware/ti_sci_static_data.h

[PATCH v4 13/13] arm: mach-k3: fixup whitespace in SPDX License IDs

2024-03-12 Thread Bryan Brattlof
The SPDX ID format usese a single space used after the 'SPDX-License-Identifier:'. Fix all files that use any other white-space character other than a single space. Signed-off-by: Bryan Brattlof --- arch/arm/mach-k3/r5/am62ax/Makefile | 2 +- arch/arm/mach-k3/r5/am62px/Makefile | 2 +-

[PATCH v4 04/13] ram: k3-ddrss: enable the am62ax's DDR controller for am62px

2024-03-12 Thread Bryan Brattlof
The am62px family of SoCs uses the same DDR controller as found on the am62ax family. Enable this option when building for the am62px family Reviewed-by: Neha Malcom Francis Signed-off-by: Bryan Brattlof --- drivers/ram/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH v4 07/13] board: ti: introduce basic board files for the am62px family

2024-03-12 Thread Bryan Brattlof
Introduce the basic files needed to support the am62px family of SoCs Co-developed-by: Hari Nagalla Signed-off-by: Hari Nagalla Reviewed-by: Tom Rini Signed-off-by: Bryan Brattlof --- board/ti/am62px/Kconfig | 26 ++ board/ti/am62px/MAINTAINERS | 9 +

[PATCH v4 00/13] Introduce basic support for TI's AM62Px SoC family

2024-03-12 Thread Bryan Brattlof
Hello Again Everyone! The AM62Px is an extension of the existing Sitara AM62x low-cost family of application processors built for Automotive and Linux Application development. Scalable Arm Cortex-A53 performance and embedded features, such as: multi high-definition display support, 3D-graphics

[PATCH v4 12/13] doc: board: ti: introduce am62px documentation

2024-03-12 Thread Bryan Brattlof
Introduce basic documentation for the am62p family of SoCs. Signed-off-by: Bryan Brattlof --- doc/board/ti/am62px_sk.rst | 289 + doc/board/ti/k3.rst| 1 + 2 files changed, 290 insertions(+) diff --git a/doc/board/ti/am62px_sk.rst

[PATCH v4 09/13] dma: ti: k3-udma: Add DMA PSIL mappings for AM62P and J722S

2024-03-12 Thread Bryan Brattlof
From: Vignesh Raghavendra Add PSIL data for the AM62P and the J722S SoC family. The PSIL mapping for the J722S is the same except for the extra instances of the CSI-RX. So let's reuse the same file for both the AM62P and J722S. Signed-off-by: Vignesh Raghavendra Signed-off-by: Ravi Gunasekaran

[PATCH v4 06/13] arch: mach-k3: introduce basic files to support the am62px SoC family

2024-03-12 Thread Bryan Brattlof
Introduce the basic functions and definitions needed to properly initialize TI's am62p family of SoCs Signed-off-by: Bryan Brattlof --- arch/arm/mach-k3/Kconfig | 7 +- arch/arm/mach-k3/Makefile | 1 + arch/arm/mach-k3/am62p5_init.c

[PATCH v4 01/13] soc: add info to identify the am62p SoC family

2024-03-12 Thread Bryan Brattlof
Include the part number for TI's am62px family of SoCs so we can properly identify it during boot Reviewed-by: Igor Opaniuk Reviewed-by: Neha Malcom Francis Signed-off-by: Bryan Brattlof --- arch/arm/mach-k3/include/mach/hardware.h | 2 ++ drivers/soc/soc_ti_k3.c | 3 +++ 2

[PATCH v4 05/13] arm: mach-k3: invert logic for split DM firmware config

2024-03-12 Thread Bryan Brattlof
Currently, for the K3 generation of SoCs, there are more SoCs that utilize the split firmware approach than the combined DMSC firmware. Invert the logic to avoid adding more and more SoCs to this list. Reviewed-by: Neha Malcom Francis Acked-by: Andrew Davis Signed-off-by: Bryan Brattlof ---

[PATCH v4 11/13] configs: introduce configs needed for the am62px

2024-03-12 Thread Bryan Brattlof
Introduce the initial configs needed to support the am62px SoC family Signed-off-by: Bryan Brattlof --- configs/am62px_evm_a53_defconfig | 177 +++ configs/am62px_evm_r5_defconfig | 137 ++ include/configs/am62px_evm.h | 14

[PATCH v4 02/13] power: domain: ti: use IS_ENABLED macro

2024-03-12 Thread Bryan Brattlof
Cleanup this list and standardize on using the IS_ENABLED macro for the power domain data list. Reviewed-by: Igor Opaniuk Signed-off-by: Bryan Brattlof --- drivers/power/domain/ti-power-domain.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git

Re: [PATCH v3 2/4] clk: qcom: clear div mask before assigning a new divider

2024-03-12 Thread Volodymyr Babchuk
Hi Sumit, Thank you for the review. Sumit Garg writes: > On Tue, 12 Mar 2024 at 03:03, Volodymyr Babchuk > wrote: >> >> The current behaviour does a bitwise OR of the previous and new >> divider values, this is wrong as some bits maybe be set already. We > > nit: s/maybe be/maybe/ > Oops,

Re: About U-Boot License

2024-03-12 Thread Tom Rini
To be clear, having both U-Boot and RTEMS on a flash image together is not considered linking. Having U-Boot start RTEMS by the mechanism that exists upstream today is also not considered linking. On Tue, Mar 12, 2024 at 09:37:32AM +0900, 강훈 wrote: > Dear Tom > > Thank you for your reply soon. >

Re: [PATCH] arm64: zynqmp: Do not describe u-boot.itb if SPL is disabled

2024-03-12 Thread Ilias Apalodimas
On Tue, 12 Mar 2024 at 17:55, Michal Simek wrote: > > > > On 3/12/24 07:14, Ilias Apalodimas wrote: > > Hi Michal > > > > Apologies for the late reply > > > > On Wed, 6 Mar 2024 at 09:48, Michal Simek wrote: > >> > >> > >> > >> On 3/5/24 16:47, Ilias Apalodimas wrote: > >>> On Fri, Feb 23, 2024

Re: [PATCH 7/7 v2] arm: remove redundant section alignments

2024-03-12 Thread Richard Henderson
On 3/12/24 04:08, Ilias Apalodimas wrote: index 33f4624b561d..ccdd1966cfbc 100644 --- a/arch/arm/cpu/armv8/u-boot.lds +++ b/arch/arm/cpu/armv8/u-boot.lds @@ -132,7 +132,7 @@ SECTIONS _end = .; - .bss ALIGN(8): { + .bss : { __bss_start = .;

Re: [PATCH 6/7 v2] arm: move image_copy_start/end to linker symbols

2024-03-12 Thread Richard Henderson
On 3/12/24 04:08, Ilias Apalodimas wrote: image_copy_start/end are defined as c variables in order to force the compiler emit relative references. However, defining those within a section definition will do the same thing since [0]. So let's remove the special sections from the linker scripts,

Re: [PATCH 5/7 v2] arm: fix __efi_runtime_start/end definitions

2024-03-12 Thread Richard Henderson
On 3/12/24 04:08, Ilias Apalodimas wrote: __efi_runtime_start/end are defined as c variables for arm7 only in order to force the compiler emit relative references. However, defining those within a section definition will do the same thing since [0]. On top of that the v8 linker scripts define it

Re: [PATCH 4/7 v2] arm: clean up v7 and v8 linker scripts for __rel_dyn_start/end

2024-03-12 Thread Richard Henderson
On 3/12/24 04:08, Ilias Apalodimas wrote: commit 47bd65ef057f ("arm: make __rel_dyn_{start, end} compiler-generated") were moving the __rel_dyn_start/end on c generated variables that were injected in their own sections. The reason was that we needed relative relocations for position independent

Re: [PATCH 3/7 v2] arm: fix __efi_runtime_rel_start/end definitions

2024-03-12 Thread Richard Henderson
On 3/12/24 04:08, Ilias Apalodimas wrote: __efi_runtime_rel_start/end are defined as c variables for arm7 only in order to force the compiler emit relative references. However, defining those within a section definition will do the same thing since [0]. On top of that the v8 linker scripts

Re: [PATCH 2/7 v2] arm: clean up v7 and v8 linker scripts for bss_start/end

2024-03-12 Thread Richard Henderson
On 3/12/24 04:08, Ilias Apalodimas wrote: - .bss_start (NOLOAD) : { - . = ALIGN(8); This alignment got lost. - KEEP(*(.__bss_start)); - } >.sdram - - .bss (NOLOAD) : { + .bss : { If it is required, the best replacement would be here on

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

2024-03-12 Thread Tom Rini
On Tue, Mar 12, 2024 at 04:51:50PM +0800, Leo Liang wrote: > Hi Tom, > > The following changes since commit f3c979dd0053c082d2df170446923e7ce5edbc2d: > > Prepare v2024.04-rc4 (2024-03-11 13:11:46 -0400) > > are available in the Git repository at: > >

Re: [PATCH v3 1/4] qcom: board: validate fdt before trying to use it

2024-03-12 Thread Caleb Connolly
On 11/03/2024 21:33, Volodymyr Babchuk wrote: > There are cases when previous bootloader stage leaves some seemingly > valid value in r0, which in fact does not point to valid FDT > blob. This behavior was encountered when trying to boot U-Boot as > "hyp" loader on SA8155P-ADP. > > To be sure

Re: [PATCH v3 3/4] clk: qcom: add support for power domains uclass

2024-03-12 Thread Caleb Connolly
On 11/03/2024 21:33, Volodymyr Babchuk wrote: > Now sub-drivers for particular SoCs can register them as power domain > drivers. This is needed for upcoming SM8150 support, because it needs > to power up the Ethernet module. > > Signed-off-by: Volodymyr Babchuk Reviewed-by: Caleb Connolly >

Re: [PATCH 2/7 v2] arm: clean up v7 and v8 linker scripts for bss_start/end

2024-03-12 Thread Ilias Apalodimas
Hi Richard, Thanks for taking a look On Tue, 12 Mar 2024 at 17:51, Richard Henderson wrote: > > On 3/12/24 04:08, Ilias Apalodimas wrote: > > - .bss_start (NOLOAD) : { > > - . = ALIGN(8); > > This alignment got lost. > > > - KEEP(*(.__bss_start)); > > - } >.sdram

Re: [PATCH] arm64: zynqmp: Do not describe u-boot.itb if SPL is disabled

2024-03-12 Thread Michal Simek
On 3/12/24 07:14, Ilias Apalodimas wrote: Hi Michal Apologies for the late reply On Wed, 6 Mar 2024 at 09:48, Michal Simek wrote: On 3/5/24 16:47, Ilias Apalodimas wrote: On Fri, Feb 23, 2024 at 05:18:42PM +0100, Michal Simek wrote: There is no reason to describe u-boot.itb on system

[PATCH] arm64: zynqmp: Do not setup default SPL options without SPL

2024-03-12 Thread Michal Simek
mach Kconfig file select some options independently of if SPL is actually enabled. It ends up in situation that SPL is disabled but symbols are still present in .config. That's why add SPL dependency for them but also group them with other SPL options in ARCH_ZYNQMP fragment with using imply

Re: [PATCH] arm64: zynqmp: Do not describe u-boot.itb if SPL is disabled

2024-03-12 Thread Michal Simek
On 3/12/24 13:29, Heinrich Schuchardt wrote: On 05.03.24 16:47, Ilias Apalodimas wrote: On Fri, Feb 23, 2024 at 05:18:42PM +0100, Michal Simek wrote: There is no reason to describe u-boot.itb on system without SPL. Pretty much this is cover all systems which are using only boot.bin which

Re: [PATCH v4 2/2] android_ab: Fix ANDROID_AB_BACKUP_OFFSET

2024-03-12 Thread Mattijs Korpershoek
Hi Colin, Thank you for the patch. On mar., mars 12, 2024 at 07:57, Colin McAllister wrote: Sam also gave his review here: https://lore.kernel.org/all/CAPLW+4kHmPtfACyND4Vc2p0ZrsyGY=+bRU=fdub4k1ux5p3...@mail.gmail.com/ Please include his review tag in the next submission. I will add it at

Re: [PATCH] arm64: zynqmp: Describe USB wakeup interrupt

2024-03-12 Thread Michal Simek
On 3/8/24 09:41, Michal Simek wrote: Describe usb wakeup interrupt. Signed-off-by: Michal Simek --- arch/arm/dts/zynqmp.dtsi | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/arch/arm/dts/zynqmp.dtsi b/arch/arm/dts/zynqmp.dtsi index

Re: [PATCH] arm64: zynqmp: Remove additional compatible string for sc-revB

2024-03-12 Thread Michal Simek
On 3/8/24 09:41, Michal Simek wrote: Based on dt schema there is no need to specify flash via additional compatible string and generic are enough. Signed-off-by: Michal Simek --- arch/arm/dts/zynqmp-sc-revB.dts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [PATCH v4 1/2] android_ab: Add missing semicolon

2024-03-12 Thread Mattijs Korpershoek
Hi Colin, Thank you for the patch. On mar., mars 12, 2024 at 07:57, Colin McAllister wrote: > From: Colin McAllister > > Found a missing semicolon in code protected by a #if that will never > evaluate to true due to a separate issue. Fixing this issue before > addressing the #if. > > Fixes:

Re: [PATCH] arm64: zynqmp: Fix gpio-key DT description

2024-03-12 Thread Michal Simek
On 3/8/24 09:40, Michal Simek wrote: All gpio-key descriptionos with dt-schema. Signed-off-by: Michal Simek --- arch/arm/dts/zynqmp-sc-revB.dts| 2 +- arch/arm/dts/zynqmp-vp-x-a2785-00-revA.dts | 2 +- arch/arm/dts/zynqmp-vpk120-revA.dts| 2 +-

Re: [PATCH v2] arm64: zynqmp: Add label to pmu fwnode

2024-03-12 Thread Michal Simek
On 3/7/24 16:29, lukas.funke-...@weidmueller.com wrote: From: Lukas Funke ZynqMP CG series devices only have two cpus. In this case the interrupt-affinity property has to adapted, because cpu3 and cpu4 are missing. By adding a label to the pmu fwnode the interrupt-affinity can be adapted in

Re: [PATCH v2] mtd: nand: arasan: Fix the crash caused by use after free

2024-03-12 Thread Michal Simek
On 3/6/24 04:34, Venkatesh Yadav Abbarapu wrote: The below exception observed on QEMU, as it doesn't support NAND controller. "Synchronous Abort" handler, esr 0x9605, far 0x17acfc878 elr: 0803ad40 lr : 0805f438 (reloc) elr: 7fcb4d40 lr : 7fcd9438 x0 :

Re: [PATCH v4] mtd: nand: arasan: Print warning for unsupported ecc modes

2024-03-12 Thread Michal Simek
On 3/6/24 04:27, Venkatesh Yadav Abbarapu wrote: Currently only hw ecc is supported in U-Boot. If any other ecc mode is given in DT, it simply through an error. So better print what is being done. Revert this patch once soft ecc support is fixed in future. Signed-off-by: Ashok Reddy Soma

RE: [PATCH v3 0/2] Fix Android A/B backup

2024-03-12 Thread Mattijs Korpershoek
Hi Colin, On mar., mars 12, 2024 at 14:04, "McAllister, Colin" wrote: > Hi Mattijs, > > I’ve been using git send-email, but there might be issues with what the > Garmin smtp server is doing to the email, like adding the footer. I sent a v4 > PS in a new thread using my personal email, but

Re: [PATCH] arm64: zynqmp: Add usb4 to the boot targets

2024-03-12 Thread Michal Simek
On 3/5/24 12:02, Venkatesh Yadav Abbarapu wrote: USB4 has been added to the boot targets and also add support to enable JTAG. Signed-off-by: Shubhangi Shrikrushna Mahalle Signed-off-by: Venkatesh Yadav Abbarapu --- board/xilinx/zynqmp/zynqmp_kria.env | 3 ++- 1 file changed, 2

Re: [PATCH 0/4] Generate capsules from config files

2024-03-12 Thread Bence Balogh
Hello Sughosh, We are planning to use these changes in the Corstone-1000 platform. Do you have a timeline when these are going to get merged? Thanks, Bence On Tue, Nov 21, 2023 at 12:30:24PM +0530, Sughosh Ganu wrote: > > Add support for generating capsules through a config file. Multiple >

[PATCH v4 2/2] android_ab: Fix ANDROID_AB_BACKUP_OFFSET

2024-03-12 Thread Colin McAllister
From: Colin McAllister Currently, setting CONFIG_AB_BACKUP_OFFSET in a target's defconfig will not actually enable the #if protected code in android_ab.c. This is because "CONFIG_" should have been prepended to the config macro, or the macros defined in kconfig.h could have been used. The code

[PATCH v4 1/2] android_ab: Add missing semicolon

2024-03-12 Thread Colin McAllister
From: Colin McAllister Found a missing semicolon in code protected by a #if that will never evaluate to true due to a separate issue. Fixing this issue before addressing the #if. Fixes: 3430f24bc6 ("android_ab: Try backup booloader_message") Signed-off-by: Colin McAllister Cc: Joshua Watt Cc:

[PATCH v4 0/2] Fix Android A/B backup

2024-03-12 Thread Colin McAllister
- Addresses compiler error due to missing semicolon - Removes use of preprocessor macros with ANDROID_AB_BACKUP_OFFSET Bug was found by noticing a semicolon was missing and not causing a compiler error when CONFIG_ANDROID_AB_BACKUP_OFFSET was set. I submitted a patch to fix the semicolon before

[PATCH v3 4/5] include: env: Add phytec RAUC boot logic

2024-03-12 Thread Leonard Anderweit
Add logic for booting systems with the RAUC update mechanism. This can be reused by other phytec boards. Signed-off-by: Leonard Anderweit Tested-by: Teresa Remmet --- include/env/phytec/rauc.env | 52 + 1 file changed, 52 insertions(+) create mode 100644

[PATCH v3 5/5] board: phytec: phycore_imx8mp: Add RAUC boot logic to environment

2024-03-12 Thread Leonard Anderweit
Add RAUC boot logic to the environment. This is the first board to utilize this environment. Signed-off-by: Leonard Anderweit Tested-by: Teresa Remmet --- board/phytec/phycore_imx8mp/phycore_imx8mp.env | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git

[PATCH v3 3/5] configs: phycore-imx8mp_defconfig: Use redundant environment

2024-03-12 Thread Leonard Anderweit
Add support for the redundant environment. Signed-off-by: Leonard Anderweit Tested-by: Teresa Remmet --- configs/phycore-imx8mp_defconfig | 2 ++ 1 file changed, 2 insertions(+) diff --git a/configs/phycore-imx8mp_defconfig b/configs/phycore-imx8mp_defconfig index fd492eba3c10..6c67f14b5559

[PATCH v3 2/5] phycore_imx8mp: Move default bootcmd to board env

2024-03-12 Thread Leonard Anderweit
Move the default bootcmd from the defconfig to the board environment. No change in functionality. Signed-off-by: Leonard Anderweit Tested-by: Teresa Remmet --- board/phytec/phycore_imx8mp/phycore_imx8mp.env | 12 configs/phycore-imx8mp_defconfig | 2 -- 2 files

[PATCH v3 1/5] phycore_imx8mp: Move environment from include/config to board

2024-03-12 Thread Leonard Anderweit
Move the environment into the board directory and convert it from a C header to a text file. Sort the variables alphabetically. No functional changes. Signed-off-by: Leonard Anderweit Reviewed-by: Teresa Remmet Tested-by: Teresa Remmet --- .../phytec/phycore_imx8mp/phycore_imx8mp.env | 45

[PATCH v3 0/5] Add RAUC boot logic for phycore_imx8mp

2024-03-12 Thread Leonard Anderweit
This series adds RAUC boot logic for the phycore_imx8mp. The first patch converts the environment from a CFG_EXTRA_ENV_SETTINGS #define to a text environment for better readability and maintainability. The second patch moves the default bootcmd from the defconfig to the board environment. The

Re: [PATCH v2 5/5] board: phytec: phycore_imx8mp: Add RAUC boot logic to environment

2024-03-12 Thread Teresa Remmet
Am Dienstag, dem 12.03.2024 um 09:59 +0100 schrieb Leonard Anderweit: > Add RAUC boot logic to the environment. This is the first board to > utilize this environment. > > Signed-off-by: Leonard Anderweit Tested-by: Teresa Remmet > --- >  board/phytec/phycore_imx8mp/phycore_imx8mp.env | 7

Re: [PATCH v2 4/5] include: env: Add phytec RAUC boot logic

2024-03-12 Thread Teresa Remmet
Am Dienstag, dem 12.03.2024 um 09:59 +0100 schrieb Leonard Anderweit: > Add logic for booting systems with the RAUC update mechanism. This > can > be reused by other phytec boards. > > Signed-off-by: Leonard Anderweit Tested-by: Teresa Remmet > --- >  include/env/phytec/rauc.env | 52 >

Re: [PATCH v2 3/5] configs: phycore-imx8mp_defconfig: Use redundant environment

2024-03-12 Thread Teresa Remmet
Am Dienstag, dem 12.03.2024 um 09:59 +0100 schrieb Leonard Anderweit: > Add support for the redundant environment. > > Signed-off-by: Leonard Anderweit Tested-by: Teresa Remmet > --- >  configs/phycore-imx8mp_defconfig | 2 ++ >  1 file changed, 2 insertions(+) > > diff --git

Re: [PATCH v2 2/5] phycore_imx8mp: Move default bootcmd to board env

2024-03-12 Thread Teresa Remmet
Am Dienstag, dem 12.03.2024 um 09:59 +0100 schrieb Leonard Anderweit: > Move the default bootcmd from the defconfig to the board environment. > No change in functionality. > > Signed-off-by: Leonard Anderweit Tested-by: Teresa Remmet > --- >  board/phytec/phycore_imx8mp/phycore_imx8mp.env |

Re: [PATCH v2 1/5] phycore_imx8mp: Move environment from include/config to board

2024-03-12 Thread Teresa Remmet
Hello Leonard, Am Dienstag, dem 12.03.2024 um 09:59 +0100 schrieb Leonard Anderweit: > Move the environment into the board directory and convert it from a C > header to a text file. Sort the variables alphabetically. > No functional changes. > > Signed-off-by: Leonard Anderweit > --- >  

[PATCH 7/7 v2] arm: remove redundant section alignments

2024-03-12 Thread Ilias Apalodimas
Previous patches cleaning up linker symbols, also merged any explicit . = ALIGN(x); into section definitions -- e.g '.bss ALIGN(x) :' instead of '. - ALIGN(x); bss : {...}' However, if the output address is not specified then one will be chosen for the section. This address will be adjusted to

[PATCH 6/7 v2] arm: move image_copy_start/end to linker symbols

2024-03-12 Thread Ilias Apalodimas
image_copy_start/end are defined as c variables in order to force the compiler emit relative references. However, defining those within a section definition will do the same thing since [0]. So let's remove the special sections from the linker scripts, the variable definitions from sections.c and

[PATCH 5/7 v2] arm: fix __efi_runtime_start/end definitions

2024-03-12 Thread Ilias Apalodimas
__efi_runtime_start/end are defined as c variables for arm7 only in order to force the compiler emit relative references. However, defining those within a section definition will do the same thing since [0]. On top of that the v8 linker scripts define it as a symbol. So let's remove the special

[PATCH 4/7 v2] arm: clean up v7 and v8 linker scripts for __rel_dyn_start/end

2024-03-12 Thread Ilias Apalodimas
commit 47bd65ef057f ("arm: make __rel_dyn_{start, end} compiler-generated") were moving the __rel_dyn_start/end on c generated variables that were injected in their own sections. The reason was that we needed relative relocations for position independent code and linker bugs back then prevented us

[PATCH 3/7 v2] arm: fix __efi_runtime_rel_start/end definitions

2024-03-12 Thread Ilias Apalodimas
__efi_runtime_rel_start/end are defined as c variables for arm7 only in order to force the compiler emit relative references. However, defining those within a section definition will do the same thing since [0]. On top of that the v8 linker scripts define it as a symbol. So let's remove the

[PATCH 2/7 v2] arm: clean up v7 and v8 linker scripts for bss_start/end

2024-03-12 Thread Ilias Apalodimas
commit 3ebd1cbc49f0 ("arm: make __bss_start and __bss_end__ compiler-generated") and commit f84a7b8f54db ("ARM: Fix __bss_start and __bss_end in linker scripts") were moving the bss_start/end on c generated variables that were injected in their own sections. The reason was that we needed relative

[PATCH 1/7 v2] arm: baltos: remove custom linker script

2024-03-12 Thread Ilias Apalodimas
commit 3d74a0977f514 ("ti: am335x: Remove unused linker script") removed the linker script for the TI variant. This linker script doesn't seem to do anything special and on top of that, has no definitions for the EFI runtime sections. So let's get rid of it and use the generic linker script which

[PATCH 0/7 v2] Clean up arm linker scripts

2024-03-12 Thread Ilias Apalodimas
The arm linker scripts had a mix of symbols and C defined variables in an effort to emit relative references instead of absolute ones e.g [0]. A linker bug prevented us from doing so [1] -- fixed since 2016. This has led to confusion over the years, ending up with mixed section definitions. Some

RE: [PATCH v3 0/2] Fix Android A/B backup

2024-03-12 Thread McAllister, Colin
Hi Mattijs, I’ve been using git send-email, but there might be issues with what the Garmin smtp server is doing to the email, like adding the footer. I sent a v4 PS in a new thread using my personal email, but that email isn’t subscribed to this ML so I think the patches are pending approval

[PATCH v4 1/1] arch: arm: Agilex5 enablement

2024-03-12 Thread Jit Loon Lim
This patch is to enable Agilex5 platform for Intel product. Changes, modification and new files are created for board, dts, configs and makefile to create the base for Agilex5. Signed-off-by: Jit Loon Lim --- Changes v3 -> v4: - Update defined to is_enabled Changes v2 -> v3: - Added FPGA 240G

[PATCH v4 0/1] Agilex5 enablement

2024-03-12 Thread Jit Loon Lim
V1: https://patchwork.ozlabs.org/project/uboot/patch/20240220143603.22091-1-jit.loon@intel.com/ V2: Fixed Git auto-merge causing misalignment of code and insert/delete V3: Added 240G FPGA DDR region V4: Update defined to is_enabled Jit Loon Lim (1): arch: arm: Agilex5 enablement

Re: [PATCH 2/3] riscv: dts: sophgo: Add ethernet node

2024-03-12 Thread Conor Dooley
On Tue, Mar 12, 2024 at 05:59:44PM +0800, Leo Liang wrote: > On Sun, Mar 10, 2024 at 01:56:45PM +0800, Kongyang Liu wrote: > > Add ethernet node for cv1800b SoC > > > > Signed-off-by: Kongyang Liu > > --- > > > > arch/riscv/dts/cv18xx.dtsi | 6 ++ > > 1 file changed, 6 insertions(+) > >

Re: [PATCH 11/11] net: dwc_eth_qos: Add support of STM32MP13xx platform

2024-03-12 Thread Patrice CHOTARD
On 3/9/24 03:11, Marek Vasut wrote: > From: Christophe Roullier > > Add compatible "st,stm32mp13-dwmac" to manage STM32MP13 boards. > > Signed-off-by: Christophe Roullier > Signed-off-by: Marek Vasut # Rebase, reshuffle, squash code > --- > Cc: Christophe Roullier > Cc: Joe Hershberger >

Re: [PATCH 10/11] net: dwc_eth_qos: Add DT parsing for STM32MP13xx platform

2024-03-12 Thread Patrice CHOTARD
On 3/9/24 03:11, Marek Vasut wrote: > From: Christophe Roullier > > Manage 2 ethernet instances, select which instance to configure with mask > If mask is not present in DT, it is stm32mp15 platform. > > Signed-off-by: Christophe Roullier > Signed-off-by: Marek Vasut # Rework the code >

Re: [PATCH 09/11] net: dwc_eth_qos: Request clk-ck earlier in probe on STM32

2024-03-12 Thread Patrice CHOTARD
On 3/9/24 03:11, Marek Vasut wrote: > From: Patrick Delaunay > > Request the clk-ck earlier in probe in preparation for obtaining > the clock rate from these clk-ck in eqos_probe_syscfg_stm32() in > the follow up patch. > > Signed-off-by: Patrick Delaunay > --- > Cc: Christophe Roullier >

Re: [PATCH 08/11] net: dwc_eth_qos: Constify st,eth-* values parsed out of DT

2024-03-12 Thread Patrice CHOTARD
On 3/9/24 03:11, Marek Vasut wrote: > Use const bool for the values parsed out of DT. Drop the duplicate > assignment of false into those bool variables, assign them directly > with the content parsed out of DT. Abbreviate the variable name too. > > Signed-off-by: Marek Vasut > --- > Cc:

Re: [PATCH 07/11] net: dwc_eth_qos: Use consistent logging prints

2024-03-12 Thread Patrice CHOTARD
On 3/9/24 03:11, Marek Vasut wrote: > Use dev_*() only to print all the logs from this glue code, > instead of mixing dev_*(), log_*(), pr_*() all in one code. > > Signed-off-by: Marek Vasut > --- > Cc: Christophe Roullier > Cc: Joe Hershberger > Cc: Patrice Chotard > Cc: Patrick Delaunay

Re: [PATCH 06/11] net: dwc_eth_qos: Move log_debug statements on top of case block

2024-03-12 Thread Patrice CHOTARD
On 3/9/24 03:11, Marek Vasut wrote: > Move the log_debug() calls on top of the bit manipulation code. > No functional change. > > Signed-off-by: Marek Vasut > --- > Cc: Christophe Roullier > Cc: Joe Hershberger > Cc: Patrice Chotard > Cc: Patrick Delaunay > Cc: Ramon Fried > Cc:

  1   2   >