[PATCH] lib: sparse: Make CHUNK_TYPE_RAW buffer aligned

2021-11-04 Thread qianfanguijin
From: qianfan Zhao CHUNK_TYPE_RAW buffer is not aligned, and flash sparse images by fastboot will report "Misaligned operation" if DCACHE is enabled. Flashing Sparse Image CACHE: Misaligned operation at range [8428, 84001028] CACHE: Misaligned operation at range [84001034, 84002034] CACHE:

Re: [PATCH v5 05/11] test/py: efi_capsule: add image authentication test

2021-11-04 Thread AKASHI Takahiro
On Thu, Nov 04, 2021 at 08:02:37PM -0600, Simon Glass wrote: > Hi Takahiro, > > On Thu, 4 Nov 2021 at 19:21, AKASHI Takahiro > wrote: > > > > On Wed, Nov 03, 2021 at 08:49:04PM -0600, Simon Glass wrote: > > > Hi Takahiro, > > > > > > On Wed, 3 Nov 2021 at 20:04, AKASHI Takahiro > > > wrote: >

Re: [PATCH v5 04/11] doc: update UEFI document for usage of mkeficapsule

2021-11-04 Thread AKASHI Takahiro
On Thu, Nov 04, 2021 at 09:11:51AM -0600, Simon Glass wrote: > Hi Takahiro, > > On Wed, 3 Nov 2021 at 19:49, AKASHI Takahiro > wrote: > > > > On Tue, Nov 02, 2021 at 08:57:48AM -0600, Simon Glass wrote: > > > Hi Takahiro, > > > > > > On Thu, 28 Oct 2021 at 23:20, AKASHI Takahiro > > > wrote: >

Re: [RFC 07/22] dm: blk: add UCLASS_PARTITION

2021-11-04 Thread AKASHI Takahiro
On Thu, Nov 04, 2021 at 08:02:05PM -0600, Simon Glass wrote: > Hi, > > On Tue, 2 Nov 2021 at 01:43, Heinrich Schuchardt wrote: > > > > > > > > On 11/1/21 03:14, Simon Glass wrote: > > > Hi Takahiro, > > > > > > On Sun, 31 Oct 2021 at 19:52, AKASHI Takahiro > > > wrote: > > >> > > >> On Sun, Oct

Re: [PATCH v5 02/11] tools: mkeficapsule: add firmwware image signing

2021-11-04 Thread AKASHI Takahiro
On Thu, Nov 04, 2021 at 08:02:40PM -0600, Simon Glass wrote: > Hi Takahiro, > > On Thu, 4 Nov 2021 at 19:04, AKASHI Takahiro > wrote: > > > > Hi, Simon, > > > > On Thu, Nov 04, 2021 at 09:11:59AM -0600, Simon Glass wrote: > > > Hi Mark, > > > > > > On Thu, 4 Nov 2021 at 08:31, Mark Kettenis >

fastboot flash sparse image: buffer aligned

2021-11-04 Thread qianfan
Hi: I'm testing 'fastboot flash' command with sparse images. Due to we write CHUNK_TYPE_RAW format data from unaligned buffer to mmc, the u-boot will report warinnig messages like this: "CACHE: Misaligned operation at range [8428, 84001028]" if DCACHE is enabled. Is it a bug that we need

fastboot flash sparse image: buffer aligned

2021-11-04 Thread qianfan
Hi: I'm testing 'fastboot flash' command with sparse images. Due to we write CHUNK_TYPE_RAW format data from unaligned buffer to mmc, the u-boot will report warinnig messages like this: "CACHE: Misaligned operation at range [8428, 84001028]" if DCACHE is enabled. Is it a bug that we need

Re: [PATCH 00/31] passage: Define a standard for firmware data flow

2021-11-04 Thread Simon Glass
Hi François, On Tue, 2 Nov 2021 at 10:03, François Ozog wrote: > > Hi Simon, > > On Tue, 2 Nov 2021 at 15:59, Simon Glass wrote: >> >> Hi François, >> >> On Mon, 1 Nov 2021 at 02:53, François Ozog wrote: >> > >> > Hi Simon, >> > >> > this seems a great endeavor. I'd like to better understand

Re: [PATCH v2 10/12] env: Add support for overwriting default environment via sysinfo

2021-11-04 Thread Simon Glass
On Wed, 3 Nov 2021 at 17:23, Marek Behún wrote: > > From: Marek Behún > > The default_environment[] buffer is built at compile time, but sometimes > it makes sense for some default environment variables to be determined > at runtime, for example: > - one board code may support different boards,

Re: [PATCH v5 02/11] tools: mkeficapsule: add firmwware image signing

2021-11-04 Thread Simon Glass
Hi Mark, On Thu, 4 Nov 2021 at 10:51, Mark Kettenis wrote: > > > From: Simon Glass > > Date: Thu, 4 Nov 2021 09:11:59 -0600 > > > > Hi Mark, > > > > On Thu, 4 Nov 2021 at 08:31, Mark Kettenis wrote: > > > > > > > From: Simon Glass > > > > Date: Wed, 3 Nov 2021 20:51:25 -0600 > > > > > > > >

Re: [PATCH] bootstage: Differentiate boot progress kconfig entries

2021-11-04 Thread Simon Glass
On Wed, 3 Nov 2021 at 08:09, Jan Kiszka wrote: > > From: Jan Kiszka > > Both U-Boot proper and SPL entries were using the same description. > > Fixes: b55881dd ("bootstage: Add SPL support") > Signed-off-by: Jan Kiszka > --- > common/Kconfig.boot | 2 +- > 1 file changed, 1 insertion(+), 1

Re: [PATCH v2 11/12] arm: mvebu: Espressobin: Use new API for setting default env at runtime

2021-11-04 Thread Simon Glass
On Wed, 3 Nov 2021 at 17:23, Marek Behún wrote: > > From: Marek Behún > > ESPRESSObin's board code uses an ad-hoc solution for ensuring that > ethaddrs are not overwritten by `env default -a` command and that the > `fdtfile` is set to correct value when `env default -a` is called. > > This

Re: [PATCH v2 12/12] env: Remove support for read-write default_environment[]

2021-11-04 Thread Simon Glass
On Wed, 3 Nov 2021 at 17:23, Marek Behún wrote: > > From: Marek Behún > > Since the default_environment[] buffer is not overwritten anymore by any > board, remove support for read-write default_environment[]. > > Signed-off-by: Marek Behún > --- > include/env_default.h | 2 -- >

Re: [RFC 1/2] patman: introduce RunException

2021-11-04 Thread Simon Glass
On Thu, 4 Nov 2021 at 12:53, Heiko Thiery wrote: > > The RunException will be throws when the a command's return_code is not > equal zero. With this an external caller can catch that and has access > to the command/args, the result code, the stdout and stderr output. > > Signed-off-by: Heiko

Re: [PATCH 4/6 v4] configs: Enable tpmv2 mmio on qemu for arm/arm64

2021-11-04 Thread Simon Glass
On Wed, 3 Nov 2021 at 09:09, Ilias Apalodimas wrote: > > A previous commit is adding an MMIO TPMv2 driver. Include in the default > qemu arm configs, since we plan on using them on EFI testing > > Signed-off-by: Ilias Apalodimas > --- > configs/qemu_arm64_defconfig | 2 ++ >

Re: [PATCH v2 06/12] sysinfo: Add get_str_list() method

2021-11-04 Thread Simon Glass
On Wed, 3 Nov 2021 at 17:23, Marek Behún wrote: > > From: Marek Behún > > Add get_str_list() method to sysinfo operations. > > The get_str_list() method is similar to get_str(), but receives one > additional argument, @idx, and fills in the @idx-th string from a given > list. > > Add sandbox

Re: [PATCH 2/6 v4] tpm2: Add a TPMv2 MMIO TIS driver

2021-11-04 Thread Simon Glass
Hi Ilias, On Wed, 3 Nov 2021 at 09:09, Ilias Apalodimas wrote: > > Add support for devices that expose a TPMv2 though MMIO. > Apart from those devices, we can use the driver in our QEMU setups and > test TPM related code which is difficult to achieve using the sandbox > driver (e.g test the EFI

Re: [PATCH 5/6 v4] doc: qemu: Add instructions for swtpm usage

2021-11-04 Thread Simon Glass
On Wed, 3 Nov 2021 at 09:09, Ilias Apalodimas wrote: > > A previous patch added support for an mmio based TPM. > Add an example in QEMU on it's usage > > Signed-off-by: Ilias Apalodimas > --- > doc/board/emulation/qemu-arm.rst | 25 + > 1 file changed, 25 insertions(+) >

Re: [PATCH 3/6 v4] tpm: Use the new API on tpm2 spi driver

2021-11-04 Thread Simon Glass
On Wed, 3 Nov 2021 at 09:09, Ilias Apalodimas wrote: > > Convert our SPI TPM driver and use the newly added API > > Signed-off-by: Ilias Apalodimas > --- > drivers/tpm/Makefile | 2 +- > drivers/tpm/tpm2_tis_spi.c | 440 +++-- > 2 files changed, 32

Re: [PATCH v2 07/12] sysinfo: Make .detect() non-mandatory

2021-11-04 Thread Simon Glass
On Wed, 3 Nov 2021 at 17:23, Marek Behún wrote: > > From: Marek Behún > > Make it so that if the .detect() method is not implemented, the sysinfo > is considered to be present. > > Signed-off-by: Marek Behún > --- > drivers/sysinfo/sysinfo-uclass.c | 5 + > 1 file changed, 1 insertion(+),

Re: [PATCH 3/3] bootstage: Enable SPL bootstage in sandbox_spl config

2021-11-04 Thread Simon Glass
On Wed, 3 Nov 2021 at 08:20, Marek Vasut wrote: > > Enable SPL bootstage support in sandbox_spl config to detect build issues in > CI. > > Signed-off-by: Marek Vasut > Cc: Simon Glass > --- > configs/sandbox_spl_defconfig | 1 + > 1 file changed, 1 insertion(+) > Reviewed-by: Simon Glass

Re: [RFC 07/22] dm: blk: add UCLASS_PARTITION

2021-11-04 Thread Simon Glass
Hi, On Tue, 2 Nov 2021 at 01:43, Heinrich Schuchardt wrote: > > > > On 11/1/21 03:14, Simon Glass wrote: > > Hi Takahiro, > > > > On Sun, 31 Oct 2021 at 19:52, AKASHI Takahiro > > wrote: > >> > >> On Sun, Oct 31, 2021 at 07:15:17PM -0600, Simon Glass wrote: > >>> Hi Takahiro, > >>> > >>> On

Re: [PATCH] image-board: fix wrong implementation ram disk address setup from cmdline

2021-11-04 Thread Simon Glass
Hi Art, On Mon, 1 Nov 2021 at 00:52, Art Nikpal wrote: > > hi Simon > > > Fixes: f33a2c1bd0f ("image: Remove #ifdefs from select_ramdisk()") > > (I believe, can you confirm?) > > confirm f33a2c1bd0f has wrong implementation logic for raw ramdisk image > sure i have tested it ... Thanks! BTW

Re: [PATCH 2/3] bootstage: Fix up remaining bootstage config misuse

2021-11-04 Thread Simon Glass
On Wed, 3 Nov 2021 at 08:20, Marek Vasut wrote: > > All these macros really guard show_boot_progress() function, which > is used only if BOOT_PROGRESS is enabled in Kconfig. Fix up the > macro guards. > > Signed-off-by: Marek Vasut > Cc: Simon Glass > --- > board/Seagate/dockstar/dockstar.c

Re: [PATCH v2 1/2] test/dm: fix watchdog test

2021-11-04 Thread Simon Glass
On Tue, 2 Nov 2021 at 12:44, Heinrich Schuchardt wrote: > > For successful execution of the watchdog test we need both the GPIO as well > as the SANDBOX watchdog. > > Avoid a build failure for CONFIG_WDT_GPIO=n. > > Signed-off-by: Heinrich Schuchardt > --- > v2: > move changes to

Re: [PATCH 4/7] common: board_r: drop initr_addr_map wrapper

2021-11-04 Thread Simon Glass
On Sat, 30 Oct 2021 at 00:52, Ovidiu Panait wrote: > > Add a return value to init_addr_map and use it directly in the > post-relocation init sequence, rather than using a wrapper stub. > > Signed-off-by: Ovidiu Panait > --- > > arch/arm/include/asm/mmu.h | 2 +- >

Re: [RFC 2/2] binman: catch RunException for mkimage runtime failure

2021-11-04 Thread Simon Glass
Hi Heiko, On Thu, 4 Nov 2021 at 12:53, Heiko Thiery wrote: > > In case mkimage exits with a return code other than zero do not stop. > Print an error message and go on. > > Signed-off-by: Heiko Thiery > --- > tools/binman/etype/mkimage.py | 8 +++- > 1 file changed, 7 insertions(+), 1

Re: [PATCH v3 0/6] Improved sysreset/watchdog uclass integration

2021-11-04 Thread Simon Glass
Hi, On Thu, 4 Nov 2021 at 19:22, Stefan Roese wrote: > > Hi Andre, > > On 05.11.21 00:11, Andre Przywara wrote: > > On Thu, 4 Nov 2021 11:37:57 +0100 > > Stefan Roese wrote: > > > > Hi Stefan, > > > >> On 04.11.21 04:55, Samuel Holland wrote: > >>> This series hooks up the watchdog uclass to

Re: [PATCH v5 02/11] tools: mkeficapsule: add firmwware image signing

2021-11-04 Thread Simon Glass
Hi Takahiro, On Thu, 4 Nov 2021 at 19:04, AKASHI Takahiro wrote: > > Hi, Simon, > > On Thu, Nov 04, 2021 at 09:11:59AM -0600, Simon Glass wrote: > > Hi Mark, > > > > On Thu, 4 Nov 2021 at 08:31, Mark Kettenis wrote: > > > > > > > From: Simon Glass > > > > Date: Wed, 3 Nov 2021 20:51:25 -0600 >

Re: [PATCH v5 05/11] test/py: efi_capsule: add image authentication test

2021-11-04 Thread Simon Glass
Hi Takahiro, On Thu, 4 Nov 2021 at 19:21, AKASHI Takahiro wrote: > > On Wed, Nov 03, 2021 at 08:49:04PM -0600, Simon Glass wrote: > > Hi Takahiro, > > > > On Wed, 3 Nov 2021 at 20:04, AKASHI Takahiro > > wrote: > > > > > > On Tue, Nov 02, 2021 at 08:58:15AM -0600, Simon Glass wrote: > > > > Hi

Re: [PATCH v2 1/1] efi_selftest: add selftest for EFI_TCG2_PROTOCOL and Measured Boot

2021-11-04 Thread Simon Glass
Hi, On Thu, 4 Nov 2021 at 04:43, Masahisa Kojima wrote: > > This commit adds the missing EFI_TCG2_PROTOCOL selftest > and Measured Boot selftest in lib/efi_selftest. > > This selftest includes PE/COFF image measurement test, > some PCR values are different in each architecture. > With that, this

Re: [PATCH v2 09/12] env: Change return behaviour of env_set_default_vars()

2021-11-04 Thread Simon Glass
On Wed, 3 Nov 2021 at 17:23, Marek Behún wrote: > > From: Marek Behún > > The env_set_default_vars() function does not document return value and > behaves differently from other env_* functions. > > Change the return value to return 0 on success and -ve on error. > > Signed-off-by: Marek Behún

Re: [PATCH v2 08/12] sysinfo: Add support for iterating string list

2021-11-04 Thread Simon Glass
Hi Marek, On Wed, 3 Nov 2021 at 17:23, Marek Behún wrote: > > From: Marek Behún > > Add function > sysinfo_get_str_list_max_len() > to determine length of the longest string in a string list, functions > sysinfo_str_list_first() and > sysinfo_str_list_next() > to support iteration over

Re: [PATCH v2 04/12] sysinfo: Make sysinfo_get_str() behave like snprintf()

2021-11-04 Thread Simon Glass
Hi Marek, On Wed, 3 Nov 2021 at 17:23, Marek Behún wrote: > > From: Marek Behún > > Currently sysinfo_get_str() returns 0 if a string is filled in the > given buffer, and otherwise gives no simple mechanism to determine > actual string length. > > One implementation returns -ENOSPC if buffer is

Re: [PATCH v2 05/12] test: Use ut_asserteq_str() instead of ut_assertok(strcmp())

2021-11-04 Thread Simon Glass
On Wed, 3 Nov 2021 at 17:23, Marek Behún wrote: > > From: Marek Behún > > Use the ut_asserteq_str() assertion instead of strcmp() function and > ut_assertok()ing it's result. > > Signed-off-by: Marek Behún > --- > test/dm/cpu.c | 4 ++-- > test/dm/soc.c | 4 ++-- > test/dm/sysinfo.c

Re: [PATCH 1/6 v4] tpm2: Introduce TIS tpm core

2021-11-04 Thread Simon Glass
Hi Ilias, On Wed, 3 Nov 2021 at 09:09, Ilias Apalodimas wrote: > > There's a lot of code duplication in U-Boot right now. All the TPM TIS > compatible drivers we have at the moment have their own copy of a TIS > implementation. > > So let's create a common layer which implements the core TIS

Re: [PATCH 1/3] bootstage: Update SPL support Kconfig description

2021-11-04 Thread Simon Glass
On Wed, 3 Nov 2021 at 08:20, Marek Vasut wrote: > > Add "in SPL" suffix to the Kconfig description, to make it differ > from the "proper" U-Boot entry. > > Signed-off-by: Marek Vasut > Cc: Simon Glass > --- > common/Kconfig.boot | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-)

Re: [PATCH] boards: siemens: iot2050: Re-enable bootstage reporting

2021-11-04 Thread Simon Glass
On Wed, 3 Nov 2021 at 08:11, Jan Kiszka wrote: > > From: Jan Kiszka > > This got lost while fixing up the condition in > board/siemens/iot2050/board.c > > Fixes: b55881dd ("bootstage: Add SPL support") > Signed-off-by: Jan Kiszka > --- > configs/iot2050_defconfig | 2 ++ > 1 file changed, 2

Re: [PATCH] cmd: pxe_utils: sysboot: add label override support

2021-11-04 Thread Simon Glass
Hi Amjad, On Fri, 22 Oct 2021 at 09:55, Amjad Ouled-Ameur wrote: > > This will allow consumers to choose a pxe label at runtime instead of > having to prompt the user. One good use-case for this, is choosing > whether or not to apply a dtbo depending on the hardware configuration. > e.g: for

Re: [PATCH] introduce CONFIG_DEVICE_TREE_INCLUDES

2021-11-04 Thread Simon Glass
Hi Roman, Good luck! I must get a copy of that BOFH book. Regards, Simon On Sun, 31 Oct 2021 at 22:30, Roman Kopytin wrote: > > Hi, all > Currently I am waiting some help from our IT infrastructure department. > > -Original Message- > From: Simon Glass > Sent: Tuesday, October 26,

Re: [PATCH 0/2] add selftest for EFI_TCG2_PROTOCOL and Measured Boot

2021-11-04 Thread Simon Glass
Hi Ilias, On Tue, 2 Nov 2021 at 10:28, Ilias Apalodimas wrote: > > Hi Simon. > > On Tue, 2 Nov 2021 at 16:55, Simon Glass wrote: > > > > Hi Masahisa, > > > > On Tue, 2 Nov 2021 at 02:03, Masahisa Kojima > > wrote: > > > > > > Hi Simon, > > > > > > On Mon, 25 Oct 2021 at 04:54, Simon Glass

Re: [PATCH v1] env: setenv add resolve value option

2021-11-04 Thread Simon Glass
Hi Art, On Wed, 3 Nov 2021 at 01:41, Art Nikpal wrote: > > > The high level problem I have with this patch is that we keep going back > > to "we really need to update to a modern hush (or other shell) rather > > than patching new features in to our ancient fork". > > Yes it will be fine ! Does

Re: [PATCH v2 2/2] sandbox: fix sandbox_wdt_expire_now()

2021-11-04 Thread Simon Glass
On Tue, 2 Nov 2021 at 12:44, Heinrich Schuchardt wrote: > > With CONFIG_SYSRESET_WATCHDOG=y the sandbox can use a watchdog based system > reset. > > To make this work calling sandbox_wdt_expire_now() must lead to a reset. > > With this change we can test the development suggested in > > [PATCH

Re: [PATCH v3 0/6] Improved sysreset/watchdog uclass integration

2021-11-04 Thread Stefan Roese
Hi Andre, On 05.11.21 00:11, Andre Przywara wrote: On Thu, 4 Nov 2021 11:37:57 +0100 Stefan Roese wrote: Hi Stefan, On 04.11.21 04:55, Samuel Holland wrote: This series hooks up the watchdog uclass to automatically register watchdog devices for use with sysreset, doing a bit of minor

Re: [PATCH v5 05/11] test/py: efi_capsule: add image authentication test

2021-11-04 Thread AKASHI Takahiro
On Wed, Nov 03, 2021 at 08:49:04PM -0600, Simon Glass wrote: > Hi Takahiro, > > On Wed, 3 Nov 2021 at 20:04, AKASHI Takahiro > wrote: > > > > On Tue, Nov 02, 2021 at 08:58:15AM -0600, Simon Glass wrote: > > > Hi Takahiro, > > > > > > On Thu, 28 Oct 2021 at 23:25, AKASHI Takahiro > > > wrote: >

Re: [PATCH v5 02/11] tools: mkeficapsule: add firmwware image signing

2021-11-04 Thread AKASHI Takahiro
Hi, Simon, On Thu, Nov 04, 2021 at 09:11:59AM -0600, Simon Glass wrote: > Hi Mark, > > On Thu, 4 Nov 2021 at 08:31, Mark Kettenis wrote: > > > > > From: Simon Glass > > > Date: Wed, 3 Nov 2021 20:51:25 -0600 > > > > > > Hi Mark, > > > > > > On Tue, 2 Nov 2021 at 09:13, Mark Kettenis > > >

Re: [PATCH] net: phy: realtek: Set correct bit for 8211e delays

2021-11-04 Thread Tom Rini
On Thu, Nov 04, 2021 at 07:36:01PM -0500, Samuel Holland wrote: > commit cb1d40151489 ("net: phy: realtek: Add tx/rx delay config for > 8211e") added support for all four possible delay combinations, but set > the wrong bit in the RXID and TXID cases. This broke Ethernet on boards > using one of

[PATCH] net: phy: realtek: Set correct bit for 8211e delays

2021-11-04 Thread Samuel Holland
commit cb1d40151489 ("net: phy: realtek: Add tx/rx delay config for 8211e") added support for all four possible delay combinations, but set the wrong bit in the RXID and TXID cases. This broke Ethernet on boards using one of those settings. Fix that by fixing the bit selection. Fixes:

Re: [PATCH v3 0/6] Improved sysreset/watchdog uclass integration

2021-11-04 Thread Andre Przywara
On Thu, 4 Nov 2021 11:37:57 +0100 Stefan Roese wrote: Hi Stefan, > On 04.11.21 04:55, Samuel Holland wrote: > > This series hooks up the watchdog uclass to automatically register > > watchdog devices for use with sysreset, doing a bit of minor cleanup > > along the way. > > > > The goal is for

Re: Please pull u-boot-marvell (watchdog related)

2021-11-04 Thread Tom Rini
On Thu, Nov 04, 2021 at 11:38:25AM +0100, Stefan Roese wrote: > Hi Tom, > > please pull the following watchdog related patches: > Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature

Re: [PATCH] net: uclass: Save ethernet MAC address when generated

2021-11-04 Thread Ramon Fried
On Thu, Nov 4, 2021 at 3:40 PM Michael Walle wrote: > > Am 2021-11-04 14:15, schrieb Michal Simek: > > On 11/4/21 13:27, Michael Walle wrote: > >> Am 2021-11-04 12:16, schrieb Michal Simek: > >>> On 11/4/21 03:09, Grygorii Strashko wrote: > > > On 02/11/2021 12:27, Michal Simek

Re: [RFC 0/2] Do not stop with an error when mkimage fails

2021-11-04 Thread Heiko Thiery
Hi Wolfgang, Am Do., 4. Nov. 2021 um 20:12 Uhr schrieb Wolfgang Denk : > > Dear Heiko, > > In message <20211104185231.2927-1-heiko.thi...@gmail.com> you wrote: > > While converting to binman for an imx8mq board, it has been found that > > building in the u-boot CI fails. This is because an imx8mq

Re: [PATCH] xilinx: zynqmp: fix ZYNQMP_RESTORE_JTAG check

2021-11-04 Thread Oleksandr Suvorov
On Thu, Nov 4, 2021 at 9:28 PM Ricardo Salveti wrote: > > Config check should be done without the SPL_ prefix. > > Signed-off-by: Ricardo Salveti Reviewed-by: Oleksandr Suvorov > --- > board/xilinx/zynqmp/zynqmp.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git

[PATCH] xilinx: zynqmp: fix ZYNQMP_RESTORE_JTAG check

2021-11-04 Thread Ricardo Salveti
Config check should be done without the SPL_ prefix. Signed-off-by: Ricardo Salveti --- board/xilinx/zynqmp/zynqmp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/board/xilinx/zynqmp/zynqmp.c b/board/xilinx/zynqmp/zynqmp.c index dfb5dab7afa..2b5239ccb47 100644 ---

Re: [RFC 0/2] Do not stop with an error when mkimage fails

2021-11-04 Thread Wolfgang Denk
Dear Heiko, In message <20211104185231.2927-1-heiko.thi...@gmail.com> you wrote: > While converting to binman for an imx8mq board, it has been found that > building in the u-boot CI fails. This is because an imx8mq requires an > external binary (signed_hdmi_imx8m.bin). If this file cannot be

[RFC 2/2] binman: catch RunException for mkimage runtime failure

2021-11-04 Thread Heiko Thiery
In case mkimage exits with a return code other than zero do not stop. Print an error message and go on. Signed-off-by: Heiko Thiery --- tools/binman/etype/mkimage.py | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/tools/binman/etype/mkimage.py

[RFC 1/2] patman: introduce RunException

2021-11-04 Thread Heiko Thiery
The RunException will be throws when the a command's return_code is not equal zero. With this an external caller can catch that and has access to the command/args, the result code, the stdout and stderr output. Signed-off-by: Heiko Thiery --- tools/patman/tools.py | 16 +--- 1 file

[RFC 0/2] Do not stop with an error when mkimage fails

2021-11-04 Thread Heiko Thiery
While converting to binman for an imx8mq board, it has been found that building in the u-boot CI fails. This is because an imx8mq requires an external binary (signed_hdmi_imx8m.bin). If this file cannot be found mkimage fails. To work around the problem the exception is caught, an error message is

[PATCH 5/5] vexpress64: Enable VIRTIO_NET network driver

2021-11-04 Thread Peter Hoyes
From: Peter Hoyes The SMSC driver is using the old driver model. Init the virtio system in vexpress64.c so that the network device is discovered. Signed-off-by: Peter Hoyes --- board/armltd/vexpress64/vexpress64.c | 7 +++ include/configs/vexpress_aemv8.h | 4 ++-- 2 files changed, 9

[PATCH 4/5] vexpress64: Enable OF_CONTROL and OF_BOARD for VExpress64

2021-11-04 Thread Peter Hoyes
From: Peter Hoyes Capture x0 in lowlevel_init.S as potential fdt address. Modify board_fdt_blob_setup to use fdt address from either vexpress_aemv8.h or lowlevel_init.S. Signed-off-by: Peter Hoyes --- board/armltd/vexpress64/Makefile| 5 + board/armltd/vexpress64/lowlevel_init.S

[PATCH 2/5] vexpress64: Refactor header file to make it easier to add new FVPs

2021-11-04 Thread Peter Hoyes
From: Peter Hoyes Rename from vexpress_aemv8a.h -> vepxress_aemv8.h as new FVPs may not be v8-A. No change in behavior. This is towards future work to enable support for the FVP_BaseR. Signed-off-by: Peter Hoyes --- board/armltd/vexpress64/Kconfig | 2 +-

[PATCH 3/5] vexpress64: Clean up BASE_FVP boot configuration

2021-11-04 Thread Peter Hoyes
From: Peter Hoyes Move env var address values to #defines so they can be reused elsewhere. Rename env var names to those recommended in the README. Fix issue where fdt is called with invalid arguments when booting without a ramdisk. Signed-off-by: Peter Hoyes ---

[PATCH 1/5] doc: Add documentation for the Arm VExpress64 board configs

2021-11-04 Thread Peter Hoyes
From: Peter Hoyes Create a new documentation section for Arm Ltd boards with a sub-page for the VExpress64 boards (FVP-A and Juno). Signed-off-by: Peter Hoyes --- doc/board/armltd/index.rst | 9 ++ doc/board/armltd/vexpress64.rst | 49 +

[PATCH 0/5] VExpress64 board family improvements

2021-11-04 Thread Peter Hoyes
From: Peter Hoyes These patches add the follow improvements to the VExpress64 board family (BASE_FVP and Juno): * Add documentation * Allow use of OF_BOARD for BASE_FVP (off by default) * Allow use of the virtio-net driver (off by default) * Refactor header file to make it easier to add new

Re: [PATCH v5 02/11] tools: mkeficapsule: add firmwware image signing

2021-11-04 Thread Mark Kettenis
> From: Simon Glass > Date: Thu, 4 Nov 2021 09:11:59 -0600 > > Hi Mark, > > On Thu, 4 Nov 2021 at 08:31, Mark Kettenis wrote: > > > > > From: Simon Glass > > > Date: Wed, 3 Nov 2021 20:51:25 -0600 > > > > > > Hi Mark, > > > > > > On Tue, 2 Nov 2021 at 09:13, Mark Kettenis > > > wrote: > > >

Re: [PATCH v2 02/41] Makefile: Allow LTO to be disabled for a build

2021-11-04 Thread Tom Rini
On Thu, Nov 04, 2021 at 09:12:02AM -0600, Simon Glass wrote: > Hi Tom, > > On Thu, 4 Nov 2021 at 08:55, Tom Rini wrote: > > > > On Wed, Nov 03, 2021 at 08:49:01PM -0600, Simon Glass wrote: > > > Hi Tom, > > > > > > On Mon, 1 Nov 2021 at 15:58, Tom Rini wrote: > > > > > > > > On Sun, Oct 31,

Re: [PATCH] sf: Querying write-protect status before operating the flash

2021-11-04 Thread Tom Rini
On Wed, Nov 03, 2021 at 01:18:42PM +0100, Jan Kiszka wrote: > On 04.10.21 15:36, Jan Kiszka wrote: > > On 13.09.21 09:48, Jan Kiszka wrote: > >> Hi all, > >> > >> Chao, please no top-post on mailing list. Also check your mail client, > >> it seems to inject a lot of bogus newlines. > >> > >> On

Testing PCIe on Armada 370 (or older) in U-Boot

2021-11-04 Thread Pali Rohár
Hello! Recently I have sent more PCIe related patches for mvebu/armada SoCs into U-Boot. Now all were merged into U-Boot git master branch. I have tested everything on Armada 38x. I would like to ask if you have some Armada 370 board (or older, e.g. Dove, Kirwood, Discovery, Orion) and if you

Re: [External] - Slow MMC IO with Raspberry CM3+ Module

2021-11-04 Thread Mario Lombardo
Vincent, thank you very much. Applying the two patches mentioned in the series resolve the issues immediately. Is it expected to have this enhancement in the upcoming u-boot release? Cheers, Mario > On 3. Nov 2021, at 21:57, Vincent Fazio wrote: > > Mario, > >> -Original Message-

Re: [PATCH v2 02/41] Makefile: Allow LTO to be disabled for a build

2021-11-04 Thread Simon Glass
Hi Tom, On Thu, 4 Nov 2021 at 08:55, Tom Rini wrote: > > On Wed, Nov 03, 2021 at 08:49:01PM -0600, Simon Glass wrote: > > Hi Tom, > > > > On Mon, 1 Nov 2021 at 15:58, Tom Rini wrote: > > > > > > On Sun, Oct 31, 2021 at 05:46:43PM -0600, Simon Glass wrote: > > > > Hi Tom, > > > > > > > > On Wed,

Re: [PATCH v5 02/11] tools: mkeficapsule: add firmwware image signing

2021-11-04 Thread Simon Glass
Hi Mark, On Thu, 4 Nov 2021 at 08:31, Mark Kettenis wrote: > > > From: Simon Glass > > Date: Wed, 3 Nov 2021 20:51:25 -0600 > > > > Hi Mark, > > > > On Tue, 2 Nov 2021 at 09:13, Mark Kettenis wrote: > > > > > > > From: Simon Glass > > > > Date: Tue, 2 Nov 2021 08:56:50 -0600 > > > > > > > >

Re: [PATCH 4/4] watchdog: Automatically register device with sysreset

2021-11-04 Thread Simon Glass
Hi Samuel, On Wed, 3 Nov 2021 at 21:49, Samuel Holland wrote: > > On 10/31/21 6:46 PM, Simon Glass wrote: > > Hi Samuel, > > > > On Sun, 22 Aug 2021 at 14:41, Samuel Holland wrote: > >> > >> Add an option to automatically register the first watchdog device with > >> the wdt_reboot driver for

Re: [PATCH] boot: don't enable booti/bootz by default if FIT_SIGNATURE is set

2021-11-04 Thread Simon Glass
Hi, On Thu, 4 Nov 2021 at 05:23, Heinrich Schuchardt wrote: > > > > On 11/4/21 04:11, Rover Mo wrote: > > Dear Heinrich, > > > > > > Thank for your comments. > > > > > > >How about CONFIG_EFI_SECURE_BOOT? Should this also disable the default? > > > > I think yes. > > I will update the relation

Re: [PATCH v5 04/11] doc: update UEFI document for usage of mkeficapsule

2021-11-04 Thread Simon Glass
Hi Takahiro, On Wed, 3 Nov 2021 at 19:49, AKASHI Takahiro wrote: > > On Tue, Nov 02, 2021 at 08:57:48AM -0600, Simon Glass wrote: > > Hi Takahiro, > > > > On Thu, 28 Oct 2021 at 23:20, AKASHI Takahiro > > wrote: > > > > > > On Thu, Oct 28, 2021 at 09:17:48PM -0600, Simon Glass wrote: > > > > Hi

Re: [PATCH v2 02/41] Makefile: Allow LTO to be disabled for a build

2021-11-04 Thread Tom Rini
On Wed, Nov 03, 2021 at 08:49:01PM -0600, Simon Glass wrote: > Hi Tom, > > On Mon, 1 Nov 2021 at 15:58, Tom Rini wrote: > > > > On Sun, Oct 31, 2021 at 05:46:43PM -0600, Simon Glass wrote: > > > Hi Tom, > > > > > > On Wed, 27 Oct 2021 at 07:13, Tom Rini wrote: > > > > > > > > On Wed, Oct 27,

Re: [PATCH v5 02/11] tools: mkeficapsule: add firmwware image signing

2021-11-04 Thread Mark Kettenis
> From: Simon Glass > Date: Wed, 3 Nov 2021 20:51:25 -0600 > > Hi Mark, > > On Tue, 2 Nov 2021 at 09:13, Mark Kettenis wrote: > > > > > From: Simon Glass > > > Date: Tue, 2 Nov 2021 08:56:50 -0600 > > > > > > Hi Takahiro, > > > > > > > > - can we just build the tool always? > > > > > > > >

Re: [PATCH v3] efi_loader: add EFI_TCG2_PROTOCOL.SubmitCommand

2021-11-04 Thread Ilias Apalodimas
On Thu, 4 Nov 2021 at 15:56, Masahisa Kojima wrote: > > This commit adds the EFI_TCG2_PROTOCOL.SubmitCommand > required in the TCG PC Client PFP spec. > SubmitCommand enables to send the raw command to the TPM device. > > To implement this api, tpm2_submit_command() is added > into tpm-v2.c. > >

[PATCH 10/14] [WIP] dts: arm: stm32mp15: enable SCMI regulators (PWR & PMIC) on MP15-DK*

2021-11-04 Thread Etienne Carriere
Signed-off-by: Etienne Carriere --- arch/arm/dts/stm32mp151.dtsi | 40 +- arch/arm/dts/stm32mp15xx-dkx.dtsi | 122 +- 2 files changed, 138 insertions(+), 24 deletions(-) diff --git a/arch/arm/dts/stm32mp151.dtsi b/arch/arm/dts/stm32mp151.dtsi index

[PATCH 12/14] [HACK] mailbox: add new arm smc/hvc mailbox

2021-11-04 Thread Etienne Carriere
From: Etienne Carriere Basic mailbox inspired from https://patchwork.kernel.org/patch/11166071/. When sending a message, the mailbox invokes the Cortex-A Arm Trustzone secure monitor with an SMC or HVC instruction providing a function identifier in register R0/X0 defined by the DTB.

[PATCH 13/14] [TEST] dts: stm32mp15: scmi mailbox transport

2021-11-04 Thread Etienne Carriere
Signed-off-by: Etienne Carriere --- arch/arm/dts/stm32mp15-u-boot.dtsi | 4 arch/arm/dts/stm32mp151.dtsi | 21 - 2 files changed, 20 insertions(+), 5 deletions(-) diff --git a/arch/arm/dts/stm32mp15-u-boot.dtsi b/arch/arm/dts/stm32mp15-u-boot.dtsi index

[PATCH 09/14] [WIP] arm: dts: stm32mp15: enable OP-TEE and SCMI on EV1/DK2

2021-11-04 Thread Etienne Carriere
Enable OP-TEE resources and SMCI over OP-TEE on STM32MP15xx-EV1/DK2 boards. Signed-off-by: Etienne Carriere --- arch/arm/dts/stm32mp15-u-boot.dtsi | 81 +-- arch/arm/dts/stm32mp151.dtsi| 145 arch/arm/dts/stm32mp153.dtsi|

[PATCH 14/14] [HACK] configs: stm32mp15: enable earlyprintk

2021-11-04 Thread Etienne Carriere
Increase CONFIG_SYS_MALLOC_F_LEN for pre-reloc trace buffers and enable earlyprintk config for STM32. Change-Id: Ibcfcff1f0e6af7eb73b5c53c717bfc6ea20881af Signed-off-by: Etienne Carriere --- configs/stm32mp15_trusted_defconfig | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git

[PATCH 11/14] [TEST] dts: stm32mp1: use scmi smccc transport

2021-11-04 Thread Etienne Carriere
Requires OP-TEE OS to be built with CFG_STM32MP1_SCMI_SIP=y. Signed-off-by: Etienne Carriere --- arch/arm/dts/stm32mp151.dtsi | 5 +++-- arch/arm/dts/stm32mp157c-dk2.dts | 1 - arch/arm/dts/stm32mp157c-ed1.dts | 1 - 3 files changed, 3 insertions(+), 4 deletions(-) diff --git

[PATCH 08/14] dt-bindings: add stm32mp15 regulators binding IDs

2021-11-04 Thread Etienne Carriere
Define stm32mp15 voltage domain DT bindings IDs. Signed-off-by: Etienne Carriere --- .../regulator/st,stm32mp15-regulator.h| 29 +++ 1 file changed, 29 insertions(+) create mode 100644 include/dt-bindings/regulator/st,stm32mp15-regulator.h diff --git

[PATCH 04/14] firmware: scmi: smccc transport: use plat data, not priv data

2021-11-04 Thread Etienne Carriere
Change SCMI smccc transport drivers to use platform data rather than private data for channel reference since it only stores platform data retrieved from the DT. Consequently the probe handler is replaced with a of_to_plat handler. Signed-off-by: Etienne Carriere ---

[PATCH 07/14] configs: stm32mp15_trusted: enable SCMI voltage domains

2021-11-04 Thread Etienne Carriere
Enable CONFIG_DM_REGULATOR_SCMI for stm32mp15_trusted_defconfig to access voltage regulators exposed by SCMI server. Signed-off-by: Etienne Carriere --- configs/stm32mp15_trusted_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/configs/stm32mp15_trusted_defconfig

[PATCH 05/14] firmware: scmi: smccc transport: simplify probe sequence

2021-11-04 Thread Etienne Carriere
Minor simplification in scmi_smccc_probe() exit sequence. Signed-off-by: Etienne Carriere --- drivers/firmware/scmi/smccc_agent.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/firmware/scmi/smccc_agent.c b/drivers/firmware/scmi/smccc_agent.c index

[PATCH 03/14] firmware: scmi: mailbox transport: use plat data, not priv data

2021-11-04 Thread Etienne Carriere
Change SCMI mailbox transport drivers to use platform data rather than private data for channel reference since it only stores platform data retrieved from the DT. Consequently the probe handler is replaced with a of_to_plat handler. Signed-off-by: Etienne Carriere ---

[PATCH 06/14] configs: stm32mp15_trusted: increase heap for OP-TEE

2021-11-04 Thread Etienne Carriere
Increase malloc heap needed before relocation to invoke OP-TEE and eventually print some early debug traces. Change-Id: I23837c8c879ce5a080b23cb3ac5033d5ba73fb0d Signed-off-by: Etienne Carriere --- configs/stm32mp15_trusted_defconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

[PATCH 01/14] firmware: scmi: fix description of an API function

2021-11-04 Thread Etienne Carriere
Correct inline comment describing API function devm_scmi_process_msg(). Signed-off-by: Etienne Carriere --- include/scmi_agent.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/include/scmi_agent.h b/include/scmi_agent.h index f1be9ff209..ed40c7360a 100644 ---

[PATCH 02/14] firmware: scmi: mailbox transport: fix probe failure implementation

2021-11-04 Thread Etienne Carriere
Correct scmi mailbox probe function that can't free the scmi channel instance since its auto-allocated by the device model framework. Signed-off-by: Etienne Carriere --- drivers/firmware/scmi/mailbox_agent.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git

[PATCH 5/5] firmware: scmi: Add OP-TEE transport

2021-11-04 Thread Etienne Carriere
This change implements an SCMI transport for agent interfacing the OP-TEE SCMI service. OP-TEE provides an SCMI PTA (Pseudo-TA) for non-secure world to send SCMI messages over an identified channel. The driver implemented here uses a SMT shared memory for passing messages between client and

[PATCH 3/5] tee: optee: define TEE error code TEE_ERROR_SHORT_BUFFER

2021-11-04 Thread Etienne Carriere
Adds TEE_ERROR_SHORT_BUFFER as TEE error code. This error code is commonly used by TEEs to inform caller that the buffer(s) it provided is too small for the desired operation. Cc: Jens Wiklander Signed-off-by: Etienne Carriere --- include/tee.h | 1 + 1 file changed, 1 insertion(+) diff --git

[PATCH 4/5] dt-bindings: arm: scmi: OP-TEE as transport channel for SCMI messages

2021-11-04 Thread Etienne Carriere
Introduce compatible "linaro,scmi-optee" for SCMI transport channel based on an OP-TEE service invocation. Define "linaro,optee-channel-id" property to identify the OP-TEE SCMI channel used by the protocol(s). OP-TEE SCMI transport can either use shared memory or a static shared memory buffer

[PATCH 2/5] tee: optee: remove unused duplicated login Id macros

2021-11-04 Thread Etienne Carriere
Remove unused OPTEE_MSG_LOGIN_* ID macros as suitable TEE_LOGIN_* ID macros are already defined tee.h. Cc: Jens Wiklander Signed-off-by: Etienne Carriere --- This change was previously discussed in the U-Boot ML, latest is "[PATCH v2 2/2] tee: optee: support session login as REE kernel", at:

[PATCH 1/5] tee: define session login identifiers

2021-11-04 Thread Etienne Carriere
Define identifiers for clnt_login field in struct tee_open_session_arg based in GlobalPlatform Device TEE IDs and on the REE_KERNEL identifier extension from OP-TEE OS. Cc: Jens Wiklander Signed-off-by: Etienne Carriere --- This change was previously discussed in the U-Boot ML, see v2 at:

[PATCH v3] efi_loader: add EFI_TCG2_PROTOCOL.SubmitCommand

2021-11-04 Thread Masahisa Kojima
This commit adds the EFI_TCG2_PROTOCOL.SubmitCommand required in the TCG PC Client PFP spec. SubmitCommand enables to send the raw command to the TPM device. To implement this api, tpm2_submit_command() is added into tpm-v2.c. Signed-off-by: Masahisa Kojima --- Changes in v3: - use ternary

Re: [PATCH] net: phy: realtek: Add tx/rx delay config for 8211e

2021-11-04 Thread Tom Rini
On Thu, Nov 04, 2021 at 01:40:42PM +, Andre Przywara wrote: > On Thu, 4 Nov 2021 09:13:49 -0400 > Tom Rini wrote: > > Hi Tom, > > > On Tue, Oct 12, 2021 at 09:07:32PM -0500, Samuel Holland wrote: > > > > > Some boards need to change the tx/rx delay config in order for > > > gigabit

Re: [PATCH] net: phy: realtek: Add tx/rx delay config for 8211e

2021-11-04 Thread Andre Przywara
On Thu, 4 Nov 2021 09:13:49 -0400 Tom Rini wrote: Hi Tom, > On Tue, Oct 12, 2021 at 09:07:32PM -0500, Samuel Holland wrote: > > > Some boards need to change the tx/rx delay config in order for > > gigabit Ethernet to work. > > > > In Linux commit bbc4d71d6354 ("net: phy: realtek: fix rtl8211e

Re: [PATCH] net: uclass: Save ethernet MAC address when generated

2021-11-04 Thread Michael Walle
Am 2021-11-04 14:15, schrieb Michal Simek: On 11/4/21 13:27, Michael Walle wrote: Am 2021-11-04 12:16, schrieb Michal Simek: On 11/4/21 03:09, Grygorii Strashko wrote: On 02/11/2021 12:27, Michal Simek wrote: On 11/2/21 10:00, Michael Walle wrote: On Fri, Oct 29, 2021 at 2:14 PM Michal

  1   2   >