Re: [PATCH] Revert "arm: bootm: Disable LMB reservation for command line and board info on arm64"

2021-08-08 Thread Wolfgang Denk
Dear Marek, In message <03720507-5ea4-0fb9-0549-37df3128b...@denx.de> you wrote: > > The IMX thing is odd indeed and raises a bigger question -- what is the > "right" amount of stack to reserve ? In the original U-Boot design (which is still documented in the README, see section "Memory Manageme

Re: [PATCH] xilinx: Disable ARCH_FIXUP_FDT_MEMORY

2021-08-08 Thread Michal Simek
On 8/6/21 8:46 PM, Tom Rini wrote: > On Fri, Aug 06, 2021 at 02:22:56PM +0200, Michal Simek wrote: > >> Based on DT spec you can have one memory node which multiple ranges or >> multiple nodes. >> fdt_fixup_memory_banks() is not implemented in a correct way when multiple >> memory nodes are pre

Re: [PATCH 05/12] mmc: nds32: ftsdc010: Convert to livetree

2021-08-08 Thread Rick Chen
> From: U-Boot On Behalf Of Simon Glass > Sent: Saturday, August 07, 2021 9:24 PM > To: U-Boot Mailing List > Cc: Tom Rini ; Simon Glass ; Jaehoon > Chung ; Macpaul Lin ; Peng Fan > > Subject: [PATCH 05/12] mmc: nds32: ftsdc010: Convert to livetree > > Use the livetree API for this driver. > >

Please pull u-boot-dm

2021-08-08 Thread Simon Glass
Hi Tom, https://source.denx.de/u-boot/custodians/u-boot-dm/-/pipelines/8631 The following changes since commit 8f07f5376a21a57ce191e5548b1a30c9709994d3: Merge tag 'video-2021-08-05' of https://source.denx.de/u-boot/custodians/u-boot-video (2021-08-06 13:46:40 -0400) are available in the Git r

Re: [PATCH] dm: migrate the dm_warn to use the log macro

2021-08-08 Thread Simon Glass
On Fri, 30 Jul 2021 at 04:13, Patrick Delaunay wrote: > > Migrate the dm_warn function to log macro with > LOGC_DM category and LOGL_WARNING level. > > This macro allows filtering with log command and allows > output on all log backend. > > Signed-off-by: Patrick Delaunay > --- > > drivers/core/

Re: [PATCH] patman: Avoid blank lines between tags

2021-08-08 Thread Simon Glass
In some cases 'patman status' leaves a blank line between the sign-off and the tags it collects from patchwork. Fix this and add a test. Signed-off-by: Simon Glass --- tools/patman/func_test.py | 23 ++- tools/patman/patchstream.py | 7 ++- 2 files changed, 28 inserti

Re: [PATCH] dm: core: Don't allow uclass use before ready

2021-08-08 Thread Simon Glass
At present it is possible to call uclass_get() before driver model is inited. In fact this happens on x86 boards which use Intel FSPv1, since mrccache_get_region() tries to get the SPI flash device very early during init. This has always been undefined behaviour. Previously it generally worked, i.

Re: [PATCH] dtoc: Correct the intarray-widening test case

2021-08-08 Thread Simon Glass
Hi Simon, On 8/2/21 10:37 AM, Simon Glass wrote: > This case was intended to check that widening an int array with an int > does nothing. Fix it. > > Reported-by: Walter Lozano > Signed-off-by: Simon Glass > --- > > tools/dtoc/test_fdt.py | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(

[PATCH 01/23] mmc: Rename MMC_SUPPORT to MMC

2021-08-08 Thread Simon Glass
Rename these options so that CONFIG_IS_ENABLED can be used with them. Signed-off-by: Simon Glass --- arch/arm/Kconfig| 4 ++-- arch/arm/cpu/armv7/ls102xa/spl.c| 2 +- arch/arm/cpu/armv8/fsl-layerscape/spl.c | 2 +- arch/arm/

[PATCH 23/23] net: Move network rules to drivers/net

2021-08-08 Thread Simon Glass
Drop these from the top-level Makefile and also move the phy rule into drivers/net/Makefile which is where it belongs. Signed-off-by: Simon Glass --- Makefile | 3 --- drivers/Makefile | 4 +--- drivers/net/Makefile | 3 +++ 3 files changed, 4 insertions(+), 6 deletions(-) diff

[PATCH 22/23] net: Rename SPL_NET_SUPPORT to SPL_NET

2021-08-08 Thread Simon Glass
Rename this option so that CONFIG_IS_ENABLED can be used with it. Signed-off-by: Simon Glass --- common/Makefile | 2 +- common/spl/Kconfig | 10 +- common/spl/Makefile | 4 ++-- configs/am335x_boneblack_vboot_defconf

Re: [PATCH] Revert "arm: bootm: Disable LMB reservation for command line and board info on arm64"

2021-08-08 Thread Tom Rini
On Thu, Jul 29, 2021 at 09:22:02AM +0200, Jan Kiszka wrote: > From: Jan Kiszka > > This reverts commit 2359fa7a87848626bcbd3399e92c657595880cd7. > > While the goal is valid and there is surely unused memory in that area, > we also have a lot of crucial things still located at the top-of-memory

[PATCH 19/23] i2c: Convert CONFIG_POWER_I2C et al to Kconfig

2021-08-08 Thread Simon Glass
This converts the following to Kconfig: CONFIG_POWER_I2C CONFIG_POWER_LEGACY They are handled at the same time due to a dependency between them. Update the Makefile rule to use legacy power only in U-Boot proper. Unfortunately a separate rule is needed in SPL to be able to build legacy powe

[PATCH 21/23] power: Refactor Makefile rules

2021-08-08 Thread Simon Glass
Move the power/ rules into drivers/power to avoid clutter in the Makefile and drivers/Makefile files. We must select SPL_POWER if SPL_POWER_DOMAIN is used, since the two are currently independent and boards do not necessarily enable SPL_POWER. Add a TPL_POWER as well, as that is used by one board

[PATCH 18/23] power: Rename CONFIG_POWER to CONFIG_POWER_LEGACY

2021-08-08 Thread Simon Glass
This option is used in pre-driver model code and much of it has never been converted to driver model. We want to add a new option to enable power support, so we can use a simple rule in the Makefile. Rename this one, which is really about a particular implementation of power. Also update the pmic

Re: [PULL] u-boot-sh/master

2021-08-08 Thread Tom Rini
On Sun, Aug 08, 2021 at 01:39:30PM +0200, Marek Vasut wrote: > The following changes since commit 8f07f5376a21a57ce191e5548b1a30c9709994d3: > > Merge tag 'video-2021-08-05' of > https://source.denx.de/u-boot/custodians/u-boot-video (2021-08-06 13:46:40 > -0400) > > are available in the Git rep

[PATCH 20/23] power: Add a POWER config

2021-08-08 Thread Simon Glass
At present we have SPL_POWER but not piain POWER. This works because there is a special build rule in Makefile that always includes the drivers/power directory. It is better to have all driver directories included by drivers/Makefile and there is already a rule in there for this purpose. It just n

[PATCH 15/23] tegra: gpio: Drop use of CONFIG_xxx variables

2021-08-08 Thread Simon Glass
It is not a good idea to use things called CONFIG_xxx in the source code since this prefix is reserved for use by Kconfig. Rename these variables. Signed-off-by: Simon Glass --- drivers/gpio/tegra_gpio.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/gp

[PATCH 17/23] power: Tidy up #undef of CONFIG_DM_PMIC

2021-08-08 Thread Simon Glass
Add a proper Kconfig option for SPL so we can remove the hack in some of the board config files. This involves adding CONFIG_SPL_DM_PMIC to some of the configs as well as updateing the Makefile rule for PMIC_RK8XX to exclude SPL. Signed-off-by: Simon Glass --- configs/imx8mm-cl-iot-gate_defcon

[PATCH 16/23] gpio: Add a GPIO config

2021-08-08 Thread Simon Glass
At present we have SPL_GPIO and TPL_GPIO but not piain GPIO. This works because there is a special build rule in Makefile that always includes the drivers/gpio directory. It is better to have all driver directories included by drivers/Makefile and there is already a rule in there for this purpose.

[PATCH 14/23] Makefile: Move drivers/dma/ into drivers/Makefile

2021-08-08 Thread Simon Glass
This rule should not be in the top-level Makefile. Move it, making use of the new LEGACY_DMA Kconfig. Signed-off-by: Simon Glass --- Makefile| 1 - drivers/Makefile| 3 ++- drivers/dma/Kconfig | 1 + 3 files changed, 3 insertions(+), 2 deletions(-) diff --git a/Makefile b/Makef

[PATCH 13/23] Convert CONFIG_DMA_LPC32XX to Kconfig

2021-08-08 Thread Simon Glass
This converts the following to Kconfig: CONFIG_DMA_LPC32XX Signed-off-by: Simon Glass --- configs/devkit3250_defconfig | 1 + drivers/dma/Kconfig | 11 +++ include/configs/devkit3250.h | 3 --- scripts/config_whitelist.txt | 1 - 4 files changed, 12 insertions(+), 4 delet

[PATCH 08/23] ppc: Rename MPC8XXX_INIT_DDR_SUPPORT to MPC8XXX_INIT_DDR

2021-08-08 Thread Simon Glass
Rename these options so that CONFIG_IS_ENABLED can be used with them. Signed-off-by: Simon Glass --- common/spl/Kconfig | 6 +++--- configs/P1010RDB-PA_36BIT_NAND_defconfig | 2 +- configs/P1010RDB-PA_36BIT_SDCARD_defconfig | 2 +- configs

[PATCH 09/23] sata: Rename SATA_SUPPORT to SATA

2021-08-08 Thread Simon Glass
Rename this options so that CONFIG_IS_ENABLED can be used with it. Signed-off-by: Simon Glass --- arch/arm/mach-mvebu/Kconfig | 2 +- arch/arm/mach-mvebu/spl.c| 10 +- arch/arm/mach-zynqmp/spl.c | 2 +- arch/powerpc/cpu/mpc83xx/Kconfig | 4 ++-- common/spl/Kconfig

[PATCH 12/23] ti: Convert CONFIG_TI_EDMA3 to Kconfig

2021-08-08 Thread Simon Glass
This converts the following to Kconfig: CONFIG_TI_EDMA3 Signed-off-by: Simon Glass --- configs/k2e_evm_defconfig| 1 + configs/k2e_hs_evm_defconfig | 1 + configs/k2g_evm_defconfig| 1 + configs/k2g_hs_evm_defconfig | 1 + configs/k2hk_evm_defconfig

[PATCH 11/23] dma: Add a Kconfig for legacy DMA

2021-08-08 Thread Simon Glass
We cannot use the existing DMA config for the MCD driver because it is not migrated to driver model. In order to move it to drivers/Makefile we need some sort of option for it. Add a new DMA_LEGACY option, which also acts as a signal that it should be migrated. Enable this for devkit3250 which use

[PATCH 10/23] Makefile: Unify the rules for BOOTCOUNT_LIMIT

2021-08-08 Thread Simon Glass
Use a single rule that works for all phases. Signed-off-by: Simon Glass --- drivers/Makefile | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/Makefile b/drivers/Makefile index 3709cf781d8..1bce26877bc 100644 --- a/drivers/Makefile +++ b/drivers/Makefile @@ -1,5 +1,

[PATCH 06/23] spi: Rename SPI_SUPPORT to SPI

2021-08-08 Thread Simon Glass
Rename these options so that CONFIG_IS_ENABLED can be used with them. Signed-off-by: Simon Glass --- Makefile | 1 - arch/Kconfig | 2 +- arch/arm/Kconfig | 2 +- arc

[PATCH 07/23] Rename CACHE_SUPPORT to CACHE

2021-08-08 Thread Simon Glass
Rename this option so that CONFIG_IS_ENABLED can be used with it. Oddly there is already an SPL_CACHE option. Drop it in favour of this one. Drop the special SPL Makefile rule which is now superfluous. Signed-off-by: Simon Glass --- common/spl/Kconfig | 8 +--- drivers/Makefile | 1 - 2

[PATCH 05/23] serial: Add a SERIAL config

2021-08-08 Thread Simon Glass
At present we have SPL_SERIAL and TPL_SERIAL but not piain SERIAL. This works because there is a special build rule in Makefile that always includes the drivers/serial directory. It is better to have all driver directories included by drivers/Makefile and there is already a rule in there for this

[PATCH 00/23] Makefile: Continue simplifying Makefile rules (Part B)

2021-08-08 Thread Simon Glass
At present a few of the Makefiles have confusing layouts, with separate sections for U-Boot proper and SPL/TPL. It is generally possible to have a simple rule of the form: obj-$(CONFIG_$(SPL_TPL_)NET) += net/ so that net/ is built for the different U-Boot phases based on which of CONFIG_NET, C

[PATCH 03/23] rtc: Rename RTC_SUPPORT to RTC

2021-08-08 Thread Simon Glass
Rename these options so that CONFIG_IS_ENABLED can be used with them. Signed-off-by: Simon Glass --- common/spl/Kconfig | 4 ++-- configs/chromebook_link64_defconfig | 2 +- configs/openpiton_riscv64_spl_defconfig | 2 +- configs/qemu-x86_64_defconfig | 2 +-

[PATCH 02/23] pch: Rename PCH_SUPPORT to PCH

2021-08-08 Thread Simon Glass
Rename these options so that CONFIG_IS_ENABLED can be used with them. Signed-off-by: Simon Glass --- arch/x86/cpu/apollolake/Kconfig| 6 +++--- common/spl/Kconfig | 4 ++-- configs/chromebook_link64_defconfig| 2 +- configs/chromebook_samus_tpl_defconfig | 4 ++--

Re: [PATCH] Revert "arm: bootm: Disable LMB reservation for command line and board info on arm64"

2021-08-08 Thread Tom Rini
On Sun, Aug 08, 2021 at 05:25:33PM +0200, Marek Vasut wrote: > On 8/8/21 4:54 PM, Tom Rini wrote: > > [...] > > > > > I expect it was not simply because up > > > > until rather recently we didn't have any checks for "don't overwrite > > > > specific areas of memory" other than right before firing

Re: [PATCH v1] imx: mkimage_fit_atf: replace @ with hyphen

2021-08-08 Thread Heiko Thiery
Hi all, Am Mo., 2. Aug. 2021 um 11:57 Uhr schrieb Peng Fan (OSS) : > > Hi Tim, > > > Subject: Re: [PATCH v1] imx: mkimage_fit_atf: replace @ with hyphen > > > > On Tue, Jul 27, 2021 at 12:17 PM Heiko Thiery > > wrote: > > > > > > Hi all, > > > > > > Am Mi., 5. Mai 2021 um 15:30 Uhr schrieb ZHIZHI

Re: [PATCH] Revert "arm: bootm: Disable LMB reservation for command line and board info on arm64"

2021-08-08 Thread Marek Vasut
On 8/8/21 4:54 PM, Tom Rini wrote: [...] I expect it was not simply because up until rather recently we didn't have any checks for "don't overwrite specific areas of memory" other than right before firing off the OS (and modify whatever memory you want to modify is a feature not a bug). The L

Re: [PATCH] Revert "arm: bootm: Disable LMB reservation for command line and board info on arm64"

2021-08-08 Thread Tom Rini
On Sun, Aug 08, 2021 at 04:28:14PM +0200, Marek Vasut wrote: > On 8/8/21 4:00 PM, Tom Rini wrote: > > On Sun, Aug 08, 2021 at 03:45:30PM +0200, Marek Vasut wrote: > > > On 8/6/21 6:49 PM, Tom Rini wrote: > > > > On Fri, Aug 06, 2021 at 12:22:43AM +0200, Marek Vasut wrote: > > > > > On 8/3/21 11:51

Re: [PATCH] Revert "arm: bootm: Disable LMB reservation for command line and board info on arm64"

2021-08-08 Thread Marek Vasut
On 8/8/21 4:00 PM, Tom Rini wrote: On Sun, Aug 08, 2021 at 03:45:30PM +0200, Marek Vasut wrote: On 8/6/21 6:49 PM, Tom Rini wrote: On Fri, Aug 06, 2021 at 12:22:43AM +0200, Marek Vasut wrote: On 8/3/21 11:51 PM, Tom Rini wrote: On Mon, Aug 02, 2021 at 05:27:59PM -0400, Tom Rini wrote: On Thu

Re: [PATCH] Revert "arm: bootm: Disable LMB reservation for command line and board info on arm64"

2021-08-08 Thread Tom Rini
On Sun, Aug 08, 2021 at 03:45:30PM +0200, Marek Vasut wrote: > On 8/6/21 6:49 PM, Tom Rini wrote: > > On Fri, Aug 06, 2021 at 12:22:43AM +0200, Marek Vasut wrote: > > > On 8/3/21 11:51 PM, Tom Rini wrote: > > > > On Mon, Aug 02, 2021 at 05:27:59PM -0400, Tom Rini wrote: > > > > > On Thu, Jul 29, 20

Re: [PATCH] Revert "arm: bootm: Disable LMB reservation for command line and board info on arm64"

2021-08-08 Thread Marek Vasut
On 8/6/21 6:49 PM, Tom Rini wrote: On Fri, Aug 06, 2021 at 12:22:43AM +0200, Marek Vasut wrote: On 8/3/21 11:51 PM, Tom Rini wrote: On Mon, Aug 02, 2021 at 05:27:59PM -0400, Tom Rini wrote: On Thu, Jul 29, 2021 at 09:22:02AM +0200, Jan Kiszka wrote: From: Jan Kiszka This reverts commit 235

Re: [PATCH] Revert "arm: bootm: Disable LMB reservation for command line and board info on arm64"

2021-08-08 Thread Marek Vasut
On 8/6/21 1:31 AM, Tom Rini wrote: On Thu, Aug 05, 2021 at 11:53:24PM +0200, Marek Vasut wrote: On 8/2/21 3:00 PM, Tom Rini wrote: On Mon, Aug 02, 2021 at 02:54:22AM +0200, Marek Vasut wrote: On 7/29/21 6:58 PM, Tom Rini wrote: [...] so when did rcar3 introduce something there that shouldn'

[PULL] u-boot-sh/master

2021-08-08 Thread Marek Vasut
The following changes since commit 8f07f5376a21a57ce191e5548b1a30c9709994d3: Merge tag 'video-2021-08-05' of https://source.denx.de/u-boot/custodians/u-boot-video (2021-08-06 13:46:40 -0400) are available in the Git repository at: git://source.denx.de/u-boot-sh.git master for you to fet

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

2021-08-08 Thread Pali Rohár
Christophe, could you please review this change? According to your commit https://source.denx.de/u-boot/u-boot/-/commit/907208c452999427cb2f43450308045bf8b42958 mpc8xx code will have to be maintained until at least 2027. On Sunday 08 August 2021 13:20:38 Pali Rohár wrote: > It is unknown why versi

[PATCH v2] powerpc: mpc: Put U-Boot version string at correct place by linker script

2021-08-08 Thread Pali Rohár
It is unknown why version string is placed at specific position on these powerpc mpc platforms. But there is no need to overload version_string symbol. Just use common definition of version_string and modify linker script to put it at "correct place". Signed-off-by: Pali Rohár --- Changes in v2: