Re: [PATCH 08/12] net/httpd: add httpd common code

2024-07-02 Thread Mikhail Kshevetskiy
On 7/1/24 19:54, Peter Robinson wrote: > Hi Mikhail, > >> This patch adds HTTP/1.1 compatible web-server that can be used >> by other. Server supports GET, POST, and HEAD requests. On client >> request it will call user specified GET/POST callback. Then results >> will be transmitted to client. >

Re: [PATCH] gpt: allow spaces in partition list

2024-07-02 Thread Mikhail Kshevetskiy
On 7/2/24 19:51, Simon Glass wrote: > Hi Mikhail, > > On Tue, 2 Jul 2024 at 10:42, Mikhail Kshevetskiy > wrote: >> >> On 27.06.2024 22:05, Simon Glass wrote: >>> Hi Mikhail, >>> >>> On Thu, 27 Jun 2024 at 12:29, Mikhail Kshevetskiy >>> wrote: This allows spliting partition list to several

Re: [PATCH 1/2] spi: soft_spi: fix miso gpio property name

2024-07-02 Thread Fabio Estevam
Hi Mikhail, On Tue, Jul 2, 2024 at 10:16 PM Mikhail Kshevetskiy wrote: > > The patch fix a missprint introduced in commit 2e9fe73a883a ("spi: soft_spi: > Support the recommended soft spi properties"). > > Signed-off-by: Mikhail Kshevetskiy Thanks for the fix: Reviewed-by: Fabio Estevam

Re: [PATCH v4 00/29] Integrate MbedTLS v3.6 LTS with U-Boot

2024-07-02 Thread Tom Rini
On Tue, Jul 02, 2024 at 11:22:36AM -0700, Raymond Mao wrote: > Integrate MbedTLS v3.6 LTS (currently v3.6.0-RC1) with U-Boot. > > Motivations: > > > 1. MbedTLS is well maintained with LTS versions. > 2. LWIP is integrated with MbedTLS and easily to enable HTTPS. > 3. MbedTLS recentl

[PATCH 1/2] spi: soft_spi: fix miso gpio property name

2024-07-02 Thread Mikhail Kshevetskiy
The patch fix a missprint introduced in commit 2e9fe73a883a ("spi: soft_spi: Support the recommended soft spi properties"). Signed-off-by: Mikhail Kshevetskiy --- drivers/spi/soft_spi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/spi/soft_spi.c b/drivers/spi/soft_

[PATCH 2/2] spi: soft_spi: Parse cs-gpios only if num-chipselects is not <0>

2024-07-02 Thread Mikhail Kshevetskiy
From: Michael Polyntsov Some boards don't have chipselect lines for leds so cs-gpios is not specified in the dts leading to probing error. Fix it by making behavior similar to the one in Linux, parse num-chipselects and if it is zero, ignore cs-gpios. Signed-off-by: Michael Polyntsov Signed-off

[PATCH 2/2] led: Add dts property to specify blinking of the led

2024-07-02 Thread Mikhail Kshevetskiy
From: Michael Polyntsov The standard property linux,default-trigger = "pattern"; used to get an effect. No blinking parameters can be set yet. Signed-off-by: Michael Polyntsov Signed-off-by: Mikhail Kshevetskiy --- drivers/led/led-uclass.c | 34 ++ 1 file

[PATCH 1/2] led: Implement software led blinking

2024-07-02 Thread Mikhail Kshevetskiy
From: Michael Polyntsov If hardware (or driver) doesn't support leds blinking, it's now possible to use software implementation of blinking instead. This relies on cyclic functions. v2 changes: * Drop sw_blink_state structure, move its necessary fields to led_uc_plat structure. * Add cyclic

led blinking patches

2024-07-02 Thread Mikhail Kshevetskiy
Hi Simon and all, This patch series implements: * software led blinking (via cyclic functions) * add support of dts property to specify blinking of the led v2 changes: * Drop sw_blink_state structure, move its necessary fields to led_uc_plat structure. * Add cyclic_info pointer to led_uc_p

RE: [PATCH] mx9: Correct repeatable build error

2024-07-02 Thread Peng Fan
> Subject: [PATCH] mx9: Correct repeatable build error > > For some reason every second time imx93_11x11_evk is built it gives > an > error: > >make O=/tmp/x BINMAN_ALLOW_MISSING=1 > > It seems to sometimes skip generation of the .cfgout file and then > eventually Binman complains: > >V

Re: [PATCH v4 04/29] lib: Adapt digest header files to MbedTLS

2024-07-02 Thread Tom Rini
On Tue, Jul 02, 2024 at 08:02:37PM -0400, Raymond Mao wrote: > Hi Tom, > > On Tue, 2 Jul 2024 at 18:48, Tom Rini wrote: > > > On Tue, Jul 02, 2024 at 11:22:40AM -0700, Raymond Mao wrote: > > > > > Adapt digest header files to support both original libs and MbedTLS > > > by switching on/off MBEDT

Re: [PATCH v4 04/29] lib: Adapt digest header files to MbedTLS

2024-07-02 Thread Raymond Mao
Hi Tom, On Tue, 2 Jul 2024 at 18:48, Tom Rini wrote: > On Tue, Jul 02, 2024 at 11:22:40AM -0700, Raymond Mao wrote: > > > Adapt digest header files to support both original libs and MbedTLS > > by switching on/off MBEDTLS_LIB_CRYPTO. > > Introduce _LEGACY kconfig for legacy hash implementations.

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

2024-07-02 Thread Tom Rini
On Thu, Jun 27, 2024 at 09:37:18AM +0100, Simon Glass wrote: > Hi Tom, > > On Wed, 26 Jun 2024 at 15:29, Tom Rini wrote: > > > > On Wed, Jun 26, 2024 at 09:00:33AM +0100, Simon Glass wrote: > > > Hi Tom, > > > > > > On Tue, 25 Jun 2024 at 15:27, Tom Rini wrote: > > > > > > > > On Tue, Jun 25, 20

Re: [PATCH v4 04/29] lib: Adapt digest header files to MbedTLS

2024-07-02 Thread Tom Rini
On Tue, Jul 02, 2024 at 11:22:40AM -0700, Raymond Mao wrote: > Adapt digest header files to support both original libs and MbedTLS > by switching on/off MBEDTLS_LIB_CRYPTO. > Introduce _LEGACY kconfig for legacy hash implementations. [snip] > diff --git a/lib/mbedtls/Kconfig b/lib/mbedtls/Kconfig

Re: [PATCH v4 02/29] mbedtls: Add script to update MbedTLS subtree

2024-07-02 Thread Tom Rini
On Tue, Jul 02, 2024 at 11:22:38AM -0700, Raymond Mao wrote: > lib/mbedtls/update-mbedtls-subtree.sh is a wrapper of git subtree > commands. > Usage from U-Boot top directory, run: > > $ ./lib/mbedtls/update-mbedtls-subtree.sh pull > $ ./lib/mbedtls/update-mbedtls-subtree.sh pick > > Signed-of

[PATCH v1] mmc: am654_sdhci: Add the quirk to set TESTCD bit

2024-07-02 Thread Emanuele Ghidoli
From: Emanuele Ghidoli The ARASAN MMC controller on Keystone 3 class of devices need the SDCD line to be connected for proper functioning. In cases where this can't be connected, add a quirk to force the controller into test mode and set the TESTCD bit. Use the flag "ti,fails-without-test-cd", t

[PATCH 2/2] ext4: Fix zalloc()

2024-07-02 Thread Richard Weinberger
The zalloc() function suffers from two problems. 1. If memalign() fails it will return NULL and memset() will use a NULL pointer. 2. memalign() itself seems to crash when more than 2^32 bytes are requested. So, check the return value of memalign() and allocate only of size is less than CONFIG_SYS_

[PATCH 1/2] ext4: Fix integer overflow in ext4fs_read_symlink()

2024-07-02 Thread Richard Weinberger
While zalloc() takes a size_t type, adding 1 to the le32 variable will overflow. A carefully crafted ext4 filesystem can exhibit an inode size of 0x and as consequence zalloc() will do a zero allocation. Later in the function the inode size is again used for copying data. So an attacker ca

[PATCH v2] ext4: Improve feature checking

2024-07-02 Thread Richard Weinberger
Evaluate the filesystem incompat and ro_compat bit fields to judge whether the filesystem can be read or written. For the read side only a scary warning is shown so far. I'd love to abort mounting too, but I fear this will break some setups where the driver works by chance. Signed-off-by: Richard

Re: [RESEND PATCH v4 0/3] provide names for emmc hardware partitions

2024-07-02 Thread Tom Rini
On Tue, Jul 02, 2024 at 11:14:15AM -0700, Tim Harvey wrote: > On Fri, May 31, 2024 at 8:36 AM Tim Harvey wrote: > > > > Modern eMMC v4+ devices have multiple hardware partitions per the JEDEC > > specification described as: > > Boot Area Partition 1 > > Boot Area Partition 2 > > RPMB Partition

Re: Please pull u-boot-dm/next

2024-07-02 Thread Tom Rini
On Tue, Jul 02, 2024 at 04:51:30PM +0100, Simon Glass wrote: > Hi Tom, > > This is for -next but we need to figure out the patch I just sent > first. I can redo the pull if that patch is acceptable, or you can > pick that up first. I thought you meant the i.MX9 patch, but with that applied befor

Re: [RESEND PATCH v4 0/3] provide names for emmc hardware partitions

2024-07-02 Thread Tim Harvey
On Tue, Jul 2, 2024 at 11:25 AM Dragan Simic wrote: > > Hello Tim, > > On 2024-07-02 20:14, Tim Harvey wrote: > > On Fri, May 31, 2024 at 8:36 AM Tim Harvey > > wrote: > >> > >> Modern eMMC v4+ devices have multiple hardware partitions per the > >> JEDEC > >> specification described as: > >> Boo

[PATCH v4 29/29] configs: enable MbedTLS as default setting

2024-07-02 Thread Raymond Mao
Enable MbedTLS as default setting for qemu arm64 Signed-off-by: Raymond Mao --- Changes in v2 - None. Changes in v3 - None. Changes in v4 - removed the unused CONFIG_MBEDTLS_LIB_TLS. configs/qemu_arm64_defconfig | 4 configs/sandbox_defconfig| 3 +++ 2 files changed, 7 insertions(+) d

[PATCH v4 28/29] test: Remove ASN1 library test

2024-07-02 Thread Raymond Mao
With MBEDTLS_LIB_X509 enabled, we don't build the original ASN1 lib, So remove it from test. Signed-off-by: Raymond Mao Reviewed-by: Ilias Apalodimas --- Changes in v2 - Initial patch. Changes in v3 - None. Changes in v4 - None. test/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-

[PATCH v4 27/29] asn1_decoder: add build options for ASN1 decoder

2024-07-02 Thread Raymond Mao
When building with MbedTLS, we are using MbedTLS to decode ASN1 data for x509, pkcs7 and mscode. Introduce _LEGACY and _MBEDTLS kconfigs for ASN1 decoder legacy and MbedTLS implementations respectively. Signed-off-by: Raymond Mao --- Changes in v2 - Initial patch. Changes in v3 - None. Changes in

[PATCH v4 26/29] lib/rypto: Adapt rsa_helper to MbedTLS

2024-07-02 Thread Raymond Mao
Previous patch has introduced MbedTLS porting layer for RSA helper, here to adjust the makefile accordingly. Signed-off-by: Raymond Mao --- Changes in v2 - Initial patch. Changes in v3 - Update commit message. Changes in v4 - Control building legacy library via '_LEGACY' Kconfig. lib/crypto/Mak

[PATCH v4 25/29] mbedtls: add RSA helper layer on MbedTLS

2024-07-02 Thread Raymond Mao
Add RSA helper layer on top on MbedTLS PK and RSA library. Introduce _LEGACY and _MBEDTLS kconfigs for RSA helper legacy and MbedTLS implementations respectively. Signed-off-by: Raymond Mao --- Changes in v2 - Initial patch. Changes in v3 - None. Changes in v4 - Introduce _LEGACY and _MBEDTLS kco

[PATCH v4 24/29] lib/crypto: Adapt mscode_parser to MbedTLS

2024-07-02 Thread Raymond Mao
Previous patch has introduced MbedTLS porting layer for mscode parser, here to adjust the header and makefiles accordingly. Signed-off-by: Raymond Mao --- Changes in v2 - Move the porting layer to MbedTLS dir. Changes in v3 - Update commit message. Changes in v4 - Control building legacy library

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

2024-07-02 Thread Raymond Mao
Add porting layer for MSCode on top of MbedTLS ASN1 library. Introduce _LEGACY and _MBEDTLS kconfigs for MSCode legacy and MbedTLS implementations respectively. Signed-off-by: Raymond Mao --- Changes in v2 - Move the porting layer to MbedTLS dir. Changes in v3 - None. Changes in v4 - Introduce _L

[PATCH v4 22/29] lib/crypto: Adapt PKCS7 parser to MbedTLS

2024-07-02 Thread Raymond Mao
Previous patch has introduced MbedTLS porting layer for PKCS7 parser, here to adjust the header and makefiles accordingly. Signed-off-by: Raymond Mao --- Changes in v2 - Move the porting layer to MbedTLS dir. Changes in v3 - Update commit message. Changes in v4 - Control building legacy library v

[PATCH v4 21/29] mbedtls: add PKCS7 parser porting layer

2024-07-02 Thread Raymond Mao
Add porting layer for PKCS7 parser on top of MbedTLS PKCS7 library. Introduce _LEGACY and _MBEDTLS kconfigs for PKCS7 parser legacy and MbedTLS implementations respectively. Signed-off-by: Raymond Mao --- Changes in v2 - Move the porting layer to MbedTLS dir. - Fix EFI Capsule CI test failures. C

[PATCH v4 20/29] lib/crypto: Adapt x509_cert_parser to MbedTLS

2024-07-02 Thread Raymond Mao
Previous patch has introduced MbedTLS porting layer for x509 cert parser, here to adjust the header and makefiles accordingly. Signed-off-by: Raymond Mao --- Changes in v2 - Move the porting layer to MbedTLS dir. Changes in v3 - Update commit message. Changes in v4 - Control building legacy libra

[PATCH v4 19/29] mbedtls: add X509 cert parser porting layer

2024-07-02 Thread Raymond Mao
Add porting layer for X509 cert parser on top of MbedTLS X509 library. Introduce _LEGACY and _MBEDTLS kconfigs for X509 cert parser legacy and MbedTLS implementations respectively. Signed-off-by: Raymond Mao --- Changes in v2 - Move the porting layer to MbedTLS dir. Changes in v3 - None. Changes

[PATCH v4 18/29] lib/crypto: Adapt public_key header with MbedTLS

2024-07-02 Thread Raymond Mao
Previous patch has introduced MbedTLS porting layer for public key, here to adjust the header and makefiles accordingly. Signed-off-by: Raymond Mao --- Changes in v2 - Move the porting layer to MbedTLS dir. Changes in v3 - Update commit message. Changes in v4 - Control building legacy library via

[PATCH v4 17/29] mbedtls: add public key porting layer

2024-07-02 Thread Raymond Mao
Add porting layer for public key on top of MbedTLS X509 library. Introduce _LEGACY and _MBEDTLS kconfigs for public key legacy and MbedTLS implementations respectively. Signed-off-by: Raymond Mao --- Changes in v2 - Move the porting layer to MbedTLS dir. Changes in v3 - None. Changes in v4 - Intr

[PATCH v4 16/29] pkcs7: move common functions to PKCS7 helper

2024-07-02 Thread Raymond Mao
Move pkcs7_get_content_data as a helper function that can be shared by legacy crypto lib and MbedTLS implementation. Signed-off-by: Raymond Mao --- Changes in v4 - Initial patch. lib/crypto/Makefile | 1 + lib/crypto/pkcs7_helper.c | 40 +++ lib/crypto

[PATCH v4 15/29] x509: move common functions to x509 helper

2024-07-02 Thread Raymond Mao
Move x509_check_for_self_signed as a common helper function that can be shared by legacy crypto lib and MbedTLS implementation. Signed-off-by: Raymond Mao --- Changes in v4 - Initial patch. lib/crypto/Makefile | 1 + lib/crypto/x509_helper.c | 67 ++

[PATCH v4 14/29] public_key: move common functions to public key helper

2024-07-02 Thread Raymond Mao
Move public_key_free and public_key_signature_free as helper functions that can be shared by legacy crypto lib and MbedTLS implementation. Signed-off-by: Raymond Mao --- Changes in v4 - Initial patch. lib/crypto/Makefile| 4 +++- lib/crypto/public_key.c| 31

[PATCH v4 13/29] mbedtls/external: update MbedTLS PKCS7 test suites

2024-07-02 Thread Raymond Mao
Update the PKCS7 test suites for multiple certs. The PR for this patch is at: https://github.com/Mbed-TLS/mbedtls/pull/9001 For enabling EFI loader PKCS7 features with MbedTLS build, we need this patch on top of MbedTLS v3.6.0 before it is merged into the next MbedTLS LTS release. Signed-off-by:

[PATCH v4 12/29] mbedtls/external: support decoding multiple signer's cert

2024-07-02 Thread Raymond Mao
Support decoding multiple signer's cert in the signed data within a PKCS7 message. The PR for this patch is at: https://github.com/Mbed-TLS/mbedtls/pull/9001 For enabling EFI loader PKCS7 features with MbedTLS build, we need this patch on top of MbedTLS v3.6.0 before it is merged into the next Mb

[PATCH v4 11/29] mbedtls/external: support PKCS9 Authenticate Attributes

2024-07-02 Thread Raymond Mao
Populate PKCS9 Authenticate Attributes from signer info if it exists in a PKCS7 message. Add OIDs for describing objects using for Authenticate Attributes. The PR for this patch is at: https://github.com/Mbed-TLS/mbedtls/pull/9001 For enabling EFI loader PKCS7 features with MbedTLS build, we need

[PATCH v4 10/29] mbedtls/external: support Microsoft Authentication Code

2024-07-02 Thread Raymond Mao
Populate Microsoft Authentication Code from the content data into PKCS7 decoding context if it exists in a PKCS7 message. Add OIDs for describing objects using for Microsoft Authentication Code. The PR for this patch is at: https://github.com/Mbed-TLS/mbedtls/pull/9001 For enabling EFI loader PKC

[PATCH v4 09/29] makefile: add mbedtls include directories

2024-07-02 Thread Raymond Mao
Add the mbedtls include directories into the build system. Signed-off-by: Raymond Mao --- Changes in v2 - None. Changes in v3 - Remove changes for PLATFORM_CPPFLAGS. Changes in v4 - Fix errors when building without "O=". - Minor fix of the include directories. Makefile | 6 ++ 1 file change

[PATCH v4 08/29] hash: integrate hash on mbedtls

2024-07-02 Thread Raymond Mao
Integrate common/hash.c on the hash shim layer so that hash APIs from mbedtls can be leveraged by boot/image and efi_loader. Signed-off-by: Raymond Mao --- Changes in v2 - Use the original head files instead of creating new ones. Changes in v3 - Add handle checkers for malloc. Changes in v4 - Non

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

2024-07-02 Thread Raymond Mao
Implement digest shim layer on top of MbedTLS crypto library. Introduce _MBEDTLS kconfig for MbedTLS crypto implementations. Signed-off-by: Raymond Mao --- Changes in v2 - Split the shim layer into separated files and use the original head files instead of creating new ones. Changes in v3 - Ref

[PATCH v4 06/29] sha1: Remove sha1 non-watchdog API

2024-07-02 Thread Raymond Mao
We don't need an API specially for non-watchdog since sha1_csum_wd supports it by disabling CONFIG_HW_WATCHDOG and CONFIG_WATCHDOG. Set 0x1 as default chunk size for SHA1. Signed-off-by: Raymond Mao --- Changes in v4 - Initial patch. board/gdsys/a38x/hre.c | 2 +- include/u-boot/sha1.h |

[PATCH v4 05/29] md5: Remove md5 non-watchdog API

2024-07-02 Thread Raymond Mao
We don't need an API specially for non-watchdog since md5_wd supports it by disabling CONFIG_HW_WATCHDOG and CONFIG_WATCHDOG. Set 0x1 as default chunk size for MD5. Signed-off-by: Raymond Mao Reviewed-by: Ilias Apalodimas Reviewed-by: Michal Simek --- Changes in v3 - Initial patch. Changes

[PATCH v4 04/29] lib: Adapt digest header files to MbedTLS

2024-07-02 Thread Raymond Mao
Adapt digest header files to support both original libs and MbedTLS by switching on/off MBEDTLS_LIB_CRYPTO. Introduce _LEGACY kconfig for legacy hash implementations. FIXME: `IS_ENABLED` or `CONFIG_IS_ENABLED` is not applicable here, since including causes undefined reference on schedule() with s

Re: [RESEND PATCH v4 0/3] provide names for emmc hardware partitions

2024-07-02 Thread Dragan Simic
Hello Tim, On 2024-07-02 20:14, Tim Harvey wrote: On Fri, May 31, 2024 at 8:36 AM Tim Harvey wrote: Modern eMMC v4+ devices have multiple hardware partitions per the JEDEC specification described as: Boot Area Partition 1 Boot Area Partition 2 RPMB Partition General Purpose Partition 1

[PATCH v4 03/29] mbedtls: add mbedtls into the build system

2024-07-02 Thread Raymond Mao
Port mbedtls with adapted libc header files. Add mbedtls default config header file. Optimize mbedtls default config by disabling unused features to reduce the target size. Add mbedtls kbuild makefile. Add Kconfig skeleton and config submenu entry for selecting crypto libraries between mbedtls and

[PATCH v4 02/29] mbedtls: Add script to update MbedTLS subtree

2024-07-02 Thread Raymond Mao
lib/mbedtls/update-mbedtls-subtree.sh is a wrapper of git subtree commands. Usage from U-Boot top directory, run: $ ./lib/mbedtls/update-mbedtls-subtree.sh pull $ ./lib/mbedtls/update-mbedtls-subtree.sh pick Signed-off-by: Raymond Mao --- Changes in v2 - Initial patch. Changes in v3 - None. Ch

[PATCH v4 01/29] CI: Exclude MbedTLS subtree for CONFIG checks

2024-07-02 Thread Raymond Mao
Since MbedTLS is an external repo with its own coding style, exclude it from Azure and gitlab CI CONFIG checks. Signed-off-by: Raymond Mao Reviewed-by: Tom Rini Reviewed-by: Ilias Apalodimas --- Changes in v2 - Initial patch. Changes in v3 - None. Changes in v4 - None. .azure-pipelines.yml |

[PATCH v4 00/29] Integrate MbedTLS v3.6 LTS with U-Boot

2024-07-02 Thread Raymond Mao
Integrate MbedTLS v3.6 LTS (currently v3.6.0-RC1) with U-Boot. Motivations: 1. MbedTLS is well maintained with LTS versions. 2. LWIP is integrated with MbedTLS and easily to enable HTTPS. 3. MbedTLS recently switched license back to GPLv2. Prerequisite: - This patch ser

Re: [RESEND PATCH v4 0/3] provide names for emmc hardware partitions

2024-07-02 Thread Tim Harvey
On Fri, May 31, 2024 at 8:36 AM Tim Harvey wrote: > > Modern eMMC v4+ devices have multiple hardware partitions per the JEDEC > specification described as: > Boot Area Partition 1 > Boot Area Partition 2 > RPMB Partition > General Purpose Partition 1 > General Purpose Partition 2 > General P

Re: [PATCH v2] binman: Update cbfstool

2024-07-02 Thread Tom Rini
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 signature.asc Description: PGP signature

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

2024-07-02 Thread Andrew Goodbody
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/-/issues/35 Signed-off-by: Andrew Goodbody --- Changes in v2:

[PATCH v2] binman: Update cbfstool

2024-07-02 Thread Simon Glass
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 --- Changes in v2: - Use the same version as CI (4.22.01) tools/binman/btool/cbfstool.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) di

Re: [RFC v2 2/2] doc: add missing table of content links

2024-07-02 Thread Simon Glass
Hi Sam, On Tue, 2 Jul 2024 at 02:14, Sam Povilus wrote: > Please can you add a commit message? > Signed-off-by: Sam Povilus > --- > doc/usage/fit/index.rst | 20 > 1 file changed, 20 insertions(+) > > diff --git a/doc/usage/fit/index.rst b/doc/usage/fit/index.rst > index

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

2024-07-02 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 1/1] doc: fix heading level of itest examples

2024-07-02 Thread Simon Glass
On Mon, 1 Jul 2024 at 21:43, Heinrich Schuchardt wrote: > > The Examples section should be on the second heading level. > > Signed-off-by: Heinrich Schuchardt > --- > doc/usage/cmd/itest.rst | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Reviewed-by: Simon Glass

Re: [PATCH 1/2] led: Implement software led blinking

2024-07-02 Thread Simon Glass
Hi Mikhail, On Tue, 2 Jul 2024 at 12:54, Mikhail Kshevetskiy wrote: > > > On 27.06.2024 22:05, Simon Glass wrote: > > Hi Mikhail, > > > > On Thu, 27 Jun 2024 at 12:31, Mikhail Kshevetskiy > > wrote: > >> From: Michael Polyntsov > >> > >> If hardware (or driver) doesn't support leds blinking, it

Please pull u-boot-dm/next

2024-07-02 Thread Simon Glass
Hi Tom, This is for -next but we need to figure out the patch I just sent first. I can redo the pull if that patch is acceptable, or you can pick that up first. https://source.denx.de/u-boot/custodians/u-boot-dm/-/pipelines/21414 Note failure fixed by: https://patchwork.ozlabs.org/project/uboot/p

Re: [PATCH] gpt: allow spaces in partition list

2024-07-02 Thread Simon Glass
Hi Mikhail, On Tue, 2 Jul 2024 at 10:42, Mikhail Kshevetskiy wrote: > > > On 27.06.2024 22:05, Simon Glass wrote: > > Hi Mikhail, > > > > On Thu, 27 Jun 2024 at 12:29, Mikhail Kshevetskiy > > wrote: > >> This allows spliting partition list to several lines in environment file > >> > >> ex: > >>

Warnings with xilinx_zynqmp_virt

2024-07-02 Thread Simon Glass
Hi Michal, I am seeing errors when building xilinx_zynqmp_virt: Can't set hash 'value' property for 'hash' node(FDT_ERR_NOSPACE) Can't set hash value for 'hash' hash node in 'fdt_35' image node Can't add verification data for node 'fdt_35' () The problem is visible in CI, e.g. [1] A bisect poin

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

2024-07-02 Thread Simon Glass
Hi Andrew, On Mon, 1 Jul 2024 at 16:01, Andrew Goodbody wrote: > > On 01/07/2024 14:57, Simon Glass wrote: > > Hi Andrew, > > > > On Mon, 24 Jun 2024 at 10:29, Andrew Goodbody > > wrote: > >> > >> Add the STACKPROTECTOR symbols to the script that generates the > >> symbols that should not be rem

[PATCH] mx9: Correct repeatable build error

2024-07-02 Thread Simon Glass
For some reason every second time imx93_11x11_evk is built it gives an error: make O=/tmp/x BINMAN_ALLOW_MISSING=1 It seems to sometimes skip generation of the .cfgout file and then eventually Binman complains: ValueError: Error 1 running 'mkimage -d ./mkimage.spl.mkimage -n spl/u-bo

Re: [PATCH] usb: cdns3: continue probe even when USB PHY device does not exist

2024-07-02 Thread Roger Quadros
On 02/07/2024 16:36, Siddharth Vadapalli wrote: > On Tue, Jul 02, 2024 at 04:20:43PM +0300, Roger Quadros wrote: >> >> >> On 02/07/2024 15:07, Siddharth Vadapalli wrote: >>> Prior to commit cd295286c786 ("usb: cdns3: avoid error messages if phys >>> don't exist"), cdns3_probe() errors out only o

Re: [PATCH] Proposed changes to dynamic UUIDs v3

2024-07-02 Thread Caleb Connolly
t approach and just make the the v5 generator produce a little endian GUID rather than a BE UUID. V4 is here: https://lore.kernel.org/u-boot/20240702-b4-dynamic-uuid-v4-0-a00c82d1f...@linaro.org Kind regards, Signed-off-by: Vincent Stehlé Cc: Caleb Connolly Cc: Tom Rini Cc: Heinrich Schu

Re: [PATCH] usb: cdns3: continue probe even when USB PHY device does not exist

2024-07-02 Thread Siddharth Vadapalli
On Tue, Jul 02, 2024 at 04:20:43PM +0300, Roger Quadros wrote: > > > On 02/07/2024 15:07, Siddharth Vadapalli wrote: > > Prior to commit cd295286c786 ("usb: cdns3: avoid error messages if phys > > don't exist"), cdns3_probe() errors out only on failing to initialize the > > USB2/USB3 PHY. However

[PATCH v4 10/10] test: lib/uuid: add unit tests for dynamic UUIDs

2024-07-02 Thread Caleb Connolly
Add some basic unit tests to validate that the UUID generation behaves as expected. This matches the implementation in efi_loader for sandbox and a Qualcomm board and should catch any regressions. Signed-off-by: Caleb Connolly --- test/lib/uuid.c | 82

[PATCH v4 09/10] tools: mkeficapsule: support generating dynamic GUIDs

2024-07-02 Thread Caleb Connolly
Add a tool that can generate GUIDs that match those generated internally by U-Boot for capsule update fw_images. Dynamic UUIDs in U-Boot work by taking a namespace UUID and hashing it with the board compatible and fw_image name. This tool accepts the same inputs and will produce the same GUID as

[PATCH v4 08/10] tools: mkeficapsule: use u-boot UUID library

2024-07-02 Thread Caleb Connolly
Replace the use of libuuid with U-Boot's own UUID library. This prepares us to add support for generating v5 GUIDs. Signed-off-by: Caleb Connolly --- tools/Makefile | 8 tools/mkeficapsule.c | 53 ++-- 2 files changed, 10 insertions

[PATCH v4 07/10] include: export uuid.h

2024-07-02 Thread Caleb Connolly
Move this header to include/u-boot/ so that it can be used by external tools. Signed-off-by: Caleb Connolly --- arch/arm/mach-rockchip/board.c | 2 +- board/cobra5272/flash.c| 2 +- board/gardena/smart-gateway-mt7688/board.c | 2 +- board/socrates/socrates.c

[PATCH v4 06/10] lib: uuid: supporting building as part of host tools

2024-07-02 Thread Caleb Connolly
Adjust the UUID library code so that it can be compiled as part of a host tool. This removes the one redundant log_debug() call, as well as the incorrectly defined LOG_CATEGORY. In general this is a fairly trivial change, just adjusting includes and disabling list_guid. This will be used by a ne

[PATCH v4 05/10] sandbox: switch to dynamic UUIDs

2024-07-02 Thread Caleb Connolly
Migrate sandbox over to generating it's capsule update image GUIDs dynamically from the namespace and board/image info. Update the reference and tests to use the new GUIDs. Signed-off-by: Caleb Connolly --- board/sandbox/sandbox.c | 16 include/s

[PATCH v4 04/10] doc: uefi: document dynamic UUID generation

2024-07-02 Thread Caleb Connolly
Document how platforms can generate GUIDs at runtime rather than maintaining a list of UUIDs per-board. Reviewed-by: Ilias Apalodimas Signed-off-by: Caleb Connolly --- doc/develop/uefi/uefi.rst | 27 +++ 1 file changed, 27 insertions(+) diff --git a/doc/develop/uefi/uef

[PATCH v4 03/10] efi: add a helper to generate dynamic UUIDs

2024-07-02 Thread Caleb Connolly
Introduce a new helper efi_capsule_update_info_gen_ids() which populates the capsule update fw images image_type_id field. This allows for determinstic UUIDs to be used that can scale to a large number of different boards and board variants without the need to maintain a big list. We call this fro

[PATCH v4 02/10] lib: uuid: add UUID v5 support

2024-07-02 Thread Caleb Connolly
Add support for generating version 5 UUIDs, these are determistic and work by hashing a "namespace" UUID together with some unique data. One intended usecase is to allow for dynamically generate payload UUIDs for UEFI capsule updates, so that supported boards can have their own UUIDs without needin

[PATCH v4 01/10] efi: define struct efi_guid

2024-07-02 Thread Caleb Connolly
This let's us forward declare efi_guid_t in the UUID code without pulling in efi.h Signed-off-by: Caleb Connolly --- include/efi.h | 2 +- tools/eficapsule.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/include/efi.h b/include/efi.h index c3c4b93f860a..b92c961a2afd

[PATCH v4 00/10] efi: CapsuleUpdate: support for dynamic UUIDs

2024-07-02 Thread Caleb Connolly
As more boards adopt support for the EFI CapsuleUpdate mechanism, there is a growing issue of being able to target updates to them properly. The current mechanism of hardcoding UUIDs for each board at compile time is unsustainable, and maintaining lists of GUIDs is similarly cumbersome. In this se

Re: [PATCH] usb: cdns3: continue probe even when USB PHY device does not exist

2024-07-02 Thread Roger Quadros
On 02/07/2024 15:07, Siddharth Vadapalli wrote: > Prior to commit cd295286c786 ("usb: cdns3: avoid error messages if phys > don't exist"), cdns3_probe() errors out only on failing to initialize the > USB2/USB3 PHY. However, since commit cd295286c786, absence of the PHY > device is also treated a

Re: [PATCH] board: phytec: k3: k3_ddrss_patch: Add ddr phy reg count

2024-07-02 Thread Wadim Egorov
Am 02.07.24 um 11:22 schrieb Dominik Haller: Add and use the correct number of ddr phy registers to update the corresponding settings. Fixes: cbf5c99ef317 ("board: phytec: common: Introduce a method to inject DDR timings deltas") Signed-off-by: Dominik Haller Reviewed-by: Wadim Egorov -

Re: [PATCH 0/2] Import environment variables from FIT configuration

2024-07-02 Thread Quentin Schulz
Hi Lukas, On 7/2/24 2:58 PM, Lukas Funke wrote: Hi Quentin, On 02.07.2024 13:37, Quentin Schulz wrote: Hi Lukas, On 7/2/24 1:01 PM, Lukas Funke wrote: Hi Quentin, On 02.07.2024 11:16, Quentin Schulz wrote: Hi Lukas, On 7/2/24 8:48 AM, lukas.funke-...@weidmueller.com wrote: From: Lukas Fu

Re: [PATCH 1/2] led: Implement software led blinking

2024-07-02 Thread Mikhail Kshevetskiy
On 27.06.2024 22:05, Simon Glass wrote: > Hi Mikhail, > > On Thu, 27 Jun 2024 at 12:31, Mikhail Kshevetskiy > wrote: >> From: Michael Polyntsov >> >> If hardware (or driver) doesn't support leds blinking, it's >> now possible to use software implementation of blinking instead. >> This relies on

Re: [PATCH 1/2] spi: soft_spi: fix miso gpio property name

2024-07-02 Thread Mikhail Kshevetskiy
On 27.06.2024 14:34, Fabio Estevam wrote: > [You don't often get email from feste...@gmail.com. Learn why this is > important at https://aka.ms/LearnAboutSenderIdentification ] > > On Thu, Jun 27, 2024 at 8:31 AM Mikhail Kshevetskiy > wrote: >> Signed-off-by: Mikhail Kshevetskiy >> --- >> dri

Re: [PATCH] gpt: allow spaces in partition list

2024-07-02 Thread Mikhail Kshevetskiy
On 27.06.2024 22:05, Simon Glass wrote: > Hi Mikhail, > > On Thu, 27 Jun 2024 at 12:29, Mikhail Kshevetskiy > wrote: >> This allows spliting partition list to several lines in environment file >> >> ex: >> >> gpt_partition_list= >> name=boot1,size=5MiB,start=0x10

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

2024-07-02 Thread Paul Geurts
Hi Ye, > Hi Paul, > >On 7/1/2024 8:39 PM, Paul Geurts wrote: >> Hi Ye, >> >>> Hi Paul, >>> >>> On 6/26/2024 3:17 PM, Paul Geurts wrote: Hi, Thanks for the feedback. > Hi Paul, > > On 6/24/2024 8:09 PM, Fabio Estevam wrote: > >> Hi Paul, >> >> On Fri, Jun

Re: [PATCH] ARM: socfpga: fix broken function call for arria10

2024-07-02 Thread Lothar Rubusch
Hi U-boot-teers! (...) > Reviewed-by: Marek Vasut > > Could you by any chance also send a patch to enable this QSPI NOR > support on one of the A10 targets, so the CI would compile this code and > this code wouldn't bitrot again ? > > Thanks I've sent you now a series of one patch showing the e

[PATCH] ARM: socfpga: fix broken function call for arria10

2024-07-02 Thread Lothar Rubusch
Remove obsolete arguments in the function call. The call's argument list differs from its more recent definition. This breaks compilation of the 'socfpga_arria10_defconfig' target, with additionally enabled: CONFIG_CADENCE_QSPI=y' The removed arguments are obtained from device-tree declaration.

[PATCH] ARM: socfpga: show broken function call for arria10

2024-07-02 Thread Lothar Rubusch
This patch only shows the problem on an existing platform. Turning on the cadence QSPI flash breaks the build for arria10. Can you reproduce? make socfpga_arria10_defconfig make Signed-off-by: Lothar Rubusch --- NB: Don't apply this config/patch permanently! The Cadence QSPI flash can be tri

Re: [PATCH 0/2] Import environment variables from FIT configuration

2024-07-02 Thread Lukas Funke
Hi Quentin, On 02.07.2024 13:37, Quentin Schulz wrote: Hi Lukas, On 7/2/24 1:01 PM, Lukas Funke wrote: Hi Quentin, On 02.07.2024 11:16, Quentin Schulz wrote: Hi Lukas, On 7/2/24 8:48 AM, lukas.funke-...@weidmueller.com wrote: From: Lukas Funke This series enables U-Boot to import enviro

[PATCH] usb: cdns3: continue probe even when USB PHY device does not exist

2024-07-02 Thread Siddharth Vadapalli
Prior to commit cd295286c786 ("usb: cdns3: avoid error messages if phys don't exist"), cdns3_probe() errors out only on failing to initialize the USB2/USB3 PHY. However, since commit cd295286c786, absence of the PHY device is also treated as an error, resulting in a regression. Extend commit cd295

Re: [PATCH 0/2] Import environment variables from FIT configuration

2024-07-02 Thread Quentin Schulz
Hi Lukas, On 7/2/24 1:01 PM, Lukas Funke wrote: Hi Quentin, On 02.07.2024 11:16, Quentin Schulz wrote: Hi Lukas, On 7/2/24 8:48 AM, lukas.funke-...@weidmueller.com wrote: From: Lukas Funke This series enables U-Boot to import environment variables from the selectd FIT configuration. One u

Re: [PATCH 0/2] Import environment variables from FIT configuration

2024-07-02 Thread Lukas Funke
Hi Quentin, On 02.07.2024 11:16, Quentin Schulz wrote: Hi Lukas, On 7/2/24 8:48 AM, lukas.funke-...@weidmueller.com wrote: From: Lukas Funke This series enables U-Boot to import environment variables from the selectd FIT configuration. One use-case is that the overall build process enriches

Re: [PATCH 1/2] spi: soft_spi: fix miso gpio property name

2024-07-02 Thread Fabio Estevam
Hi Mikhail, On Tue, Jul 2, 2024 at 6:50 AM Mikhail Kshevetskiy wrote: > This is exactly what the patch did. Actually it just fix a miss-print. > Other properties already have a proper fallback. Ah, you are right. I introduced this issue in commit 2e9fe73a883a ("spi: soft_spi: Support the recom

[PATCH 2/2] clk: imx: Fix wrong flags assignment clk-composite-93

2024-07-02 Thread Michael Trimarchi
The mux flags (u8), div flags (u8), and gate flags (u8) are not the clk flags (unsigned long). They have different meanings Signed-off-by: Michael Trimarchi --- drivers/clk/imx/clk-composite-93.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/clk/imx/clk-composite-93.c b/drivers/

[PATCH 1/2] clk: imx: Fix wrong flags assignment clk-composite-8m

2024-07-02 Thread Michael Trimarchi
The mux flags (u8), div flags (u8), and gate flags (u8) are not the clk flags (unsigned long). They have different meanings Signed-off-by: Michael Trimarchi --- drivers/clk/imx/clk-composite-8m.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/clk/imx/clk-composit

[PATCH] board: phytec: k3: k3_ddrss_patch: Add ddr phy reg count

2024-07-02 Thread Dominik Haller
Add and use the correct number of ddr phy registers to update the corresponding settings. Fixes: cbf5c99ef317 ("board: phytec: common: Introduce a method to inject DDR timings deltas") Signed-off-by: Dominik Haller --- board/phytec/common/k3/k3_ddrss_patch.c | 3 ++- 1 file changed, 2 insertion

Re: [PATCH 0/2] Import environment variables from FIT configuration

2024-07-02 Thread Quentin Schulz
Hi Lukas, On 7/2/24 8:48 AM, lukas.funke-...@weidmueller.com wrote: From: Lukas Funke This series enables U-Boot to import environment variables from the selectd FIT configuration. One use-case is that the overall build process enriches the FIT configuration node with dm-verity information wh

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

2024-07-02 Thread Ye Li
Hi Paul, On 7/1/2024 8:39 PM, Paul Geurts wrote: Hi Ye, Hi Paul, On 6/26/2024 3:17 PM, Paul Geurts wrote: Hi, Thanks for the feedback. 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 { +s

  1   2   >