Re: Converting to DM SERIAL for Kirkwood boards

2022-12-15 Thread Pali Rohár
On Thursday 15 December 2022 18:42:20 Tony Dinh wrote: > Pogo V4 (tested with u-boot-2022.10, since Pogo V4 itself was broken > in u-boot-2023.01-rc3 for unknown reason). Could you try to git bisect which commit broke it?

Re: [PATCH v1] mtd: parsers: ofpart: Fix parsing when size-cells is 0

2022-12-15 Thread Francesco Dolcini
On Fri, Dec 16, 2022 at 01:36:03AM +0100, Marek Vasut wrote: > On 12/15/22 09:04, Miquel Raynal wrote: > > > > That would fix all cases and only have an impact on the affected > > > > boards. > > > > > > Sadly, it does only fix the known cases, not the unknown cases > > > like downstream forks whi

Re: [PATCH] fastboot: remove #ifdef CONFIG when it is possible

2022-12-15 Thread Mattijs Korpershoek
On Thu, Dec 15, 2022 at 14:45, Mattijs Korpershoek wrote: > On Thu, Dec 15, 2022 at 10:15, Patrick Delaunay > wrote: > >> Much of the fastboot code predates the introduction of Kconfig and >> has quite a few #ifdefs in it which is unnecessary now that we can use >> IS_ENABLED() et al. >> >> Si

Re: [PATCH v1] mtd: parsers: ofpart: Fix parsing when size-cells is 0

2022-12-15 Thread Francesco Dolcini
Hello Marek and Miquel, On Thu, Dec 15, 2022 at 08:16:04AM +0100, Miquel Raynal wrote: > So my first first idea was to avoid using the broken "fixup mtdparts" > function in U-Boot and I am still convinced this is what we should do > in priority. This is something that was already discussed, but I

[PATCH] imxtract: specify max gunzip size

2022-12-15 Thread Nikita Shubin
From: Nikita Shubin Specify max gunzip size from config to override SYS_XIMG_LEN default value wich is 0x80. Signed-off-by: Nikita Shubin --- cmd/Kconfig | 7 +++ 1 file changed, 7 insertions(+) diff --git a/cmd/Kconfig b/cmd/Kconfig index d93731f2af6..06e200802d1 100644 --- a/cmd/Kco

Re: [PATCH 2/2] eficonfig: avoid SetVariable between GetNextVariableName calls

2022-12-15 Thread Masahisa Kojima
On Fri, 16 Dec 2022 at 10:58, Heinrich Schuchardt wrote: > > On 12/8/22 05:40, Masahisa Kojima wrote: > > The current code calls efi_set_variable_int() to delete the > > invalid boot option between calls to efi_get_next_variable_name_int(), > > it may produce unpredictable results. > > > > This co

Re: Converting to DM SERIAL for Kirkwood boards

2022-12-15 Thread Stefan Roese
Hi Tony, On 12/16/22 03:42, Tony Dinh wrote: Hi all, On Mon, Dec 12, 2022 at 5:18 PM Tony Dinh wrote: Hi Michael, On Mon, Dec 12, 2022 at 1:03 AM Michael Walle wrote: On 12/9/22 04:55, Tony Dinh wrote: Hi Simon et al, (Resend to include u-boot mailing list) I'm in the process of conv

Re: [PATCH 1/2] eficonfig: curve out efi_get_next_variable_name_int calls

2022-12-15 Thread Masahisa Kojima
On Fri, 16 Dec 2022 at 10:51, Heinrich Schuchardt wrote: > > %s/curve/carve/ Thank you for pointing out the typo. > > On 12/8/22 05:40, Masahisa Kojima wrote: > > To retrieve the EFI variable name by efi_get_next_variable_name_int(), > > the sequence of alloc -> efi_get_next_variable_name_int ->

Re: [PATCH] Prevent buffer overflow on USB control endpoint

2022-12-15 Thread Marek Vasut
On 11/20/22 18:42, Szymon Heidrich wrote: On 20/11/2022 18:25, Marek Vasut wrote: On 11/20/22 16:29, Szymon Heidrich wrote: On 20/11/2022 15:43, Marek Vasut wrote: On 11/17/22 12:50, Fabio Estevam wrote: [Adding Lukasz and Marek] On Thu, Nov 17, 2022 at 6:50 AM Szymon Heidrich wrote: Assu

[PATCH] [RFC] usb: gadget: Convert to usb_gadget_driver callback wrappers

2022-12-15 Thread Marek Vasut
Instead of having the entire USB gadget code base littered with direct calls to usb_gadget_driver members, add wrappers which call those members. These wrappers can perform additional request sanitization as well, esp. the setup wrapper. Convert all the drivers using regex and slight manual adjust

Re: Converting to DM SERIAL for Kirkwood boards

2022-12-15 Thread Tony Dinh
Hi all, On Mon, Dec 12, 2022 at 5:18 PM Tony Dinh wrote: > > Hi Michael, > > On Mon, Dec 12, 2022 at 1:03 AM Michael Walle wrote: > > > > >> On 12/9/22 04:55, Tony Dinh wrote: > > >> > Hi Simon et al, > > >> > > > >> > (Resend to include u-boot mailing list) > > >> > > > >> > I'm in the process

Re: [PATCH 2/2] eficonfig: avoid SetVariable between GetNextVariableName calls

2022-12-15 Thread Heinrich Schuchardt
On 12/8/22 05:40, Masahisa Kojima wrote: The current code calls efi_set_variable_int() to delete the invalid boot option between calls to efi_get_next_variable_name_int(), it may produce unpredictable results. This commit moves removal of the invalid boot option outside of the efi_get_next_varia

Re: [PATCH 1/2] eficonfig: curve out efi_get_next_variable_name_int calls

2022-12-15 Thread Heinrich Schuchardt
%s/curve/carve/ On 12/8/22 05:40, Masahisa Kojima wrote: To retrieve the EFI variable name by efi_get_next_variable_name_int(), the sequence of alloc -> efi_get_next_variable_name_int -> realloc -> efi_get_next_variable_name_int is required. In current code, this sequence repeatedly appears in t

[PATCH 1/1] cmd: allow sound command to play multiple sounds

2022-12-15 Thread Heinrich Schuchardt
Currently the sound command accepts only one value each for duration and frequency. Allowing more duration and frequency arguments enables playing a tune. Signed-off-by: Heinrich Schuchardt --- cmd/sound.c | 41 - doc/usage/cmd/sound.rst | 23 +

[PATCH 1/1] cmd: allow sound command to play multiple sounds

2022-12-15 Thread Heinrich Schuchardt
Currently the sound command accepts only one value each for duration and frequency. Allowing more duration and frequency arguments enables playing a tune. Signed-off-by: Heinrich Schuchardt --- cmd/sound.c | 41 - doc/usage/cmd/sound.rst | 23 +

Re: [PATCH v4 0/2] arm: dts: rockchip: rk3399: usb: ehci: Fix EHCI probe in rk3399 to access peripherals by USB 2.

2022-12-15 Thread Marek Vasut
On 12/9/22 16:39, Xavier Drudis Ferran wrote: arch/arm/dts/rk3399.dtsi has a node usb_host0_ehci: usb@fe38 { compatible = "generic-ehci"; with clocks: clocks = <&cru HCLK_HOST0>, <&cru HCLK_HOST0_ARB>, <&u2phy0>; The first 2 refer to nodes with class UC

Re: [PATCH v1] mtd: parsers: ofpart: Fix parsing when size-cells is 0

2022-12-15 Thread Marek Vasut
On 12/15/22 09:04, Miquel Raynal wrote: Hi Marek, Hi, [...] Yesterday while talking about an ACPI mis-description which needed fixing, I realized fixing up what the firmware provides to Linux should preferably be handled as early as possible. So my first first idea was to avoid using the bro

Re: [PATCH] lds: align u-boot-nodtb with 8 bytes boundary

2022-12-15 Thread Pali Rohár
On Thursday 15 December 2022 06:24:16 Simon Glass wrote: > Hi Eugen, > > On Thu, 15 Dec 2022 at 03:58, Eugen Hristev > wrote: > > > > Newer DTC require that the DTB start address is aligned at 8 bytes. > > In the u-boot.bin case, the u-boot-nodtb.bin is concatenated with the > > DTB, but there i

[PATCH] usb: dwc3: Look up reference clock DT phandle in both controller DT nodes

2022-12-15 Thread Marek Vasut
There are currently four disparate placement possibilities of DWC3 reference clock phandle in SoC DTs: - in top level glue node, with generic subnode without clock (ZynqMP) - in top level generic node, with no subnode (i.MX8MQ) - in generic subnode, with other clock in top level node (i.MX8MP) - in

[PATCH] pylibfdt: Fix version normalization warning

2022-12-15 Thread Marek Vasut
Fix the following version normalization warning: " /usr/lib/python3/dist-packages/setuptools/dist.py:530: UserWarning: Normalizing '2023.01' to '2023.1' " Using suggestion from Richard Jones: https://github.com/pypa/setuptools/issues/308#issuecomment-405817468 Signed-off-by: Marek Vasut --- Cc:

[RFC] [PATCH] imx8mp: power-domain: force usb powerdomain off on startup

2022-12-15 Thread Börge Strümpfel
This patch fixes a bug, where usb-ether is not usable, after an imx8mp board has been booted using the uuu tool. In order to solve this, the usb power domain is turned off during the probe of the imx8mp-hsiomix driver. Thereby the usb controller is reset during startup and can operate normally ther

Re: TFTP boot using DNS

2022-12-15 Thread Valerio Nappi
Hi Harald, Thank you for your reply. As a workaround right now I'm using a patched version that comments out the check for the serverip handler, and it is working. Nevertheless, let's see if we can understand how this should be handled. Best, Valerio On 12/15/22 13:55, Harald Seiler wrote:

[PATCH] adc: stm32mp15: add calibration support

2022-12-15 Thread Olivier Moysan
Add support of offset and linear calibration for STM32MP15. The calibration is performed once at probe. The ADC is set in power on state for calibration. It remains in this state after calibration, to give to the kernel the opportunity to retrieve calibration data, directly from the ADC. Signed-of

Re: [PATCH] mkimage: fit: Support signed configurations in 'auto' FITs

2022-12-15 Thread Simon Glass
Hi Pegorer, On Sun, 11 Dec 2022 at 06:54, Pegorer Massimo wrote: > > Hi, > > The patch follows, as per discussion in email thread "Patch proposal > - mkimage: fit: Support signed conf 'auto' FITs". Let me know if you > prefer something to be changed, or patch to be split in several > commits. >

Re: [PATCH] lds: align u-boot-nodtb with 8 bytes boundary

2022-12-15 Thread Michal Simek
On 12/15/22 16:00, Simon Glass wrote: Hi Eugen, On Thu, 15 Dec 2022 at 06:37, wrote: On 12/15/22 16:24, Simon Glass wrote: Hi Eugen, On Thu, 15 Dec 2022 at 03:58, Eugen Hristev wrote: Newer DTC require that the DTB start address is aligned at 8 bytes. In the u-boot.bin case, the u-boo

Re: [PATCH] fastboot: remove #ifdef CONFIG when it is possible

2022-12-15 Thread Sean Anderson
On 12/15/22 04:15, Patrick Delaunay wrote: > Much of the fastboot code predates the introduction of Kconfig and > has quite a few #ifdefs in it which is unnecessary now that we can use > IS_ENABLED() et al. > > Signed-off-by: Patrick Delaunay > --- > > cmd/fastboot.c | 35

Re: [PATCH] fastboot: remove #ifdef CONFIG when it is possible

2022-12-15 Thread Sean Anderson
On 12/15/22 09:30, Marek Vasut wrote: > On 12/15/22 10:15, Patrick Delaunay wrote: >> Much of the fastboot code predates the introduction of Kconfig and >> has quite a few #ifdefs in it which is unnecessary now that we can use >> IS_ENABLED() et al. >> >> Signed-off-by: Patrick Delaunay >> --- >>

Re: [PATCH] fastboot: remove #ifdef CONFIG when it is possible

2022-12-15 Thread Marek Vasut
On 12/15/22 10:15, Patrick Delaunay wrote: Much of the fastboot code predates the introduction of Kconfig and has quite a few #ifdefs in it which is unnecessary now that we can use IS_ENABLED() et al. Signed-off-by: Patrick Delaunay --- cmd/fastboot.c | 35 +-- dri

Re: [PATCH] imx8mq_pins: fix configuration for UART4 on ECSPI2 pads

2022-12-15 Thread Fabio Estevam
On Thu, Dec 15, 2022 at 11:51 AM Arnaud Ferraris wrote: > diff --git a/arch/arm/include/asm/arch-imx8m/imx8mq_pins.h > b/arch/arm/include/asm/arch-imx8m/imx8mq_pins.h > index c71913f209..16d418c687 100644 > --- a/arch/arm/include/asm/arch-imx8m/imx8mq_pins.h > +++ b/arch/arm/include/asm/arch-imx

Re: [PATCH] lds: align u-boot-nodtb with 8 bytes boundary

2022-12-15 Thread Simon Glass
Hi Eugen, On Thu, 15 Dec 2022 at 06:37, wrote: > > On 12/15/22 16:24, Simon Glass wrote: > > Hi Eugen, > > > > On Thu, 15 Dec 2022 at 03:58, Eugen Hristev > > wrote: > >> > >> Newer DTC require that the DTB start address is aligned at 8 bytes. > >> In the u-boot.bin case, the u-boot-nodtb.bin i

[PATCH] imx8mq_pins: fix configuration for UART4 on ECSPI2 pads

2022-12-15 Thread Arnaud Ferraris
When routing UART4 using the ECSPI2 pads, register IOMUXC_UART4_RXD_SELECT_INPUT (offset 0x050C) should be changed only when dealing with RX, as its name suggests. Signed-off-by: Arnaud Ferraris --- arch/arm/include/asm/arch-imx8m/imx8mq_pins.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(

Re: [PATCH] lds: align u-boot-nodtb with 8 bytes boundary

2022-12-15 Thread Eugen.Hristev
On 12/15/22 16:24, Simon Glass wrote: > Hi Eugen, > > On Thu, 15 Dec 2022 at 03:58, Eugen Hristev > wrote: >> >> Newer DTC require that the DTB start address is aligned at 8 bytes. >> In the u-boot.bin case, the u-boot-nodtb.bin is concatenated with the >> DTB, but there is no alignment/padding

Re: [PATCH] lds: align u-boot-nodtb with 8 bytes boundary

2022-12-15 Thread Simon Glass
Hi Eugen, On Thu, 15 Dec 2022 at 03:58, Eugen Hristev wrote: > > Newer DTC require that the DTB start address is aligned at 8 bytes. > In the u-boot.bin case, the u-boot-nodtb.bin is concatenated with the > DTB, but there is no alignment/padding to the next 8byte aligned address. > This causes th

Re: [PATCH v5 1/3] efi_loader: Add SPI I/O protocol support

2022-12-15 Thread Simon Glass
Yes, that's it. Thanks. - Simon On Wed, 14 Dec 2022 at 01:57, Paul Barker wrote: > > On 14/12/2022 04:39, Simon Glass wrote: > > Hi Paul, > > > > On Wed, 23 Nov 2022 at 10:50, Paul Barker wrote: > >> > >> This addition allows UEFI applications running under u-boot to access > >> peripherals on

Re: [PATCH v5 2/3] efi_selftest: Add tests for SPI protocol support

2022-12-15 Thread Simon Glass
On Wed, 23 Nov 2022 at 09:50, Paul Barker wrote: > > These tests ensure that the new UEFI SPI I/O protocol support works as > expected. They intended to execute on the sandbox target. > > Signed-off-by: Paul Barker > --- > MAINTAINERS | 1 + > arch/sandbox/dts/

Re: [PATCH v2] cmd: pxe_utils: Limit fdtcontroladdr usage to non-fitImage

2022-12-15 Thread Peter Hoyes
On 14/12/2022 15:25, Marek Vasut wrote: On 12/14/22 16:23, Quentin Schulz wrote: Hi Marek, Hi, On 12/14/22 07:45, Marek Vasut wrote: Commit d5ba6188dfb ("cmd: pxe_utils: Check fdtcontroladdr in label_boot") forces '$fdtcontroladdr' DT address as a third parameter of bootm command even if t

Re: [PATCH] fastboot: remove #ifdef CONFIG when it is possible

2022-12-15 Thread Mattijs Korpershoek
On Thu, Dec 15, 2022 at 10:15, Patrick Delaunay wrote: > Much of the fastboot code predates the introduction of Kconfig and > has quite a few #ifdefs in it which is unnecessary now that we can use > IS_ENABLED() et al. > > Signed-off-by: Patrick Delaunay Hi Patrick, Thank you for this, it's a

Re: TFTP boot using DNS

2022-12-15 Thread Harald Seiler
Hi Valerio, On Thu, 2022-12-15 at 10:27 +0100, Valerio Nappi wrote: > Hello everyone, > > Context: > I'm new to the u-boot project. I am trying to boot a Zynq SoC from an > internal TFTP server, but i have no guarantee that the IP of the > server is going to stay the same. A DNS service is offere

TFTP boot using DNS

2022-12-15 Thread Valerio Nappi
Hello everyone, Context: I'm new to the u-boot project. I am trying to boot a Zynq SoC from an internal TFTP server, but i have no guarantee that the IP of the server is going to stay the same. A DNS service is offered, and kept updated. When i try to update the serverip variable using the dns

[PATCH v4 07/25] ram: rockchip: Add rv1126 ddr driver support

2022-12-15 Thread Jagan Teki
Add DDR driver for Rockchip RV1126 SoC. Signed-off-by: YouMin Chen Signed-off-by: Jagan Teki Reviewed-by: Kever Yang --- Changes for v4: - none Changes for v3: - collect Kever r-b Changes for v2: - none .../asm/arch-rockchip/dram_spec_timing.h | 452 +++ .../include/asm/arch-rockchip/sd

[PATCH] lds: align u-boot-nodtb with 8 bytes boundary

2022-12-15 Thread Eugen Hristev
Newer DTC require that the DTB start address is aligned at 8 bytes. In the u-boot.bin case, the u-boot-nodtb.bin is concatenated with the DTB, but there is no alignment/padding to the next 8byte aligned address. This causes the board to fail booting, because the FDT will claim that the DTB inside u

Re: [PATCH] net: zynq_gem: Add support for new compatible str with xlnx prefix

2022-12-15 Thread Michal Simek
On 12/9/22 16:19, Michal Simek wrote: cdns prefix was deprecated and replaced by xlnx one. Signed-off-by: Michal Simek --- Link: https://lore.kernel.org/r/20220726070802.26579-1-krzysztof.kozlow...@linaro.org --- drivers/net/zynq_gem.c | 6 +- 1 file changed, 5 insertions(+), 1 dele

Re: [PATCH 0/5] arm64: zynqmp: DT sync with Linux kernel

2022-12-15 Thread Michal Simek
On 12/9/22 13:56, Michal Simek wrote: Hi, I found some differences between kernel and u-boot that's why do small sync up. Thanks, Michal Michal Simek (5): arm64: zynqmp: Sync #dma-cells property location arm64: zynqmp: Added GEM reset definitions arm64: zynqmp: Fix AMS device tree

Re: [PATCH v2] fastboot: Add OEM run command

2022-12-15 Thread Patrick DELAUNAY
Hi, On 12/6/22 11:23, Patrick DELAUNAY wrote: Hi, On 12/5/22 20:15, Sean Anderson wrote: On 12/5/22 14:04, Patrick DELAUNAY wrote: Hi, On 12/2/22 22:03, Sean Anderson wrote: This adds the UUU UCmd functionality as an OEM command. While the fastboot tool allows sending arbitrary commands as

[PATCH] fastboot: remove #ifdef CONFIG when it is possible

2022-12-15 Thread Patrick Delaunay
Much of the fastboot code predates the introduction of Kconfig and has quite a few #ifdefs in it which is unnecessary now that we can use IS_ENABLED() et al. Signed-off-by: Patrick Delaunay --- cmd/fastboot.c | 35 +-- drivers/fastboot/fb_command.c | 104

[PATCH] optee: bind the TA drivers on OP-TEE node

2022-12-15 Thread Patrick Delaunay
In U-Boot driver model the devices can be referenced by phandle in the U-Boot configuration nodes. Without a valid node provided during the bind, the driver associated to OP-TEE TA can't be referenced. For example to force the sequence number with alias (.flags = DM_UC_FLAG_SEQ_ALIAS) al

Re: [PATCH v1] mtd: parsers: ofpart: Fix parsing when size-cells is 0

2022-12-15 Thread Miquel Raynal
Hi Marek, ma...@denx.de wrote on Thu, 15 Dec 2022 08:45:33 +0100: > On 12/15/22 08:16, Miquel Raynal wrote: > > Hi Marek & Francesco, > > Hi, > > > ma...@denx.de wrote on Mon, 5 Dec 2022 17:25:11 +0100: > > > >> On 12/5/22 14:49, Miquel Raynal wrote: > >>> Hi Francesco, > >> > >> Hi, >

Re: [PATCH 3/3] ARM: dts: stm32: Add timer interrupts on stm32mp15

2022-12-15 Thread Patrice CHOTARD
On 12/14/22 16:25, Patrick Delaunay wrote: > The timer units in the stm32mp15x CPUs have interrupts, depending on the > timer flavour either one "global" or four dedicated ones. Add the irqs > to the timer units on stm32mp15x. > > Sync the DT Files with linux kernel v6.1 and with commit a9b7010

Re: [PATCH 2/3] ARM: dts: stm32mp15: fix typo in stm32mp15xx-dkx.dtsi

2022-12-15 Thread Patrice CHOTARD
On 12/14/22 16:25, Patrick Delaunay wrote: > Remove unnecessary space in device tree stm32mp15xx-dkx.dtsi. > > Signed-off-by: Patrick Delaunay > --- > > arch/arm/dts/stm32mp15xx-dkx.dtsi | 10 +- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a/arch/arm/dts/stm32mp

Re: [PATCH 1/3] ARM: dts: stm32mp15: remove clksrc include in SCMI dtsi file

2022-12-15 Thread Patrice CHOTARD
Hi Patrick On 12/14/22 16:24, Patrick Delaunay wrote: > The include file stm32mp1-clksrc.h is not necessary for the SCMI STM32MP15 > dtsi files as the clock tree is not defined in the U-Boot SCMI device tree; > these SCMI device tree only support TFABOOT with stm32mp15_defconfig, > SPL with the ba