September 2021 - U-Boot

[PATCH] doc: Remove information about CAPSULE_FMP_HEADER
by Michal Simek 03 Sep '21

03 Sep '21

This Kconfig symbol was never added to U-Boot but it was mentioned in the origin commit c35df7c9e43e ("qemu: arm64: Add documentation for capsule update"). That's why remove it from documentation to be accurate. Signed-off-by: Michal Simek <michal.simek(a)xilinx.com> --- doc/develop/uefi/uefi.rst | 1 - 1 file changed, 1 deletion(-) diff --git a/doc/develop/uefi/uefi.rst b/doc/develop/uefi/uefi.rst index 64fe9346c7f2..f17138f5c765 100644 --- a/doc/develop/uefi/uefi.rst +++ b/doc/develop/uefi/uefi.rst @@ -392,7 +392,6 @@ settings:: CONFIG_EFI_CAPSULE_FIRMWARE_MANAGEMENT=y CONFIG_EFI_CAPSULE_FIRMWARE=y CONFIG_EFI_CAPSULE_FIRMWARE_RAW=y - CONFIG_EFI_CAPSULE_FMP_HEADER=y In addition, the following config needs to be disabled(QEMU ARM specific):: -- 2.33.0

2 2

Re: [PATCH 0/2] Fix qspi read and write issues
by Michal Simek 03 Sep '21

03 Sep '21

On 8/27/21 10:36 AM, Ashok Reddy Soma wrote: > This patch series fixes below issues > - Switch genfifo start to manual start mode > - Unaligned DMA reads > - Minor improvements in initialization > > > > Ashok Reddy Soma (2): > spi: zynqmp_gqspi: Switch genfifo start to manual mode > spi: zynqmp_gqspi: Fix dma alignment issue > > drivers/spi/zynqmp_gqspi.c | 30 ++++++++++++++---------------- > 1 file changed, 14 insertions(+), 16 deletions(-) > Applied. M

1 0

string literals in efi_loader code
by AKASHI Takahiro 03 Sep '21

03 Sep '21

Heinrich, You started to use u"..." literals in efi_loader code in commit 99bfab8b5832("efi_loader: identify PK, KEK, db, dbx correctly"), but there are still lots of L"..." literals seen. Do you want to replace all the 'L' to 'u'? I think that mixing both usages would cause confusion. What is your policy? (I know -fshort-wchar is still on in Makefile.) -Takahiro Akashi

1 0

[PATCH] lib: add crc16.o if CONFIG_MMC_SPI_CRC_ON for SPL build
by Andreas Schwab 03 Sep '21

03 Sep '21

CONFIG_MMC_SPI_CRC_ON needs the crc16 functions, but it was not included in an SPL build. For non-SPL builds, crc16.o is already added unconditionally. This also removes CONFIG_SPL_YMODEM_SUPPORT from the sifive board configs, which is only relevant for some ARM boards and was only set for its side effect of adding crc16.o. --- configs/sifive_unleashed_defconfig | 1 - configs/sifive_unmatched_defconfig | 1 - lib/Makefile | 1 + 3 files changed, 1 insertion(+), 2 deletions(-) diff --git a/configs/sifive_unleashed_defconfig b/configs/sifive_unleashed_defconfig index fd686dfadc..d6448dbf5e 100644 --- a/configs/sifive_unleashed_defconfig +++ b/configs/sifive_unleashed_defconfig @@ -22,7 +22,6 @@ CONFIG_SPL_SEPARATE_BSS=y CONFIG_SPL_DM_SPI_FLASH=y CONFIG_SPL_DM_RESET=y CONFIG_SPL_SPI_LOAD=y -CONFIG_SPL_YMODEM_SUPPORT=y CONFIG_SYS_RELOC_GD_ENV_ADDR=y CONFIG_SPL_DM_SEQ_ALIAS=y CONFIG_SPL_CLK=y diff --git a/configs/sifive_unmatched_defconfig b/configs/sifive_unmatched_defconfig index 6c40a35bc2..59c4a383e0 100644 --- a/configs/sifive_unmatched_defconfig +++ b/configs/sifive_unmatched_defconfig @@ -20,7 +20,6 @@ CONFIG_DISPLAY_BOARDINFO=y CONFIG_DISPLAY_BOARDINFO_LATE=y CONFIG_SPL_SEPARATE_BSS=y CONFIG_SPL_DM_RESET=y -CONFIG_SPL_YMODEM_SUPPORT=y CONFIG_CMD_EEPROM=y CONFIG_CMD_MEMINFO=y CONFIG_CMD_PWM=y diff --git a/lib/Makefile b/lib/Makefile index 8ba745faa0..2d29cdad1e 100644 --- a/lib/Makefile +++ b/lib/Makefile @@ -88,6 +88,7 @@ endif ifdef CONFIG_SPL_BUILD obj-$(CONFIG_SPL_YMODEM_SUPPORT) += crc16.o obj-$(CONFIG_$(SPL_TPL_)HASH_SUPPORT) += crc16.o +obj-$(CONFIG_MMC_SPI_CRC_ON) += crc16.o obj-y += net_utils.o endif obj-$(CONFIG_ADDR_MAP) += addr_map.o -- 2.33.0 -- Andreas Schwab, SUSE Labs, schwab(a)suse.de GPG Key fingerprint = 0196 BAD8 1CE9 1970 F4BE 1748 E4D4 88E3 0EEA B9D7 "And now for something completely different."

3 3

[PATCH v4 0/2] mmc: Add support for enumrating MMC card in user specified mode
by Aswath Govindraju 03 Sep '21

03 Sep '21

The following series of patches, - add support enumerating MMC card in user specified speed mode - add documentation for the usage of above feature changes since v3: - split the patch in two, seperating the documentation changes Aswath Govindraju (2): mmc: Add support for enumerating MMC card in a given mode using mmc command doc: usage: mmc: Document usage of speed mode in "mmc dev" and "mmc rescan" cmd/Kconfig | 10 ++++++++ cmd/mmc.c | 52 +++++++++++++++++++++++++++++++++------- doc/usage/mmc.rst | 49 +++++++++++++++++++++++++++++++++++-- drivers/mmc/mmc-uclass.c | 5 +++- drivers/mmc/mmc.c | 22 ++++++++++++++++- include/mmc.h | 2 ++ 6 files changed, 128 insertions(+), 12 deletions(-) -- 2.17.1

2 5

[PATCH 0/1] tools: env: Handle shorter read calls
by Thibault Ferrante 03 Sep '21

03 Sep '21

On specific devices, read can return less bytes than expected. I noticed this problem on a EEPROM chip with 16Kbyte of memory. Defining an environment with more than 4Kbyte leads to less bytes read than expected and failures from fw_printenv/fw_setenv. This patch target to handle this issue by doing multiple reads instead of exiting directly. Thibault Ferrante (1): tools: env: Handle shorter read calls tools/env/fw_env.c | 22 ++++++++++++---------- 1 file changed, 12 insertions(+), 10 deletions(-) -- 2.25.1

2 2

[PATCH] psci: fix double declaration
by Oleksandr Suvorov 03 Sep '21

03 Sep '21

The prototype of psci_features() duplicated. Remove extra declaration. Fixed: e21e3ffdd1 ("psci: Fix warnings when compiling with W=1") Reported-by: Michael Scott <mike(a)foundries.io> Signed-off-by: Oleksandr Suvorov <oleksandr.suvorov(a)foundries.io> --- arch/arm/include/asm/system.h | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm/include/asm/system.h b/arch/arm/include/asm/system.h index 8b3a54e64c..9eec19946b 100644 --- a/arch/arm/include/asm/system.h +++ b/arch/arm/include/asm/system.h @@ -551,7 +551,6 @@ s32 psci_affinity_info(u32 function_id, u32 target_affinity, u32 psci_migrate_info_type(void); void psci_system_off(void); void psci_system_reset(void); -s32 psci_features(u32 function_id, u32 psci_fid); #endif #endif /* __ASSEMBLY__ */ -- 2.31.1

2 1

[PATCH v1] pci: iproc: fix compilation errors and warnings
by Bharat Kumar Reddy Gooty 03 Sep '21

03 Sep '21

From: Bharat Gooty <bharat.gooty(a)broadcom.com> Signed-off-by: Bharat Gooty <bharat.gooty(a)broadcom.com> --- drivers/pci/pcie_iproc.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/drivers/pci/pcie_iproc.c b/drivers/pci/pcie_iproc.c index 12ce9d525c..be03dcbd97 100644 --- a/drivers/pci/pcie_iproc.c +++ b/drivers/pci/pcie_iproc.c @@ -1,6 +1,6 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * Copyright (C) 2020 Broadcom + * Copyright (C) 2020-2021 Broadcom * */ @@ -12,6 +12,7 @@ #include <malloc.h> #include <asm/io.h> #include <dm/device_compat.h> +#include <linux/delay.h> #include <linux/log2.h> #define EP_PERST_SOURCE_SELECT_SHIFT 2 @@ -884,7 +885,7 @@ static int iproc_pcie_map_ranges(struct udevice *dev) for (i = 0; i < hose->region_count; i++) { if (hose->regions[i].flags == PCI_REGION_MEM || hose->regions[i].flags == PCI_REGION_PREFETCH) { - debug("%d: bus_addr %p, axi_addr %p, size 0x%lx\n", + debug("%d: bus_addr %p, axi_addr %p, size 0x%llx\n", i, &hose->regions[i].bus_start, &hose->regions[i].phys_start, hose->regions[i].size); @@ -1049,7 +1050,7 @@ static int iproc_pcie_map_dma_ranges(struct iproc_pcie *pcie) while (!pci_get_dma_regions(pcie->dev, &regions, i)) { dev_dbg(pcie->dev, - "dma %d: bus_addr %#lx, axi_addr %#llx, size %#lx\n", + "dma %d: bus_addr %#llx, axi_addr %#llx, size %#llx\n", i, regions.bus_start, regions.phys_start, regions.size); /* Each range entry corresponds to an inbound mapping region */ -- 2.17.1 -- This electronic communication and the information and any files transmitted with it, or attached to it, are confidential and are intended solely for the use of the individual or entity to whom it is addressed and may contain information that is confidential, legally privileged, protected by privacy laws, or otherwise restricted from disclosure to anyone else. If you are not the intended recipient or the person responsible for delivering the e-mail to the intended recipient, you are hereby notified that any use, copying, distributing, dissemination, forwarding, printing, or copying of this e-mail is strictly prohibited. If you received this e-mail in error, please return the e-mail to the sender, delete it from your computer, and destroy any printed copy of it.

2 1

[PATCH] mailmap: Update e-mail for Jernej Skrabec
by Jernej Skrabec 03 Sep '21

03 Sep '21

Old address doesn't exist anymore. Map it to new one. Signed-off-by: Jernej Skrabec <jernej.skrabec(a)gmail.com> --- .mailmap | 1 + 1 file changed, 1 insertion(+) diff --git a/.mailmap b/.mailmap index b76f02283c57..3d15a09d1c2d 100644 --- a/.mailmap +++ b/.mailmap @@ -29,6 +29,7 @@ Jagan Teki <jaganna(a)gmail.com> Jagan Teki <jaganna(a)xilinx.com> Jagan Teki <jagannadh.teki(a)gmail.com> Jagan Teki <jagannadha.sutradharudu-teki(a)xilinx.com> +Jernej Skrabec <jernej.skrabec(a)gmail.com> <jernej.skrabec(a)siol.net> Igor Opaniuk <igor.opaniuk(a)gmail.com> <igor.opaniuk(a)linaro.org> Igor Opaniuk <igor.opaniuk(a)gmail.com> <igor.opaniuk(a)toradex.com> Markus Klotzbuecher <mk(a)denx.de> -- 2.33.0

3 2

[PATCH 1/1] cmd/part: correct alignment of partition list
by Heinrich Schuchardt 03 Sep '21

03 Sep '21

When running 'part list' for an ISO partition the numbers are not under the labels. Correct the alignment of the ISO partition list. With the patch the output looks like: Part Start Sect x Size Type 1 3720 5024 512 U-Boot Signed-off-by: Heinrich Schuchardt <xypron.glpk(a)gmx.de> --- disk/part_iso.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/disk/part_iso.c b/disk/part_iso.c index 822f2c4d9f..1061f341d3 100644 --- a/disk/part_iso.c +++ b/disk/part_iso.c @@ -220,7 +220,7 @@ static void part_print_iso(struct blk_desc *dev_desc) printf("Part Start Sect x Size Type\n"); i=1; do { - printf(" %2d " LBAFU " " LBAFU " %6ld %.32s\n", + printf(" %2d %8" LBAFlength "u %8" LBAFlength "u %6ld %.32s\n", i, info.start, info.size, info.blksz, info.type); i++; } while (part_get_info_iso_verb(dev_desc, i, &info, 0) != -1); -- 2.30.2

2 1