November 2019 - U-Boot

[U-Boot] [PATCH] moveconfig.py: Fix more Python3 UTF issues
by Tom Rini 24 Nov '19

24 Nov '19

With the move to using Python 3 for real, we encounter two different issues. First, the file include/video_font_data.h includes at least one UTF-16 character. Given that it does not include any CONFIG symbols it is easiest to just ignore this file. Next, we encounter similar problems with some dts/dtsi files that come from Linux. In this case it's easiest to simply ignore all dts/dtsi files as there will not be CONFIG symbols for us to migrate in them. Cc: Masahiro Yamada <yamada.masahiro(a)socionext.com> Signed-off-by: Tom Rini <trini(a)konsulko.com> --- tools/moveconfig.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/tools/moveconfig.py b/tools/moveconfig.py index e2ff4cfc88b2..36160a397726 100755 --- a/tools/moveconfig.py +++ b/tools/moveconfig.py @@ -667,8 +667,11 @@ def cleanup_headers(configs, options): if dirpath == os.path.join('include', 'generated'): continue for filename in filenames: - if not fnmatch.fnmatch(filename, '*~'): + if not filename.endswith(('~', '.dts', '.dtsi')): header_path = os.path.join(dirpath, filename) + # This file contains UTF-16 data and no CONFIG symbols + if header_path == 'include/video_font_data.h': + continue cleanup_one_header(header_path, patterns, options) cleanup_empty_blocks(header_path, options) -- 2.17.1

1 1

[U-Boot] [PATCH] scripts: dtc: ignore files generated generated by python
by Bartosz Golaszewski 24 Nov '19

24 Nov '19

From: Bartosz Golaszewski <bgolaszewski(a)baylibre.com> Add __pycache__ to ignored files and extend the rule for _libfdt to also include generated shared objects (e.g. _libfdt.cpython-37m-x86_64-linux-gnu.so). Signed-off-by: Bartosz Golaszewski <bgolaszewski(a)baylibre.com> --- scripts/dtc/pylibfdt/.gitignore | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/scripts/dtc/pylibfdt/.gitignore b/scripts/dtc/pylibfdt/.gitignore index 033f23dfdd..3a512001c9 100644 --- a/scripts/dtc/pylibfdt/.gitignore +++ b/scripts/dtc/pylibfdt/.gitignore @@ -1,4 +1,5 @@ -/_libfdt.so +/_libfdt.* /libfdt.py /libfdt.pyc /libfdt_wrap.c +/__pycache__ -- 2.23.0

2 1

[U-Boot] [PATCH] buildman: Fix problem with non-existent output directories
by Tom Rini 24 Nov '19

24 Nov '19

Now that we have buildman telling genboards.cfg to use an output directory we need to ensure that it exists. Cc: Bin Meng <bmeng.cn(a)gmail.com> Cc: Simon Glass <sjg(a)chromium.org> Fixes: bc750bca1246 ("tools: buildman: Honor output directory when generating boards.cfg") Signed-off-by: Tom Rini <trini(a)konsulko.com> --- tools/buildman/control.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tools/buildman/control.py b/tools/buildman/control.py index 9787b8674761..5988ada72b75 100644 --- a/tools/buildman/control.py +++ b/tools/buildman/control.py @@ -201,6 +201,8 @@ def DoBuildman(options, args, toolchains=None, make_func=None, boards=None, # Work out what subset of the boards we are building if not boards: + if not os.path.exists(options.output_dir): + os.mkdir(options.output_dir) board_file = os.path.join(options.output_dir, 'boards.cfg') genboardscfg = os.path.join(options.git, 'tools/genboardscfg.py') status = subprocess.call([genboardscfg, '-o', board_file]) -- 2.17.1

2 5

[U-Boot] [PATCH v2 1/1] arm: fix -march for ARM11
by Heinrich Schuchardt 24 Nov '19

24 Nov '19

In GCC 9 support for the Armv5 and Armv5E architectures (which have no known implementations) has been removed, cf. https://gcc.gnu.org/gcc-9/changes.html ARM11 is an armv6 implementation. So change the architecture flag for the compiler to armv6 for ARM11. Suggested-by: Fabio Estevam <festevam(a)gmail.com> Signed-off-by: Heinrich Schuchardt <xypron.glpk(a)gmx.de> --- v2: change commit message --- arch/arm/Makefile | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/arm/Makefile b/arch/arm/Makefile index 5384981c17..60af7e3199 100644 --- a/arch/arm/Makefile +++ b/arch/arm/Makefile @@ -12,8 +12,8 @@ arch-$(CONFIG_CPU_ARM926EJS) =-march=armv5te arch-$(CONFIG_CPU_ARM946ES) =-march=armv5te arch-$(CONFIG_CPU_SA1100) =-march=armv4 arch-$(CONFIG_CPU_PXA) = -arch-$(CONFIG_CPU_ARM1136) =-march=armv5 -arch-$(CONFIG_CPU_ARM1176) =-march=armv5t +arch-$(CONFIG_CPU_ARM1136) =-march=armv6 +arch-$(CONFIG_CPU_ARM1176) =-march=armv6 arch-$(CONFIG_CPU_V7A) =$(call cc-option, -march=armv7-a, \ $(call cc-option, -march=armv7)) arch-$(CONFIG_CPU_V7M) =-march=armv7-m -- 2.24.0

3 2

[U-Boot] Pull request for UEFI sub-system for efi-2020-01-rc4 (2)
by Heinrich Schuchardt 24 Nov '19

24 Nov '19

The following changes since commit 2800540d101d7b0dd0629c5177fca48331d2927d: Update MAINTAINERS to include environment files (2019-11-21 08:36:40 -0500) are available in the Git repository at: https://gitlab.denx.de/u-boot/custodians/u-boot-efi.git tags/efi-2020-01-rc4-2 for you to fetch changes up to b7cdecfc196686ffae34b6849fc8b5b7ef62237a: efi_loader: default EFI_LOADER=n on ARM11 (2019-11-23 09:19:31 +0100) ---------------------------------------------------------------- Pull request for UEFI sub-system for efi-2020-01-rc4 (2) Use CONFIG_EFI_LOADER=n for ARM11 except for the Raspberry Pi and the Raspberry Pi Zero. ---------------------------------------------------------------- Heinrich Schuchardt (1): efi_loader: default EFI_LOADER=n on ARM11 configs/rpi_0_w_defconfig | 1 + configs/rpi_defconfig | 1 + lib/efi_loader/Kconfig | 2 +- 3 files changed, 3 insertions(+), 1 deletion(-)

2 1

[U-Boot] [PATCH 1/1] efi_loader: default EFI_LOADER=n on ARM11
by Heinrich Schuchardt 24 Nov '19

24 Nov '19

Some of the ARM11 boards have tight limits on the size of U-Boots. Hence use EFI_LOADER=n as default on ARM11. Set EFI_LOADER=y for the Raspberry Pi and Raspberry Pi Zero as these boards have sufficient storage on the SD card. Suggested-by: Tom Rini <trini(a)konsulko.com> Signed-off-by: Heinrich Schuchardt <xypron.glpk(a)gmx.de> --- configs/rpi_0_w_defconfig | 1 + configs/rpi_defconfig | 1 + lib/efi_loader/Kconfig | 2 +- 3 files changed, 3 insertions(+), 1 deletion(-) diff --git a/configs/rpi_0_w_defconfig b/configs/rpi_0_w_defconfig index fe5a7763a6..6614adfc9b 100644 --- a/configs/rpi_0_w_defconfig +++ b/configs/rpi_0_w_defconfig @@ -41,3 +41,4 @@ CONFIG_SYS_WHITE_ON_BLACK=y CONFIG_CONSOLE_SCROLL_LINES=10 CONFIG_PHYS_TO_BUS=y CONFIG_OF_LIBFDT_OVERLAY=y +CONFIG_EFI_LOADER=y diff --git a/configs/rpi_defconfig b/configs/rpi_defconfig index 2c04b3334e..fa4d9c0b04 100644 --- a/configs/rpi_defconfig +++ b/configs/rpi_defconfig @@ -41,3 +41,4 @@ CONFIG_SYS_WHITE_ON_BLACK=y CONFIG_CONSOLE_SCROLL_LINES=10 CONFIG_PHYS_TO_BUS=y CONFIG_OF_LIBFDT_OVERLAY=y +CONFIG_EFI_LOADER=y diff --git a/lib/efi_loader/Kconfig b/lib/efi_loader/Kconfig index 7984d6f42d..91411ad124 100644 --- a/lib/efi_loader/Kconfig +++ b/lib/efi_loader/Kconfig @@ -10,7 +10,7 @@ config EFI_LOADER depends on !EFI_STUB || !X86_64 || EFI_STUB_64BIT # We need EFI_STUB_32BIT to be set on x86_32 with EFI_STUB depends on !EFI_STUB || !X86 || X86_64 || EFI_STUB_32BIT - default y + default y if !ARM || SYS_CPU = armv7 || SYS_CPU = armv8 select LIB_UUID select HAVE_BLOCK_DEVICE select REGEX -- 2.24.0

4 3

[U-Boot] [PATCH v4 0/5] Add helper function for linking a DM Eth device to a PHY
by Alex Marginean 24 Nov '19

24 Nov '19

The patch set introduces dm_eth_phy_connect which takes in an ethernet device and uses DT information to find the associated PHY and connect the Ethernet interface to it. This should simplify similar code in ethernet drivers. A new binding is introduced for scanning an MDIO bus, useful if the bus is private to the Ethernet interface. We're going to try to get this binding accepted in Linux too. Finally the patch set updates fsl_enetc driver to use the new helper function. This patch set supersedes v2 series: https://patchwork.ozlabs.org/project/uboot/list/?series=124537 and v3 series: https://patchwork.ozlabs.org/project/uboot/list/?series=140114 Changes in v4: - rebased on current head Changes in v3: - added cover letter - check for null PHY pointer before using it in dm_eth_connect_phy_handle - moved the code dealing with MDIO scanning into a separate patch - renames several arguments and variables for a bit more clarity and consistency Changes in v2: - Moved MDIO scan code into dm_mdio_phy_scan which is also exported - Use interface instead of if_type for consistency - don't use phy pointer if NULL in fsl_enetc code Alex Marginean (5): net: mdio-uclass: rename arguments of dm_mdio_phy_connect for consistency net: mdio-uclass: add dm_eth_phy_connect helper function net: mdio-uclass: Add helper functions for scanning the MDIO bus doc: bindings: add mdio-handle property to ethernet nodes drivers: net: fsl_enetc: use the new MDIO DM helper functions doc/device-tree-bindings/net/ethernet.txt | 2 + drivers/net/fsl_enetc.c | 52 ++------ drivers/net/fsl_enetc.h | 1 + include/miiphy.h | 32 ++++- net/mdio-uclass.c | 139 +++++++++++++++++++++- 5 files changed, 175 insertions(+), 51 deletions(-) -- 2.17.1

4 16

[U-Boot] [PATCH] drivers: mmc: fsl_esdhc_imx: add m68k support
by Angelo Dureghello 23 Nov '19

23 Nov '19

From: Angelo Durgehello <angelo.dureghello(a)timesys.com> Add DM m68k support. Signed-off-by: Angelo Durgehello <angelo.dureghello(a)timesys.com> --- drivers/mmc/fsl_esdhc_imx.c | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/drivers/mmc/fsl_esdhc_imx.c b/drivers/mmc/fsl_esdhc_imx.c index 4099386313..4437ed82ae 100644 --- a/drivers/mmc/fsl_esdhc_imx.c +++ b/drivers/mmc/fsl_esdhc_imx.c @@ -1398,7 +1398,9 @@ void fdt_fixup_esdhc(void *blob, bd_t *bd) #endif #if CONFIG_IS_ENABLED(DM_MMC) +#ifndef CONFIG_MCF5441x #include <asm/arch/clock.h> +#endif __weak void init_clk_usdhc(u32 index) { } @@ -1528,7 +1530,11 @@ static int fsl_esdhc_probe(struct udevice *dev) priv->sdhc_clk = clk_get_rate(&priv->per_clk); } else { +#ifndef CONFIG_MCF5441x priv->sdhc_clk = mxc_get_clock(MXC_ESDHC_CLK + dev->seq); +#else + priv->sdhc_clk = gd->arch.sdhc_clk; +#endif if (priv->sdhc_clk <= 0) { dev_err(dev, "Unable to get clk for %s\n", dev->name); return -EINVAL; -- 2.24.0

1 0

[U-Boot] Question about U-Boot development model
by deanzerder 23 Nov '19

23 Nov '19

Hi, I'm a bit confused about the U-Boot development model, so I am hoping that someone might be able to clear it up: In Linux, you can send patches at any time to the maintainer of a subsystem. They will apply them to a fixes/next branch and it will be merged into mainline during the next merge window. It seems to me that this is not really the case for U-Boot. There is much more code without a custodian Git tree, and even many of the existing custodian Git trees seem to have deserted "next" branches. Does that mean that patches which add new functionality are essentially supposed to be posted during the merge window? I would say that it's usually helpful to send out patches early to get feedback. What happens if the patch is reviewed/deemed ready while the merge window is closed? Will it just get picked up automatically during the next merge window, or does it need to be sent again? Thanks, Dean

1 0

[U-Boot] [PATCH] The sata port can be configured when driver model enabled.
by liu hao 23 Nov '19

23 Nov '19

- CONFIG_SYS_SCSI_MAX_SCSI_ID - CONFIG_SYS_SCSI_MAX_LUN - These two configs can be used here. Signed-off-by: Steven Hao <steven_hao5189(a)outlook.com> --- Changes for v1: - The sata driver only support 2 ports. It is not suited for all user. We can make it can be configured. --- drivers/ata/ahci.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/ata/ahci.c b/drivers/ata/ahci.c index d10f9f0..594982c 100644 --- a/drivers/ata/ahci.c +++ b/drivers/ata/ahci.c @@ -1167,8 +1167,8 @@ int ahci_probe_scsi(struct udevice *ahci_dev, ulong base) return -ENODEV; uc_plat = dev_get_uclass_platdata(dev); uc_plat->base = base; - uc_plat->max_lun = 1; - uc_plat->max_id = 2; + uc_plat->max_lun = CONFIG_SYS_SCSI_MAX_LUN; + uc_plat->max_id = CONFIG_SYS_SCSI_MAX_SCSI_ID; uc_priv = dev_get_uclass_priv(ahci_dev); ret = ahci_init_one(uc_priv, dev); -- 2.7.4

2 1