February 2020 - U-Boot

imx8m: composite clock wrong clock parent ?
by Sébastien Szymanski 18 Feb '20

18 Feb '20

Hi, I am working on a i.MX8MM based board and I got issue to make the Ethernet phy work. I noticed that the MDIO clock is not configured properly and the FEC outputs a 26,6MHz MDIO clock. That's because the clk_get_rate() call at line 1389 returns 24MHz whereas it should returns 266MHz as it is set in the device tree and configured in the SoC: BIOS> md.l 0x30388880 1 30388880: 11000000 With some debug traces we can see that the enet_axi's parent is wrong: fecmxc_probe 1388 clk_get_rate 447: clk clock-controller@30380000 clk_get_rate 447: clk enet1_root_clk clk_get_rate 447: clk enet_axi clk_get_parent_rate 489: clk enet_axi parent clock-osc-24m clk_get_parent_rate 489: clk enet1_root_clk parent enet_axi fecmxc_probe 1390 Is this a known issue ? Regards, -- Sébastien Szymanski, Armadeus Systems Software engineer

3 2

[PATCH V2 1/2] ARM: socfpga: Permit overriding the default timer frequency
by Marek Vasut 18 Feb '20

18 Feb '20

The default timer rate may be different than 25 MHz, permit overriding the default rate in board configuration file. Ultimatelly, this should be properly handled by a clock driver, however that is not available on Gen5 yet. Signed-off-by: Marek Vasut <marex(a)denx.de> Cc: Ley Foon Tan <ley.foon.tan(a)intel.com> Cc: Simon Goldschmidt <simon.k.r.goldschmidt(a)gmail.com> --- V2: Drop misleading comment --- include/configs/socfpga_common.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/include/configs/socfpga_common.h b/include/configs/socfpga_common.h index 8d10469e7c..54a43569dc 100644 --- a/include/configs/socfpga_common.h +++ b/include/configs/socfpga_common.h @@ -94,12 +94,13 @@ * L4 OSC1 Timer 0 */ #ifndef CONFIG_TIMER -/* This timer uses eosc1, whose clock frequency is fixed at any condition. */ #define CONFIG_SYS_TIMERBASE SOCFPGA_OSC1TIMER0_ADDRESS #define CONFIG_SYS_TIMER_COUNTS_DOWN #define CONFIG_SYS_TIMER_COUNTER (CONFIG_SYS_TIMERBASE + 0x4) +#ifndef CONFIG_SYS_TIMER_RATE #define CONFIG_SYS_TIMER_RATE 25000000 #endif +#endif /* * L4 Watchdog -- 2.24.1

3 4

[PATCH] image.h: Change android_image_get_dtb* to use uint and not u32
by Tom Rini 17 Feb '20

17 Feb '20

The image.h header can be used fairly widely in U-Boot builds. We cannot use u32 here as it may be used in cases where we don't have that typedef available and don't want to expose it either. Use uint instead here. Cc: Eugeniu Rosca <roscaeugeniu(a)gmail.com> Cc: Sam Protsenko <joe.skb7(a)gmail.com> Signed-off-by: Tom Rini <trini(a)konsulko.com> --- include/image.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/include/image.h b/include/image.h index b316d167d8d7..1dc3b48d8689 100644 --- a/include/image.h +++ b/include/image.h @@ -1425,9 +1425,9 @@ int android_image_get_ramdisk(const struct andr_img_hdr *hdr, ulong *rd_data, ulong *rd_len); int android_image_get_second(const struct andr_img_hdr *hdr, ulong *second_data, ulong *second_len); -bool android_image_get_dtbo(ulong hdr_addr, ulong *addr, u32 *size); -bool android_image_get_dtb_by_index(ulong hdr_addr, u32 index, ulong *addr, - u32 *size); +bool android_image_get_dtbo(ulong hdr_addr, ulong *addr, uint *size); +bool android_image_get_dtb_by_index(ulong hdr_addr, uint index, ulong *addr, + uint *size); ulong android_image_get_end(const struct andr_img_hdr *hdr); ulong android_image_get_kload(const struct andr_img_hdr *hdr); ulong android_image_get_kcomp(const struct andr_img_hdr *hdr); -- 2.17.1

4 6

[PATCH v2 00/13] Add DM/DTS support for omap video driver
by Dario Binacchi 17 Feb '20

17 Feb '20

Changes: - Fix coding style - Refactor some parts of the driver - Add support for driver model and device-tree - Update the boards device-tree - Add device-tree binding documentation Changes for v2: - Use the GENMASK macro to manage bitfield mask - Move clock enabling sequence of LCD module to arch/arm/mach-omap2/ - Create a Kconfig symbol for CONFIG_AM335X_LCD - Move 'u-boot,dm-pre-reloc' property to *-u-boot.dtsi file for boards that support Linux - Add a patch to fix the pixel-per-line bitfield setting - Add a patch to fix building errors for brxre1 board in case of CONFIG_DM_VIDEO is enabled Changes in v2: - Use GENMASK macro too Dario Binacchi (13): video: omap: use BIT() and GENMASK() macros video: omap: add missing bitfield masks video: omap: fix coding style on use of spaces video: omap: fix bitfields order video: omap: rename LCD controller registers video: omap: fix debug message video: omap: add loop exit conditions to the dpll setup video: omap: create two routines to set the pixel clock rate video: omap: add support for DM/DTS arm: dts: am335x: add 'u-boot,dm-pre-reloc' to panel fdt: video: omap: add framebuffer and panel bindings video: omap: fix pixel-per-line bitfield setting board: brxre1: fix building errors arch/arm/dts/am335x-brppt1-mmc.dts | 2 + arch/arm/dts/am335x-brppt1-nand.dts | 2 + arch/arm/dts/am335x-brppt1-spi.dts | 2 + arch/arm/dts/am335x-brsmarc1.dts | 1 + arch/arm/dts/am335x-brxre1.dts | 2 + arch/arm/dts/am335x-evm-u-boot.dtsi | 4 + arch/arm/dts/am335x-evm.dts | 2 +- arch/arm/dts/am335x-evmsk-u-boot.dtsi | 10 + arch/arm/dts/am335x-evmsk.dts | 2 +- arch/arm/dts/am335x-guardian-u-boot.dtsi | 4 + arch/arm/dts/am335x-guardian.dts | 2 +- arch/arm/dts/am335x-pdu001-u-boot.dtsi | 4 + arch/arm/dts/am335x-pdu001.dts | 2 +- arch/arm/dts/am335x-pxm50-u-boot.dtsi | 10 + arch/arm/dts/am335x-pxm50.dts | 2 +- arch/arm/dts/am335x-rut-u-boot.dtsi | 10 + arch/arm/dts/am335x-rut.dts | 2 +- arch/arm/dts/da850-evm-u-boot.dtsi | 4 + arch/arm/dts/da850-evm.dts | 2 +- arch/arm/mach-omap2/am33xx/clock_am33xx.c | 4 + board/BuR/common/br_resetc.c | 2 +- board/BuR/common/bur_common.h | 3 + board/BuR/common/common.c | 2 +- .../video/tilcdc/panel.txt | 66 ++ .../video/tilcdc/tilcdc.txt | 82 +++ drivers/video/Kconfig | 6 + drivers/video/am335x-fb.c | 567 +++++++++++++++--- drivers/video/am335x-fb.h | 16 +- include/configs/brxre1.h | 2 + 29 files changed, 712 insertions(+), 107 deletions(-) create mode 100644 arch/arm/dts/am335x-evmsk-u-boot.dtsi create mode 100644 arch/arm/dts/am335x-pxm50-u-boot.dtsi create mode 100644 arch/arm/dts/am335x-rut-u-boot.dtsi create mode 100644 doc/device-tree-bindings/video/tilcdc/panel.txt create mode 100644 doc/device-tree-bindings/video/tilcdc/tilcdc.txt -- 2.24.0

5 17

Fwd: use invd instead of wbinvd in real mode start code
by Andy Shevchenko 17 Feb '20

17 Feb '20

---------- Forwarded message --------- From: Andy Shevchenko <andy.shevchenko(a)gmail.com> Date: Mon, Feb 17, 2020 at 2:31 PM Subject: use invd instead of wbinvd in real mode start code To: Masahiro Yamada <masahiroy(a)kernel.org>, Simon Glass <sjg(a)chromium.org>, Bin Meng <bmeng.cn(a)gmail.com> Hi! It seems Masahiro's patches (don't know yet which one out of two, probably invd one) broke the boot on Intel Edison. Reverting (both for now) helps. Please, revert or fix ASAP before v2020.04 release! P.S. I dunno how it has been tested, so, if you have Intel Edison in possession, please, don't forget to test on it. It's not first time the Intel Edison behaviour is broken due to poor testing. -- With Best Regards, Andy Shevchenko -- With Best Regards, Andy Shevchenko

1 1

[PATCH] rtc: m41t62: add compatible for m41st87
by Marek Vasut 17 Feb '20

17 Feb '20

This adds a compatible string for m41st87. This ensures that this driver can be used for m41st87 . Signed-off-by: Marek Vasut <marex(a)denx.de> Cc: Simon Goldschmidt <simon.k.r.goldschmidt(a)gmail.com> Cc: Stefan Roese <sr(a)denx.de> --- drivers/rtc/m41t62.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/rtc/m41t62.c b/drivers/rtc/m41t62.c index 6161b76712..f52e1291a9 100644 --- a/drivers/rtc/m41t62.c +++ b/drivers/rtc/m41t62.c @@ -173,6 +173,7 @@ static const struct rtc_ops m41t62_rtc_ops = { static const struct udevice_id m41t62_rtc_ids[] = { { .compatible = "st,m41t62" }, { .compatible = "st,m41t82" }, + { .compatible = "st,m41st87" }, { .compatible = "microcrystal,rv4162" }, { } }; -- 2.25.0

2 1

[PATCH 1/1] arm: mvebu: drivers/ddr: remove redundant assignment
by Heinrich Schuchardt 17 Feb '20

17 Feb '20

The value of local variable ecc is immediately overwritten. So we can remove the first assignment. Signed-off-by: Heinrich Schuchardt <xypron.glpk(a)gmx.de> --- drivers/ddr/marvell/axp/ddr3_init.c | 4 ---- 1 file changed, 4 deletions(-) diff --git a/drivers/ddr/marvell/axp/ddr3_init.c b/drivers/ddr/marvell/axp/ddr3_init.c index 13df9126e7..30ad5d4151 100644 --- a/drivers/ddr/marvell/axp/ddr3_init.c +++ b/drivers/ddr/marvell/axp/ddr3_init.c @@ -434,10 +434,6 @@ static u32 ddr3_init_main(void) scrub_size = 0; #endif -#if defined(ECC_SUPPORT) && defined(AUTO_DETECTION_SUPPORT) - ecc = DRAM_ECC; -#endif - #if defined(ECC_SUPPORT) && defined(AUTO_DETECTION_SUPPORT) ecc = 0; if (ddr3_check_config(BUS_WIDTH_ECC_TWSI_ADDR, CONFIG_ECC)) -- 2.25.0

2 1

Support UBI u-boot
by JH 17 Feb '20

17 Feb '20

Hi, Sorry for cross-posting, I have been asking for helping solving the issue many days ago, but could not get those work. (1) UBI support I added NAND and UBIFS support to my Yocto layer machine config file: UBOOT_CONFIG = "nand" UBOOT_CONFIG[nand] += "mx6ull_14x14_evk_config" UBOOT_CONFIG[nand] += "mx6ull_14x14_evk_nand_config" UBOOT_CONFIG[nand] += "ubi" UBOOT_CONFIG[nand] += "ubifs" But my u-boot.imx-nand still could not support ubi: => ubi part rootfs Unknown command 'ubi' - try 'help' What I could be missing here? (2) Patch u-boot parameters I am not clear how you do to change u-boot parameters, it seems to me I have to create my own u-boot-imx_2017.03.bbappend and patch files to mx6ullevk.h, to change u-boot parameters of mtdparts, NAND_ROOT_UBI, bootcmd, etc based on my configuration. But Yocto won't allow the patch running unless that the patch is committed to the git repository in https://source.codeaurora.org/external/imx/uboot-imx.git. Ideally, I thought all u-boot parameters can be changed in a bbappend file, could you advise if it is possible to do that way or not? If so, how to do it? If not, what is the policy for making parches and commit to the git repository https://source.codeaurora.org/external/imx/uboot-imx.git? Thank you. Kind regards, - jh

4 12

[PATCH] {Makefile,config.mk,a/Kconfig}: introduce SUPPLIER
by Jens Rehsack 17 Feb '20

17 Feb '20

From: Jens Rehsack <sno(a)NetBSD.org> Introduce SUPPLIER analogous to VENDOR to allow (from customer perspective) a VENDOR using it's SUPPLIER's common/ code. This is reasonable, when a VENDOR (from customer perspective) builds several machines sharing some features (e.g. some FPGA which has to be initialized during u-boot) but wants to use common NXP or Samsung code for the BSP instead of copying and create merge overhead. Signed-off-by: Jens Rehsack <sno(a)NetBSD.org> --- Makefile | 4 +++- arch/Kconfig | 12 ++++++++++++ config.mk | 6 +++++- 3 files changed, 20 insertions(+), 2 deletions(-) diff --git a/Makefile b/Makefile index 0af89e0a78..94ef1e5003 100644 --- a/Makefile +++ b/Makefile @@ -427,7 +427,7 @@ UBOOTRELEASE = $(shell cat include/config/uboot.release 2> /dev/null) UBOOTVERSION = $(VERSION)$(if $(PATCHLEVEL),.$(PATCHLEVEL)$(if $(SUBLEVEL),.$(SUBLEVEL)))$(EXTRAVERSION) export VERSION PATCHLEVEL SUBLEVEL UBOOTRELEASE UBOOTVERSION -export ARCH CPU BOARD VENDOR SOC CPUDIR BOARDDIR +export ARCH CPU BOARD VENDOR SUPPLIER SOC CPUDIR BOARDDIR export CONFIG_SHELL HOSTCC HOSTCFLAGS HOSTLDFLAGS CROSS_COMPILE AS LD CC export CPP AR NM LDR STRIP OBJCOPY OBJDUMP export MAKE LEX YACC AWK PERL PYTHON PYTHON2 PYTHON3 @@ -721,9 +721,11 @@ c_flags := $(KBUILD_CFLAGS) $(cpp_flags) # U-Boot objects....order is important (i.e. start must be first) HAVE_VENDOR_COMMON_LIB = $(if $(wildcard $(srctree)/board/$(VENDOR)/common/Makefile),y,n) +HAVE_SUPPLIER_COMMON_LIB = $(if $(wildcard $(srctree)/board/$(SUPPLIER)/common/Makefile),y,n) libs-y += lib/ libs-$(HAVE_VENDOR_COMMON_LIB) += board/$(VENDOR)/common/ +libs-$(HAVE_SUPPLIER_COMMON_LIB) += board/$(SUPPLIER)/common/ libs-$(CONFIG_OF_EMBED) += dts/ libs-y += fs/ libs-y += net/ diff --git a/arch/Kconfig b/arch/Kconfig index ae9c93ed7b..512ba7c0ac 100644 --- a/arch/Kconfig +++ b/arch/Kconfig @@ -248,6 +248,18 @@ config SYS_SOC This is optional. For those targets without the SoC directory, leave this option empty. +config SYS_SUPPLIER + string + help + This option should contain the supplier name of the target soc. + One could even distinguish between SYS_SUPPLIER and SYS_VENDOR as + SYS_SOC_VENDOR and SYS_BOARD_VENDOR. If it is set and + board/<CONFIG_SYS_SUPPLIER>/common/Makefile exists, the supplier + common directory is compiled. + + This is optional. For those targets without the vendor directory, + leave this option empty. + config SYS_VENDOR string help diff --git a/config.mk b/config.mk index caf0dd9b81..993d5a2b40 100644 --- a/config.mk +++ b/config.mk @@ -17,8 +17,9 @@ LDFLAGS := LDFLAGS_FINAL := LDFLAGS_STANDALONE := OBJCOPYFLAGS := -# clear VENDOR for tcsh +# clear VENDOR & SUPPLIER for tcsh VENDOR := +SUPPLIER := ######################################################################### ARCH := $(CONFIG_SYS_ARCH:"%"=%) @@ -32,6 +33,9 @@ BOARD := $(CONFIG_SYS_BOARD:"%"=%) ifneq ($(CONFIG_SYS_VENDOR),) VENDOR := $(CONFIG_SYS_VENDOR:"%"=%) endif +ifneq ($(CONFIG_SYS_SUPPLIER),) +SUPPLIER := $(CONFIG_SYS_SUPPLIER:"%"=%) +endif ifneq ($(CONFIG_SYS_SOC),) SOC := $(CONFIG_SYS_SOC:"%"=%) endif -- 2.17.1

6 15

[PATCH] ARM: zynq: Rename defconfig to be align with ZynqMP and Versal
by Michal Simek 17 Feb '20

17 Feb '20

From: Michal Simek <michal.simek(a)xilinx.com> Just cosmetic change before real switch. Signed-off-by: Michal Simek <michal.simek(a)xilinx.com> Signed-off-by: Michal Simek <monstr(a)monstr.eu> --- configs/{zynq_virt_defconfig => xilinx_zynq_virt_defconfig} | 0 1 file changed, 0 insertions(+), 0 deletions(-) rename configs/{zynq_virt_defconfig => xilinx_zynq_virt_defconfig} (100%) diff --git a/configs/zynq_virt_defconfig b/configs/xilinx_zynq_virt_defconfig similarity index 100% rename from configs/zynq_virt_defconfig rename to configs/xilinx_zynq_virt_defconfig -- 2.25.0

1 0