Re: [PATCH 11/11] pci: pcie_layerscape_fixup_common: lx2_board_fix_fdt can be static

2021-09-17 Thread Bin Meng
On Fri, Sep 17, 2021 at 8:11 PM Vladimir Oltean wrote: > > To avoid W=1 build warnings, declare this function as static, since it > is not used outside of this translation module. > > Signed-off-by: Vladimir Oltean > --- > drivers/pci/pcie_layerscape_fixup_common.c | 2 +- > 1 file changed, 1 in

Re: [PATCH 10/11] pci: pcie_layerscape_fixup_common: include fdt_support.h for ft_pci_setup

2021-09-17 Thread Bin Meng
On Fri, Sep 17, 2021 at 8:11 PM Vladimir Oltean wrote: > > The function prototype for ft_pci_setup is inside fdt_support.h, we need > to include that header. > > Signed-off-by: Vladimir Oltean > --- > drivers/pci/pcie_layerscape_fixup_common.c | 1 + > 1 file changed, 1 insertion(+) > Reviewed-

Re: [PATCH 09/11] pci: layerscape: ls_pcie_conf_address can be static

2021-09-17 Thread Bin Meng
On Fri, Sep 17, 2021 at 8:11 PM Vladimir Oltean wrote: > > To avoid W=1 build warnings, declare this function as static, since it > is not used outside of this translation module. > > Signed-off-by: Vladimir Oltean > --- > drivers/pci/pcie_layerscape_rc.c | 4 ++-- > 1 file changed, 2 insertions

Re: [PATCH 08/11] pci: _dm_pci_phys_to_bus can be static

2021-09-17 Thread Bin Meng
On Fri, Sep 17, 2021 at 8:11 PM Vladimir Oltean wrote: > > To avoid W=1 build warnings, declare this function as static, since it > is not used outside of this translation module. > > Signed-off-by: Vladimir Oltean > --- > drivers/pci/pci-uclass.c | 6 +++--- > 1 file changed, 3 insertions(+), 3

Re: [PATCH 07/11] pci: pci_read_config can be static

2021-09-17 Thread Bin Meng
On Fri, Sep 17, 2021 at 8:11 PM Vladimir Oltean wrote: > > To avoid W=1 build warnings, declare this function as static, since it > is not used outside of this translation module. > > Signed-off-by: Vladimir Oltean > --- > drivers/pci/pci-uclass.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 d

Re: [PATCH 06/11] pci: pci_write_config can be static

2021-09-17 Thread Bin Meng
On Fri, Sep 17, 2021 at 8:11 PM Vladimir Oltean wrote: > > To avoid W=1 build warnings, declare this function as static, since it > is not used outside of this translation module. > > Signed-off-by: Vladimir Oltean > --- > drivers/pci/pci-uclass.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 d

Re: [PATCH 05/11] pci: pciinfo_header can be static

2021-09-17 Thread Bin Meng
On Fri, Sep 17, 2021 at 8:11 PM Vladimir Oltean wrote: > > To avoid W=1 build warnings, declare this function as static, since it > is not used outside of this translation module. > > Signed-off-by: Vladimir Oltean > --- > cmd/pci.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > Revi

Re: [PATCH 04/11] pci: pci_header_show can be static

2021-09-17 Thread Bin Meng
On Fri, Sep 17, 2021 at 8:11 PM Vladimir Oltean wrote: > > To avoid W=1 build warnings, declare this function as static, since it > is not used outside of this translation module. > > Signed-off-by: Vladimir Oltean > --- > cmd/pci.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > Revi

Re: [PATCH 03/11] pci: pci_bar_show can be static

2021-09-17 Thread Bin Meng
On Fri, Sep 17, 2021 at 8:11 PM Vladimir Oltean wrote: > > To avoid W=1 build warnings, declare this function as static, since it > is not used outside of this translation module. > > Signed-off-by: Vladimir Oltean > --- > cmd/pci.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > Revi

Re: [PATCH 02/11] pci: include pci_internal.h inside pci_auto.c

2021-09-17 Thread Bin Meng
On Fri, Sep 17, 2021 at 8:11 PM Vladimir Oltean wrote: > > To avoid a build warning with W=1, provide a function prototype for > dm_pciauto_prescan_setup_bridge, which is a non-static function whose > definition is inside pci_auto.c. > > Signed-off-by: Vladimir Oltean > --- > drivers/pci/pci_aut

Re: [PATCH 01/11] pci: provide prototype for pci_skip_dev outside of #if defined(CONFIG_DM_PCI_COMPAT)

2021-09-17 Thread Bin Meng
On Fri, Sep 17, 2021 at 8:11 PM Vladimir Oltean wrote: > > The weak definition of pci_skip_dev from drivers/pci/pci_common.c is not > under CONFIG_DM_PCI_COMPAT, and that definition needs a previous > function prototype declaration to avoid W=1 build warnings. > > That prototype is not available d

Re: Question about extension board used in U-boot

2021-09-17 Thread François Ozog
Le sam. 18 sept. 2021 à 08:49, François Ozog a écrit : > Hi Paul > > Too posting because I think we also need to address this at a higher level. > > i think we discussed this topic quite a while back. I may be wrong but it > may be Bill Mills who proposed to have an eeprom on the extensions that

Re: Question about extension board used in U-boot

2021-09-17 Thread François Ozog
Hi Paul Too posting because I think we also need to address this at a higher level. i think we discussed this topic quite a while back. I may be wrong but it may be Bill Mills who proposed to have an eeprom on the extensions that the carrier board can use to detect and fetch proper overlay. Anoth

[PATCH v8 4/5] watchdog: rti_wdt: Add support for loading firmware

2021-09-17 Thread Jan Kiszka
From: Jan Kiszka To avoid the need of extra boot scripting on AM65x for loading a watchdog firmware, add the required rproc init and loading logic for the first R5F core to the watchdog start handler. In case the R5F cluster is in lock-step mode, also initialize the second core. The firmware itse

[PATCH v8 5/5] iot2050: Enable watchdog support, but do not auto-start it

2021-09-17 Thread Jan Kiszka
From: Jan Kiszka This allows to use the watchdog in custom scripts but does not enforce that the OS has to support it as well. Signed-off-by: Jan Kiszka --- arch/arm/dts/k3-am65-iot2050-boot-image.dtsi | 25 configs/iot2050_defconfig| 6 + tools/bi

[PATCH v8 3/5] arm64: dts: ti: k3-am65-mcu: Add RTI watchdog entry

2021-09-17 Thread Jan Kiszka
From: Jan Kiszka Add the DT entry for a watchdog based on RTI1. It requires additional firmware on the MCU R5F cores to handle the expiry, e.g. https://github.com/siemens/k3-rti-wdt. As this firmware will also lock the power domain to protect it against premature shutdown, mark it shared. Aligns

[PATCH v8 1/5] arm: dts: Add IOT2050 device tree files

2021-09-17 Thread Jan Kiszka
From: Jan Kiszka Prepares for the addition of the IOT2050 board which is based on the TI AM65x. The board comes in four variants, Basic and Advanced, each as product generation 1 (SR1.0) and 2 (SR2.x), so there are separate dts files needed. Furthermore, the SPL has its own device tree. Based on

[PATCH v8 2/5] board: siemens: Add support for SIMATIC IOT2050 devices

2021-09-17 Thread Jan Kiszka
From: Jan Kiszka This adds support for the IOT2050 Basic and Advanced devices. The Basic used the dual-core AM6528 GP processor, the Advanced one the AM6548 HS quad-core version. Both variants are booted via a Siemens-provided FSBL that runs on the R5 cores. Consequently, U-Boot support is targe

[PATCH v8 0/5] Add SIMATIC IOT2050 board support

2021-09-17 Thread Jan Kiszka
This is the baseline support for the SIMATIC IOT2050 devices. Changes in v8: - rebased - added support for product generation 2 board - align device trees with upstream [pending] kernel version - only warn on missing watchdog firmware binary - removed bits only needed for incomplete prueth su

Question about extension board used in U-boot

2021-09-17 Thread Ying-Chun Liu (PaulLiu)
Hi all, I have some questions about how to implement extension board usage. My case is on imx8mm-cl-iot-gate. It can add three different types of extension boards. One of the extension boards is SPI extension which have 3 empty slots. And you can add some small boards onto it. One of them is a "T

Re: [PATCH V2] arm: mach-k3: am6_init: Prioritize MSMC traffic over DDR in NAVSS Northbridge

2021-09-17 Thread Tom Rini
On Wed, Sep 08, 2021 at 03:28:59PM -0500, Nishanth Menon wrote: > From: Roger Quadros > > NB0 is bridge to SRAM and NB1 is bridge to DDR. > > To ensure that SRAM transfers are not stalled due to delays during DDR > refreshes, SRAM traffic should be higher priority (threadmap=2) than > DDR traff

Re: [PATCH v2 5/5] clk: ti: k3: Update driver to account for divider flags

2021-09-17 Thread Tom Rini
On Tue, Sep 07, 2021 at 05:16:58PM -0500, Dave Gerlach wrote: > From: Suman Anna > > The K3 SoCs have some PLL output clocks (POSTDIV clocks) which in > turn serve as inputs to other HSDIV output clocks. These clocks use > the actual value to compute the divider clock rate, and need to be > regi

Re: [PATCH v2 4/5] clk: ti: k3-pll: Change DIV_CTRL programming to read-modify-write

2021-09-17 Thread Tom Rini
On Tue, Sep 07, 2021 at 05:16:57PM -0500, Dave Gerlach wrote: > There are three different divider values in the DIV_CTRL register > controlled by the k3-pll driver. Currently the ti_pll_clk_set_rate > function writes the entire register when programming plld, even though > plld only resides in the

Re: [PATCH v2 3/5] arm: mach-k3: Add note to auto-generated files

2021-09-17 Thread Tom Rini
On Tue, Sep 07, 2021 at 05:16:56PM -0500, Dave Gerlach wrote: > Add a note to the automatically generated clk-data and dev-data files > for j721e and j7200 to indicate that they are in fact auto-generated and > should not be hand edited. > > Also adjust TI URL to use https instead of http and als

Re: [PATCH v2 2/5] arm: mach-k3: j7200: Fix clk-data parenting for postdiv PLL clocks

2021-09-17 Thread Tom Rini
On Tue, Sep 07, 2021 at 05:16:55PM -0500, Dave Gerlach wrote: > From: Suman Anna > > The TI K3 Fractional PLLs use two programmable POSTDIV1 and POSTDIV2 > divisors to generate the final FOUTPOSTDIV clock. These are in sequence > with POSTDIV2 following the POSTDIV1 clock. The current J7200 cloc

Re: [PATCH v2 1/5] arm: mach-k3: j721e: Fix clk-data parenting for postdiv PLL clocks

2021-09-17 Thread Tom Rini
On Tue, Sep 07, 2021 at 05:16:54PM -0500, Dave Gerlach wrote: > From: Suman Anna > > The TI K3 Fractional PLLs use two programmable POSTDIV1 and POSTDIV2 > divisors to generate the final FOUTPOSTDIV clock. These are in sequence > with POSTDIV2 following the POSTDIV1 clock. The current J721E cloc

Re: [PATCH 2/2] arm: mach-k3: common: Add a release_resources_for_core_shutdown() stub

2021-09-17 Thread Tom Rini
On Tue, Jul 27, 2021 at 06:24:40PM -0500, Suman Anna wrote: > Add a weak release_resources_for_core_shutdown() stub implementation > that can be overridden by actual implementation if a SoC supports that > function. > > Signed-off-by: Suman Anna > Reviewed-by: Nishanth Menon Applied to u-boot/

Re: [PATCH 1/2] firmware: ti_sci: Include linux/err.h in ti_sci_protocol.h

2021-09-17 Thread Tom Rini
On Tue, Jul 27, 2021 at 06:24:39PM -0500, Suman Anna wrote: > The common TI SCI header file uses some macros from err.h and these > get exercised when CONFIG_TI_SCI_PROTOCOL is not defined. Include > the linux/err.h header file in this header file directly rather > than relying on source files to

Re: [PATCH 1/2] arm: mach-k3: common: Make sure firmware sections are loaded prior to armv8 startup

2021-09-17 Thread Tom Rini
On Tue, Aug 31, 2021 at 01:20:48PM -0500, Nishanth Menon wrote: > With Device Manager firmware in an elf file form, we cannot load the FIT > image to the exact same address as any of the executable sections of the > elf file itself is located. > > However, the device tree descriptions for the ARM

Re: IMX8MM fsl_esdhc_imx UHS/HS200/HS400 not working?

2021-09-17 Thread Tim Harvey
On Thu, Sep 16, 2021 at 4:32 PM Tim Harvey wrote: > > Greetings, > > Can anyone confirm if IMX8MM UHS/HS200/HS400 is working for them? I > had this working in my U-Boot v2021.01 imx8mm-veice branch but it > seems to not work in U-Boot master or v2021.07. I know there was a lot > of back and forth

Re: [PATCH] riscv: Fix setting no-map in reserved memory nodes

2021-09-17 Thread Atish Patra
On Sun, Sep 12, 2021 at 9:05 AM Samuel Holland wrote: > > The no-map property is wrongly skipped if a no-map reserved memory > node follows one without that property. Fix this by not remembering > the absence of a no-map property across loop iterations. > > Fixes: d4ea649f179a ("riscv: Provide a m

[RFC 1/2] mtd: Add partial support for Linux slc-mode for MLC NAND

2021-09-17 Thread Chris Morgan
From: Chris Morgan Add support for setting the slc-mode parameter for a given MLC NAND device. This allows a given device to write only to the first set of paried pages to help reduce data corruption and increase reliability of the NAND. I haven't tested these patches extensively yet, but thus f

[RFC 2/2] mtd: Add pairing info for Toshiba TC58TEG5DCLTA00 NAND

2021-09-17 Thread Chris Morgan
From: Chris Morgan Add pairing info for the Toshiba TC58TEG5DCLTA00 NAND so that we may utilize slc-mode emulation for enhanced reliability and compatibility with upstream Linux. Signed-off-by: Chris Morgan --- drivers/mtd/nand/raw/nand_base.c | 9 + 1 file changed, 9 insertions(+) di

[RFC 0/2] mtd: Support slc-mode for Toshiba TC58TEG5DCLTA00

2021-09-17 Thread Chris Morgan
From: Chris Morgan Add support for slc-mode implemented in Linux for the Toshiba TC58TEG5DCLTA00 NAND flash found on the NTC CHIP. This requires the addition of a paired-pages scheme, a new parameter for MTD partitions of slc-mode, and setting the correct paired-pages scheme for the TC58TEG5DCLTA

Re: Problem with U-boot | Configuration Signature not being checked while booting

2021-09-17 Thread Tom Rini
On Fri, Sep 17, 2021 at 10:21:38AM -0600, Simon Glass wrote: > Hi François, > > On Wed, 15 Sept 2021 at 04:26, François Ozog wrote: > > > > > > > > Le mer. 15 sept. 2021 à 12:13, Simon Glass a écrit : > >> > >> Hi Mark, > >> > >> On Sat, 11 Sept 2021 at 13:18, Mark Kettenis > >> wrote: > >> >

Re: Problem with U-boot | Configuration Signature not being checked while booting

2021-09-17 Thread Tom Rini
On Fri, Sep 17, 2021 at 10:21:15AM -0600, Simon Glass wrote: > Hi Tom, > > On Wed, 15 Sept 2021 at 07:35, Tom Rini wrote: > > > > On Wed, Sep 15, 2021 at 01:51:51PM +0200, Mark Kettenis wrote: > > > > From: Simon Glass > > > > Date: Wed, 15 Sep 2021 04:13:24 -0600 > > > > > > Hi Simon, > > > > >

Re: Problem with U-boot | Configuration Signature not being checked while booting

2021-09-17 Thread Tom Rini
On Fri, Sep 17, 2021 at 10:19:18AM -0600, Simon Glass wrote: > Hi Mark, > > On Wed, 15 Sept 2021 at 05:52, Mark Kettenis wrote: > > > > > From: Simon Glass > > > Date: Wed, 15 Sep 2021 04:13:24 -0600 > > > > Hi Simon, > > > > > Hi Mark, > > > > > > On Sat, 11 Sept 2021 at 13:18, Mark Kettenis

Re: Problem with U-boot | Configuration Signature not being checked while booting

2021-09-17 Thread François Ozog
HI Simon, On Fri, 17 Sept 2021 at 18:21, Simon Glass wrote: > Hi François, > > On Wed, 15 Sept 2021 at 04:26, François Ozog > wrote: > > > > > > > > Le mer. 15 sept. 2021 à 12:13, Simon Glass a écrit : > >> > >> Hi Mark, > >> > >> On Sat, 11 Sept 2021 at 13:18, Mark Kettenis > wrote: > >> > >

Re: [PATCH] mtd: spi-nor-core: Add fixups for s25fs512s

2021-09-17 Thread Pratyush Yadav
On 15/09/21 12:49PM, tkuw584...@gmail.com wrote: > From: Takahiro Kuwano > > The current S25FS512S support has following issues that need to be fixed. > > - Non-uniform sectors by factory default. The setting needs to be > checked and assign erase hook as needed. > - Page size is wrongly

Re: [PATCH 2/2] mtd: spi-nor-core: Add fixups for Spansion S25FL256L

2021-09-17 Thread Pratyush Yadav
On 08/09/21 05:47PM, tkuw584...@gmail.com wrote: > From: Takahiro Kuwano > > The S25FL256L is 32MB NOR Flash that does not support Bank Address > Register. This fixup is activated if CONFIG_SPI_FLASH_BAR is enabled and > returns ENOTSUPP in setup() hook to avoid further ops. > > Signed-off-by: T

Re: [PATCH 1/2] mtd: spi-nor-ids: Add support for Spansion S25FL256L

2021-09-17 Thread Pratyush Yadav
On 08/09/21 05:47PM, tkuw584...@gmail.com wrote: > From: Takahiro Kuwano > > The S25FL256L is a part of the S25FL-L family and has the same feature set > as S25FL128L except the density. > > The datasheet can be found in the following link. > https://www.cypress.com/file/316171/download > > Tes

Re: Problem with U-boot | Configuration Signature not being checked while booting

2021-09-17 Thread Simon Glass
Hi François, On Wed, 15 Sept 2021 at 04:26, François Ozog wrote: > > > > Le mer. 15 sept. 2021 à 12:13, Simon Glass a écrit : >> >> Hi Mark, >> >> On Sat, 11 Sept 2021 at 13:18, Mark Kettenis wrote: >> > >> > > From: Moiz Imtiaz >> > > Date: Sat, 11 Sep 2021 23:19:05 +0500 >> > > >> > > Hi Sim

Re: Problem with U-boot | Configuration Signature not being checked while booting

2021-09-17 Thread Simon Glass
Hi Tom, On Wed, 15 Sept 2021 at 07:35, Tom Rini wrote: > > On Wed, Sep 15, 2021 at 01:51:51PM +0200, Mark Kettenis wrote: > > > From: Simon Glass > > > Date: Wed, 15 Sep 2021 04:13:24 -0600 > > > > Hi Simon, > > > > > Hi Mark, > > > > > > On Sat, 11 Sept 2021 at 13:18, Mark Kettenis > > > wrot

Re: Problem with U-boot | Configuration Signature not being checked while booting

2021-09-17 Thread Simon Glass
Hi Mark, On Wed, 15 Sept 2021 at 05:52, Mark Kettenis wrote: > > > From: Simon Glass > > Date: Wed, 15 Sep 2021 04:13:24 -0600 > > Hi Simon, > > > Hi Mark, > > > > On Sat, 11 Sept 2021 at 13:18, Mark Kettenis > > wrote: > > > > > > > From: Moiz Imtiaz > > > > Date: Sat, 11 Sep 2021 23:19:05 +

[PATCH 15/17] Makefile: Support ad-hoc COMFIX_SPL_IMAGE_SIZE et al

2021-09-17 Thread Simon Glass
Ad-hoc CONFIGs are not supported by the new split-config mechanism. Use the provided work-around for these symbols, just to show the mechanism. Signed-off-by: Simon Glass --- scripts/Makefile.spl | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/scripts/Makefile.spl b/

[PATCH 12/17] kconfig: Support writing separate SPL files

2021-09-17 Thread Simon Glass
At present kconfig writes out several files, including: auto.conf - CONFIG settings used by make autoconf.h - header file used by C code This works well but is a bit ugly in places, for example requiring the use of a SPL_TPL_ macro in Makefiles to distinguish between options intended for S

[PATCH 17/17] kconfig: Add a new CONFIG() macro

2021-09-17 Thread Simon Glass
It is annoying to have to use IS_ENABLED() and CONFIG_IS_ENABLED(), depending on whether the option being checked has an SPL version. Also we use #ifdef CONFIG_xxx and #if defined(CONFIG_xxx) in some contexts. It would be nice to use a single style consistenty. Add a CONFIG() macro to this end, to

[PATCH 16/17] kconfig: Update CONFIG_IS_ENABLED() for split files

2021-09-17 Thread Simon Glass
Update this file to include the correct autoconf.h or autoconf_spl.h file for each phase. This allows the macros to be simplified. With this, CONFIG_IS_ENABLED() is the same as IS_ENABLED() apart from a migration detail. Signed-off-by: Simon Glass --- include/linux/kconfig.h | 87 -

[PATCH 10/17] virtio: Add SPL Kconfig for virtio

2021-09-17 Thread Simon Glass
At present we rely on this not existing to avoid building virtio for SPL. But with the new split configs this does not work. Add a separate Kconfig instead. Signed-off-by: Simon Glass --- drivers/virtio/Kconfig | 6 ++ 1 file changed, 6 insertions(+) diff --git a/drivers/virtio/Kconfig b/

[PATCH 14/17] Makefile: Use empty SPL_ and SPL_TPL_ vars

2021-09-17 Thread Simon Glass
Now that we include an auto.conf file specific to the phase being built we can make these variables empty. With this done, we can also remove them from U-Boot entirely, dropping them from all Makefiles. That is left for a future clean-up since we cannot apply this series until CONFIG migration is

[PATCH 11/17] kconfig: Refactor code into separate writer functions

2021-09-17 Thread Simon Glass
Separate out the code that writes the Makefile and headers so we can reuse these functions when writing out SPL files. This makes no functional change. Signed-off-by: Simon Glass --- scripts/kconfig/confdata.c | 65 ++ scripts/kconfig/expr.h | 9 ++

[PATCH 13/17] Makefile: Include the config for the phase being built

2021-09-17 Thread Simon Glass
At present there is only a single auto.conf file used within the makefiles. Update them to use the correct one for each phase. Signed-off-by: Simon Glass --- Makefile | 11 ++- scripts/Makefile.build | 10 +- scripts/Makefile.spl | 8 +++- 3 files changed, 2

[PATCH 09/17] serial: Tidy up the Makefile condition for DM_SERIAL

2021-09-17 Thread Simon Glass
There is no need to check for SPL/TPL build again as the 'ifdef' above does this already. Drop it, so that the build rule works with split configs. Signed-off-by: Simon Glass --- drivers/serial/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/serial/Makefile

[PATCH 08/17] sandbox: Enable MISC support for the SPL build

2021-09-17 Thread Simon Glass
At present this is always enabled in SPL by virtue of the Makefile condition, which ignores SPL. Add the required config to tidy this up, so it will works with split configs. Signed-off-by: Simon Glass --- configs/sandbox_noinst_defconfig | 1 + configs/sandbox_spl_defconfig| 1 + 2 files c

[PATCH 06/17] power: Add SPL Kconfig for sandbox pmc emulator

2021-09-17 Thread Simon Glass
At present we rely on this not existing to avoid building the emulator for SPL. But with the new split configs this does not work. Add a separate Kconfig instead. Signed-off-by: Simon Glass --- drivers/power/acpi_pmc/Kconfig | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/powe

[PATCH 07/17] test: Add SPL Kconfig for compression tests

2021-09-17 Thread Simon Glass
At present we rely on this not existing to avoid building these tests for SPL. But with the new split configs this does not work. Add a separate Kconfig instead. Signed-off-by: Simon Glass --- test/Kconfig | 5 + 1 file changed, 5 insertions(+) diff --git a/test/Kconfig b/test/Kconfig ind

[PATCH 05/17] cros_ec: Add SPL Kconfigs for cros_ec features

2021-09-17 Thread Simon Glass
At present we rely on this not existing to avoid building various EC features. But with the new split configs this does not work. Add separates Kconfigs instead. Signed-off-by: Simon Glass --- drivers/i2c/Kconfig | 10 ++ 1 file changed, 10 insertions(+) diff --git a/drivers/i2c/Kconf

[PATCH 03/17] acpi: Add a Kconfig for SPL

2021-09-17 Thread Simon Glass
At present we rely on this not existing to avoid building ACPI for SPL. But with the new split configs this does not work. Add a separate Kconfig instead. Signed-off-by: Simon Glass --- drivers/core/Kconfig | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/core/Kconfig b/drivers/

[PATCH 04/17] cmd: Add an SPL Kconfig for CMDLINE and HUSH

2021-09-17 Thread Simon Glass
At present we rely on this not existing to avoid building various command-line features. But with the new split configs this does not work. Add separates Kconfigs instead. Signed-off-by: Simon Glass --- cmd/Kconfig | 10 ++ 1 file changed, 10 insertions(+) diff --git a/cmd/Kconfig b/c

[PATCH 01/17] env: Avoid checking ENV_IS_IN when env disabled

2021-09-17 Thread Simon Glass
This check is not needed when the environment is not enabled, e.g. in SPL. Add a condition to handle this. Signed-off-by: Simon Glass --- cmd/nvedit.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/cmd/nvedit.c b/cmd/nvedit.c index ddc715b4f91..1c8d49646e5 100644 --- a/cmd/nvedit.c +++ b

[PATCH 02/17] x86: Fix qemu condition for arch_cpu_init()

2021-09-17 Thread Simon Glass
This is written incorrectly, since SPL_ should not be used with the CONFIG_IS_ENABLED macro. Fix it by dropping the prefix and inverting the logic, which produces the same result as now. Signed-off-by: Simon Glass --- arch/x86/cpu/qemu/qemu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-

[PATCH 00/17] RFC: Split configs between U-Boot proper and SPL builds

2021-09-17 Thread Simon Glass
At present a single .config file is used to configure the build for all phases, with SPL phases having a SPL_ or TPL_ prefix on each CONFIG to indicate which build it controls. So, for example, we have CONFIG_MISC and CONFIG_SPL_MISC to control the inclusion of MISC drivers in each phase. >From t

[PATCH] m68k: Add missing ENTRY entry in linker script

2021-09-17 Thread Tom Rini
Currently, there is no ENTRY() entry in the m68k linker script. However, since they currently contain a version_string symbol, which then overrides the default weak version_string symbol, the linker decides we must keep start.o in the resulting link. Add an ENTRY() line so that the linker will kno

Re: [PATCH] distro_bootcmd: change the default dtb search path to include default kernel dtbs directory

2021-09-17 Thread Qu Wenruo
On 2021/9/17 22:25, Mark Kettenis wrote: Date: Fri, 17 Sep 2021 19:42:29 +0800 Content-Language: en-US On 2021/9/17 19:34, Mark Kettenis wrote: From: Qu Wenruo Date: Fri, 17 Sep 2021 19:02:35 +0800 When booting using U-boot -> systemd-boot (EFI payload) -> kernel on RK3399, systemd-boot by

Re: [PATCH] distro_bootcmd: change the default dtb search path to include default kernel dtbs directory

2021-09-17 Thread Mark Kettenis
> Date: Fri, 17 Sep 2021 19:42:29 +0800 > Content-Language: en-US > > On 2021/9/17 19:34, Mark Kettenis wrote: > >> From: Qu Wenruo > >> Date: Fri, 17 Sep 2021 19:02:35 +0800 > >> > >> When booting using U-boot -> systemd-boot (EFI payload) -> kernel on > >> RK3399, systemd-boot by some bug can't

Re: [PATCH v3 2/2] mtd: spi: nor: force mtd name to "nor%d"

2021-09-17 Thread Marek Vasut
On 9/17/21 3:06 PM, Patrick DELAUNAY wrote: Hi Marek, > Marek VasutSept. 16, 2021, 5:27 p.m. UTC | #3 > On 9/16/21 4:01 PM, Patrick Delaunay wrote: > [...] > > @@ -3664,6 +3666,11 @@ int spi_nor_scan(struct spi_nor *nor) > >       struct mtd_info *mtd = &nor->mtd; > >       struct spi_sl

Re: [PATCH v3 1/2] mtd: cfi_flash: use cfi_flash_num_flash_banks only when supported

2021-09-17 Thread Marek Vasut
On 9/17/21 12:55 PM, Patrick DELAUNAY wrote: Hi Marek, > Marek VasutSept. 16, 2021, 5:24 p.m. UTC | #1 > On 9/16/21 4:01 PM, Patrick Delaunay wrote: >> When CONFIG_SYS_MAX_FLASH_BANKS_DETECT is activated, >> CONFIG_SYS_MAX_FLASH_BANKS is replaced by cfi_flash_num_flash_banks, >> but this

Re: [ANN] U-Boot v2021.10-rc4 released

2021-09-17 Thread Tom Rini
On Fri, Sep 17, 2021 at 08:29:07AM +0200, Christophe Leroy wrote: > Hi Tom, > > Le 15/09/2021 à 01:01, Tom Rini a écrit : > > Hey all, > > > > Alright, I'm a day late, but, here's v2021.10-rc4. We've had a few > > regressions pop up of late, unfortunately. I've pushed the fix for > > (what I b

Re: [systemd-devel] Systemd-boot not properly loading device tree, when loaded by U-boot (ARM64, tested on RK3399)

2021-09-17 Thread Qu Wenruo
On 2021/9/17 19:45, Lennart Poettering wrote: On Fr, 17.09.21 19:25, Qu Wenruo (w...@suse.com) wrote: Hi, I'm recently testing booting my RK3399 boards with the following boot sequence: U-boot -> systemd-boot (EFI payload) -> kernel Which provides much more flex than plain extlinux conf f

[PATCH v3 2/2] spi: rockchip_sfc: Using read_poll

2021-09-17 Thread Jon Lin
Using read_poll logic. Tested-by: Chris Morgan Signed-off-by: Jon Lin --- (no changes since v2) Changes in v2: - Fix assigned but never used return error codes drivers/spi/rockchip_sfc.c | 67 -- 1 file changed, 35 insertions(+), 32 deletions(-) diff --gi

[PATCH v3 1/2] spi: rockchip_sfc: Implement set_speed logic

2021-09-17 Thread Jon Lin
Set clock related processing into set_speed logic. And Optimize printing format. Tested-by: Chris Morgan Signed-off-by: Jon Lin --- Changes in v3: - Remove useless headfile - Fix misspelling drivers/spi/rockchip_sfc.c | 82 ++ 1 file changed, 39 insertions(

Re: [PATCH v3 2/2] mtd: spi: nor: force mtd name to "nor%d"

2021-09-17 Thread Patrick DELAUNAY
Hi Marek, > Marek VasutSept. 16, 2021, 5:27 p.m. UTC | #3 > On 9/16/21 4:01 PM, Patrick Delaunay wrote: > [...] > > @@ -3664,6 +3666,11 @@ int spi_nor_scan(struct spi_nor *nor) > >       struct mtd_info *mtd = &nor->mtd; > >       struct spi_slave *spi = nor->spi; > >       int ret; > > +    in

Re: [PATCH 2/3] efi_loader: GOP: Add 30bpp support

2021-09-17 Thread Mark Kettenis
> From: Heinrich Schuchardt > Date: Fri, 17 Sep 2021 13:29:06 +0200 > > On 9/17/21 11:23 AM, Mark Kettenis wrote: > >> From: Heinrich Schuchardt > >> Date: Fri, 17 Sep 2021 04:56:31 +0200 > > > > Hi Heinrich, > > > >> On 9/16/21 3:01 PM, Mark Kettenis wrote: > >>> Provide correct framebuffer inf

Re: [PATCH] xtensa: Put U-Boot version string at correct place by linker script

2021-09-17 Thread Max Filippov
On Fri, Sep 17, 2021 at 5:04 AM Tom Rini wrote: > So this is in context with: > https://patchwork.ozlabs.org/project/uboot/list/?series=256258 > and specifically: > https://patchwork.ozlabs.org/project/uboot/patch/20210802131838.21097-7-p...@kernel.org/ Yes, I've found that series and the purpose

RE: [PATCH] configs: lx2162a: Enable CONFIG_SPI_FLASH_MT35XU for lx2162a-qds

2021-09-17 Thread Wasim Khan
> -Original Message- > From: U-Boot On Behalf Of Kuldeep Singh > Sent: Wednesday, September 15, 2021 3:34 PM > To: u-boot@lists.denx.de > Cc: Priyanka Jain ; Kuldeep Singh > > Subject: [PATCH] configs: lx2162a: Enable CONFIG_SPI_FLASH_MT35XU for > lx2162a-qds > > LX2162A-QDS has micro

[PATCH] board: freescale: lx216x : increase fdt blob size

2021-09-17 Thread Wasim Khan
From: Wasim Khan Increase fdt blob size for lx2160 and lx2162 series to fix below errors/warnings during device tree fixup. Unable to update property /soc/spi@210:status, err=FDT_ERR_NOSPACE Unable to update property /soc/spi@211:status, err=FDT_ERR_NOSPACE Unable to update property /soc

[PATCH 10/11] pci: pcie_layerscape_fixup_common: include fdt_support.h for ft_pci_setup

2021-09-17 Thread Vladimir Oltean
The function prototype for ft_pci_setup is inside fdt_support.h, we need to include that header. Signed-off-by: Vladimir Oltean --- drivers/pci/pcie_layerscape_fixup_common.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/pci/pcie_layerscape_fixup_common.c b/drivers/pci/pcie_layers

[PATCH 09/11] pci: layerscape: ls_pcie_conf_address can be static

2021-09-17 Thread Vladimir Oltean
To avoid W=1 build warnings, declare this function as static, since it is not used outside of this translation module. Signed-off-by: Vladimir Oltean --- drivers/pci/pcie_layerscape_rc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/pci/pcie_layerscape_rc.c b/dr

[PATCH 11/11] pci: pcie_layerscape_fixup_common: lx2_board_fix_fdt can be static

2021-09-17 Thread Vladimir Oltean
To avoid W=1 build warnings, declare this function as static, since it is not used outside of this translation module. Signed-off-by: Vladimir Oltean --- drivers/pci/pcie_layerscape_fixup_common.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/pci/pcie_layerscape_fix

[PATCH 08/11] pci: _dm_pci_phys_to_bus can be static

2021-09-17 Thread Vladimir Oltean
To avoid W=1 build warnings, declare this function as static, since it is not used outside of this translation module. Signed-off-by: Vladimir Oltean --- drivers/pci/pci-uclass.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/pci/pci-uclass.c b/drivers/pci/pci-

[PATCH 04/11] pci: pci_header_show can be static

2021-09-17 Thread Vladimir Oltean
To avoid W=1 build warnings, declare this function as static, since it is not used outside of this translation module. Signed-off-by: Vladimir Oltean --- cmd/pci.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cmd/pci.c b/cmd/pci.c index ebe7c0fc5377..7febff497c54 100644 --

[PATCH 07/11] pci: pci_read_config can be static

2021-09-17 Thread Vladimir Oltean
To avoid W=1 build warnings, declare this function as static, since it is not used outside of this translation module. Signed-off-by: Vladimir Oltean --- drivers/pci/pci-uclass.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/pci/pci-uclass.c b/drivers/pci/pci-uc

[PATCH 05/11] pci: pciinfo_header can be static

2021-09-17 Thread Vladimir Oltean
To avoid W=1 build warnings, declare this function as static, since it is not used outside of this translation module. Signed-off-by: Vladimir Oltean --- cmd/pci.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cmd/pci.c b/cmd/pci.c index 7febff497c54..cfabdc0f3012 100644 --

[PATCH 06/11] pci: pci_write_config can be static

2021-09-17 Thread Vladimir Oltean
To avoid W=1 build warnings, declare this function as static, since it is not used outside of this translation module. Signed-off-by: Vladimir Oltean --- drivers/pci/pci-uclass.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/pci/pci-uclass.c b/drivers/pci/pci-uc

[PATCH 03/11] pci: pci_bar_show can be static

2021-09-17 Thread Vladimir Oltean
To avoid W=1 build warnings, declare this function as static, since it is not used outside of this translation module. Signed-off-by: Vladimir Oltean --- cmd/pci.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cmd/pci.c b/cmd/pci.c index af75a6cfff4a..ebe7c0fc5377 100644 --

[PATCH 02/11] pci: include pci_internal.h inside pci_auto.c

2021-09-17 Thread Vladimir Oltean
To avoid a build warning with W=1, provide a function prototype for dm_pciauto_prescan_setup_bridge, which is a non-static function whose definition is inside pci_auto.c. Signed-off-by: Vladimir Oltean --- drivers/pci/pci_auto.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/pci/pci

[PATCH 01/11] pci: provide prototype for pci_skip_dev outside of #if defined(CONFIG_DM_PCI_COMPAT)

2021-09-17 Thread Vladimir Oltean
The weak definition of pci_skip_dev from drivers/pci/pci_common.c is not under CONFIG_DM_PCI_COMPAT, and that definition needs a previous function prototype declaration to avoid W=1 build warnings. That prototype is not available due to it being under CONFIG_DM_PCI_COMPAT, so move it outside of th

[PATCH 00/11] Drive-by PCI W=1 cleanups

2021-09-17 Thread Vladimir Oltean
These are some trivial cleanup patches I made after compiling U-Boot for ls1028ardb_tfa_defconfig with W=1. Vladimir Oltean (11): pci: provide prototype for pci_skip_dev outside of #if defined(CONFIG_DM_PCI_COMPAT) pci: include pci_internal.h inside pci_auto.c pci: pci_bar_show can be st

Re: [PATCH] test/py: tpm2: Skip tpm pytest based on env variable

2021-09-17 Thread Tom Rini
On Fri, Sep 17, 2021 at 11:49:51AM +, T Karthik Reddy wrote: > Hi Tom, > > > -Original Message- > > From: Tom Rini > > Sent: Thursday, August 5, 2021 6:53 PM > > To: T Karthik Reddy > > Cc: Ashok Reddy Soma ; u-boot@lists.denx.de; > > miquel.ray...@bootlin.com; git > > Subject: Re:

Re: [systemd-devel] Systemd-boot not properly loading device tree, when loaded by U-boot (ARM64, tested on RK3399)

2021-09-17 Thread Lennart Poettering
On Fr, 17.09.21 19:25, Qu Wenruo (w...@suse.com) wrote: > Hi, > > I'm recently testing booting my RK3399 boards with the following boot > sequence: > > U-boot -> systemd-boot (EFI payload) -> kernel > > Which provides much more flex than plain extlinux conf from U-boot. > (More choice, easier to w

Re: [PATCH] xtensa: Put U-Boot version string at correct place by linker script

2021-09-17 Thread Tom Rini
On Thu, Sep 16, 2021 at 10:14:03PM -0700, Max Filippov wrote: > On Thu, Sep 16, 2021 at 10:30 AM Tom Rini wrote: > > > > Update the linker script macros to know that we need to include the > > .text_version_string section now as well. > > > > Signed-off-by: Tom Rini > > --- > > arch/xtensa/inclu

Re: [PATCH] xtensa: Put U-Boot version string at correct place by linker script

2021-09-17 Thread Tom Rini
On Thu, Sep 16, 2021 at 10:21:29PM -0700, Max Filippov wrote: > On Thu, Sep 16, 2021 at 12:42 PM Tom Rini wrote: > > We also don't have qemu support for it > > I'm curious what happened to it and what I should do to update it? > > xtensa is still supported in the QEMU and AFAIK nothing has chan

RE: [PATCH] test/py: tpm2: Skip tpm pytest based on env variable

2021-09-17 Thread T Karthik Reddy
Hi Tom, > -Original Message- > From: Tom Rini > Sent: Thursday, August 5, 2021 6:53 PM > To: T Karthik Reddy > Cc: Ashok Reddy Soma ; u-boot@lists.denx.de; > miquel.ray...@bootlin.com; git > Subject: Re: [PATCH] test/py: tpm2: Skip tpm pytest based on env variable > > On Thu, Aug 05, 2

Re: [systemd-devel] Systemd-boot not properly loading device tree, when loaded by U-boot (ARM64, tested on RK3399)

2021-09-17 Thread Qu Wenruo
On 2021/9/17 19:45, Lennart Poettering wrote: On Fr, 17.09.21 19:25, Qu Wenruo (w...@suse.com) wrote: Hi, I'm recently testing booting my RK3399 boards with the following boot sequence: U-boot -> systemd-boot (EFI payload) -> kernel Which provides much more flex than plain extlinux conf f

Re: [PATCH] distro_bootcmd: change the default dtb search path to include default kernel dtbs directory

2021-09-17 Thread Qu Wenruo
On 2021/9/17 19:34, Mark Kettenis wrote: From: Qu Wenruo Date: Fri, 17 Sep 2021 19:02:35 +0800 When booting using U-boot -> systemd-boot (EFI payload) -> kernel on RK3399, systemd-boot by some bug can't execute its "devicetree" key correctly to load its proper dtb from files. In that case,

Re: [PATCH] distro_bootcmd: change the default dtb search path to include default kernel dtbs directory

2021-09-17 Thread Mark Kettenis
> From: Qu Wenruo > Date: Fri, 17 Sep 2021 19:02:35 +0800 > > When booting using U-boot -> systemd-boot (EFI payload) -> kernel on > RK3399, systemd-boot by some bug can't execute its "devicetree" key > correctly to load its proper dtb from files. > > In that case, it will use fallback dtb from

Re: [PATCH 2/3] efi_loader: GOP: Add 30bpp support

2021-09-17 Thread Heinrich Schuchardt
On 9/17/21 11:23 AM, Mark Kettenis wrote: From: Heinrich Schuchardt Date: Fri, 17 Sep 2021 04:56:31 +0200 Hi Heinrich, On 9/16/21 3:01 PM, Mark Kettenis wrote: Provide correct framebuffer information for 30bpp modes. This is not enough to get a correct GOP implementation for the 30bpp

[PATCH] net: update NXP copyright text

2021-09-17 Thread Vladimir Oltean
NXP Legal insists that the following are not fine: - Saying "NXP Semiconductors" instead of "NXP", since the company's registered name is "NXP" - Putting a "(c)" sign in the copyright string - Putting a comma in the copyright string The only accepted copyright string format is "Copyright NXP

Systemd-boot not properly loading device tree, when loaded by U-boot (ARM64, tested on RK3399)

2021-09-17 Thread Qu Wenruo
Hi, I'm recently testing booting my RK3399 boards with the following boot sequence: U-boot -> systemd-boot (EFI payload) -> kernel Which provides much more flex than plain extlinux conf from U-boot. (More choice, easier to write config, runtime kernel change). So far "kernel" and "initramfs" k

Re: [PATCH] arm: rpi: perform XHCI firmware upload only once

2021-09-17 Thread nicolas saenz julienne
On Fri, 2021-09-17 at 10:19 +0200, Marek Szyprowski wrote: > XHCI firmware upload must be performed only once after initializing the > PCI bridge. This fixes USB stack initialization after calling "usb stop; > usb start" on Raspberry Pi 4B. > > Signed-off-by: Marek Szyprowski > --- Reviewed-by:

  1   2   >