Re: [PATCH 25/42] test: Allow connecting to a running board

2024-06-25 Thread Andrejs Cainikovs
On Tue, Jun 25, 2024 at 08:14:17AM -0600, Tom Rini wrote: > On Tue, Jun 25, 2024 at 01:31:19PM +0100, Simon Glass wrote: > > Hi Andrejs, > > > > On Tue, 25 Jun 2024 at 00:56, Andrejs Cainikovs > > wrote: > > > > > > On Tue, Jun 11, 2024 at 02:01:39PM -0600, Simon Glass wrote: > > > > Sometimes we

Re: [PATCH v3 4/6] buildman: Always use the full path in CROSS_COMPILE

2024-06-25 Thread Andrejs Cainikovs
On Tue, Jun 25, 2024 at 01:38:07PM +0100, Simon Glass wrote: > Hi Andrejs, > > On Tue, 25 Jun 2024 at 01:06, Andrejs Cainikovs > wrote: > > > > On Sun, Jun 23, 2024 at 11:56:20AM -0600, Simon Glass wrote: > > > The feature to set the toolchain path does not seem to be needed. It > > > causes prob

Re: [ANN] U-Boot v2024.07-rc5 released

2024-06-25 Thread LEROY Christophe
Hi All, Le 24/06/2024 à 21:30, Tom Rini a écrit : > Hey all, > > Well, once again I need to check my calender reminders since, whoops, > I'm a week late. That said, looking at the list of changes in this > slightly elongated period, I think it's OK. At this point we're a week > out from the relea

Re: [PATCH v5 0/3] Introduce mtdblock device

2024-06-25 Thread Alexey Romanov
Hi Simon, On Wed, Jun 05, 2024 at 07:17:34AM -0600, Simon Glass wrote: > Hi Alexey, > > On Wed, 5 Jun 2024 at 04:09, Alexey Romanov > wrote: > > > > Hi Simon, > > your message is empty. > > > > On Tue, Jun 04, 2024 at 08:13:34PM -0600, Simon Glass wrote: > > > Hi Alexey, > > > > > > On Mon, Jun

Re: [ANN] U-Boot v2024.07-rc5 released

2024-06-25 Thread Christophe Leroy
Le 25/06/2024 à 17:19, Tom Rini a écrit : On Tue, Jun 25, 2024 at 03:12:38PM +, LEROY Christophe wrote: Hi All, Le 24/06/2024 à 21:30, Tom Rini a écrit : Hey all, Well, once again I need to check my calender reminders since, whoops, I'm a week late. That said, looking at the list of ch

Re: [ANN] U-Boot v2024.07-rc5 released

2024-06-25 Thread Tom Rini
On Tue, Jun 25, 2024 at 03:12:38PM +, LEROY Christophe wrote: > Hi All, > > Le 24/06/2024 à 21:30, Tom Rini a écrit : > > Hey all, > > > > Well, once again I need to check my calender reminders since, whoops, > > I'm a week late. That said, looking at the list of changes in this > > slightly

Re: [PATCH v1] board: mpfs_icicle: implement board_fdt_blob_setup()

2024-06-25 Thread Tom Rini
On Tue, Jun 25, 2024 at 10:08:06AM +0100, Conor Dooley wrote: > The firmware on the Icicle is capable of providing a devicetree in a1 to > U-Boot, but until now the devicetree has been packaged in a "payload" [1] > alongside U-Boot (or other bootloaders/RTOSes) and appended to the image. > The add

Re: [PATCH v3 08/19] test: Introduce the concept of a role

2024-06-25 Thread Tom Rini
On Tue, Jun 25, 2024 at 01:38:08PM +0100, Simon Glass wrote: > Hi Tom, > > On Mon, 24 Jun 2024 at 19:13, Tom Rini wrote: > > > > On Sun, Jun 23, 2024 at 02:32:02PM -0600, Simon Glass wrote: > > > > > In Labgrid there is the concept of a 'role', which is similar to the > > > U-Boot board ID in U-B

Re: [PATCH v3 06/19] test: Avoid failing skipped tests

2024-06-25 Thread Tom Rini
On Tue, Jun 25, 2024 at 01:38:00PM +0100, Simon Glass wrote: > Hi Tom, > > On Mon, 24 Jun 2024 at 19:06, Tom Rini wrote: > > > > On Sun, Jun 23, 2024 at 02:32:00PM -0600, Simon Glass wrote: > > > > > When a test returns -EAGAIN this should not be considered a failure. > > > Fix what seems to be a

Re: [PATCH 13/14] Update u-boot.cfg to include CFG also

2024-06-25 Thread Tom Rini
On Tue, Jun 25, 2024 at 01:38:04PM +0100, Simon Glass wrote: > Hi Tom, > > On Mon, 24 Jun 2024 at 19:29, Tom Rini wrote: > > > > On Sun, Jun 23, 2024 at 02:30:32PM -0600, Simon Glass wrote: > > > > > Some configuration is now in variables with a CFG_ prefix. Add these to > > > the .cfg file so th

Re: [PATCH 25/42] test: Allow connecting to a running board

2024-06-25 Thread Tom Rini
On Tue, Jun 25, 2024 at 01:31:19PM +0100, Simon Glass wrote: > Hi Andrejs, > > On Tue, 25 Jun 2024 at 00:56, Andrejs Cainikovs > wrote: > > > > On Tue, Jun 11, 2024 at 02:01:39PM -0600, Simon Glass wrote: > > > Sometimes we know that the board is already running the right software, > > > so provi

Re: [PATCH] imx: hab: Make imx_hab_is_enabled dependent on FIELD_RETURN

2024-06-25 Thread Ye Li
Hi Paul, On 6/24/2024 8:09 PM, Fabio Estevam wrote: Hi Paul, On Fri, Jun 21, 2024 at 10:06 AM Paul Geurts wrote: -struct imx_sec_config_fuse_t { +struct imx_fuse_t { Please make the struct renaming a separate patch. Peng Fan, Ye Li, Could you please help review this patch? Thanks Can

Re: [PATCH v3 08/19] test: Introduce the concept of a role

2024-06-25 Thread Simon Glass
Hi Tom, On Mon, 24 Jun 2024 at 19:13, Tom Rini wrote: > > On Sun, Jun 23, 2024 at 02:32:02PM -0600, Simon Glass wrote: > > > In Labgrid there is the concept of a 'role', which is similar to the > > U-Boot board ID in U-Boot's pytest subsystem. > > > > The role indicates both the target and inform

Re: [PATCH v3 4/6] buildman: Always use the full path in CROSS_COMPILE

2024-06-25 Thread Simon Glass
Hi Andrejs, On Tue, 25 Jun 2024 at 01:06, Andrejs Cainikovs wrote: > > On Sun, Jun 23, 2024 at 11:56:20AM -0600, Simon Glass wrote: > > The feature to set the toolchain path does not seem to be needed. It > > causes problems with venv (see [1]). Let's remove it. > > > > Add some tests while we ar

Re: [PATCH 13/14] Update u-boot.cfg to include CFG also

2024-06-25 Thread Simon Glass
Hi Tom, On Mon, 24 Jun 2024 at 19:29, Tom Rini wrote: > > On Sun, Jun 23, 2024 at 02:30:32PM -0600, Simon Glass wrote: > > > Some configuration is now in variables with a CFG_ prefix. Add these to > > the .cfg file so that we can see everything in one place. Sort the > > options so they are easie

Re: [PATCH v3 06/19] test: Avoid failing skipped tests

2024-06-25 Thread Simon Glass
Hi Tom, On Mon, 24 Jun 2024 at 19:06, Tom Rini wrote: > > On Sun, Jun 23, 2024 at 02:32:00PM -0600, Simon Glass wrote: > > > When a test returns -EAGAIN this should not be considered a failure. > > Fix what seems to be a problem case, where the pytests see a failure > > when a test has merely bee

Re: [PATCH 25/42] test: Allow connecting to a running board

2024-06-25 Thread Simon Glass
Hi Andrejs, On Tue, 25 Jun 2024 at 00:56, Andrejs Cainikovs wrote: > > On Tue, Jun 11, 2024 at 02:01:39PM -0600, Simon Glass wrote: > > Sometimes we know that the board is already running the right software, > > so provide an option to allow running of tests directly, without first > > resetting

Re: [RESEND PATCH v2 1/5] x86: provide mb() macro

2024-06-25 Thread Simon Glass
Hi Philip, On Mon, 24 Jun 2024 at 02:35, Philip Oberfichtner wrote: > > Implement memory barrier using mfence. Linux does it equivalently [1]. > > "The MFENCE instruction establishes a memory fence for both loads and > stores" [2]. > > [1] linux/arch/x86/include/asm/barrier.h > [2] Intel® 64 and

Re: [PATCH 14/14] smbios: Correct error handling when writing tables

2024-06-25 Thread Simon Glass
Hi Heinrich, On Sun, 23 Jun 2024 at 22:41, Heinrich Schuchardt wrote: > > > > Am 23. Juni 2024 22:30:33 MESZ schrieb Simon Glass : > >Since write_smbios_table() returns an address, we cannot use it to > >return and error number. Also, failing on sysinfo_detect() breaks > > IS_ERR_VALUE() could se

Re: [PATCH v3 19/19] CI: Allow running tests on sjg lab

2024-06-25 Thread Simon Glass
Hi, On Mon, 24 Jun 2024 at 19:01, Tom Rini wrote: > > On Mon, Jun 24, 2024 at 04:56:02PM +0200, Michael Nazzareno Trimarchi wrote: > > Hi Simon > > > > On Mon, Jun 24, 2024 at 2:52 PM Andrejs Cainikovs > > wrote: > > > > > > On Sun, Jun 23, 2024 at 02:32:13PM -0600, Simon Glass wrote: > > > > Ad

Re: [PATCH 1/3] net: give a different name to rtl8169 interfaces

2024-06-25 Thread Etienne Dublé
Hello, Le 25/06/2024 à 12:25, Dragan Simic a écrit : Another option may be to name them "rtl8169@", with "" reflecting the PCI region address (so it is unique and stable). What do you think? I guess that's one way, I'm also wondering how systemd renames those to be unique but stable on the

Re: [PATCH 2/3] rockchip: fix wrong PCI range address in rk3568 dtsi

2024-06-25 Thread Etienne Dublé
Hello Jonas, Le 25/06/2024 à 12:46, Jonas Karlman a écrit : Ahh, linux is still missing a patch to be able to use full address ranges as a root complex. Will re-run some tests on my R5C on both u-boot and linux to see if I can replicate your issue. OK. To use the second interface in u-boot,

Re: [PATCH 2/3] rockchip: fix wrong PCI range address in rk3568 dtsi

2024-06-25 Thread Jonas Karlman
Hello Etienne, On 2024-06-25 12:12, Etienne Dublé wrote: > > Hello Jonas, > > Le 25/06/2024 à 10:29, Jonas Karlman a écrit : >> I do not understand the need for such/this patch. The changed address is >> the internal io address that is shared with the pci controller and >> devices. >> >> Do you

Re: [PATCH v1 1/3] mtd: rawnand: nand_base: support for 'NAND_IS_BOOT_MEDIUM' flag

2024-06-25 Thread Arseniy Krasnov
On 24.06.2024 12:14, Alexander Dahl wrote: > Hello Arseniy, > > Am Sun, Jun 02, 2024 at 11:08:34PM +0300 schrieb Arseniy Krasnov: >> Based on Linux kernel: >> commit f922bd798bb9 ("mtd: rawnand: add an option to specify NAND chip as a >> boot device") >> >> Allow to define a NAND chip as a boo

Re: [PATCH 1/3] net: give a different name to rtl8169 interfaces

2024-06-25 Thread Dragan Simic
Hello, On 2024-06-25 12:03, Quentin Schulz wrote: On 6/25/24 8:38 AM, Etienne Dublé wrote: Thanks for reviewing my patches. Le 24/06/2024 à 15:08, Quentin Schulz a écrit : [...]   1 file changed, 11 insertions(+) diff --git a/drivers/net/rtl8169.c b/drivers/net/rtl8169.c index 93e83661ce..

Re: [PATCH 2/3] rockchip: fix wrong PCI range address in rk3568 dtsi

2024-06-25 Thread Etienne Dublé
Hello Jonas, Le 25/06/2024 à 10:29, Jonas Karlman a écrit : I do not understand the need for such/this patch. The changed address is the internal io address that is shared with the pci controller and devices. Do you have any issue in linux or is it only in U-Boot?, I suspect this change is on

Re: [PATCH 2/3] rockchip: fix wrong PCI range address in rk3568 dtsi

2024-06-25 Thread Mark Kettenis
> Date: Tue, 25 Jun 2024 10:29:48 +0200 > From: Jonas Karlman Hi Jonas, > Hi Etienne, > > On 2024-06-25 08:47, Etienne Dublé wrote: > > Hi Quentin, > > > > Le 24/06/2024 à 15:15, Quentin Schulz a écrit : > >> Hi Etienne, > >> > >> On 6/24/24 2:40 PM, ETIENNE DUBLE wrote: > >>> One of the PCI r

Re: [PATCH 1/3] net: give a different name to rtl8169 interfaces

2024-06-25 Thread Quentin Schulz
Hi Etienne, On 6/25/24 8:38 AM, Etienne Dublé wrote: Hi Quentin, Thanks for reviewing my patches. Le 24/06/2024 à 15:08, Quentin Schulz a écrit : [...]   1 file changed, 11 insertions(+) diff --git a/drivers/net/rtl8169.c b/drivers/net/rtl8169.c index 93e83661ce..b30d51731f 100644 --- a/dr

Re: [PATCH 5/7] powerpc: Implement a default flush_dcache_all

2024-06-25 Thread Ilias Apalodimas
On Thu, 20 Jun 2024 at 00:28, Tom Rini wrote: > > Implement a weak default version of flush_dcache_all which is based on > the ARM default, which is to flush the entire range via > flush_dcache_range(...). > > Signed-off-by: Tom Rini > --- > arch/powerpc/lib/cache.c | 9 + > 1 file chang

[PATCH v1] board: mpfs_icicle: implement board_fdt_blob_setup()

2024-06-25 Thread Conor Dooley
The firmware on the Icicle is capable of providing a devicetree in a1 to U-Boot, but until now the devicetree has been packaged in a "payload" [1] alongside U-Boot (or other bootloaders/RTOSes) and appended to the image. The address of this appended devicetree is placed in a1 by the firmware. This

Re: [PATCH 2/3] rockchip: fix wrong PCI range address in rk3568 dtsi

2024-06-25 Thread Jonas Karlman
Hi Etienne, On 2024-06-25 08:47, Etienne Dublé wrote: > Hi Quentin, > > Le 24/06/2024 à 15:15, Quentin Schulz a écrit : >> Hi Etienne, >> >> On 6/24/24 2:40 PM, ETIENNE DUBLE wrote: >>> One of the PCI ranges was wrong in this device tree. >>> When testing with a FriendlyElec Nanopi R5C board, the

Re: [PATCH v3] ARM: imx: verdin-imx8mm: Set CAN oscillator frequency based on model

2024-06-25 Thread Francesco Dolcini
Hello Marek, thank for taking care of this. On Tue, Jun 25, 2024 at 10:26:15AM +0200, Marek Vasut wrote: > The older i.MX8M Mini Verdin SoMs may came with 20 MHz SPI CAN controller > oscillator, the newer SoMs always use 40 MHz oscillator. Handle both by > overriding the oscillator frequency just

[PATCH v3] ARM: imx: verdin-imx8mm: Set CAN oscillator frequency based on model

2024-06-25 Thread Marek Vasut
The older i.MX8M Mini Verdin SoMs may came with 20 MHz SPI CAN controller oscillator, the newer SoMs always use 40 MHz oscillator. Handle both by overriding the oscillator frequency just before booting the kernel. These are the known variants with 20 MHz oscillator: - 0055, V1.1A, V1.1B, V1.1C and

Re: [PATCH 3/7] sh: Implement a default flush_dcache_all

2024-06-25 Thread Ilias Apalodimas
On Thu, 20 Jun 2024 at 00:28, Tom Rini wrote: > > Implement a weak default version of flush_dcache_all which is based on > the ARM default, which is to flush the entire range via > flush_dcache_range(...). > > Signed-off-by: Tom Rini > --- > Cc: Marek Vasut > Cc: Nobuhiro Iwamatsu > --- > arch

Re: [PATCH 1/7] m68k: Implement a default flush_dcache_all

2024-06-25 Thread Ilias Apalodimas
On Thu, 20 Jun 2024 at 00:28, Tom Rini wrote: > > Implement a weak default version of flush_dcache_all which is based on > the ARM default, which is to flush the entire range via > flush_dcache_range(...). > > Signed-off-by: Tom Rini > --- > Cc: Huan Wang > Cc: Angelo Dureghello > --- > arch/m

Re: [PATCH v4 00/14] Introduce the lwIP network stack

2024-06-25 Thread Jerome Forissier
On 6/25/24 00:28, Tim Harvey wrote: > On Sat, Jun 22, 2024 at 1:09 AM Maxim Uvarov wrote: >> >> пт, 21 июн. 2024 г. в 21:42, Fabio Estevam : >>> >>> Hi Tim and Jerome, >>> >>> On Fri, Jun 21, 2024 at 1:08 PM Tim Harvey wrote: >>> I tried your to-upstream/v5-wip branch (042bea36eb9731079

Re: [PATCH v4 10/14] net-lwip: add wget command

2024-06-25 Thread Jerome Forissier
On 6/25/24 05:20, Jon Humphreys wrote: > Jerome Forissier writes: > >> On 6/24/24 08:21, Jon Humphreys wrote: >>> Jerome Forissier writes: >>> Add support for the wget command with NET_LWIP. Based on code initially developed by Maxim U. Signed-off-by: Jerome Forissier >