November 2024 - U-Boot

[ANN] U-Boot v2025.01-rc3 released
by Tom Rini 26 Nov '24

26 Nov '24

Hey all, Once again it's release day and so here's v2025.01-rc3 and I'll update the -next branch soon. At this point, the biggest outstanding changes I see are what fixes are still requires to the spi subsystem to bring spi-nor functionality back in to line with v2024.10, and what further cleanups are needed to lwIP or LMB now and shouldn't wait until v2025.04. In terms of a changelog, git log --merges v2025.01-rc2..v2025.01-rc3 As always, more details in pull requests (or the tags referenced by them) will result in more details here. I continue to plan to do an rc release every two weeks with the exception of I'm also doing -rc6 on Monday, December 30th to have one more tag over the end of year holidays, to see if that helps with people finding time to test. We'll how that went in January. Thanks all! -- Tom

1 0

[PULL] u-boot-usb/master-asix
by Marek Vasut 25 Nov '24

25 Nov '24

The following changes since commit 52c0e5f8a39ef022ee5ab2dfd67661d1d34941c7: Merge branch '2024-11-22-assorted-fixes' (2024-11-22 15:04:54 -0600) are available in the Git repository at: git://source.denx.de/u-boot-usb.git master-asix for you to fetch changes up to 05278af8b3df1c0c02e345ccf4c8c59bd0111f68: eth: asix88179: packet drop when receiving large fragmented packets (2024-11-25 19:11:19 +0100) ---------------------------------------------------------------- Khoa Hoang (2): eth: asix88179: Fix ASIX AX88179A PHY hang eth: asix88179: packet drop when receiving large fragmented packets drivers/usb/eth/asix88179.c | 45 +++++++++++++++++++++++++++++++++++++-------- 1 file changed, 37 insertions(+), 8 deletions(-)

2 1

[PATCH] spi: Fix missed rename from SPI_ADVANCE to SPI_STACKED_PARALLEL
by Dominik Wernberger 25 Nov '24

25 Nov '24

Fix missed rename from SPI_ADVANCE to SPI_STACKED_PARALLEL. This fixes an issue encountered while testing the Zynq-7000 QSPI parallel Flash implementation. Signed-off-by: Dominik Wernberger <dominik.wernberger(a)gmx.de> --- drivers/spi/spi-uclass.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/spi/spi-uclass.c b/drivers/spi/spi-uclass.c index 36b7d383aa..d604975374 100644 --- a/drivers/spi/spi-uclass.c +++ b/drivers/spi/spi-uclass.c @@ -446,7 +446,7 @@ int _spi_get_bus_and_cs(int busnum, int cs, int speed, int mode, slave = dev_get_parent_priv(dev); bus_data = dev_get_uclass_priv(bus); -#if CONFIG_IS_ENABLED(SPI_ADVANCE) +#if CONFIG_IS_ENABLED(SPI_STACKED_PARALLEL) if ((dev_read_bool(dev, "parallel-memories")) && !slave->multi_cs_cap) { dev_err(dev, "controller doesn't support multi CS\n"); return -EINVAL; @@ -515,7 +515,7 @@ int spi_slave_of_to_plat(struct udevice *dev, struct dm_spi_slave_plat *plat) int mode = 0; int value; -#if CONFIG_IS_ENABLED(SPI_ADVANCE) +#if CONFIG_IS_ENABLED(SPI_STACKED_PARALLEL) int ret; ret = dev_read_u32_array(dev, "reg", plat->cs, SPI_CS_CNT_MAX); -- 2.34.1 --- Another approach to get the commit message right. In case there are still some issues feel free to fix them up. I'm obviously not yet used to the Linux mailing list which will hopefully change soon. Until then I don't want to annoy you with too many mails. Thanks for your help! :) BR Dominik

2 1

[PATCH] spi: Fix missed rename from SPI_ADVANCE to SPI_STACKED_PARALLEL
by Dominik Wernberger 25 Nov '24

25 Nov '24

Signed-off-by: Dominik Wernberger <dominik.wernberger(a)gmx.de> --- drivers/spi/spi-uclass.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/spi/spi-uclass.c b/drivers/spi/spi-uclass.c index 36b7d383aa..d604975374 100644 --- a/drivers/spi/spi-uclass.c +++ b/drivers/spi/spi-uclass.c @@ -446,7 +446,7 @@ int _spi_get_bus_and_cs(int busnum, int cs, int speed, int mode, slave = dev_get_parent_priv(dev); bus_data = dev_get_uclass_priv(bus); -#if CONFIG_IS_ENABLED(SPI_ADVANCE) +#if CONFIG_IS_ENABLED(SPI_STACKED_PARALLEL) if ((dev_read_bool(dev, "parallel-memories")) && !slave->multi_cs_cap) { dev_err(dev, "controller doesn't support multi CS\n"); return -EINVAL; @@ -515,7 +515,7 @@ int spi_slave_of_to_plat(struct udevice *dev, struct dm_spi_slave_plat *plat) int mode = 0; int value; -#if CONFIG_IS_ENABLED(SPI_ADVANCE) +#if CONFIG_IS_ENABLED(SPI_STACKED_PARALLEL) int ret; ret = dev_read_u32_array(dev, "reg", plat->cs, SPI_CS_CNT_MAX); -- 2.34.1 --- Moved the URL from the commit message here into the annotation section. Original renaming was performed in https://lore.kernel.org/u-boot/20241026201741.171073-8-marek.vasut+renesas@… I encountered this issue while testing the Zynq-7000 QSPI parallel Flash implementation but there is no corresponding issue in the bug tracker that I'm aware of.

3 2

[PULL] u-boot-usb/master
by Marek Vasut 25 Nov '24

25 Nov '24

The following changes since commit 52c0e5f8a39ef022ee5ab2dfd67661d1d34941c7: Merge branch '2024-11-22-assorted-fixes' (2024-11-22 15:04:54 -0600) are available in the Git repository at: git://source.denx.de/u-boot-usb.git master for you to fetch changes up to 57e30b09fcfc88da72c310f979b35ed99a9b277c: usb: onboard-hub: Bail out if peer hub is already probed (2024-11-25 05:40:49 +0100) ---------------------------------------------------------------- Venkatesh Yadav Abbarapu (5): usb: onboard-hub: Add reset-gpio support usb: onboard-hub: Fix the return values of regulator APIs usb: onboard-hub: add support for Microchip USB5744 usb: onboard-hub: Add i2c initialization for usb5744 hub usb: onboard-hub: Bail out if peer hub is already probed common/usb_onboard_hub.c | 195 +++++++++++++++++++++++++++++++++++++++++++++-- 1 file changed, 190 insertions(+), 5 deletions(-)

2 1

[PATCH] ast2600: spl: Use readl for reading mmio
by Joel Stanley 25 Nov '24

25 Nov '24

u-boot was crashing in qemu as the modeled hardware enforced overly strict memory reads. While this code will work on existing hardware, fix to avoid future issues. Fixes: 12770d0df0e8 ("ast2600: spl: Add boot mode detection") Link: https://gitlab.com/qemu-project/qemu/-/issues/2636 Signed-off-by: Joel Stanley <joel(a)jms.id.au> --- GCC 14 was using this sequence to read the value, leading to the crash: ldrbpl.w r0, [r0, #1280] @ 0x500 arch/arm/mach-aspeed/ast2600/spl.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/arm/mach-aspeed/ast2600/spl.c b/arch/arm/mach-aspeed/ast2600/spl.c index 05390c16f3af..0c5a82ed0947 100644 --- a/arch/arm/mach-aspeed/ast2600/spl.c +++ b/arch/arm/mach-aspeed/ast2600/spl.c @@ -45,10 +45,10 @@ u32 spl_boot_device(void) } /* boot from UART has higher priority */ - if (scu->hwstrap2 & SCU_HWSTRAP2_BOOT_UART) + if (readl(&scu->hwstrap2) & SCU_HWSTRAP2_BOOT_UART) return BOOT_DEVICE_UART; - if (scu->hwstrap1 & SCU_HWSTRAP1_BOOT_EMMC) + if (readl(&scu->hwstrap1) & SCU_HWSTRAP1_BOOT_EMMC) return BOOT_DEVICE_MMC1; out: -- 2.45.2

2 1

[PATCH v1 1/1] mtd: Drop superfluous const qualifier
by Andy Shevchenko 25 Nov '24

25 Nov '24

The compiler will ignore it anyway: include/linux/mtd/mtd.h:342:15: warning: type qualifiers ignored on function return type [-Wignored-qualifiers] Signed-off-by: Andy Shevchenko <andriy.shevchenko(a)linux.intel.com> --- include/linux/mtd/mtd.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/linux/mtd/mtd.h b/include/linux/mtd/mtd.h index 6751fb52c5a5..3ffcd40807d0 100644 --- a/include/linux/mtd/mtd.h +++ b/include/linux/mtd/mtd.h @@ -339,7 +339,7 @@ static inline void mtd_set_ofnode(struct mtd_info *mtd, ofnode node) dev_set_ofnode(mtd->dev, node); } -static inline const ofnode mtd_get_ofnode(struct mtd_info *mtd) +static inline ofnode mtd_get_ofnode(struct mtd_info *mtd) { return dev_ofnode(mtd->dev); } -- 2.43.0.rc1.1336.g36b5255a03ac

3 2

[PATCH] net-lwip: zero terminate string with headers in wget_lwip_fill_info()
by Adriano Cordova 25 Nov '24

25 Nov '24

This patch comes as a companion to the same patch but for the legacy net stack. Commit 1327c2a8d6 ("net/lwip: wget: integrate struct wget_info into wget code") introduced function wget_fill_info() which retrieves the headers from the HTTP server response. As we want to parse the string in later patches we need to ensure that it is NUL terminated. We must further check that wget_info->headers in not NULL. Otherwise a crash occurs. Fixes: 1327c2a8d6 ("net/lwip: wget: integrate struct wget_info into wget code") Signed-off-by: Adriano Cordova <adrianox(a)gmail.com> --- net/lwip/wget.c | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/net/lwip/wget.c b/net/lwip/wget.c index 53c3b169e0..15cf4e25bd 100644 --- a/net/lwip/wget.c +++ b/net/lwip/wget.c @@ -36,8 +36,13 @@ struct wget_ctx { static void wget_lwip_fill_info(struct pbuf *hdr, u16_t hdr_len, u32_t hdr_cont_len) { - if (wget_info->headers && hdr_len < MAX_HTTP_HEADERS_SIZE) - pbuf_copy_partial(hdr, (void *)wget_info->headers, hdr_len, 0); + if (wget_info->headers) { + if (wget_info->headers && hdr_len < MAX_HTTP_HEADERS_SIZE) + pbuf_copy_partial(hdr, (void *)wget_info->headers, hdr_len, 0); + else + hdr_len = 0; + wget_info->headers[hdr_len] = 0; + } wget_info->hdr_cont_len = (u32)hdr_cont_len; } -- 2.43.0

1 0

Question: Looking for x86 platform for u-boot development
by Naresh Bhat 25 Nov '24

25 Nov '24

Hi, I am new to u-boot. To learn the u-boot source code development (i.e PCI driver in u-boot), Can some one please suggest any x86 platform which has a PCI slot ? Thanks and Regards -Naresh Bhat

1 0

[PATCH 1/1] net: remove duplicate definition of ip_to_string()
by Heinrich Schuchardt 25 Nov '24

25 Nov '24

ip_to_string() was added to net-common.h but not removed from net-legacy.h in commit 9063dba2d326 ("net: net_utils: Move ip_to_string to lib/net_utils.c"). Fixes: 9063dba2d326 ("net: net_utils: Move ip_to_string to lib/net_utils.c") Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt(a)canonical.com> --- include/net-legacy.h | 3 --- 1 file changed, 3 deletions(-) diff --git a/include/net-legacy.h b/include/net-legacy.h index ca1efd17af7..5a431fc6b16 100644 --- a/include/net-legacy.h +++ b/include/net-legacy.h @@ -502,9 +502,6 @@ static inline void net_copy_u32(void *to, void *from) memcpy((void *)to, (void *)from, sizeof(u32)); } -/* Convert an IP address to a string */ -void ip_to_string(struct in_addr x, char *s); - /** * string_to_ip() - Convert a string to ip address * -- 2.45.2

3 2