Re: [PATCH 1/1] efi_loader: fix 'efidebug bootorder'

On Wed, Apr 29, 2020 at 09:23:01PM +0200, Heinrich Schuchardt wrote: > * don't copy GUIDs for no reason > * shorten print format strings by using variable names > * don't use the run-time table to access exported functions I don't much care about those changes, but my initial intent was that any e

Re: [PATCH] lib: rsa: Fix unaligned 64-bit fdt accesses

On 06.05.20 22:00, Tom Rini wrote: On Wed, May 06, 2020 at 06:32:03PM +0200, Jan Kiszka wrote: From: Jan Kiszka The fdt only provides 32-bit alignment of data. If the public_exponent happens to be not 64-bit aligned, we can trigger an exception on certain architectures. Seen on TI AM64x. Not

Re: [PATCH 04/10] efi_loader: capsule: add capsule_on_disk support

On Thu, Apr 30, 2020 at 09:51:51PM +0200, Heinrich Schuchardt wrote: > On 4/30/20 2:52 PM, Sughosh Ganu wrote: > > > > On Tue, 28 Apr 2020 at 05:58, AKASHI Takahiro > > mailto:takahiro.aka...@linaro.org>> wrote: > > > > Heinrich, > > > > On Mon, Apr 27, 2020 at 10:28:35PM +0200, Heinrich Sc

Re: [PATCH 3/8] qemu: arm64: Add support for efi firmware management protocol routines

On Fri, May 01, 2020 at 11:33:42AM +0200, Heinrich Schuchardt wrote: > On 4/30/20 9:13 PM, Sughosh Ganu wrote: > > > > On Fri, 1 May 2020 at 00:09, Heinrich Schuchardt > > wrote: > > > > On 4/30/20 7:36 PM, Sughosh Ganu wrote: > > > Add support for the get_image_

Re: [PATCH 8/8] qemu: arm64: Add documentation for capsule update

On Fri, May 01, 2020 at 11:17:27AM +0530, Sughosh Ganu wrote: > On Fri, 1 May 2020 at 00:57, Tom Rini wrote: > > > On Fri, May 01, 2020 at 12:38:45AM +0530, Sughosh Ganu wrote: > > > On Fri, 1 May 2020 at 00:07, Heinrich Schuchardt > > wrote: > > > > > > > On 4/30/20 7:36 PM, Sughosh Ganu wrote:

Re: [PATCH 10/13] imx: load calibration parameters from fuse for i.MX8MP

Hi Peng, On Wed, May 6, 2020 at 9:08 PM Peng Fan wrote: > I'll give a look. Thanks for raising the issue. I understand the issue now. I will send a patch soon. Thanks

Re: [PATCH 08/10] cmd: add "efidebug capsule" command

On Thu, Apr 30, 2020 at 06:08:11PM +0530, Sughosh Ganu wrote: > On Mon, 27 Apr 2020 at 15:19, AKASHI Takahiro > wrote: > > > "efidebug capsule" is more or less a debugging utility. > > efidebug capsule update: invoke UpdateCapsule against data on memory > > efidebug capsule show: show a capsu

Re: [PATCH 1/1] test: stabilize test_efi_secboot

Heinrich, On Mon, May 04, 2020 at 12:33:26PM +0200, Heinrich Schuchardt wrote: > When setting up the console via function efi_console_register() we call > query_console_serial(). This functions sends an escape sequence to the > terminal to query the display size. The response is another escape > s

efi_loader: pkcs7_parse_message() returns error pointer

Since pkcs7_parse_message() returns an error pointer, we must not check for NULL. We have to explicitly set msg to NULL in the error case, otherwise the call to pkcs7_free_message() on the goto err path will assume it's a valid object. Signed-off-by: Patrick Wildt diff --git a/lib/efi_loader/ef

efi_loader: efi_variable_parse_signature() returns NULL on error

efi_variable_parse_signature() returns NULL on error, so IS_NULL() is an incorrect check. The goto err leads to pkcs7_free_message(), which works fine on a NULL ptr. Signed-off-by: Patrick Wildt diff --git a/lib/efi_loader/efi_variable.c b/lib/efi_loader/efi_variable.c index 58f8fae358..c5fe896

RE: [PATCH 10/13] imx: load calibration parameters from fuse for i.MX8MP

> Subject: Re: [PATCH 10/13] imx: load calibration parameters from fuse for > i.MX8MP > > Hi Peng, > > On Mon, May 4, 2020 at 11:50 PM Peng Fan wrote: > > > Busfreq could be disabled by set the device tree node to disabled. > > This does not help. If I use the NXP U-Boot I can boot the NXP 5.4

Re: [PATCH 1/2] test: describe naming conventions for macro UNIT_TEST

On 5/6/20 10:26 AM, Heinrich Schuchardt wrote: > Strict naming conventions have to be followed for Python function > generate_ut_subtest() to collect C unit tests to be executed via > command 'ut'. > > Describe the requirements both on the C as well on the Python side. > +/** > + * UNIT_TEST() -

TPM: remove duplicate definitions

With the recent change to tpm-v2.h, some enums are now defined twice and tpm2_tis_spi.c fails to build. Unfortunately I fear removing the defines from tpm_tis.h, like in this diff, will break the TPMv1 drivers tpm_tis_infineon.c and pm_tis_st33zp24_i2c.c, which depend on those defines. Maybe they

[PATCH 3/6] patman: Don't try to process checkpatch lines twice

Once we have determined what the line refers to there is no point in processing it further. Update the logic to continue to the next line in these cases. Signed-off-by: Simon Glass --- tools/patman/checkpatch.py | 11 --- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/too

[PATCH 5/6] patman: Support warnings in the patch subject

Sometimes checkpatch outputs problems in the patch subject. Add support for parsing this output and reporting it correctly. Signed-off-by: Simon Glass --- tools/patman/checkpatch.py | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/tools/patman/checkpatch.py b/tools/patm

[PATCH 4/6] patman: Handle checkpatch output with notes and code

If checkpatch is configured to output code we should ignore it. Similarly, notes should be ignored. Update the logic to handle these situations. Signed-off-by: Simon Glass --- tools/patman/checkpatch.py | 11 ++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/tools/patma

[PATCH 6/6] patman: Complain if a checkpatch line is not understood

Rather than suffering in silence, output a warning if something about the checkpatch output cannot be understood. Signed-off-by: Simon Glass --- tools/patman/checkpatch.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tools/patman/checkpatch.py b/tools/patman/checkpatch.py index 5ae450d

Re: [PATCH v4 4/4] patman: Modify functional tests for new behavior

On Mon, 4 May 2020 at 14:29, Sean Anderson wrote: > > This patch adds or modifies functional tests for the Cover-changes, > Commit-changes, and Series-process-log tags in order to account for new > behavior added in the previous few patches. The '(no changes since v1)' > case is not tested for, si

[PATCH 2/6] patman: Support emacs mode with checkpatch

If checkpatch is run in 'emacs' mode it shows the filename at the start of each line. Add support for this so that the warnings and errors are correctly detected. Signed-off-by: Simon Glass --- tools/patman/checkpatch.py | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --g

[PATCH 1/6] patman: Fix 'warning' typo

If no warnings are detected due to checkpatch having unexpected options, patman currently shows an error: TypeError: unsupported operand type(s) for +=: 'int' and 'property' Fix this by initing the variable correctly. Signed-off-by: Simon Glass --- tools/patman/checkpatch.py | 2 +- 1 file

[PATCH 0/6] patman: Fixes to allow patman to work with Zephyr OS

At present patman doesn't correctly parse the output of checkpatch if the --emacs and --show-types options are given in the .checkpatch.conf file. This series corrects these problems, allowing patman to be used to check patches intended for Zephyr OS. Simon Glass (6): patman: Fix 'warning' typ

[PATCH v4 10/12] phy: atheros: add device tree bindings and config

Add support for configuring the CLK_25M pin as well as the RGMII I/O voltage by the device tree. By default the AT803x PHYs outputs the 25MHz clock of the XTAL input. But this output can also be changed by software to other frequencies. This commit introduces a generic way to configure this output

[PATCH v4 12/12] phy: atheros: consolidate {ar8031|ar8035}_config()

The two functions are now exactly the same, remove one of them. Signed-off-by: Michael Walle Acked-by: Joe Hershberger --- drivers/net/phy/atheros.c | 30 +++--- 1 file changed, 3 insertions(+), 27 deletions(-) diff --git a/drivers/net/phy/atheros.c b/drivers/net/phy/at

[PATCH v4 11/12] phy: atheros: ar8035: remove static clock config

We can configure the clock output in the device tree. Disable the hardcoded one in here. This is highly board-specific and should have never been enabled in the PHY driver. If bisecting shows that this commit breaks your board it probably depends on the clock output of your Atheros AR8035 PHY. Ple

[PATCH v4 08/12] phy: atheros: introduce debug read and write functions

Provide functions to read and write the Atheros debug registers. Signed-off-by: Michael Walle Acked-by: Joe Hershberger --- drivers/net/phy/atheros.c | 57 --- 1 file changed, 41 insertions(+), 16 deletions(-) diff --git a/drivers/net/phy/atheros.c b/drivers

[PATCH v4 09/12] phy: atheros: move delay config to common function

Signed-off-by: Michael Walle Acked-by: Joe Hershberger --- drivers/net/phy/atheros.c | 38 ++ 1 file changed, 22 insertions(+), 16 deletions(-) diff --git a/drivers/net/phy/atheros.c b/drivers/net/phy/atheros.c index 660dcd9491..22035c2496 100644 --- a/driver

[PATCH v4 07/12] phy: atheros: use defines for PHY IDs

Signed-off-by: Michael Walle Acked-by: Joe Hershberger --- drivers/net/phy/atheros.c | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/drivers/net/phy/atheros.c b/drivers/net/phy/atheros.c index 01953a1390..5ff5875d3d 100644 --- a/drivers/net/phy/atheros.c +++ b/driv

[PATCH v4 04/12] phy: atheros: Explicitly disable RGMII delays

From: Vladimir Oltean To eliminate any doubts about the out-of-reset value of the PHY, that the driver previously relied on. If bisecting shows that this commit breaks your board you probably have a wrong PHY interface mode. You probably want the PHY_INTERFACE_MODE_RGMII_RXID or PHY_INTERFACE_MO

[PATCH v4 06/12] phy: atheros: fix AR8021 PHY ID mask

The upper bits are all the OUI. Signed-off-by: Michael Walle Acked-by: Joe Hershberger --- drivers/net/phy/atheros.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/phy/atheros.c b/drivers/net/phy/atheros.c index 3cc162828c..01953a1390 100644 --- a/drivers/net/ph

[PATCH v4 05/12] phy: atheros: Clarify the intention of ar8021_config

From: Vladimir Oltean Debug register 5 contains TX_CLK DELAY at bit 8 and reserved values at the other bit positions, just like the other PHYs in the family do. Therefore, it is not necessary to hardcode the reserved values, but instead simply follow the read-modify-write procedure from the commo

[PATCH v4 00/12] phy: atheros: dt bindings and cleanup

This patch series superseeds the following two: >From Vladimir Oltean https://patchwork.ozlabs.org/cover/1031360/ >From me: https://patchwork.ozlabs.org/cover/1184507/ Although the first is marked as accepted into u-boot-net I guess it was removed due to broken boards ("DT as ABI", RGMII delay

[PATCH v4 02/12] phy: atheros: Use common functions for RGMII internal delays

From: Vladimir Oltean Signed-off-by: Vladimir Oltean Acked-by: Joe Hershberger --- drivers/net/phy/atheros.c | 69 +++ 1 file changed, 41 insertions(+), 28 deletions(-) diff --git a/drivers/net/phy/atheros.c b/drivers/net/phy/atheros.c index 537c1a9125..c0c

[PATCH v4 01/12] phy: atheros: Make RGMII Tx delays actually configurable for AR8035

From: Vladimir Oltean Delete the extraneous write to debug reg 5 that enables Tx delay When the driver was originally introduced in commit "6027384a phylib: Add Atheros AR8035 GETH PHY support", the Tx delay was being unconditionally enabled. Then during "2ec4d10b phy: atheros: add support for

[PATCH v4 03/12] phy: atheros: Clarify the configuration of the CLK_25M output pin

From: Vladimir Oltean Also take the opportunity to use the phy_read_mmd and phy_write_mmd convenience functions. Signed-off-by: Vladimir Oltean Acked-by: Joe Hershberger --- drivers/net/phy/atheros.c | 19 ++- 1 file changed, 14 insertions(+), 5 deletions(-) diff --git a/driv

Re: raspberrypi cm3 v2019.07 not working

On Tue, May 5, 2020 at 10:18 PM Belisko Marek wrote: > > Hi, > > I'm trying to run 2019.07 u-boot on raspberrypi cm3 module but I got > no feedback on console. When used 2018.01 it boots fine. Probably I > need to do git bisect but maybe someone have some idea or experience > what can cause that.

Re: [PATCH 32/36] bdinfo: m68k: Move m68k-specific info into its own file

On Tue, May 5, 2020 at 1:19 AM Simon Glass wrote: > > We don't really want to have m68k-specific code in a generic file. Create > a new arch-specific function to hold it, and move it into that. > > Make the function weak so that any arch can implement it. > > Signed-off-by: Simon Glass > --- > >

Re: [PATCH] lib: rsa: Fix unaligned 64-bit fdt accesses

On Wed, May 06, 2020 at 06:32:03PM +0200, Jan Kiszka wrote: > From: Jan Kiszka > > The fdt only provides 32-bit alignment of data. If the public_exponent > happens to be not 64-bit aligned, we can trigger an exception on certain > architectures. Seen on TI AM64x. > > Note that the normal way of

Re: [PATCH v3 00/12] phy: atheros: dt bindings and cleanup

On Wed, May 06, 2020 at 09:47:10PM +0200, Michael Walle wrote: > Am 2020-05-06 21:14, schrieb Michael Walle: > > This patch series superseeds the following two: > > From Vladimir Oltean > > https://patchwork.ozlabs.org/cover/1031360/ > > From me: > > https://patchwork.ozlabs.org/cover/1184507/

Re: [PATCH v3 00/12] phy: atheros: dt bindings and cleanup

Am 2020-05-06 21:14, schrieb Michael Walle: This patch series superseeds the following two: From Vladimir Oltean https://patchwork.ozlabs.org/cover/1031360/ From me: https://patchwork.ozlabs.org/cover/1184507/ Although the first is marked as accepted into u-boot-net I guess it was removed du

[PATCH v2 1/2] armv8: ls1028a: move FSL_LAYERSCAPE to kconfig

CONFIG_FSL_LAYERSCAPE is available in kconfig. There is no need to define it per board; the ls1028a_common.h is really board dependent and only fits to the NXP eval boards. Instead select CONFIG_FSL_LAYERSCAPE when ARCH_LS1028A is selected. Signed-off-by: Michael Walle --- changes since v1: non

[PATCH v2 2/2] board: kontron: add sl28 support

Add basic support for the Kontron SMARC-sAL28 board. This includes just the bare minimum to be able to bring up the board and boot linux. Also only one board variant is supported for now. That is the Single PHY variant. Other variants will fall back to this one. In particular, there is no watchdo

[PATCH v3 11/12] phy: atheros: ar8035: remove static clock config

We can configure the clock output in the device tree. Disable the hardcoded one in here. This is highly board-specific and should have never been enabled in the PHY driver. If bisecting shows that this commit breaks your board it probably depends on the clock output of your Atheros AR8035 PHY. Ple

[PATCH v3 08/12] phy: atheros: introduce debug read and write functions

Provide functions to read and write the Atheros debug registers. Signed-off-by: Michael Walle --- drivers/net/phy/atheros.c | 57 --- 1 file changed, 41 insertions(+), 16 deletions(-) diff --git a/drivers/net/phy/atheros.c b/drivers/net/phy/atheros.c index 5f

[PATCH v3 07/12] phy: atheros: use defines for PHY IDs

Signed-off-by: Michael Walle --- drivers/net/phy/atheros.c | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/drivers/net/phy/atheros.c b/drivers/net/phy/atheros.c index 01953a1390..5ff5875d3d 100644 --- a/drivers/net/phy/atheros.c +++ b/drivers/net/phy/atheros.c @@ -2

[PATCH v3 09/12] phy: atheros: move delay config to common function

Signed-off-by: Michael Walle --- drivers/net/phy/atheros.c | 38 ++ 1 file changed, 22 insertions(+), 16 deletions(-) diff --git a/drivers/net/phy/atheros.c b/drivers/net/phy/atheros.c index 660dcd9491..22035c2496 100644 --- a/drivers/net/phy/atheros.c +++ b/d

[PATCH v3 12/12] phy: atheros: consolidate {ar8031|ar8035}_config()

The two functions are now exactly the same, remove one of them. Signed-off-by: Michael Walle --- drivers/net/phy/atheros.c | 30 +++--- 1 file changed, 3 insertions(+), 27 deletions(-) diff --git a/drivers/net/phy/atheros.c b/drivers/net/phy/atheros.c index 3142733105..4

[PATCH v3 10/12] phy: atheros: add device tree bindings and config

Add support for configuring the CLK_25M pin as well as the RGMII I/O voltage by the device tree. By default the AT803x PHYs outputs the 25MHz clock of the XTAL input. But this output can also be changed by software to other frequencies. This commit introduces a generic way to configure this output

[PATCH v3 03/12] phy: atheros: Clarify the configuration of the CLK_25M output pin

From: Vladimir Oltean Also take the opportunity to use the phy_read_mmd and phy_write_mmd convenience functions. Signed-off-by: Vladimir Oltean Acked-by: Joe Hershberger --- drivers/net/phy/atheros.c | 19 ++- 1 file changed, 14 insertions(+), 5 deletions(-) diff --git a/driv

[PATCH v3 05/12] phy: atheros: Clarify the intention of ar8021_config

From: Vladimir Oltean Debug register 5 contains TX_CLK DELAY at bit 8 and reserved values at the other bit positions, just like the other PHYs in the family do. Therefore, it is not necessary to hardcode the reserved values, but instead simply follow the read-modify-write procedure from the commo

[PATCH v3 02/12] phy: atheros: Use common functions for RGMII internal delays

From: Vladimir Oltean Signed-off-by: Vladimir Oltean Acked-by: Joe Hershberger --- drivers/net/phy/atheros.c | 69 +++ 1 file changed, 41 insertions(+), 28 deletions(-) diff --git a/drivers/net/phy/atheros.c b/drivers/net/phy/atheros.c index 537c1a9125..c0c

[PATCH v3 04/12] phy: atheros: Explicitly disable RGMII delays

From: Vladimir Oltean To eliminate any doubts about the out-of-reset value of the PHY, that the driver previously relied on. If bisecting shows that this commit breaks your board you probably have a wrong PHY interface mode. You probably want the PHY_INTERFACE_MODE_RGMII_RXID or PHY_INTERFACE_MO

[PATCH v3 00/12] phy: atheros: dt bindings and cleanup

This patch series superseeds the following two: >From Vladimir Oltean https://patchwork.ozlabs.org/cover/1031360/ >From me: https://patchwork.ozlabs.org/cover/1184507/ Although the first is marked as accepted into u-boot-net I guess it was removed due to broken boards ("DT as ABI", RGMII delay

[PATCH v3 06/12] phy: atheros: fix AR8021 PHY ID mask

The upper bits are all the OUI. Signed-off-by: Michael Walle --- drivers/net/phy/atheros.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/phy/atheros.c b/drivers/net/phy/atheros.c index 3cc162828c..01953a1390 100644 --- a/drivers/net/phy/atheros.c +++ b/drivers/n

[PATCH v3 01/12] phy: atheros: Make RGMII Tx delays actually configurable for AR8035

From: Vladimir Oltean Delete the extraneous write to debug reg 5 that enables Tx delay When the driver was originally introduced in commit "6027384a phylib: Add Atheros AR8035 GETH PHY support", the Tx delay was being unconditionally enabled. Then during "2ec4d10b phy: atheros: add support for

[PATCH 3/6] efi_loader: Implement EFI variable handling via OP-TEE

In OP-TEE we can run EDK2's StandAloneMM on a secure partition. StandAloneMM is responsible for the UEFI variable support. In combination with OP-TEE and it's U-Boot supplicant, variables are authenticated/validated in secure world and stored on an RPMB partition. So let's add a new config option

[PATCH 6/6] doc: uefi.rst: Add OP-TEE variable storage config options

If OP-TEE is compiled with an EDK2 application running in secure world it can process and store UEFI variables in an RPMB. Add documentation for the config options enabling this Signed-off-by: Ilias Apalodimas --- doc/uefi/uefi.rst | 10 ++ 1 file changed, 10 insertions(+) diff --git a/

[PATCH 2/6] efi_loader: Add headers for EDK2 StandAloneMM communication

From: Sughosh Ganu In Arm devices OP-TEE has the ability to run StandAloneMM (from EDK2) in a separate partition and handle UEFI variables. A following patch introduces this functionality. Add the headers needed for OP-TEE <--> StandAloneMM communication Signed-off-by: Sughosh Ganu Signed-off-

[PATCH 4/6] cmd: efidebug: Add support for querying UEFI variable storage

With the previous patches that use OP-TEE and StandAloneMM for UEFI variable storage we've added functionality for efi_query_variable_info. So let's add the relevant command to efidebug and retrieve information about the container used to store UEFI variables Signed-off-by: Ilias Apalodimas ---

[PATCH 5/6] MAINTAINERS: Add maintainer for EFI variables via OP-TEE

Add myself as maintainer for the OP-TEE related UEFI variable storage and add the headers files on the existing EFI list Signed-off-by: Ilias Apalodimas --- MAINTAINERS | 7 +++ 1 file changed, 7 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index ec59ce8b8802..f33fd74b330b 100644 --

[PATCH 1/6] charset: Add support for calculating bytes occupied by a u16 string

From: Sughosh Ganu The current code uses 'u16_strlen(x) + 1) * sizeof(u16)' in various places to calculate the number of bytes occupied by a u16 string. Let's introduce a wrapper around this. This wrapper is used on following patches Signed-off-by: Sughosh Ganu --- include/charset.h | 11 +

[PATCH 0/6] EFI variable support via OP-TEE

With new OP-TEE and EDK2 patches (not yet upstreamed [1][2]) we can run a secure world application which manages UEFI variables. Leveraging the U-Boot's OP-TEE supplicant we can then store those values in an RPMB device. The Secure World application responsible for doing that is coming from EDK2

Re: [PATCH 1/2] JFFS2: Process obsolete nodes as well as accurate ones

On Mon, Apr 27, 2020 at 06:43:25PM +0200, petr.bors...@i.cz wrote: > From: Petr Borsodi > > Obsolete nodes (ie. without the JFFS2_NODE_ACCURATE flag) were ignored because > they had seemingly invalid crc. This could lead to finding the phantom node > header in obsolete node data. > > Signed-off

Re: [PATCH v2 1/2] net: cortina_ni: Addd eth support for Cortina Access CAxxxx SoCs

On Wed, Apr 29, 2020 at 05:07:29PM -0700, Alex Nemirovsky wrote: Now, > From: Aaron Tseng > > Add Cortina Access Ethernet device driver for CA SoCs. > This driver supports both legacy and DM_ETH network models. > > Signed-off-by: Aaron Tseng > Signed-off-by: Alex Nemirovsky > Signed-off-by

[PATCH 2/2] board: kontron: add sl28 support

Add basic support for the Kontron SMARC-sAL28 board. This includes just the bare minimum to be able to bring up the board and boot linux. Also only one board variant is supported for now. That is the Single PHY variant. Other variants will fall back to this one. In particular, there is no watchdo

[PATCH 1/2] armv8: ls1028a: move FSL_LAYERSCAPE to kconfig

CONFIG_FSL_LAYERSCAPE is available in kconfig. There is no need to define it per board; the ls1028a_common.h is really board dependent and only fits to the NXP eval boards. Instead select CONFIG_FSL_LAYERSCAPE when ARCH_LS1028A is selected. Signed-off-by: Michael Walle --- arch/arm/cpu/armv8/fsl

[PATCH 1/1] efi_loader: put device tree into EfiACPIReclaimMemory

According to the UEFI spec ACPI tables should be placed in EfiACPIReclaimMemory. Let's do the same with the device tree. Suggested-by: Ard Biesheuvel Cc: Grant Likely Signed-off-by: Heinrich Schuchardt --- cmd/bootefi.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/cm

Re: [PATCH V2] mkimage: fit: Do not tail-pad fitImage with external data

On 5/6/20 7:02 PM, Tom Rini wrote: > On Wed, May 06, 2020 at 06:35:52PM +0200, Marek Vasut wrote: >> On 5/6/20 6:33 PM, Tom Rini wrote: >>> On Wed, May 06, 2020 at 06:17:47PM +0200, Marek Vasut wrote: On 5/6/20 6:04 PM, Tom Rini wrote: > On Wed, May 06, 2020 at 05:52:45PM +0200, Marek Vasu

[PATCH 6/6] qemu-x86*_defconfig: Enable CONFIG_PCI_INIT_R

Enable CONFIG_PCI_INIT_R for qemux86 and qemux86-64 pci enumeration during boot in order to eliminate the custom preboot commands in include/configs/qemu-x86.h. Signed-off-by: Ovidiu Panait --- configs/qemu-x86_64_defconfig | 1 + configs/qemu-x86_defconfig| 1 + include/configs/qemu-x86.h

[PATCH 5/6] qemu_arm_defconfig: Enable CONFIG_PCI_INIT_R

Replace the "pci enum" preboot sequence with CONFIG_PCI_INIT_R=y. Signed-off-by: Ovidiu Panait --- configs/qemu_arm_defconfig | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/configs/qemu_arm_defconfig b/configs/qemu_arm_defconfig index f807dfc10e..a8473988bd 100644 --- a/co

[PATCH 1/6] env: Convert CONFIG_DELAY_ENVIRONMENT to Kconfig

This converts ad-hoc CONFIG_DELAY_ENVIRONMENT to Kconfig. Signed-off-by: Ovidiu Panait --- env/Kconfig | 12 scripts/config_whitelist.txt | 1 - 2 files changed, 12 insertions(+), 1 deletion(-) diff --git a/env/Kconfig b/env/Kconfig index af63ac52f7..ed94e83ec1 10

[PATCH 4/6] qemu_arm64_defconfig: Enable CONFIG_PCI_INIT_R

Replace the "pci enum" preboot sequence with CONFIG_PCI_INIT_R=y. Signed-off-by: Ovidiu Panait --- configs/qemu_arm64_defconfig | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/configs/qemu_arm64_defconfig b/configs/qemu_arm64_defconfig index 80e0ad55e0..53c653df21 100644 --

[PATCH 2/6] board_r: env: Use IS_ENABLED() instead of #ifdefs

Use IS_ENABLED() instead of #ifdef in should_load_env and initr_env functions. No functional change intended. Signed-off-by: Ovidiu Panait --- common/board_r.c | 21 +++-- 1 file changed, 11 insertions(+), 10 deletions(-) diff --git a/common/board_r.c b/common/board_r.c index d

[PATCH 3/6] board_r: Introduce CONFIG_PCI_INIT_R Kconfig option

With CONFIG_DM_PCI enabled, PCI buses are not enumerated at boot, as they are without that config option enabled. However, there are cases such as DM PCI-based Ethernet devices that need the PCI bus enumerated so that they can be discovered by their drivers. Currently, to solve this, some boards e

Re: [PATCH V2] mkimage: fit: Do not tail-pad fitImage with external data

On Wed, May 06, 2020 at 06:35:52PM +0200, Marek Vasut wrote: > On 5/6/20 6:33 PM, Tom Rini wrote: > > On Wed, May 06, 2020 at 06:17:47PM +0200, Marek Vasut wrote: > >> On 5/6/20 6:04 PM, Tom Rini wrote: > >>> On Wed, May 06, 2020 at 05:52:45PM +0200, Marek Vasut wrote: > On 5/6/20 5:43 PM, Ale

Re: [PATCH] misc: i2c_eeprom: implement different probe test eeprom offset

Hi Eugen, On Wed, May 06 2020, Eugen Hristev wrote: > Because of this commit : > 5ae84860b0 ("misc: i2c_eeprom: verify that the chip is functional at probe()") > at probe time, each eeprom is tested for read at offset 0. > > The Atmel AT24MAC402 eeprom has different mapping. One i2c slave address

Re: [PATCH V2] mkimage: fit: Do not tail-pad fitImage with external data

On 5/6/20 6:33 PM, Tom Rini wrote: > On Wed, May 06, 2020 at 06:17:47PM +0200, Marek Vasut wrote: >> On 5/6/20 6:04 PM, Tom Rini wrote: >>> On Wed, May 06, 2020 at 05:52:45PM +0200, Marek Vasut wrote: On 5/6/20 5:43 PM, Alex Kiernan wrote: > On Wed, May 6, 2020 at 3:41 PM Marek Vasut wrot

Re: [PATCH V2] mkimage: fit: Do not tail-pad fitImage with external data

On Wed, May 06, 2020 at 06:17:47PM +0200, Marek Vasut wrote: > On 5/6/20 6:04 PM, Tom Rini wrote: > > On Wed, May 06, 2020 at 05:52:45PM +0200, Marek Vasut wrote: > >> On 5/6/20 5:43 PM, Alex Kiernan wrote: > >>> On Wed, May 6, 2020 at 3:41 PM Marek Vasut wrote: > > On 5/6/20 4:37 PM, To

[PATCH] lib: rsa: Fix unaligned 64-bit fdt accesses

From: Jan Kiszka The fdt only provides 32-bit alignment of data. If the public_exponent happens to be not 64-bit aligned, we can trigger an exception on certain architectures. Seen on TI AM64x. Note that the normal way of accessing such a number would be fdtdec_get_number. However, this is not a

[PATCH 1/2] test: describe naming conventions for macro UNIT_TEST

Strict naming conventions have to be followed for Python function generate_ut_subtest() to collect C unit tests to be executed via command 'ut'. Describe the requirements both on the C as well on the Python side. Signed-off-by: Heinrich Schuchardt --- include/test/test.h | 24 +

[PATCH 0/2] test: fix naming of test functions in the log test suite

Some Python subtests executed via the 'ut' console command were not executed due to not following a required naming convention. Add a description of the naming convention. Fix the tests. Heinrich Schuchardt (2): test: describe naming conventions for macro UNIT_TEST test: fix naming of test fu

[PATCH 2/2] test: fix naming of test functions in the log test suite

Both the nolog as well as the syslog tests were not found by Python function generate_ut_subtest() due to not following the nameing requirements imposed by the regular expression used to find linker generated list entries in file u-boot.sym. Adjust the naming of test functions. With the patch the

Re: [PATCH] Revert "mkimage: fit: Do not tail-pad fitImage with external data"

On 5/6/20 5:31 PM, Tom Rini wrote: > On Wed, May 06, 2020 at 05:23:55PM +0200, Marek Vasut wrote: >> On 5/6/20 5:19 PM, Tom Rini wrote: >>> On Wed, May 06, 2020 at 05:08:12PM +0200, Marek Vasut wrote: On 5/6/20 5:05 PM, Tom Rini wrote: > This has been reported to break booting of U-Boot fr

Re: [PATCH V2] mkimage: fit: Do not tail-pad fitImage with external data

On 5/6/20 6:04 PM, Tom Rini wrote: > On Wed, May 06, 2020 at 05:52:45PM +0200, Marek Vasut wrote: >> On 5/6/20 5:43 PM, Alex Kiernan wrote: >>> On Wed, May 6, 2020 at 3:41 PM Marek Vasut wrote: On 5/6/20 4:37 PM, Tom Rini wrote: > On Wed, May 06, 2020 at 04:33:37PM +0200, Marek Vasut

Re: [PATCH 10/13] imx: load calibration parameters from fuse for i.MX8MP

Hi Peng, On Mon, May 4, 2020 at 11:50 PM Peng Fan wrote: > Busfreq could be disabled by set the device tree node to disabled. This does not help. If I use the NXP U-Boot I can boot the NXP 5.4.3 without problems. > > I have also tried to boot NXP 5.4.3 on a i.MX8MM EVK with the latest U-Boot >

Re: [PATCH 30/36] bdinfo: arm: Move ARM-specific info into its own file

Am 05.05.20 um 01:17 schrieb Simon Glass: > We don't really want to have ARM-specific code in a generic file. Create > a new arch-specific function to hold it, and move it into that. > > Make the function weak so that any arch can implement it. > > Signed-off-by: Simon Glass > --- > > arch/

[PATCH] misc: i2c_eeprom: implement different probe test eeprom offset

Because of this commit : 5ae84860b0 ("misc: i2c_eeprom: verify that the chip is functional at probe()") at probe time, each eeprom is tested for read at offset 0. The Atmel AT24MAC402 eeprom has different mapping. One i2c slave address is used for the lower 0x80 bytes and another i2c slave address

Re: [PATCH V2] mkimage: fit: Do not tail-pad fitImage with external data

On Wed, May 06, 2020 at 05:52:45PM +0200, Marek Vasut wrote: > On 5/6/20 5:43 PM, Alex Kiernan wrote: > > On Wed, May 6, 2020 at 3:41 PM Marek Vasut wrote: > >> > >> On 5/6/20 4:37 PM, Tom Rini wrote: > >>> On Wed, May 06, 2020 at 04:33:37PM +0200, Marek Vasut wrote: > On 5/6/20 4:27 PM, Tom

Re: [PATCH 07/36] bdinfo: microblaze: Use the generic bd command

Am 05.05.20 um 01:17 schrieb Simon Glass: > Microblaze prints out ethernet and FDT information. This is useful to > most archs, so move it into the generic code and move microblaze over to > use it. > > Signed-off-by: Simon Glass > --- > > cmd/bdinfo.c | 30 +++--- >

Re: [PATCH V2] mkimage: fit: Do not tail-pad fitImage with external data

On 5/6/20 5:43 PM, Alex Kiernan wrote: > On Wed, May 6, 2020 at 3:41 PM Marek Vasut wrote: >> >> On 5/6/20 4:37 PM, Tom Rini wrote: >>> On Wed, May 06, 2020 at 04:33:37PM +0200, Marek Vasut wrote: On 5/6/20 4:27 PM, Tom Rini wrote: > On Wed, May 06, 2020 at 04:17:35PM +0200, Marek Vasut w

Re: [PATCH V2] mkimage: fit: Do not tail-pad fitImage with external data

On Wed, May 6, 2020 at 3:41 PM Marek Vasut wrote: > > On 5/6/20 4:37 PM, Tom Rini wrote: > > On Wed, May 06, 2020 at 04:33:37PM +0200, Marek Vasut wrote: > >> On 5/6/20 4:27 PM, Tom Rini wrote: > >>> On Wed, May 06, 2020 at 04:17:35PM +0200, Marek Vasut wrote: > On 5/6/20 3:48 PM, Tom Rini wr

Re: [PATCH 00/36] Tidy up the 'bd' command.

Hi Alexey, On Wed, 6 May 2020 at 03:28, Alexey Brodkin wrote: > > Hi Simon, > > > -Original Message- > > From: Simon Glass > > Sent: Tuesday, May 5, 2020 2:17 AM > > To: U-Boot Mailing List > > [snip] > > > Subject: [PATCH 00/36] Tidy up the 'bd' command. > > > > The code for the 'bd'

Re: [PATCH] Revert "mkimage: fit: Do not tail-pad fitImage with external data"

On Wed, May 06, 2020 at 05:23:55PM +0200, Marek Vasut wrote: > On 5/6/20 5:19 PM, Tom Rini wrote: > > On Wed, May 06, 2020 at 05:08:12PM +0200, Marek Vasut wrote: > >> On 5/6/20 5:05 PM, Tom Rini wrote: > >>> This has been reported to break booting of U-Boot from SPL on a number > >>> of platforms

Re: [PATCH 05/36] bdinfo: mips: Use the generic bd command

Am 05.05.20 um 01:17 schrieb Simon Glass: > MIPS currently has a few extra things which are generally useful. Add them > to the generic function and move MIPS over to use it. > > Signed-off-by: Simon Glass > --- > > cmd/bdinfo.c | 13 - > 1 file changed, 4 insertions(+), 9 deleti

Re: [PATCH] Revert "mkimage: fit: Do not tail-pad fitImage with external data"

On 5/6/20 5:19 PM, Tom Rini wrote: > On Wed, May 06, 2020 at 05:08:12PM +0200, Marek Vasut wrote: >> On 5/6/20 5:05 PM, Tom Rini wrote: >>> This has been reported to break booting of U-Boot from SPL on a number >>> of platforms due to a lack of alignment of the external data. The >>> issues this c

Re: [PATCH] Revert "mkimage: fit: Do not tail-pad fitImage with external data"

On Wed, May 06, 2020 at 05:08:12PM +0200, Marek Vasut wrote: > On 5/6/20 5:05 PM, Tom Rini wrote: > > This has been reported to break booting of U-Boot from SPL on a number > > of platforms due to a lack of alignment of the external data. The > > issues this commit is addressing will need to be re

Re: [PATCH 33/36] bdinfo: m68k: ppc: Move arch-specific code from bdinfo

On 05.05.20 01:17, Simon Glass wrote: We don't have an easy way to share these three lines of code with two architectures. We also want to make it clear that this code is actually arch-specific. So just duplicate it in each arch-specific file. Signed-off-by: Simon Glass Reviewed-by: Stefan R

Re: [PATCH 3/3] lib: Allow hexdump to be used in SPL

On 06.05.20 16:03, Simon Glass wrote: It is sometimes useful to output hex dumps in SPL. Add a config option to allow this. Signed-off-by: Simon Glass Reviewed-by: Stefan Roese Thanks, Stefan --- lib/Kconfig | 6 ++ lib/Makefile | 2 +- 2 files changed, 7 insertions(+), 1 deleti

Re: [PATCH 31/36] bdinfo: ppc: Move PPC-specific info into its own file

On 05.05.20 01:17, Simon Glass wrote: We don't really want to have PPC-specific code in a generic file. Create a new arch-specific function to hold it, and move it into that. Make the function weak so that any arch can implement it. Signed-off-by: Simon Glass Reviewed-by: Stefan Roese Than

Re: [PATCH 25/36] bdinfo: net: ppc: Drop prints for CONFIG_HAS_ETHn

On 05.05.20 01:17, Simon Glass wrote: These config options have not been migrated to Kconfig. This should be handled using driver model, iterating over the available Ethernet devices. For now, remove the code. Signed-off-by: Simon Glass Reviewed-by: Stefan Roese Thanks, Stefan --- cmd/

Re: [PATCH 24/36] bdinfo: net: ppc: Drop bi_enet1addr and other similar info

On 05.05.20 01:17, Simon Glass wrote: These values were 'old' in 2013 so it should be safe to remove them. They are never set in U-Boot anyway, so the values will always be zero. Signed-off-by: Simon Glass Reviewed-by: Stefan Roese Thanks, Stefan --- common/board_r.c | 16 +

  1   2   3   >