April 2024 - U-Boot

[PATCH v5 0/5] imx93: Conver to OF_UPSTREAM
by Peng Fan (OSS) 12 Apr '24

12 Apr '24

A few nodes were added to soc and board u-boot.dtsi(lpi2c, usbotg), those nodes could be dropped after upstream linux supports them. To support OF_UPSTREAM, a few driver changes are included. For TMU, still use U-Boot node, I will prepare a kernel update, then back to U-Boot support. Mathieu: please help test the boards you maintain when you have time. Thanks, Peng. To: Stefano Babic <sbabic(a)denx.de> To: Fabio Estevam <festevam(a)gmail.com> To: "NXP i.MX U-Boot Team" <uboot-imx(a)nxp.com> To: Mathieu Othacehe <m.othacehe(a)gmail.com> Cc: u-boot(a)lists.denx.de Signed-off-by: Peng Fan <peng.fan(a)nxp.com> Changes in v5: - Correct uart clk checking - Link to v4: https://lore.kernel.org/r/20240328-imx93-of-v2-v4-0-338d15a65c36@nxp.com Changes in v4: - Convert all i.MX93 boards - Link to v3: https://lore.kernel.org/r/20240328-imx93-of-v3-0-4e7f341ed7ea@nxp.com Changes in v3: - Update patch 5, to drop the imx8mp-evk changes which are wrongly included - Link to v2: https://lore.kernel.org/r/20240328-imx93-of-v2-0-909f5d37da87@nxp.com Changes in v2: - Add a new patch to sync clock header to avoid breaking - Drop the Makefile change which change including order - Link to v1: https://lore.kernel.org/r/20240327-imx93-of-v1-0-afab6b31422a@nxp.com --- Peng Fan (5): gpio: imx_rgpio2p: support one address serial: lpuart: use ipg clk for i.MX7ULP cpu: drop imx9_cpu clk: imx93: fix anatop base imx93: convert to OF_UPSTREAM arch/arm/dts/Makefile | 5 - arch/arm/dts/imx93-11x11-evk-u-boot.dtsi | 119 ++++ arch/arm/dts/imx93-11x11-evk.dts | 322 --------- arch/arm/dts/imx93-phyboard-segin.dts | 117 --- arch/arm/dts/imx93-phycore-som.dtsi | 126 ---- arch/arm/dts/imx93-pinfunc.h | 623 ---------------- arch/arm/dts/imx93-u-boot.dtsi | 80 +++ arch/arm/dts/imx93-var-som-symphony-u-boot.dtsi | 22 + arch/arm/dts/imx93-var-som-symphony.dts | 323 --------- arch/arm/dts/imx93-var-som.dtsi | 111 --- arch/arm/dts/imx93.dtsi | 906 ------------------------ arch/arm/mach-imx/imx9/Kconfig | 1 + configs/imx93-phyboard-segin_defconfig | 2 +- configs/imx93_11x11_evk_defconfig | 2 +- configs/imx93_11x11_evk_ld_defconfig | 2 +- configs/imx93_var_som_defconfig | 2 +- drivers/clk/imx/clk-imx93.c | 2 +- drivers/cpu/imx9_cpu.c | 224 ------ drivers/gpio/imx_rgpio2p.c | 42 +- drivers/serial/serial_lpuart.c | 28 +- include/dt-bindings/clock/imx93-clock.h | 208 ------ include/dt-bindings/power/fsl,imx93-power.h | 15 - 22 files changed, 284 insertions(+), 2998 deletions(-) --- base-commit: 777c28460947371ada40868dc994dfe8537d7115 change-id: 20240328-imx93-of-v2-f879efef737d Best regards, -- Peng Fan <peng.fan(a)nxp.com>

3 7

[PATCH 0/3] arm: Add Analog Devices SC5xx Machine Type
by Greg Malysa 12 Apr '24

12 Apr '24

This series adds support for the ADI SC5xx machine type and includes two core drivers that are required for being able to boot any board--a UART driver and the clock tree driver. Our corresponding Linux support relies on u-boot configuring the clocks correctly before booting, so it is not possible to boot any board without the CGU/CDU configuration happening here. The clock tree itself is only used by the UART in this minimal patch set, but is of course broadly useful for all other peripheral drivers to be submitted in future patch sets. There are also no board files or defconfigs included here, but some common definitions that will be used to build board files currently are. Some of the configuration code in dmcinit and clkinit is quite scary and causes a lot of checkpatch violations. It is modified from code initially provided by ADI, but it has not been fully rewritten. There's a question of how important it is to clean up this code--it has some quality violations, but it has been in use (including in production) for over two years and is known to work for performing the low level SoC initialization, while a rewrite might introduce bugs that could take a significant amount of time to detect in the future. Please provide any feedback or comments that will be helpful in creating a v2 of this patch that will be ready to submit for inclusion into the main tree, as well as general observations that we should consider in other driver patches before submitting them. Thank you! Nathan Barrett-Morrison (3): arch: arm: Add Analog Devices SC5xx machine type drivers: clk: adi: Add in SC5XX-family clock driver drivers: serial: Add in UART for ADI SC5XX-family processors MAINTAINERS | 15 + arch/arm/Kconfig | 6 + arch/arm/Makefile | 1 + arch/arm/include/asm/arch-adi/sc5xx/sc5xx.h | 115 +++ arch/arm/include/asm/arch-adi/sc5xx/soc.h | 18 + arch/arm/include/asm/arch-adi/sc5xx/spl.h | 41 + arch/arm/include/asm/mach-types.h | 4 + arch/arm/mach-sc5xx/Kconfig | 464 +++++++++ arch/arm/mach-sc5xx/Makefile | 19 + arch/arm/mach-sc5xx/config.mk | 21 + arch/arm/mach-sc5xx/init/Makefile | 11 + arch/arm/mach-sc5xx/init/clkinit.c | 543 +++++++++++ arch/arm/mach-sc5xx/init/clkinit.h | 18 + arch/arm/mach-sc5xx/init/dmcinit.c | 973 +++++++++++++++++++ arch/arm/mach-sc5xx/init/dmcinit.h | 29 + arch/arm/mach-sc5xx/init/init.c | 68 ++ arch/arm/mach-sc5xx/init/init.h | 37 + arch/arm/mach-sc5xx/init/mem/is43tr16512bl.h | 63 ++ arch/arm/mach-sc5xx/init/mem/mt41k128m16jt.h | 51 + arch/arm/mach-sc5xx/init/mem/mt41k512m16ha.h | 51 + arch/arm/mach-sc5xx/init/mem/mt47h128m16rt.h | 50 + arch/arm/mach-sc5xx/rcu.c | 22 + arch/arm/mach-sc5xx/sc57x.c | 21 + arch/arm/mach-sc5xx/sc58x.c | 21 + arch/arm/mach-sc5xx/sc59x.c | 32 + arch/arm/mach-sc5xx/sc59x_64.c | 36 + arch/arm/mach-sc5xx/soc.c | 142 +++ arch/arm/mach-sc5xx/spl.c | 140 +++ drivers/clk/Kconfig | 1 + drivers/clk/Makefile | 1 + drivers/clk/adi/Kconfig | 83 ++ drivers/clk/adi/Makefile | 16 + drivers/clk/adi/clk-adi-pll.c | 94 ++ drivers/clk/adi/clk-adi-sc57x.c | 205 ++++ drivers/clk/adi/clk-adi-sc58x.c | 221 +++++ drivers/clk/adi/clk-adi-sc594.c | 230 +++++ drivers/clk/adi/clk-adi-sc598.c | 307 ++++++ drivers/clk/adi/clk-shared.c | 48 + drivers/clk/adi/clk.h | 122 +++ drivers/serial/Makefile | 1 + drivers/serial/serial_adi_uart4.c | 228 +++++ include/configs/sc_adi_common.h | 226 +++++ include/dt-bindings/clock/adi-sc5xx-clock.h | 271 ++++++ 43 files changed, 5066 insertions(+) create mode 100644 arch/arm/include/asm/arch-adi/sc5xx/sc5xx.h create mode 100644 arch/arm/include/asm/arch-adi/sc5xx/soc.h create mode 100644 arch/arm/include/asm/arch-adi/sc5xx/spl.h create mode 100644 arch/arm/mach-sc5xx/Kconfig create mode 100644 arch/arm/mach-sc5xx/Makefile create mode 100644 arch/arm/mach-sc5xx/config.mk create mode 100644 arch/arm/mach-sc5xx/init/Makefile create mode 100644 arch/arm/mach-sc5xx/init/clkinit.c create mode 100644 arch/arm/mach-sc5xx/init/clkinit.h create mode 100644 arch/arm/mach-sc5xx/init/dmcinit.c create mode 100644 arch/arm/mach-sc5xx/init/dmcinit.h create mode 100644 arch/arm/mach-sc5xx/init/init.c create mode 100644 arch/arm/mach-sc5xx/init/init.h create mode 100644 arch/arm/mach-sc5xx/init/mem/is43tr16512bl.h create mode 100644 arch/arm/mach-sc5xx/init/mem/mt41k128m16jt.h create mode 100644 arch/arm/mach-sc5xx/init/mem/mt41k512m16ha.h create mode 100644 arch/arm/mach-sc5xx/init/mem/mt47h128m16rt.h create mode 100644 arch/arm/mach-sc5xx/rcu.c create mode 100644 arch/arm/mach-sc5xx/sc57x.c create mode 100644 arch/arm/mach-sc5xx/sc58x.c create mode 100644 arch/arm/mach-sc5xx/sc59x.c create mode 100644 arch/arm/mach-sc5xx/sc59x_64.c create mode 100644 arch/arm/mach-sc5xx/soc.c create mode 100644 arch/arm/mach-sc5xx/spl.c create mode 100644 drivers/clk/adi/Kconfig create mode 100644 drivers/clk/adi/Makefile create mode 100644 drivers/clk/adi/clk-adi-pll.c create mode 100644 drivers/clk/adi/clk-adi-sc57x.c create mode 100644 drivers/clk/adi/clk-adi-sc58x.c create mode 100644 drivers/clk/adi/clk-adi-sc594.c create mode 100644 drivers/clk/adi/clk-adi-sc598.c create mode 100644 drivers/clk/adi/clk-shared.c create mode 100644 drivers/clk/adi/clk.h create mode 100644 drivers/serial/serial_adi_uart4.c create mode 100644 include/configs/sc_adi_common.h create mode 100644 include/dt-bindings/clock/adi-sc5xx-clock.h -- 2.43.2

3 8

Re: [U-Boot] [PATCH] mmc: dw-mmc: support DesignWare MMC controller
by 18304935041 12 Apr '24

12 Apr '24

Dear Mr. Jaehoon Chung, Hello! I am learning to write an mmc driver, I saw this driver code you wrote in 2012 in U-boot source code, I can't download the documentation of DesignWare MMC controller, so I can't work on my driver development. I hope you can provide some help! Have a nice life!

1 0

[PATCH 0/7] qcom: mmc fixes and sdm845 support
by Caleb Connolly 12 Apr '24

12 Apr '24

This series does some long overdue cleanup to the msm_sdhci driver, fixes v5 support, and adds the necessary clock configuration to get the sdcard up and running on the RB3. --- Caleb Connolly (7): mmc: msm_sdhci: correct vendor_spec_cap0 register for v5 mmc: msm_sdhci: use modern DT handling mmc: msm_sdhci: print core version mmc: msm_sdhci: use a more sensible default clock rate clk/qcom: sdm845: enable SDCC2 core clock pinctrl: qcom: sdm845: add special pin names dts: sdm845-db845c-u-boot: adjust MMC clocks arch/arm/dts/sdm845-db845c-u-boot.dtsi | 7 ++++++ drivers/clk/qcom/clock-qcom.h | 1 + drivers/clk/qcom/clock-sdm845.c | 17 ++++++++++++++ drivers/mmc/msm_sdhci.c | 43 +++++++++++++++++++++++----------- drivers/pinctrl/qcom/pinctrl-sdm845.c | 13 +++++++++- 5 files changed, 66 insertions(+), 15 deletions(-) --- change-id: 20240409-b4-qcom-mmc-fixes-d30782746f4a base-commit: b40edaf6383b1494222ea9c043d7c1716d4d118f // Caleb (they/them)

3 18

[PATCH] net: ti: am65-cpsw: Fix buffer overflow
by Michael Walle 12 Apr '24

12 Apr '24

The device name is a concatenation of the device node name of the cpsw device and of the device node name of the port. In my case that is ethernet@8000000 port@1 First the buffer is really too small, but more importantly, there is no boundary check. Use snprintf() and increase the buffer size. Fixes: 38922b1f4acc ("net: ti: am65-cpsw: Add support for multi port independent MAC mode") Signed-off-by: Michael Walle <mwalle(a)kernel.org> --- drivers/net/ti/am65-cpsw-nuss.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/ti/am65-cpsw-nuss.c b/drivers/net/ti/am65-cpsw-nuss.c index d68ed671836..b151e25d6a4 100644 --- a/drivers/net/ti/am65-cpsw-nuss.c +++ b/drivers/net/ti/am65-cpsw-nuss.c @@ -664,7 +664,7 @@ static int am65_cpsw_port_probe(struct udevice *dev) struct am65_cpsw_priv *priv = dev_get_priv(dev); struct eth_pdata *pdata = dev_get_plat(dev); struct am65_cpsw_common *cpsw_common; - char portname[15]; + char portname[32]; int ret; priv->dev = dev; @@ -672,7 +672,7 @@ static int am65_cpsw_port_probe(struct udevice *dev) cpsw_common = dev_get_priv(dev->parent); priv->cpsw_common = cpsw_common; - sprintf(portname, "%s%s", dev->parent->name, dev->name); + snprintf(portname, sizeof(portname), "%s%s", dev->parent->name, dev->name); device_set_name(dev, portname); ret = am65_cpsw_ofdata_parse_phy(dev); -- 2.39.2

2 1

[PATCH v1] arm: mach-k3: common: EFI loader map memory below ram top
by Vitor Soares 12 Apr '24

12 Apr '24

From: Vitor Soares <vitor.soares(a)toradex.com> During the boot, the EFI loader maps the memory from ram_top to ram_end as EFI_BOOT_SERVICES_DATA. When LMB does boot_fdt_add_mem_rsv_regions() to OPTEE, TFA, R5, and M4F DMA/memory "no-map" for the kernel it produces the following error message: ERROR: reserving fdt memory region failed (addr=9cb00000 size=100000 flags=4) ERROR: reserving fdt memory region failed (addr=9cc00000 size=e00000 flags=4) ERROR: reserving fdt memory region failed (addr=9da00000 size=100000 flags=4) ERROR: reserving fdt memory region failed (addr=9db00000 size=c00000 flags=4) ERROR: reserving fdt memory region failed (addr=9e780000 size=80000 flags=4) ERROR: reserving fdt memory region failed (addr=9e800000 size=1800000 flags=4) To avoid this, don't flag with EFI_BOOT_SERVICES_DATA the memory from ram_top to ram_end by the EFI loader. Signed-off-by: Vitor Soares <vitor.soares(a)toradex.com> --- Similar fix for the same issue can be found at https://lore.kernel.org/all/20240308101230.2595220-1-patrice.chotard@foss.s… arch/arm/mach-k3/common.c | 12 ++++++++++++ 1 file changed, 12 insertions(+) diff --git a/arch/arm/mach-k3/common.c b/arch/arm/mach-k3/common.c index b54f2ddc47..80e89b2a16 100644 --- a/arch/arm/mach-k3/common.c +++ b/arch/arm/mach-k3/common.c @@ -24,6 +24,7 @@ #include <asm/io.h> #include <fs_loader.h> #include <fs.h> +#include <efi_loader.h> #include <env.h> #include <elf.h> #include <soc.h> @@ -607,3 +608,14 @@ void setup_qos(void) writel(qos_data[i].val, (uintptr_t)qos_data[i].reg); } #endif + +void efi_add_known_memory(void) +{ + if (IS_ENABLED(CONFIG_EFI_LOADER)) + /* + * Memory over ram_top can be used by various firmware + * Declare to EFI only memory area below ram_top + */ + efi_add_memory_map(gd->ram_base, gd->ram_top - gd->ram_base, + EFI_CONVENTIONAL_MEMORY); +} -- 2.34.1

2 1

[PATCH] am625x_evm_a53: Tweak boot command to set fdt
by Martyn Welch 12 Apr '24

12 Apr '24

With the current config for tha SK-AM62, fdtfile isn't set in the U-Boot environment. When using bootflow to boot from a block device, where the extlinux.conf file specifies `fdtdir`, a fallback device tree is being constructed from the `soc` (`k3`) and `board` (`am62x`) environment variables, resulting in u-Boot trying to retrieve `/dtbs/6.8.1+/k3-am62x.dtb`. This file doesn't exist. The environment variables `default_device_tree` and `default_device_tree_arch` are set in the config, the `findfdt` script can be called to construct `fdtfile` from the environment variables set by these config options, however this script currently isn't being run. Calling this script results in the correct device tree being retrieved: Retrieving file: /dtbs/6.8.1+/ti/k3-am625-sk.dtb Many boards are calling this script as part of their boot command. The am62x currently isn't. Rectify this so that booting works correctly. Signed-off-by: Martyn Welch <martyn.welch(a)collabora.com> --- configs/am62x_evm_a53_defconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configs/am62x_evm_a53_defconfig b/configs/am62x_evm_a53_defconfig index 78287ec18a..e373e44a93 100644 --- a/configs/am62x_evm_a53_defconfig +++ b/configs/am62x_evm_a53_defconfig @@ -30,7 +30,7 @@ CONFIG_SPL_LOAD_FIT=y CONFIG_SPL_LOAD_FIT_ADDRESS=0x81000000 CONFIG_BOOTSTD_FULL=y CONFIG_SYS_BOOTM_LEN=0x800000 -CONFIG_BOOTCOMMAND="run envboot; bootflow scan -lb" +CONFIG_BOOTCOMMAND="run findfdt; run envboot; bootflow scan -lb" CONFIG_BOARD_LATE_INIT=y CONFIG_SPL_MAX_SIZE=0x58000 CONFIG_SPL_HAS_BSS_LINKER_SECTION=y -- 2.43.0

4 4

[PATCH] tools: binman: ti_board_cfg: improve error message
by Michael Walle 12 Apr '24

12 Apr '24

When there is a lint error the user gets the following cryptic message: binman: Node '/path/to/some/node': Yamllint error: 18: comments This isn't very helpful. Improve the message to tell the user that the number is actually a line number and also tell the user in which file they have to look. Signed-off-by: Michael Walle <mwalle(a)kernel.org> --- tools/binman/etype/ti_board_config.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/binman/etype/ti_board_config.py b/tools/binman/etype/ti_board_config.py index 2c3bb8f7b56..c10d66edcb1 100644 --- a/tools/binman/etype/ti_board_config.py +++ b/tools/binman/etype/ti_board_config.py @@ -248,7 +248,7 @@ class Entry_ti_board_config(Entry_section): yaml_config = config.YamlLintConfig("extends: default") for p in yamllint.linter.run(open(self._config_file, "r"), yaml_config): - self.Raise(f"Yamllint error: {p.line}: {p.rule}") + self.Raise(f"Yamllint error: Line {p.line} in {self._config_file}: {p.rule}") try: validate(self.file_yaml, self.schema_yaml) except Exception as e: -- 2.39.2

3 2

[PATCH] binman: ti-secure: Enable debug extension for combined boot
by Manorit Chawdhry 12 Apr '24

12 Apr '24

To debug using jtag, ROM needs to unlock jtag debugging on HS devices and it does that looking at this debug extension. Add the debug extension and enable it by default. Link: https://software-dl.ti.com/tisci/esd/latest/2_tisci_msgs/security/sec_cert_… Signed-off-by: Manorit Chawdhry <m-chawdhry(a)ti.com> --- Keeping this enabled by default as this is enabled for non-combined boot as well by default. --- tools/binman/btool/openssl.py | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/tools/binman/btool/openssl.py b/tools/binman/btool/openssl.py index fe81a1f51b1e..c6df64c5316d 100644 --- a/tools/binman/btool/openssl.py +++ b/tools/binman/btool/openssl.py @@ -283,6 +283,7 @@ emailAddress = {req_dist_name_dict['emailAddress']} basicConstraints = CA:true 1.3.6.1.4.1.294.1.3=ASN1:SEQUENCE:swrv 1.3.6.1.4.1.294.1.9=ASN1:SEQUENCE:ext_boot_info +1.3.6.1.4.1.294.1.8=ASN1:SEQUENCE:debug [swrv] swrv=INTEGER:{sw_rev} @@ -323,6 +324,12 @@ compSize = INTEGER:{imagesize_sysfw_data} shaType = OID:{sha_type} shaValue = FORMAT:HEX,OCT:{hashval_sysfw_data} +[ debug ] +debugUID = FORMAT:HEX,OCT:0000000000000000000000000000000000000000000000000000000000000000 +debugType = INTEGER:4 +coreDbgEn = INTEGER:0 +coreDbgSecEn = INTEGER:0 + {sysfw_inner_cert_ext_boot_block} {dm_data_ext_boot_block} --- base-commit: fb49d6c289d942ff7de309a5c5eaa37a7f4235db change-id: 20240326-b4-upstream-jtag-debug-unlock-6fc21e344cfb Best regards, -- Manorit Chawdhry <m-chawdhry(a)ti.com>

3 2

[PATCH 0/2] arm: mach-k3: am62: change a53 clock frequency by speed grade
by Joao Paulo Goncalves 12 Apr '24

12 Apr '24

From: Joao Paulo Goncalves <joao.goncalves(a)toradex.com> This series introduces a method to dynamically set the AM62 A53 CPU frequency based on its speed grade. It adds a new function to retrieve the A53 frequency value by reading the speed grade from the hardware. Subsequently, it adjusts the Cortex R5 device tree at runtime with the frequency value before initializing the remote processor. Regards, João Paulo Goncalves Joao Paulo Goncalves (2): arm: mach-k3: am62: Get a53 max cpu frequency by speed grade arm: mach-k3: am625: Fixup a53 cpu frequency by speed grade arch/arm/mach-k3/am625_init.c | 62 +++++++++++++++++++ arch/arm/mach-k3/include/mach/am62_hardware.h | 15 +++++ 2 files changed, 77 insertions(+) -- 2.34.1

2 3