Re: Please pull u-boot-dm

2024-07-26 Thread Tom Rini
On Fri, Jul 26, 2024 at 08:54:50AM -0600, Simon Glass wrote: > Hi Tom, > > https://source.denx.de/u-boot/custodians/u-boot-dm/-/pipelines/21781 > https://dev.azure.com/simon0972/u-boot/_build/results?buildId=65&view=results > > The following changes since commit f4e163ece454bcf949a9b2005de3b7039

[PATCH] boot: android: fix booting without a ramdisk

2024-07-26 Thread Michael Walle
android_image_get_ramdisk() will return an error if there is no ramdisk. Using the android image without a ramdisk worked until commit 1ce8e10f3b4b ("image: Fix up ANDROID_BOOT_IMAGE ramdisk code") because that return code wasn't checked. Now that it is checked, don't return an error in the (valid)

Re: [PATCHv3 04/12] cpu: imx: fix the CPU frequency in cpu_imx_get_info()

2024-07-26 Thread Fabio Estevam
On Fri, Jul 26, 2024 at 2:30 PM Zhiqiang Hou wrote: > > From: Hou Zhiqiang > > 1MHz = 100Hzz Please provide a proper commit log with an explanation. This Hzz makes no sense. Please retain the Reviewed-by tags from Simon on the previous version.

Re: [PATCHv2] clk: imx8m: register ARM A53 core clock

2024-07-26 Thread Marek Vasut
On 7/26/24 6:01 PM, Z.Q. Hou wrote: Hi Marek, Hi, -Original Message- From: Marek Vasut Sent: Friday, July 26, 2024 10:05 PM To: Z.Q. Hou ; u-boot@lists.denx.de; tr...@konsulko.com; lu...@denx.de; sean...@gmail.com; feste...@gmail.com; Peng Fan Subject: Re: [PATCHv2] clk: imx8m: regi

Re: M68K Vectors

2024-07-26 Thread Peter LaDow
Scratch that. I forgot I hard coded the vector table with 0x400 to test things. Restoring _start still results in 0x for the reset vector. On Fri, Jul 26, 2024 at 1:16 PM Peter LaDow wrote: > > I should mention I was using the gcc-m68k-linux-gnu package on Ubuntu > 22.04.4, which pulled

Re: M68K Vectors

2024-07-26 Thread Peter LaDow
I should mention I was using the gcc-m68k-linux-gnu package on Ubuntu 22.04.4, which pulled in gcc-11-m68k-linux-gnu. I just downloaded the bootlin m68k-coldfire--uclibc--stable-2024.02-1, and tried that. It generates the proper value in the vector table (0x400 for _start). But the call to memse

Re: M68K Vectors

2024-07-26 Thread Fabio Estevam
Adding the Coldfire maintainers on Cc. On Fri, Jul 26, 2024 at 4:46 PM Peter LaDow wrote: > > After some digging it appears that this is a toolchain issue. It seems the > linker fixups are sometimes not computed correctly. For example, in > board_init_f_init_reserve, the object file disassemble

Re: M68K Vectors

2024-07-26 Thread Peter LaDow
After some digging it appears that this is a toolchain issue. It seems the linker fixups are sometimes not computed correctly. For example, in board_init_f_init_reserve, the object file disassembled has: : 0: 2f02movel %d2,%sp@- 2: 242f 0008 movel %sp@(8),%d2

[PATCH] patman: Resolve python string vs. regex escaping syntax

2024-07-26 Thread Brian Norris
Python strings have their own notion of backslash-escaping, and that can conflict with the intentions for strings passed to the 're' module. In particular, I get warnings like this: tools/patman/../patman/commit.py:9: SyntaxWarning: invalid escape sequence '\s' re_subject_tag = re.compile('([^:\

Re: [PATCH v2 00/40] mmc: dw_mmc: Enable eMMC on E850-96 board

2024-07-26 Thread Sam Protsenko
On Fri, Jul 26, 2024 at 12:27 PM Minkyu Kang wrote: > > Hello, > > > 2024년 7월 26일 (금) 05:58, Sam Protsenko 님이 작성: >> >> On Thu, Jul 25, 2024 at 1:25 PM Michael Nazzareno Trimarchi >> wrote: >> > >> > Hi all >> > >> > Il mar 9 lug 2024, 02:48 Sam Protsenko ha >> > scritto: >> >> >> >> On Thu, Ju

[PATCHv3 12/12] configs: imx93: enable the 'cpu' command

2024-07-26 Thread Zhiqiang Hou
From: Hou Zhiqiang Enable the 'cpu' command to display the CPU info and release CPU core to run baremetal or RTOS applications. Signed-off-by: Hou Zhiqiang --- V3: - No change. configs/imx93_11x11_evk_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/configs/imx93_11x11_evk_defco

[PATCHv3 11/12] configs: imx8m: enable the 'cpu' command

2024-07-26 Thread Zhiqiang Hou
From: Hou Zhiqiang Enable the 'cpu' command and the depended imx CPU driver to display the CPU info and release CPU core to run baremetal or RTOS applications. Signed-off-by: Hou Zhiqiang --- V3: - No change. configs/imx8mm_evk_defconfig | 3 +++ configs/imx8mn_evk_defconfig | 3 +++ configs

[PATCHv3 10/12] MAINTAINERS: add entry for cpu command

2024-07-26 Thread Zhiqiang Hou
From: Hou Zhiqiang Added the original author Simon and myself. Signed-off-by: Hou Zhiqiang --- V3: - New patch. MAINTAINERS | 7 +++ 1 file changed, 7 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index a6e47e8a217..f1aa802758d 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -996,6

[PATCHv3 09/12] doc: cmd: add documentation for cpu command

2024-07-26 Thread Zhiqiang Hou
From: Hou Zhiqiang Add documentation for the 'cpu' command, taking NXP i.MX 8M Plus as a example. Signed-off-by: Hou Zhiqiang --- V3: - New patch. doc/usage/cmd/cpu.rst | 82 +++ 1 file changed, 82 insertions(+) create mode 100644 doc/usage/cmd/cpu.rs

[PATCHv3 07/12] cpu: imx: implement release_core callback

2024-07-26 Thread Zhiqiang Hou
From: Hou Zhiqiang Release the secondary cores through the PSCI request. Signed-off-by: Hou Zhiqiang --- V3: - No change. drivers/cpu/imx8_cpu.c | 24 1 file changed, 24 insertions(+) diff --git a/drivers/cpu/imx8_cpu.c b/drivers/cpu/imx8_cpu.c index c1315cc32dd..85

[PATCHv3 08/12] cmd: cpu: add release subcommand

2024-07-26 Thread Zhiqiang Hou
From: Hou Zhiqiang Add a new subcommand 'release' to bring up a core to run baremetal and RTOS applications. For example on i.MX8M Plus EVK, release the LAST core to run a RTOS application, passing the sequence number of the CPU core to release, here it is 3: u-boot=> cpu list 0: cpu@0

[PATCHv3 06/12] cpu: imx: Add i.MX 8M series SoCs

2024-07-26 Thread Zhiqiang Hou
From: Hou Zhiqiang Add i.MX 8M Mini, Nano and Plus SoCs support. Signed-off-by: Hou Zhiqiang --- V3: - No change. drivers/cpu/imx8_cpu.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/drivers/cpu/imx8_cpu.c b/drivers/cpu/imx8_cpu.c index 6a97b7b9ad0..c1315cc32dd 100644 --- a/drive

[PATCHv3 05/12] cpu: imx: fix the CPU type field width

2024-07-26 Thread Zhiqiang Hou
From: Hou Zhiqiang Increase one more bit to cover all CPU types. Otherwise it shows wrong CPU info on some platforms, such as i.MX8M Plus: U-Boot 2024.04+g674440bc73e+p0 (Jun 06 2024 - 10:05:34 +) CPU: NXP i.MX8MM Rev1.1 A53 at 4154504685 MHz at 30C Model: NXP i.MX8MPlus LPDD

[PATCHv3 04/12] cpu: imx: fix the CPU frequency in cpu_imx_get_info()

2024-07-26 Thread Zhiqiang Hou
From: Hou Zhiqiang 1MHz = 100Hzz Signed-off-by: Hou Zhiqiang --- V3: - New patch. drivers/cpu/imx8_cpu.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/cpu/imx8_cpu.c b/drivers/cpu/imx8_cpu.c index 4781a565547..601a7071a64 100644 --- a/drivers/cpu/imx

[PATCHv3 03/12] test: cpu: add test for release CPU core.

2024-07-26 Thread Zhiqiang Hou
From: Hou Zhiqiang Add test for API cpu_release_core(). Signed-off-by: Hou Zhiqiang --- V3: - No change. test/dm/cpu.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/test/dm/cpu.c b/test/dm/cpu.c index acba8105996..8af25316cea 100644 --- a/test/dm/cpu.c +++ b/test/dm/cpu.c @@ -43,6 +4

[PATCHv3 02/12] cpu: sandbox: implement release_core callback

2024-07-26 Thread Zhiqiang Hou
From: Hou Zhiqiang Add empty release CPU core function for testing. Signed-off-by: Hou Zhiqiang --- V3: - No change. drivers/cpu/cpu_sandbox.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/drivers/cpu/cpu_sandbox.c b/drivers/cpu/cpu_sandbox.c index e65e1bdc51b..b1527957831 100644

[PATCHv3 01/12] cpu: add release_core callback

2024-07-26 Thread Zhiqiang Hou
From: Hou Zhiqiang Add a new callback release_core to the cpu_ops, which is used to release a CPU core to run baremetal or RTOS application on a SoC with multiple CPU cores. Signed-off-by: Hou Zhiqiang --- V3: - No change. drivers/cpu/cpu-uclass.c | 10 ++ include/cpu.h|

[PATCHv3 0/12] Add a subcommand 'release' to cmd/cpu.c

2024-07-26 Thread Zhiqiang Hou
From: Hou Zhiqiang This patch set is to add a subcommand 'release' to the 'cpu' command in cmd/cpu.c, making the command is able to release a core to run baremetal and RTOS applications. Fixed some problems of the imx8_cpu.c and added i.MX 8M series SoCs support. And enabled the 'cpu' command a

Re: [PATCH v2 00/40] mmc: dw_mmc: Enable eMMC on E850-96 board

2024-07-26 Thread Minkyu Kang
Hello, 2024년 7월 26일 (금) 05:58, Sam Protsenko 님이 작성: > On Thu, Jul 25, 2024 at 1:25 PM Michael Nazzareno Trimarchi > wrote: > > > > Hi all > > > > Il mar 9 lug 2024, 02:48 Sam Protsenko ha > scritto: > >> > >> On Thu, Jun 27, 2024 at 9:42 AM Tom Rini wrote: > >> > > >> > On Wed, Jun 26, 2024 a

[PATCH 3/3] doc: stm32mp1: fix/improve basic reStructuredText syntax

2024-07-26 Thread Grzegorz Szymaszek
Drop some unnecessary/unintended blockquotes, improve formatting consistency, simplify tables, add some inline literals, fix several other minor issues. Signed-off-by: Grzegorz Szymaszek --- doc/board/st/stm32mp1.rst | 859 +++--- 1 file changed, 421 insertions(+)

[PATCH 2/3] doc: stm32mp1: fix literal block markers (::)

2024-07-26 Thread Grzegorz Szymaszek
Signed-off-by: Grzegorz Szymaszek --- doc/board/st/stm32mp1.rst | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/doc/board/st/stm32mp1.rst b/doc/board/st/stm32mp1.rst index 239e18b5e17..98167e80618 100644 --- a/doc/board/st/stm32mp1.rst +++ b/doc/board/st/stm32mp1.rst @@ -2

[PATCH 1/3] doc: board: stm32mp1: add missing newline at EOF

2024-07-26 Thread Grzegorz Szymaszek
Signed-off-by: Grzegorz Szymaszek --- doc/board/st/stm32mp1.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doc/board/st/stm32mp1.rst b/doc/board/st/stm32mp1.rst index 63b44776ffc..239e18b5e17 100644 --- a/doc/board/st/stm32mp1.rst +++ b/doc/board/st/stm32mp1.rst @@ -838,4

RE: [PATCHv2] clk: imx8m: register ARM A53 core clock

2024-07-26 Thread Z.Q. Hou
Hi Marek, > -Original Message- > From: Marek Vasut > Sent: Friday, July 26, 2024 10:05 PM > To: Z.Q. Hou ; u-boot@lists.denx.de; > tr...@konsulko.com; lu...@denx.de; sean...@gmail.com; > feste...@gmail.com; Peng Fan > Subject: Re: [PATCHv2] clk: imx8m: register ARM A53 core clock > > On

Re: [PATCHv2] clk: imx8m: register ARM A53 core clock

2024-07-26 Thread Marek Vasut
On 7/26/24 12:29 PM, Zhiqiang Hou wrote: From: Hou Zhiqiang Register ARM A53 core clock for i.MX 8M Mini, Nano and Plus. Why is MX8M(Q) not included here ?

Re: [PATCH v4 3/4] alist: Add support for an allocated pointer list

2024-07-26 Thread Simon Glass
Hi Tom, On Fri, 26 Jul 2024 at 08:33, Tom Rini wrote: > > On Fri, Jul 26, 2024 at 08:00:37AM -0600, Simon Glass wrote: > > In various places it is useful to have an array of structures, but allow > > it to grow. In some cases we work around it by setting maximum number of > > entries, using a Kco

Please pull u-boot-dm

2024-07-26 Thread Simon Glass
Hi Tom, https://source.denx.de/u-boot/custodians/u-boot-dm/-/pipelines/21781 https://dev.azure.com/simon0972/u-boot/_build/results?buildId=65&view=results The following changes since commit f4e163ece454bcf949a9b2005de3b70392ca05d1: Merge branch 'qcom-main' of https://gitlab.denx.de/u-boot/cust

Re: [PATCH v4 1/4] binman: Add nxp_imx8mcst etype for i.MX8M flash.bin signing

2024-07-26 Thread Simon Glass
Hi Marek, On Tue, 9 Jul 2024 at 03:24, Simon Glass wrote: > > Hi Marek, > > On Sun, 7 Jul 2024 at 01:55, Marek Vasut wrote: > > > > On 6/27/24 10:19 AM, Simon Glass wrote: > > > Hi Marek, > > > > Hi, > > > > >> Add new binman etype which allows signing both the SPL and fitImage > > >> s

Re: [PATCH 5/6] efi: Use malloc() for the EFI pool

2024-07-26 Thread Simon Glass
Hi Ilias, On Fri, 26 Jul 2024 at 02:31, Ilias Apalodimas wrote: > > Hi Simon, > > > On Fri, 26 Jul 2024 at 02:33, Simon Glass wrote: > > > > Hi Heinrich, > > > > On Thu, 25 Jul 2024 at 09:54, Heinrich Schuchardt > > wrote: > > > > > > On 25.07.24 15:56, Simon Glass wrote: > > > > This API call

Re: [PATCH v4 3/4] alist: Add support for an allocated pointer list

2024-07-26 Thread Tom Rini
On Fri, Jul 26, 2024 at 08:00:37AM -0600, Simon Glass wrote: > In various places it is useful to have an array of structures, but allow > it to grow. In some cases we work around it by setting maximum number of > entries, using a Kconfig option. In other places we use a linked list, > which does no

Re: [PATCH v4 07/29] mbedtls: add digest shim layer for MbedTLS

2024-07-26 Thread Raymond Mao
Hi Ilias, On Fri, 26 Jul 2024 at 06:19, Ilias Apalodimas wrote: > Hi Raymond > > [...] > > > > > +if MBEDTLS_LIB_CRYPTO > > + > > +config SHA1_MBEDTLS > > + bool "Enable SHA1 support with MbedTLS crypto library" > > + depends on MBEDTLS_LIB_CRYPTO && SHA1 > > + help > > +

Re: [PATCH v4 23/29] mbedtls: add MSCode parser porting layer

2024-07-26 Thread Raymond Mao
Hi Ilias, On Fri, 26 Jul 2024 at 06:10, Ilias Apalodimas wrote: > Hi Raymond > > On Tue, 2 Jul 2024 at 21:33, Raymond Mao wrote: > > > > Add porting layer for MSCode on top of MbedTLS ASN1 library. > > Introduce _LEGACY and _MBEDTLS kconfigs for MSCode legacy and > > MbedTLS implementations res

Re: [PATCH v5 16/20] configs: add qemu_arm64_lwip_defconfig

2024-07-26 Thread Tom Rini
On Fri, Jul 26, 2024 at 02:27:05PM +0200, Jerome Forissier wrote: > > > On 7/25/24 18:25, Tom Rini wrote: > > On Thu, Jul 25, 2024 at 06:10:48PM +0200, Jerome Forissier wrote: > >> > >> > >> On 7/25/24 17:58, Tom Rini wrote: > >>> On Thu, Jul 25, 2024 at 02:57:37PM +0200, Jerome Forissier wrote:

Re: [PATCH v4 07/29] mbedtls: add digest shim layer for MbedTLS

2024-07-26 Thread Tom Rini
On Fri, Jul 26, 2024 at 01:18:57PM +0300, Ilias Apalodimas wrote: > Hi Raymond > > [...] > > > > > +if MBEDTLS_LIB_CRYPTO > > + > > +config SHA1_MBEDTLS > > + bool "Enable SHA1 support with MbedTLS crypto library" > > + depends on MBEDTLS_LIB_CRYPTO && SHA1 > > + help > > +

[PATCH v4 4/4] RFC: lib: Convert str_to_list() to use alist

2024-07-26 Thread Simon Glass
Use this new data structure in the utility function. Signed-off-by: Simon Glass --- (no changes since v3) Changes in v3: - Update for alist_add() API change lib/strto.c | 35 +++ 1 file changed, 19 insertions(+), 16 deletions(-) diff --git a/lib/strto.c b/lib/

[PATCH v4 3/4] alist: Add support for an allocated pointer list

2024-07-26 Thread Simon Glass
In various places it is useful to have an array of structures, but allow it to grow. In some cases we work around it by setting maximum number of entries, using a Kconfig option. In other places we use a linked list, which does not provide for random access and can complicate the code. Introduce a

[PATCH v4 2/4] lib: Handle a special case with str_to_list()

2024-07-26 Thread Simon Glass
The current implementation can return an extra result at the end when the string ends with a space. Fix this by adding a special case. Signed-off-by: Simon Glass --- (no changes since v1) lib/strto.c | 4 +++- test/str_ut.c | 4 +--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --g

[PATCH v4 1/4] malloc: Support testing with realloc()

2024-07-26 Thread Simon Glass
At present in tests it is possible to cause an out-of-memory condition with malloc() but not realloc(). Add support to realloc() too, so code which uses that function can be tested. Signed-off-by: Simon Glass --- (no changes since v1) common/dlmalloc.c | 4 1 file changed, 4 insertions(+)

[PATCH v4 0/4] alist: Implement a pointer list / array of structs

2024-07-26 Thread Simon Glass
This data structure provides a list of pointers / array of structures. I was planning to use it for the lmb restructure, to allow it to support any number of entries, but then I gave up on it. There are quite a few places in U-Boot where such a list would be useful, since it supports growing the a

cmd: bind/unbind by class index doesn't work with usb gadget

2024-07-26 Thread Zixun LI
Bind/unbind a gadget function driver to a USB gadget device doesn't work with class index, while using node path is ok. For example: U-Boot> dm tree Class Index Probed DriverName --- root 0 [ + ] root_driv

Re: [PULL] Please pull qcom/qcom-main

2024-07-26 Thread Tom Rini
On Fri, Jul 26, 2024 at 01:55:06AM +0200, Caleb Connolly wrote: > Hi Tom, > > Just a few things: > > * Qualcomm platforms >~2016 gain support for the RPMh (Resource Power Manager) > peripheral which is used to control most regulators. The RB5 is now able to > power up its USB VBUS regulator

Re: [PATCH V2] clk: Propagate clk_set_rate() if CLK_SET_PARENT_RATE present for gate and mux

2024-07-26 Thread Michael Nazzareno Trimarchi
Hi Sam On Thu, Jul 25, 2024 at 4:41 AM Sam Protsenko wrote: > > On Wed, Jul 24, 2024 at 4:44 AM Michael Nazzareno Trimarchi > wrote: > > [snip] > > > > Ok , so you suggest to have something like in the core > > > > diff --git a/drivers/clk/clk-uclass.c b/drivers/clk/clk-uclass.c > > index 8faf5a

Re: [PATCH v2] tools: patman: fix `pip install` with Python 3.12

2024-07-26 Thread Simon Glass
On Mon, 1 Jul 2024 at 18:51, Brandon Maier wrote: > > Installing patman with `cd ./tools/patman && pip install -e .` fails > with the error below. > > As described in the error output below, the license line is not allowed > to be only defined in the setup.py. We remove the 'license' field > entir

Re: [PATCH v2] binman: Update cbfstool

2024-07-26 Thread Simon Glass
On Tue, Jul 02, 2024 at 05:37:28PM +0100, Simon Glass wrote: > Update to a newer version of this tool, 4.22.01. This runs OK with the > current binman tests and matches the one in CI. > > Signed-off-by: Simon Glass Reviewed-by: Tom Rini -- Tom Applied to u-boot-dm, thanks!

Re: [PATCH v2] sandbox: Fix LTO to work with STACKPROTECTOR

2024-07-26 Thread Simon Glass
On Tue, 2 Jul 2024 at 18:00, Andrew Goodbody wrote: > > Add the STACKPROTECTOR symbols to the script that generates the > symbols that should not be removed by the use of LTO when linking > a shared object. This prevents a fail to build due to link errors. > > https://source.denx.de/u-boot/u-boot/

Re: [PATCH] buildman: Show board list with -x

2024-07-26 Thread Simon Glass
On Thu, Jul 11, 2024 at 09:10:04AM +0100, Simon Glass wrote: > When -x is used, buildman does not show the list of boards that will be > built, since there are no terms which cause boards to be added, only > terms which cause them to be removed. > > Add a special case to fix this. > > Signed-off-b

Re: [PATCH 01/24] qconfig: Fix pylint error in read_database()

2024-07-26 Thread Simon Glass
Fix this error by initing the variable before the loop: tools/qconfig.py:880:22: E0606: Possibly using variable 'defconfig' before assignment (possibly-used-before-assignment) Signed-off-by: Simon Glass --- tools/qconfig.py | 1 + 1 file changed, 1 insertion(+) Applied to u-boot-dm, thanks

Re: [PATCH 02/24] qconfig: Drop the try_expand() function

2024-07-26 Thread Simon Glass
This is not used anymore, so drop it. Signed-off-by: Simon Glass --- tools/qconfig.py | 21 - 1 file changed, 21 deletions(-) Applied to u-boot-dm, thanks!

Re: [PATCH 03/24] qconfig: Make KconfigScanner a function

2024-07-26 Thread Simon Glass
This doesn't have any methods so is not good as a class. Make it a function instead, to keep pylint happy. Signed-off-by: Simon Glass --- tools/qconfig.py | 25 + 1 file changed, 13 insertions(+), 12 deletions(-) Applied to u-boot-dm, thanks!

Re: [PATCH 04/24] qconfig: Tidy up some pylint warnings

2024-07-26 Thread Simon Glass
Reduce the number of warnings in this file a little bit. Add my own name to the copyright message. Signed-off-by: Simon Glass --- tools/qconfig.py | 33 ++--- 1 file changed, 22 insertions(+), 11 deletions(-) Applied to u-boot-dm, thanks!

Re: [PATCH 06/24] qconfig: Rename the doc link

2024-07-26 Thread Simon Glass
This was missed during the renaming of the tool. Fix it. Signed-off-by: Simon Glass Fixes: ea4d6dead37 ("moveconfig: Rename the tool to qconfig") --- tools/qconfig.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) Applied to u-boot-dm, thanks!

Re: [PATCH 05/24] qconfig: Correct format string in do_imply_config()

2024-07-26 Thread Simon Glass
One of the strings was converted incorrectly. Fix it. Signed-off-by: Simon Glass Fixes: 1bd43060b3e ("moveconfig: Use f strings where possible") --- tools/qconfig.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) Applied to u-boot-dm, thanks!

Re: [PATCH 07/24] qconfig: Move arg parsing into a separate function

2024-07-26 Thread Simon Glass
Reduce the size of main() by putting this code into its own function. For now the parser object needs to be returned too. Signed-off-by: Simon Glass --- tools/qconfig.py | 16 +--- 1 file changed, 13 insertions(+), 3 deletions(-) Applied to u-boot-dm, thanks!

Re: [PATCH 08/24] qconfig: Move arg checking a little higher

2024-07-26 Thread Simon Glass
Check for scan_source as one of the possible operations for this tool, moving the check above the scan_source implementation. Signed-off-by: Simon Glass --- tools/qconfig.py | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) Applied to u-boot-dm, thanks!

Re: [PATCH 09/24] qconfig: Move arg checking to the top of main()

2024-07-26 Thread Simon Glass
Check for 'test' as one of the possible operations for this tool, moving the check above the implementation. Signed-off-by: Simon Glass --- tools/qconfig.py | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) Applied to u-boot-dm, thanks!

Re: [PATCH 10/24] qconfig: Move getting the colour to where it is needed

2024-07-26 Thread Simon Glass
Move this assignment down to just above where it is needed. Signed-off-by: Simon Glass --- tools/qconfig.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) Applied to u-boot-dm, thanks!

Re: [PATCH 11/24] qconfig: Move checking directory to the top

2024-07-26 Thread Simon Glass
Move this check to the top, so it happens always. The tool should be run from the U-Boot source directory. Signed-off-by: Simon Glass --- tools/qconfig.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) Applied to u-boot-dm, thanks!

Re: [PATCH 12/24] qconfig: Move converting config args to the top

2024-07-26 Thread Simon Glass
Move this check to the top, so it happens always. There is no harm to doing this earlier and it separates the setup from actual program logic. Update the arg rather than adding a new variable, with the new variable only created when moving or building, since it is used more heavily. Signed-off-by

Re: [PATCH 13/24] qconfig: Move testing into a separate function

2024-07-26 Thread Simon Glass
Reduce the size of main() by putting this code into its own function. Signed-off-by: Simon Glass --- tools/qconfig.py | 16 +++- 1 file changed, 11 insertions(+), 5 deletions(-) Applied to u-boot-dm, thanks!

Re: [PATCH 14/24] qconfig: Add a return value to do_scan_source()

2024-07-26 Thread Simon Glass
Return an exit code so we can use this function like do_tests(). Refactor the caller to handle this. Reduce the size of main() by putting this code into its own function. Signed-off-by: Simon Glass --- tools/qconfig.py | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) Applied to u-bo

Re: [PATCH 15/24] qconfig: Move imply into a separate function

2024-07-26 Thread Simon Glass
Reduce the size of main() by putting this code into its own function, with the usage message staying in main(). Tidy up the comments for do_imply_config() while we are here. Signed-off-by: Simon Glass --- tools/qconfig.py | 66 1 file changed, 3

Re: [PATCH 19/24] qconfig: Move progress output into the class

2024-07-26 Thread Simon Glass
Rather than create these outputs separately, put them in the class so that the main program doesn't need to deal with them. Signed-off-by: Simon Glass --- tools/qconfig.py | 28 +++- 1 file changed, 19 insertions(+), 9 deletions(-) Applied to u-boot-dm, thanks!

Re: [PATCH 16/24] qconfig: Add a return value to do_find_config()

2024-07-26 Thread Simon Glass
Return an exit code so we can use this function like do_tests(). Refactor the caller to handle this. Reduce the size of main() by putting this code into its own function. Signed-off-by: Simon Glass --- tools/qconfig.py | 17 ++--- 1 file changed, 10 insertions(+), 7 deletions(-) A

Re: [PATCH 17/24] qconfig: Move all move_config code into move_config()

2024-07-26 Thread Simon Glass
Move the setup and completion code into the move_config() function so it is all in one place. Signed-off-by: Simon Glass --- tools/qconfig.py | 45 + 1 file changed, 25 insertions(+), 20 deletions(-) Applied to u-boot-dm, thanks!

Re: [PATCH 18/24] qconfig: Move commit code into a separate function

2024-07-26 Thread Simon Glass
Reduce the size of main() by putting this code into its own function. Signed-off-by: Simon Glass --- tools/qconfig.py | 31 --- 1 file changed, 20 insertions(+), 11 deletions(-) Applied to u-boot-dm, thanks!

Re: [PATCH 20/24] qconfig: Move the last two operations into their own functions

2024-07-26 Thread Simon Glass
Put the summary and database-writing code into separate functions to reduce the size of main(). Signed-off-by: Simon Glass --- tools/qconfig.py | 66 ++-- 1 file changed, 47 insertions(+), 19 deletions(-) Applied to u-boot-dm, thanks!

Re: [PATCH 21/24] qconfig: Use the Color object in Progress

2024-07-26 Thread Simon Glass
Since the Progress class has the required object, use it from there instead of passing it around. Signed-off-by: Simon Glass --- tools/qconfig.py | 21 ++--- 1 file changed, 10 insertions(+), 11 deletions(-) Applied to u-boot-dm, thanks!

Re: [PATCH 22/24] qconfig: Drop col argument from Slots()

2024-07-26 Thread Simon Glass
This is not needed since the progress indicator has the object. Use that instead. Signed-off-by: Simon Glass --- tools/qconfig.py | 15 ++- 1 file changed, 6 insertions(+), 9 deletions(-) Applied to u-boot-dm, thanks!

Re: [PATCH 23/24] qconfig: Move operation check into parse_args()

2024-07-26 Thread Simon Glass
Put the check for an operation being provided into the parse_args() function, to reduce the size of main(). Signed-off-by: Simon Glass --- tools/qconfig.py | 13 +++-- 1 file changed, 7 insertions(+), 6 deletions(-) Applied to u-boot-dm, thanks!

Re: [PATCH v5 02/20] net: introduce alternative implementation as net-lwip/

2024-07-26 Thread Jerome Forissier
On 7/26/24 10:44, Michal Simek wrote: > čt 25. 7. 2024 v 15:28 odesílatel Jerome Forissier > napsal: >> >> Prepare the introduction of the lwIP (lightweight IP) TCP/IP stack by >> adding a new net/lwip/ directory and the NET_LWIP symbol. Network >> support is either NO_NET, NET (legacy stack) o

Re: [PATCH v5 02/20] net: introduce alternative implementation as net-lwip/

2024-07-26 Thread Jerome Forissier
On 7/25/24 18:17, Tom Rini wrote: > On Thu, Jul 25, 2024 at 02:57:23PM +0200, Jerome Forissier wrote: > >> Prepare the introduction of the lwIP (lightweight IP) TCP/IP stack by >> adding a new net/lwip/ directory and the NET_LWIP symbol. Network >> support is either NO_NET, NET (legacy stack) o

[PATCH 1/1] riscv: define find_{first,next}_zero_bit in asm/bitops.h

2024-07-26 Thread Maxim Kochetkov
These seem to be missing, and trying to build fastboot cmd without them is causing errors due to these being missing. Signed-off-by: Maxim Kochetkov --- arch/riscv/include/asm/bitops.h | 40 + 1 file changed, 40 insertions(+) diff --git a/arch/riscv/include/asm/b

[PATCH 1/1] clk: fix clk_enable() parent enable

2024-07-26 Thread Maxim Kochetkov
Move clkp->enable_count++ after clk_enable(clkp->dev->parent). It allows the parent clock increment its enable counters for proper disable sequence. Signed-off-by: Maxim Kochetkov --- drivers/clk/clk-uclass.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/drivers/cl

[PATCH 1/1] serial: ns16550: Add clock ctrl to driver

2024-07-26 Thread Maxim Kochetkov
Add code to get all clock inputs as in serial DT node. A clock property is an optional feature, so do not fail if a clock property is not present. If a clock property is discovered, then use it to get clock. It marks clock as used and prevents disabling shared clocks. Signed-off-by: Maxim Kochetk

Re: [PATCH v5 02/20] net: introduce alternative implementation as net-lwip/

2024-07-26 Thread Michal Simek
čt 25. 7. 2024 v 15:28 odesílatel Jerome Forissier napsal: > > Prepare the introduction of the lwIP (lightweight IP) TCP/IP stack by > adding a new net/lwip/ directory and the NET_LWIP symbol. Network > support is either NO_NET, NET (legacy stack) or NET_LWIP. Subsequent > commits will introduce t

[PATCH] usb: gadget: ether: Handle gadget driver registration in start and stop

2024-07-26 Thread Zixun LI
Revert part of 718f1d41 to move usb_gadget_register_driver()/usb_gadget_unregister_driver() back to usb_eth_start()/usb_eth_stop(). usb_gadget_register_driver() will initialize the USB controller which enters ready to connect state with pull-up resistor enabled. >From the host's point of view, a

[PATCH] usb: bootm: Drop old USB-device-removal code

2024-07-26 Thread Simon Glass
USB is stopped using driver model now, in dm_remove_devices_flags() in announce_and_cleanup() at the top of this file. The usb_stop() call actually unbinds devices. When a USB device is unbound, it causes any bootflows attached to it to be removed, via a call to bootdev_clear_bootflows() from boo

Re: [PATCH v5 16/20] configs: add qemu_arm64_lwip_defconfig

2024-07-26 Thread Jerome Forissier
On 7/25/24 18:25, Tom Rini wrote: > On Thu, Jul 25, 2024 at 06:10:48PM +0200, Jerome Forissier wrote: >> >> >> On 7/25/24 17:58, Tom Rini wrote: >>> On Thu, Jul 25, 2024 at 02:57:37PM +0200, Jerome Forissier wrote: >>> Add qemu_arm64_lwip_defconfig which #include's qemu_arm64_defconfig and

Re: [PATCH v4 07/29] mbedtls: add digest shim layer for MbedTLS

2024-07-26 Thread Ilias Apalodimas
Hi Raymond [...] > > +if MBEDTLS_LIB_CRYPTO > + > +config SHA1_MBEDTLS > + bool "Enable SHA1 support with MbedTLS crypto library" > + depends on MBEDTLS_LIB_CRYPTO && SHA1 > + help > + This option enables support of hashing using SHA1 algorithm > + with MbedTLS c

Re: [PATCH v4 23/29] mbedtls: add MSCode parser porting layer

2024-07-26 Thread Ilias Apalodimas
Hi Raymond On Tue, 2 Jul 2024 at 21:33, Raymond Mao wrote: > > Add porting layer for MSCode on top of MbedTLS ASN1 library. > Introduce _LEGACY and _MBEDTLS kconfigs for MSCode legacy and > MbedTLS implementations respectively. You should mention explicitly on the commit message, that this patch

[PATCHv2] clk: imx8m: register ARM A53 core clock

2024-07-26 Thread Zhiqiang Hou
From: Hou Zhiqiang Register ARM A53 core clock for i.MX 8M Mini, Nano and Plus. Signed-off-by: Hou Zhiqiang --- V2: - Fixed the change log: s/A55/A53 drivers/clk/imx/clk-imx8mm.c | 6 ++ drivers/clk/imx/clk-imx8mn.c | 7 +++ drivers/clk/imx/clk-imx8mp.c | 7 +++ 3 files changed,

Re: [PATCH v4 0/7] usb: gadget: atmel: Code refactor and DM_USB_GADGET support

2024-07-26 Thread Mattijs Korpershoek
Hi, On jeu., juil. 25, 2024 at 20:57, Marek Vasut wrote: > On 7/25/24 8:48 PM, Zixun LI wrote: >> On Thu, Jul 25, 2024 at 8:24 PM Marek Vasut wrote: >>> >>> On 7/25/24 5:31 PM, Zixun LI wrote: Changes in v4: - Release clocks if probe failed - Add missing endpoint data free -

Re: [PATCH v1 0/4] android_ab: fix slot_suffix issue and introduce ab_dump command

2024-07-26 Thread Dmitry Rokosov
Azure pipeline is successfully DONE https://github.com/u-boot/u-boot/pull/625/checks On Thu, Jul 25, 2024 at 10:47:00PM +0300, Dmitry Rokosov wrote: > The patch series include changes: > - fix indentation problems for --no-dec parameter in the ab_select > command > - introduce the a

Re: [PATCH v5 01/20] flash: prefix error codes with FL_

2024-07-26 Thread Jerome Forissier
On 7/25/24 20:18, Tom Rini wrote: > On Thu, Jul 25, 2024 at 02:57:22PM +0200, Jerome Forissier wrote: > >> Prefix the flash status codes (ERR_*) with FL_ in order to avoid clashes >> with third-party libraries. Case in point: including the lwIP library >> header file which defines err_enum_t a

Re: [PATCH v3 1/8] usb: cdns3: Set USB PHY mode in cdns3_drd_update_mode()

2024-07-26 Thread Minda Chen
> > USB PHY maybe need to set PHY mode in different USB dr mode. So translate > USB PHY mode to generic PHY mode and call generic_phy_set_mode(). > > Signed-off-by: Minda Chen > --- > drivers/usb/cdns3/drd.c | 14 ++ > 1 file changed, 14 insertions(+) > > diff --git a/drivers/us

Re: [PATCH v5 06/20] net: eth-uclass: add function eth_start_udev()

2024-07-26 Thread Jerome Forissier
On 7/25/24 20:27, Tom Rini wrote: > On Thu, Jul 25, 2024 at 02:57:27PM +0200, Jerome Forissier wrote: > >> Add a function to start a given etwork device, and update eth_init() > > Typo, "network". Fixed in v6. >> to use it. >> >> Signed-off-by: Jerome Forissier > > And this is where most

Re: [PATCH v3 5/8] pinctrl: starfive: Setup USB default disable overcurrent pin

2024-07-26 Thread Minda Chen
> > For some JH7110 boards, USB host overcurent pin is not reserved, To make USB > host work, overcurrent pin must be disabled. So set the pin default disabled. > > Signed-off-by: Minda Chen > --- > drivers/pinctrl/starfive/pinctrl-jh7110-sys.c | 11 +-- > 1 file changed, 9 insertion

Re: [PATCH 5/6] efi: Use malloc() for the EFI pool

2024-07-26 Thread Ilias Apalodimas
Hi Simon, On Fri, 26 Jul 2024 at 02:33, Simon Glass wrote: > > Hi Heinrich, > > On Thu, 25 Jul 2024 at 09:54, Heinrich Schuchardt wrote: > > > > On 25.07.24 15:56, Simon Glass wrote: > > > This API call is intended for allocating small amounts of memory, > > > similar to malloc(). The current i

Re: [PATCH] arm64: zynqmp: dts: Add rts delay property for rs485 mode on KD240

2024-07-26 Thread Michal Simek
On 7/18/24 12:15, Michal Simek wrote: From: Manikanta Guntupalli Add "rs485-rts-delay" property to uartps node with delay_rts_before_send and delay_rts_after_send values as 10ms for rs485 mode on KD240. 10ms rts delay values have been chosen based on testing with rs485 temperature sensor (w

Re: [PATCH] arm64: xilinx: Describe TPM reset for Kria CCs

2024-07-26 Thread Michal Simek
On 7/16/24 15:29, Michal Simek wrote: Describe carrier card TPM reset behavior and show message about it on boot console to let users know what to expect from it. Signed-off-by: Michal Simek --- board/xilinx/zynqmp/zynqmp_kria.env | 9 ++--- 1 file changed, 6 insertions(+), 3 deletio

Re: [PATCH] arm64: versal: Remove undocumented cadence,qspi compatible

2024-07-26 Thread Michal Simek
On 7/15/24 16:39, Michal Simek wrote: Compatible string is not the part of dt-schema and also not used by U-Boot or Linux that's why remove it completely. Signed-off-by: Michal Simek --- arch/arm/dts/versal-mini-ospi.dtsi | 2 +- arch/arm/dts/versal-net-mini-ospi.dtsi | 2 +- 2 file

Re: [PATCH] arm64: versal-net: Align node names with dt-schema

2024-07-26 Thread Michal Simek
On 7/15/24 16:38, Michal Simek wrote: dt-schema is forcing some rules for node names that's why align them with it. Labels are not changing that's why this change is not breaking any other board specific DTSes. Signed-off-by: Michal Simek --- arch/arm/dts/versal-net-mini-emmc.dts | 4 ++-

Re: [PATCH] arm64: zynqmp: Add resets property for UART nodes

2024-07-26 Thread Michal Simek
On 7/15/24 16:23, Michal Simek wrote: From: Manikanta Guntupalli Add resets property for UART0 and UART1 nodes Signed-off-by: Manikanta Guntupalli Signed-off-by: Michal Simek --- arch/arm/dts/zynqmp.dtsi | 2 ++ 1 file changed, 2 insertions(+) diff --git a/arch/arm/dts/zynqmp.dtsi b/