Re: [PATCH] Revert "tree: imx: remove old fit generator script"

2022-01-07 Thread Marcel Ziswiler
gt; > > > > u-boot.its:84.12-89.5: Warning (unit_address_vs_reg): > > > > > > > > > /configurations/config@5: node has a unit name, but no reg > > > > > > > > > property > > > > > > > > > ./tools/

Re: mkimage_fit_atf.sh: not found

2022-01-05 Thread Marcel Ziswiler
On Wed, 2022-01-05 at 17:04 -0500, Tom Rini wrote: > On Wed, Jan 05, 2022 at 10:51:23PM +0100, Marcel Ziswiler wrote: > > Hi Tim et al. > > > > On Wed, 2022-01-05 at 11:08 -0800, Tim Harvey wrote: > > > On Wed, Jan 5, 2022 at 3:34 AM ZHIZHIKIN Andrey > >

Re: mkimage_fit_atf.sh: not found

2022-01-05 Thread Marcel Ziswiler
Tuesday, January 4, 2022 11:48 PM > > > To: u-boot ; Stefano Babic ; Fabio > > > Estevam > > > > > > Cc: Schrempf Frieder ; Adam Ford > > > ; Marcel Ziswiler ; Jagan Teki > > > > > > Subject: mkimage_fit_atf.sh: not found > > >

DeprecationWarning: distutils package and distutils.sysconfig module deprecated

2021-11-10 Thread Marcel Ziswiler
distutils.sysconfig import get_python_lib tools/binman/binman:12: DeprecationWarning: The distutils.sysconfig module is deprecated, use sysconfig instead from distutils.sysconfig import get_python_lib Cheers Marcel

Re: [PATCH] imx8mn_var_som: Add support for Variscite VAR-SOM-MX8M-NANO board

2021-11-03 Thread Marcel Ziswiler
Hi Ariel On Tue, 2021-11-02 at 20:18 -0300, Ariel D'Alessandro wrote: > Add support for iMX8MN VAR-SOM-MX8M-NANO board. Enables support for: > > - 1GiB DDR4 RAM > - 16 GiB eMMC > - SD card > - Gigabit ethernet > - USBOTG1 peripheral - fastboot > > Signed-off-by: Ariel D'Alessandro > --- >  

Re: [PATCH v3 0/7] arm64: dts: imx8mm: use common binman configuration and further clean-up

2021-10-22 Thread Marcel Ziswiler
On Sat, 2021-10-23 at 01:15 +0200, Marcel Ziswiler wrote: > From: Marcel Ziswiler > > > With the move to using binman to generate SPL aka u-boot-spl-ddr.bin and > U-Boot proper aka u-boot.itb every board now covers such configuration > in its own U-Boot specific device

Re: [PATCH v4 0/5] board: sl28: target refresh

2021-10-22 Thread Marcel Ziswiler
On Sat, 2021-10-23 at 00:19 +0200, Marcel Ziswiler wrote: > From: Marcel Ziswiler > > > Note that this re-base is only compile tested. > > Changes in v4: > - Re-based on top of imx/master. > > Changes in v3: > - none >  - use CONFIG

Re: [PATCH v4 0/5] board: sl28: target refresh

2021-10-22 Thread Marcel Ziswiler
Hi Michael On Sat, 2021-10-23 at 01:28 +0200, Michael Walle wrote: > Hi Marcel, > > Am 2021-10-23 01:09, schrieb Marcel Ziswiler: > > Sorry, I did not want to upset anybody. I probably should have asked > > first before doing this. > > You could have said th

[PATCH v3 7/7] arm64: dts: imx8mm-u-boot.dtsi: improve odd blob-ext naming

2021-10-22 Thread Marcel Ziswiler
From: Marcel Ziswiler Rather than using odd implicit blob-ext naming, explicitly specify the type to be of blob-ext and therefore also simplify the node naming. Signed-off-by: Marcel Ziswiler Reviewed-by: Peng Fan --- (no changes since v2) Changes in v2: - Rather than doing that clean-up

[PATCH v3 4/7] arm64: dts: imx8mm: use common binman configuration

2021-10-22 Thread Marcel Ziswiler
From: Marcel Ziswiler With the move to using binman to generate SPL aka u-boot-spl-ddr.bin and U-Boot proper aka u-boot.itb every board now covers such configuration in its own U-Boot specific device tree include. Move the comon part of that configuration to the common imx8mm-u-boot.dtsi include

[PATCH v3 6/7] arm64: dts: imx8mm-u-boot.dtsi: explicitly add spl filename

2021-10-22 Thread Marcel Ziswiler
From: Marcel Ziswiler Explicitly add SPL aka u-boot-spl.bin filename. Signed-off-by: Marcel Ziswiler Reviewed-by: Peng Fan --- (no changes since v2) Changes in v2: - Rather than doing that clean-up to the separate binman dtsi now do it to the common u-boot dtsi one. arch/arm/dts/imx8mm

[PATCH v3 5/7] arm64: dts: imx8mm-u-boot.dtsi: alphabetically re-order properties

2021-10-22 Thread Marcel Ziswiler
From: Marcel Ziswiler Alphabetically re-order properties. Signed-off-by: Marcel Ziswiler Reviewed-by: Peng Fan --- (no changes since v2) Changes in v2: - Rather than doing that clean-up to the separate binman dtsi now do it to the common u-boot dtsi one. arch/arm/dts/imx8mm-u-boot.dtsi

[PATCH v3 3/7] arm64: dts: imx8mm-cl-iot-gate-u-boot.dtsi: use common imx8mm-u-boot.dtsi

2021-10-22 Thread Marcel Ziswiler
From: Marcel Ziswiler Use common imx8mm-u-boot.dtsi. Signed-off-by: Marcel Ziswiler Reviewed-by: Peng Fan --- (no changes since v2) Changes in v2: - New patch preparing cl-iot-gate. .../dts/imx8mm-cl-iot-gate-optee-u-boot.dtsi | 37 +-- arch/arm/dts/imx8mm-cl-iot-gate-u

[PATCH v3 2/7] arm64: dts: imx8mm-cl-iot-gate-u-boot.dtsi: alphabetically re-order

2021-10-22 Thread Marcel Ziswiler
From: Marcel Ziswiler Alphabetically re-order nodes and properties. Signed-off-by: Marcel Ziswiler Reviewed-by: Peng Fan --- (no changes since v2) Changes in v2: - New patch preparing cl-iot-gate. .../dts/imx8mm-cl-iot-gate-optee-u-boot.dtsi | 202 +- arch/arm/dts/imx8mm

[PATCH v3 0/7] arm64: dts: imx8mm: use common binman configuration and further clean-up

2021-10-22 Thread Marcel Ziswiler
From: Marcel Ziswiler With the move to using binman to generate SPL aka u-boot-spl-ddr.bin and U-Boot proper aka u-boot.itb every board now covers such configuration in its own U-Boot specific device tree include. Move the comon part of that configuration to the common imx8mm-u-boot.dtsi

[PATCH v3 1/7] imx8mm-cl-iot-gate: fix imximage intermediate binary naming

2021-10-22 Thread Marcel Ziswiler
From: Marcel Ziswiler This fixes the following build time issue: ... BINMAN all binman: Error 1 running 'mkimage -d ./mkimage.spl.mkimage -n spl/u-boot-spl.cfgout -T imx8mimage -e 0x7e1000 ./mkimage-out.spl.mkimage': mkimage.flash.mkimage: Can't open: No such file or directory make

Re: [PATCH v4 0/5] board: sl28: target refresh

2021-10-22 Thread Marcel Ziswiler
Hi Michael Sorry, I did not want to upset anybody. I probably should have asked first before doing this. On Sat, 2021-10-23 at 00:52 +0200, Michael Walle wrote: > Hi Marcel, > > Am 2021-10-23 00:19, schrieb Marcel Ziswiler: > > From: Marcel Ziswiler > > > >

[PATCH v4 1/5] board: sl28: enable EFI_SET_TIME support

2021-10-22 Thread Marcel Ziswiler
From: Michael Walle Allow EFI to actually set the time before ExitBootServices(). Signed-off-by: Michael Walle Signed-off-by: Marcel Ziswiler --- Changes in v4: - Re-based on top of imx/master. Changes in v3: - none Changes in v2: - none configs/kontron_sl28_defconfig | 1 + 1 file

[PATCH v4 0/5] board: sl28: target refresh

2021-10-22 Thread Marcel Ziswiler
From: Marcel Ziswiler Note that this re-base is only compile tested. Changes in v4: - Re-based on top of imx/master. Changes in v3: - none - use CONFIG_EFI_CAPSULE_FIRMWARE_FIT instead of CONFIG_EFI_CAPSULE_FIRMWARE_RAW Changes in v2: - none - new patch Frieder Schrempf (1): doc

[PATCH v4 3/5] board: sl28: enable EFI UpdateCapsule support

2021-10-22 Thread Marcel Ziswiler
From: Michael Walle Enable support for update over EFI UpdateCapsule mechanism. This board doesn't support setting EFI variables after ExitBootservices(). Therefore, we are also enabling EFI_IGNORE_OSINDICATIONS. Signed-off-by: Michael Walle Signed-off-by: Marcel Ziswiler --- Changes in v4

[PATCH v4 4/5] doc: board: kontron: sl28: Reduce section levels and change title

2021-10-22 Thread Marcel Ziswiler
From: Frieder Schrempf In order to add other Kontron boards to the docs alongside the existing sl28 board, we need to reduce the levels of the sections and change the title. Cc: Fabio Estevam Signed-off-by: Frieder Schrempf Acked-by: Michael Walle Signed-off-by: Marcel Ziswiler --- >F

[PATCH v4 5/5] board: sl28: add update image documentation

2021-10-22 Thread Marcel Ziswiler
From: Michael Walle Document the update image and how to use the EFI UpdateCapsule. Signed-off-by: Michael Walle Signed-off-by: Marcel Ziswiler --- Changes in v4: - Re-based on top of imx/master. Changes in v3: - none Changes in v2: - new patch doc/board/kontron/sl28.rst | 15

[PATCH v4 2/5] board: sl28: generate FIT update image

2021-10-22 Thread Marcel Ziswiler
Signed-off-by: Marcel Ziswiler --- Changes in v4: - Re-based on top of imx/master. Changes in v3: - none Changes in v2: - new patch .../dts/fsl-ls1028a-kontron-sl28-u-boot.dtsi | 35 +-- 1 file changed, 33 insertions(+), 2 deletions(-) diff --git a/arch/arm/dts/fsl-ls1028a-kont

[PATCH v2] imx8mm_beacon: Switch to binman for generating image

2021-10-22 Thread Marcel Ziswiler
From: Adam Ford To eliminate a warning when using custom imx tools for generating a binary, use binman to generate flash.bin. Signed-off-by: Adam Ford Signed-off-by: Marcel Ziswiler --- Note that this re-base is only compile tested. Changes in v2: - Re-based on top of imx/master. arch/arm

[PATCH v2] board: toradex: add verdin imx8m plus support

2021-10-22 Thread Marcel Ziswiler
From: Marcel Ziswiler This adds initial support for the Toradex Verdin iMX8M Plus Quad 4GB WB IT V1.0B module. They are strapped to boot from eFuses which are factory fused to properly boot from their on-module eMMC. U-Boot supports booting from the on-module eMMC only, SDP support is disabled

Re: i.MX LPDDR4 Firmware

2021-10-20 Thread Marcel Ziswiler
On Tue, 2021-10-19 at 15:04 +0200, Heiko Thiery wrote: >  Hi Marcel, > > Am Di., 19. Okt. 2021 um 11:47 Uhr schrieb Marcel Ziswiler > : > > > > Hi Heiko > > > > On Mon, 2021-10-18 at 11:23 +0200, Heiko Thiery wrote: > > > Hi all, > > > >

Re: i.MX LPDDR4 Firmware

2021-10-19 Thread Marcel Ziswiler
6.bin > lpddr4_pmu_train_1d_imem.bin > lpddr4_pmu_train_2d_dmem_201904.bin > lpddr4_pmu_train_2d_dmem_202006.bin > lpddr4_pmu_train_2d_dmem.bin > lpddr4_pmu_train_2d_imem_201904.bin > lpddr4_pmu_train_2d_imem_202006.bin > lpddr4_pmu_train_2d_imem.bin [1] https://www.nxp.com/docs/en/user-guide/IMX_LINUX_USERS_GUIDE.pdf > Thanks Cheers Marcel

Re: [PATCH v6 00/11] board: toradex: verdin-imx8mm: target refresh

2021-10-12 Thread Marcel Ziswiler
Hi Tim On Tue, 2021-10-12 at 12:46 -0700, Tim Harvey wrote: > On Sat, Oct 9, 2021 at 1:43 PM Marcel Ziswiler wrote: > > > > From: Marcel Ziswiler > > > > > > An assortment of fixes and improvements like an Ethernet PHY > > configuration fix, DEK bl

Re: IMX8M OP-TEE

2021-10-12 Thread Marcel Ziswiler
Hi Tim On Mon, 2021-10-11 at 16:32 -0700, Tim Harvey wrote: > > ... > Marcel, > > Thanks for checking into OP-TEE for me. Sure thing. I meanwhile got reminded that Sergio Prado actually held a talk about this topic back at the Embedded World [1]. Not sure whether he played

[PATCH v1] board: toradex: add verdin imx8m plus support

2021-10-12 Thread Marcel Ziswiler
From: Marcel Ziswiler This adds initial support for the Toradex Verdin iMX8M Plus Quad 4GB WB IT V1.0B module. They are strapped to boot from eFuses which are factory fused to properly boot from their on-module eMMC. U-Boot supports booting from the on-module eMMC only, SDP support is disabled

Re: IMX8M OP-TEE

2021-10-11 Thread Marcel Ziswiler
ZE - 32MIB but I'm not entirely sure. > > I'm currently using NXP's ATF (imx_5.4.3_2.0.0) and NXP's TEE > (imx_5.4.70_2.3.0) and would also like to understand if NXP's branches > are strictly required here and if so what the pros and cons of using > them are. > > Anyone using IMX8MM OP-TEE that could point me in the right direction? > > Best regards, > > Tim BTW: Have you tried my latest stuff [1] on venice? Did/does it work? And yeah, my cover letter subject is slightly misleading as I forgot to update that one (;-p). [1] https://marc.info/?l=u-boot=163372696806292 Cheers Marcel

Re: [PATCH 1/7] arm: mach-imx: Update MAC fuse for i.MX8MP

2021-10-10 Thread Marcel Ziswiler
On Mon, 2021-08-16 at 18:44 +0800, Ye Li wrote: > i.MX8MP has two ENET controllers, have to update the function to > enable loading two MAC addresses. > > Signed-off-by: Ye Li Whole series. Tested on iMX8MP_EVK and Verdin iMX8M Plus on Verdin development board. Tested-by: Marcel

[PATCH v6 00/11] board: toradex: verdin-imx8mm: target refresh

2021-10-09 Thread Marcel Ziswiler
From: Marcel Ziswiler An assortment of fixes and improvements like an Ethernet PHY configuration fix, DEK blob encapsulation preparation, migration to using binman to pack images, SLEEP_MOCI# enablement, dropping of V1.0 hardware support [1], renaming kernel image variable, using preboot

[PATCH v6 09/11] include/configs: apalis-imx8/verdin-imx8mm: rename kernel image variable

2021-10-09 Thread Marcel Ziswiler
an hard-coding fdtfile default fdt_board to dev for the Verdin iMX8M Mini and fix its closing #endif comment. Signed-off-by: Oleksandr Suvorov Signed-off-by: Marcel Ziswiler Reviewed-by: Fabio Estevam --- Changes in v6: - Update commit message as requested by Wolfgang. Changes in v3: - F

[PATCH v6 05/11] verdin-imx8mm: switch to use binman to pack images

2021-10-09 Thread Marcel Ziswiler
From: Marcel Ziswiler Use binman to pack images. Signed-off-by: Marcel Ziswiler Reviewed-by: Heiko Thiery Reviewed-by: Fabio Estevam Reviewed-by: Heiko Schocher --- (no changes since v4) Changes in v4: - Add Heiko Schocher's reviewed-by tag. - Fix copyright periods. Changes in v3

[PATCH v6 10/11] verdin-imx8mm: use preboot for fdtfile evaluation

2021-10-09 Thread Marcel Ziswiler
mode without manual setting fdtfile value, as it it's already evaluated before entering interactive mode. Signed-off-by: Igor Opaniuk Signed-off-by: Marcel Ziswiler Reviewed-by: Fabio Estevam --- (no changes since v2) Changes in v2: - New patch addressing dynamic fdtfile definition. configs

[PATCH v6 06/11] verdin-imx8mm: enable sleep_moci output

2021-10-09 Thread Marcel Ziswiler
From: Max Krummenacher This powers some peripherals on the carrier board e.g. the USB hub. Related-to: ELB-3206 Signed-off-by: Max Krummenacher Signed-off-by: Marcel Ziswiler Reviewed-by: Fabio Estevam --- (no changes since v1) arch/arm/dts/imx8mm-verdin.dts | 18

[PATCH v6 11/11] verdin-imx8mm: fix watchdog pinctrl issue

2021-10-09 Thread Marcel Ziswiler
From: Marcel Ziswiler Finally, found the root cause of the issue already once mentioned back here [2] which caused the following error message during boot: imx_wdt watchdog@3028: pinctrl_select_state_full: uclass_get_device_by_phandle_id: err=-19 Turns out while the watchdog node itself

[PATCH v6 08/11] verdin-imx8mm: drop support for v1.0 hardware

2021-10-09 Thread Marcel Ziswiler
From: Max Krummenacher We drop support for Verdin iMX8M Mini V1.0B. Related-to: ELB-3551 Signed-off-by: Max Krummenacher Signed-off-by: Marcel Ziswiler Reviewed-by: Fabio Estevam --- (no changes since v5) Changes in v5: - Re-based on top of u-boot-imx, master yet again. Changes in v4

[PATCH v6 07/11] verdin-imx8mm: clean-up include order

2021-10-09 Thread Marcel Ziswiler
From: Marcel Ziswiler Alphabetically order includes. While at it also update copyright year resp. period. Signed-off-by: Marcel Ziswiler --- (no changes since v3) Changes in v3: - New patch cleaning up include order. board/toradex/verdin-imx8mm/verdin-imx8mm.c | 4 ++-- 1 file changed, 2

[PATCH v6 04/11] arm64: dts: imx8mm-verdin-u-boot.dtsi: alphabetically re-order

2021-10-09 Thread Marcel Ziswiler
From: Marcel Ziswiler Alphabetically re-order nodes and properties. Signed-off-by: Marcel Ziswiler --- Changes in v6: - New patch. arch/arm/dts/imx8mm-verdin-u-boot.dtsi | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/arch/arm/dts/imx8mm-verdin-u

[PATCH v6 03/11] ARM: dts: imx8mm-verdin: prepare for dek blob encapsulation

2021-10-09 Thread Marcel Ziswiler
From: Marcel Ziswiler Prepare for DEK blob encapsulation support through "dek_blob" command. On ARMv8, u-boot runs in non-secure, thus cannot encapsulate a DEK blob for encrypted boot. The DEK blob is encapsulated by OP-TEE through a trusted application call. U-boot sends and receiv

[PATCH v6 01/11] imx8m: clean-up kconfig indentation

2021-10-09 Thread Marcel Ziswiler
From: Marcel Ziswiler Replace spurious spaces with proper tabs. Signed-off-by: Marcel Ziswiler Reviewed-by: Fabio Estevam --- (no changes since v1) arch/arm/mach-imx/imx8m/Kconfig | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/arch/arm/mach-imx

[PATCH v6 02/11] verdin-imx8mm: fix ethernet

2021-10-09 Thread Marcel Ziswiler
From: Marcel Ziswiler Turns out Microship (formerly Micrel) meanwhile integrated proper support for the DLL setup on their KSZ9131. Unfortunately, this conflicts with our previous board code doing that. Fix this by getting rid of our board code and just relying on the generic implementation

[PATCH v2 7/7] arm64: dts: imx8mm-u-boot.dtsi: improve odd blob-ext naming

2021-10-08 Thread Marcel Ziswiler
From: Marcel Ziswiler Rather than using odd implicit blob-ext naming, explicitly specify the type to be of blob-ext and therefore also simplify the node naming. Signed-off-by: Marcel Ziswiler --- Changes in v2: - Rather than doing that clean-up to the separate binman dtsi now do

[PATCH v2 5/7] arm64: dts: imx8mm-u-boot.dtsi: alphabetically re-order properties

2021-10-08 Thread Marcel Ziswiler
From: Marcel Ziswiler Alphabetically re-order properties. Signed-off-by: Marcel Ziswiler --- Changes in v2: - Rather than doing that clean-up to the separate binman dtsi now do it to the common u-boot dtsi one. arch/arm/dts/imx8mm-u-boot.dtsi | 34 - 1

[PATCH v2 6/7] arm64: dts: imx8mm-u-boot.dtsi: explicitly add spl filename

2021-10-08 Thread Marcel Ziswiler
From: Marcel Ziswiler Explicitly add SPL aka u-boot-spl.bin filename. Signed-off-by: Marcel Ziswiler --- Changes in v2: - Rather than doing that clean-up to the separate binman dtsi now do it to the common u-boot dtsi one. arch/arm/dts/imx8mm-u-boot.dtsi | 1 + 1 file changed, 1

[PATCH v2 4/7] arm64: dts: imx8mm: use common binman configuration

2021-10-08 Thread Marcel Ziswiler
From: Marcel Ziswiler With the move to using binman to generate SPL aka u-boot-spl-ddr.bin and U-Boot proper aka u-boot.itb every board now covers such configuration in its own U-Boot specific device tree include. Move the comon part of that configuration to the common imx8mm-u-boot.dtsi include

[PATCH v2 2/7] arm64: dts: imx8mm-cl-iot-gate-u-boot.dtsi: alphabetically re-order

2021-10-08 Thread Marcel Ziswiler
From: Marcel Ziswiler Alphabetically re-order nodes and properties. Signed-off-by: Marcel Ziswiler --- Changes in v2: - New patch preparing cl-iot-gate. .../dts/imx8mm-cl-iot-gate-optee-u-boot.dtsi | 202 +- arch/arm/dts/imx8mm-cl-iot-gate-u-boot.dtsi | 202

[PATCH v2 3/7] arm64: dts: imx8mm-cl-iot-gate-u-boot.dtsi: use common imx8mm-u-boot.dtsi

2021-10-08 Thread Marcel Ziswiler
From: Marcel Ziswiler Use common imx8mm-u-boot.dtsi. Signed-off-by: Marcel Ziswiler --- Changes in v2: - New patch preparing cl-iot-gate. .../dts/imx8mm-cl-iot-gate-optee-u-boot.dtsi | 37 +-- arch/arm/dts/imx8mm-cl-iot-gate-u-boot.dtsi | 37 +-- 2 files

[PATCH v2 0/7] arm64: dts: imx8mm: add common -binman.dtsi and further clean-up

2021-10-08 Thread Marcel Ziswiler
From: Marcel Ziswiler With the move to using binman to generate SPL aka u-boot-spl-ddr.bin and U-Boot proper aka u-boot.itb every board now covers such configuration in its own U-Boot specific device tree include. Move the comon part of that configuration to the common imx8mm-u-boot.dtsi

[PATCH v2 1/7] imx8mm-cl-iot-gate: fix imximage intermediate binary naming

2021-10-08 Thread Marcel Ziswiler
From: Marcel Ziswiler This fixes the following build time issue: ... BINMAN all binman: Error 1 running 'mkimage -d ./mkimage.spl.mkimage -n spl/u-boot-spl.cfgout -T imx8mimage -e 0x7e1000 ./mkimage-out.spl.mkimage': mkimage.flash.mkimage: Can't open: No such file or directory make

[PATCH v5 08/10] include/configs: apalis-imx8/verdin-imx8mm: rename kernel image variable

2021-10-07 Thread Marcel Ziswiler
;. Setting kernel_image="Image" as a pre-defined u-boot variable breaks booting systems with modern versions of boot.scr, whereas renaming it fixes booting with modern scripts and does not break working of earlier versions of boot.scr. Signed-off-by: Oleksandr Suvorov Signed-off-by: Mar

[PATCH v5 00/10] board: toradex: verdin-imx8mm: target refresh

2021-10-07 Thread Marcel Ziswiler
From: Marcel Ziswiler An assortment of fixes and improvements like an Ethernet PHY configuration fix, DEK blob encapsulation preparation, migration to using binman to pack images, SLEEP_MOCI# enablement, dropping of V1.0 hardware support [1], renaming kernel image variable, using preboot

[PATCH v5 10/10] verdin-imx8mm: fix watchdog pinctrl issue

2021-10-07 Thread Marcel Ziswiler
From: Marcel Ziswiler Finally, found the root cause of the issue already once mentioned back here [2] which caused the following error message during boot: imx_wdt watchdog@3028: pinctrl_select_state_full: uclass_get_device_by_phandle_id: err=-19 Turns out while the watchdog node itself

[PATCH v5 04/10] verdin-imx8mm: switch to use binman to pack images

2021-10-07 Thread Marcel Ziswiler
From: Marcel Ziswiler Use binman to pack images. Signed-off-by: Marcel Ziswiler Reviewed-by: Heiko Thiery Reviewed-by: Fabio Estevam Reviewed-by: Heiko Schocher --- (no changes since v4) Changes in v4: - Add Heiko Schocher's reviewed-by tag. - Fix copyright periods. Changes in v3

[PATCH v5 09/10] verdin-imx8mm: use preboot for fdtfile evaluation

2021-10-07 Thread Marcel Ziswiler
mode without manual setting fdtfile value, as it it's already evaluated before entering interactive mode. Signed-off-by: Igor Opaniuk Signed-off-by: Marcel Ziswiler Reviewed-by: Fabio Estevam --- (no changes since v2) Changes in v2: - New patch addressing dynamic fdtfile definition. configs

[PATCH v5 05/10] verdin-imx8mm: enable sleep_moci output

2021-10-07 Thread Marcel Ziswiler
From: Max Krummenacher This powers some peripherals on the carrier board e.g. the USB hub. Related-to: ELB-3206 Signed-off-by: Max Krummenacher Signed-off-by: Marcel Ziswiler Reviewed-by: Fabio Estevam --- (no changes since v1) arch/arm/dts/imx8mm-verdin.dts | 18

[PATCH v5 07/10] verdin-imx8mm: drop support for v1.0 hardware

2021-10-07 Thread Marcel Ziswiler
From: Max Krummenacher We drop support for Verdin iMX8M Mini V1.0B. Related-to: ELB-3551 Signed-off-by: Max Krummenacher Signed-off-by: Marcel Ziswiler Reviewed-by: Fabio Estevam --- Changes in v5: - Re-based on top of u-boot-imx, master yet again. Changes in v4: - Re-based. Changes

[PATCH v5 01/10] imx8m: clean-up kconfig indentation

2021-10-07 Thread Marcel Ziswiler
From: Marcel Ziswiler Replace spurious spaces with proper tabs. Signed-off-by: Marcel Ziswiler Reviewed-by: Fabio Estevam --- (no changes since v1) arch/arm/mach-imx/imx8m/Kconfig | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/arch/arm/mach-imx

[PATCH v5 06/10] verdin-imx8mm: clean-up include order

2021-10-07 Thread Marcel Ziswiler
From: Marcel Ziswiler Alphabetically order includes. While at it also update copyright year resp. period. Signed-off-by: Marcel Ziswiler --- (no changes since v3) Changes in v3: - New patch cleaning up include order. board/toradex/verdin-imx8mm/verdin-imx8mm.c | 4 ++-- 1 file changed, 2

[PATCH v5 03/10] ARM: dts: imx8mm-verdin: prepare for dek blob encapsulation

2021-10-07 Thread Marcel Ziswiler
From: Marcel Ziswiler Prepare for DEK blob encapsulation support through "dek_blob" command. On ARMv8, u-boot runs in non-secure, thus cannot encapsulate a DEK blob for encrypted boot. The DEK blob is encapsulated by OP-TEE through a trusted application call. U-boot sends and receiv

[PATCH v5 02/10] verdin-imx8mm: fix ethernet

2021-10-07 Thread Marcel Ziswiler
From: Marcel Ziswiler Turns out Microship (formerly Micrel) meanwhile integrated proper support for the DLL setup on their KSZ9131. Unfortunately, this conflicts with our previous board code doing that. Fix this by getting rid of our board code and just relying on the generic implementation

Re: [PATCH] ARM: dts: imx8mm-verdin: Set PHY mode to RGMII-ID

2021-10-07 Thread Marcel Ziswiler
On Thu, 2021-10-07 at 23:56 +, Marcel Ziswiler wrote: > On Thu, 2021-10-07 at 16:12 +0200, sba...@denx.de wrote: > > > Since c6df0e2ffdc ("net: phy: micrel: add support for DLL setup on > > > ksz9131") > > > the Micrel PHY driver correctly configures t

Re: [PATCH] ARM: dts: imx8mm-verdin: Set PHY mode to RGMII-ID

2021-10-07 Thread Marcel Ziswiler
, otherwise the ethernet no longer works. > > Signed-off-by: Marek Vasut > > Cc: Marcel Ziswiler > > Cc: Max Krummenacher > > Cc: Oleksandr Suvorov > Applied to u-boot-imx, master, thanks ! No, remember, you should not have picked that one but rather my patch series instead! > Best regards, > Stefano Babic

Re: [PATCH v4 04/10] verdin-imx8mm: switch to use binman to pack images

2021-10-07 Thread Marcel Ziswiler
t; > +   blob_1: blob-ext@1 { > > +   filename = "lpddr4_pmu_train_1d_imem.bin"; > > +   size = <0x8000>; > > +   }; > > You can use the 'type' if you like, so you don't need the @: > > blob_1: 1d-imem { >     type = "blob-ext"; >     filename = "..."; >     ... Yeah, remember, we agreed on doing such further clean-up in a separate step once we migrated towards using a common binman configuration [1]. I am about to send out a v2 thereof. [1] https://marc.info/?l=u-boot=162998010613817 > Regards, > Simon Cheers Marcel

Re: [PATCH v2] imx8mm_venice: switch to use binman to pack images

2021-10-07 Thread Marcel Ziswiler
On Thu, 2021-10-07 at 10:26 -0700, Tim Harvey wrote: > ... > Marvell, (;-p) > Indeed that patch resolves the issue - thanks for reminding me that > hasn't made it in yet! You are very welcome. Yeah, I hope somebody starts pulling that stuff (and everything else still in-flight) soon. As it

Re: [PATCH v2] imx8mm_venice: switch to use binman to pack images

2021-10-07 Thread Marcel Ziswiler
Hi Tim On Thu, 2021-10-07 at 08:52 -0700, Tim Harvey wrote: > )On Wed, Oct 6, 2021 at 2:44 PM Marcel Ziswiler wrote: > > > > On Wed, 2021-10-06 at 13:17 -0700, Tim Harvey wrote: > > > Use binman to pack images. > > > > > > Note that imx8mm_venice suppo

Re: [RFC PATCH v1 0/5] arm64: dts: imx8mm: add common -binman.dtsi and further clean-up

2021-10-06 Thread Marcel Ziswiler
On Thu, 2021-09-02 at 14:08 -0300, Fabio Estevam wrote: > Hi Marcel, > > On Thu, Aug 26, 2021 at 9:14 AM Marcel Ziswiler wrote: > > > > From: Marcel Ziswiler > > > > > > With the move to using binman to generate SPL aka u-boot-spl-ddr.bin and > >

Re: [RFC PATCH v1 2/5] arm64: dts: imx8mm: add common -binman.dtsi

2021-10-06 Thread Marcel Ziswiler
Hi Tim On Thu, 2021-08-26 at 07:57 -0700, Tim Harvey wrote: > ... > Marcel, > > This would break imx8mm-venice. > > The of-list, @fdt-SEQ and @config-SEQ are required to support > automatic generation of fdt and config nodes when you have multiple > device-trees. See

Re: [PATCH v2] imx8mm_venice: switch to use binman to pack images

2021-10-06 Thread Marcel Ziswiler
On Wed, 2021-10-06 at 13:17 -0700, Tim Harvey wrote: > Use binman to pack images. > > Note that imx8mm_venice supports several boards via multiple DTB's thus > in the fit node we must use: > - fit,fdt-list = "of-list" > - fdt-SEQ > - config-SEQ > > Signed-off

[PATCH v4 10/10] verdin-imx8mm: fix watchdog pinctrl issue

2021-10-06 Thread Marcel Ziswiler
From: Marcel Ziswiler Finally, found the root cause of the issue already once mentioned back here [2] which caused the following error message during boot: imx_wdt watchdog@3028: pinctrl_select_state_full: uclass_get_device_by_phandle_id: err=-19 Turns out while the watchdog node itself

[PATCH v4 00/10] board: toradex: verdin-imx8mm: target refresh

2021-10-06 Thread Marcel Ziswiler
From: Marcel Ziswiler An assortment of fixes and improvements like an Ethernet PHY configuration fix, DEK blob encapsulation preparation, migration to using binman to pack images, SLEEP_MOCI# enablement, dropping of V1.0 hardware support [1], renaming kernel image variable, using preboot

[PATCH v4 08/10] include/configs: apalis-imx8/verdin-imx8mm: rename kernel image variable

2021-10-06 Thread Marcel Ziswiler
;. Setting kernel_image="Image" as a pre-defined u-boot variable breaks booting systems with modern versions of boot.scr, whereas renaming it fixes booting with modern scripts and does not break working of earlier versions of boot.scr. Signed-off-by: Oleksandr Suvorov Signed-off-by: Mar

[PATCH v4 09/10] verdin-imx8mm: use preboot for fdtfile evaluation

2021-10-06 Thread Marcel Ziswiler
mode without manual setting fdtfile value, as it it's already evaluated before entering interactive mode. Signed-off-by: Igor Opaniuk Signed-off-by: Marcel Ziswiler Reviewed-by: Fabio Estevam --- (no changes since v2) Changes in v2: - New patch addressing dynamic fdtfile definition. configs

[PATCH v4 07/10] verdin-imx8mm: drop support for v1.0 hardware

2021-10-06 Thread Marcel Ziswiler
From: Max Krummenacher We drop support for Verdin iMX8M Mini V1.0B. Related-to: ELB-3551 Signed-off-by: Max Krummenacher Signed-off-by: Marcel Ziswiler Reviewed-by: Fabio Estevam --- Changes in v4: - Re-based. Changes in v3: - Add Fabio's reviewed-by tag. Changes in v2: - Drop first

[PATCH v4 04/10] verdin-imx8mm: switch to use binman to pack images

2021-10-06 Thread Marcel Ziswiler
From: Marcel Ziswiler Use binman to pack images. Signed-off-by: Marcel Ziswiler Reviewed-by: Heiko Thiery Reviewed-by: Fabio Estevam Reviewed-by: Heiko Schocher --- Changes in v4: - Add Heiko Schocher's reviewed-by tag. - Fix copyright periods. Changes in v3: - Case fold hex string

[PATCH v4 05/10] verdin-imx8mm: enable sleep_moci output

2021-10-06 Thread Marcel Ziswiler
From: Max Krummenacher This powers some peripherals on the carrier board e.g. the USB hub. Related-to: ELB-3206 Signed-off-by: Max Krummenacher Signed-off-by: Marcel Ziswiler Reviewed-by: Fabio Estevam --- (no changes since v1) arch/arm/dts/imx8mm-verdin.dts | 18

[PATCH v4 03/10] ARM: dts: imx8mm-verdin: prepare for dek blob encapsulation

2021-10-06 Thread Marcel Ziswiler
From: Marcel Ziswiler Prepare for DEK blob encapsulation support through "dek_blob" command. On ARMv8, u-boot runs in non-secure, thus cannot encapsulate a DEK blob for encrypted boot. The DEK blob is encapsulated by OP-TEE through a trusted application call. U-boot sends and receiv

[PATCH v4 06/10] verdin-imx8mm: clean-up include order

2021-10-06 Thread Marcel Ziswiler
From: Marcel Ziswiler Alphabetically order includes. While at it also update copyright year resp. period. Signed-off-by: Marcel Ziswiler --- (no changes since v3) Changes in v3: - New patch cleaning up include order. board/toradex/verdin-imx8mm/verdin-imx8mm.c | 4 ++-- 1 file changed, 2

[PATCH v4 02/10] verdin-imx8mm: fix ethernet

2021-10-06 Thread Marcel Ziswiler
From: Marcel Ziswiler Turns out Microship (formerly Micrel) meanwhile integrated proper support for the DLL setup on their KSZ9131. Unfortunately, this conflicts with our previous board code doing that. Fix this by getting rid of our board code and just relying on the generic implementation

[PATCH v4 01/10] imx8m: clean-up kconfig indentation

2021-10-06 Thread Marcel Ziswiler
From: Marcel Ziswiler Replace spurious spaces with proper tabs. Signed-off-by: Marcel Ziswiler Reviewed-by: Fabio Estevam --- (no changes since v1) arch/arm/mach-imx/imx8m/Kconfig | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/arch/arm/mach-imx

[PATCH v2 5/5] colibri-imx6ull: add emmc variant

2021-10-06 Thread Marcel Ziswiler
From: Max Krummenacher Add code to build the eMMC variant of the Colibri iMX6ULL, i.e. the 'Colibri iMX6ULL 1GB' which has a eMMC instead of the raw NAND used on other SKUs. Related-to: ELB-4056, ELB-4057 Signed-off-by: Max Krummenacher Signed-off-by: Marcel Ziswiler --- Changes in v2: - Re

[PATCH v2 0/5] board: toradex: prepare and add colibri imx6ull 1gb (emmc) support

2021-10-06 Thread Marcel Ziswiler
From: Marcel Ziswiler Some preparatory steps concerning tdx-cfg-block including a fix for the interactive mode IT handling and adding them new SKUs and then adding initial support for the Toradex Colibri iMX6ULL 1GB (eMMC) V1.1A module. They are now strapped to boot from eFuses which

[PATCH v2 4/5] colibri-imx6ull: imximage.cfg: integrate new 1GiB RAM variant

2021-10-06 Thread Marcel Ziswiler
for all Toradex Colibri iMX6ULL SKUs Related-to: ELB-4055, ELB-4057 Signed-off-by: Philippe Schenker Signed-off-by: Marcel Ziswiler --- (no changes since v1) board/toradex/colibri-imx6ull/imximage.cfg | 13 + include/configs/colibri-imx6ull.h | 2 +- 2 files changed, 6

[PATCH v2 3/5] toradex: tdx-cfg-block: add new i.mx 6ull and 8m plus skus

2021-10-06 Thread Marcel Ziswiler
From: Marcel Ziswiler Add new i.MX 6ULL and 8M Plus SKUs to ConfigBlock handling: 0062: Colibri iMX6ULL 1GB IT (eMMC) 0063: Verdin iMX8M Plus Quad 4GB IT 0064: Verdin iMX8M Plus Quad 2GB Wi-Fi / BT IT 0065: Verdin iMX8M Plus QuadLite 1GB IT 0066: Verdin iMX8M Plus Quad 8GB Wi-Fi / BT

[PATCH v2 2/5] toradex: configblock: fix interactive mode it handling

2021-10-06 Thread Marcel Ziswiler
us") Related-to: ELB-3482 Signed-off-by: Denys Drozdov Signed-off-by: Marcel Ziswiler --- (no changes since v1) board/toradex/common/tdx-cfg-block.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/board/toradex/common/tdx-cfg-block.c b/board/toradex/common/tdx

[PATCH v2 1/5] toradex: take over maintainership

2021-10-06 Thread Marcel Ziswiler
From: Marcel Ziswiler Also take over maintainership of remaining Toradex SoMs as Oleksandr has left our company. Signed-off-by: Marcel Ziswiler CC: Oleksandr Suvorov --- (no changes since v1) board/toradex/apalis-imx8x/MAINTAINERS| 2 +- board/toradex/apalis_imx6/MAINTAINERS | 2

Re: [PATCH v2] apalis/colibri_imx6: move setting bootcmd to defconfig

2021-09-16 Thread Marcel Ziswiler
ewed-by: Igor Opaniuk Acked-by: Marcel Ziswiler > --- > > Changes in v2: > - fix spaces after semicolon. > - add reviewed-by records. > >  configs/apalis_imx6_defconfig  | 1 + >  configs/colibri_imx6_defconfig | 1 + >  include/configs/apalis_imx6.h  | 4 >  include

Re: [PATCH] apalis/colibri_imx6: move setting bootcmd to defconfig

2021-09-15 Thread Marcel Ziswiler
t; BOOTENV \ > -   "bootcmd=run distro_bootcmd; " \ > -   "usb start ; " \ > -   "setenv stdout serial,vidconsole; " \ > -   "setenv stdin serial,usbkbd\0" \ > "boot_file=zImage\0" \ > "console=ttymxc0\0" \ > "defargs=enable_wait_mode=off galcore.contiguousSize=50331648\0" \ Cheers Marcel

Re: [PATCH] ARM: dts: imx8mm-verdin: Set PHY mode to RGMII-ID

2021-09-11 Thread Marcel Ziswiler
ethernet no longer works. Yes, however, one should also get rid of the proprietary PHY setup in our board setup. Remember, I already did send this as part of my target refresh series: https://marc.info/?l=u-boot=162990456210415 > Signed-off-by: Marek Vasut > Cc: Marcel Ziswiler >

Re: [PATCH v2 1/2] GPIO: fxl6408: Add support for FXL6408 GPIO expander

2021-09-09 Thread Marcel Ziswiler
bles support for such devices. > > Based on: https://patchwork.kernel.org/patch/9148419/ > > Signed-off-by: Oleksandr Suvorov Hehe. > Reviewed-by: Simon Glass > Signed-off-by: Oleksandr Suvorov Tested-by: Marcel Ziswiler > --- > > Changes in v2: None > >  driver

Re: [PATCH] doc: board: toradex: fix file names

2021-09-09 Thread Marcel Ziswiler
; Signed-off-by: Oleksandr Suvorov Acked-by: Marcel Ziswiler > --- > >  board/toradex/apalis-imx8/MAINTAINERS    | 2 +- >  doc/board/toradex/{apalix-imx8.rst => apalis-imx8.rst}   | 0 >  doc/board/toradex/{apalix-imx8x.rst => apali

Re: [PATCH] watchdog: Fix watchdog enablement in SPL and TPL

2021-09-02 Thread Marcel Ziswiler
On Tue, 2021-08-31 at 11:17 +0200, Marek Vasut wrote: > On 8/31/21 8:49 AM, Marcel Ziswiler wrote: > > Hi Marek > > > > On Tue, 2021-08-31 at 00:03 +0200, Marek Vasut wrote: > > > Commit 830d29ac372 ("watchdog: Allow to use CONFIG_WDT without starting > >

Re: [[PATCH] colibri-imx6ull: move CONFIG_BOOTCOMMAND from header to defconfig

2021-09-01 Thread Marcel Ziswiler
On Wed, 2021-09-01 at 16:33 +0200, liu.min...@gmail.com wrote: > From: Ming Liu Please note that we do have a patch set introducing Colibri iMX6ULL 1GB (eMMC) support [1] in-flight and you likely would need to re-base on top of that one. Thanks! [1] https://marc.info/?l=u-boot=162911970822560

Re: [PATCH] watchdog: Fix watchdog enablement in SPL and TPL

2021-08-31 Thread Marcel Ziswiler
TPL phase. Enabled by default. Disable this option if you want > + to compile U-Boot with CONFIG_WDT support but do not want to > + activate watchdog, like when CONFIG_WDT option is disabled. > + >  endmenu Those Kconfig entries look fine. However, I am wondering where exactly they get used. Am I missing anything? Cheers Marcel

Re: [PATCH 3/3] imx8qm_mek: Increase CONFIG_SYS_BOOTM_LEN to 64MB

2021-08-30 Thread Marcel Ziswiler
On Mon, 2021-08-30 at 13:46 -0400, Tom Rini wrote: > On Mon, Aug 30, 2021 at 03:05:32PM +0000, Marcel Ziswiler wrote: > > On Mon, 2021-08-30 at 14:18 +0200, Marek Vasut wrote: > > > On 8/30/21 1:11 PM, Oleksandr Suvorov wrote: > > > > On Sun, Aug 29, 2021

Re: [PATCH 3/3] imx8qm_mek: Increase CONFIG_SYS_BOOTM_LEN to 64MB

2021-08-30 Thread Marcel Ziswiler
On Mon, 2021-08-30 at 17:10 +0200, Marek Vasut wrote: > On 8/30/21 5:05 PM, Marcel Ziswiler wrote: > > On Mon, 2021-08-30 at 14:18 +0200, Marek Vasut wrote: > > > On 8/30/21 1:11 PM, Oleksandr Suvorov wrote: > > > > On Sun, Aug 29, 2021 at 10:55 PM Marek Vasut wrote

Re: [PATCH 3/3] imx8qm_mek: Increase CONFIG_SYS_BOOTM_LEN to 64MB

2021-08-30 Thread Marcel Ziswiler
On Mon, 2021-08-30 at 14:18 +0200, Marek Vasut wrote: > On 8/30/21 1:11 PM, Oleksandr Suvorov wrote: > > On Sun, Aug 29, 2021 at 10:55 PM Marek Vasut wrote: > > > > > > On 8/29/21 9:39 PM, Oleksandr Suvorov wrote: > > > > The BSP platform LmP supports the board NXP iMX8QM MEK. The > > > > kernel

Re: [PATCH v2 3/3] RFC: doc: Add documentation about devicetree usage

2021-08-30 Thread Marcel Ziswiler
ed, but it is > +  not pretty. We could use binman to support more complex arrangements, but > only > +  some boards use this at present, so it would be a big change. > + > +- **API** - How would another project provide two devicetree files to U-Boot > at > +  runtime? Presumably this would just be too painful. But if it doesn't, it > +  would be unable to configure run-time features of U-Boot during the boot. > + > +- **Confusion** - No other project has two devicetrees. U-Boot would be in > the > +  unfortunate position of having to describe this fact to new users, along > with > +  the (arguably contrived) reason for the arrangement. > + > +- **Signing flow** - The current signing flow is simple as it involves > running > +  `mkimage` with the U-Boot devicetree. This would have to be updated to use > the > +  special devicetree. Some way of telling the user that they have done it > wrong > +  would have to be invented. > + > +Overall, adding a second devicetree would create enormous confusion and > +complexity. It seems a lot cheaper to solve this by a change of attitude. > + > + > +.. _rpi_patch: > https://patchwork.ozlabs.org/project/uboot/patch/20170402082520.32546-1-de...@google.com/ > +.. _bcm_patch: > https://patchwork.ozlabs.org/project/uboot/patch/16fc0901f4521d3c399eac950c52a634b2f9473b.1528485916.git.fitz...@fitzsim.org/ > +.. _`TF-A`: https://www.trustedfirmware.org/projects/tf-a > +.. _`QEMU ARM`: https://github.com/qemu/qemu/blob/master/hw/arm/virt.c > +.. _`QEMU RISC-V`: https://github.com/qemu/qemu/blob/master/hw/riscv/virt.c > +.. _`/chosen node`: > https://www.kernel.org/doc/Documentation/devicetree/bindings/chosen.txt > +.. _fdt_add_pubkey: > https://patchwork.ozlabs.org/project/uboot/list/?series=157843=* > +.. _dtspec: https://www.devicetree.org/specifications/ > diff --git a/doc/develop/package/index.rst b/doc/develop/package/index.rst > index 9374be2e62c..188c376950e 100644 > --- a/doc/develop/package/index.rst > +++ b/doc/develop/package/index.rst > @@ -17,3 +17,4 @@ SPI flash. >     :maxdepth: 2 >   >     binman > +   devicetree Thanks for that write-up. Reviewed-by: Marcel Ziswiler

<    1   2   3   4   5   6   7   8   9   10   >