Re: [PATCH v2 0/4] FMP versioning support

2023-03-03 Thread Takahiro Akashi
Kojima-san, On Wed, Mar 01, 2023 at 06:15:18PM +0900, Masahisa Kojima wrote: > Firmware version management is not implemented in the current > FMP implementation. This series aims to add the versioning support > in FMP. I think that you need to think of A/B update case, especially when a capsule

Re: [PATCH v5 44/44] command: Don't allow commands in SPL

2023-03-03 Thread Tom Rini
On Wed, Feb 22, 2023 at 09:34:25AM -0700, Simon Glass wrote: > At present we compile commands into U-Boot SPL even though they cannot > be used. This wastes space. Adjust the condition to avoid this. > > Signed-off-by: Simon Glass Applied to u-boot/next, thanks! -- Tom signature.asc

Re: [PATCH v5 43/44] venice: Simplify conditions for network init

2023-03-03 Thread Tom Rini
On Wed, Feb 22, 2023 at 09:34:24AM -0700, Simon Glass wrote: > The conditions in this code do not align when doing an SPL build with > split config. Use __maybe_unused to avoid needing to be so explicit. > > Of course a better solution would be to refactor all of this to avoid > using #ifdef. >

Re: [PATCH v5 42/44] power: wandboard: Add a missing CONFIG

2023-03-03 Thread Tom Rini
On Wed, Feb 22, 2023 at 09:34:23AM -0700, Simon Glass wrote: > We should enable pmic in SPL since it is used. > > Signed-off-by: Simon Glass Applied to u-boot/next, thanks! -- Tom signature.asc Description: PGP signature

Re: [PATCH v5 41/44] x86: coral: Add missing TPL options

2023-03-03 Thread Tom Rini
On Wed, Feb 22, 2023 at 09:34:22AM -0700, Simon Glass wrote: > Some options should be enabled which are missing. Fix this. > > Signed-off-by: Simon Glass Applied to u-boot/next, thanks! -- Tom signature.asc Description: PGP signature

Re: [PATCH v5 40/44] dm: Add a TPL symbol for simple-bus

2023-03-03 Thread Tom Rini
On Wed, Feb 22, 2023 at 09:34:21AM -0700, Simon Glass wrote: > This is used in some x86 code, so add a symbol for it. > > Signed-off-by: Simon Glass Applied to u-boot/next, thanks! -- Tom signature.asc Description: PGP signature

Re: [PATCH v5 39/44] serial: Support ns16550 driver in TPL

2023-03-03 Thread Tom Rini
On Wed, Feb 22, 2023 at 09:34:20AM -0700, Simon Glass wrote: > Add options for this since they are needed by P1020RDB-PC_NAND. > > Signed-off-by: Simon Glass Applied to u-boot/next, thanks! -- Tom signature.asc Description: PGP signature

Re: [PATCH v5 37/44] net: Add an SPL config for atheros

2023-03-03 Thread Tom Rini
On Wed, Feb 22, 2023 at 09:34:18AM -0700, Simon Glass wrote: > Add a new SPL_PHY_ATHEROS to avoid a build error on am335x_evm with split > config. > > Signed-off-by: Simon Glass > Reviewed-by: Ramon Fried Applied to u-boot/next, thanks! -- Tom signature.asc Description: PGP signature

Re: [PATCH v5 36/44] imx: Use SATA instead of CMD_SATA

2023-03-03 Thread Tom Rini
On Wed, Feb 22, 2023 at 09:34:17AM -0700, Simon Glass wrote: > This causes a build failure on mx6cuboxi with split config, since CMD_SATA > shows up as enabled in SPl (because there is no SPL_CMD_SATA). > > The condition is wrong anyway, so change it to use SATA instead. > > Signed-off-by:

Re: [PATCH v5 35/44] freescale: Drop old pre-DM_ETH code

2023-03-03 Thread Tom Rini
On Wed, Feb 22, 2023 at 09:34:16AM -0700, Simon Glass wrote: > This is used by ls1021atwr_sdcard_ifc_SECURE_BOOT with split config, but > is not needed anymore, since Ethernet migration is complete. Drop it. > > Signed-off-by: Simon Glass Applied to u-boot/next, thanks! -- Tom

Re: [PATCH v5 33/44] Add VPL options for BLOBLIST

2023-03-03 Thread Tom Rini
On Wed, Feb 22, 2023 at 09:34:14AM -0700, Simon Glass wrote: > We can use this feature in VPL, so add some options for it. Also fix a > typo in the SPL help while we are here. > > Signed-off-by: Simon Glass Applied to u-boot/next, thanks! -- Tom signature.asc Description: PGP signature

Re: [PATCH v5 31/44] test: Tidy up sandbox handling in test-main

2023-03-03 Thread Tom Rini
On Wed, Feb 22, 2023 at 09:34:12AM -0700, Simon Glass wrote: > This is pretty messy at present since it relies on a SPL_SANDBOX option > that does not exist. Use the normal options instead, so that it will work > with split config. > > Signed-off-by: Simon Glass Applied to u-boot/next, thanks!

Re: [PATCH v5 30/44] lib: Add an SPL config for LIB_UUID

2023-03-03 Thread Tom Rini
On Wed, Feb 22, 2023 at 09:34:11AM -0700, Simon Glass wrote: > This is selected by PARTITION_UUIDS which has a separate option for SPL. > Add an SPL option for LIB_UUID also, so that we can keep them consistent. > > Also add one for PARTITION_TYPE_GUID to avoid a build error in part_efi.c >

Re: [PATCH v5 24/44] sandbox: Use the generic VPL option to enable VPL

2023-03-03 Thread Tom Rini
On Wed, Feb 22, 2023 at 09:34:05AM -0700, Simon Glass wrote: > Avoid using CONFIG_SANDBOX_VPL since we have a generic option which works > just as well. > > Signed-off-by: Simon Glass Applied to u-boot/next, thanks! -- Tom signature.asc Description: PGP signature

Re: [PATCH v5 25/44] sandbox: Tidy up I2C options

2023-03-03 Thread Tom Rini
On Wed, Feb 22, 2023 at 09:34:06AM -0700, Simon Glass wrote: > At present we enable the sandbox I2C driver for all builds. Add a separate > Kconfig option to control this, so that it can be disabled in TPL, where > it is not needed. > > Signed-off-by: Simon Glass > Reviewed-by: Heiko Schocher

Re: [PATCH v5 23/44] sandbox: Tidy up RTC options

2023-03-03 Thread Tom Rini
On Wed, Feb 22, 2023 at 09:34:04AM -0700, Simon Glass wrote: > At present we enable the sandbox RTC driver for all builds. Add a separate > Kconfig option to control this, so that it can be disabled in TPL, where > it is not needed. > > Signed-off-by: Simon Glass Applied to u-boot/next,

Re: [PATCH v5 22/44] lib: Fix build condition for tiny-printf

2023-03-03 Thread Tom Rini
On Wed, Feb 22, 2023 at 09:34:03AM -0700, Simon Glass wrote: > This should be checking for any SPL build. Drop the use of SPL_TPL_ since > it is not necessary and will not work with split config. > > Signed-off-by: Simon Glass Applied to u-boot/next, thanks! -- Tom signature.asc

Re: [PATCH v5 20/44] lib: Add VPL options for SHA1 and SHA256

2023-03-03 Thread Tom Rini
On Wed, Feb 22, 2023 at 09:34:01AM -0700, Simon Glass wrote: > Add these options so these algorithms can be used in VPL. > > Signed-off-by: Simon Glass Applied to u-boot/next, thanks! -- Tom signature.asc Description: PGP signature

Re: [PATCH v5 19/44] env: Allow VPL environment to be nowhere

2023-03-03 Thread Tom Rini
On Wed, Feb 22, 2023 at 09:34:00AM -0700, Simon Glass wrote: > Add an option to put the VPL environment nowhere (not in storage). > > Signed-off-by: Simon Glass Applied to u-boot/next, thanks! -- Tom signature.asc Description: PGP signature

Re: [PATCH v5 17/44] bootstd: Correct 'VPL' typo

2023-03-03 Thread Tom Rini
On Wed, Feb 22, 2023 at 09:33:58AM -0700, Simon Glass wrote: > Correct a 'VPL' typo in the Kconfig. > > Signed-off-by: Simon Glass Applied to u-boot/next, thanks! -- Tom signature.asc Description: PGP signature

Re: [PATCH v5 16/44] event: Add Kconfig options for SPL

2023-03-03 Thread Tom Rini
On Wed, Feb 22, 2023 at 09:33:57AM -0700, Simon Glass wrote: > Add options to enable events in SPL. This is mostly so the code can be > excluded from SPL builds. > > Signed-off-by: Simon Glass Applied to u-boot/next, thanks! -- Tom signature.asc Description: PGP signature

Re: [PATCH v5 15/44] sandbox: Expand size for VPL image

2023-03-03 Thread Tom Rini
On Wed, Feb 22, 2023 at 09:33:56AM -0700, Simon Glass wrote: > Allow this to get larger to accommodate more test code with LTO disabled. > > Signed-off-by: Simon Glass Applied to u-boot/next, thanks! -- Tom signature.asc Description: PGP signature

Re: [PATCH v5 13/44] lib: Add a Kconfig for SPL_BZIP2

2023-03-03 Thread Tom Rini
On Wed, Feb 22, 2023 at 09:33:54AM -0700, Simon Glass wrote: > This is implicitly used in the source and seems useful, so add it. > > Signed-off-by: Simon Glass Applied to u-boot/next, thanks! -- Tom signature.asc Description: PGP signature

Re: [PATCH v5 12/44] Correct SPL use of DM_RNG

2023-03-03 Thread Tom Rini
On Wed, Feb 22, 2023 at 09:33:53AM -0700, Simon Glass wrote: > This converts 1 usage of this option to the non-SPL form, since there is > no SPL_DM_RNG defined in Kconfig > > Signed-off-by: Simon Glass Applied to u-boot/next, thanks! -- Tom signature.asc Description: PGP signature

Re: [PATCH v5 11/44] boot: Add Kconfigs for BOOTMETH_VBE_REQUEST

2023-03-03 Thread Tom Rini
On Wed, Feb 22, 2023 at 09:33:52AM -0700, Simon Glass wrote: > Allow this to be enabled separately in U-Boot proper and in SPL, since > it is not needed in SPL. > > Signed-off-by: Simon Glass Applied to u-boot/next, thanks! -- Tom signature.asc Description: PGP signature

Re: [PATCH v5 10/44] Correct SPL uses of PHY_FIXED

2023-03-03 Thread Tom Rini
On Wed, Feb 22, 2023 at 09:33:51AM -0700, Simon Glass wrote: > This converts 3 usages of this option to the non-SPL form, since there is > no SPL_PHY_FIXED defined in Kconfig > > Signed-off-by: Simon Glass Applied to u-boot/next, thanks! -- Tom signature.asc Description: PGP signature

Re: [PATCH v5 09/44] Correct SPL use of PG_WCOM_UBOOT_UPDATE_SUPPORTED

2023-03-03 Thread Tom Rini
On Wed, Feb 22, 2023 at 09:33:50AM -0700, Simon Glass wrote: > This converts 1 usage of this option to the non-SPL form, since there is > no SPL_PG_WCOM_UBOOT_UPDATE_SUPPORTED defined in Kconfig > > Signed-off-by: Simon Glass > Aleksandar Gerasimovski Applied to u-boot/next, thanks! -- Tom

Re: [PATCH v5 08/44] Correct SPL uses of MULTIPLEXER

2023-03-03 Thread Tom Rini
On Wed, Feb 22, 2023 at 09:33:49AM -0700, Simon Glass wrote: > This converts 3 usages of this option to the non-SPL form, since there is > no SPL_MULTIPLEXER defined in Kconfig > > Signed-off-by: Simon Glass > Reviewed-by: Tom Rini Applied to u-boot/next, thanks! -- Tom signature.asc

Re: [PATCH v5 07/44] Correct SPL uses of DISPLAY_AER_FULL

2023-03-03 Thread Tom Rini
On Wed, Feb 22, 2023 at 09:33:48AM -0700, Simon Glass wrote: > This converts 2 usages of this option to the non-SPL form, since there is > no SPL_DISPLAY_AER_FULL defined in Kconfig > > Signed-off-by: Simon Glass Applied to u-boot/next, thanks! -- Tom signature.asc Description: PGP

Re: [PATCH v5 05/44] bootstd: Disable QFW bootmeth in SPL

2023-03-03 Thread Tom Rini
On Wed, Feb 22, 2023 at 09:33:46AM -0700, Simon Glass wrote: > Move this Makefile line into the non-SPL area so we don't have to repy > on the SPL_TPL_ macro. > > Signed-off-by: Simon Glass Applied to u-boot/next, thanks! -- Tom signature.asc Description: PGP signature

Re: [PATCH v5 03/44] sh4: Drop unused twl6030 driver

2023-03-03 Thread Tom Rini
On Wed, Feb 22, 2023 at 09:33:44AM -0700, Simon Glass wrote: > This is not used. Drop the driver and Kconfig option. > > Signed-off-by: Simon Glass Applied to u-boot/next, thanks! -- Tom signature.asc Description: PGP signature

Re: [PATCH v5 02/44] mtd: Drop unused CONFIG_ONENAND_U_BOOT

2023-03-03 Thread Tom Rini
On Wed, Feb 22, 2023 at 09:33:43AM -0700, Simon Glass wrote: > This option does not exist, so the Makefile rule does nothing. Drop it. > > Signed-off-by: Simon Glass Applied to u-boot/next, thanks! -- Tom signature.asc Description: PGP signature

Re: [PATCH v5 01/44] mtd: Drop unused kb9202_nand driver

2023-03-03 Thread Tom Rini
On Wed, Feb 22, 2023 at 09:33:42AM -0700, Simon Glass wrote: > This is not used since time out of mind. > > Drop the driver and Kconfig option. > > Signed-off-by: Simon Glass > Acked-by: Michael Trimarchi Applied to u-boot/next, thanks! -- Tom signature.asc Description: PGP signature

Re: [PATCH 09/13] s5p: Remove empty arch_misc_init

2023-03-03 Thread Tom Rini
On Wed, Feb 15, 2023 at 10:36:55PM -0500, Tom Rini wrote: > We don't need to provide an empty arch_misc_init function here, we can > just not enable the hook. > > Cc: Stefan Bosch > Signed-off-by: Tom Rini > Reviewed-by: Simon Glass > Reviewed-by: Minkyu Kang Applied to u-boot/next, thanks!

Re: [PATCH 08/13] mvebu: Drop empty arch_misc_init

2023-03-03 Thread Tom Rini
On Wed, Feb 15, 2023 at 10:36:54PM -0500, Tom Rini wrote: > If this hooks is needed later, it should be added and populated for > real. > > Cc: Stefan Roese > Signed-off-by: Tom Rini > Reviewed-by: Stefan Roese Applied to u-boot/next, thanks! -- Tom signature.asc Description: PGP

Re: [PATCH 07/13] mvebe: Drop ARCH_MISC_INIT from alleycat 5

2023-03-03 Thread Tom Rini
On Wed, Feb 15, 2023 at 10:36:53PM -0500, Tom Rini wrote: > In this platform, arch_misc_init doesn't perform any real function. The > call to get_soc_type_rev has no lasting side effects. > > Cc: Chris Packham > Signed-off-by: Tom Rini > Reviewed-by: Simon Glass > Reviewed-by: Chris Packham

Re: [PATCH 06/13] imx9: Remove ARCH_MISC_INIT

2023-03-03 Thread Tom Rini
On Wed, Feb 15, 2023 at 10:36:52PM -0500, Tom Rini wrote: > We don't need an empty function, we can just not enable the hook we > don't use. > > Cc: Peng Fan > Signed-off-by: Tom Rini > Reviewed-by: Simon Glass > Reviewed-by: Peng Fan Applied to u-boot/next, thanks! -- Tom signature.asc

Re: [PATCH 05/13] common: Make ARCH_EARLY_INIT_R be selected only

2023-03-03 Thread Tom Rini
On Wed, Feb 15, 2023 at 10:36:51PM -0500, Tom Rini wrote: > As platforms which require this hook need this hook enabled, in order to > function, or do not need this hook, it doesn't make sense to prompt the > user. As all platforms that need this hook now select the symbol, remove > the prompt

Re: [PATCH 04/13] arm: rk3368: Select ARCH_EARLY_INIT_R when used

2023-03-03 Thread Tom Rini
On Wed, Feb 15, 2023 at 10:36:50PM -0500, Tom Rini wrote: > On the lion and evb-px5 platforms, we need this function, so select it. > > Cc: Andy Yan > Cc: Quentin Schulz > Cc: Klaus Goger > Signed-off-by: Tom Rini > Reviewed-by: Simon Glass > Reviewed-by: Quentin Schulz Applied to

Re: [PATCH 03/13] arm: zynq: Move to select'ing ARCH_EARLY_INIT_R if we have FPGA

2023-03-03 Thread Tom Rini
On Wed, Feb 15, 2023 at 10:36:49PM -0500, Tom Rini wrote: > The function arch_early_init_r only does anything on these platforms if > we have FPGA (or SPL and SPL_FPGA) enabled, so move the logic to select > based on that. > > Cc: Michal Simek > Signed-off-by: Tom Rini > Reviewed-by: Michal

Re: [PATCH 02/13] arm: mvebu: Add select on ARCH_EARLY_INIT_R if ARM64

2023-03-03 Thread Tom Rini
On Wed, Feb 15, 2023 at 10:36:48PM -0500, Tom Rini wrote: > We need to be calling arch_early_init_r() on 64bit mvebu platforms, so > move this to a select. > > Cc: Stefan Roese > Signed-off-by: Tom Rini > Reviewed-by: Stefan Roese Applied to u-boot/next, thanks! -- Tom signature.asc

Re: [PATCH 01/13] common/Kconfig: Reword text for BOARD_TYPES

2023-03-03 Thread Tom Rini
On Wed, Feb 15, 2023 at 10:36:47PM -0500, Tom Rini wrote: > While it is true that for some Samsung platforms, we call > get_board_type() the main usage of this CONFIG switch is to enable > board_types in global data, which is then used by various platforms. > > Signed-off-by: Tom Rini >

Re: [PULL] u-boot-usb/next

2023-03-03 Thread Tom Rini
On Fri, Mar 03, 2023 at 01:05:10AM +0100, Marek Vasut wrote: > The following changes since commit 49cba67852f1fb5ef481bef3532b2cda96816e45: > > Merge branch 'next' of > https://source.denx.de/u-boot/custodians/u-boot-marvell into next > (2023-03-01 11:00:22 -0500) > > are available in the Git

[PATCH 1/1] api: move API related config options into submenu

2023-03-03 Thread Heinrich Schuchardt
Kconfig settings that are related to the API for standalone applications should be in the API sub-menu and not on the top level. CONFIG_STANDALONE_LOAD_ADDR is only relevant if standalone example applications are built. Signed-off-by: Heinrich Schuchardt --- Kconfig | 8

[PATCH 1/1] doc: man-page for panic command

2023-03-03 Thread Heinrich Schuchardt
Provide a man-page for the panic command. Signed-off-by: Heinrich Schuchardt --- doc/usage/cmd/panic.rst | 33 + doc/usage/index.rst | 1 + 2 files changed, 34 insertions(+) create mode 100644 doc/usage/cmd/panic.rst diff --git a/doc/usage/cmd/panic.rst

Re: [PATCH v3 03/17] dt-bindings: reset: Add StarFive JH7110 reset definitions

2023-03-03 Thread Conor Dooley
Hey Yanhong, In the future, if you forget to send a patch as part of the series itself, please make sure to send it as a reply to the original series. Since you have the message-id's from your previous invocation of git send-email, or can trivially find them in your mail client (or somewhere

Re: [PATCH v3 16/17] riscv: dts: jh7110: Add initial StarFive VisionFive v2 board device tree

2023-03-03 Thread Conor Dooley
On Fri, Mar 03, 2023 at 11:24:31AM +0800, Yanhong Wang wrote: > Add initial device tree for StarFive VisionFive v2 board. > > Signed-off-by: Yanhong Wang > --- > arch/riscv/dts/Makefile | 3 +- > ...10-starfive-visionfive-2-v1.2a-u-boot.dtsi | 85 ++ >

Re: [PATCH v3 14/17] riscv: dts: jh7110: Add initial StarFive JH7110 device tree

2023-03-03 Thread Conor Dooley
On Fri, Mar 03, 2023 at 11:24:29AM +0800, Yanhong Wang wrote: > Add initial device tree for the JH7110 RISC-V SoC. > > Signed-off-by: Yanhong Wang > --- > arch/riscv/dts/jh7110.dtsi | 582 + > 1 file changed, 582 insertions(+) > create mode 100644

Re: [PATCH v3 00/17] Basic StarFive JH7110 RISC-V SoC support

2023-03-03 Thread Conor Dooley
Hey Yanhong, On Fri, Mar 03, 2023 at 11:24:15AM +0800, Yanhong Wang wrote: > This series of patches base on the latest branch/master, and add support > for the StarFive JH7110 RISC-V SoC and VisionFive V2 board. In order for > this to be achieved, the respective DT nodes have been added, and the

Re: [PATCH v3 02/17] cache: starfive: Add StarFive JH7110 support

2023-03-03 Thread Conor Dooley
On Fri, Mar 03, 2023 at 11:24:17AM +0800, Yanhong Wang wrote: > This adds support for the StarFive JH7110 SoC which also > feature this SiFive cache controller. > > Signed-off-by: Yanhong Wang > --- > drivers/cache/cache-sifive-ccache.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git

[PATCH 1/1] efi_loader: describe term_get_char()

2023-03-03 Thread Heinrich Schuchardt
Add a function description. Signed-off-by: Heinrich Schuchardt --- lib/efi_loader/efi_console.c | 8 1 file changed, 8 insertions(+) diff --git a/lib/efi_loader/efi_console.c b/lib/efi_loader/efi_console.c index 4317630907..d970b667a6 100644 --- a/lib/efi_loader/efi_console.c +++

[PATCH 4/4] net: e1000: make use of multiple rx packets

2023-03-03 Thread Christian Gmeiner
When facing a very busy network, the single rx packet buffer was oftentimes overwritten before a desired response packet (e.g. a Ping reply) could have been processed. This change improves resistance to this by utilising multiple buffers. Signed-off-by: Christian Gmeiner --- drivers/net/e1000.c

[PATCH 3/4] net: e1000: dynamically allocate rx data buffers per instance

2023-03-03 Thread Christian Gmeiner
Signed-off-by: Christian Gmeiner --- drivers/net/e1000.c | 25 + 1 file changed, 17 insertions(+), 8 deletions(-) diff --git a/drivers/net/e1000.c b/drivers/net/e1000.c index 08c84ce3d1..ea9ca76917 100644 --- a/drivers/net/e1000.c +++ b/drivers/net/e1000.c @@ -54,14

[PATCH 2/4] net: e1000: move all rx data structures to hw instance

2023-03-03 Thread Christian Gmeiner
Preparation for per driver instance allocated data structures. Signed-off-by: Christian Gmeiner --- drivers/net/e1000.c | 26 +++--- drivers/net/e1000.h | 5 + 2 files changed, 20 insertions(+), 11 deletions(-) diff --git a/drivers/net/e1000.c b/drivers/net/e1000.c

[PATCH 1/4] net: e1000: add and make use of NUM_RX_DESC macro

2023-03-03 Thread Christian Gmeiner
The call to DEFINE_ALIGN_BUFFER for the rx_desc array conained an icnonsistency as 16 receive descriptors were allocated when the remaining code would only use 8 of them. Signed-off-by: Christian Gmeiner --- drivers/net/e1000.c | 6 +++--- drivers/net/e1000.h | 2 ++ 2 files changed, 5

Re: [PATCH v8 6/6] doc: Add measured boot documentation

2023-03-03 Thread Heinrich Schuchardt
On 3/3/23 20:25, Eddie James wrote: Briefly describe the feature and specify the requirements. Signed-off-by: Eddie James Reviewed-by: Simon Glass --- doc/usage/index.rst | 1 + doc/usage/measured_boot.rst | 23 +++ 2 files changed, 24 insertions(+) create

Re: [PATCH v8 4/6] bootm: Support boot measurement

2023-03-03 Thread Heinrich Schuchardt
On 3/3/23 20:25, Eddie James wrote: Add a configuration option to measure the boot through the bootm function. Add the measurement state to the booti and bootz paths as well. Signed-off-by: Eddie James Reviewed-by: Simon Glass --- Changes since v6: - Added comment for bootm_measure -

Re: [PATCH v3 1/1] editorconfig: introduce .editorconfig

2023-03-03 Thread Heinrich Schuchardt
On 3/3/23 16:33, Dzmitry Sankouski wrote: Current process of sending patches includes running checkpatch.pl script for each patch, and fixing found style problems. EditorConfig may help to prevent some style related problems (like spaces vs tab indentation) on the fly. Reviewed-by: Simon Glass

[PATCH v8 5/6] test: Add sandbox TPM boot measurement

2023-03-03 Thread Eddie James
Use the sandbox TPM driver to measure some boot images in a unit test case. Signed-off-by: Eddie James Reviewed-by: Simon Glass Acked-by: Ilias Apalodimas --- Changes since v5: - Only compile in the measurement u-boot command when CONFIG_MEASURED_BOOT is enabled

[PATCH v8 3/6] tpm: Support boot measurements

2023-03-03 Thread Eddie James
Add TPM2 functions to support boot measurement. This includes starting up the TPM, initializing/appending the event log, and measuring the U-Boot version. Much of the code was used in the EFI subsystem, so remove it there and use the common functions. Signed-off-by: Eddie James --- Changes since

[PATCH v8 6/6] doc: Add measured boot documentation

2023-03-03 Thread Eddie James
Briefly describe the feature and specify the requirements. Signed-off-by: Eddie James Reviewed-by: Simon Glass --- doc/usage/index.rst | 1 + doc/usage/measured_boot.rst | 23 +++ 2 files changed, 24 insertions(+) create mode 100644 doc/usage/measured_boot.rst

[PATCH v8 4/6] bootm: Support boot measurement

2023-03-03 Thread Eddie James
Add a configuration option to measure the boot through the bootm function. Add the measurement state to the booti and bootz paths as well. Signed-off-by: Eddie James Reviewed-by: Simon Glass --- Changes since v6: - Added comment for bootm_measure - Fixed line length in bootm_measure

[PATCH v8 2/6] tpm: sandbox: Update for needed TPM2 capabilities

2023-03-03 Thread Eddie James
The driver needs to support getting the PCRs in the capabilities command. Fix various other things and support the max number of PCRs for TPM2. Remove the !SANDBOX dependency for EFI TCG2 as well. Signed-off-by: Eddie James Reviewed-by: Simon Glass Acked-by: Ilias Apalodimas --- Changes since

[PATCH v8 0/6] tpm: Support boot measurements

2023-03-03 Thread Eddie James
This series adds support for measuring the boot images more generically than the existing EFI support. Several EFI functions have been moved to the TPM layer. The series includes optional measurement from the bootm command. A new test case has been added for the bootm measurement to test the new

[PATCH v8 1/6] tpm: Fix spelling for tpmu_ha union

2023-03-03 Thread Eddie James
tmpu -> tpmu Signed-off-by: Eddie James Reviewed-by: Ilias Apalodimas --- include/tpm-v2.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/include/tpm-v2.h b/include/tpm-v2.h index 2b6980e441..6684033deb 100644 --- a/include/tpm-v2.h +++ b/include/tpm-v2.h @@ -169,7

Re: [PATCH v7 3/6] tpm: Support boot measurements

2023-03-03 Thread Eddie James
On 3/2/23 14:22, Ilias Apalodimas wrote: Hi Eddie, I found the issue. I still think we could squeeze things even more in our abstraction. Specifically the measure_event() tcg2_agile_log_append() contain some efi specific bits and I am trying to figure out if we can make those more generic.

[PATCH] arm: mach-k3: introduce generic board detction kconfig option

2023-03-03 Thread Christian Gmeiner
For non TI boards it is not possible to enable the do_board_detect() call as TI_I2C_BOARD_DETECT is defined in board/ti/common/Kconfig. I want to use do_board_detect() to dectect boards and properties based on some SPI communication with a FPGA. Signed-off-by: Christian Gmeiner ---

can't reproduce XHCI hang in Rock Pi 4

2023-03-03 Thread Xavier Drudis Ferran
I'm sorry but I haven't been able to reproduce your issue. El Fri, Mar 03, 2023 at 11:26:46AM +0100, Xavier Drudis Ferran deia: > El Fri, Mar 03, 2023 at 10:42:20AM +0100, Christoph Fritz deia: > > Unfortunately I don't think I have any usb-ethernet dongle here to test... > [...] > > => usb

Re: [PATCH v3 3/4] Kconfig: j721e: Change K3_MCU_SCRATCHPAD_BASE to non firewalled region

2023-03-03 Thread Tom Rini
On Fri, Mar 03, 2023 at 03:59:35PM +0530, Manorit Chawdhry wrote: > In non-combined boot flow for K3, all the firewalls are locked by default > until sysfw comes up. Rom configures some of the firewall for its usage > along with the SRAM for R5 but the PSRAM region is still locked. > > The K3

[PATCH v3 1/1] editorconfig: introduce .editorconfig

2023-03-03 Thread Dzmitry Sankouski
Current process of sending patches includes running checkpatch.pl script for each patch, and fixing found style problems. EditorConfig may help to prevent some style related problems (like spaces vs tab indentation) on the fly. Reviewed-by: Simon Glass Signed-off-by: Dzmitry Sankouski ---

[PATCH v3 0/1] introduce .editorconfig

2023-03-03 Thread Dzmitry Sankouski
Current process of sending patches includes running checkpatch.pl script for each patch, and fixing found style problems. Editorconfig may help to prevent some style related problems (like spaces vs tab indentation) on the fly. Changes in v3: - add 'the' article in docs - fix spacing - add sign

Re: [PATCH v5 32/44] x86: Fix up use of X86_32BIT_INIT and X86_64 options

2023-03-03 Thread Tom Rini
On Wed, Feb 22, 2023 at 09:34:13AM -0700, Simon Glass wrote: > Drop the invalid SPL_ in a CONFIG_IS_ENABLED() usage. Use the correct > X86_64 option in msr.h since SPL may be 32-bit when U-Bout proper is > not. > > Signed-off-by: Simon Glass > --- > > (no changes since v4) > > Changes in v4:

Re: Question regarding U-boot MultiCore SMP

2023-03-03 Thread Sean Anderson
On 3/2/23 05:04, Jan Reinhard wrote: Hello, I am working on the PolarFire RISC-V icicle kit and use u-boot to start my application. I configured the firmware to start u-boot on all harts (cores) and found out that u-boot uses a "HART lottery system" to decide which core/hart it runs on. In

[PATCH v1 17/17] colibri-imx8x: enable fuse command

2023-03-03 Thread Andrejs Cainikovs
This command is required for initial SoC provisioning. Signed-off-by: Andrejs Cainikovs --- configs/colibri-imx8x_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/configs/colibri-imx8x_defconfig b/configs/colibri-imx8x_defconfig index 8b4592020d5..43aa4230ff9 100644 ---

[PATCH v1 16/17] Revert "imx: imx8x: colibri: switch to binman"

2023-03-03 Thread Andrejs Cainikovs
This reverts commit bdadc140a127b14a666d2007eddc3f65c8de7d5a. We do not want this, see [1]. [1] https://lore.kernel.org/all/56cf058164f331ce99ce75b0751b825ee2e07fc0.ca...@toradex.com/ Signed-off-by: Andrejs Cainikovs --- arch/arm/dts/fsl-imx8qxp-colibri-u-boot.dtsi | 2 --

[PATCH v1 14/17] colibri-imx8x: enable environment bootcount limit

2023-03-03 Thread Andrejs Cainikovs
From: Marcel Ziswiler Enable optional environment bootcount limit functionality. Signed-off-by: Marcel Ziswiler Signed-off-by: Andrejs Cainikovs --- configs/colibri-imx8x_defconfig | 2 ++ 1 file changed, 2 insertions(+) diff --git a/configs/colibri-imx8x_defconfig

[PATCH v1 15/17] colibri-imx8x: set bootdelay

2023-03-03 Thread Andrejs Cainikovs
From: Marcel Ziswiler Set the boot delay to one second. Signed-off-by: Marcel Ziswiler Signed-off-by: Andrejs Cainikovs --- configs/colibri-imx8x_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/configs/colibri-imx8x_defconfig b/configs/colibri-imx8x_defconfig index

[PATCH v1 13/17] colibri-imx8x: set bootaux memory base and size

2023-03-03 Thread Andrejs Cainikovs
Move i.MX auxiliary core memory base and size configuration to defconfig where it should belong. Signed-off-by: Andrejs Cainikovs --- configs/colibri-imx8x_defconfig | 2 ++ include/configs/colibri-imx8x.h | 5 - 2 files changed, 2 insertions(+), 5 deletions(-) diff --git

[PATCH v1 12/17] colibri-imx8x: update env memory layout

2023-03-03 Thread Andrejs Cainikovs
From: Marcel Ziswiler Update the distro config env memory layout for the Colibri iMX8X: - kernel_comp_addr_r=0xb000 temporary area for uncompressing (ie FIT images or Image.gz booted using booti) - kernel_comp_size=0x0800 - loadaddr=0x9540 avoiding any reserved areas located before

[PATCH v1 11/17] colibri-imx8x: switch from fatload to load

2023-03-03 Thread Andrejs Cainikovs
Make sure M4 binary loading works equally well on ext4 as well as fat file systems. Signed-off-by: Andrejs Cainikovs --- include/configs/colibri-imx8x.h | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/include/configs/colibri-imx8x.h b/include/configs/colibri-imx8x.h

[PATCH v1 10/17] colibri-imx8x: provide proper CONFIG_SYS_PROMPT

2023-03-03 Thread Andrejs Cainikovs
From: Igor Opaniuk Provide proper sys prompt, which should be "Colibri iMX8X #". Signed-off-by: Igor Opaniuk Signed-off-by: Andrejs Cainikovs --- configs/colibri-imx8x_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/configs/colibri-imx8x_defconfig

[PATCH v1 09/17] colibri-imx8x: drop obsolete environment variables

2023-03-03 Thread Andrejs Cainikovs
Drop obsolete environment variables boot_file, bootcmd_mfg, fdt_addr, finduuid, image, mfgtool_args, mmcargs, mmcdev, mmcpart, panel, sec_boot, vidargs. Signed-off-by: Andrejs Cainikovs --- include/configs/colibri-imx8x.h | 35 + 1 file changed, 1 insertion(+),

[PATCH v1 08/17] colibri-imx8x: construct fdtfile dynamically

2023-03-03 Thread Andrejs Cainikovs
The following expression is used to construct the device tree name: fdtfile=${soc}-colibri-${fdt_board}.dtb - soc is set dynamically (either imx8qxp or imx8dx) - fdt_board can be modified by the user (eval-v3, aster, iris/iris-v2) Signed-off-by: Andrejs Cainikovs ---

[PATCH v1 07/17] colibri-imx8x: extract is_imx8dx() from ram detection

2023-03-03 Thread Andrejs Cainikovs
From: Max Krummenacher Refactor the detection of QXP vs. DX SoC into its own helper function. Signed-off-by: Max Krummenacher Signed-off-by: Andrejs Cainikovs --- board/toradex/colibri-imx8x/colibri-imx8x.c | 22 - 1 file changed, 13 insertions(+), 9 deletions(-) diff

[PATCH v1 06/17] colibri-imx8x: update update_uboot confirmation message

2023-03-03 Thread Andrejs Cainikovs
From: Marcel Ziswiler Update update_uboot confirmation message. Signed-off-by: Marcel Ziswiler Signed-off-by: Andrejs Cainikovs --- include/configs/colibri-imx8x.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/configs/colibri-imx8x.h

[PATCH v1 04/17] colibri-imx8x: remove obsolete net usb start

2023-03-03 Thread Andrejs Cainikovs
From: Marcel Ziswiler Remove obsolete net USB start. While at it also add a comment about enabling distro-boot. Signed-off-by: Marcel Ziswiler Signed-off-by: Andrejs Cainikovs --- include/configs/colibri-imx8x.h | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git

[PATCH v1 05/17] colibri-imx8x: remove obsolete sdhc related config defines

2023-03-03 Thread Andrejs Cainikovs
From: Marcel Ziswiler Remove obsolete SDHC related config defines. Nowadays, all SDHC related hardware configuration comes from the device tree. Signed-off-by: Marcel Ziswiler Signed-off-by: Andrejs Cainikovs --- include/configs/colibri-imx8x.h | 11 --- 1 file changed, 11

[PATCH v1 02/17] colibri-imx8x: Remove baudrate from console argument

2023-03-03 Thread Andrejs Cainikovs
From: Philippe Schenker This commit does remove the options argument from the console kernel-argument as it prevents the serial driver from outputting anything. Do this by switchting to use the variable "setup" as it is done on other Toradex modules. Signed-off-by: Philippe Schenker

[PATCH v1 01/17] board: colibri-imx8x: add 2nd ethernet address

2023-03-03 Thread Andrejs Cainikovs
All Colibri iMX8X variants have 2nd RGMII on SoC, so add the address for 2nd ethernet. Signed-off-by: Andrejs Cainikovs --- board/toradex/colibri-imx8x/Kconfig | 3 +++ 1 file changed, 3 insertions(+) diff --git a/board/toradex/colibri-imx8x/Kconfig b/board/toradex/colibri-imx8x/Kconfig index

[PATCH v1 03/17] defconfig: colibri-imx8x: enable CONFIG_OF_SYSTEM_SETUP

2023-03-03 Thread Andrejs Cainikovs
From: Philippe Schenker Enable a call to ft_system_setup() which reserves M4 memory region. Signed-off-by: Philippe Schenker Signed-off-by: Andrejs Cainikovs --- configs/colibri-imx8x_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/configs/colibri-imx8x_defconfig

[PATCH v1 00/17] colibri-imx8x: support refresh

2023-03-03 Thread Andrejs Cainikovs
This series refreshes Colibri iMX8X support as follows: - enable fuse command - revert switch to binman - set boot delay - enable bootcount limit - move bootaux memory base and size configuration to defconfig - update env memory layout - use load instead of fatload in M4 boot environment helpers -

Re: [PATCH v3 3/4] Kconfig: j721e: Change K3_MCU_SCRATCHPAD_BASE to non firewalled region

2023-03-03 Thread Kamlesh Gurudasani
Manorit Chawdhry writes: > In non-combined boot flow for K3, all the firewalls are locked by default > until sysfw comes up. Rom configures some of the firewall for its usage > along with the SRAM for R5 but the PSRAM region is still locked. > > The K3 MCU Scratchpad for j721e was set to a PSRAM

[PATCH v3 4/4] arm: mach-k3: Increase SYSFW max image size

2023-03-03 Thread Manorit Chawdhry
From: Dave Gerlach When booting with HS silicon, the system firmware image is 278270, which is slightly larger than currently allocated amount. This can cause unexpected behavior if this overlap interferes with other things in memory, so increase this with a slightly margin added as well to

[PATCH v3 3/4] Kconfig: j721e: Change K3_MCU_SCRATCHPAD_BASE to non firewalled region

2023-03-03 Thread Manorit Chawdhry
In non-combined boot flow for K3, all the firewalls are locked by default until sysfw comes up. Rom configures some of the firewall for its usage along with the SRAM for R5 but the PSRAM region is still locked. The K3 MCU Scratchpad for j721e was set to a PSRAM region triggering the firewall

[PATCH v3 2/4] configs: j721e: Merge the HS and non-HS defconfigs

2023-03-03 Thread Manorit Chawdhry
K3 devices have runtime type board detection. Make the default defconfig include the secure configuration. Then remove the HS specific config. Non-HS devices will continue to boot due to runtime device type detection. If TI_SECURE_DEV_PKG is not set the build will emit warnings, for non-HS

[PATCH v3 1/4] arm: k3: config.mk: Add missing dependencies on tispl.bin HS

2023-03-03 Thread Manorit Chawdhry
From: Andrew Davis When building for secure devices using non-buildman based image generation the signed tispl.bin file is called tispl.bin_HS. Also build the unsigned tispl.bin file as expected. Signed-off-by: Andrew Davis Signed-off-by: Manorit Chawdhry --- arch/arm/mach-k3/config.mk | 2

[PATCH v3 0/4] J721E HS Fixes

2023-03-03 Thread Manorit Chawdhry
This series fixes the support for J721E board along with some maintainance for it. v3: - drop fixes related to other boards ( make series j721e specific ) - memory map added in docs - applied reviewed-by v2: - Merge two series - Include some more commits that fix the gp builds when merging

Re: [PATCH v5 0/2] arm: dts: rockchip: rk3399: usb: ehci: Fix EHCI probe in rk3399 to access peripherals by USB 2.

2023-03-03 Thread Xavier Drudis Ferran
El Fri, Mar 03, 2023 at 10:42:20AM +0100, Christoph Fritz deia: > >    Changes: > > > >    v5: fixes a bug that Christoph Fritz discovered, consisting in the > >    wrong eror code returned when enabling or disabling the clock > >    because property_enable() returns an error code in

using binman sym for size of u-boot itb

2023-03-03 Thread Rasmus Villemoes
On imx8mp, binman is heavily used for the assembling of all the various pieces. Partly for my own future reference: There's u-boot-spl-ddr which is u-boot-spl.bin with the DDR firmware blobs tacked on at the end, and the sizes and positions of those blobs then updated via the binman syms

Re: [PATCH v5 0/2] arm: dts: rockchip: rk3399: usb: ehci: Fix EHCI probe in rk3399 to access peripherals by USB 2.

2023-03-03 Thread Christoph Fritz
>    Changes: > >    v5: fixes a bug that Christoph Fritz discovered, consisting in the >    wrong eror code returned when enabling or disabling the clock >    because property_enable() returns an error code in linux but >    the modified register value in U-Boot. This caused the clk

  1   2   >