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

2021-12-02 Thread Simon Glass
With Ilias' efforts we have dropped OF_PRIOR_STAGE and OF_HOSTFILE so there are only three ways to obtain a devicetree: - OF_SEPARATE - the normal way, where the devicetree is built and appended to U-Boot - OF_EMBED - for development purposes, the devicetree is embedded in the

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

2021-12-02 Thread Simon Glass
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 included. Signed-off-by: Simon Glass --- (no changes since v5) Changes in v5: - Merge RISC-V and ARM patches since they are

[PATCH v6 02/25] arm: qemu: Mention -nographic in the docs

2021-12-02 Thread Simon Glass
Without this option QEMU appears to hang. Add it to avoid confusion. Signed-off-by: Simon Glass --- (no changes since v1) doc/board/emulation/qemu-arm.rst | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/doc/board/emulation/qemu-arm.rst

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

Re: [PATCH] mtd: sf: Set SF parameters as env variables

2021-12-02 Thread Sean Anderson
On 10/7/21 8:40 AM, Pratyush Yadav wrote: On 23/09/21 10:00PM, Marek Vasut wrote: On 9/23/21 8:53 PM, Pratyush Yadav wrote: > On 14/09/21 05:28AM, Marek Vasut wrote: > > Set the SF page size, erase block size and total size as an environment > > variable after "sf probe". This lets us

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 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.

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: 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 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: > > >

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: >> >> >> >>

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 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 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 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

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

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 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 > > > > > > > >

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 >

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

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] 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

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 +

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 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

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

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

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

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 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, > > > > > > >

[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

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 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

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: [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

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 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: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,

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: [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 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,

[PATCH v6 19/25] fdt: Drop #ifdefs with MULTI_DTB_FIT

2021-12-02 Thread Simon Glass
Refactor the code to drop the #ifdefs for this feature. Signed-off-by: Simon Glass --- (no changes since v1) dts/Kconfig | 1 - include/asm-generic/global_data.h | 8 lib/fdtdec.c | 31 +++ 3 files changed, 19

[PATCH v6 20/25] fdt: Drop CONFIG_SPL_BUILD check in fdtdec_setup()

2021-12-02 Thread Simon Glass
Move this to the header file to clean up the C code. Signed-off-by: Simon Glass --- (no changes since v1) include/fdtdec.h | 14 ++ lib/fdtdec.c | 6 +- 2 files changed, 15 insertions(+), 5 deletions(-) diff --git a/include/fdtdec.h b/include/fdtdec.h index

[PATCH v6 25/25] fdt: Don't call board_fdt_blob_setup() without OF_BOARD

2021-12-02 Thread Simon Glass
At present this override function is called even when OF_BOARD Is not enabled. This makes it impossible to disable this feature and in fact makes the OF_BOARD option useless. Reinstate its intended purpose, so that it is possible to switch between the appended devicetree and one provided by the

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

2021-12-02 Thread Heinrich Schuchardt
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 included. Information like the number of CPUs and the memory must be accurate in the

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

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

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,

[PATCH 08/15] rtc: pcf2127: remove U-Boot specific compatible string

2021-12-02 Thread Vladimir Oltean
Now that all in-tree boards have been converted to the compatible strings from Linux, delete the support for the ad-hoc "pcf2127-rtc" one. Cc: Simon Glass Signed-off-by: Vladimir Oltean --- drivers/rtc/pcf2127.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/rtc/pcf2127.c

[PATCH 15/15] arm: dts: ls1028a-qds: declare in-band autoneg for Ethernet ports

2021-12-02 Thread Vladimir Oltean
As explained in the previous patch "arm: dts: ls1028a-rdb: sync Ethernet device tree nodes with Linux", the commit below broke traffic through switch ports where the SERDES protocol requires in-band autoneg: SGMII, QSGMII, USXGMII (with 2500Base-X, in-band autoneg isn't supported). The

[PATCH 11/15] arm: dts: ls1028a-rdb: disable DSPI nodes

2021-12-02 Thread Vladimir Oltean
There is no SPI peripheral on the LS1028A-RDB, therefore no reason to enable these nodes in the U-Boot device tree (and Linux does not enable them either). Signed-off-by: Vladimir Oltean --- arch/arm/dts/fsl-ls1028a-rdb.dts | 12 1 file changed, 12 deletions(-) diff --git

[PATCH 14/15] arm: dts: ls1028a-rdb: sync device tree with Linux

2021-12-02 Thread Vladimir Oltean
Allow device trees to be reused between Linux and U-Boot. The source for these device trees is today's linux-next plus these changes that have been made so that the sources can be shared. These other patches are currently in flight:

[PATCH 12/15] arm: dts: ls1028a-rdb: disable I2C buses 1 through 7

2021-12-02 Thread Vladimir Oltean
There is no I2C peripheral on these buses on the reference design board, and the Linux device tree does not enable them either. Signed-off-by: Vladimir Oltean --- arch/arm/dts/fsl-ls1028a-rdb.dts | 28 1 file changed, 28 deletions(-) diff --git

[PATCH 13/15] arm: dts: ls1028a-rdb: enable PCIe controllers from U-Boot dtsi

2021-12-02 Thread Vladimir Oltean
Reuse the scheme implemented by the Kontron SL28 boards in commit d08011d7f9b4 ("arm: dts: ls1028a: disable the PCIe controller by default") and move the 'status = "okay"' lines for the PCIe controllers inside a separate U-Boot dtsi for the LS1028A-RDB board. This way, the existing Linux device

Re: [PATCH] pci: When disabling pref MEM set all base bits

2021-12-02 Thread Stefan Roese
On 12/2/21 00:12, Pali Rohár wrote: On Tuesday 30 November 2021 07:10:37 Stefan Roese wrote: On 11/25/21 11:34, Pali Rohár wrote: It is common to set all base address bits to one and all limit address bits to zero for disabling address forwarding. Forwarding is disabled when base address is

[PATCH v6 01/25] doc: Add documentation about devicetree usage

2021-12-02 Thread Simon Glass
At present some of the ideas and techniques behind devicetree in U-Boot are assumed, implied or unsaid. Add some documentation to cover how devicetree is build, how it can be modified and the rules about using the various CONFIG_OF_... options. Signed-off-by: Simon Glass Reviewed-by: Marcel

[PATCH v6 06/25] riscv: qemu: Add devicetree files for qemu_riscv32/64

2021-12-02 Thread Simon Glass
Add these files, generated from qemu, so there is a reference devicetree in the U-Boot tree. Split the existing qemu-virt into two, since we need a different devicetree for 32- and 64-bit machines. Signed-off-by: Simon Glass --- (no changes since v1) arch/riscv/dts/Makefile |

[PATCH v6 04/25] arm: qemu: Add a devicetree file for qemu_arm

2021-12-02 Thread Simon Glass
Add this file, generated from QEMU, so there is a reference devicetree in the U-Boot tree. So long as OF_BOARD is enabled, it is overridden at runtime by the QEMU version. However, with this change it becomes possible to run with the U-Boot devicetree, if desired, including any local

[PATCH v6 05/25] arm: qemu: Add a devicetree file for qemu_arm64

2021-12-02 Thread Simon Glass
Add this file, generated from QEMU, so there is a reference devicetree in the U-Boot tree. So long as OF_BOARD is enabled, it is overridden at runtime by the QEMU version. However, with this change it becomes possible to run with the U-Boot devicetree, if desired, including any local

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:00, Simon Glass a écrit : > With Ilias' efforts we have dropped OF_PRIOR_STAGE and OF_HOSTFILE so > there are only three ways to obtain a devicetree: > >- OF_SEPARATE - the normal way, where the devicetree is built and > appended to U-Boot >-

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

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 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,

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,

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,

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

[PATCH 00/15] Sync NXP LS1028A-RDB device trees between U-Boot and Linux

2021-12-02 Thread Vladimir Oltean
The changes were intended to be minimal, but unfortunately I discovered some other stuff as well: - we need to make some changes to the compatible strings of RTC devices and I2C muxes. This has ramifications to other NXP boards which were also updated. - I broke Ethernet on LS1028A boards

[PATCH 05/15] arm: ls1088a-rdb: use Linux compatible string for RTC

2021-12-02 Thread Vladimir Oltean
During the LS1028A-RDB sync with Linux device trees, it was observed that the same RTC is present on the two boards, and the wrong compatible string is used in both places. This change updates the RTC from the LS1088A-RDB to use the compatible string that was established in Linux. Signed-off-by:

[PATCH 06/15] arm: lx2160a-qds: use Linux compatible string for RTC

2021-12-02 Thread Vladimir Oltean
During the LS1028A-RDB sync with Linux device trees, it was observed that the same RTC is present on the two boards, and the wrong compatible string is used in both places. This change updates the RTC from the LX2160A-QDS to use the compatible string that was established in Linux. Signed-off-by:

[PATCH 10/15] arm: dts: ls1028a-rdb: sync Ethernet device tree nodes with Linux

2021-12-02 Thread Vladimir Oltean
In a bit of a blunder, the blamed commit made the mscc_felix switch driver look at the 'managed = "in-band-status"' device tree property, forgetting that the U-Boot device tree had not been updated to include that property, whereas the Linux one does. The switch is therefore described in the

[PATCH 07/15] arm: dts: lx2160a-rdb: use Linux compatible string for RTC

2021-12-02 Thread Vladimir Oltean
During the LS1028A-RDB sync with Linux device trees, it was observed that the same RTC is present on the two boards, and the wrong compatible string is used in both places. This change updates the RTC from the LX2160A-RDB to use the compatible string that was established in Linux. Signed-off-by:

[PATCH 09/15] arm: dts: ls1028a-rdb: sort nodes alphabetically

2021-12-02 Thread Vladimir Oltean
The nodes in the NXP LS1028A-RDB device tree are out of order, regroup them alphabetically to have a simple delta when the Linux device tree is brought in. Signed-off-by: Vladimir Oltean --- arch/arm/dts/fsl-ls1028a-rdb.dts | 110 +++ 1 file changed, 55

[PATCH v6 15/25] fdt: Make OF_BOARD a bool option

2021-12-02 Thread Simon Glass
This should not be a separate option from OF_SEPARATE. It is a run-time option to override the devicetree, even if present. Move the option out of the choice. Disable BINMAN_FDT for a few boards which don't actually use it. Signed-off-by: Simon Glass --- (no changes since v1)

[PATCH v6 17/25] doc: Update info on devicetree update

2021-12-02 Thread Simon Glass
Since OF_BOARD has been corrected to be a run-time option, we can drop the historical info from this documentation. Signed-off-by: Simon Glass --- (no changes since v1) doc/develop/devicetree/dt_update.rst | 74 +++- 1 file changed, 8 insertions(+), 66 deletions(-)

[PATCH v6 18/25] fdt: Move MULTI_DTB_FIT handling out of fdtdec_setup()

2021-12-02 Thread Simon Glass
This logic is a bit convoluted for one function. Move the mulit-FIT part into its own function. Signed-off-by: Simon Glass Reviewed-by: Ilias Apalodimas --- (no changes since v1) lib/fdtdec.c | 62 1 file changed, 38 insertions(+), 24

[PATCH v6 16/25] Drop CONFIG_BINMAN_STANDALONE_FDT

2021-12-02 Thread Simon Glass
This was added as a hack to work around not having an in-tree devicetree. Now that this is fixed it is not needed. Drop it. Signed-off-by: Simon Glass --- (no changes since v1) Makefile| 3 +-- dts/Kconfig | 18 -- tools/binman/binman.rst | 20

[PATCH v6 22/25] fdt: Use if() for fdtcontroladdr check

2021-12-02 Thread Simon Glass
Change this to use if() instead of #if Signed-off-by: Simon Glass --- (no changes since v1) lib/fdtdec.c | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/lib/fdtdec.c b/lib/fdtdec.c index 299a2c3a32f..659aeffd82e 100644 --- a/lib/fdtdec.c +++ b/lib/fdtdec.c @@

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

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

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

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 >

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 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,

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

2021-12-02 Thread Simon Glass
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 CONFIG_OF_BOARD is enabled. The previous firmware stage provides a

[PATCH v6 13/25] arm: qemu-ppce500: Add a devicetree file

2021-12-02 Thread Simon Glass
Add a devicetree file obtained from qemu for this board. This was obtained with: qemu-system-ppc64 -machine ppce500 -cpu e6500 -M dumpdtb=dtb.dtb Signed-off-by: Simon Glass --- (no changes since v1) arch/powerpc/dts/Makefile | 1 + arch/powerpc/dts/qemu-ppce500.dts | 264

[PATCH v6 11/25] arm: xilinx_versal_virt: Add a devicetree file

2021-12-02 Thread Simon Glass
Add a devicetree file obtained from qemu for this board. This was obtained with: qemu-system-aarch64 -M xlnx-versal-virt -machine dumpdtb=dtb.dtb Signed-off-by: Simon Glass --- (no changes since v1) arch/arm/dts/Makefile| 3 +- arch/arm/dts/xilinx-versal-virt.dts | 307

[PATCH v6 14/25] arm: highbank: Add a fake devicetree file

2021-12-02 Thread Simon Glass
Add an empty file to prevent build errors when building with CONFIG_OF_SEPARATE enabled. Unfortunately there are no build instructions in the U-Boot tree to enable a real file to be created. Signed-off-by: Simon Glass --- (no changes since v1) arch/arm/dts/Makefile | 2 ++

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

2021-12-02 Thread Simon Glass
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. Signed-off-by: Simon Glass --- (no changes since v1) arch/arm/dts/Makefile| 2 ++

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

[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

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

2021-12-02 Thread Vladimir Oltean
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 a "nxp,pca9547" compatible string. Create a new compatible for

[PATCH 02/15] rtc: pcf2127: sync with Linux compatible strings

2021-12-02 Thread Vladimir Oltean
Allow this driver to be used by boards which inherit their device trees from Linux. Compatibility is temporarily retained with the old compatible string which is U-Boot specific, and will be removed after a few changes. Cc: Simon Glass Signed-off-by: Vladimir Oltean --- drivers/rtc/pcf2127.c |

[PATCH 04/15] arm: ls1088a-qds: use Linux compatible string for RTC

2021-12-02 Thread Vladimir Oltean
During the LS1028A-RDB sync with Linux device trees, it was observed that the same RTC is present on the two boards, and the wrong compatible string is used in both places. This change updates the RTC from the LS1088A-QDS to use the compatible string that was established in Linux. Signed-off-by:

[PATCH 03/15] arm: dts: ls1028a-qds: use Linux compatible string for RTC

2021-12-02 Thread Vladimir Oltean
The LS1028A-QDS board won't be synced with the Linux device trees right now, since those are currently still in progress (Ethernet is missing). However, while we're at converting the RDB, it can be observed that the same RTC is present on the two boards, and the wrong compatible string is used in

Re: [PATCH 5/5] clk: Add clk_get_by_name_optional

2021-12-02 Thread Simon Glass
Hi Sean, On Thu, 2 Dec 2021 at 07:25, Sean Anderson wrote: > > Hi Simon, > > On 12/2/21 8:43 AM, Simon Glass wrote: > > Hi Sean, > > > > On Wed, 1 Dec 2021 at 11:43, Sean Anderson wrote: > >> > >> This adds a helper function for clk_get_by_name in cases where the clock is > >> optional.

Question/issues about i.MX6 DDR configuration

2021-12-02 Thread Francesco Dolcini
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 effect is that SPL is not able to load U-Boot from the eMMC to DDR and

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 00/25] fdt: Make OF_BOARD a boolean option

2021-12-02 Thread Simon Glass
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 >> there are only three ways to obtain a devicetree: >> >>- OF_SEPARATE - the normal

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

  1   2   >