July 2022 - U-Boot

[ANN] U-Boot release v2022.07 DELAYED 1 week
by Tom Rini 04 Jul '22

04 Jul '22

Hey all, Given all of the last minutes PRs, I'm going to delay the release by a week and do one more -rc on the 4th, and final release on 11 July 2022. Thanks for your understanding all. -- Tom

1 0

[PATCH 1/1] disk: honor CONFIG_SYS_64BIT_LBA
by Heinrich Schuchardt 04 Jul '22

04 Jul '22

Without the patch for qemu-x86_defconfig: * sizeof(lbaint_t) = 4 in dev_read() * sizeof(lbaint_t) = 8 in blkcache_read() CONFIG_SYS_64BIT_LBA is defined in common.h via include/configs/x86-common.h:22. We have to include common.h before including blk.h. Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt(a)canonical.com> --- disk/disk-uclass.c | 1 + 1 file changed, 1 insertion(+) diff --git a/disk/disk-uclass.c b/disk/disk-uclass.c index 72ff62ebf5..f3fb942a6b 100644 --- a/disk/disk-uclass.c +++ b/disk/disk-uclass.c @@ -8,6 +8,7 @@ #define LOG_CATEGORY UCLASS_PARTITION +#include <common.h> #include <blk.h> #include <dm.h> #include <log.h> -- 2.36.1

3 2

net: designware: frequent network problem like CPU not owner of tx frame
by LABBE Corentin 03 Jul '22

03 Jul '22

Hello We have an HSDK board in kernelCI, and it fail often to do networking tasks. (getting image via tftp) Example: tftp 0x82000000 2156673/tftp-deploy-l8y_ehqa/kernel/uImage Speed: 100, full duplex Using ethernet@f0008000 device TFTP from server 10.201.1.3; our IP address is 10.201.4.179 Filename '2156673/tftp-deploy-l8y_ehqa/kernel/uImage'. Load address: 0x82000000 Loading: *␈CPU not owner of tx frame CPU not owner of tx frame CPU not owner of tx frame CPU not owner of tx frame ARP Retry count exceeded; starting again Memory error exception @ 0xfffffffe ECR: 0x00010000 RET: 0xfffffffe BLINK: 0xbffb04e2 STAT32: 0x00080400 GP: 0x1003e000 r25: 0xbfd92ee0 BTA: 0xbffc5744 SP: 0xbfd92d60 FP: 0x00000000 LPS: 0x000011f2 LPE: 0x000011fa LPC: 0x00000000 r00: 0x00000000 r01: 0xffffffff r02: 0x00000000 r03: 0x00000000 r04: 0x00000000 r05: 0xbfd92d54 r06: 0xbffdc700 r07: 0x00000000 r08: 0xf0020028 r09: 0x1003e2b0 r10: 0x80000000 r11: 0x0000000e r12: 0x00000010 r13: 0xbfda6218 r14: 0xbfd933b8 r15: 0x00000003 r16: 0x00000000 r17: 0x00000003 r18: 0xbffeff90 r19: 0x00000000 r20: 0xbfd949c8 r21: 0x00000000 r22: 0x00000000 r23: 0x00000000 r24: 0x00000000 Resetting CPU ... ### ERROR ### Please RESET the board ### tftp 0x82000000 2156279/tftp-deploy-o6vloihf/kernel/uImage Speed: 100, full duplex Using ethernet@f0008000 device TFTP from server 10.201.1.3; our IP address is 10.201.3.203 Filename '2156279/tftp-deploy-o6vloihf/kernel/uImage'. Load address: 0x82000000 Loading: *␈ ARP Retry count exceeded; starting again tftp 0x82000000 2155981/tftp-deploy-wm6qiwhy/kernel/uImage Speed: 100, full duplex Using ethernet@f0008000 device TFTP from server 10.201.1.3; our IP address is 10.201.4.197 Filename '2155981/tftp-deploy-wm6qiwhy/kernel/uImage'. Load address: 0x82000000 Loading: *␈CPU not owner of tx frame CPU not owner of tx frame [...] (lot of CPU not owner of tx frame) CPU not owner of tx frame ARP Retry count exceeded; starting again I tried to update uboot until 2022.04, without any change. Regards

2 1

[PATCH] sunxi: lcd: Move range from kconfig description to definition.
by Michal Suchanek 03 Jul '22

03 Jul '22

KConfig has range option, use it instead of notice in the option descrition. Signed-off-by: Michal Suchanek <msuchanek(a)suse.de> --- arch/arm/mach-sunxi/Kconfig | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/arch/arm/mach-sunxi/Kconfig b/arch/arm/mach-sunxi/Kconfig index 10645fc644..0ef79849a4 100644 --- a/arch/arm/mach-sunxi/Kconfig +++ b/arch/arm/mach-sunxi/Kconfig @@ -854,8 +854,9 @@ config VIDEO_LCD_DCLK_PHASE int "LCD panel display clock phase" depends on VIDEO_SUNXI || DM_VIDEO default 1 + range 0 3 ---help--- - Select LCD panel display clock phase shift, range 0-3. + Select LCD panel display clock phase shift config VIDEO_LCD_POWER string "LCD panel power enable pin" -- 2.36.1

1 0

Pull request for efi-2022-07-rc6
by Heinrich Schuchardt 03 Jul '22

03 Jul '22

Dear Tom, The following changes since commit 936d468b7bbc8c02071b3d0d24f9c48167cb5ce7: chameleonv3: Add MAINTAINERS file (2022-07-01 15:55:54 -0400) are available in the Git repository at: https://source.denx.de/u-boot/custodians/u-boot-efi.git tags/efi-2022-07-rc6 for you to fetch changes up to 7a85f32413a34b18a2f84c865d7b448790200b13: EFI: Fix ReadBlocks API reading incorrect sector for UCLASS_PARTITION devices (2022-07-02 14:19:12 +0200) Gitlab CI showed no problems: https://source.denx.de/u-boot/custodians/u-boot-efi/-/pipelines/12600 ---------------------------------------------------------------- Pull request for efi-2022-07-rc6 UEFI: * Fix EFI_IO_BLOCK_PROTOCOL: read correct blocks on partitions Other: * Honor CONFIG_SYS_64BIT_LBA in the disk uclass ---------------------------------------------------------------- Heinrich Schuchardt (1): disk: honor CONFIG_SYS_64BIT_LBA Paul Barbieri (1): EFI: Fix ReadBlocks API reading incorrect sector for UCLASS_PARTITION devices disk/disk-uclass.c | 1 + lib/efi_loader/efi_disk.c | 8 +------- lib/efi_selftest/efi_selftest_block_device.c | 27 +++++++++++++++++++++++++++ 3 files changed, 29 insertions(+), 7 deletions(-)

2 1

[PATCH] MAINTAINERS: Add Dario and Michael as nand maintainers
by Michael Trimarchi 03 Jul '22

03 Jul '22

Both of us are working on Nand subsystem on several architectures and we have boards and projects to improve the subsystem in uboot. The idea is to garantee quick feedback on patches sent on mailing list and most of the time the possibilities to test them. Cc: Dario Binacchi <dario.binacchi(a)amarulasolutions.com> Cc: Tom Rini <trini(a)konsulko.com> Signed-off-by: Michael Trimarchi <michael(a)amarulasolutions.com> --- MAINTAINERS | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index a15ba7abdc..7f27ff4c20 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1064,8 +1064,9 @@ T: git https://source.denx.de/u-boot/custodians/u-boot-mmc.git F: drivers/mmc/ NAND FLASH -#M: Scott Wood <oss(a)buserror.net> -S: Orphaned (Since 2018-07) +M: Dario Binacchi <dario.binacchi(a)amarulasolutions.com> +M: Michael Trimarchi <michael(a)amarulasolutions.com> +S: Maintained T: git https://source.denx.de/u-boot/custodians/u-boot-nand-flash.git F: drivers/mtd/nand/raw/ -- 2.25.1

3 3

[PATCH] board/BuR/*: replace maintainer of BuR boards
by Hannes Schmelzer 03 Jul '22

03 Jul '22

Since i'm leaving the company with end of june, the maintainership will be transferred to Wolfgang Wallner. Signed-off-by: Hannes Schmelzer <hannes.schmelzer(a)br-automation.com> Seriec-cc: trini(a)konsulko.com Series-version : 2 Series-changes : 2 - fix typo in Wolgangs email-address --- board/BuR/brppt1/MAINTAINERS | 2 +- board/BuR/brppt2/MAINTAINERS | 2 +- board/BuR/brsmarc1/MAINTAINERS | 2 +- board/BuR/brxre1/MAINTAINERS | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-) diff --git a/board/BuR/brppt1/MAINTAINERS b/board/BuR/brppt1/MAINTAINERS index 9eddab4..6b45508 100644 --- a/board/BuR/brppt1/MAINTAINERS +++ b/board/BuR/brppt1/MAINTAINERS @@ -1,5 +1,5 @@ BRPPT1 BOARD -M: Hannes Schmelzer <hannes.schmelzer(a)br-automation.com> +M: Wolfgang Wallner <wolfgang.wallner(a)br-automation.com> S: Maintained F: board/BuR/brppt1/ F: include/configs/brppt1.h diff --git a/board/BuR/brppt2/MAINTAINERS b/board/BuR/brppt2/MAINTAINERS index a1b5bd4..fe65188 100644 --- a/board/BuR/brppt2/MAINTAINERS +++ b/board/BuR/brppt2/MAINTAINERS @@ -1,5 +1,5 @@ BUR_PPT2 BOARD -M: Hannes Schmelzer <hannes.schmelzer(a)br-automation.com> +M: Wolfgang Wallner <wolfgang.wallner(a)br-automation.com> S: Maintained F: board/BuR/brppt2/ F: include/configs/brppt2.h diff --git a/board/BuR/brsmarc1/MAINTAINERS b/board/BuR/brsmarc1/MAINTAINERS index c6dfc20..8d1fe21 100644 --- a/board/BuR/brsmarc1/MAINTAINERS +++ b/board/BuR/brsmarc1/MAINTAINERS @@ -1,5 +1,5 @@ BRSMARC1 BOARD -M: Hannes Schmelzer <hannes.schmelzer(a)br-automation.com> +M: Wolfgang Wallner <wolfgang.wallner(a)br-automation.com> S: Maintained F: board/BuR/brsmarc1/ F: include/configs/brsmarc1.h diff --git a/board/BuR/brxre1/MAINTAINERS b/board/BuR/brxre1/MAINTAINERS index eb0fe8b..5aa3671 100644 --- a/board/BuR/brxre1/MAINTAINERS +++ b/board/BuR/brxre1/MAINTAINERS @@ -1,5 +1,5 @@ BRXRE1 BOARD -M: Hannes Schmelzer <hannes.schmelzer(a)br-automation.com> +M: Wolfgang Wallner <wolfgang.wallner(a)br-automation.com> S: Maintained F: board/BuR/brxre1/ F: include/configs/brxre1.h -- 2.7.4

3 2

[PATCH] image: fit: Use stack allocation macro
by Joel Stanley 03 Jul '22

03 Jul '22

The documentation above the DEFINE_ALIGN_BUFFER says it's for use outside functions, but we're inside one. Instead use ALLOC_CACHE_ALIGN_BUFFER, the stack based macro, which also includes the cache alignment. Fixes: b583348ca8c8 ("image: fit: Align hash output buffers") Signed-off-by: Joel Stanley <joel(a)jms.id.au> --- This fixes booting the ast2600-evb image in qemu, which was getting all zeroes for the output of the FIT hash check. The 'static' buffer was in BSS but the output image didn't contain a BSS section. The pointer was left pointing to the text, so the code was trying to write to the (read only?) text area in SPI NOR memory space. tools/mkimage.h | 3 +-- boot/image-fit.c | 3 +-- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/tools/mkimage.h b/tools/mkimage.h index 7652c8b001c3..f5ca65e2edfd 100644 --- a/tools/mkimage.h +++ b/tools/mkimage.h @@ -41,8 +41,7 @@ static inline ulong map_to_sysmem(void *ptr) return (ulong)(uintptr_t)ptr; } -#define ARCH_DMA_MINALIGN 1 -#define DEFINE_ALIGN_BUFFER(type, name, size, alugn) type name[size] +#define ALLOC_CACHE_ALIGN_BUFFER(type, name, size) type name[size] #define MKIMAGE_TMPFILE_SUFFIX ".tmp" #define MKIMAGE_MAX_TMPFILE_LEN 256 diff --git a/boot/image-fit.c b/boot/image-fit.c index f57d97f55229..df3e5df8836a 100644 --- a/boot/image-fit.c +++ b/boot/image-fit.c @@ -1264,8 +1264,7 @@ int calculate_hash(const void *data, int data_len, const char *name, static int fit_image_check_hash(const void *fit, int noffset, const void *data, size_t size, char **err_msgp) { - DEFINE_ALIGN_BUFFER(uint8_t, value, FIT_MAX_HASH_LEN, - ARCH_DMA_MINALIGN); + ALLOC_CACHE_ALIGN_BUFFER(uint8_t, value, FIT_MAX_HASH_LEN); int value_len; const char *algo; uint8_t *fit_value; -- 2.35.1

4 3

[PATCH 1/1] cmd: simplify do_adc_single()
by Heinrich Schuchardt 03 Jul '22

03 Jul '22

If argc is not < 3, it must be >= 3. If argc >= 3, argv[2] cannot be NULL. Fixes: 9de612ae4ded ("cmd: adc: Add support for storing ADC result in env variable") Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt(a)canonical.com> --- cmd/adc.c | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/cmd/adc.c b/cmd/adc.c index 8de9121cad..195efa8661 100644 --- a/cmd/adc.c +++ b/cmd/adc.c @@ -71,7 +71,6 @@ static int do_adc_info(struct cmd_tbl *cmdtp, int flag, int argc, static int do_adc_single(struct cmd_tbl *cmdtp, int flag, int argc, char *const argv[]) { - char *varname = NULL; struct udevice *dev; unsigned int data; int ret, uV, val; @@ -79,9 +78,6 @@ static int do_adc_single(struct cmd_tbl *cmdtp, int flag, int argc, if (argc < 3) return CMD_RET_USAGE; - if (argc >= 3) - varname = argv[2]; - ret = adc_channel_single_shot(argv[1], simple_strtol(argv[2], NULL, 0), &data); if (ret) { @@ -99,8 +95,7 @@ static int do_adc_single(struct cmd_tbl *cmdtp, int flag, int argc, printf("%u\n", data); } - if (varname) - env_set_ulong(varname, val); + env_set_ulong(argv[2], val); return CMD_RET_SUCCESS; } -- 2.34.1

4 4

[PATCH] tools/imx8mimage: Remove the usage of IVT reserved1 field
by Peng Fan (OSS) 03 Jul '22

03 Jul '22

From: Peng Fan <peng.fan(a)nxp.com> Commit log from NXP imx-mkimage: On iMX8MQ B2, ROM will check IVT header and requires the reserved fields in IVT to be 0. However, in imx-mkimage we set the reserved1 field for the offset from second boot image to SPL boot image, which was used for mfgtool but has deprecated. So remove it to fix the boot failure on iMX8MQ B2. Update imx8mimage to address same issue Cc: Mihai Chelalau <mihai.chelalau(a)nxp.com> Cc: Dan Douglass <dan.douglass(a)nxp.com> Signed-off-by: Peng Fan <peng.fan(a)nxp.com> --- tools/imx8mimage.c | 8 -------- 1 file changed, 8 deletions(-) diff --git a/tools/imx8mimage.c b/tools/imx8mimage.c index 4eed683396f..52baf4c9903 100644 --- a/tools/imx8mimage.c +++ b/tools/imx8mimage.c @@ -505,14 +505,6 @@ void build_image(int ofd) exit(EXIT_FAILURE); } else { sld_header_off = sld_src_off - rom_image_offset; - /* - * Record the second bootloader relative offset in - * image's IVT reserved1 - */ - if (rom_version == ROM_V1) { - imx_header[IMAGE_IVT_ID].fhdr.reserved1 = - sld_header_off - header_image_off; - } sld_fd = open(sld_img, O_RDONLY | O_BINARY); if (sld_fd < 0) { fprintf(stderr, "%s: Can't open: %s\n", -- 2.36.0

2 3