November 2023 - U-Boot

[PATCH v2 0/9] ufs: Add a PCI UFS controller support
by Bin Meng 27 Nov '23

27 Nov '23

This adds a PCI UFS controller support and enables the support on QEMU RISC-V for testing. Requiring QEMU v8.2+. This series is avaiable at u-boot-x86/ufs for testing. Changes in v2: - fix a build warning Bin Meng (9): ufs: Correct the UFS terminlogy ufs: Add a line feed to the end of some dev_xxx() messages cmd: kconfig: Make ufs prompt look similar to other commands cmd: ufs: Correct the help text pci_ids: Add Red Hat vendor and device IDs ufs: Allow mmio registers on the PCI bus ufs: Add a PCI based UFS controller driver ufs: Handle UFS 3.1 controllers qemu: riscv: Enable UFS support board/emulation/qemu-riscv/Kconfig | 2 ++ cmd/Kconfig | 2 +- cmd/ufs.c | 2 +- doc/board/emulation/qemu-riscv.rst | 8 +++++- drivers/ufs/Kconfig | 11 ++++++++ drivers/ufs/Makefile | 1 + drivers/ufs/ufs-pci.c | 45 ++++++++++++++++++++++++++++++ drivers/ufs/ufs-uclass.c | 2 +- drivers/ufs/ufs.c | 31 ++++++++++++-------- drivers/ufs/ufs.h | 1 + include/pci_ids.h | 7 +++++ 11 files changed, 97 insertions(+), 15 deletions(-) create mode 100644 drivers/ufs/ufs-pci.c -- 2.34.1

6 30

[PATCH v2 0/3] J721S2 Add remoteproc early boot support
by Manorit Chawdhry 27 Nov '23

27 Nov '23

The series adds the support to boot early remote cores from u-boot and changes the rproc_fw_binaries to be compatible with Linux DT Syncs. Logs: https://gist.github.com/manorit2001/2f2ef0699619e0b496367b400ff02d3c Cc: Apurva Nandan <a-nandan(a)ti.com> Cc: Udit Kumar <u-kumar1(a)ti.com> Cc: Vignesh Raghavendra <vigneshr(a)ti.com> Cc: Neha Malcom Francis <n-francis(a)ti.com> Cc: Nishanth Menon <nm(a)ti.com> Cc: Andrew Davis <afd(a)ti.com> Cc: <u-boot(a)lists.denx.de> To: Tom Rini <trini(a)konsulko.com> Signed-off-by: Manorit Chawdhry <m-chawdhry(a)ti.com> Changes in v2: - Update the signoff --- Hari Nagalla (2): remoteproc: k3-r5: Extend support for R5F clusters on J721S2 SoCs remoteproc: k3-dsp: Extend support for C71x DSPs on J721S2 SoCs Manorit Chawdhry (1): env: ti: j721s2.env: Remove the processors that are in lockstep board/ti/j721s2/j721s2.env | 2 +- drivers/remoteproc/ti_k3_dsp_rproc.c | 4 +++- drivers/remoteproc/ti_k3_r5f_rproc.c | 6 ++++-- 3 files changed, 8 insertions(+), 4 deletions(-) --- base-commit: 054222eb68f6d7500e21dd1f01af8b9a59b89fd4 change-id: 20231127-b4-upstream-j721e-rproc-support-15074716a401 Best regards, -- Manorit Chawdhry <m-chawdhry(a)ti.com>

2 4

Licensing discrepancies or ambiguities
by Vagrant Cascadian 27 Nov '23

27 Nov '23

I've been reviewing the copyright and license information for Das U-Boot in preparation for uploading to Debian, and found a few surprises. tools/libfdt/fdt_rw.c: /* SPDX-License-Identifier: GPL-2.0+ BSD-2-Clause */ I *think* according to the SPDX spec this needs an OR or an AND. I also see no copyright declaration, although maybe there is a standard interpretation for this. Similar issue with (though thankfully they include copyright declarations): include/bloblist.h:/* SPDX-License-Identifier: GPL-2.0+ BSD-3-Clause */ common/bloblist.c:// SPDX-License-Identifier: GPL-2.0+ BSD-3-Clause doc/README.ubispl:# SPDX-License-Identifier: GPL 2.0+ BSD-3-Clause This one has a non-existent license: test/lib/strlcat.c: // SPDX-License-Identifier: GPL-2.1+ No such license exists, though thankfully it references the exact file in the original glibc sources it came from, which is listed as LGPL-2.1+. There are certainly more ambiguous issues, but figured I would start with those! live well, vagrant

2 3

[PATCH] boards: Disable NET on platforms without NETDEVICES
by Tom Rini 27 Nov '23

27 Nov '23

None of these platforms enabled a networking devices, and disabled CMD_NET. Given that we used to gate network support on CMD_NET rather than NET, we disable CONFIG_NET on these platforms now. Signed-off-by: Tom Rini <trini(a)konsulko.com> --- Cc: Simon Glass <sjg(a)chromium.org> Cc: Heinrich Schuchardt <xypron.glpk(a)gmx.de> Cc: Matthias Winker <matthias.winker(a)de.bosch.com> Cc: Philip Oberfichtner <pro(a)denx.de> Cc: Minkyu Kang <mk7.kang(a)samsung.com> Cc: Jaehoon Chung <jh80.chung(a)samsung.com> Cc: Patrice Chotard <patrice.chotard(a)foss.st.com> Cc: Patrick Delaunay <patrick.delaunay(a)foss.st.com> Cc: Sergey Temerkhanov <s.temerkhanov(a)gmail.com> --- configs/efi-x86_app32_defconfig | 3 +-- configs/efi-x86_app64_defconfig | 3 +-- configs/imx6q_bosch_acc_defconfig | 3 +-- configs/origen_defconfig | 2 +- configs/s5pc210_universal_defconfig | 2 +- configs/stm32mp25_defconfig | 2 +- configs/thunderx_88xx_defconfig | 2 +- configs/trats2_defconfig | 2 +- configs/trats_defconfig | 2 +- 9 files changed, 9 insertions(+), 12 deletions(-) diff --git a/configs/efi-x86_app32_defconfig b/configs/efi-x86_app32_defconfig index 682ba5b89576..e2e6f42c9a9d 100644 --- a/configs/efi-x86_app32_defconfig +++ b/configs/efi-x86_app32_defconfig @@ -21,7 +21,6 @@ CONFIG_SYS_PBSIZE=532 CONFIG_CMD_BOOTZ=y CONFIG_CMD_PART=y # CONFIG_CMD_SCSI is not set -# CONFIG_CMD_NET is not set CONFIG_CMD_TIME=y CONFIG_CMD_EXT2=y CONFIG_CMD_EXT4=y @@ -36,7 +35,7 @@ CONFIG_ENV_OVERWRITE=y CONFIG_SYS_RELOC_GD_ENV_ADDR=y CONFIG_USE_BOOTFILE=y CONFIG_BOOTFILE="bzImage" -CONFIG_USE_ROOTPATH=y +# CONFIG_NET is not set CONFIG_REGMAP=y CONFIG_SYSCON=y # CONFIG_REGEX is not set diff --git a/configs/efi-x86_app64_defconfig b/configs/efi-x86_app64_defconfig index d6b6c3d82995..14b5cd0fa0ee 100644 --- a/configs/efi-x86_app64_defconfig +++ b/configs/efi-x86_app64_defconfig @@ -22,7 +22,6 @@ CONFIG_SYS_PBSIZE=532 CONFIG_CMD_BOOTZ=y CONFIG_CMD_PART=y # CONFIG_CMD_SCSI is not set -# CONFIG_CMD_NET is not set CONFIG_CMD_CACHE=y CONFIG_CMD_TIME=y CONFIG_CMD_EXT2=y @@ -38,7 +37,7 @@ CONFIG_ENV_OVERWRITE=y CONFIG_SYS_RELOC_GD_ENV_ADDR=y CONFIG_USE_BOOTFILE=y CONFIG_BOOTFILE="bzImage" -CONFIG_USE_ROOTPATH=y +# CONFIG_NET is not set CONFIG_REGMAP=y CONFIG_SYSCON=y CONFIG_CONSOLE_SCROLL_LINES=5 diff --git a/configs/imx6q_bosch_acc_defconfig b/configs/imx6q_bosch_acc_defconfig index 05bab034e0f9..6abb7954ba2d 100644 --- a/configs/imx6q_bosch_acc_defconfig +++ b/configs/imx6q_bosch_acc_defconfig @@ -63,7 +63,6 @@ CONFIG_CMD_PART=y # CONFIG_CMD_ITEST is not set # CONFIG_CMD_SOURCE is not set # CONFIG_CMD_SETEXPR is not set -# CONFIG_CMD_NET is not set # CONFIG_CMD_BLOCK_CACHE is not set # CONFIG_CMD_SLEEP is not set # CONFIG_CMD_MP is not set @@ -82,7 +81,7 @@ CONFIG_SYS_MMC_ENV_PART=1 CONFIG_ENV_WRITEABLE_LIST=y CONFIG_ENV_ACCESS_IGNORE_FORCE=y CONFIG_VERSION_VARIABLE=y -CONFIG_TFTP_BLOCKSIZE=512 +# CONFIG_NET is not set CONFIG_SPL_DM=y CONFIG_BOOTCOUNT_LIMIT=y CONFIG_DM_BOOTCOUNT=y diff --git a/configs/origen_defconfig b/configs/origen_defconfig index 3a71d6e582a7..5e55b712df9e 100644 --- a/configs/origen_defconfig +++ b/configs/origen_defconfig @@ -35,13 +35,13 @@ CONFIG_CMD_DFU=y CONFIG_CMD_GPT=y CONFIG_CMD_MMC=y CONFIG_CMD_USB_MASS_STORAGE=y -# CONFIG_CMD_NET is not set CONFIG_CMD_CACHE=y # CONFIG_CMD_SLEEP is not set CONFIG_CMD_EXT4_WRITE=y CONFIG_OF_CONTROL=y CONFIG_ENV_OVERWRITE=y CONFIG_SYS_RELOC_GD_ENV_ADDR=y +# CONFIG_NET is not set CONFIG_DFU_MMC=y CONFIG_SYS_DFU_DATA_BUF_SIZE=0x2000000 CONFIG_MMC_DW=y diff --git a/configs/s5pc210_universal_defconfig b/configs/s5pc210_universal_defconfig index f585c510635a..ac236c82a1fc 100644 --- a/configs/s5pc210_universal_defconfig +++ b/configs/s5pc210_universal_defconfig @@ -33,7 +33,6 @@ CONFIG_CMD_GPT=y CONFIG_CMD_I2C=y CONFIG_CMD_MMC=y CONFIG_CMD_USB_MASS_STORAGE=y -# CONFIG_CMD_NET is not set CONFIG_CMD_CACHE=y # CONFIG_CMD_SLEEP is not set CONFIG_CMD_EXT4_WRITE=y @@ -42,6 +41,7 @@ CONFIG_OF_CONTROL=y CONFIG_ENV_OVERWRITE=y CONFIG_SYS_RELOC_GD_ENV_ADDR=y CONFIG_ENV_VARS_UBOOT_RUNTIME_CONFIG=y +# CONFIG_NET is not set CONFIG_DFU_MMC=y CONFIG_SYS_DFU_DATA_BUF_SIZE=0x2000000 CONFIG_SYS_I2C_S3C24X0=y diff --git a/configs/stm32mp25_defconfig b/configs/stm32mp25_defconfig index 8423943f091c..75f27c98b441 100644 --- a/configs/stm32mp25_defconfig +++ b/configs/stm32mp25_defconfig @@ -23,7 +23,6 @@ CONFIG_CMD_MEMTEST=y CONFIG_CMD_CLK=y CONFIG_CMD_GPIO=y # CONFIG_CMD_LOADB is not set -# CONFIG_CMD_NET is not set CONFIG_CMD_CACHE=y CONFIG_CMD_TIME=y CONFIG_CMD_RNG=y @@ -31,6 +30,7 @@ CONFIG_CMD_TIMER=y CONFIG_CMD_REGULATOR=y CONFIG_CMD_LOG=y CONFIG_OF_LIVE=y +# CONFIG_NET is not set CONFIG_GPIO_HOG=y CONFIG_DM_I2C=y CONFIG_SYS_I2C_STM32F7=y diff --git a/configs/thunderx_88xx_defconfig b/configs/thunderx_88xx_defconfig index 5c24b38bb1fa..0c71b298ada5 100644 --- a/configs/thunderx_88xx_defconfig +++ b/configs/thunderx_88xx_defconfig @@ -32,8 +32,8 @@ CONFIG_SYS_BOOTM_LEN=0x800000 # CONFIG_CMD_EDITENV is not set # CONFIG_CMD_SAVEENV is not set # CONFIG_CMD_ENV_EXISTS is not set -# CONFIG_CMD_NET is not set CONFIG_SYS_RELOC_GD_ENV_ADDR=y +# CONFIG_NET is not set # CONFIG_MMC is not set CONFIG_DM_SERIAL=y CONFIG_DEBUG_UART_SKIP_INIT=y diff --git a/configs/trats2_defconfig b/configs/trats2_defconfig index 6c4909f12be0..045c74f95efb 100644 --- a/configs/trats2_defconfig +++ b/configs/trats2_defconfig @@ -36,7 +36,6 @@ CONFIG_CMD_GPT=y CONFIG_CMD_I2C=y CONFIG_CMD_MMC=y CONFIG_CMD_USB_MASS_STORAGE=y -# CONFIG_CMD_NET is not set CONFIG_CMD_CACHE=y # CONFIG_CMD_SLEEP is not set CONFIG_CMD_EXT4_WRITE=y @@ -44,6 +43,7 @@ CONFIG_OF_CONTROL=y CONFIG_ENV_OVERWRITE=y CONFIG_SYS_RELOC_GD_ENV_ADDR=y CONFIG_ENV_VARS_UBOOT_RUNTIME_CONFIG=y +# CONFIG_NET is not set CONFIG_DFU_MMC=y CONFIG_SYS_DFU_DATA_BUF_SIZE=0x2000000 CONFIG_DM_I2C_GPIO=y diff --git a/configs/trats_defconfig b/configs/trats_defconfig index 78484c071945..b1adaef17a01 100644 --- a/configs/trats_defconfig +++ b/configs/trats_defconfig @@ -35,7 +35,6 @@ CONFIG_CMD_GPT=y CONFIG_CMD_I2C=y CONFIG_CMD_MMC=y CONFIG_CMD_USB_MASS_STORAGE=y -# CONFIG_CMD_NET is not set CONFIG_CMD_CACHE=y # CONFIG_CMD_SLEEP is not set CONFIG_CMD_EXT4_WRITE=y @@ -43,6 +42,7 @@ CONFIG_OF_CONTROL=y CONFIG_ENV_OVERWRITE=y CONFIG_SYS_RELOC_GD_ENV_ADDR=y CONFIG_ENV_VARS_UBOOT_RUNTIME_CONFIG=y +# CONFIG_NET is not set CONFIG_DFU_MMC=y CONFIG_SYS_DFU_DATA_BUF_SIZE=0x2000000 CONFIG_DM_I2C_GPIO=y -- 2.34.1

4 6

[PATCH v1 0/3] efi_vars: SPI Flash store
by Shantur Rathore 27 Nov '23

27 Nov '23

With this patch series we implement SPI Flash storage for EFI variables. Shantur Rathore (3): efi: filestore: don't compile when config disabled efi_vars: Implement SPI Flash store defconfig: rockpro64: Enable SF EFI var store configs/rockpro64-rk3399_defconfig | 2 + include/efi_variable.h | 46 ++++++++++++--- lib/efi_loader/Kconfig | 18 ++++++ lib/efi_loader/Makefile | 1 + lib/efi_loader/efi_var_file.c | 13 +++-- lib/efi_loader/efi_var_sf.c | 91 ++++++++++++++++++++++++++++++ lib/efi_loader/efi_variable.c | 14 ++++- 7 files changed, 169 insertions(+), 16 deletions(-) create mode 100644 lib/efi_loader/efi_var_sf.c -- 2.40.1

3 10

[PATCH] net: mv88e6xxx: add missing SMI address init
by Marek Mojík 27 Nov '23

27 Nov '23

Driver does not currently initialize the smi_addr field, but instead keeps the default value. This leads to issues on systems with different hardware configuration. We can fix this problem by reading the SMI address from device tree. Signed-off-by: Marek Mojík <marek.mojik(a)nic.cz> --- drivers/net/mv88e6xxx.c | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/drivers/net/mv88e6xxx.c b/drivers/net/mv88e6xxx.c index c073f81e72..4636dbf156 100644 --- a/drivers/net/mv88e6xxx.c +++ b/drivers/net/mv88e6xxx.c @@ -753,6 +753,12 @@ static int mv88e6xxx_probe(struct udevice *dev) return -ENODEV; } + priv->smi_addr = dev_read_addr(dev); + if (priv->smi_addr == FDT_ADDR_T_NONE) { + dev_err(dev, "Invalid or missing SMI address\n"); + return -EINVAL; + } + /* probe internal mdio bus */ ret = mv88e6xxx_probe_mdio(dev); if (ret) -- 2.42.1

3 4

Re: [PATCH] net: designware: Support high memory nodes
by Andre Przywara 27 Nov '23

27 Nov '23

On Sun, 26 Nov 2023 16:58:52 +0100 Nils Le Roux <gilbsgilbert(a)gmail.com> wrote: Hi Nils, (adding back the list) > Hi Andre, > > Thank you for the code review and spot-on comments. I'll send a revised > patch soon. In the meantime, I have some silly questions regarding your > first point, please see below: > > On 11/17/23 00:22, Andre Przywara wrote: > > On Sat, 11 Nov 2023 16:26:14 +0100 > > Nils Le Roux <gilbsgilbert(a)gmail.com> wrote: > > > > Hi Nils, > > > >> Some platforms (such as the Lichee Pi 4A) have their dwmac device > >> addressable only in high memory space. Storing the node's base address > >> on 32 bits is not possible in such case. > > > > That's indeed true, so thanks for the patch. > > Some comments below. > > > >> Use platform's physical address type to store the base address. > >> > >> Signed-off-by: Nils Le Roux <gilbsgilbert(a)gmail.com> > >> --- > >> > >> drivers/net/designware.c | 8 ++++---- > >> 1 file changed, 4 insertions(+), 4 deletions(-) > >> > >> diff --git a/drivers/net/designware.c b/drivers/net/designware.c > >> index a174344b3e..327732fbf7 100644 > >> --- a/drivers/net/designware.c > >> +++ b/drivers/net/designware.c > >> @@ -678,8 +678,8 @@ int designware_eth_probe(struct udevice *dev) > >> { > >> struct eth_pdata *pdata = dev_get_plat(dev); > >> struct dw_eth_dev *priv = dev_get_priv(dev); > >> - u32 iobase = pdata->iobase; > >> - ulong ioaddr; > >> + phys_addr_t iobase = pdata->iobase; > >> + phys_addr_t ioaddr; > > > > So I agree that iobase should be a phys_addr_t, since pdata->iobase is > > of this type, but ioaddr seems to be a virtual address, not a physical > > one. So either we make this a void*, or keep it at ulong. > > > > How can you tell ioaddr is a virtual address? Because the value gets assigned to the mac_regs_p and dma_regs_p members of struct dw_eth_dev, and those are clearly pointers, and they are dereferenced at some point. > As far as I can see, it is > only used as a temporary variable to hold iobase which is a physical Well, this assignment is semantically problematic, but works in practice, see below. > address (that's how I inferred it should be the same type). Is virtual > addressing a thing in U-Boot anyways, or are you just speaking of > semantics? I feel like I'm missing something. So conceptually U-Boot has the notion of virtual and physical addresses being something different, but indeed most architectures only ever use a 1:1 mapping when using the MMU, so in practice this doesn't fail. There are exceptions, though, MIPS and some PowerPC builds on the hardware side, and more prominently the sandbox. So we have virt_to_phys() and phys_to_virt() to do conversions, even though they just do casts for most architectures. grep'ing the code for those functions might give you more hints. So you should change the assignment to: ioaddr = phys_to_virt(iobase); That might make it clearer what types are actually required. > More directly: would it be wrong to remove the ioaddr variable and just > set the registers pointers to iobase directly? Aren't they pointers to > physical memory anyways? It would work, and if you have a hardware driver that never runs on MIPS or PPC you will get away with it. But it is of course much cleaner to use the conversion functions. Cheers, Andre > > Thank you again, > Best regards, > Nils > > >> int ret, err; > >> struct reset_ctl_bulk reset_bulk; > >> #ifdef CONFIG_CLK > >> @@ -740,7 +740,7 @@ int designware_eth_probe(struct udevice *dev) > >> * or via a PCI bridge, fill in plat before we probe the hardware. > >> */ > >> if (IS_ENABLED(CONFIG_PCI) && device_is_on_pci_bus(dev)) { > >> - dm_pci_read_config32(dev, PCI_BASE_ADDRESS_0, &iobase); > >> + dm_pci_read_config32(dev, PCI_BASE_ADDRESS_0, (u32 *)&iobase); > > > > This looks sketchy, relies on little endian, and assumes that iobase > > has been initialised to 0 before. Since the read is explicitly 32 bits > > wide, I'd declare a new local u32 variable (inside this if-statement) > > and read it into there, then assign this to iobase. > > > >> iobase &= PCI_BASE_ADDRESS_MEM_MASK; > >> iobase = dm_pci_mem_to_phys(dev, iobase); > >> > >> @@ -748,7 +748,7 @@ int designware_eth_probe(struct udevice *dev) > >> pdata->phy_interface = PHY_INTERFACE_MODE_RMII; > >> } > >> > >> - debug("%s, iobase=%x, priv=%p\n", __func__, iobase, priv); > >> + debug("%s, iobase=%llx, priv=%p\n", __func__, (u64)iobase, priv); > > > > This assumes that u64 is always %llx, which is not universally true. > > Fortunately we have %pa for that (see doc/develop/printf.rst), so you > > can just use this and drop the cast. > > > > Cheers, > > Andre > > > >> ioaddr = iobase; > >> priv->mac_regs_p = (struct eth_mac_regs *)ioaddr; > >> priv->dma_regs_p = (struct eth_dma_regs *)(ioaddr + DW_DMA_BASE_OFFSET); > >

1 0

[PATCH] sunxi: H616: OrangePi Zero 2: enable USB power supply
by Andre Przywara 26 Nov '23

26 Nov '23

The OrangePi Zero 2 has a USB VBUS regulator, controlled by pin PC16. This is correctly described in the DT, but the patches for supporting this are still pending. Meanwhile add our good old CONFIG_USB1_VBUS_PIN to the defconfig file, to enable power on the USB port and allow using a USB flash drive, for instance. Fixes: 6acc5fa581b4 ("sunxi: H616: enable USB support for H616 boards") Reported-by: Mikhail Kalashnikov <iuncuim(a)gmail.com> Signed-off-by: Andre Przywara <andre.przywara(a)arm.com> --- configs/orangepi_zero2_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/configs/orangepi_zero2_defconfig b/configs/orangepi_zero2_defconfig index 127cf9e365a..98aed846f1f 100644 --- a/configs/orangepi_zero2_defconfig +++ b/configs/orangepi_zero2_defconfig @@ -8,6 +8,7 @@ CONFIG_DRAM_SUN50I_H616_CA_DRI=0x0e0e CONFIG_DRAM_SUN50I_H616_TPR10=0xf83438 CONFIG_MACH_SUN50I_H616=y CONFIG_SUNXI_DRAM_H616_DDR3_1333=y +CONFIG_USB1_VBUS_PIN="PC16" CONFIG_R_I2C_ENABLE=y CONFIG_SPL_SPI_SUNXI=y # CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set -- 2.35.8

1 0

Adding support for Orange Pi Zero 3
by electricworry 26 Nov '23

26 Nov '23

I would like to contribute improvements u-boot to enable support for the Orange Pi Zero 3 board. I have already gained a good understanding of the project and the standards for changes, and so I have made modifications and tested them on my board. Before submitting patches for review I've got a few questions I'd like to discuss. The board is an Allwinner H618. From what I can gather this is just an H616 with minor changes. The changes are so minor that I've been able to implement the support as changes to the existing files for H616. For example, to add LPDDR4 support I have been able to make changes to arch/arm/mach-sunxi/dram_sun50i_h616.c and done so in a well configured way that won't impact anything else. Is doing it in this way acceptable or would the preference be to have a separate file (e.g. dram_sun50i_h618.c) to keep H616 and H618 separate? The way I've done it feels right because it really is just a minor upgrade, but there might be other opinions I should hear first. Further, with regards to attribution, all of my changes are based on code in https://github.com/orangepi-xunlong/u-boot-orangepi. I've not been able to cherry-pick changes because that codebase has departed severely from the u-boot master branch and it's not as elegantly maintained, so I've had to surgically move just the changes required to get my board working. Their repo is correctly marked as GPL-2.0 so compatible with merging, but the exact author of every line of code will be difficult to attribute. Is it sufficient for me to cite that downstream repo as the source of the patches? Thanks in advance. I've not submitted a patch here before so I could benefit from some hand-holding to not create problems for people, but I'm pretty confident that the changes I've made comply with the standards of the project and that there won't be much of an issue having them accepted.

4 5

[GIT PULL] Please pull u-boot-dfu-20231124
by Mattijs Korpershoek 26 Nov '23

26 Nov '23

Hi Tom, Here are some fixes for master including: - Fix reinit for ChipIdea controller - Add missing newline in fastboot error handling This CI job is at https://source.denx.de/u-boot/custodians/u-boot-dfu/-/pipelines/18703 Thanks, Mattijs The following changes since commit 24ca49b33af98d54d6cd2e845f071f6565345ffd: Prepare v2024.01-rc3 (2023-11-20 08:43:46 -0500) are available in the Git repository at: https://source.denx.de/u-boot/custodians/u-boot-dfu.git tags/u-boot-dfu-20231124 for you to fetch changes up to 6f68dcd0259f9f1c03f097669e9490b13b5325c7: usb: fastboot: Add missing newline in pr_err (2023-11-21 09:19:48 +0100) ---------------------------------------------------------------- u-boot-dfu-20231124 - Fix reinit for ChipIdea controller - Add missing newline in fastboot error handling ---------------------------------------------------------------- Simon Holesch (2): usb: ci: Fix gadget reinit usb: fastboot: Add missing newline in pr_err drivers/usb/gadget/ci_udc.c | 38 ++++++++++++++++++++++++++++++++++++++ drivers/usb/gadget/f_fastboot.c | 2 +- 2 files changed, 39 insertions(+), 1 deletion(-)

2 1