Re: [PATCH v2 2/2] pmic: pca9450: Add regulator driver

2022-05-20 Thread Stefano Babic
Hi Tim, Marek, On 20.05.22 00:24, Tim Harvey wrote: On Sun, Apr 24, 2022 at 2:41 PM Marek Vasut wrote: Add PCA9450 regulator driver. This is complementary driver for the BUCKn and LDOn regulators provided by the PCA9450 PMIC driver. Currently the driver permits reading the settngs and configu

Re: [PATCH 11/16] board: stm32pm1: add stm32mp13 board support

2022-05-20 Thread Patrice CHOTARD
Hi Patrick One typo and one remark below On 5/6/22 16:06, Patrick Delaunay wrote: > Add stm32mp15x prefix to all STM32MP15x board specific function, > this patch is a preliminary step for STM32MP13x support. > > This patch also add the RCC probe to avoid circular access with s/add/adds > usbph

Re: [PATCH 12/16] ram: stm32mp1: add support of STM32MP13x

2022-05-20 Thread Patrice CHOTARD
HI Patrick On 5/6/22 16:06, Patrick Delaunay wrote: > Add support for new compatible "st,stm32mp13-ddr" to manage the > DDR sub system (Controller and PHY) in STM32MP13x SOC: > - only one AXI port > - support of 16 port output (MEMC_DRAM_DATA_WIDTH = 2) > > The STM32MP15x SOC have 2 AXI ports and

Re: [PATCH v3 2/2] boot: build arch/*/lib/bootm.c if support boot linux

2022-05-20 Thread Bin Meng
On Fri, May 20, 2022 at 10:59 AM Rover Mo wrote: > > arch/*/lib/bootm.c is for the boot Linux commands, not just for > bootm command. > > This commit makes CONFIG_BOOTM_LINUX selected by the boot Linux > commands and arch/*/lib/bootm.c will be built if enabled > CONFIG_BOOTM_LINUX. NAK. bootm is

Re: [PATCH 13/16] mmc: stm32_sdmmc2: make reset property optional

2022-05-20 Thread Patrice CHOTARD
Hi Patrick On 5/6/22 16:06, Patrick Delaunay wrote: > Although not recommended, the reset property could be made optional. > This way the driver will probe even if no reset property is provided > in an sdmmc node in DT. This reset is already optional in Linux. > > Signed-off-by: Yann Gautier > S

Re: [PATCH 14/16] arm: dts: stm32mp: add stm32mp13 device tree for U-Boot

2022-05-20 Thread Patrice CHOTARD
HI Patrick On 5/6/22 16:06, Patrick Delaunay wrote: > Compile the device tree of STM32MP13x boards and add the needed > U-Boot add-on. > > Signed-off-by: Patrick Delaunay > --- > > arch/arm/dts/Makefile | 3 + > arch/arm/dts/stm32mp13-u-boot.dtsi | 91 ++

Re: [PATCH 15/16] configs: add stm32mp13 defconfig

2022-05-20 Thread Patrice CHOTARD
Hi Patrick On 5/6/22 16:06, Patrick Delaunay wrote: > Add a initial config for STM32M13x SOC family, using the stm32mp135f-dk > device tree. > > Signed-off-by: Patrick Delaunay > --- > > board/st/stm32mp1/MAINTAINERS | 1 + > configs/stm32mp13_defconfig | 54

Re: [PATCH v2 2/2] pmic: pca9450: Add regulator driver

2022-05-20 Thread Marek Vasut
On 5/20/22 09:00, Stefano Babic wrote: Hi Tim, Marek, Hi, It's defined differently in include/power/bd71837.h (where it should also likely be changed to something that doesn't collide) $ git grep DVS_BUCK_RUN_MASK include/ include/power/bd71837.h:#define DVS_BUCK_RUN_MASK   0x3f i

Re: [PATCH] ARM: dts: stm32: Configure Buck3 voltage per PMIC NVM on Avenger96

2022-05-20 Thread Patrick DELAUNAY
Hi, On 5/17/22 14:39, Patrick DELAUNAY wrote: Hi, On 5/11/22 23:09, Marek Vasut wrote: The Avenger96 board comes in multiple regulator configurations.   - rev.100 or rev.200 have Buck3 preconfigured to 3V3 operation on     boot and contains extra Enpirion EP53A8LQI DCDC converter which     sup

Re: [Uboot-stm32] [PATCH v1 1/2] spi: stm32_qspi: Always check SR_TCF flags in stm32_qspi_wait_cmd()

2022-05-20 Thread Patrick DELAUNAY
Hi, On 5/17/22 10:23, Patrick DELAUNAY wrote: Hi Patrice, On 5/12/22 09:17, Patrice Chotard wrote: Currently, SR_TCF flag is checked in case there is data, this criteria is not correct. SR_TCF flags is set when programmed number of bytes have been transferred to the memory device ("bytes" c

Re: [PATCH 16/16] doc: st: stm32mp1: add STM32MP13x support

2022-05-20 Thread Patrice CHOTARD
Hi PAtrick typos below On 5/6/22 16:06, Patrick Delaunay wrote: > Add in U-Boot documentation the quick instruction for s/for/to > setup the STMicroelectronics STM32MP13x boards. > > Signed-off-by: Patrick Delaunay > --- > > doc/board/st/stm32mp1.rst | 181 ++

Re: [PATCH v1 2/2] spi: stm32_qspi: Remove SR_BUSY bit check before sending command

2022-05-20 Thread Patrick DELAUNAY
Hi, On 5/17/22 10:24, Patrick DELAUNAY wrote: Hi Patrice On 5/12/22 09:17, Patrice Chotard wrote: Waiting for SR_BUSY bit when receiving a new command is not needed. SR_BUSY bit is already managed in the previous command treatment. Signed-off-by: Patrice Chotard ---   drivers/spi/stm32_qsp

Re: [Uboot-stm32] [PATCH 08/16] arm: stm32mp: add support of STM32MP13x

2022-05-20 Thread Patrice CHOTARD
another typo found below On 5/20/22 08:49, Patrice CHOTARD wrote: > Hi Patrick > > 2 minor typo below > > Thanks > Patrice > > On 5/6/22 16:06, Patrick Delaunay wrote: >> Introduce the code in mach-stm32mp and the configuration file >> stm32mp13_defconfig for the new STM32MP family. >> >> Signe

[PULL] Pull request for u-boot master / v2022.07 = u-boot-stm32-20220520

2022-05-20 Thread Patrick DELAUNAY
Hi Tom, Please pull the STM32 related fixes for u-boot/master, v2022.07: u-boot-stm32-20220520 - spi: fix busy bit check in stm32_qspi driver - stm32mp15: configure Buck3 voltage per PMIC NVM on Avenger96 board CI status: https://source.denx.de/u-boot/custodians/u-boot-stm/-/pipelines/12043

Re: [PATCH v2 2/2] pmic: pca9450: Add regulator driver

2022-05-20 Thread Stefano Babic
On 20.05.22 09:19, Marek Vasut wrote: On 5/20/22 09:00, Stefano Babic wrote: Hi Tim, Marek, Hi, It's defined differently in include/power/bd71837.h (where it should also likely be changed to something that doesn't collide) $ git grep DVS_BUCK_RUN_MASK include/ include/power/bd71837.h:#def

Re: [PATCH 4/4] ARM: imx: imx5: Introduce and use UART_BASE_ADDR(n)

2022-05-20 Thread Stefano Babic
Hi Marek, On 24.04.22 23:44, Marek Vasut wrote: Introduce helper macro UART_BASE_ADDR(n), which returns Nth UART base address. Convert all board configurations to this new macro. This is the first step toward switching CONFIG_MXC_UART_BASE to Kconfig. This is a clean up, no functional change. T

Re: [Uboot-stm32] [PATCH 08/16] arm: stm32mp: add support of STM32MP13x

2022-05-20 Thread Marek Vasut
On 5/20/22 09:24, Patrice CHOTARD wrote: [...] @@ -0,0 +1,17 @@ +/* SPDX-License-Identifier: GPL-2.0-or-later OR BSD-3-Clause */ +/* + * Copyright (C) 2022, STMicroelectronics - All Rights Reserved + * + * Configuration settings for the STMicroelectonics STM32MP15x boards s/STMicroelectonics/

Re: [PATCH 4/4] ARM: imx: imx5: Introduce and use UART_BASE_ADDR(n)

2022-05-20 Thread Marek Vasut
On 5/20/22 09:30, Stefano Babic wrote: Hi Marek, On 24.04.22 23:44, Marek Vasut wrote: Introduce helper macro UART_BASE_ADDR(n), which returns Nth UART base address. Convert all board configurations to this new macro. This is the first step toward switching CONFIG_MXC_UART_BASE to Kconfig. This

Re: [PATCH 4/4] ARM: imx: imx5: Introduce and use UART_BASE_ADDR(n)

2022-05-20 Thread Stefano Babic
On 20.05.22 09:45, Marek Vasut wrote: On 5/20/22 09:30, Stefano Babic wrote: Hi Marek, On 24.04.22 23:44, Marek Vasut wrote: Introduce helper macro UART_BASE_ADDR(n), which returns Nth UART base address. Convert all board configurations to this new macro. This is the first step toward switchin

Re: [PATCH] board: turris: Find atsha device by atsha driver

2022-05-20 Thread Pali Rohár
On Friday 20 May 2022 07:05:50 Stefan Roese wrote: > On 19.05.22 11:11, Pali Rohár wrote: > > It does not matter what is DT node name of atsha device. So find it via > > atsha driver and not by DT node name. > > > > Signed-off-by: Pali Rohár > > Just curious: What exactly does this patch fix? Is

Re: [PATCH 4/4] ARM: imx: imx5: Introduce and use UART_BASE_ADDR(n)

2022-05-20 Thread Stefano Babic
On 20.05.22 10:11, Stefano Babic wrote: On 20.05.22 09:45, Marek Vasut wrote: On 5/20/22 09:30, Stefano Babic wrote: Hi Marek, On 24.04.22 23:44, Marek Vasut wrote: Introduce helper macro UART_BASE_ADDR(n), which returns Nth UART base address. Convert all board configurations to this new macr

Re: i.MX8MP usb status

2022-05-20 Thread Tommaso Merciai
On Mon, May 16, 2022 at 12:50:34PM -0300, Fabio Estevam wrote: > Hi Marcel, > > On 16/05/2022 12:41, Marcel Ziswiler wrote: > > > Talking about uuu, has anybody managed to get that going on the i.MX > > 8M Mini yet? Regular USB device/host > > functionality works great but last I tried gadget fun

[PATCH v2 0/8] ARM: imx: Add support for iMX6QDL DHCOM DRC02 and DH picoITX

2022-05-20 Thread Philip Oberfichtner
This patch series adds support for the DHCOM DRC02 and DH picoITX baseboards by DH electronics. The two boards can be equipped with different SoMs. The STM32MP15xx based versions are already mainlined. This patch adds support for the iMX6QDL based variants. Changes in v2: - Rewrite board_fit_co

[PATCH v2 1/8] ARM: imx6: Fix broken DT path in DH board file

2022-05-20 Thread Philip Oberfichtner
In the DH electronics iMX6 board file fix the outdated eeprom path by using a DT label instead. The label has been newly created for all iMX6QDL DHCOM boards. Reviewed-by: Marek Vasut Signed-off-by: Philip Oberfichtner --- Changes in v2: - Reviewed-by Marek arch/arm/dts/imx6qdl-dhcom-u-boot

[PATCH v2 2/8] ARM: dts: imx: Migrate iMX6QDL DRC02 DTs from Linux

2022-05-20 Thread Philip Oberfichtner
Migrate DH DRC02 device trees from Linux commit 42226c989789 (tag v5.18-rc7). No changes have been made, the DTs are exact copies. Furthermore add the DTB to dh_imx6_defconfig. Reviewed-by: Marek Vasut Signed-off-by: Philip Oberfichtner --- (no changes since v1) arch/arm/dts/Makefile

[PATCH v2 3/8] ARM: dts: imx: Migrate iMX6QDL picoITX DTs from Linux

2022-05-20 Thread Philip Oberfichtner
Migrate DH picoITX device trees from Linux commit 42226c989789 (tag v5.18-rc7). No changes have been made, the DTs are exact copies. Furthermore add the DTB to dh_imx6_defconfig. Reviewed-by: Marek Vasut Signed-off-by: Philip Oberfichtner --- (no changes since v1) arch/arm/dts/Makefile

[PATCH v2 5/8] ARM: dts: imx: Simplify fec node for iMX6QDL DHCOM boards

2022-05-20 Thread Philip Oberfichtner
Firstly the FEC can now use the regulator reg_eth_vio from imx6qdl-dhcom-som.dtsi instead of defining its own. Secondly the &fec node is moved to the more generic SoM device tree file, because it can be used by multiple boards. Reviewed-by: Marek Vasut Signed-off-by: Philip Oberfichtner --- C

[PATCH v2 4/8] ARM: imx6: Remove CONFIG_FEC_MXC_PHYADDR from DH header

2022-05-20 Thread Philip Oberfichtner
Use phy address from device tree instead of CONFIG_FEC_MXC_PHYADDR from board header. This is required, because the DH picoITX and DRC02 boards require different settings than PDK2. The corresponding 'phy-handle' device tree properties are already there. I tested this change on picoITX and DRC02,

[PATCH v2 7/8] ARM: dts: imx: Configure FEC for iMX6QDL DRC02

2022-05-20 Thread Philip Oberfichtner
Add a u-boot dtsi for configuring the FEC node of the DH DRC02. Reviewed-by: Marek Vasut Signed-off-by: Philip Oberfichtner --- Changes in v2: - Reviewed-by Marek arch/arm/dts/imx6s-dhcom-drc02-u-boot.dtsi | 10 ++ 1 file changed, 10 insertions(+) create mode 100644 arch/arm/dts/imx

[PATCH v2 6/8] ARM: dts: imx: Configure FEC for iMX6QDL picoITX

2022-05-20 Thread Philip Oberfichtner
Add a u-boot dtsi for configuring the FEC node of the DH picoITX. Reviewed-by: Marek Vasut Signed-off-by: Philip Oberfichtner --- Changes in v2: - Reviewed-by Marek arch/arm/dts/imx6dl-dhcom-picoitx-u-boot.dtsi | 10 ++ 1 file changed, 10 insertions(+) create mode 100644 arch/arm/dt

[PATCH v2 8/8] ARM: imx6: Adapt device tree selection in DH board file

2022-05-20 Thread Philip Oberfichtner
Before this commit device tree selection could rely solely on differentiating the iMX6 processor variant Q and DL. After adding two new carrier boards, the DRC02 and the picoITX, the interchangeability of SoMs makes this approach infeasible. It is now required to specify the carrier board (dhcom-d

Re:Re: [PATCH v3 1/2] boot: don't enable the non-secure boot commands by default if secure boot enabled

2022-05-20 Thread Rover Mo
At 2022-05-20 14:34:18, "Heinrich Schuchardt" wrote: >Am 20. Mai 2022 04:58:46 MESZ schrieb Rover Mo : > >Having EFI_SECURE_BOOT=y is not enough to use secure boot. You must also >supply variables PK, KEK, db, dbx. > >Furthermore you would have to disable a whole lot more commands to secure the

Re: [PATCH v2] usb: dwc3: add a SPL_USB_DWC3_GENERIC option for the dwc3 driver

2022-05-20 Thread Stefano Babic
Hi Marek, don't you mind if I apply to my u-booz-imx this (that really belongs to your competence area) ? It fixes warnings for the librem5, and it is a pity if I cannot merge it. Best regards, Stefano On 24.04.22 16:08, Angus Ainslie wrote: Suppress warnings when building the SPL without U

Re: [PATCH v2] usb: dwc3: add a SPL_USB_DWC3_GENERIC option for the dwc3 driver

2022-05-20 Thread Marek Vasut
On 5/20/22 11:08, Stefano Babic wrote: Hi Marek, don't you mind if I apply to my u-booz-imx this (that really belongs to your competence area) ? It fixes warnings for the librem5, and it is a pity if I cannot merge it. Just pick it via imx, that's fine, I don't expect conflict. Reviewed-by

[PATCH v3 1/3] arm: mvebu: dts: sync DTS

2022-05-20 Thread Robert Marko
Update the uDPU DTS to the version that is pending upstream [1][2][3][4]. [1] https://patchwork.kernel.org/project/linux-arm-kernel/patch/20220516124828.45144-4-robert.ma...@sartura.hr/ [2] https://patchwork.kernel.org/project/linux-arm-kernel/patch/20220516124828.45144-5-robert.ma...@sartura.hr

[PATCH v3 3/3] MAINTAINERS: add myself as Methode maintainer

2022-05-20 Thread Robert Marko
I am currently maintaing the Methode uDPU and eDPU boards so add myself as the maintainer for them. Signed-off-by: Robert Marko --- MAINTAINERS | 8 1 file changed, 8 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 56be0bfad0..3d72b0c11f 100644 --- a/MAINTAINERS +++ b/MAINTA

[PATCH v3 2/3] arm: mvebu: add support for Methode eDPU

2022-05-20 Thread Robert Marko
Methode eDPU is an Armada 3720 power board based on the Methode uDPU. They feature the same CPU, RAM, and storage as well as the form factor. However, eDPU only has one SFP slot plus a copper G.hn port which does not work under U-boot. In order to reduce duplication, split the uDPU DTS into a co

Re: [PATCH 2/2] arm: mvebu: add support for Methode eDPU

2022-05-20 Thread Robert Marko
On Mon, May 16, 2022 at 10:54 AM Stefan Roese wrote: > > On 16.05.22 10:52, Robert Marko wrote: > > On Mon, May 16, 2022 at 8:40 AM Stefan Roese wrote: > >> > >> On 06.05.22 20:01, Robert Marko wrote: > >>> Methode eDPU is an Armada 3720 power board based on the Methode uDPU. > >>> > >>> They fea

Re: [PATCH v2 8/8] ARM: imx6: Adapt device tree selection in DH board file

2022-05-20 Thread Marek Vasut
On 5/20/22 10:46, Philip Oberfichtner wrote: Before this commit device tree selection could rely solely on differentiating the iMX6 processor variant Q and DL. After adding two new carrier boards, the DRC02 and the picoITX, the interchangeability of SoMs makes this approach infeasible. It is now

[PATCH] include/configs: Remove rootwait=1 to all the affected boards

2022-05-20 Thread Michael Trimarchi
rootwait=1 is not a valid kernel boot parameters. According to the documenation is only rootwait rootwait[KNL] Wait (indefinitely) for root device to show up. Useful for devices that are detected asynchronously (e.g. USB and MMC devices).

Re: [PATCH v3 2/3] arm: mvebu: add support for Methode eDPU

2022-05-20 Thread Pali Rohár
On Friday 20 May 2022 12:54:23 Robert Marko wrote: > Methode eDPU is an Armada 3720 power board based on the Methode uDPU. > > They feature the same CPU, RAM, and storage as well as the form factor. > > However, eDPU only has one SFP slot plus a copper G.hn port which does not > work under U-boot

Re: [PATCH v3 2/3] arm: mvebu: add support for Methode eDPU

2022-05-20 Thread Robert Marko
On Fri, May 20, 2022 at 1:01 PM Pali Rohár wrote: > > On Friday 20 May 2022 12:54:23 Robert Marko wrote: > > Methode eDPU is an Armada 3720 power board based on the Methode uDPU. > > > > They feature the same CPU, RAM, and storage as well as the form factor. > > > > However, eDPU only has one SFP

[PATCH v4 1/3] arm: mvebu: dts: sync DTS

2022-05-20 Thread Robert Marko
Update the uDPU DTS to the version that is pending upstream [1][2][3][4]. [1] https://patchwork.kernel.org/project/linux-arm-kernel/patch/20220516124828.45144-4-robert.ma...@sartura.hr/ [2] https://patchwork.kernel.org/project/linux-arm-kernel/patch/20220516124828.45144-5-robert.ma...@sartura.hr

[PATCH v4 3/3] MAINTAINERS: add myself as Methode maintainer

2022-05-20 Thread Robert Marko
I am currently maintaing the Methode uDPU and eDPU boards so add myself as the maintainer for them. Signed-off-by: Robert Marko --- MAINTAINERS | 8 1 file changed, 8 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 56be0bfad0..3d72b0c11f 100644 --- a/MAINTAINERS +++ b/MAINTA

[PATCH v4 2/3] arm: mvebu: add support for Methode eDPU

2022-05-20 Thread Robert Marko
Methode eDPU is an Armada 3720 power board based on the Methode uDPU. They feature the same CPU, RAM, and storage as well as the form factor. However, eDPU only has one SFP slot plus a copper G.hn port which does not work under U-boot. In order to reduce duplication, split the uDPU DTS into a co

Re: [PATCH] include/configs: Remove rootwait=1 to all the affected boards

2022-05-20 Thread Tommaso Merciai
On Fri, May 20, 2022 at 01:00:13PM +0200, Michael Trimarchi wrote: > rootwait=1 is not a valid kernel boot parameters. According > to the documenation is only rootwait > > rootwait [KNL] Wait (indefinitely) for root device to show up. > Useful for devices that are dete

Re: [PATCH v4 3/3] MAINTAINERS: add myself as Methode maintainer

2022-05-20 Thread Stefan Roese
On 20.05.22 13:12, Robert Marko wrote: I am currently maintaing the Methode uDPU and eDPU boards so add myself as the maintainer for them. Signed-off-by: Robert Marko --- MAINTAINERS | 8 1 file changed, 8 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 56be0bfad0..3d72b

[PATCH] common/board_r.c: drop legacy and unused bi_enetaddr

2022-05-20 Thread Rasmus Villemoes
The bi_enetaddr field in struct bd_info is write-only; nothing ever reads back the value. Moreover, the value we write is more or less random, and certainly not something one can rely on: If the board has a writable environment and the mac address has been stored there, we fetch that value. But if

Re: [PATCH v4 3/3] MAINTAINERS: add myself as Methode maintainer

2022-05-20 Thread Robert Marko
On Fri, May 20, 2022 at 1:17 PM Stefan Roese wrote: > > On 20.05.22 13:12, Robert Marko wrote: > > I am currently maintaing the Methode uDPU and eDPU boards so add myself > > as the maintainer for them. > > > > Signed-off-by: Robert Marko > > --- > > MAINTAINERS | 8 > > 1 file change

Re: [PATCH v4 3/3] MAINTAINERS: add myself as Methode maintainer

2022-05-20 Thread Stefan Roese
On 20.05.22 13:26, Robert Marko wrote: On Fri, May 20, 2022 at 1:17 PM Stefan Roese wrote: On 20.05.22 13:12, Robert Marko wrote: I am currently maintaing the Methode uDPU and eDPU boards so add myself as the maintainer for them. Signed-off-by: Robert Marko --- MAINTAINERS | 8

Re: [PATCH v2 0/8] ARM: imx: Add support for iMX6QDL DHCOM DRC02 and DH picoITX

2022-05-20 Thread Fabio Estevam
Hi Philip, On 20/05/2022 05:46, Philip Oberfichtner wrote: This patch series adds support for the DHCOM DRC02 and DH picoITX baseboards by DH electronics. The two boards can be equipped with different SoMs. The STM32MP15xx based versions are already mainlined. This patch adds support for the

[PATCH v5 1/3] arm: mvebu: dts: sync DTS

2022-05-20 Thread Robert Marko
Update the uDPU DTS to the version that is pending upstream [1][2][3][4]. [1] https://patchwork.kernel.org/project/linux-arm-kernel/patch/20220516124828.45144-4-robert.ma...@sartura.hr/ [2] https://patchwork.kernel.org/project/linux-arm-kernel/patch/20220516124828.45144-5-robert.ma...@sartura.hr

[PATCH v5 2/3] arm: mvebu: add support for Methode eDPU

2022-05-20 Thread Robert Marko
Methode eDPU is an Armada 3720 power board based on the Methode uDPU. They feature the same CPU, RAM, and storage as well as the form factor. However, eDPU only has one SFP slot plus a copper G.hn port which does not work under U-boot. In order to reduce duplication, split the uDPU DTS into a co

[PATCH v5 3/3] MAINTAINERS: add myself as Methode maintainer

2022-05-20 Thread Robert Marko
I am currently maintaing the Methode uDPU and eDPU boards so add myself as the maintainer for them. Remove the old entry from board/Marvell/mvebu_armada-37xx/MAINTAINERS. Signed-off-by: Robert Marko --- Changes in v5: * Remove entry from the board/Marvell/mvebu_armada-37xx/MAINTAINERS --- MAINT

[PATCH] riscv: cpu: set gp before board_init_f_init_reserve

2022-05-20 Thread Nikita Shubin
From: Nikita Shubin Restore global pointer before board_init_f_init_reserve call, as "a0" can be set in harts_early_init call and we end up with invalid global pointer. Signed-off-by: Nikita Shubin --- arch/riscv/cpu/start.S | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/riscv/cpu/st

Re: [PATCH v2] usb: dwc3: add a SPL_USB_DWC3_GENERIC option for the dwc3 driver

2022-05-20 Thread Stefano Babic
On 20.05.22 12:21, Marek Vasut wrote: On 5/20/22 11:08, Stefano Babic wrote: Hi Marek, don't you mind if I apply to my u-booz-imx this (that really belongs to your competence area) ? It fixes warnings for the librem5, and it is a pity if I cannot merge it. Just pick it via imx, that's fine

Re: [PATCH v2] usb: dwc3: add a SPL_USB_DWC3_GENERIC option for the dwc3 driver

2022-05-20 Thread Angus Ainslie
On 2022-05-20 05:31, Stefano Babic wrote: On 20.05.22 12:21, Marek Vasut wrote: On 5/20/22 11:08, Stefano Babic wrote: Hi Marek, don't you mind if I apply to my u-booz-imx this (that really belongs to your competence area) ? It fixes warnings for the librem5, and it is a pity if I cannot me

[PATCH V4 3/8] imx: imx8mm-icore: migrate to use BINMAN

2022-05-20 Thread Peng Fan (OSS)
From: Peng Fan Use BINMAN instead of imx specific packing method. Signed-off-by: Peng Fan --- arch/arm/mach-imx/imx8m/Kconfig | 1 + arch/arm/mach-imx/imx8m/imximage-8mm-lpddr4.cfg | 10 +- configs/imx8mm-icore-mx8mm-ctouch2_defconfig| 2 +- configs/imx8mm-icore-m

[PATCH V4 1/8] spl: guard u_boot_any with X86

2022-05-20 Thread Peng Fan (OSS)
From: Peng Fan set the symbol as weak not work if LTO is enabled. Since u_boot_any is only used on X86 for now, so guard it with X86, otherwise build break if we use BINMAN_SYMBOLS on i.MX. Tested-by: Tim Harvey #imx8m[m,n,p]-venice Signed-off-by: Peng Fan --- common/spl/spl.c | 8 ++-

[PATCH V4 4/8] armv8: u-boot-spl.lds: mark __image_copy_start as symbol

2022-05-20 Thread Peng Fan (OSS)
From: Peng Fan In arch/arm/lib/sections.c there is below code: char __image_copy_start[0] __section(".__image_copy_start"); But actually 'objdump -t spl/u-boot-spl' not able to find out symbol '__image_copy_start' for binman update image-pos/size. So update link file Tested-by: Tim Harvey #imx

[PATCH V4 5/8] tools: binman: section: replace @ with -

2022-05-20 Thread Peng Fan (OSS)
From: Peng Fan In arch/arm/dts/imx8mp-u-boot.dtsi, there are blob-ext@1, blob-ext@2 and etc which is for packing ddr phy firmware. However we could not declare symbol name such as 'binman_sym_declare(ulong, blob_ext@1, image_pos)', because '@' is not allowed, so we choose to declare the symbol 'b

[PATCH V4 0/8] arm64: binman: use binman symbols for imx

2022-05-20 Thread Peng Fan (OSS)
From: Peng Fan V4: Fix three boards build failure V3: Add R-b/T-b Fix build warning V2: resolve some CI failure include patch 7 binman symbol is a good feature, but only used on X86 for now. This patchset is to use it for i.MX8M platform. The current imx8m ddr phy firmware consumes lots

[PATCH V4 6/8] ddr: imx8m: helper: load ddr firmware according to binman symbols

2022-05-20 Thread Peng Fan (OSS)
From: Peng Fan By reading binman symbols, we no need hard coded IMEM_LEN/DMEM_LEN after we update the binman dtsi to drop 0x8000/0x4000 length for the firmware. And that could save binary size for many KBs. Tested-by: Tim Harvey #imx8m[m,n,p]-venice Signed-off-by: Peng Fan --- drivers/ddr/im

[PATCH V4 2/8] arm: dts: imx8m: update binman ddr firmware node name

2022-05-20 Thread Peng Fan (OSS)
From: Peng Fan We are migrating to use BINMAN SYMBOLS, the current name is not a valid binman type, so update to use blob-ext@[1,2,3,4]. Tested-by: Tim Harvey #imx8m[m,n,p]-venice Signed-off-by: Peng Fan --- arch/arm/dts/imx8mm-u-boot.dtsi | 8 arch/arm/dts/imx8mn-b

[PATCH V4 7/8] arm: dts: imx8m: shrink ddr firmware size to actual file size

2022-05-20 Thread Peng Fan (OSS)
From: Peng Fan After we switch to use BINMAN_SYMBOLS, there is no need to pad the file size to 0x8000 and 0x4000. After we use BINMAN_SYMBOLS, the u-boot-spl-ddr.bin shrink about 36KB with i.MX8MP-EVK. Tested-by: Tim Harvey #imx8m[m,n,p]-venice Signed-off-by: Peng Fan --- arch/arm/dts/imx8mm-

[PATCH V4 8/8] binman_sym: guard with CONFIG_SPL_BINMAN_SYMBOLS

2022-05-20 Thread Peng Fan (OSS)
From: Peng Fan There is case that CONFIG_BINMAN is defined, but CONFIG_SPL_BINMAN_SYMBOLS is not defined. In that case, there will be build failure. So use CONFIG_SPL_BINMAN_SYMBOLS to guard the macros, and define CONFIG_SPL_BINMAN_SYMBOLS in binman syms test. Tested-by: Tim Harvey #imx8m[m,n,p

[PATCH 3/4] ARM: imx: imx31: Introduce and use UART_BASE_ADDR(n)

2022-05-20 Thread sbabic
> Introduce helper macro UART_BASE_ADDR(n), which returns Nth UART base > address. Convert all board configurations to this new macro. This is the > first step toward switching CONFIG_MXC_UART_BASE to Kconfig. This is a > clean up, no functional change. > The new macro contains compile-time test to

[PATCH] board: gateworks: gw_ventana: add support for GPY111 PHY

2022-05-20 Thread sbabic
> The MaxLinear GPY111 PHY is being used on some boards due to part > availability. Add support for this PHY which requires a longer reset > post-delay and RGMII delay configuration. > Signed-off-by: Tim Harvey Applied to u-boot-imx, master, thanks ! Best regards, Stefano Babic -- =

[PATCH V2 01/16] imx: imx8mn_evk: enable pinctrl_wdog in SPL

2022-05-20 Thread sbabic
> From: Peng Fan > Mark pinctrl_wdog as u-boot,dm-spl to clean up board code, > The set_wdog_reset() function is not necessary as this is handled by > the imx_watchdog.c driver due to the 'fsl,ext-reset-output' property > being set. > Signed-off-by: Peng Fan Applied to u-boot-imx, master, thanks

[PATCH v6] bosch: Add initial board support for ACC

2022-05-20 Thread sbabic
> The Bosch ACC (Air Center Control) Board is based on the i.MX6D. > The device tree is copied from Linux, see [1]. The only difference > compared to the Linux DT is the removal of usbphynop properties. They are > defined in the Linux version of imx6qdl.dtsi, but not in the u-boot > version. > [1]

[PATCH 2/3] cgtqmx8: Enable cache in SPL

2022-05-20 Thread sbabic
> From: Fabio Estevam > There is no reason for disabling I-cache and D-cache > in SPL. > > Remove the unneeded CONFIG_SPL_SYS_ICACHE_OFF and > CONFIG_SPL_SYS_DCACHE_OFF options. > Signed-off-by: Fabio Estevam Applied to u-boot-imx, master, thanks ! Best regards, Stefano Babic -- =

[PATCH v2 4/8] ARM: imx6: Remove CONFIG_FEC_MXC_PHYADDR from DH header

2022-05-20 Thread sbabic
> Use phy address from device tree instead of CONFIG_FEC_MXC_PHYADDR from > board header. This is required, because the DH picoITX and DRC02 boards > require different settings than PDK2. The corresponding 'phy-handle' > device tree properties are already there. > I tested this change on picoITX an

[PATCH V2 02/16] imx: imx8mm_evk: enable pinctrl_wdog in SPL

2022-05-20 Thread sbabic
> From: Peng Fan > Mark pinctrl_wdog as u-boot,dm-spl to clean up board code, > The set_wdog_reset() function is not necessary as this is handled by > the imx_watchdog.c driver due to the 'fsl,ext-reset-output' property > being set. > Signed-off-by: Peng Fan Applied to u-boot-imx, master, thanks

[PATCH v2 6/8] ARM: dts: imx: Configure FEC for iMX6QDL picoITX

2022-05-20 Thread sbabic
> Add a u-boot dtsi for configuring the FEC node of the DH picoITX. > Reviewed-by: Marek Vasut > Signed-off-by: Philip Oberfichtner Applied to u-boot-imx, master, thanks ! Best regards, Stefano Babic -- = DENX Software Enginee

[PATCH] i.MX8ULP: add display_ele_fw_version api

2022-05-20 Thread sbabic
> implement get f/w version api. > print ele f/w version in spl. > Signed-off-by: Gaurav Jain > Reviewed-by: Peng Fan > Reviewed-by: Pankaj Gupta Applied to u-boot-imx, master, thanks ! Best regards, Stefano Babic -- = DENX S

[PATCH V2 16/16] imx: toradex/verdin-imx8mm/p: cleanup board watchdog code

2022-05-20 Thread sbabic
> From: Peng Fan > pinctrl_wdog already marked u-boot,dm-spl, so clean up board code. > The set_wdog_reset() function is not necessary as this is handled by > the imx_watchdog.c driver due to the 'fsl,ext-reset-output' property > being set. > Signed-off-by: Peng Fan Applied to u-boot-imx, master,

[PATCH] imx8m: fix reading of DDR4 MR registers

2022-05-20 Thread sbabic
> I was trying to employ lpddr4_mr_read() to something similar to what > the imx8mm-cl-iot-gate board is doing for auto-detecting the RAM > type. However, the version in drivers/ddr/imx/imx8m/ddrphy_utils.c > differs from the private one used by that board in how it extracts the > byte value, and I

[PATCH v3 1/2] imx8mn_ddr4_evk: Add USB Mass Storage support

2022-05-20 Thread sbabic
> From: Fabio Estevam > Add USB Mass Storage support, which is a convenient way to flash > the eMMC card, for example. > Signed-off-by: Fabio Estevam Applied to u-boot-imx, master, thanks ! Best regards, Stefano Babic -- = DEN

[PATCH v2 1/8] ARM: imx6: Fix broken DT path in DH board file

2022-05-20 Thread sbabic
> In the DH electronics iMX6 board file fix the outdated eeprom path by > using a DT label instead. > The label has been newly created for all iMX6QDL DHCOM boards. > Reviewed-by: Marek Vasut > Signed-off-by: Philip Oberfichtner Applied to u-boot-imx, master, thanks ! Best regards, Stefano Babic

[PATCH V2 13/14] imx: imx8mq-pico: enable CONFIG_DM_SERIAL

2022-05-20 Thread sbabic
> From: Peng Fan > Marked related nodes as u-boot,dm-spl for serial driver model > Enable CONFIG_DM_SERIAL > Signed-off-by: Peng Fan > Reviewed-by: Fabio Estevam Applied to u-boot-imx, master, thanks ! Best regards, Stefano Babic --

[PATCH] board: gateworks: gw_ventana: remove obsolete file

2022-05-20 Thread sbabic
> commit 61cf22505339 ("board: gateworks: gw_ventana: use comomn GSC driver") > moved to the common GSC driver and moved remaining board-specific > functions to eeprom.c. The functions in gsc.c are no longer used and it > was removed from the Makefile but the file itself was not removed. > Remove i

[PATCH 2/2] mx6sllevk: Remove duplicated "mmc dev" command

2022-05-20 Thread sbabic
> From: Fabio Estevam > The "mmc dev ${mmcdev}" command is done twice. > Remove one ocurrence to avoid the duplication. > Signed-off-by: Fabio Estevam Applied to u-boot-imx, master, thanks ! Best regards, Stefano Babic -- = DE

[PATCH 2/4] ARM: imx: imx27: Introduce and use UART_BASE_ADDR(n)

2022-05-20 Thread sbabic
> Introduce helper macro UART_BASE_ADDR(n), which returns Nth UART base > address. Convert all board configurations to this new macro. This is the > first step toward switching CONFIG_MXC_UART_BASE to Kconfig. This is a > clean up, no functional change. > The new macro contains compile-time test to

[PATCH] ARM: dts: imx: Use 100 kHz I2C2 on Data Modul i.MX8M Mini eDM SBC

2022-05-20 Thread sbabic
> The I2C2 has SMBus device SMSC USB2514Bi connected to it, the device is > capable of up to 100 kHz operation. Reduce the bus frequency to 100 kHz > to guarantee this I2C device can work correctly. > Signed-off-by: Marek Vasut > Cc: Fabio Estevam > Cc: Peng Fan > Cc: Stefano Babic Applied to u

[PATCH V2 10/16] imx: imx8mn_smm_s2: clean up board watchdog code

2022-05-20 Thread sbabic
> From: Peng Fan > pinctrl_wdog already marked u-boot,dm-spl, so clean up board code. > The set_wdog_reset() function is not necessary as this is handled by > the imx_watchdog.c driver due to the 'fsl,ext-reset-output' property > being set. > Signed-off-by: Peng Fan > Tested-by: Ariel D'Alessandr

[PATCH] configs: imx8mp_venice: add FEC_QUIRK_ENET_MAC

2022-05-20 Thread sbabic
> The IMX8MP SoC FEC needs to have the FEC_QUIRK_ENET_MAC defined. > Fixes: commit 2395625209cc ("board: gateworks: venice: add > imx8mp-venice-gw740x support") > Signed-off-by: Tim Harvey Applied to u-boot-imx, master, thanks ! Best regards, Stefano Babic -- ==

[PATCH V2 09/14] imx: imx8mm_edm_sbc: Enable SPL_DM_SERIAL

2022-05-20 Thread sbabic
> From: Peng Fan > Enable CONFIG_SPL_DM_SERIAL. uart and its pinmux was already > marked with u-boot,dm-spl. > Move preloader_console_init after spl_early_init to make sure driver > model work. > Signed-off-by: Peng Fan > Reviewed-by: Fabio Estevam Applied to u-boot-imx, master, thanks ! Best r

[PATCH 3/4] imx: bootaux: get stack from elf file

2022-05-20 Thread sbabic
> From: Peng Fan > To i.MX8, M core stack is pre-coded in source code, so need to get it > before kicking M core. The stack pointer is stored in the first word of > the first PT_LOAD section __isr_vector. So use a num to index the > section loading. > Signed-off-by: Peng Fan Applied to u-boot-imx

[PATCH v2 7/8] ARM: dts: imx: Configure FEC for iMX6QDL DRC02

2022-05-20 Thread sbabic
> Add a u-boot dtsi for configuring the FEC node of the DH DRC02. > Reviewed-by: Marek Vasut > Signed-off-by: Philip Oberfichtner Applied to u-boot-imx, master, thanks ! Best regards, Stefano Babic -- = DENX Software Engineeri

[PATCH 4/4] imx: imx8m: add rproc_att

2022-05-20 Thread sbabic
> From: Peng Fan > With rpoc_att, bootaux able to kick elf file for M core > Signed-off-by: Peng Fan Applied to u-boot-imx, master, thanks ! Best regards, Stefano Babic -- = DENX Software Engineering GmbH, Managing Direct

[PATCH] board: gateworks: venice: enable SPL_DM_SERIAL

2022-05-20 Thread sbabic
> The uart2 and its pinmux are already marked with u-boot,dm-spl but we > need to move the call to preloader_console_init() after spl_early_init() > to avoid a board hang as dm can't be used until after spl_early_init() > due to the uart driver not enabling the uart clock. > Remove the manual confi

[PATCH v1] ARM: imx8mm: verdin-imx8mm: fix board hang in spl

2022-05-20 Thread sbabic
> From: Marcel Ziswiler > Move the preloader_console_init() call after spl_early_init() to avoid > board hang in SPL. > While at it remove explicit in-code console/debug UART pinmuxing (uart1 > and its pinmuxing are already marked as u-boot,dm-spl via device tree). > Fixes: 4551e1898769 ("configs:

[PATCH V2 03/16] imx: imx8mp_evk: enable pinctrl_wdog in SPL

2022-05-20 Thread sbabic
> From: Peng Fan > Mark pinctrl_wdog as u-boot,dm-spl to clean up board code, > The set_wdog_reset() function is not necessary as this is handled by > the imx_watchdog.c driver due to the 'fsl,ext-reset-output' property > being set. > Signed-off-by: Peng Fan Applied to u-boot-imx, master, thanks

[PATCH V4 3/5] arm: mach-imx: cmd_nandbcb fix bad block handling

2022-05-20 Thread sbabic
> The badblock should be skipped properly in reading and writing. > Fix the logic. The bcb struct is written, skipping the bad block, > so we need to read using the same logic. This was tested create > bad block in the area and then flash it and read it back. > Acked-by: Han Xu > Tested-By: Tim Ha

[PATCH 1/4] ARM: imx: imx8m: Introduce and use UART_BASE_ADDR(n)

2022-05-20 Thread sbabic
> Introduce helper macro UART_BASE_ADDR(n), which returns Nth UART base > address. Convert all board configurations to this new macro. This is the > first step toward switching CONFIG_MXC_UART_BASE to Kconfig. This is a > clean up, no functional change. > The new macro contains compile-time test to

[PATCH 1/2] mx6slevk: Remove duplicated "mmc dev" command

2022-05-20 Thread sbabic
> From: Fabio Estevam > The "mmc dev ${mmcdev}" command is done twice. > Remove one ocurrence to avoid the duplication. > Signed-off-by: Fabio Estevam Applied to u-boot-imx, master, thanks ! Best regards, Stefano Babic -- = DE

[PATCH V2 03/14] imx: imx8mm_mx8menlo: Enable DM_SERIAL

2022-05-20 Thread sbabic
> From: Peng Fan > Enable CONFIG_DM_SERIAL. uart2 and its pinmux was already > marked with u-boot,dm-spl. > Signed-off-by: Peng Fan > Reviewed-by: Fabio Estevam Applied to u-boot-imx, master, thanks ! Best regards, Stefano Babic -- =

[PATCH V4 1/5] nand: raw: mxs_nand: Fix specific hook registration

2022-05-20 Thread sbabic
> Move the hook after nand_scan_tail is called. The hook must be replaced > to the mxs specific one but those must to be assignment later in the > probe function. > With this fix markbad is working again. Before this change: > nand markbad 0xDEC00 > NXS NAND: Writing OOB isn't supported > NXS NAND:

[PATCH V2 01/14] imx: imx8mp_rsb3720a1: convert to DM_SERIAL

2022-05-20 Thread sbabic
> From: Peng Fan > Enable CONFIG_DM_SERIAL. uart2 and its pinmux was already > marked with u-boot,dm-spl. > Signed-off-by: Peng Fan > Reviewed-by: Fabio Estevam Applied to u-boot-imx, master, thanks ! Best regards, Stefano Babic -- =

[PATCH V2 15/16] imx: verdin-imx8mm/p: cleanup board watchdog code

2022-05-20 Thread sbabic
> From: Peng Fan > pinctrl_wdog already marked u-boot,dm-spl, so clean up board code. > The set_wdog_reset() function is not necessary as this is handled by > the imx_watchdog.c driver due to the 'fsl,ext-reset-output' property > being set. > Signed-off-by: Peng Fan Applied to u-boot-imx, master,

  1   2   >