Re: [PATCH v2 1/2] binman: Do not pollute source tree when build with `make O=...`

2021-12-02 Thread Andy Shevchenko
On Friday, December 3, 2021, Simon Glass wrote: > Hi Andy, > > On Tue, 30 Nov 2021 at 12:04, Andy Shevchenko > wrote: > > > > Importing libraries in Python caches the bytecode by default. > > Since we run scripts in source tree it ignores the current directory > > settings, which is $(srctree),

Re: [PATCH v6 00/25] fdt: Make OF_BOARD a boolean option

2021-12-02 Thread Ilias Apalodimas
Hi Simon, > > > > > [...] > > > > > Changes in v6: > > > > > - Fix description of OF_BOARD so it refers just to the current state > > > > > - Explain that the 'two devicetrees' refers to two *control* > > > > > devicetrees > > > > > - Expand the commit message based on comments > > > > > - Exp

Re: [PATCH v6 00/25] fdt: Make OF_BOARD a boolean option

2021-12-02 Thread Ilias Apalodimas
Hi Mark, > > > > > [...] > > > > > Changes in v6: > > > > > - Fix description of OF_BOARD so it refers just to the current state > > > > > - Explain that the 'two devicetrees' refers to two *control* > > > > > devicetrees > > > > > - Expand the commit message based on comments > > > > > - Expan

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

2021-12-02 Thread AKASHI Takahiro
Heinrich, On Tue, Nov 16, 2021 at 12:01:27PM +0900, AKASHI Takahiro wrote: > On Tue, Nov 16, 2021 at 01:02:55AM +0100, Heinrich Schuchardt wrote: > > On 11/16/21 00:51, AKASHI Takahiro wrote: > > > > > Is the patch good enough to include in the series? > > > > > > > > > > If not, you could reply

Re: [BUG] efi_loader: incorrect creation of device paths

2021-12-02 Thread AKASHI Takahiro
Heinrich, On Thu, Nov 25, 2021 at 02:44:28PM +0900, AKASHI Takahiro wrote: > Heinrich, > > On Wed, Nov 24, 2021 at 12:10:32PM +0900, AKASHI Takahiro wrote: > > On Sat, Nov 20, 2021 at 01:54:30PM +0100, Heinrich Schuchardt wrote: > > > Hello Takahiro, > > > > > > in a prior mail we have discussed

Re: [PATCH v7 00/12] efi_loader: capsule: improve capsule authentication support

2021-12-02 Thread AKASHI Takahiro
Heinrich, On Thu, Nov 25, 2021 at 03:02:35PM +0900, AKASHI Takahiro wrote: > Hi Heinrich > > On Tue, Nov 16, 2021 at 01:32:26PM +0900, AKASHI Takahiro wrote: > > As I proposed and discussed in [1] and [2], I have made a couple of > > improvements on the current implementation of capsule update in

[PULL] u-boot-riscv/master

2021-12-02 Thread Leo Liang
Hi Tom, The following changes since commit 4a14bfffd42f968ed9d72a780a8d44a9053c5b95: Merge https://source.denx.de/u-boot/custodians/u-boot-marvell (2021-11-30 08:59:22 -0500) are available in the Git repository at: https://source.denx.de/u-boot/custodians/u-boot-riscv.git for you to fetc

Re: [RESEND RFC PATCH 04/10] FWU: Add metadata access functions for GPT partitioned block devices

2021-12-02 Thread Sughosh Ganu
hi Simon, On Thu, 2 Dec 2021 at 19:04, Simon Glass wrote: > Hi Sughosh, > > On Thu, 2 Dec 2021 at 01:05, Sughosh Ganu wrote: > > > > hi Simon, > > > > On Wed, 1 Dec 2021 at 23:32, Simon Glass wrote: > >> > >> Hi Sughosh, > >> > >> On Thu, 25 Nov 2021 at 00:13, Sughosh Ganu > wrote: > >> > > >

Re: [PATCH v6 09/25] arm: xenguest_arm64: Add a fake devicetree file

2021-12-02 Thread Oleksandr Andrushchenko
Hi, Simon! On 02.12.21 19:57, Simon Glass wrote: > Hi Oleksandr, > > On Thu, 2 Dec 2021 at 10:40, Oleksandr Andrushchenko > wrote: >> Hi, Simon! >> >> Sorry for being late to the party >> >> On 02.12.21 17:59, Simon Glass wrote: >>> Add an empty file to prevent build errors when building with >>>

Re: [PATCH 01/15] i2c: muxes: pca954x: add PCA9847 variant

2021-12-02 Thread Heiko Schocher
Hello Vladimir, On 02.12.21 15:53, Vladimir Oltean wrote: > This seems to be very similar to the already existing PCA9547, save for > the fact that it supports 0.8V and doesn't support 5V. In fact, it is so > similar to the PCA9547 that the NXP LS1028A-RDB board has been driving > this chip using

[PATCH 3/3] efi_loader: correctly handle tcg2_measure_pe_image() error

2021-12-02 Thread Masahisa Kojima
When the TCG2 protocol is installed in efi_tcg2_register(), TPM2 device must be present. tcg2_measure_pe_image() expects that TCP2 protocol is installed and TPM device is available. If TCG2 Protocol is installed but TPM device is not found, tcg2_measure_pe_image() returns EFI_SECURITY_VIOLATION efi

[PATCH 2/3] efi_loader: check tcg2 protocol installation outside the TCG protocol

2021-12-02 Thread Masahisa Kojima
There are functions that calls tcg2_agile_log_append() outside of the TCG protocol invocation (e.g tcg2_measure_pe_image). These functions must to check that TCG2 protocol is installed. If not, measurement shall be skipped. Signed-off-by: Masahisa Kojima --- lib/efi_loader/efi_tcg2.c | 18 ++

[PATCH 0/3] fix TCG2 error handling

2021-12-02 Thread Masahisa Kojima
This series fix the efi_tcg2.c error handling. Masahisa Kojima (3): efi_loader: efi_tcg2_register returns appropriate error efi_loader: check tcg2 protocol installation outside the TCG protocol efi_loader: correctly handle tcg2_measure_pe_image() error include/efi_loader.h |

[PATCH 1/3] efi_loader: efi_tcg2_register returns appropriate error

2021-12-02 Thread Masahisa Kojima
This commit modify efi_tcg2_register() to return the appropriate error. With this fix, sandbox will not boot because efi_tcg2_register() fails due to some missing feature in GetCapabilities. So disable sandbox if EFI_TCG2_PROTOCOL is enabled. UEFI secure boot variable measurement is not directly r

Re: [PATCH 0/4] rockchip: Improve support for Bob chromebook and add support for Kevin

2021-12-02 Thread Simon Glass
Hi Peter, On Wed, 1 Dec 2021 at 07:23, Peter Robinson wrote: > > On Thu, Nov 25, 2021 at 5:39 PM Alper Nebi Yasak > wrote: > > > > I have recently started testing booting U-Boot from SPI on my gru-kevin > > (as opposed to chainloading it from vendor coreboot + depthcharge) and > > brought it to

Re: [PATCH 0/4] rockchip: Improve support for Bob chromebook and add support for Kevin

2021-12-02 Thread Simon Glass
Hi Alper, On Thu, 25 Nov 2021 at 10:39, Alper Nebi Yasak wrote: > > I have recently started testing booting U-Boot from SPI on my gru-kevin > (as opposed to chainloading it from vendor coreboot + depthcharge) and > brought it to a better working state based on an initial support patch > from Mart

Re: [PATCH u-boot-marvell RESEND 04/11] fdt_support: Remove FDT_STATUS_FAIL_ERROR_CODE

2021-12-02 Thread Simon Glass
On Fri, 26 Nov 2021 at 06:57, Marek Behún wrote: > > From: Marek Behún > > Since no one uses this feature and I am not aware of any parsers of this > in Linux, remove it. > > Signed-off-by: Marek Behún > Reviewed-by: Stefan Roese > Cc: Simon Glass > Cc: Andy Shevchenko > Cc: Pratyush Yadav >

Re: [PATCH 4/4] rockchip: rk3399: Add support for chromebook_kevin

2021-12-02 Thread Simon Glass
Hi Alper, On Thu, 25 Nov 2021 at 10:40, Alper Nebi Yasak wrote: > > From: "Marty E. Plummer" > > Add support for Kevin, an RK3399-based convertible chromebook that is > very similar to Bob. This patch is mostly based on existing support for > Bob, with only minor changes for Kevin-specific thing

Re: [PATCH u-boot-marvell RESEND 03/11] fdt_support: Remove fdt_alloc_phandle() in favor of fdt_generate_phandle()

2021-12-02 Thread Simon Glass
On Fri, 26 Nov 2021 at 06:57, Marek Behún wrote: > > From: Marek Behún > > Commit f0921f5098d ("fdt: Sync up to the latest libfdt") introduced > fdt_generate_phandle() in libfdt, making fdt_alloc_phandle() obsolete in > fdt_support. > > Signed-off-by: Marek Behún > Reviewed-by: Stefan Roese > C

Re: [PATCH 1/4] rockchip: gru: Set up SoC IO domain registers

2021-12-02 Thread Simon Glass
Hi Alper, On Thu, 25 Nov 2021 at 10:40, Alper Nebi Yasak wrote: > > The RK3399 SoC needs to know the voltage value provided by some > regulators, which is done by setting relevant register bits. Configure > these the way other RK3399 boards do, but with values set in coreboot. What do you mean b

Re: [PATCH v2 1/2] binman: Do not pollute source tree when build with `make O=...`

2021-12-02 Thread Simon Glass
Hi Andy, On Tue, 30 Nov 2021 at 12:04, Andy Shevchenko wrote: > > Importing libraries in Python caches the bytecode by default. > Since we run scripts in source tree it ignores the current directory > settings, which is $(srctree), and creates cache just in the middle > of the source tree. Move c

Re: [PATCH v2 2/2] binman: Use less hard coded magic when inserting new PATH

2021-12-02 Thread Simon Glass
Hi Andy, On Tue, 30 Nov 2021 at 12:04, Andy Shevchenko wrote: > > Instead of joining hard coded '..' to the run-time path of the executable, > take just a dirname out of it. Besides that, use $(srctree) where it makes > sense. > > Signed-off-by: Andy Shevchenko > --- > v2: new patch > tools/bin

Re: [RFC Patch v2] binman: add support for creating dummy files for external blobs

2021-12-02 Thread Simon Glass
Hi Heiko, On Mon, 29 Nov 2021 at 02:48, Heiko Thiery wrote: > > While converting to binman for an imx8mq board, it has been found that > building in the u-boot CI fails. This is because an imx8mq requires an > external binary (signed_hdmi_imx8m.bin). If this file cannot be found > mkimage fails.

[PATCH v2] RFC: gitlab: x86: Add a coreboot test

2021-12-02 Thread Simon Glass
Coreboot supports U-Boot as a payload and this recently got a bit of a facelist. Add a test for this. For now this uses a binary build of coreboot (v4.15). Future work could potentially build it from source, but we need to figure out the toolchain problems first, since coreboot uses its own toolch

[PATCH] RFC: gitlab: x86: Add a coreboot test

2021-12-02 Thread Simon Glass
Coreboot supports U-Boot as a payload and this recently got a bit of a facelist. Add a test for this. For now this uses a binary build of coreboot (v4.15). Future work could potentially build it from source, but we need to figure out the toolchain problems first, since coreboot uses its own toolch

[u-boot-test-hooks PATCH] travis-ci: Add tests for booting from coreboot

2021-12-02 Thread Simon Glass
Add a means of testing a coreboot + U-Boot build using qemu. Signed-off-by: Simon Glass --- bin/ellesmere/conf.coreboot_qemu | 1 + bin/travis-ci/conf.coreboot_qemu | 28 2 files changed, 29 insertions(+) create mode 12 bin/ellesmere/conf.coreboot_qemu create

Re: [PATCH v6 00/25] fdt: Make OF_BOARD a boolean option

2021-12-02 Thread Simon Glass
Hi Andre, On Thu, 2 Dec 2021 at 18:59, Andre Przywara wrote: > > On Thu, 2 Dec 2021 13:34:07 -0500 > Tom Rini wrote: > > Hi, > > > On Thu, Dec 02, 2021 at 11:17:38AM -0700, Simon Glass wrote: > > > Hi Tom, > > > > > > On Thu, 2 Dec 2021 at 11:03, Tom Rini wrote: > > > > > > > > On Thu, Dec 02,

Re: [PATCH v6 00/25] fdt: Make OF_BOARD a boolean option

2021-12-02 Thread Andre Przywara
On Thu, 2 Dec 2021 13:34:07 -0500 Tom Rini wrote: Hi, > On Thu, Dec 02, 2021 at 11:17:38AM -0700, Simon Glass wrote: > > Hi Tom, > > > > On Thu, 2 Dec 2021 at 11:03, Tom Rini wrote: > > > > > > On Thu, Dec 02, 2021 at 10:07:13AM -0700, Simon Glass wrote: > > > > Hi Tom, > > > > > > > > On

Re: [PATCH v6 00/25] fdt: Make OF_BOARD a boolean option

2021-12-02 Thread Simon Glass
Hi Andre, On Thu, 2 Dec 2021 at 18:31, Andre Przywara wrote: > > On Thu, 2 Dec 2021 11:17:38 -0700 > Simon Glass wrote: > > > Hi Tom, > > > > On Thu, 2 Dec 2021 at 11:03, Tom Rini wrote: > > > > > > On Thu, Dec 02, 2021 at 10:07:13AM -0700, Simon Glass wrote: > > > > Hi Tom, > > > > > > > > On

Re: [PATCH v6 00/25] fdt: Make OF_BOARD a boolean option

2021-12-02 Thread Andre Przywara
On Thu, 2 Dec 2021 11:17:38 -0700 Simon Glass wrote: > Hi Tom, > > On Thu, 2 Dec 2021 at 11:03, Tom Rini wrote: > > > > On Thu, Dec 02, 2021 at 10:07:13AM -0700, Simon Glass wrote: > > > Hi Tom, > > > > > > On Thu, 2 Dec 2021 at 09:59, Tom Rini wrote: > > > > > > > > On Thu, Dec 02, 2021 a

Re: [PATCH v6 00/25] fdt: Make OF_BOARD a boolean option

2021-12-02 Thread Andre Przywara
On Thu, 2 Dec 2021 19:03:46 +0200 Ilias Apalodimas wrote: Hi, > On Thu, 2 Dec 2021 at 18:38, Tom Rini wrote: > > > > On Thu, Dec 02, 2021 at 05:33:53PM +0100, François Ozog wrote: > > > Hi Simon > > > > > > Le jeu. 2 déc. 2021 à 17:00, Simon Glass a écrit : > > > > > > > With Ilias' effort

Re: [PATCH v6 00/25] fdt: Make OF_BOARD a boolean option

2021-12-02 Thread Andre Przywara
On Thu, 2 Dec 2021 09:39:52 -0700 Simon Glass wrote: Hi, wow, this thread quickly exploded, jumping in here randomly to add my thoughts: > Hi François, > > On Thu, 2 Dec 2021 at 09:34, François Ozog wrote: > > > > Hi Simon > > > > Le jeu. 2 déc. 2021 à 17:00, Simon Glass a écrit : > >> >

[PATCH 1/2] board/sunxi/dram_sun4i_auto: use DRAM_MEMORY_TYPE_DDR3 instead of magic number 3

2021-12-02 Thread Giulio Benetti
Since DRAM_MEMORY_TYPE_DDR3 is defined let's use it instead of magic number 3. Signed-off-by: Giulio Benetti --- board/sunxi/dram_sun4i_auto.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/board/sunxi/dram_sun4i_auto.c b/board/sunxi/dram_sun4i_auto.c index e8bbee4ee5..547d1

[PATCH 2/2] board/sunxi/dram_sun5i_auto: use DRAM_MEMORY_TYPE_DDR3 instead of magic number 3

2021-12-02 Thread Giulio Benetti
Since DRAM_MEMORY_TYPE_DDR3 is defined let's use it instead of magic number 3. Signed-off-by: Giulio Benetti --- board/sunxi/dram_sun5i_auto.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/board/sunxi/dram_sun5i_auto.c b/board/sunxi/dram_sun5i_auto.c index a5f4f8b743..51750

Re: [PATCH v6 00/25] fdt: Make OF_BOARD a boolean option

2021-12-02 Thread Simon Glass
Hi Tom, On Thu, 2 Dec 2021 at 15:47, Tom Rini wrote: > > On Thu, Dec 02, 2021 at 12:12:16PM -0700, Simon Glass wrote: > > Hi Tom, > > > > On Thu, 2 Dec 2021 at 11:34, Tom Rini wrote: > > > > > > On Thu, Dec 02, 2021 at 11:17:38AM -0700, Simon Glass wrote: > > > > Hi Tom, > > > > > > > > On Thu,

Re: [PATCH v6 00/25] fdt: Make OF_BOARD a boolean option

2021-12-02 Thread Simon Glass
Hi Tom, On Thu, 2 Dec 2021 at 15:53, Tom Rini wrote: > > On Thu, Dec 02, 2021 at 11:36:59PM +0100, François Ozog wrote: > > Hi Simon,Tom > > > > Le jeu. 2 déc. 2021 à 19:34, Tom Rini a écrit : > > > > > On Thu, Dec 02, 2021 at 11:17:38AM -0700, Simon Glass wrote: > > > > Hi Tom, > > > > > > > >

Re: [PATCH v6 00/25] fdt: Make OF_BOARD a boolean option

2021-12-02 Thread Tom Rini
On Thu, Dec 02, 2021 at 11:36:59PM +0100, François Ozog wrote: > Hi Simon,Tom > > Le jeu. 2 déc. 2021 à 19:34, Tom Rini a écrit : > > > On Thu, Dec 02, 2021 at 11:17:38AM -0700, Simon Glass wrote: > > > Hi Tom, > > > > > > On Thu, 2 Dec 2021 at 11:03, Tom Rini wrote: > > > > > > > > On Thu, De

Re: [PATCH v6 00/25] fdt: Make OF_BOARD a boolean option

2021-12-02 Thread Tom Rini
On Thu, Dec 02, 2021 at 12:12:16PM -0700, Simon Glass wrote: > Hi Tom, > > On Thu, 2 Dec 2021 at 11:34, Tom Rini wrote: > > > > On Thu, Dec 02, 2021 at 11:17:38AM -0700, Simon Glass wrote: > > > Hi Tom, > > > > > > On Thu, 2 Dec 2021 at 11:03, Tom Rini wrote: > > > > > > > > On Thu, Dec 02, 2021

Re: [PATCH v6 00/25] fdt: Make OF_BOARD a boolean option

2021-12-02 Thread Simon Glass
Hi Tom, On Thu, 2 Dec 2021 at 14:53, Tom Rini wrote: > > On Thu, Dec 02, 2021 at 12:24:24PM -0700, Simon Glass wrote: > > Hi Mark, > > > > On Thu, 2 Dec 2021 at 12:22, Mark Kettenis wrote: > > > > > > > From: Simon Glass > > > > Date: Thu, 2 Dec 2021 11:53:53 -0700 > > > > > > > > Hi Mark, > >

Re: [PATCH v6 00/25] fdt: Make OF_BOARD a boolean option

2021-12-02 Thread François Ozog
Hi Simon,Tom Le jeu. 2 déc. 2021 à 19:34, Tom Rini a écrit : > On Thu, Dec 02, 2021 at 11:17:38AM -0700, Simon Glass wrote: > > Hi Tom, > > > > On Thu, 2 Dec 2021 at 11:03, Tom Rini wrote: > > > > > > On Thu, Dec 02, 2021 at 10:07:13AM -0700, Simon Glass wrote: > > > > Hi Tom, > > > > > > > >

Re: [PATCH v6 00/25] fdt: Make OF_BOARD a boolean option

2021-12-02 Thread Tom Rini
On Thu, Dec 02, 2021 at 12:24:24PM -0700, Simon Glass wrote: > Hi Mark, > > On Thu, 2 Dec 2021 at 12:22, Mark Kettenis wrote: > > > > > From: Simon Glass > > > Date: Thu, 2 Dec 2021 11:53:53 -0700 > > > > > > Hi Mark, > > > > > > On Thu, 2 Dec 2021 at 11:47, Mark Kettenis > > > wrote: > > > >

Re: [PATCH 04/17] binman: Tidy up style in cmdline

2021-12-02 Thread Simon Glass
Update this file to improve the pylint score a little. The remaining item is: Function name "ParseArgs" doesn't conform to snake_case naming style which needs some binman-wide renaming. Signed-off-by: Simon Glass --- scripts/pylint.base | 2 +- tools/binman/cmdline.py | 45 +++

Re: [PATCH] drivers: core: lists: fix for loop index type

2021-12-02 Thread Simon Glass
On Sat, 6 Nov 2021 at 09:11, wrote: > > From: Alexander Preißner > > * fixes the bug in function bind_drivers_pass that for > CONFIG_CC_OPTIMIZE_FOR_SIZE=n and no entries in the driver_info list, > i.e. n_ents == 0, the processor steps into the first loop iteration > despite the loop condition be

Re: [PATCH 1/1] sandbox: replace putchar(ch) by fputc(ch, stdout)

2021-12-02 Thread Simon Glass
On Sat, 20 Nov 2021 at 05:28, Heinrich Schuchardt wrote: > > When compiled with -Og for better debugability u-boot ends up in a stack > overflow using > > gcc (Ubuntu 11.2.0-7ubuntu2) 11.2.0 > GNU Binutils for Ubuntu 2.37 > > putchar(ch) is defined as a macro which ends up calling U-Boot's

Re: [PATCH 01/17] dtoc: Bring in the libfdt module automatically

2021-12-02 Thread Simon Glass
Use the same technique as with binman to load this module from the U-Boot tree if available. This allows running tests without having to specify the PYTHONPATH variable. Signed-off-by: Simon Glass --- tools/dtoc/test_fdt.py | 6 ++ 1 file changed, 6 insertions(+) Applied to u-boot-dm/next,

Re: [PATCH 02/17] dtoc: Add support for reading 64-bit ints

2021-12-02 Thread Simon Glass
Add functions to read a 64-bit integer property from the devicetree. Signed-off-by: Simon Glass --- tools/dtoc/fdt_util.py | 35 tools/dtoc/test/dtoc_test_simple.dts | 1 + tools/dtoc/test_dtoc.py | 2 ++ tools/dtoc/test_fdt.py

Re: [PATCH 05/17] binman: Add a way to obtain the version

2021-12-02 Thread Simon Glass
Add a -V option which shows the version number of binman. For now this just uses a local 'version' file. Once the tool is packaged in some way we can figure out an approach that suits. Signed-off-by: Simon Glass --- tools/binman/cmdline.py | 29 + tools/binman/ftest.

Re: [PATCH 03/17] dtoc: Add support for reading fixed-length bytes properties

2021-12-02 Thread Simon Glass
Add functions to read a sequence of bytes from the devicetree. Signed-off-by: Simon Glass --- scripts/pylint.base| 4 ++-- tools/dtoc/fdt_util.py | 20 tools/dtoc/test_fdt.py | 17 + 3 files changed, 39 insertions(+), 2 deletions(-) Applied to u-boot-d

Re: [PATCH 06/17] binman: Correct init of entry in Entry class

2021-12-02 Thread Simon Glass
This should not have an underscore. Drop it so that derived classes can rely on it being set correctly. Signed-off-by: Simon Glass --- tools/binman/entry.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) Applied to u-boot-dm/next, thanks!

Re: [PATCH 08/17] binman: Drop the underscore in _ReadEntries()

2021-12-02 Thread Simon Glass
This function can be overridden so should not have an underscore. Drop it. Signed-off-by: Simon Glass --- tools/binman/etype/blob_phase.py | 2 +- tools/binman/etype/cbfs.py | 4 ++-- tools/binman/etype/files.py | 2 +- tools/binman/etype/section.py| 8 4 files changed,

Re: [PATCH 07/17] binman: Correct comments for ReadChildData()

2021-12-02 Thread Simon Glass
The comment here is incomplete. Fix it. Signed-off-by: Simon Glass --- tools/binman/entry.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) Applied to u-boot-dm/next, thanks!

Re: [PATCH 10/17] binman: Allow overriding BuildSectionData()

2021-12-02 Thread Simon Glass
This method is currently marked private. However it is useful to be able to subclass it, since much of the entry_Section code can be reused. Rename it. Also document one confusing part of this code, so people can understand how to add a test for this case. Fix up a few pylint warnings to avoid re

Re: [PATCH 11/17] binman: Allow control of which entries to read

2021-12-02 Thread Simon Glass
The ObtainContents() and GetEntryContents() methods in this file read every single entry in the section. This is the common case. However when one of the entries has had its data updated (e.g. with 'binman replace') we don't want to read it again from the file. Allow the entry to be skipped, for t

Re: [PATCH 09/17] binman: Drop the filename property in entry_Section

2021-12-02 Thread Simon Glass
This is not used and does nothing. Drop it. Add a tweak to avoid reducing the pylint score. Signed-off-by: Simon Glass --- tools/binman/etype/section.py | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) Applied to u-boot-dm/next, thanks!

Re: [PATCH 12/17] binman: Update the section documentation

2021-12-02 Thread Simon Glass
Expand this to explain subclassing better and also to tidy up formatting for rST. Fix a few pylint warnings to avoid dropping the score. Signed-off-by: Simon Glass --- tools/binman/entries.rst | 149 +++--- tools/binman/etype/section.py | 148 ++

Re: [PATCH 13/17] binman: Move cbfs.ObtainContents() down a bit

2021-12-02 Thread Simon Glass
It is easier to understand this file if reading the entries comes before obtaining the contents, since that is the order in which Binman proceeds. Move the function down a bit. Signed-off-by: Simon Glass --- tools/binman/etype/cbfs.py | 30 +++--- 1 file changed, 15 inse

Re: [PATCH 14/17] binman: Use normal entries in cbfs

2021-12-02 Thread Simon Glass
This currently uses _cbfs_entries[] to store entries. Since the entries are in fact valid etypes, we may as well use the same name as entry_Section uses, which is _entries. This allows reusing more of the code there (in a future patch). Signed-off-by: Simon Glass --- tools/binman/etype/cbfs.py

Re: [PATCH 16/17] binman: cfbs: Refactor ObtainContents() for consistency

2021-12-02 Thread Simon Glass
Update this to use the same arguments as entry_Section uses. Signed-off-by: Simon Glass --- tools/binman/etype/cbfs.py | 40 ++ 1 file changed, 23 insertions(+), 17 deletions(-) Applied to u-boot-dm/next, thanks!

Re: [PATCH 15/17] binman: cbfs: Refactor the init process

2021-12-02 Thread Simon Glass
Update the constructor to work in the recommended way, where the node properties are read in a separate function. This makes it more similar to entry_Section. Signed-off-by: Simon Glass --- tools/binman/etype/cbfs.py | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) Applied to u-bo

Re: [PATCH 17/17] binman: Rename testCbfsNoCOntents()

2021-12-02 Thread Simon Glass
Use a lower-case O as was intended. Signed-off-by: Simon Glass --- tools/binman/ftest.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) Applied to u-boot-dm/next, thanks!

Re: Question/issues about i.MX6 DDR configuration

2021-12-02 Thread Fabio Estevam
Hi Michael, On Thu, Dec 2, 2021 at 5:36 PM Michael Nazzareno Trimarchi wrote: > The bootrom loads the dcd using some logic and you write the register > in sequence. > You don't respect the ddr initialization or this delay on MMDC > according to 44.4.2. > Is that not necessary? I don't see in 44

Re: Question/issues about i.MX6 DDR configuration

2021-12-02 Thread Michael Nazzareno Trimarchi
Hi Fabio On Thu, Dec 2, 2021 at 9:14 PM Fabio Estevam wrote: > > Hi Francesco, > > On Thu, Dec 2, 2021 at 1:14 PM Francesco Dolcini > wrote: > > > > Hello Fabio, Tim and all, > > in the last few weeks I have been debugging some sporadic i.MX6 board > > boot failures (2020.07 U-Boot, if that matt

Re: Question/issues about i.MX6 DDR configuration

2021-12-02 Thread Fabio Estevam
Hi Francesco, On Thu, Dec 2, 2021 at 1:14 PM Francesco Dolcini wrote: > > Hello Fabio, Tim and all, > in the last few weeks I have been debugging some sporadic i.MX6 board > boot failures (2020.07 U-Boot, if that matters) and we have some loose > indication that they could be RAM related. The eff

Re: Please pull u-boot-net/master

2021-12-02 Thread Tom Rini
On Thu, Dec 02, 2021 at 10:22:52AM +0200, Ramon Fried wrote: > Hi Tom, > The below contains two fixes from Marek for designware and mdio. > > The following changes since commit 4a14bfffd42f968ed9d72a780a8d44a9053c5b95: > > Merge https://source.denx.de/u-boot/custodians/u-boot-marvell > (2021-1

[PATCH v3] xilinx: Kconfig: add XILINX_OF_BOARD_DTB_ADDR default value for microblaze

2021-12-02 Thread Ovidiu Panait
The xilinx board_fdt_blob_setup() implementation makes use of XILINX_OF_BOARD_DTB_ADDR, but no default value is currently defined for microblaze. Add one so that microblaze could also work with CONFIG_OF_SEPARATE. Signed-off-by: Ovidiu Panait --- board/xilinx/Kconfig | 1 + 1 file changed, 1 in

Re: [PATCH v6 00/25] fdt: Make OF_BOARD a boolean option

2021-12-02 Thread Simon Glass
Hi Mark, On Thu, 2 Dec 2021 at 12:22, Mark Kettenis wrote: > > > From: Simon Glass > > Date: Thu, 2 Dec 2021 11:53:53 -0700 > > > > Hi Mark, > > > > On Thu, 2 Dec 2021 at 11:47, Mark Kettenis wrote: > > > > > > > From: Simon Glass > > > > Date: Thu, 2 Dec 2021 11:21:22 -0700 > > > > > > > > Hi

Re: [PATCH v6 09/25] arm: xenguest_arm64: Add a fake devicetree file

2021-12-02 Thread Simon Glass
Hi François, On Thu, 2 Dec 2021 at 11:44, François Ozog wrote: > > Hi Simon > > On Thu, 2 Dec 2021 at 19:29, Simon Glass wrote: >> >> Hi François, >> >> On Thu, 2 Dec 2021 at 11:17, François Ozog wrote: >> > >> > Hi Simon >> > >> > On Thu, 2 Dec 2021 at 19:05, Simon Glass wrote: >> >> >> >> Hi

Re: [PATCH v6 00/25] fdt: Make OF_BOARD a boolean option

2021-12-02 Thread Mark Kettenis
> From: Simon Glass > Date: Thu, 2 Dec 2021 11:53:53 -0700 > > Hi Mark, > > On Thu, 2 Dec 2021 at 11:47, Mark Kettenis wrote: > > > > > From: Simon Glass > > > Date: Thu, 2 Dec 2021 11:21:22 -0700 > > > > > > Hi Mark, > > > > > > On Thu, 2 Dec 2021 at 11:15, Mark Kettenis > > > wrote: > > >

[PATCH] mkimage: fix segfault on MacOS arm64

2021-12-02 Thread Sergey V. Lobanov
mkimage segfaults due ASLR mechasim on MacOS arm64 It is required to use _dyld_get_image_vmaddr_slide() to prevent segfault on MacOS arm64 This patch ased on the discussion https://github.com/u-boot/u-boot/commit/3b142045e8a7f0ab17b6099e9226296af45967d0 Thanks to Ronny Kotzschmar and ptpt52 gith

Re: [PATCH v6 00/25] fdt: Make OF_BOARD a boolean option

2021-12-02 Thread Simon Glass
Hi Tom, On Thu, 2 Dec 2021 at 11:34, Tom Rini wrote: > > On Thu, Dec 02, 2021 at 11:17:38AM -0700, Simon Glass wrote: > > Hi Tom, > > > > On Thu, 2 Dec 2021 at 11:03, Tom Rini wrote: > > > > > > On Thu, Dec 02, 2021 at 10:07:13AM -0700, Simon Glass wrote: > > > > Hi Tom, > > > > > > > > On Thu,

Re: [PATCH v6 00/25] fdt: Make OF_BOARD a boolean option

2021-12-02 Thread Mark Kettenis
> From: François Ozog > Date: Thu, 2 Dec 2021 19:32:17 +0100 > > On Thu, 2 Dec 2021 at 19:15, Mark Kettenis wrote: > > > From: Ilias Apalodimas > > Date: Thu, 2 Dec 2021 19:03:46 +0200 > > > > On Thu, 2 Dec 2021 at 18:38, Tom Rini wrote: > > > > > > On Thu, Dec 02, 2021 at 05:33:53PM +

Re: [PATCH v6 00/25] fdt: Make OF_BOARD a boolean option

2021-12-02 Thread Simon Glass
Hi Mark, On Thu, 2 Dec 2021 at 11:47, Mark Kettenis wrote: > > > From: Simon Glass > > Date: Thu, 2 Dec 2021 11:21:22 -0700 > > > > Hi Mark, > > > > On Thu, 2 Dec 2021 at 11:15, Mark Kettenis wrote: > > > > > > > From: Ilias Apalodimas > > > > Date: Thu, 2 Dec 2021 19:03:46 +0200 > > > > > > >

Re: [PATCH v6 00/25] fdt: Make OF_BOARD a boolean option

2021-12-02 Thread Mark Kettenis
> From: Simon Glass > Date: Thu, 2 Dec 2021 11:21:22 -0700 > > Hi Mark, > > On Thu, 2 Dec 2021 at 11:15, Mark Kettenis wrote: > > > > > From: Ilias Apalodimas > > > Date: Thu, 2 Dec 2021 19:03:46 +0200 > > > > > > On Thu, 2 Dec 2021 at 18:38, Tom Rini wrote: > > > > > > > > On Thu, Dec 02, 20

Re: [PATCH v6 09/25] arm: xenguest_arm64: Add a fake devicetree file

2021-12-02 Thread François Ozog
Hi Simon On Thu, 2 Dec 2021 at 19:29, Simon Glass wrote: > Hi François, > > On Thu, 2 Dec 2021 at 11:17, François Ozog > wrote: > > > > Hi Simon > > > > On Thu, 2 Dec 2021 at 19:05, Simon Glass wrote: > >> > >> Hi Tom, > >> > >> On Thu, 2 Dec 2021 at 10:56, Tom Rini wrote: > >> > > >> > On Th

Re: [PATCH v6 00/25] fdt: Make OF_BOARD a boolean option

2021-12-02 Thread Tom Rini
On Thu, Dec 02, 2021 at 11:21:22AM -0700, Simon Glass wrote: > Hi Mark, > > On Thu, 2 Dec 2021 at 11:15, Mark Kettenis wrote: > > > > > From: Ilias Apalodimas > > > Date: Thu, 2 Dec 2021 19:03:46 +0200 > > > > > > On Thu, 2 Dec 2021 at 18:38, Tom Rini wrote: > > > > > > > > On Thu, Dec 02, 2021

Re: [PATCH v6 00/25] fdt: Make OF_BOARD a boolean option

2021-12-02 Thread Tom Rini
On Thu, Dec 02, 2021 at 11:17:38AM -0700, Simon Glass wrote: > Hi Tom, > > On Thu, 2 Dec 2021 at 11:03, Tom Rini wrote: > > > > On Thu, Dec 02, 2021 at 10:07:13AM -0700, Simon Glass wrote: > > > Hi Tom, > > > > > > On Thu, 2 Dec 2021 at 09:59, Tom Rini wrote: > > > > > > > > On Thu, Dec 02, 2021

Re: [PATCH v6 00/25] fdt: Make OF_BOARD a boolean option

2021-12-02 Thread François Ozog
On Thu, 2 Dec 2021 at 19:15, Mark Kettenis wrote: > > From: Ilias Apalodimas > > Date: Thu, 2 Dec 2021 19:03:46 +0200 > > > > On Thu, 2 Dec 2021 at 18:38, Tom Rini wrote: > > > > > > On Thu, Dec 02, 2021 at 05:33:53PM +0100, François Ozog wrote: > > > > Hi Simon > > > > > > > > Le jeu. 2 déc. 2

Re: [PATCH v6 09/25] arm: xenguest_arm64: Add a fake devicetree file

2021-12-02 Thread Simon Glass
Hi François, On Thu, 2 Dec 2021 at 11:17, François Ozog wrote: > > Hi Simon > > On Thu, 2 Dec 2021 at 19:05, Simon Glass wrote: >> >> Hi Tom, >> >> On Thu, 2 Dec 2021 at 10:56, Tom Rini wrote: >> > >> > On Thu, Dec 02, 2021 at 05:40:46PM +, Oleksandr Andrushchenko wrote: >> > > Hi, Simon! >

Re: [PATCH v6 07/25] arm: rpi: Add a devicetree file for rpi_4

2021-12-02 Thread Simon Glass
Hi Mark, On Thu, 2 Dec 2021 at 10:34, Mark Kettenis wrote: > > > From: Simon Glass > > Date: Thu, 2 Dec 2021 08:59:01 -0700 > > > > Add this file, obtained from the Raspbian boot disk, so there is a > > reference devicetree in the U-Boot tree. The same one is used for > > 32- and 64-bit variant

[PATCH v4 3/6] ARM: dts: sync Actions Semi S700 DT from Linux v5.16-rc3

2021-12-02 Thread Amit Singh Tomar
From: Amit Singh Tomar This Synchronizes the Actions Semi S700 SoC DT changes from commit "g58e1100fdc59" ("Linux v5.16-rc3"). Signed-off-by: Amit Singh Tomar --- arch/arm/dts/s700.dtsi| 17 - .../dt-bindings/power/owl-s700-powergate.h| 19 ++

Re: [PATCH v6 00/25] fdt: Make OF_BOARD a boolean option

2021-12-02 Thread Simon Glass
Hi Mark, On Thu, 2 Dec 2021 at 11:15, Mark Kettenis wrote: > > > From: Ilias Apalodimas > > Date: Thu, 2 Dec 2021 19:03:46 +0200 > > > > On Thu, 2 Dec 2021 at 18:38, Tom Rini wrote: > > > > > > On Thu, Dec 02, 2021 at 05:33:53PM +0100, François Ozog wrote: > > > > Hi Simon > > > > > > > > Le je

Re: [PATCH v6 07/25] arm: rpi: Add a devicetree file for rpi_4

2021-12-02 Thread François Ozog
Hi Mark, On Thu, 2 Dec 2021 at 18:34, Mark Kettenis wrote: > > From: Simon Glass > > Date: Thu, 2 Dec 2021 08:59:01 -0700 > > > > Add this file, obtained from the Raspbian boot disk, so there is a > > reference devicetree in the U-Boot tree. The same one is used for > > 32- and 64-bit variants

Re: [PATCH v6 00/25] fdt: Make OF_BOARD a boolean option

2021-12-02 Thread Simon Glass
Hi Tom, On Thu, 2 Dec 2021 at 11:03, Tom Rini wrote: > > On Thu, Dec 02, 2021 at 10:07:13AM -0700, Simon Glass wrote: > > Hi Tom, > > > > On Thu, 2 Dec 2021 at 09:59, Tom Rini wrote: > > > > > > On Thu, Dec 02, 2021 at 09:49:51AM -0700, Simon Glass wrote: > > > > Hi Tom, > > > > > > > > On Thu,

Re: [PATCH v6 09/25] arm: xenguest_arm64: Add a fake devicetree file

2021-12-02 Thread François Ozog
Hi Simon On Thu, 2 Dec 2021 at 19:05, Simon Glass wrote: > Hi Tom, > > On Thu, 2 Dec 2021 at 10:56, Tom Rini wrote: > > > > On Thu, Dec 02, 2021 at 05:40:46PM +, Oleksandr Andrushchenko wrote: > > > Hi, Simon! > > > > > > Sorry for being late to the party > > > > > > On 02.12.21 17:59, Simo

Re: [PATCH v6 00/25] fdt: Make OF_BOARD a boolean option

2021-12-02 Thread Mark Kettenis
> From: Ilias Apalodimas > Date: Thu, 2 Dec 2021 19:03:46 +0200 > > On Thu, 2 Dec 2021 at 18:38, Tom Rini wrote: > > > > On Thu, Dec 02, 2021 at 05:33:53PM +0100, François Ozog wrote: > > > Hi Simon > > > > > > Le jeu. 2 déc. 2021 à 17:00, Simon Glass a écrit : > > > > > > > With Ilias' efforts

Re: [PATCH v6 09/25] arm: xenguest_arm64: Add a fake devicetree file

2021-12-02 Thread Tom Rini
On Thu, Dec 02, 2021 at 11:05:16AM -0700, Simon Glass wrote: > Hi Tom, > > On Thu, 2 Dec 2021 at 10:56, Tom Rini wrote: > > > > On Thu, Dec 02, 2021 at 05:40:46PM +, Oleksandr Andrushchenko wrote: > > > Hi, Simon! > > > > > > Sorry for being late to the party > > > > > > On 02.12.21 17:59, Si

Re: [PATCH v6 09/25] arm: xenguest_arm64: Add a fake devicetree file

2021-12-02 Thread Simon Glass
Hi Tom, On Thu, 2 Dec 2021 at 10:56, Tom Rini wrote: > > On Thu, Dec 02, 2021 at 05:40:46PM +, Oleksandr Andrushchenko wrote: > > Hi, Simon! > > > > Sorry for being late to the party > > > > On 02.12.21 17:59, Simon Glass wrote: > > > Add an empty file to prevent build errors when building wi

Re: [PATCH v6 00/25] fdt: Make OF_BOARD a boolean option

2021-12-02 Thread Tom Rini
On Thu, Dec 02, 2021 at 10:07:13AM -0700, Simon Glass wrote: > Hi Tom, > > On Thu, 2 Dec 2021 at 09:59, Tom Rini wrote: > > > > On Thu, Dec 02, 2021 at 09:49:51AM -0700, Simon Glass wrote: > > > Hi Tom, > > > > > > On Thu, 2 Dec 2021 at 09:38, Tom Rini wrote: > > > > > > > > On Thu, Dec 02, 2021

Re: [PATCH v6 09/25] arm: xenguest_arm64: Add a fake devicetree file

2021-12-02 Thread Simon Glass
Hi Oleksandr, On Thu, 2 Dec 2021 at 10:40, Oleksandr Andrushchenko wrote: > > Hi, Simon! > > Sorry for being late to the party > > On 02.12.21 17:59, Simon Glass wrote: > > Add an empty file to prevent build errors when building with > > CONFIG_OF_SEPARATE enabled. > > > > The build instructions

Re: [PATCH v6 09/25] arm: xenguest_arm64: Add a fake devicetree file

2021-12-02 Thread Tom Rini
On Thu, Dec 02, 2021 at 05:40:46PM +, Oleksandr Andrushchenko wrote: > Hi, Simon! > > Sorry for being late to the party > > On 02.12.21 17:59, Simon Glass wrote: > > Add an empty file to prevent build errors when building with > > CONFIG_OF_SEPARATE enabled. > > > > The build instructions in

Re: [PATCH v6 09/25] arm: xenguest_arm64: Add a fake devicetree file

2021-12-02 Thread Oleksandr Andrushchenko
Hi, Simon! Sorry for being late to the party On 02.12.21 17:59, Simon Glass wrote: > Add an empty file to prevent build errors when building with > CONFIG_OF_SEPARATE enabled. > > The build instructions in U-Boot do not provide enough detail to build a > useful devicetree, unfortunately. Xen gues

Re: [PATCH v6 07/25] arm: rpi: Add a devicetree file for rpi_4

2021-12-02 Thread Mark Kettenis
> From: Simon Glass > Date: Thu, 2 Dec 2021 08:59:01 -0700 > > Add this file, obtained from the Raspbian boot disk, so there is a > reference devicetree in the U-Boot tree. The same one is used for > 32- and 64-bit variants. > > Note that U-Boot does not normally need this at runtime, since > C

Re: [PATCH v6 00/25] fdt: Make OF_BOARD a boolean option

2021-12-02 Thread Simon Glass
Hi Ilias, On Thu, 2 Dec 2021 at 10:04, Ilias Apalodimas wrote: > > On Thu, 2 Dec 2021 at 18:38, Tom Rini wrote: > > > > On Thu, Dec 02, 2021 at 05:33:53PM +0100, François Ozog wrote: > > > Hi Simon > > > > > > Le jeu. 2 déc. 2021 à 17:00, Simon Glass a écrit : > > > > > > > With Ilias' efforts

Re: [PATCH v6 00/25] fdt: Make OF_BOARD a boolean option

2021-12-02 Thread Simon Glass
Hi Tom, On Thu, 2 Dec 2021 at 09:59, Tom Rini wrote: > > On Thu, Dec 02, 2021 at 09:49:51AM -0700, Simon Glass wrote: > > Hi Tom, > > > > On Thu, 2 Dec 2021 at 09:38, Tom Rini wrote: > > > > > > On Thu, Dec 02, 2021 at 05:33:53PM +0100, François Ozog wrote: > > > > Hi Simon > > > > > > > > Le je

Re: [PATCH v6 00/25] fdt: Make OF_BOARD a boolean option

2021-12-02 Thread Ilias Apalodimas
On Thu, 2 Dec 2021 at 18:38, Tom Rini wrote: > > On Thu, Dec 02, 2021 at 05:33:53PM +0100, François Ozog wrote: > > Hi Simon > > > > Le jeu. 2 déc. 2021 à 17:00, Simon Glass a écrit : > > > > > With Ilias' efforts we have dropped OF_PRIOR_STAGE and OF_HOSTFILE so > > > there are only three ways t

Re: [PATCH v6 03/25] arm: riscv: qemu: Explain how to extract the generated dt

2021-12-02 Thread Tom Rini
On Thu, Dec 02, 2021 at 05:30:20PM +0100, Heinrich Schuchardt wrote: > On 12/2/21 16:58, Simon Glass wrote: > > QEMU currently generates a devicetree for use with U-Boot. Explain how to > > obtain it. > > > > Also explain how to merge it to produce a devicetree with the U-Boot > > features include

Re: [PATCH v6 00/25] fdt: Make OF_BOARD a boolean option

2021-12-02 Thread Tom Rini
On Thu, Dec 02, 2021 at 09:49:51AM -0700, Simon Glass wrote: > Hi Tom, > > On Thu, 2 Dec 2021 at 09:38, Tom Rini wrote: > > > > On Thu, Dec 02, 2021 at 05:33:53PM +0100, François Ozog wrote: > > > Hi Simon > > > > > > Le jeu. 2 déc. 2021 à 17:00, Simon Glass a écrit : > > > > > > > With Ilias' e

Re: [PATCH v6 00/25] fdt: Make OF_BOARD a boolean option

2021-12-02 Thread François Ozog
Hi Simon Le jeu. 2 déc. 2021 à 17:40, Simon Glass a écrit : > Hi François, > > On Thu, 2 Dec 2021 at 09:34, François Ozog > wrote: > > > > Hi Simon > > > > Le jeu. 2 déc. 2021 à 17:00, Simon Glass a écrit : > >> > >> With Ilias' efforts we have dropped OF_PRIOR_STAGE and OF_HOSTFILE so > >> th

Re: [PATCH v6 00/25] fdt: Make OF_BOARD a boolean option

2021-12-02 Thread Peter Maydell
On Thu, 2 Dec 2021 at 16:40, Simon Glass wrote: > The word 'fake' applies to only three of the boards (highbank, bcm7xxx > and octeontx), where I could not even figure out where to get a > devicetree. One might think this would be a significant problem! Isn't highbank https://github.com/torvalds/

Re: [PATCH v6 03/25] arm: riscv: qemu: Explain how to extract the generated dt

2021-12-02 Thread Simon Glass
Hi Heinrich, On Thu, 2 Dec 2021 at 09:35, Heinrich Schuchardt wrote: > > On 12/2/21 16:58, Simon Glass wrote: > > QEMU currently generates a devicetree for use with U-Boot. Explain how to > > obtain it. > > > > Also explain how to merge it to produce a devicetree with the U-Boot > > features incl

  1   2   >