October 2021 - U-Boot

FIT image: load secure FPGA
by Jorge Ramirez-Ortiz, Foundries 09 Feb '22

09 Feb '22

Hello, We are enabling secure boot on Zynqmp with SPL. The issue however is that during secure boot, the bootrom not only validates the first loader (SPL and PMUFW combo) but it will also expect a signed bitstream during load(FPGA). Since currently the SPL load of an FPGA image from FIT does not support loading images for authentication (fpga_loads), I'd like to discuss how to best implement such support. A pretty standard file.its description of the FPGA loadable looks like this: fpga { description = "FPGA binary"; data = /incbin/("${DEPLOY_DIR_IMAGE}/${SPL_FPGA_BINARY}"); type = "fpga"; arch = "${UBOOT_ARCH}"; compression = "none"; load = <${fpgaloadaddr}>; hash-1 { algo = "${FIT_HASH_ALG}"; }; }; We could extend imagetool.h struct image_tool_params to add more params or perhpas just define different 'types' of fpga? Something like: "fpga" "fpga-auth" : authenticated "fpga-enc" : encrypted "fpga-sec" : encrypted and authenticated Then it would be a matter of modifying https://github.com/u-boot/u-boot/blob/master/common/spl/spl_fit.c#L572 any thoughts? TIA Jorge

5 12

[PATCH v4] apalis/colibri_imx6: move setting bootcmd to defconfig
by Oleksandr Suvorov 07 Feb '22

07 Feb '22

Move setting the default boot command to the apalis/colibri_imx6_defconfig. It allows replacing the command without code modification. Signed-off-by: Oleksandr Suvorov <oleksandr.suvorov(a)foundries.io> Reviewed-by: Peng Fan <peng.fan(a)nxp.com> Reviewed-by: Igor Opaniuk <igor.opaniuk(a)foundries.io> Acked-by: Marcel Ziswiler <marcel.ziswiler(a)toradex.com> --- Changes in v4: - add an acked-by record. Changes in v3: - fix Peng Fan's e-mail address. 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/configs/colibri_imx6.h | 4 ---- 4 files changed, 2 insertions(+), 8 deletions(-) diff --git a/configs/apalis_imx6_defconfig b/configs/apalis_imx6_defconfig index a0e85ba23a4..f64b1ca4378 100644 --- a/configs/apalis_imx6_defconfig +++ b/configs/apalis_imx6_defconfig @@ -23,6 +23,7 @@ CONFIG_DISTRO_DEFAULTS=y CONFIG_FIT=y CONFIG_SYS_EXTRA_OPTIONS="IMX_CONFIG=arch/arm/mach-imx/spl_sd.cfg" CONFIG_BOOTDELAY=1 +CONFIG_BOOTCOMMAND="run distro_bootcmd; usb start; setenv stdout serial,vidconsole; setenv stdin serial,usbkbd" # CONFIG_DISPLAY_BOARDINFO is not set CONFIG_DISPLAY_BOARDINFO_LATE=y CONFIG_BOARD_EARLY_INIT_F=y diff --git a/configs/colibri_imx6_defconfig b/configs/colibri_imx6_defconfig index 47b1cfb1917..373d6476ddc 100644 --- a/configs/colibri_imx6_defconfig +++ b/configs/colibri_imx6_defconfig @@ -22,6 +22,7 @@ CONFIG_DISTRO_DEFAULTS=y CONFIG_FIT=y CONFIG_SYS_EXTRA_OPTIONS="IMX_CONFIG=arch/arm/mach-imx/spl_sd.cfg" CONFIG_BOOTDELAY=1 +CONFIG_BOOTCOMMAND="run distro_bootcmd; usb start; setenv stdout serial,vidconsole; setenv stdin serial,usbkbd" # CONFIG_DISPLAY_BOARDINFO is not set CONFIG_DISPLAY_BOARDINFO_LATE=y CONFIG_BOARD_EARLY_INIT_F=y diff --git a/include/configs/apalis_imx6.h b/include/configs/apalis_imx6.h index 12de0105c6c..ade479cb943 100644 --- a/include/configs/apalis_imx6.h +++ b/include/configs/apalis_imx6.h @@ -133,10 +133,6 @@ #endif #define CONFIG_EXTRA_ENV_SETTINGS \ 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 vmalloc=400M\0" \ diff --git a/include/configs/colibri_imx6.h b/include/configs/colibri_imx6.h index 804a144a03e..f91d6b49ea8 100644 --- a/include/configs/colibri_imx6.h +++ b/include/configs/colibri_imx6.h @@ -115,10 +115,6 @@ #define FDT_FILE "imx6dl-colibri-eval-v3.dtb" #define CONFIG_EXTRA_ENV_SETTINGS \ 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" \ -- 2.31.1

3 2

[PATCH 00/20] i.MX8ULP misc update
by Peng Fan (OSS) 05 Feb '22

05 Feb '22

From: Peng Fan <peng.fan(a)nxp.com> This is to upstream NXP downstream i.MX8ULP patches including ADC/DRAM/LPAV/FUSE/ENET update/fix Alice Guo (3): imx8ulp: clock: Support to enable/disable the ADC1 clock imx8ulp: measure die temperature using PMC Temperature Sensor imx8ulp: implement to obtain the SoC current temperature Jacky Bai (1): imx8ulp:ddr: saving the dram config timing data into sram Peng Fan (1): imx8ulp: assign PXP/HIFI4/EPDC to APD domain Ye Li (15): imx8ulp: soc: Check the DBD_EN fuse before setting RDC arm: imx8ulp: Allocate LPAV resources to AP domain imx8ulp: clock: Support LPAV clocks in cgc and pcc imx8ulp: clock: Add MIPI DSI clock and DCNano clock imx8ulp: rdc: allow A35 access flexspi0 mem imx8ulp_evk: Control LPI2C0 PCA6416 and TPM0 for display imx8ulp: Set DCNANO read QoS on NIC_LPAV to highest imx8ulp: Fix DCNANO QoS setting imx8ulp: Remove freescale name from CPU revision imx8ulp: Workaround LPOSC_TRIM fuse load issue imx8ulp: clock: Reset DDR controller before clock enable imx8ulp: clock: Support to reset DCNano and MIPI DSI imx8ulp: Update ethernet mac to get from fuse imx8ulp: clock: Handle the DDRLOCKED when setting DDR clock imx8ulp: ddr: Fix DDR frequency request issue arch/arm/include/asm/arch-imx8ulp/cgc.h | 37 +- arch/arm/include/asm/arch-imx8ulp/clock.h | 4 + arch/arm/include/asm/arch-imx8ulp/imx-regs.h | 63 +++- .../include/asm/arch-imx8ulp/imx8ulp-pins.h | 4 + arch/arm/include/asm/arch-imx8ulp/pcc.h | 70 +++- arch/arm/include/asm/arch-imx8ulp/sys_proto.h | 2 + arch/arm/mach-imx/imx8ulp/cgc.c | 325 +++++++++++++++--- arch/arm/mach-imx/imx8ulp/clock.c | 156 ++++++++- arch/arm/mach-imx/imx8ulp/pcc.c | 158 ++++++++- arch/arm/mach-imx/imx8ulp/soc.c | 117 ++++++- board/freescale/imx8ulp_evk/imx8ulp_evk.c | 56 +++ board/freescale/imx8ulp_evk/spl.c | 9 + drivers/ddr/imx/imx8ulp/Kconfig | 7 + drivers/ddr/imx/imx8ulp/ddr_init.c | 55 ++- drivers/thermal/Kconfig | 7 + drivers/thermal/Makefile | 1 + drivers/thermal/imx_pmc_temperature.c | 174 ++++++++++ 17 files changed, 1148 insertions(+), 97 deletions(-) create mode 100644 drivers/thermal/imx_pmc_temperature.c -- 2.30.0

2 38

[PATCH v1 0/2] board: toradex: improve env badblock management for NAND variant boards
by Francesco Dolcini 05 Feb '22

05 Feb '22

Use the complete 512kb (4 blocks) nand partition reserved for u-boot environment instead of just the first block, this allows the module to have a working environment even if 3 blocks are bad. Francesco Dolcini (2): colibri-imx6ull: improve env badblock management colibri-imx7: improve env badblock management include/configs/colibri-imx6ull.h | 5 +++++ include/configs/colibri_imx7.h | 5 +++++ 2 files changed, 10 insertions(+) -- 2.25.1

3 7

[PATCH] mxs: power: Change sequence of enabling DCDC switches
by Lukasz Majewski 05 Feb '22

05 Feb '22

The imx28 uses following voltage supplies hierarchy: VDD_5V (VDD_BAT) -> VDDIO -> VDDA -> VDDMEM \-----> VDDD One shall first enable DCDC on the parent source (VDDIO) and then follow with its children. Signed-off-by: Lukasz Majewski <lukma(a)denx.de> --- arch/arm/cpu/arm926ejs/mxs/spl_power_init.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/arm/cpu/arm926ejs/mxs/spl_power_init.c b/arch/arm/cpu/arm926ejs/mxs/spl_power_init.c index 35ea71a5ba8..c33170f06d3 100644 --- a/arch/arm/cpu/arm926ejs/mxs/spl_power_init.c +++ b/arch/arm/cpu/arm926ejs/mxs/spl_power_init.c @@ -627,11 +627,11 @@ static void mxs_power_enable_4p2(void) mxs_power_init_dcdc_4p2_source(); - writel(vdddctrl, &power_regs->hw_power_vdddctrl); + writel(vddioctrl, &power_regs->hw_power_vddioctrl); early_delay(20); writel(vddactrl, &power_regs->hw_power_vddactrl); early_delay(20); - writel(vddioctrl, &power_regs->hw_power_vddioctrl); + writel(vdddctrl, &power_regs->hw_power_vdddctrl); /* * Check if FET is enabled on either powerout and if so, -- 2.20.1

4 3

[PATCH] imx: nandbcb: Fix printf format in write_fcb
by Pali Rohár 05 Feb '22

05 Feb '22

Correct printf format for unsigned long long is %llx and not %llxx. Signed-off-by: Pali Rohár <pali(a)kernel.org> --- arch/arm/mach-imx/cmd_nandbcb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/mach-imx/cmd_nandbcb.c b/arch/arm/mach-imx/cmd_nandbcb.c index 09622c13c980..646da0198660 100644 --- a/arch/arm/mach-imx/cmd_nandbcb.c +++ b/arch/arm/mach-imx/cmd_nandbcb.c @@ -650,7 +650,7 @@ static int write_fcb(struct boot_config *boot_cfg, struct fcb_block *fcb) }; ret = mtd_write_oob(mtd, off, &ops); - printf("NAND FCB write to 0x%llxx offset 0x%zx written: %s\n", off, ops.len, ret ? "ERROR" : "OK"); + printf("NAND FCB write to 0x%llx offset 0x%zx written: %s\n", off, ops.len, ret ? "ERROR" : "OK"); } if (ret) -- 2.20.1

3 4

[PATCH] imx8mm-venice-gw7902: fix M2_RST# gpio pinmux
by Tim Harvey 05 Feb '22

05 Feb '22

Fix the invalid gw7902 M2_RST# gpio pinmux. Signed-off-by: Tim Harvey <tharvey(a)gateworks.com> --- arch/arm/dts/imx8mm-venice-gw7902.dts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/dts/imx8mm-venice-gw7902.dts b/arch/arm/dts/imx8mm-venice-gw7902.dts index 07e436be95..d37ffc050d 100644 --- a/arch/arm/dts/imx8mm-venice-gw7902.dts +++ b/arch/arm/dts/imx8mm-venice-gw7902.dts @@ -651,7 +651,7 @@ pinctrl_hog: hoggrp { fsl,pins = < MX8MM_IOMUXC_NAND_CE0_B_GPIO3_IO1 0x40000159 /* M2_GDIS# */ - MX8MM_IOMUXC_GPIO1_IO12_GPIO1_IO12 0x40000041 /* M2_RST# */ + MX8MM_IOMUXC_GPIO1_IO13_GPIO1_IO13 0x40000041 /* M2_RST# */ MX8MM_IOMUXC_NAND_DATA01_GPIO3_IO7 0x40000119 /* M2_OFF# */ MX8MM_IOMUXC_GPIO1_IO15_GPIO1_IO15 0x40000159 /* M2_WDIS# */ MX8MM_IOMUXC_SAI1_TXD2_GPIO4_IO14 0x40000041 /* AMP GPIO1 */ -- 2.17.1

3 2

[PATCH v2 0/3] Support SPL for i.MX7ULP
by Oleksandr Suvorov 05 Feb '22

05 Feb '22

This patchset adds SPL support for mx7ulp-based boards. Changes in v2: - add a patch with SPL support for mx7ulp_com board. Oleksandr Suvorov (1): mx7ulp: select soc features Ricardo Salveti (2): mx7ulp: add base SPL support for mx7ulp mx7ulp_com: add support for SPL arch/arm/Makefile | 2 +- arch/arm/dts/imx7ulp-com-u-boot.dtsi | 37 +++++++++++++++++++++ arch/arm/dts/imx7ulp-com.dts | 1 + arch/arm/mach-imx/Makefile | 2 +- arch/arm/mach-imx/mx7ulp/Kconfig | 15 +++++++++ arch/arm/mach-imx/mx7ulp/soc.c | 2 +- arch/arm/mach-imx/spl.c | 12 +++++-- board/ea/mx7ulp_com/mx7ulp_com.c | 26 +++++++++++++++ include/configs/imx7ulp_spl.h | 48 ++++++++++++++++++++++++++++ include/configs/mx7ulp_com.h | 6 ++++ 10 files changed, 145 insertions(+), 6 deletions(-) create mode 100644 arch/arm/dts/imx7ulp-com-u-boot.dtsi create mode 100644 include/configs/imx7ulp_spl.h -- 2.31.1

5 12

[PATCH] ARM: imx8m: support env in fat and ext4
by Ricardo Salveti 05 Feb '22

05 Feb '22

Change boot device logic to also allow environment stored in fat and in ext4 when booting from SD or from eMMC. As the boot device check for SD and for eMMC was depending on ENV_IS_IN_MMC being defined, change the ifdef blocks at env_get_location to use IS_ENABLED instead for all modes, returning NOWHERE when no valid mode is found. Signed-off-by: Ricardo Salveti <ricardo(a)foundries.io> --- arch/arm/mach-imx/imx8m/soc.c | 35 +++++++++++++++-------------------- 1 file changed, 15 insertions(+), 20 deletions(-) diff --git a/arch/arm/mach-imx/imx8m/soc.c b/arch/arm/mach-imx/imx8m/soc.c index f2ddc834d4b..a3ce517a104 100644 --- a/arch/arm/mach-imx/imx8m/soc.c +++ b/arch/arm/mach-imx/imx8m/soc.c @@ -1306,40 +1306,35 @@ void do_error(struct pt_regs *pt_regs, unsigned int esr) enum env_location env_get_location(enum env_operation op, int prio) { enum boot_device dev = get_boot_device(); - enum env_location env_loc = ENVL_UNKNOWN; if (prio) - return env_loc; + return ENVL_UNKNOWN; switch (dev) { -#ifdef CONFIG_ENV_IS_IN_SPI_FLASH case QSPI_BOOT: - env_loc = ENVL_SPI_FLASH; - break; -#endif -#ifdef CONFIG_ENV_IS_IN_NAND + if (IS_ENABLED(CONFIG_ENV_IS_IN_SPI_FLASH)) + return ENVL_SPI_FLASH; + return ENVL_NOWHERE; case NAND_BOOT: - env_loc = ENVL_NAND; - break; -#endif -#ifdef CONFIG_ENV_IS_IN_MMC + if (IS_ENABLED(CONFIG_ENV_IS_IN_NAND)) + return ENVL_NAND; + return ENVL_NOWHERE; case SD1_BOOT: case SD2_BOOT: case SD3_BOOT: case MMC1_BOOT: case MMC2_BOOT: case MMC3_BOOT: - env_loc = ENVL_MMC; - break; -#endif + if (IS_ENABLED(CONFIG_ENV_IS_IN_MMC)) + return ENVL_MMC; + else if (IS_ENABLED(CONFIG_ENV_IS_IN_EXT4)) + return ENVL_EXT4; + else if (IS_ENABLED(CONFIG_ENV_IS_IN_FAT)) + return ENVL_FAT; + return ENVL_NOWHERE; default: -#if defined(CONFIG_ENV_IS_NOWHERE) - env_loc = ENVL_NOWHERE; -#endif - break; + return ENVL_NOWHERE; } - - return env_loc; } #ifndef ENV_IS_EMBEDDED -- 2.33.0

4 3

[PATCH] arm: imx: imx8mq: add support to get values for more clocks
by Heiko Thiery 05 Feb '22

05 Feb '22

Return the root clock values for MXC_CSPI_CLK, MXC_I2C_CLK, MXC_UART_CLK and MXC_QSPI_CLK. At least for the I2C clock the missing support leads to a wrong configured I2C frequency. The expected value is 100kHz but the resulting value is about 1MHz. Signed-off-by: Heiko Thiery <heiko.thiery(a)gmail.com> --- More information and measurements about the wrong configure I2C frequency can be found here: https://lists.denx.de/pipermail/u-boot/2021-September/460176.html arch/arm/mach-imx/imx8m/clock_imx8mq.c | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/arch/arm/mach-imx/imx8m/clock_imx8mq.c b/arch/arm/mach-imx/imx8m/clock_imx8mq.c index 60e2218a3c..9db62b944e 100644 --- a/arch/arm/mach-imx/imx8m/clock_imx8mq.c +++ b/arch/arm/mach-imx/imx8m/clock_imx8mq.c @@ -359,10 +359,18 @@ unsigned int mxc_get_clock(enum mxc_clock clk) clock_get_target_val(IPG_CLK_ROOT, &val); val = val & 0x3; return get_root_clk(AHB_CLK_ROOT) / (val + 1); + case MXC_CSPI_CLK: + return get_root_clk(ECSPI1_CLK_ROOT); case MXC_ESDHC_CLK: return get_root_clk(USDHC1_CLK_ROOT); case MXC_ESDHC2_CLK: return get_root_clk(USDHC2_CLK_ROOT); + case MXC_I2C_CLK: + return get_root_clk(I2C1_CLK_ROOT); + case MXC_UART_CLK: + return get_root_clk(UART1_CLK_ROOT); + case MXC_QSPI_CLK: + return get_root_clk(QSPI_CLK_ROOT); default: return get_root_clk(clk); } -- 2.30.0

4 4