Re:[PATCH v2] boot: don't enable the non-secure boot commands by default if secure boot enabled

2021-11-08 Thread Rover Mo
Dear Simon Glass, > Also please do check tests (make qcheck) since sandbox enables more > options than most boards. All tests passed, please check the detailed log in the attachment. Best regards, Rover Mo At 2021-11-09 15:21:43, "Rover Mo" wrote: >To prevent booting unsigned images, don't

[PATCH v2] boot: don't enable the non-secure boot commands by default if secure boot enabled

2021-11-08 Thread Rover Mo
To prevent booting unsigned images, don't enable the non-secure boot commands(booti, bootz .etc) by default if secure boot enabled. Signed-off-by: Rover Mo Cc: Heinrich Schuchardt Cc: Simon Glass --- Changes for v2: - Don't enable the non-secure boot commands too if !EFI_SECURE_BOOT -

Re: [PATCH 06/11] mmc: fsl_esdhc_imx: fix mmc->clock with actual clock

2021-11-08 Thread Jaehoon Chung
On 11/6/21 2:39 AM, Sean Anderson wrote: > [ fsl_esdhc commit 30f6444d024a74ee48aa6969c1531aecd3c59deb ] > > Fix mmc->clock with actual clock which is divided by the > controller, and record it with priv->clock. > > Signed-off-by: Yangbo Lu > Signed-off-by: Sean Anderson > --- > > drivers/mmc

Re: [PATCH 04/11] mmc: fsl_esdhc_imx: clean up bus width configuration code

2021-11-08 Thread Jaehoon Chung
On 11/6/21 2:39 AM, Sean Anderson wrote: > [ fsl_esdhc commit 07bae1de382723b94244096953b05225572728cd ] > > This patch is to clean up bus width setting code. > > - For DM_MMC, remove getting "bus-width" from device tree. > This has been done in mmc_of_parse(). > > - For non-DM_MMC, move bus w

[PATCH 8/8 v7] MAINTAINERS: Add entry for TPM drivers

2021-11-08 Thread Ilias Apalodimas
TPM drivers have currently no maintainers. Add myself since I contributed the TIS implementation. Reviewed-by: Heinrich Schuchardt Reviewed-by: Simon Glass Signed-off-by: Ilias Apalodimas --- MAINTAINERS | 5 + 1 file changed, 5 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index

[PATCH 7/8 v7] doc: qemu: Add instructions for swtpm usage

2021-11-08 Thread Ilias Apalodimas
A previous patch added support for an mmio based TPM. Add an example in QEMU on it's usage Reviewed-by: Simon Glass Signed-off-by: Ilias Apalodimas --- doc/board/emulation/qemu-arm.rst | 25 + 1 file changed, 25 insertions(+) diff --git a/doc/board/emulation/qemu-arm.rs

[PATCH 5/8 v7] tpm: Use the new API on tpm2 spi driver

2021-11-08 Thread Ilias Apalodimas
Convert our SPI TPM driver and use the newly added API Reviewed-by: Simon Glass Signed-off-by: Ilias Apalodimas --- drivers/tpm/Makefile | 2 +- drivers/tpm/tpm2_tis_spi.c | 447 +++-- 2 files changed, 32 insertions(+), 417 deletions(-) diff --git a/driv

[PATCH 6/8 v7] configs: Enable tpmv2 mmio on qemu for arm/arm64

2021-11-08 Thread Ilias Apalodimas
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 Reviewed-by: Simon Glass Signed-off-by: Ilias Apalodimas --- configs/qemu_arm64_defconfig | 2 ++ configs/qemu_arm_defconfig | 2 ++ 2 files changed, 4 insert

[PATCH 3/8 v7] tpm2: Introduce TIS tpm core

2021-11-08 Thread Ilias Apalodimas
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 functions. Any driver added from now own, which is compatible with the TIS spec

[PATCH 4/8 v7] tpm2: Add a TPMv2 MMIO TIS driver

2021-11-08 Thread Ilias Apalodimas
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 TCG2 protocol). It's worth noting that a previous patch added TPMv2 TIS cor

[PATCH 2/8 v7] tpm: refactor function names for LPC based TPMs

2021-11-08 Thread Ilias Apalodimas
With the upcoming TPM2 API, some of the functions name are part of the new header file. So switch conflicting internal function names and defines. Reviewed-by: Simon Glass Reviewed-by: Heinrich Schuchardt Signed-off-by: Ilias Apalodimas --- drivers/tpm/tpm_tis_lpc.c | 4 ++-- 1 file changed,

[PATCH 1/8 v7] tpm: refactor function names and macros for infineon v1.2 TPM

2021-11-08 Thread Ilias Apalodimas
With the upcoming TPM2 API, some of the functions name are part of the new header file. So switch conflicting internal function names and defines. Reviewed-by: Simon Glass Signed-off-by: Ilias Apalodimas --- drivers/tpm/tpm_tis_infineon.c | 34 +- 1 file changed

[PATCH 0/8 v7] TPM cleanups and MMIO driver

2021-11-08 Thread Ilias Apalodimas
Hi! This is the update for [1]. Changes since v6: - added a few missing @return in function documentation - switched TPM1_ defines to TPM_INFINEON_ Changes since v5: - Fixed 2 tpmv1.2 drivers. Some of the internal driver functions were conflicting with the new API Changes since v4: - renamed st

Re: Testing pci_mvebu.c with Kirkwood SoCs

2021-11-08 Thread Tony Dinh
Hi Marek and Pali, On Mon, Nov 8, 2021 at 3:04 PM Marek Behún wrote: > > On Mon, 8 Nov 2021 14:48:03 -0800 > Tony Dinh wrote: > > > > So to avoid calling 'pci enum' manually, you need to put pci_init(); > > > function call into your board board_early_init_r() function. > > > > Thanks! will do th

Re: [PATCH] cmd: pxe_utils: Check fdtcontroladdr in label_boot

2021-11-08 Thread Ramon Fried
On Mon, Nov 8, 2021 at 6:53 PM Peter Hoyes wrote: > > On 28/10/2021 19:55, Ramon Fried wrote: > > On Sun, Oct 24, 2021 at 10:56 PM Simon Glass wrote: > >> Hi, > >> > >> On Fri, 15 Oct 2021 at 05:21, Andre Przywara > >> wrote: > >>> On Thu, 14 Oct 2021 12:24:45 -0600 > >>> Simon Glass wrote: >

Re: [PATCH 2/2] exynos: mmu: use CONFIG_IS_ENABLED

2021-11-08 Thread Jaehoon Chung
On 11/4/21 4:13 PM, Minkyu Kang wrote: > to fix following checkpatch warnings. > Use 'if (IS_ENABLED(CONFIG...))' instead of '#if or #ifdef' where > possible > > Signed-off-by: Minkyu Kang > Cc: Dzmitry Sankouski Reviewed-by: Jaehoon Chung Best Regards, Jaehoon Chung > --- > arch/arm/mach-e

Re: [PATCH 1/2] exynos78x0: pinctrl: set const to structs

2021-11-08 Thread Jaehoon Chung
On 11/4/21 4:13 PM, Minkyu Kang wrote: > to fix following checkpatch warings. > WARNING: struct should normally be const > > Signed-off-by: Minkyu Kang > Cc: Dzmitry Sankouski Reviewed-by: Jaehoon Chung Best Regards, Jaehoon Chung > --- > drivers/pinctrl/exynos/pinctrl-exynos78x0.c | 14 ++

Re: [PATCH 1/1] doc: mmc rescan speed mode

2021-11-08 Thread Jaehoon Chung
On 11/6/21 12:39 AM, Heinrich Schuchardt wrote: > Provide human readable descriptions of the speed nodes instead of the name > of constants from the code. > > Signed-off-by: Heinrich Schuchardt Reviewed-by: Jaehoon Chung Best Regards, Jaehoon Chung > --- > doc/usage/mmc.rst | 36

Re: [PATCH] mmc: mtk-sd: implement waiting for DAT0 line state

2021-11-08 Thread Jaehoon Chung
On 11/5/21 10:34 PM, Julien Masson wrote: > With the recent changes on mmc driver, we saw that the boot is ~5 secs > longer compared to v2021.07 on mediatek platforms. > > This regression is seen during mmc_init and caused by the following > patch [1]. > > Indeed since we did not support poll dat

Re: [PATCH v10 1/2] net: brcm: netXtreme driver

2021-11-08 Thread Marek Behún
On Mon, 8 Nov 2021 18:20:43 -0800 Roman Bacik wrote: > On Mon, Nov 8, 2021 at 5:12 PM Marek Behún wrote: > > > > On Mon, 8 Nov 2021 16:48:33 -0800 > > Roman Bacik wrote: > > > > > To be honest changing status codes coming from FW does not seem right. But > > > we will try to make the requeste

[PULL] u-boot-riscv/master

2021-11-08 Thread Leo Liang
Hi Tom, The following changes since commit 52207514ba419a69a8105d16997b025f966c8879: Merge branch '2021-11-05-Kconfig-syncs' (2021-11-05 15:38:46 -0400) are available in the Git repository at: https://source.denx.de/u-boot/custodians/u-boot-riscv.git for you to fetch changes up to 990e1e4

Re: [PATCH v10 1/2] net: brcm: netXtreme driver

2021-11-08 Thread Roman Bacik
On Mon, Nov 8, 2021 at 5:12 PM Marek Behún wrote: > > On Mon, 8 Nov 2021 16:48:33 -0800 > Roman Bacik wrote: > > > To be honest changing status codes coming from FW does not seem right. But > > we will try to make the requested changes. > > I looked at kernel's implementation of this driver and t

Re: [PATCH 01/14] mips: mtmips: add support for MediaTek MT7621 SoC

2021-11-08 Thread Weijie Gao
On Mon, 2021-11-08 at 22:39 +0100, Daniel Schwierzeck wrote: > Am Donnerstag, dem 04.11.2021 um 17:48 +0800 schrieb Weijie Gao: > > This patch adds support for MediaTek MT7621 SoC. > > All files are dedicated for u-boot. > > > > Currently only ramboot is supported. > > The default build target is

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

2021-11-08 Thread chaochao2021666
HI jagan and ta I might have a different view, the caller can not get the correct response even though we can not operate the device sucessfully. I think it is necessary to return a valid value. if return 0, the device cannot actually be operated but the correct results are not possible

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

2021-11-08 Thread Andre Przywara
On Mon, 8 Nov 2021 17:17:33 +0100 Heinrich Schuchardt wrote: > On 11/8/21 17:05, Tom Rini wrote: > > On Mon, Nov 08, 2021 at 08:58:33AM -0700, Simon Glass wrote: > >> Hi, > >> > >> On Sun, 7 Nov 2021 at 04:18, Heinrich Schuchardt > >> wrote: > >>> > >>> > >>> > >>> On 11/6/21 14:53, Tom Rini

[RFC 3/3] cmd: efidebug: handle booting from removable media

2021-11-08 Thread AKASHI Takahiro
Support for booting from removable media is now added to UEFI boot manager. Here we should modify efidebug command in order to define a proper "Boot" variable. With this patch applied, you will be able to specify the boot order, usb and scsi: => efidebug -b 1 SCSI scsi 0:1 => efidebug boot dum

[RFC 2/3] efi_loader: bootmgr: add booting from removable media

2021-11-08 Thread AKASHI Takahiro
Under the current implementation, booting from removal media using a architecture-specific default image name, say BOOTAA64.EFI, is supported only in distro_bootcmd script. See the commit 74522c898b35 ("efi_loader: Add distro boot script for removable media"). This is, however, half-baked implemen

[RFC 1/3] efi_loader: export efi_locate_device_handle()

2021-11-08 Thread AKASHI Takahiro
This function will be used in the next commit where some behavior of EFI boot manager will be expanded. Signed-off-by: AKASHI Takahiro --- include/efi_loader.h | 4 lib/efi_loader/efi_boottime.c | 7 +++ 2 files changed, 7 insertions(+), 4 deletions(-) diff --git a/include/efi

[RFC 0/3] efi_loader: bootmgr itself should support removable media

2021-11-08 Thread AKASHI Takahiro
# This task is a bit motivated by Simon's discussion about bootflow[1]. Booting UEFI system from removable media is currently supported by distro_bootcmd script. But the behavior should be best implemented in UEFI Boot Manager. (Historically, the boot manager was added later than the support for r

Re: [PATCH v10 1/2] net: brcm: netXtreme driver

2021-11-08 Thread Marek Behún
On Mon, 8 Nov 2021 16:48:33 -0800 Roman Bacik wrote: > To be honest changing status codes coming from FW does not seem right. But > we will try to make the requested changes. I looked at kernel's implementation of this driver and these hwrm functions and they don't return STATUS_*. Marek

RE: [PATCH v10 1/2] net: brcm: netXtreme driver

2021-11-08 Thread Roman Bacik
Hi Marek, > -Original Message- > From: Marek Behún > Sent: Monday, November 8, 2021 3:43 PM > To: Roman Bacik > Cc: U-Boot Mailing List ; Pali Rohar > ; Bharat Gooty ; Joe > Hershberger ; Ramon Fried > > Subject: Re: [PATCH v10 1/2] net: brcm: netXtreme driver > > Hello Roman, > > some

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

2021-11-08 Thread Tom Rini
On Mon, Nov 08, 2021 at 05:09:14PM -0700, Simon Glass wrote: > Hi Heinrich, > > On Mon, 8 Nov 2021 at 09:17, Heinrich Schuchardt > wrote: > > > > On 11/8/21 17:05, Tom Rini wrote: > > > On Mon, Nov 08, 2021 at 08:58:33AM -0700, Simon Glass wrote: > > >> Hi, > > >> > > >> On Sun, 7 Nov 2021 at 04:

Re: [PATCH v2 2/2] image: Explicitly declare do_bdinfo()

2021-11-08 Thread Simon Glass
On Mon, 8 Nov 2021 at 11:05, Andy Shevchenko wrote: > > Compiler is not happy: > > common/image-board.c: In function ‘boot_get_kbd’: > common/image-board.c:902:17: warning: implicit declaration of function > ‘do_bdinfo’ [-Wimplicit-function-declaration] > 902 | do_bdinfo(NULL, 0

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

2021-11-08 Thread Simon Glass
Hi Heinrich, On Mon, 8 Nov 2021 at 09:17, Heinrich Schuchardt wrote: > > On 11/8/21 17:05, Tom Rini wrote: > > On Mon, Nov 08, 2021 at 08:58:33AM -0700, Simon Glass wrote: > >> Hi, > >> > >> On Sun, 7 Nov 2021 at 04:18, Heinrich Schuchardt > >> wrote: > >>> > >>> > >>> > >>> On 11/6/21 14:53, To

Re: [PATCH v2 1/2] image: Fix typo in boot_get_kbd()

2021-11-08 Thread Simon Glass
On Mon, 8 Nov 2021 at 11:04, Andy Shevchenko wrote: > > After the commit 4ed37abc49c2 ("image: Remove ifdefs around > image_setup_linux() el at"): > > common/image-board.c: In function ‘boot_get_kbd’: > common/image-board.c:902:17: error: expected ‘)’ before ‘do_bdinfo’ > 902 | d

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

2021-11-08 Thread Simon Glass
Hi, On Mon, 8 Nov 2021 at 11:45, Ilias Apalodimas wrote: > > Hi chiming in a little late but > > On Mon, 8 Nov 2021 at 06:46, AKASHI Takahiro > wrote: > > > > On Fri, Nov 05, 2021 at 10:12:16AM -0600, Simon Glass wrote: > > > Hi Takahiro, > > > > > > On Thu, 4 Nov 2021 at 20:49, AKASHI Takahiro

Re: [PATCH v10 1/2] net: brcm: netXtreme driver

2021-11-08 Thread Marek Behún
Hello Roman, some last requests from me. On Mon, 8 Nov 2021 14:46:10 -0800 Roman Bacik wrote: > +#define bnxt_down_chip(bp) bnxt_hwrm_run(down_chip, bp, 0) > +#define bnxt_bring_chip(bp)bnxt_hwrm_run(bring_chip, bp, 1) Could these be changed to functions instead of macros, please? >

Re: Testing pci_mvebu.c with Kirkwood SoCs

2021-11-08 Thread Marek Behún
On Mon, 8 Nov 2021 14:48:03 -0800 Tony Dinh wrote: > > So to avoid calling 'pci enum' manually, you need to put pci_init(); > > function call into your board board_early_init_r() function. > > Thanks! will do that. Don't do that. Instead enable CONFIG_PCI_INIT_R and CONFIG_SYS_EARLY_PCI_INIT

Re: [PATCH v8 1/2] net: brcm: netXtreme driver

2021-11-08 Thread Pali Rohár
On Monday 08 November 2021 09:31:14 Roman Bacik wrote: > > -Original Message- > > From: Pali Rohár > > Sent: Sunday, November 7, 2021 1:44 AM > > To: Roman Bacik > > Cc: U-Boot Mailing List ; Bharat Gooty > > ; Joe Hershberger > > ; Ramon Fried > > Subject: Re: [PATCH v8 1/2] net: brcm:

Re: Testing pci_mvebu.c with Kirkwood SoCs

2021-11-08 Thread Tony Dinh
Hi Pali, On Mon, Nov 8, 2021 at 2:02 PM Pali Rohár wrote: > > On Monday 08 November 2021 12:54:39 Tony Dinh wrote: > > On Mon, Nov 8, 2021 at 3:11 AM Pali Rohár wrote: > > > On Sunday 07 November 2021 18:08:56 Tony Dinh wrote: > ... > > > > > I think some more investigation is needed. Why did we

[PATCH v10 1/2] net: brcm: netXtreme driver

2021-11-08 Thread Roman Bacik
From: Bharat Gooty Broadcom bnxt L2 driver support. Used by the Broadcom iproc platforms. Signed-off-by: Bharat Gooty Reviewed-by: Ramon Fried Signed-off-by: Roman Bacik --- Changes in v10: - move unnecessary code from bnxt_read_rom_hwaddr to bnxt_eth_probe Changes in v9: - remove bnxt_ver

[PATCH v10 2/2] board: brcm-ns3: Load netXtreme firmware

2021-11-08 Thread Roman Bacik
From: Bharat Gooty Load NetXtreme firmware in board_init when BNXT_ETH is selected. Signed-off-by: Bharat Gooty Signed-off-by: Roman Bacik --- (no changes since v4) Changes in v4: - remove bnxt commands - load bnxt firmware in board_init Changes in v3: - remove commands set/get mac/speed -

Re: [PATCH v9 1/2] net: brcm: netXtreme driver

2021-11-08 Thread Roman Bacik
On Mon, Nov 8, 2021 at 10:40 AM Roman Bacik wrote: > > From: Bharat Gooty > > Broadcom bnxt L2 driver support. Used by the Broadcom > iproc platforms. > > Signed-off-by: Bharat Gooty > Reviewed-by: Ramon Fried > > Signed-off-by: Roman Bacik > --- > > Changes in v9: > - remove bnxt_ver.h > - ad

Re: Testing pci_mvebu.c with Kirkwood SoCs

2021-11-08 Thread Pali Rohár
On Monday 08 November 2021 12:54:39 Tony Dinh wrote: > On Mon, Nov 8, 2021 at 3:11 AM Pali Rohár wrote: > > On Sunday 07 November 2021 18:08:56 Tony Dinh wrote: ... > > > > I think some more investigation is needed. Why did we need to do "pci > > > > enum", and then "usb reset", in that order, to

Re: [PATCH 02/14] mips: mtmips: add two reference boards for mt7621

2021-11-08 Thread Daniel Schwierzeck
Am Donnerstag, dem 04.11.2021 um 17:48 +0800 schrieb Weijie Gao: > The mt7621_rfb board supports integrated giga PHYs plus one external > giga PHYs. It also has up to 512MiB DDR3, 16MB SPI-NOR, 3 mini PCI-e > x1 > slots, SDXC and USB. > > The mt7621_nand_rfb board is almost the same as mt7621_rfb

Re: [PATCH 01/14] mips: mtmips: add support for MediaTek MT7621 SoC

2021-11-08 Thread Daniel Schwierzeck
Am Donnerstag, dem 04.11.2021 um 17:48 +0800 schrieb Weijie Gao: > This patch adds support for MediaTek MT7621 SoC. > All files are dedicated for u-boot. > > Currently only ramboot is supported. > The default build target is u-boot-lzma.img. > This file can be booted using bootm command, or be use

Re: Testing pci_mvebu.c with Kirkwood SoCs

2021-11-08 Thread Tony Dinh
Hi Pali, On Mon, Nov 8, 2021 at 3:11 AM Pali Rohár wrote: > > Hello! > > On Sunday 07 November 2021 18:08:56 Tony Dinh wrote: > > Hi Pali, > > > > On Sun, Nov 7, 2021 at 4:58 PM Tony Dinh wrote: > > > > > > Hi Pali, > > > > > > Looks like it is working! But in a weird way. Please see my response

[PATCH v9 1/2] net: brcm: netXtreme driver

2021-11-08 Thread Roman Bacik
From: Bharat Gooty Broadcom bnxt L2 driver support. Used by the Broadcom iproc platforms. Signed-off-by: Bharat Gooty Reviewed-by: Ramon Fried Signed-off-by: Roman Bacik --- Changes in v9: - remove bnxt_ver.h - add DRIVER_VERSION_* definitions to bnxt.h - remove pci_read_/pci_write_/pci_map

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

2021-11-08 Thread Ilias Apalodimas
Hi chiming in a little late but On Mon, 8 Nov 2021 at 06:46, AKASHI Takahiro wrote: > > On Fri, Nov 05, 2021 at 10:12:16AM -0600, Simon Glass wrote: > > Hi Takahiro, > > > > On Thu, 4 Nov 2021 at 20:49, AKASHI Takahiro > > wrote: > > > > > > On Thu, Nov 04, 2021 at 08:02:05PM -0600, Simon Glass

[PATCH v9 2/2] board: brcm-ns3: Load netXtreme firmware

2021-11-08 Thread Roman Bacik
From: Bharat Gooty Load NetXtreme firmware in board_init when BNXT_ETH is selected. Signed-off-by: Bharat Gooty Signed-off-by: Roman Bacik --- (no changes since v4) Changes in v4: - remove bnxt commands - load bnxt firmware in board_init Changes in v3: - remove commands set/get mac/speed -

[PATCH v2 2/2] image: Explicitly declare do_bdinfo()

2021-11-08 Thread Andy Shevchenko
Compiler is not happy: common/image-board.c: In function ‘boot_get_kbd’: common/image-board.c:902:17: warning: implicit declaration of function ‘do_bdinfo’ [-Wimplicit-function-declaration] 902 | do_bdinfo(NULL, 0, 0, NULL); | ^ Move the forward de

[PATCH v2 1/2] image: Fix typo in boot_get_kbd()

2021-11-08 Thread Andy Shevchenko
After the commit 4ed37abc49c2 ("image: Remove ifdefs around image_setup_linux() el at"): common/image-board.c: In function ‘boot_get_kbd’: common/image-board.c:902:17: error: expected ‘)’ before ‘do_bdinfo’ 902 | do_bdinfo(NULL, 0, 0, NULL); | ^ comm

RE: [PATCH v8 1/2] net: brcm: netXtreme driver

2021-11-08 Thread Roman Bacik
> -Original Message- > From: Pali Rohár > Sent: Sunday, November 7, 2021 1:44 AM > To: Roman Bacik > Cc: U-Boot Mailing List ; Bharat Gooty > ; Joe Hershberger > ; Ramon Fried > Subject: Re: [PATCH v8 1/2] net: brcm: netXtreme driver > > Hello! > > When sending a new version of patch, tr

[PATCH u-boot-marvell 04/11] tools: kwbimage: Fix validation of kwbimage v0

2021-11-08 Thread Marek Behún
From: Pali Rohár kwbimage v0 sldo has 32-bit data checksum at the end like kwbimage v1. Use same data checksum validation for both v0 and v1 image types. Signed-off-by: Pali Rohár Signed-off-by: Marek Behún --- tools/kwbimage.c | 80 ++-- 1 file ch

[PATCH u-boot-marvell 02/11] tools: kwbimage: Explicitly set version also for kwbimage v0

2021-11-08 Thread Marek Behún
From: Pali Rohár For documentation purposes update struct main_hdr_v0 to include information where version of the image must be stored. For kwbimage v0 it obviously must be 0. By default all image header memory is initialized to zero, therefore this change has no functional effect. Signed-off-by

[PATCH u-boot-marvell 11/11] tools: kwbimage: Properly set srcaddr in kwbimage v0

2021-11-08 Thread Marek Behún
From: Pali Rohár Field srcaddr in kwbimage v0 needs to be adjusted similarly like in v1. Signed-off-by: Pali Rohár Signed-off-by: Marek Behún --- tools/kwbimage.c | 22 ++ 1 file changed, 22 insertions(+) diff --git a/tools/kwbimage.c b/tools/kwbimage.c index 952023c14c..

[PATCH u-boot-marvell 09/11] tools: kwbimage: Fill the real header size into the main header

2021-11-08 Thread Marek Behún
From: Pali Rohár Fill the real header size without padding into the main header This allows to reduce final image when converting image to another format which does not need additional padding. Signed-off-by: Pali Rohár Signed-off-by: Marek Behún --- tools/kwbimage.c | 9 + 1 file ch

[PATCH u-boot-marvell 10/11] tools: kwbimage: Properly calculate and align kwbimage v0 header size

2021-11-08 Thread Marek Behún
From: Pali Rohár Kwbimage v0 has similar alignment requirements as v1. Signed-off-by: Pali Rohár Signed-off-by: Marek Behún --- tools/kwbimage.c | 24 1 file changed, 16 insertions(+), 8 deletions(-) diff --git a/tools/kwbimage.c b/tools/kwbimage.c index 114b313b4d..

[PATCH u-boot-marvell 08/11] tools: kwbimage: Align kwbimage header to proper size

2021-11-08 Thread Marek Behún
From: Pali Rohár Currently kwbimage header is always aligned to 4096 bytes. But it does not have to be aligned to such a high value. The header needs to be just 4-byte aligned, while some image types have additional alignment restrictions. This change reduces size of kwbimage binaries by removi

[PATCH u-boot-marvell 07/11] tools: kwbimage: Do not put final image padding to the image data size

2021-11-08 Thread Marek Behún
From: Pali Rohár This change allows to convert image from one format to another without need to include unnecessary padding (e.g. when target image format has smaller alignment requirement as source image format). Do it by storing real image data size without padding to the kwbimage header. Sig

[PATCH u-boot-marvell 06/11] tools: kwbimage: Align final UART image to 128 bytes

2021-11-08 Thread Marek Behún
From: Pali Rohár xmodem block size is 128 bytes, therefore it is possible to transfer only images with size multiple of 128 bytes. kwboot automatically pads image with zero bytes at the end to align it to 128 bytes boundary. Do this padding when generating image to allow uploading with other xmo

[PATCH u-boot-marvell 05/11] tools: kwbimage: Remove unused enums and prototypes

2021-11-08 Thread Marek Behún
From: Pali Rohár There are more unused enums and function prototypes. Remove them. The function kwbimage_check_params() does not return enum kwbimage_cmd_types, but a boolean value returned as int. Signed-off-by: Pali Rohár Signed-off-by: Marek Behún --- tools/kwbimage.c | 2 +- tools/kwbima

[PATCH u-boot-marvell 03/11] tools: kwbimage: Set BOOT_FROM by default to SPI

2021-11-08 Thread Marek Behún
From: Pali Rohár kwbimage must have valid blockid member instead of zero value. Thus if config file does not contain BOOT_FROM command, use by default the value for SPI booting (which is probably the most common). Signed-off-by: Pali Rohár Signed-off-by: Marek Behún --- tools/kwbimage.c | 32

[PATCH u-boot-marvell 01/11] tools: kwbimage: Add support for new commands UART_PORT and UART_MPP

2021-11-08 Thread Marek Behún
From: Pali Rohár These two commands allow to specify custom setting of UART port used for printing BootROM messages. Signed-off-by: Pali Rohár Signed-off-by: Marek Behún --- tools/kwbimage.c | 20 +++- 1 file changed, 19 insertions(+), 1 deletion(-) diff --git a/tools/kwbimag

[PATCH u-boot-marvell 00/11] Another kwbimage series

2021-11-08 Thread Marek Behún
From: Marek Behún Hi Stefan, Pali has prepared another series of patches for kwbimage, I have reviewed them. Marek Pali Rohár (11): tools: kwbimage: Add support for new commands UART_PORT and UART_MPP tools: kwbimage: Explicitly set version also for kwbimage v0 tools: kwbimage: Set BOOT_

[PATCH u-boot-marvell 00/11] Another kwbimage series

2021-11-08 Thread Marek Behún
From: Marek Behún Hi Stefan, Pali has prepared another series of patches for kwbimage, I have reviewed them. Marek Pali Rohár (11): tools: kwbimage: Add support for new commands UART_PORT and UART_MPP tools: kwbimage: Explicitly set version also for kwbimage v0 tools: kwbimage: Set BOOT_

Re: [PATCH] cmd: pxe_utils: Check fdtcontroladdr in label_boot

2021-11-08 Thread Peter Hoyes
On 28/10/2021 19:55, Ramon Fried wrote: On Sun, Oct 24, 2021 at 10:56 PM Simon Glass wrote: Hi, On Fri, 15 Oct 2021 at 05:21, Andre Przywara wrote: On Thu, 14 Oct 2021 12:24:45 -0600 Simon Glass wrote: Hi Simon, On Thu, 14 Oct 2021 at 02:40, Peter Hoyes wrote: From: Peter Hoyes If us

Re: [PATCH 1/8 v6] tpm: refactor function names and macros for infineon v1.2 TPM

2021-11-08 Thread Ilias Apalodimas
Hi Heinrich On Mon, 8 Nov 2021 at 17:55, Heinrich Schuchardt wrote: > > > > On 11/7/21 22:33, Ilias Apalodimas wrote: > > With the upcoming TPM2 API, some of the function names are part of the new > > header file. So switch conflicting driver defined function names and > > defines. > > > > Signe

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

2021-11-08 Thread François Ozog
On Fri, 5 Nov 2021 at 18:17, Simon Glass wrote: > ) to signal Hi François, > > On Fri, 5 Nov 2021 at 10:31, François Ozog > wrote: > > > > Hi Simon, > > > > Le ven. 5 nov. 2021 à 17:12, Simon Glass a écrit : > >> > >> Hi François, > >> > >> On Fri, 5 Nov 2021 at 02:27, François Ozog > wrote: >

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

2021-11-08 Thread Heinrich Schuchardt
On 11/8/21 17:05, Tom Rini wrote: On Mon, Nov 08, 2021 at 08:58:33AM -0700, Simon Glass wrote: Hi, On Sun, 7 Nov 2021 at 04:18, Heinrich Schuchardt wrote: On 11/6/21 14:53, Tom Rini wrote: On Sat, Nov 06, 2021 at 04:55:44AM +0100, Heinrich Schuchardt wrote: On 11/6/21 02:52, Andre Przy

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

2021-11-08 Thread Tom Rini
On Mon, Nov 08, 2021 at 09:09:20AM -0700, Simon Glass wrote: > Hi Tom, > > On Mon, 8 Nov 2021 at 09:05, Tom Rini wrote: > > > > On Mon, Nov 08, 2021 at 08:58:33AM -0700, Simon Glass wrote: > > > Hi, > > > > > > On Sun, 7 Nov 2021 at 04:18, Heinrich Schuchardt > > > wrote: > > > > > > > > > > > >

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

2021-11-08 Thread Simon Glass
Hi Tom, On Mon, 8 Nov 2021 at 09:05, Tom Rini wrote: > > On Mon, Nov 08, 2021 at 08:58:33AM -0700, Simon Glass wrote: > > Hi, > > > > On Sun, 7 Nov 2021 at 04:18, Heinrich Schuchardt > > wrote: > > > > > > > > > > > > On 11/6/21 14:53, Tom Rini wrote: > > > > On Sat, Nov 06, 2021 at 04:55:44AM +

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

2021-11-08 Thread Tom Rini
On Mon, Nov 08, 2021 at 08:58:33AM -0700, Simon Glass wrote: > Hi, > > On Sun, 7 Nov 2021 at 04:18, Heinrich Schuchardt > wrote: > > > > > > > > On 11/6/21 14:53, Tom Rini wrote: > > > On Sat, Nov 06, 2021 at 04:55:44AM +0100, Heinrich Schuchardt wrote: > > >> > > >> > > >> On 11/6/21 02:52, Andr

Re: [PATCH 2/8 v6] tpm: refactor function names for LPC based TPMs

2021-11-08 Thread Heinrich Schuchardt
On 11/7/21 22:33, Ilias Apalodimas wrote: With the upcoming TPM2 API, some of the function names are part of the new header file. So switch conflicting driver defined function names and defines. Signed-off-by: Ilias Apalodimas Reviewed-by: Heinrich Schuchardt --- drivers/tpm/tpm_tis_

Re: [PATCH 3/8 v6] tpm2: Introduce TIS tpm core

2021-11-08 Thread Simon Glass
Hi Ilias, On Sun, 7 Nov 2021 at 14:33, 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 fun

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

2021-11-08 Thread Simon Glass
Hi Takahiro, On Sun, 7 Nov 2021 at 21:15, AKASHI Takahiro wrote: > > On Fri, Nov 05, 2021 at 10:12:20AM -0600, Simon Glass wrote: > > Hi Takahiro, > > > > On Thu, 4 Nov 2021 at 21:24, AKASHI Takahiro > > wrote: > > > > > > On Thu, Nov 04, 2021 at 08:02:37PM -0600, Simon Glass wrote: > > > > Hi

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

2021-11-08 Thread Simon Glass
Hi, On Sun, 7 Nov 2021 at 04:18, Heinrich Schuchardt wrote: > > > > On 11/6/21 14:53, Tom Rini wrote: > > On Sat, Nov 06, 2021 at 04:55:44AM +0100, Heinrich Schuchardt wrote: > >> > >> > >> On 11/6/21 02:52, Andre Przywara wrote: > >>> On Fri, 5 Nov 2021 18:56:34 -0400 > >>> Tom Rini wrote: > >>

Re: [PATCH] introduce CONFIG_DEVICE_TREE_INCLUDES

2021-11-08 Thread Simon Glass
Hi Roman, On Mon, 8 Nov 2021 at 08:43, Roman Kopytin wrote: > > Hi, I sent patches, please check. > But before correct emails I sent several test emails. OK I see them, not copied to me, but I see them in the mailing list, thank you. Regards, Simon > > -Original Message- > From: Simon

Re: [PATCH 4/8 v6] tpm2: Add a TPMv2 MMIO TIS driver

2021-11-08 Thread Simon Glass
Hi Ilias, On Sun, 7 Nov 2021 at 14:33, 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 T

Re: [PATCH v4 09/34] efi: Add EFI uclass for media

2021-11-08 Thread Simon Glass
Hi Heinrich, On Mon, 8 Nov 2021 at 08:39, Heinrich Schuchardt wrote: > > > > On 11/8/21 06:29, AKASHI Takahiro wrote: > > On Sun, Nov 07, 2021 at 09:43:22AM -0700, Simon Glass wrote: > >> Hi Heinrich, > >> > >> On Sun, 7 Nov 2021 at 01:21, Heinrich Schuchardt > >> wrote: > >>> > >>> > >>> > >>>

Re: [PATCH 2/8 v6] tpm: refactor function names for LPC based TPMs

2021-11-08 Thread Simon Glass
On Sun, 7 Nov 2021 at 14:33, Ilias Apalodimas wrote: > > With the upcoming TPM2 API, some of the function names are part of the new > header file. So switch conflicting driver defined function names and > defines. > > Signed-off-by: Ilias Apalodimas > --- > drivers/tpm/tpm_tis_lpc.c | 4 ++-- >

Re: [PATCH v1 2/2] image: Explicitly declare do_bdinfo()

2021-11-08 Thread Simon Glass
Hi Andy, On Mon, 8 Nov 2021 at 01:38, Andy Shevchenko wrote: > > On Sun, Nov 7, 2021 at 6:43 PM Simon Glass wrote: > > On Sat, 6 Nov 2021 at 01:59, Andy Shevchenko > > wrote: > > > On Saturday, November 6, 2021, Simon Glass wrote: > > >> On Fri, 5 Nov 2021 at 14:10, Andy Shevchenko > > >> wr

Re: [PATCH 1/8 v6] tpm: refactor function names and macros for infineon v1.2 TPM

2021-11-08 Thread Simon Glass
On Sun, 7 Nov 2021 at 14:33, Ilias Apalodimas wrote: > > With the upcoming TPM2 API, some of the function names are part of the new > header file. So switch conflicting driver defined function names and > defines. > > Signed-off-by: Ilias Apalodimas > --- > drivers/tpm/tpm_tis_infineon.c | 34 +

Re: [PATCH V2 09/17] arm: j721e: Add support for selecting DT based on EEPROM

2021-11-08 Thread Tom Rini
On Tue, Nov 02, 2021 at 07:35:50PM +0530, Sinthu Raja wrote: > From: Sinthu Raja > > Enable support for selecting DTB from FIT within SPL based on the > board name read from EEPROM. This will help to use single defconfig > for both EVM and SK. > > Also print FDT model name and board name read f

Re: [PATCH V2 07/17] board: ti: j721e: Disable probing of daughtercards

2021-11-08 Thread Tom Rini
On Tue, Nov 02, 2021 at 07:35:48PM +0530, Sinthu Raja wrote: > From: Sinthu Raja > > j721e-sk doesn't have any daughter cards, so disable daughter > card probing inside board_late_init() and spl_board_init() for > j721e-sk. > > Signed-off-by: Sinthu Raja > --- > > Changes in V2: > * J721E EAIK

Re: [PATCH V2 05/17] board: ti: j721e: Enable support for reading EEPROM at next alternate address

2021-11-08 Thread Tom Rini
On Tue, Nov 02, 2021 at 07:35:46PM +0530, Sinthu Raja wrote: > From: Sinthu Raja > > J721E EVM has EEPROM populated at 0x50. J721E SK has EEPROM populated > at next address 0x51 in order to be compatible with RBPi. So start > looking for TI specific EEPROM at 0x50, if not found look for EEPROM a

Re: [PATCH 1/8 v6] tpm: refactor function names and macros for infineon v1.2 TPM

2021-11-08 Thread Heinrich Schuchardt
On 11/7/21 22:33, Ilias Apalodimas wrote: With the upcoming TPM2 API, some of the function names are part of the new header file. So switch conflicting driver defined function names and defines. Signed-off-by: Ilias Apalodimas --- drivers/tpm/tpm_tis_infineon.c | 34 +-

Re: [PATCH V2 04/17] board: ti: j721e: Guard functions with right #ifdef to avoid build warnings

2021-11-08 Thread Tom Rini
On Tue, Nov 02, 2021 at 07:35:45PM +0530, Sinthu Raja wrote: > From: Sinthu Raja > > board_late_init(), setup_board_eeprom_env() and setup_serial() is > called only under CONFIG_BOARD_LATE_INIT, so guard these functions > with the same. > > Signed-off-by: Sinthu Raja > --- > > No chnages in V

Re: [PATCH V2 01/17] common: make: Support show_board_info() as part of SPL build

2021-11-08 Thread Tom Rini
On Tue, Nov 02, 2021 at 07:35:42PM +0530, Sinthu Raja wrote: > From: Sinthu Raja > > Enable support for show_board_info() in SPL build. > > Signed-off-by: Sinthu Raja OK, but why do we need this, and won't it cause every platform that has this defined to now grow in size? -- Tom signature

Re: [PATCH V2 11/17] arm: dts: k3-j721e-sk: Add initial A72 specific dts support

2021-11-08 Thread Tom Rini
On Tue, Nov 02, 2021 at 07:35:52PM +0530, Sinthu Raja wrote: > From: Sinthu Raja > > J721E Starter Kit (SK)[1] is a low cost, small form factor board designed > for TI’s J721E SoC. TI’s J721E SoC comprises of dual core A72, high > performance vision accelerators, video codec accelerators, latest

Re: [PATCH V2 12/17] arm: dts: k3-j721e-r5-sk: Add initial R5 specific dts support for j721e-sk

2021-11-08 Thread Tom Rini
On Tue, Nov 02, 2021 at 07:35:53PM +0530, Sinthu Raja wrote: > From: Sinthu Raja > > Add R5 specific dts for J721E-SK > > Signed-off-by: Sinthu Raja > --- > > Changes in V2: > * J721E EAIK board name is changed to J721E SK, rename all occurrences of > eaik > to sk. > * Rename file from eaik

Re: [PATCH V2 10/17] arm: dts: k3-j721e-r5-common-proc-board: Do not use power-domains for I2C

2021-11-08 Thread Tom Rini
On Tue, Nov 02, 2021 at 07:35:51PM +0530, Sinthu Raja wrote: > From: Sinthu Raja > > Board ID I2C EEPROM will be probed before SYSFW is available. > So drop the power-domains property for wakup_i2c0 on which > board ID EEPROM is connected. > > Signed-off-by: Sinthu Raja > --- > > No Changes in

RE: [PATCH] introduce CONFIG_DEVICE_TREE_INCLUDES

2021-11-08 Thread Roman Kopytin
Hi, I sent patches, please check. But before correct emails I sent several test emails. -Original Message- From: Simon Glass Sent: Friday, November 5, 2021 5:02 AM To: Roman Kopytin Cc: Rasmus Villemoes ; U-Boot Mailing List ; Alex Kiernan ; Masahiro Yamada Subject: Re: [PATCH] intro

Re: [PATCH v4 09/34] efi: Add EFI uclass for media

2021-11-08 Thread Heinrich Schuchardt
On 11/8/21 06:29, AKASHI Takahiro wrote: On Sun, Nov 07, 2021 at 09:43:22AM -0700, Simon Glass wrote: Hi Heinrich, On Sun, 7 Nov 2021 at 01:21, Heinrich Schuchardt wrote: On 11/4/21 04:09, Simon Glass wrote: At present UCLASS_EFI is used to represent an EFI filesystem among other thing

Re: [PATCH 13/20] # This is a combination of 2 commits. # This is the 1st commit message:

2021-11-08 Thread Tom Rini
On Sun, Nov 07, 2021 at 10:59:44PM -0500, Tom Rini wrote: > Convert CONFIG_SYS_PROMPT et al to Kconfig > > This converts the following to Kconfig: >CONFIG_SYS_PROMPT >CONFIG_SYS_PROMPT_HUSH_PS2 > > Signed-off-by: Tom Rini I clearly forgot to remove the very first two lines when I merge

Testing Tools

2021-11-08 Thread vish chopra
Hi All, I'm new to U-boot. What are all testing tools available in Uboot to check 1. MMC 2. DDR 3. ethernet and how to use tools if available?

[PATCH 2/2] test_vboot.py: include test of fdt_add_pubkey tool

2021-11-08 Thread Roman Kopytin
Signed-off-by: Roman Kopytin Cc: Rasmus Villemoes --- test/py/tests/test_vboot.py | 8 1 file changed, 8 insertions(+) diff --git a/test/py/tests/test_vboot.py b/test/py/tests/test_vboot.py index 6dff6779d1..cf7416b39a 100644 --- a/test/py/tests/test_vboot.py +++ b/test/py/tests/test_v

[PATCH 1/2] tools: add fdt_add_pubkey

2021-11-08 Thread Roman Kopytin
Having to use the -K option to mkimage to populate U-Boot's .dtb with the public key while signing the kernel FIT image is often a little awkward. In particular, when using a meta-build system such as bitbake/Yocto, having the tasks of the kernel and U-Boot recipes intertwined, modifying deployed a

[PATCH 0/2] RFC: add fdt_add_pubkey tool

2021-11-08 Thread Roman Kopytin
In order to reduce the coupling between building the kernel and U-Boot, I'd like a tool that can add a public key to U-Boot's dtb without simultaneously signing a FIT image. That tool doesn't seem to exist, so I stole the necessary pieces from mkimage et al and put it in a single .c file. I'm stil

[PATCH 1/2] tools: add fdt_add_pubkey

2021-11-08 Thread Roman Kopytin
Having to use the -K option to mkimage to populate U-Boot's .dtb with the public key while signing the kernel FIT image is often a little awkward. In particular, when using a meta-build system such as bitbake/Yocto, having the tasks of the kernel and U-Boot recipes intertwined, modifying deployed a

  1   2   >