RE: [EXT] [PATCH] arm: mvebu: Espressobin: Enable 'mtd' command and define SPI NOR partitions

2021-07-17 Thread Kostya Porotchkin
Hi, Pali, > -Original Message- > From: Pali Rohár > Sent: Friday, July 16, 2021 13:35 > To: Stefan Roese ; Andre Heider ; Gérald > Kerma ; Kostya Porotchkin > Cc: u-boot@lists.denx.de > Subject: [EXT] [PATCH] arm: mvebu: Espressobin: Enable 'mtd' command and > define SPI NOR partitions

[PATCH] mtd: nand: Fix typo in MXC Kconfig symbol description

2021-07-17 Thread li . haolin
From: Haolin Li Trivial typo fix. Signed-off-by: Haolin Li --- drivers/mtd/nand/raw/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mtd/nand/raw/Kconfig b/drivers/mtd/nand/raw/Kconfig index a901ce5511..6421e05da1 100644 --- a/drivers/mtd/nand/raw/Kconfig

Re: using binman fails boot

2021-07-17 Thread Simon Glass
Hi Tim, On Fri, 16 Jul 2021 at 17:15, Tim Harvey wrote: > > On Fri, Jul 16, 2021 at 3:11 PM Simon Glass wrote: > > > > () which has > > Hi Tim, > > > > On Fri, 16 Jul 2021 at 15:43, Tim Harvey wrote: > > > > > > On Thu, Jul 15, 2021 at 9:30 PM Simon Glass wrote: > > > > > > > > Hi Tim, > > >

Re: [PATCH 1/1] sandbox: don't set SA_NODEFER in signal handler

2021-07-17 Thread Simon Glass
Hi Heinrich, On Wed, 7 Jul 2021 at 11:37, Simon Glass wrote: > > Hi Heinrich, > > On Mon, 5 Jul 2021 at 11:43, Heinrich Schuchardt wrote: > > > > The sandbox can handle signals. Due to a damaged global data pointer > > additional exceptions in the signal handler may occur leading to an endless

Re: [PATCH 18/22] image: Allow @ in node names when not using signatures

2021-07-17 Thread Simon Glass
On Mon, 5 Jul 2021 at 16:33, Simon Glass wrote: > > If signature verification is not in use we don't need to worry about > the risk of using @ in node names. Update fit_image_verify() to allow > it if the function is not enabled. > > Signed-off-by: Simon Glass > --- > > common/image-fit.c | 2

Re: [PATCH 11/22] cros_ec: Allow reading the battery-charge state

2021-07-17 Thread Simon Glass
Add a function to read this information from the EC. It is useful for determining whether the battery has enough charge to boot. Signed-off-by: Simon Glass --- drivers/misc/cros_ec.c | 17 + include/cros_ec.h | 8 2 files changed, 25 insertions(+) Applied to

Re: [PATCH 01/22] dm: core: Add logging for DM_SEQ_ALIAS

2021-07-17 Thread Simon Glass
It is sometimes helpful to see which sequence is assigned to a device. Add debugging info for that. Signed-off-by: Simon Glass --- drivers/core/device.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) Applied to u-boot-dm, thanks!

Re: [PATCH 02/22] dm: Support lzma in the flashmap

2021-07-17 Thread Simon Glass
Allow lzma compression as well as lz4. Signed-off-by: Simon Glass --- drivers/core/of_extra.c | 2 ++ 1 file changed, 2 insertions(+) Applied to u-boot-dm, thanks!

Re: [PATCH 03/22] test: Allow CONFIG_SPL_LOAD_FIT to be disabled

2021-07-17 Thread Simon Glass
At present if this is not enabled on a sandbox build, the build fails. Add a condition to avoid this. Signed-off-by: Simon Glass --- test/Makefile | 2 ++ 1 file changed, 2 insertions(+) Applied to u-boot-dm, thanks!

Re: [PATCH 04/22] test: Add DM_DMA to be disabled

2021-07-17 Thread Simon Glass
At present if DM_DMA is disabled on a sandbox build, the build fails. Make the test conditional. Signed-off-by: Simon Glass --- test/dm/core.c | 2 ++ 1 file changed, 2 insertions(+) Applied to u-boot-dm, thanks!

Re: [PATCH 05/22] test: Avoid a build error with SPL

2021-07-17 Thread Simon Glass
At present this fails to build chromeos_sandbox due to a rebase error in dm_test_pre_run(). Fix it. Signed-off-by: Simon Glass --- test/test-main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) Applied to u-boot-dm, thanks!

Re: [PATCH 09/22] sandbox: Use hinting with the display

2021-07-17 Thread Simon Glass
SDL provides a hinting feature which provides a higher-quality image with the double-display option (-K). Enable it. Signed-off-by: Simon Glass --- arch/sandbox/cpu/sdl.c | 3 +++ 1 file changed, 3 insertions(+) Applied to u-boot-dm, thanks!

Re: [PATCH 10/22] sandbox: Adjust the bloblist default address

2021-07-17 Thread Simon Glass
Move this down to provide more space for the bloblist. Signed-off-by: Simon Glass --- common/Kconfig | 2 +- doc/arch/sandbox.rst | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) Applied to u-boot-dm, thanks!

Re: [PATCH 06/22] Makefile: Avoid rebuilding .dtb files each time

2021-07-17 Thread Simon Glass
Drop the FORCE from the rule that builds .dtb files and let the normal dependency checking do its work. This should work correctly, at least for .dts files that don't use /include/. Signed-off-by: Simon Glass --- scripts/Makefile.lib | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

Re: [PATCH 07/22] sandbox: Support executables for more phases

2021-07-17 Thread Simon Glass
The SPL header has a function for obtaining the phase in capital letters, e.g. 'SPL'. Add one for lower-case also, as used by sandbox. Use this to generalise the sandbox logic for determining the filename of the next sandbox executable. This can provide support for VPL. Signed-off-by: Simon

Re: [PATCH 08/22] sandbox: Add work-around for SDL2 display

2021-07-17 Thread Simon Glass
At present the display does not show on some machines, e.g. Ubunutu 20.04 but the reason is unknown. Add a work-around until this can be determined. Also include more error checking just in case. Signed-off-by: Simon Glass --- arch/sandbox/cpu/sdl.c | 23 ++- 1 file

Re: [PATCH 12/22] cros_ec: Drop cros_ec_entering_mode()

2021-07-17 Thread Simon Glass
This function is not needed anymore. Drop it. Signed-off-by: Simon Glass --- drivers/misc/cros_ec.c | 11 --- drivers/misc/cros_ec_sandbox.c | 3 --- include/cros_ec.h | 9 - 3 files changed, 23 deletions(-) Applied to u-boot-dm, thanks!

Re: [PATCH 13/22] cros_ec: Support the full-size vboot context

2021-07-17 Thread Simon Glass
The v2 format is 64-bytes in size. Support this and drop v1 since it is not used anymore. Signed-off-by: Simon Glass --- drivers/misc/cros_ec_sandbox.c | 8 +++- 1 file changed, 3 insertions(+), 5 deletions(-) Applied to u-boot-dm, thanks!

Re: [PATCH 14/22] cros_ec: Use standard calls for recovery-request checking

2021-07-17 Thread Simon Glass
Rather than calling directly into the sandbox SDL code, we can use the normal U-Boot console handling for this feature. Update the code, to make it more generic. Signed-off-by: Simon Glass --- drivers/misc/cros_ec_sandbox.c | 18 +++--- 1 file changed, 11 insertions(+), 7

Re: [PATCH 15/22] bloblist: Support resizing a blob

2021-07-17 Thread Simon Glass
Sometimes a blob needs to expand, e.g. because it needs to hold more log data. Add support for this. Note that the bloblist must have sufficient spare space for this to work. Signed-off-by: Simon Glass --- common/bloblist.c | 71 - include/bloblist.h | 13 +++

Re: [PATCH 16/22] bloblist: Tidy up a few API comments

2021-07-17 Thread Simon Glass
Some comments for struct bloblist_hdr are a bit ambiguous. Update them to clarify the meaning more precisely. Also document bloblist_get_stats() properly. Signed-off-by: Simon Glass --- include/bloblist.h | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) Applied to u-boot-dm,

Re: [PATCH 17/22] bloblist: Correct condition in bloblist_addrec()

2021-07-17 Thread Simon Glass
It is possible to add a blob that ends at the end of the bloblist, but at present this is not supported. Fix it and add a regression test for this case. Signed-off-by: Simon Glass --- common/bloblist.c | 2 +- test/bloblist.c | 23 +++ 2 files changed, 24 insertions(+),

Re: [PATCH 20/22] sandbox: mmc: Support fixed MMC devices

2021-07-17 Thread Simon Glass
On 7/6/21 7:32 AM, Simon Glass wrote: > Add support for reading devicetree flags for MMC devices. With this we > can distinguish between fixed and removable drives. Note that this > information is only available when the device is probed, not when it is > bound, since it is read in the

Re: [PATCH 19/22] spl: Provide more information on boot failure

2021-07-17 Thread Simon Glass
If SPL fails to boot, try to provide an error code to indicate what is wrong. For example, if a uclass is missing, this can return -EPFNOSUPPORT (-96) which provides useful information. Add a helper for accessing the image-loader name so we can drop the use of #ifdefs in this code. Put this

Re: [PATCH 22/22] log: Allow padding of the function name

2021-07-17 Thread Simon Glass
At present when function names are logged, the output is a little hard to read since every function is a different length. Add a way to pad the names so that the log messages line up vertically. This doesn't work if the function name is very long, but it makes a big difference in most cases. Use

Re: [PATCH 21/22] blk: Support iteration

2021-07-17 Thread Simon Glass
It is useful to be able to iterate over block devices. Typically there are fixed and removable devices. For security reasons it is sometimes useful to ignore removable devices since they are under user control. Add iterators which support selecting the block-device type. Signed-off-by: Simon

Re: [PATCH v3 01/19] image: Shorten FIT_ENABLE_SHAxxx_SUPPORT

2021-07-17 Thread Tom Rini
On Wed, Jul 14, 2021 at 05:05:29PM -0500, Alexandru Gagniuc wrote: > From: Simon Glass > > The ENABLE part of this name is redundant, since all boolean Kconfig > options serve to enable something. The SUPPORT part is also redundant > since Kconfigs can be assumed to enable support for

Re: Pull request: u-boot-imx u-boot-imx-20210717

2021-07-17 Thread Tom Rini
0) > > are available in the Git repository at: > > https://gitlab.denx.de/u-boot/custodians/u-boot-imx.git > tags/u-boot-imx-20210717 > > for you to fetch changes up to edf95bdeddeab8f0fd7b88d4865fbc6e99071c73: > > mx7ulp: wdog: Wait for WDOG unlock and reconfig

How to config ENV_IS_IN_NAND to default option in Kconfig?

2021-07-17 Thread 天田南
Hi all: I'm intend to set ENV_IS_IN_NAND to my default option in env/Kconfig. I can see the ENV_IS_IN_NAND is depends on !CHAIN_OF_TRUST, but I don't know where CHAIN_OF_TRUST is configured from. So where should I define this CHAIN_OF_TRUST in Kconfig? And What does CHAIN_OF_TRUST do? Thanks

[PATCH 3/3] doc: Update CapsuleUpdate READMEs

2021-07-17 Thread Ilias Apalodimas
Since we removed embeddingg the capsule key into a .dtb and fixed authenticated capsule updates for all boards, move the relevant documentation in the efi file and update it accordingly Signed-off-by: Ilias Apalodimas --- changes since v1: - fixed a typo

[PATCH 2/3] mkeficapsule: Remove dtb related options

2021-07-17 Thread Ilias Apalodimas
commit 322c813f4bec ("mkeficapsule: Add support for embedding public key in a dtb") added a bunch of options enabling the addition of the capsule public key in a dtb. Since now we embedded the key in U-Boot's .rodata we don't this this functionality anymore Signed-off-by: Ilias Apalodimas ---

[PATCH 1/3 v2] efi_capsule: Move signature from DTB to .rodata

2021-07-17 Thread Ilias Apalodimas
The capsule signature is now part of our DTB. This is problematic when a user is allowed to change/fixup that DTB from U-Boots command line since he can overwrite the signature as well. So Instead of adding the key on the DTB, embed it in the u-boot binary it self as part of it's .rodata. This

Re: [PATCH 1/3] efi_capsule: Move signature from DTB to .rodata

2021-07-17 Thread Ilias Apalodimas
On Sat, 17 Jul 2021 at 14:35, Ilias Apalodimas wrote: > > > > > [...] > > > > obj-$(CONFIG_EFI_HAVE_CAPSULE_SUPPORT) += efi_capsule.o > > > > +obj-$(CONFIG_EFI_CAPSULE_AUTHENTICATE) += efi_capsule_key.o > > > > We should give users another choice here to allow them to add their > > own

Pull request: u-boot-imx u-boot-imx-20210717

2021-07-17 Thread Stefano Babic
tags/u-boot-imx-20210717 for you to fetch changes up to edf95bdeddeab8f0fd7b88d4865fbc6e99071c73: mx7ulp: wdog: Wait for WDOG unlock and reconfiguration to complete (2021-07-17 14:59:56 +0200) i.MX - mx7ulp : fix WDOG

Re: [PATCH 1/9] net: eqos: Add PHY reset control for i.MX platform

2021-07-17 Thread Stefano Babic
Hi Ye, series was completely lost on my side, I tried to check it today but it is obsolete now and conflicts with Peng's series for MX8ULP. So I drop it, yorry, you should rebase and post it again. Best regards, Stefano Babic On 19.02.21 08:07, Ye Li wrote: Parse the "phy-reset-gpios",

Re: [PATCH V2 10/46] arm: imx8ulp: add container support

2021-07-17 Thread Stefano Babic
Hi peng, Ye, On 29.06.21 04:32, Peng Fan (OSS) wrote: From: Ye Li i.MX8ULP support using ROM API to load container image, it use same ROM API as i.MX8MN/MP, and use same container format as i.MX8QM/QXP. Signed-off-by: Ye Li Signed-off-by: Peng Fan ---

Re: [PATCH v2 4/5] imx: imx6ul: Add support for Kontron Electronics SL/BL i.MX6UL/ULL boards (N63xx/N64xx)

2021-07-17 Thread Stefano Babic
Hi Frieder, On 13.07.21 11:14, Frieder Schrempf wrote: From: Frieder Schrempf This adds support for i.MX6UL/ULL-based evaluation kits with SoMs by Kontron Electronics GmbH. Currently there are the following SoM flavors (SoM-Line): * N6310: SOM with i.MX6UL-2, 256MB RAM, 256MB SPI NAND

Re: [PATCH 1/3] efi_capsule: Move signature from DTB to .rodata

2021-07-17 Thread Ilias Apalodimas
On Fri, Jul 16, 2021 at 07:49:09AM -0600, Simon Glass wrote: > Hi Ilias, > > On Thu, 15 Jul 2021 at 11:00, Ilias Apalodimas > wrote: > > > > The capsule signature is now part of our DTB. This is problematic when a > > user is allowed to change/fixup that DTB from U-Boots command line since he >

Re: [PATCH 1/3] efi_capsule: Move signature from DTB to .rodata

2021-07-17 Thread Ilias Apalodimas
> > > [...] > > > +config EFI_CAPSULE_KEY_PATH > > > + string "Path to .esl file for capsule authentication" > > > + depends on EFI_CAPSULE_AUTHENTICATE > > > + help > > > + Provide the .esl file used for capsule authentication > > We might be friendly if we add what

Re: [PATCH 2/3] mkeficapsule: Remove dtb related options

2021-07-17 Thread Ilias Apalodimas
On Fri, Jul 16, 2021 at 08:03:23AM -0600, Simon Glass wrote: > Hi Ilias, > > On Thu, 15 Jul 2021 at 11:00, Ilias Apalodimas > wrote: > > > > commit 322c813f4bec ("mkeficapsule: Add support for embedding public key in > > a dtb") > > added a bunch of options enabling the addition of the capsule