December 2019 - U-Boot

[PATCH V2] ARM: stm32: Allow overriding setup_mac_address()
by Marek Vasut 17 Jan '20

17 Jan '20

Let board code override setup_mac_address(), which is useful e.g. if the board derives the MAC address from another source, like an I2C EEPROM. Signed-off-by: Marek Vasut <marex(a)denx.de> Cc: Patrick Delaunay <patrick.delaunay(a)st.com> Cc: Patrice Chotard <patrice.chotard(a)st.com> --- V2: Add prototype to sys_proto.h --- arch/arm/mach-stm32mp/cpu.c | 2 +- arch/arm/mach-stm32mp/include/mach/sys_proto.h | 2 ++ 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/arch/arm/mach-stm32mp/cpu.c b/arch/arm/mach-stm32mp/cpu.c index ed7d9f61dc..856c7ced7a 100644 --- a/arch/arm/mach-stm32mp/cpu.c +++ b/arch/arm/mach-stm32mp/cpu.c @@ -449,7 +449,7 @@ static void setup_boot_mode(void) * If there is no MAC address in the environment, then it will be initialized * (silently) from the value in the OTP. */ -static int setup_mac_address(void) +__weak int setup_mac_address(void) { #if defined(CONFIG_NET) int ret; diff --git a/arch/arm/mach-stm32mp/include/mach/sys_proto.h b/arch/arm/mach-stm32mp/include/mach/sys_proto.h index 99eefabf6e..47e57922d1 100644 --- a/arch/arm/mach-stm32mp/include/mach/sys_proto.h +++ b/arch/arm/mach-stm32mp/include/mach/sys_proto.h @@ -30,3 +30,5 @@ u32 get_cpu_package(void); /* return boot mode */ u32 get_bootmode(void); + +int setup_mac_address(void); -- 2.24.1

2 2

[PATCH] stm32mp: remove redundant SYS_TEXT_BASE prompt
by Thomas Hebb 17 Jan '20

17 Jan '20

The main prompt for this (defined in /Kconfig) is visible at all times, which means there's no reason to have an additional, machine-specific prompt to set the same option. Signed-off-by: Thomas Hebb <tommyhebb(a)gmail.com> --- arch/arm/mach-stm32mp/Kconfig | 5 ----- 1 file changed, 5 deletions(-) diff --git a/arch/arm/mach-stm32mp/Kconfig b/arch/arm/mach-stm32mp/Kconfig index ae28f6e206..bf8a18a0c3 100644 --- a/arch/arm/mach-stm32mp/Kconfig +++ b/arch/arm/mach-stm32mp/Kconfig @@ -80,12 +80,7 @@ config STM32MP1_OPTEE OP-TEE monitor provides ST SMC to access to secure resources config SYS_TEXT_BASE - prompt "U-Boot base address" default 0xC0100000 - help - configure the U-Boot base address - when DDR driver is used: - DDR + 1MB (0xC0100000) config NR_DRAM_BANKS default 1 -- 2.24.1

2 2

[PATCH] clk: uclass: clk_get_by_name() must not be available if CONFIG_OF_PLATDATA is enabled
by Giulio Benetti 17 Jan '20

17 Jan '20

clk_get_by_name() requires clk_get_by_id() that is not available if CONFIG_OF_PLATDATA is defined, so move clk_get_by_name() into #else condition of #if CONFIG_IS_ENABLED(OF_PLATDATA). Signed-off-by: Giulio Benetti <giulio.benetti(a)benettiengineering.com> --- drivers/clk/clk-uclass.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/clk/clk-uclass.c b/drivers/clk/clk-uclass.c index 9aa8537004..170e19db65 100644 --- a/drivers/clk/clk-uclass.c +++ b/drivers/clk/clk-uclass.c @@ -326,7 +326,6 @@ int clk_set_defaults(struct udevice *dev, int stage) return 0; } -# endif /* OF_PLATDATA */ int clk_get_by_name(struct udevice *dev, const char *name, struct clk *clk) { @@ -343,6 +342,7 @@ int clk_get_by_name(struct udevice *dev, const char *name, struct clk *clk) return clk_get_by_index(dev, index, clk); } +# endif /* OF_PLATDATA */ int clk_release_all(struct clk *clk, int count) { -- 2.20.1

2 7

[PATCH 0/2] Enable private ATF handoff structure
by Michal Simek 17 Jan '20

17 Jan '20

Hi, Xilinx ZynqMP platform is using from the beggining specific structure which is passed from FSBL/SPL to ATF. Maybe in future both options can be supported by ATF and we can remove this private structure completely. But as of now bl31_params are completely ignored by ATF by RESET_TO_BL31=1 build flag. Thanks, Michal Michal Simek (2): spl: atf: Enable option to rewrite bl2_plat_get_bl31_params() arm: zynqmp: Wire SPL/ATF handoff structure properly arch/arm/mach-zynqmp/handoff.c | 17 +++++++++++----- arch/arm/mach-zynqmp/include/mach/sys_proto.h | 2 -- arch/arm/mach-zynqmp/spl.c | 2 -- common/spl/spl_atf.c | 15 +------------- include/spl.h | 20 +++++++++++++++++++ 5 files changed, 33 insertions(+), 23 deletions(-) -- 2.24.0

3 7

[RFC PATCH] Revert "arm: Show cache warnings in U-Boot proper only"
by Andre Przywara 16 Jan '20

16 Jan '20

According to commit 11aa6a32eb5f ("arm: cache: Implement cache range check for v7"), which introduced check_cache_range(), this was meant as a pure debugging feature, only to be compiled in when a developer explicitly #defined DEBUG in cache.c. Presumably the intention was to help with finding *certain* alignment issues with DMA buffers. Commit bcc53bf09589 ("arm: Show cache warnings in U-Boot proper only") compiled this in *unconditionally* into U-Boot proper. This has the annoying side effect of producing tons of somewhat pointless warnings about non-aligned clean&invalidate operations, which tend to be appeased by even more pointless rounding operations in many drivers (mostly those used on ARM boards). Bring back the old behaviour, of only compiling this in for DEBUG situations, but staying silent otherwise. This reverts commit bcc53bf095893fbdae531a9a7b5d4ef4a125a7fc. Signed-off-by: Andre Przywara <andre.przywara(a)arm.com> --- Hi, if the intention was indeed to always force cache maintenance range alignments, I would like to open a discussion on this, because I believe it is not useful, especially in the clean&invalidate case. Cheers, Andre. arch/arm/lib/cache.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/arm/lib/cache.c b/arch/arm/lib/cache.c index 007d4ebc49..f6c5c774cd 100644 --- a/arch/arm/lib/cache.c +++ b/arch/arm/lib/cache.c @@ -57,8 +57,8 @@ int check_cache_range(unsigned long start, unsigned long stop) ok = 0; if (!ok) { - warn_non_spl("CACHE: Misaligned operation at range [%08lx, %08lx]\n", - start, stop); + debug("CACHE: Misaligned operation at range [%08lx, %08lx]\n", + start, stop); } return ok; -- 2.14.5

4 9

[U-Boot] [PATCH] spl: fix entry_point equal to load_addr
by Giulio Benetti 16 Jan '20

16 Jan '20

At the moment entry_point is set to image_get_load(header) that sets it to "load address" instead of "entry point", assuming entry_point is equal to load_addr, but it's not true. Then load_addr is set to "entry_point - header_size", but this is wrong too since load_addr is not an entry point. So use image_get_ep() for entry_point assignment and image_get_load() for load_addr assignment. Signed-off-by: Giulio Benetti <giulio.benetti(a)benettiengineering.com> --- common/spl/spl.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/common/spl/spl.c b/common/spl/spl.c index f1ad8dc9da..cdae346753 100644 --- a/common/spl/spl.c +++ b/common/spl/spl.c @@ -261,9 +261,9 @@ int spl_parse_image_header(struct spl_image_info *spl_image, spl_image->entry_point = image_get_ep(header); spl_image->size = image_get_data_size(header); } else { - spl_image->entry_point = image_get_load(header); + spl_image->entry_point = image_get_ep(header); /* Load including the header */ - spl_image->load_addr = spl_image->entry_point - + spl_image->load_addr = image_get_load(header) - header_size; spl_image->size = image_get_data_size(header) + header_size; -- 2.20.1

4 17

[PATCH v2 0/8] Add support for MediaTek MT8512 Soc
by mingming lee 16 Jan '20

16 Jan '20

This patch series adds basic boot support on eMMC for the MediaTek MT8512 SoC based boards. This series add the clock, pinctrl drivers and the SoC initializaton code. --- Changes for v2: - Sort device nodes reg address order and alphabetically. - Split the changes in mtk clk for 8512 into two patchs. - Delete fix patch for clock-rate overflow in mtk clk since have been included in changelist 1207053 mingming lee (8): ARM: MediaTek: Add support for MediaTek MT8512 SoC clk: mediatek: add driver support for MT8512 clk: mediatek: add set_clr_upd mux type flow clk: mediatek: add configurable pcw_chg_reg/ibits/fmin to mtk_pll pinctrl: mediatek: add driver for MT8512 mmc: mtk-sd: add support for MediaTek MT8512/MT8110 SoCs mmc: mtk-sd: fix hang when data read quickly ARM: MediaTek: add basic support for MT8512 boards arch/arm/dts/Makefile | 1 + arch/arm/dts/mt8512-bm1-emmc.dts | 106 +++ arch/arm/dts/mt8512.dtsi | 115 +++ arch/arm/mach-mediatek/Kconfig | 15 + arch/arm/mach-mediatek/Makefile | 1 + arch/arm/mach-mediatek/mt8512/Makefile | 4 + arch/arm/mach-mediatek/mt8512/init.c | 78 ++ arch/arm/mach-mediatek/mt8512/lowlevel_init.S | 32 + board/mediatek/mt8512/Kconfig | 14 + board/mediatek/mt8512/MAINTAINERS | 6 + board/mediatek/mt8512/Makefile | 3 + board/mediatek/mt8512/mt8512.c | 19 + configs/mt8512_bm1_emmc_defconfig | 44 + drivers/clk/mediatek/Makefile | 1 + drivers/clk/mediatek/clk-mt8512.c | 873 ++++++++++++++++++ drivers/clk/mediatek/clk-mtk.c | 68 +- drivers/clk/mediatek/clk-mtk.h | 26 + drivers/mmc/mtk-sd.c | 140 ++- drivers/pinctrl/mediatek/Kconfig | 4 + drivers/pinctrl/mediatek/Makefile | 1 + drivers/pinctrl/mediatek/pinctrl-mt8512.c | 387 ++++++++ include/configs/mt8512.h | 60 ++ include/dt-bindings/clock/mt8512-clk.h | 197 ++++ 23 files changed, 2138 insertions(+), 57 deletions(-) create mode 100644 arch/arm/dts/mt8512-bm1-emmc.dts create mode 100644 arch/arm/dts/mt8512.dtsi create mode 100644 arch/arm/mach-mediatek/mt8512/Makefile create mode 100644 arch/arm/mach-mediatek/mt8512/init.c create mode 100644 arch/arm/mach-mediatek/mt8512/lowlevel_init.S create mode 100644 board/mediatek/mt8512/Kconfig create mode 100644 board/mediatek/mt8512/MAINTAINERS create mode 100644 board/mediatek/mt8512/Makefile create mode 100644 board/mediatek/mt8512/mt8512.c create mode 100644 configs/mt8512_bm1_emmc_defconfig create mode 100644 drivers/clk/mediatek/clk-mt8512.c create mode 100644 drivers/pinctrl/mediatek/pinctrl-mt8512.c create mode 100644 include/configs/mt8512.h create mode 100644 include/dt-bindings/clock/mt8512-clk.h -- 2.24.1

2 17

[PATCH 1/1] configs: qemu: enable FIT images on qemu_arm(64)_defconfig
by Heinrich Schuchardt 16 Jan '20

16 Jan '20

For testing UEFI FIT images we need FIT image support on QEMU. Cf. "test/py: Create a test for launching UEFI binaries from FIT images" https://lists.denx.de/pipermail/u-boot/2019-December/394516.html Signed-off-by: Heinrich Schuchardt <xypron.glpk(a)gmx.de> --- configs/qemu_arm64_defconfig | 5 +++++ configs/qemu_arm_defconfig | 5 +++++ 2 files changed, 10 insertions(+) diff --git a/configs/qemu_arm64_defconfig b/configs/qemu_arm64_defconfig index b7c320743a..50d0aa5bf5 100644 --- a/configs/qemu_arm64_defconfig +++ b/configs/qemu_arm64_defconfig @@ -6,6 +6,11 @@ CONFIG_NR_DRAM_BANKS=1 CONFIG_ENV_SECT_SIZE=0x40000 CONFIG_AHCI=y CONFIG_DISTRO_DEFAULTS=y +CONFIG_FIT=y +CONFIG_FIT_SIGNATURE=y +CONFIG_FIT_VERBOSE=y +CONFIG_FIT_BEST_MATCH=y +CONFIG_LEGACY_IMAGE_FORMAT=y CONFIG_USE_PREBOOT=y CONFIG_PREBOOT="pci enum" # CONFIG_DISPLAY_CPUINFO is not set diff --git a/configs/qemu_arm_defconfig b/configs/qemu_arm_defconfig index 521c6793a3..ff97d6bd83 100644 --- a/configs/qemu_arm_defconfig +++ b/configs/qemu_arm_defconfig @@ -7,6 +7,11 @@ CONFIG_NR_DRAM_BANKS=1 CONFIG_ENV_SECT_SIZE=0x40000 CONFIG_AHCI=y CONFIG_DISTRO_DEFAULTS=y +CONFIG_FIT=y +CONFIG_FIT_SIGNATURE=y +CONFIG_FIT_VERBOSE=y +CONFIG_FIT_BEST_MATCH=y +CONFIG_LEGACY_IMAGE_FORMAT=y CONFIG_USE_PREBOOT=y CONFIG_PREBOOT="pci enum" # CONFIG_DISPLAY_CPUINFO is not set -- 2.24.1

2 1

[PATCH] cmd/blk_common: clarify no partition error message
by Alexandre Besnard 16 Jan '20

16 Jan '20

When no partition table is found, users should be warned so. Warning that no device is available in this case could be misleading, especially as it is the same error when no device is selected. Signed-off-by: Alexandre Besnard <alexandre.besnard(a)softathome.com> --- cmd/blk_common.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/cmd/blk_common.c b/cmd/blk_common.c index c5514cf8f8..cc6e161ba0 100644 --- a/cmd/blk_common.c +++ b/cmd/blk_common.c @@ -32,7 +32,8 @@ int blk_common_cmd(int argc, char * const argv[], enum if_type if_type, return 0; } else if (strncmp(argv[1], "part", 4) == 0) { if (blk_list_part(if_type)) - printf("\nno %s devices available\n", if_name); + printf("\nno %s partition table available\n", + if_name); return 0; } return CMD_RET_USAGE; -- 2.24.1

2 1

[PATCH 1/1] test/py: use valid device tree in test_fit.py
by Heinrich Schuchardt 16 Jan '20

16 Jan '20

The device tree compiler expects that a node with a unit-address has a reg property. Signed-off-by: Heinrich Schuchardt <xypron.glpk(a)gmx.de> --- test/py/tests/test_fit.py | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/test/py/tests/test_fit.py b/test/py/tests/test_fit.py index 356d9a20f2..84b3f95850 100755 --- a/test/py/tests/test_fit.py +++ b/test/py/tests/test_fit.py @@ -83,13 +83,16 @@ base_fdt = ''' /dts-v1/; / { - model = "Sandbox Verified Boot Test"; - compatible = "sandbox"; + #address-cells = <1>; + #size-cells = <0>; + + model = "Sandbox Verified Boot Test"; + compatible = "sandbox"; reset@0 { compatible = "sandbox,reset"; + reg = <0>; }; - }; ''' -- 2.24.0

3 2