Re: [PATCH] arm: kirkwood: Enable uart0 dm-pre-reloc for Kirkwood boards

2023-02-10 Thread Tony Dinh
Hi Stefan, Hi Pali, On Fri, Feb 10, 2023 at 9:29 AM Pali Rohár wrote: > > On Thursday 09 February 2023 17:37:54 Tony Dinh wrote: > > Hi Pali, > > > > On Thu, Feb 9, 2023 at 1:45 PM Pali Rohár wrote: > > > > > > On Thursday 09 February 2023 13:33:23 Tony Dinh wrote: > > > > Hi Stefan, > > > > >

Re: [PATCH] arm: kirkwood: Enable uart0 dm-pre-reloc for Kirkwood boards

2023-02-10 Thread Pali Rohár
On Friday 10 February 2023 13:38:44 Tony Dinh wrote: > Hi Stefan, > Hi Pali, > > On Fri, Feb 10, 2023 at 9:29 AM Pali Rohár wrote: > > > > On Thursday 09 February 2023 17:37:54 Tony Dinh wrote: > > > Hi Pali, > > > > > > On Thu, Feb 9, 2023 at 1:45 PM Pali Rohár wrote: > > > > > > > > On

Re: imx8mm USB Mass Storage fails through a HUB

2023-02-10 Thread Marcel Ziswiler
Hi Tim On Thu, 2023-02-09 at 11:58 -0800, Tim Harvey wrote: > Greetings, > > I'm finding that USB Mass Storage devices appear to fail detection on > imx8mm when hanging off a USB hub whereas other devices such as USB > ethernet work fine. > > u-boot=> usb start && usb tree > starting USB... >

Re: [PATCH v1] configs: colibri-imx7: Fix bad block table in flash configuration

2023-02-10 Thread Marcel Ziswiler
On Fri, 2023-02-10 at 11:33 +0100, Francesco Dolcini wrote: > From: Stefan Eichenberger > > Make sure that the bad block table in flash is used on Colibri iMX7. > Without this configuration enabled U-Boot corrupts the bad block table > and Linux will update the table on each reboot. The

Re: Pull request for efi-2023-04-rc2

2023-02-10 Thread Tom Rini
On Fri, Feb 10, 2023 at 07:43:56PM +0100, Heinrich Schuchardt wrote: > Dear Tom, > > The following changes since commit 81e8a51cee2b265e26272f0c67518c4844baa36c: > > Merge branch '2023-02-09-minor-updates' (2023-02-09 16:31:36 -0500) > > are available in the Git repository at: > >

Re: [PATCH v3 68/76] usb: Drop unused fotg210 gadget

2023-02-10 Thread Samuel Holland
On 2/1/23 14:20, Simon Glass wrote: > This is not used and appears to be associated with the faraday board which > has been removed. Drop the driver and Kconfig options. > > Signed-off-by: Simon Glass > --- > > (no changes since v1) > > drivers/usb/gadget/Makefile | 3 +- >

Re: [PATCH] arm: kirkwood: Enable uart0 dm-pre-reloc for Kirkwood boards

2023-02-10 Thread Tony Dinh
Hi Pali, On Fri, Feb 10, 2023 at 1:44 PM Pali Rohár wrote: > > On Friday 10 February 2023 13:38:44 Tony Dinh wrote: > > Hi Stefan, > > Hi Pali, > > > > On Fri, Feb 10, 2023 at 9:29 AM Pali Rohár wrote: > > > > > > On Thursday 09 February 2023 17:37:54 Tony Dinh wrote: > > > > Hi Pali, > > > > >

[PATCH] arm: kirkwood: Enable uart0 dm-pre-reloc for Pogoplug V4 board

2023-02-10 Thread Tony Dinh
When DM_SERIAL is enabled, the device-tree tag u-boot,dm-pre-reloc is required for this board to boot over UART with kwboot. Enable this in kirkwood-pogoplug-series-4-u-boot.dtsi. Signed-off-by: Tony Dinh --- arch/arm/dts/kirkwood-pogoplug-series-4-u-boot.dtsi | 7 +++ 1 file changed, 7

Re: [PATCH v3 68/76] usb: Drop unused fotg210 gadget

2023-02-10 Thread Tom Rini
On Fri, Feb 10, 2023 at 05:33:13PM -0600, Samuel Holland wrote: > On 2/1/23 14:20, Simon Glass wrote: > > This is not used and appears to be associated with the faraday board which > > has been removed. Drop the driver and Kconfig options. > > > > Signed-off-by: Simon Glass > > --- > > > > (no

Re: [PATCH] binman: Avoid requiring a home directory on startup

2023-02-10 Thread Mark Kettenis
> From: Simon Glass > Date: Fri, 10 Feb 2023 13:59:46 -0700 > > This is needed to download tools, but we may not need to do this. At > present binman fails to start if HOME is not set. > > Use the current directory as a default to avoid this. > > Signed-off-by: Simon Glass Not new, but

[PATCH] binman: Avoid requiring a home directory on startup

2023-02-10 Thread Simon Glass
This is needed to download tools, but we may not need to do this. At present binman fails to start if HOME is not set. Use the current directory as a default to avoid this. Signed-off-by: Simon Glass --- tools/binman/bintool.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 1/1] efi_loader: fix efi_ecpt_register()

2023-02-10 Thread Heinrich Schuchardt
num_entries should be unsigned to avoid warnings. As the target field is u16 we should use this type. lib/efi_loader/efi_conformance.c: In function ‘efi_ecpt_register’: lib/efi_loader/efi_conformance.c:30:33: warning: conversion to ‘long unsigned int’ from ‘int’ may change the

[PATCH 1/1] efi_loader: fix efi_ecpt_register()

2023-02-10 Thread Heinrich Schuchardt
num_entries should be unsigned to avoid warnings. As the target field is u16 we should use this type. lib/efi_loader/efi_conformance.c: In function ‘efi_ecpt_register’: lib/efi_loader/efi_conformance.c:30:33: warning: conversion to ‘long unsigned int’ from ‘int’ may change the

[PATCH 2/8] efi_loader: static functions in efi_boottime.c

2023-02-10 Thread Heinrich Schuchardt
Make functions that are no used externally static. Signed-off-by: Heinrich Schuchardt --- lib/efi_loader/efi_boottime.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/lib/efi_loader/efi_boottime.c b/lib/efi_loader/efi_boottime.c index ba28989f36..caaab685ee 100644 ---

[PATCH 4/8] efi_loader: static functions in efi_runtime.c

2023-02-10 Thread Heinrich Schuchardt
Functions that are not used externally should be static. Signed-off-by: Heinrich Schuchardt --- lib/efi_loader/efi_runtime.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/efi_loader/efi_runtime.c b/lib/efi_loader/efi_runtime.c index ad2ab825d1..cee96bfc7f 100644

[PATCH 5/8] efi_loader: provide definition for efi_add_known_memory()

2023-02-10 Thread Heinrich Schuchardt
We should provide a definition in an include for efi_add_known_memory(). Signed-off-by: Heinrich Schuchardt --- include/efi_loader.h | 7 +++ 1 file changed, 7 insertions(+) diff --git a/include/efi_loader.h b/include/efi_loader.h index 4560b0d04c..c664d6cdf2 100644 ---

[PATCH 3/8] efi_loader: static functions in efi_console.c

2023-02-10 Thread Heinrich Schuchardt
Define function set_shift_mask() as static as it is not used externally. Signed-off-by: Heinrich Schuchardt --- lib/efi_loader/efi_console.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/efi_loader/efi_console.c b/lib/efi_loader/efi_console.c index

[PATCH 0/8] efi_loader: fix function definitions

2023-02-10 Thread Heinrich Schuchardt
When building with 'make W=1' several warnings concerning function definitions are shown. Fix them all. Heinrich Schuchardt (8): efi_loader: static functions in helloworld.c efi_loader: static functions in efi_boottime.c efi_loader: static functions in efi_console.c efi_loader: static

[PATCH 1/8] efi_loader: static functions in helloworld.c

2023-02-10 Thread Heinrich Schuchardt
Make functions that are not used externally static. Signed-off-by: Heinrich Schuchardt --- lib/efi_loader/helloworld.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/lib/efi_loader/helloworld.c b/lib/efi_loader/helloworld.c index d565f32745..49fa8cc2f0 100644 ---

[PATCH 1/4] phy: phy-mtk-tphy: add support mt8195

2023-02-10 Thread Chunfeng Yun
The T-PHY controller is designed to use use PLL integer mode, but in fact use fractional mode for some ones on mt8195 by mistake, this causes signal degradation (e.g. eye diagram test fail), fix it by switching PLL to 26Mhz from default 48Mhz to improve signal quality. Signed-off-by: Chunfeng Yun

[PATCH 2/4] usb: xhci-mtk: modify the SOF/ITP interval for mt8195

2023-02-10 Thread Chunfeng Yun
There are 4 USB controllers on MT8195, the controllers (IP1~IP3, exclude IP0) have a wrong default SOF/ITP interval which is calculated from the frame counter clock 24Mhz by default, but in fact, the frame counter clock is 48Mhz, so we shall set the accurate interval according to 48Mhz for those

[PATCH 4/4] dt-bindings: usb: mtk-xhci: add support mt8195

2023-02-10 Thread Chunfeng Yun
Add a new compatible for mt8195 to add a workaround for hardware issue. Signed-off-by: Chunfeng Yun --- doc/device-tree-bindings/usb/mediatek,mtk-xhci.txt | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/doc/device-tree-bindings/usb/mediatek,mtk-xhci.txt

[PATCH 3/4] dt-bindings: phy-mtk-tphy: add support mt8195

2023-02-10 Thread Chunfeng Yun
Add a new compatible for mt8195 to add a workaround for hardware issue. Signed-off-by: Chunfeng Yun --- doc/device-tree-bindings/phy/phy-mtk-tphy.txt | 1 + 1 file changed, 1 insertion(+) diff --git a/doc/device-tree-bindings/phy/phy-mtk-tphy.txt

[PATCH 7/8] efi_loader: make gop_blt() static

2023-02-10 Thread Heinrich Schuchardt
This function is not used externally. Signed-off-by: Heinrich Schuchardt --- lib/efi_loader/efi_gop.c | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/lib/efi_loader/efi_gop.c b/lib/efi_loader/efi_gop.c index d1dc2f22d0..778b693f98 100644 ---

[PATCH 6/8] efi_loader: include definition of allow_unaligned()

2023-02-10 Thread Heinrich Schuchardt
Add missing include. Signed-off-by: Heinrich Schuchardt --- lib/efi_loader/efi_setup.c | 1 + 1 file changed, 1 insertion(+) diff --git a/lib/efi_loader/efi_setup.c b/lib/efi_loader/efi_setup.c index f0f01d3b1d..69aaefab63 100644 --- a/lib/efi_loader/efi_setup.c +++

[PATCH 8/8] efi_loader: static efi_query_variable_info_runtime()

2023-02-10 Thread Heinrich Schuchardt
This function is not used externally and hence should be static. Signed-off-by: Heinrich Schuchardt --- lib/efi_loader/efi_variable.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/efi_loader/efi_variable.c b/lib/efi_loader/efi_variable.c index 7c32adf6e5..548b42bfa2

Re: [PATCH] schemas: Add schema for firmware logs

2023-02-10 Thread Ard Biesheuvel
On Thu, 9 Feb 2023 at 19:05, Simon Glass wrote: > > Hi Jan, > > On Wed, 8 Feb 2023 at 01:15, Jan Lübbe wrote: > > > > On Tue, 2023-02-07 at 11:39 -0700, Simon Glass wrote: > > > Hi Jan, > > > > > > On Tue, 7 Feb 2023 at 08:39, Jan Lübbe wrote: > > > > > > > > On Tue, 2023-02-07 at 06:38 -0700,

[PATCH] ARM: meson: odroid-go-ultra: setup PMIC regulators are board init

2023-02-10 Thread Neil Armstrong
MASS_STORAGE=y # CONFIG_CMD_SETEXPR is not set +CONFIG_CMD_PMIC=y CONFIG_CMD_REGULATOR=y CONFIG_OF_CONTROL=y CONFIG_SYS_RELOC_GD_ENV_ADDR=y --- base-commit: d1f5dbe6645ad51e318dd322033fe6a08bce357b change-id: 20230210-u-boot-odroid-go-ultra-pmics-setup-0e79dad28de6 Best regards, -- Neil Armstrong

Re: [PATCH 2/4] usb: xhci-mtk: modify the SOF/ITP interval for mt8195

2023-02-10 Thread Marek Vasut
On 2/10/23 09:33, Chunfeng Yun wrote: [...] @@ -50,6 +50,27 @@ #define IPPC_U3_CTRL(p) (IPPC_U3_CTRL_0P + ((p) * 0x08)) #define IPPC_U2_CTRL(p) (IPPC_U2_CTRL_0P + ((p) * 0x08)) +/* xHCI CSR */ +#define LS_EOF_CFG 0x930 +#define LSEOF_OFFSET 0x89 +

Re: [PATCH v1] configs: colibri-imx7: Fix bad block table in flash configuration

2023-02-10 Thread Michael Nazzareno Trimarchi
On Fri, Feb 10, 2023 at 11:34 AM Francesco Dolcini wrote: > > From: Stefan Eichenberger > > Make sure that the bad block table in flash is used on Colibri iMX7. > Without this configuration enabled U-Boot corrupts the bad block table > and Linux will update the table on each reboot. The

[PATCH v2 0/3] fdt: introduce "fsapply" command

2023-02-10 Thread Andre Przywara
Hi, updating the series, basically just changing the name of the command to "fsapply", as suggested by Simon. The first two patches are still the same fixes, Simon said he merged them to u-boot-dm in July already, but I don't see them there or in master. I am still looking into a test for

[PATCH] i2c: correct I2C deblock logic

2023-02-10 Thread haibo . chen
From: Haibo Chen Current code use dm_gpio_get_value() to get SDA and SCL value, and the value depends on the flag GPIOD_ACTIVE_LOW. When toggle SCL to wait slave release SDA, the SDA are config as GPIOD_IS_IN, and whether contain the GPIOD_ACTIVE_LOW depends on the DTS setting. Usually, for I2C

[PATCH v1] configs: colibri-imx7: Fix bad block table in flash configuration

2023-02-10 Thread Francesco Dolcini
From: Stefan Eichenberger Make sure that the bad block table in flash is used on Colibri iMX7. Without this configuration enabled U-Boot corrupts the bad block table and Linux will update the table on each reboot. The corruption occurs because if CONFIG_SYS_NAND_USE_FLASH_BBT is not set, U-boot

[PATCH v2 2/3] cmd: fdt: allow standalone "fdt move"

2023-02-10 Thread Andre Przywara
At the moment every subcommand of "fdt", except "addr" itself, requires the DT address to be set first. We explicitly check for that before even comparing against the subcommands' string. This early bailout also affects the "move" subcommand, even though that does not require or rely on a previous

[PATCH v2 1/3] cmd: fdt: move: Use map_sysmem to convert pointers

2023-02-10 Thread Andre Przywara
The "fdt move" subcommand was using the provided DTB addresses directly, without trying to "map" them into U-Boot's address space. This happened to work since on the vast majority of "real" platforms there is a simple 1:1 mapping of VA to PAs, so either value works fine. However this is not true

[PATCH v2 3/3] fdt: introduce fsapply command

2023-02-10 Thread Andre Przywara
Explicitly specifying the exact filenames of devicetree overlays files on a U-Boot command line can be quite tedious for users, especially when it should be made persistent for every boot. To simplify the task of applying (custom) DT overlays, introduce a "fdt fsapply" subcommand, that iterates a

Re: [PATCH 1/5] menu: remove CTRL+C to quit

2023-02-10 Thread Heinrich Schuchardt
On 2/2/23 10:24, Masahisa Kojima wrote: On the sandbox called without "--terminal raw" CTRL+C leaves U-Boot, "ESC/CTRL+C to quit" is misleading. Let's remove CTRL+C to quit key handling from bootmenu and eficonfig menu. I guess my review was misleading. CTRL+C on the sandbox leaves the

Re: i.MX8M binman

2023-02-10 Thread Peng Fan
+Marek, I heard that from Marek on IRC, but Marek ask me to reach you. Actually I am not sure what is the issue with i.MX8M binman node. Thanks, Peng. On 2/7/2023 9:38 PM, Simon Glass wrote: Hi Peng, On Mon, 6 Feb 2023 at 05:43, Peng Fan wrote: Hi Simon, I heard that you found i.MX8M

Re: [PATCH 4/5] eficonfig: include EFI_STATUS string in error message

2023-02-10 Thread Heinrich Schuchardt
On 2/2/23 10:24, Masahisa Kojima wrote: Current eficonfig_print_msg() does not show the return value of EFI Boot/Runtime Services when the service call fails. With this commit, user can know EFI_STATUS in the error message. Why do we need function eficonfig_print_msg()? I cannot see why the

Re: [PATCH 1/5] mtd/spinand: rework detect procedure for different READ_ID operation

2023-02-10 Thread Michael Nazzareno Trimarchi
Hi I will review On Thu, Feb 9, 2023 at 5:52 PM Tom Rini wrote: > > On Thu, Feb 09, 2023 at 10:24:47AM +0100, Frieder Schrempf wrote: > > Hi, > > > > On 10.01.23 12:58, Frieder Schrempf wrote: > > > From: Mikhail Kshevetskiy > > > > > > Currently there are 3 different variants of read_id

Re: [PATCH 06/11] efi: Support copy framebuffer

2023-02-10 Thread Heinrich Schuchardt
On 2/5/23 20:46, Simon Glass wrote: Add support for this to EFI in case it becomes useful. At present it just slows things down. Enable CONFIG_VIDEO_COPY to turn it on. Signed-off-by: Simon Glass --- arch/x86/dts/efi-x86_app.dts | 1 + drivers/video/efi.c | 11 +++ 2

Re: [PATCH v2 025/169] Correct SPL uses of ARCH_MVEBU

2023-02-10 Thread Stefan Roese
On 2/9/23 18:23, Tom Rini wrote: On Sun, Feb 05, 2023 at 03:36:11PM -0700, Simon Glass wrote: This converts 2 usages of this option to the non-SPL form, since there is no SPL_ARCH_MVEBU defined in Kconfig Signed-off-by: Simon Glass --- (no changes since v1) drivers/mmc/mv_sdhci.c | 4

[PATCH 1/3] mmc: mv_sdhci: Simplify call to sdhci_mvebu_mbus_config()

2023-02-10 Thread Stefan Roese
This driver already depends on CONFIG_ARCH_MVEBU, so there is no need to have some checks for this Kconfig symbol in the driver itself. Let's remove these superfluous checks. Signed-off-by: Stefan Roese Cc: Tom Rini Cc: Simon Glass Cc: Peng Fan Cc: Jaehoon Chung --- drivers/mmc/mv_sdhci.c |

[PATCH 2/3] mmc: mv_sdhci: Remove CONFIG_MMC_SDHCI_IO_ACCESSORS support

2023-02-10 Thread Stefan Roese
CONFIG_MMC_SDHCI_IO_ACCESSORS is not supported and/or used by this driver so let's remove these unused parts completely. Signed-off-by: Stefan Roese Cc: Tom Rini Cc: Simon Glass Cc: Peng Fan Cc: Jaehoon Chung --- drivers/mmc/mv_sdhci.c | 8 1 file changed, 8 deletions(-) diff

Re: rk3328 broken between v2022.04 and v2022.07 - still not working with v2023.01

2023-02-10 Thread Jagan Teki
On Fri, Feb 10, 2023 at 6:01 PM Janpieter Sollie wrote: > > Hello, > > After a few years, I tried updating my u-boot 2021 to a more recent version, > as the linux kernel > complained "ATF needs update". > The device I'm using is a nanopi neo3 2GB (more or less the same compared to > NanoPi R2S,

Re: rk3328 broken between v2022.04 and v2022.07 - still not working with v2023.01

2023-02-10 Thread Peter Robinson
On Fri, Feb 10, 2023 at 12:31 PM Janpieter Sollie wrote: > > Hello, > > After a few years, I tried updating my u-boot 2021 to a more recent version, > as the linux kernel > complained "ATF needs update". > The device I'm using is a nanopi neo3 2GB (more or less the same compared to > NanoPi

Re: [PATCH v2 3/3] fdt: introduce fsapply command

2023-02-10 Thread Lothar Waßmann
Hi, On Fri, 10 Feb 2023 11:02:13 + Andre Przywara wrote: > Explicitly specifying the exact filenames of devicetree overlays files > on a U-Boot command line can be quite tedious for users, especially > when it should be made persistent for every boot. > > To simplify the task of applying

Re: [PATCH v2 2/3] cmd: fdt: allow standalone "fdt move"

2023-02-10 Thread Lothar Waßmann
Hi, On Fri, 10 Feb 2023 11:02:12 + Andre Przywara wrote: > At the moment every subcommand of "fdt", except "addr" itself, requires > the DT address to be set first. We explicitly check for that before even > comparing against the subcommands' string. > This early bailout also affects the

[PATCH 3/3] mmc: mv_sdhci: Depend on DM_MMC

2023-02-10 Thread Stefan Roese
All build targets using this driver already use DM_MMC. So let's depend this driver on this Kconfig symbol and remove the non-DM driver part. Signed-off-by: Stefan Roese Cc: Tom Rini Cc: Simon Glass Cc: Peng Fan Cc: Jaehoon Chung --- drivers/mmc/Kconfig| 1 + drivers/mmc/mv_sdhci.c |

Re: [PATCH 02/88] treewide: Correct invalid Kconfig syntax and warnings

2023-02-10 Thread Alexey Brodkin
Hi Tom, Daniel, all, > On 1/27/23 14:45, Tom Rini wrote: > > On Mon, Jan 23, 2023 at 02:59:05PM -0700, Simon Glass wrote: > > > >> In several places a 'select' is used to select a choice, which is not > >> supported by Kconfig. In other places, the filename for the 'source' > >> command is not in

[PATCH v1 1/1] tools: env: use /run to store lockfile

2023-02-10 Thread Tim Lee
According this issue https://github.com/ppp-project/ppp/issues/339. Eventually, the dt-utils changed lock directory to fix missing /var/lock directory error then make dt-utils can run normally. We also have a similar issue with these two utilities fw_printenv and fw_setenv will failed when the

Re: [PATCH v2 031/169] Correct SPL use of ATMEL_PIO4

2023-02-10 Thread Tom Rini
On Fri, Feb 10, 2023 at 09:25:22AM +0200, Eugen Hristev wrote: > On 2/9/23 19:36, Tom Rini wrote: > > On Sun, Feb 05, 2023 at 03:36:17PM -0700, Simon Glass wrote: > > > > > This converts 1 usage of this option to the non-SPL form, since there is > > > no SPL_ATMEL_PIO4 defined in Kconfig > > > >

Re: [PATCH v2 093/169] Correct SPL use of FSP_VERSION2

2023-02-10 Thread Tom Rini
On Sun, Feb 05, 2023 at 03:40:02PM -0700, Simon Glass wrote: > This converts 1 usage of this option to the non-SPL form, since there is > no SPL_FSP_VERSION2 defined in Kconfig > > Signed-off-by: Simon Glass This is used to avoid code on chromebook_coral in SPL. -- Tom signature.asc

Re: [PATCH v2 099/169] Correct SPL uses of I8259_PIC

2023-02-10 Thread Tom Rini
On Sun, Feb 05, 2023 at 03:40:08PM -0700, Simon Glass wrote: > This converts 4 usages of this option to the non-SPL form, since there is > no SPL_I8259_PIC defined in Kconfig > > Signed-off-by: Simon Glass This is used to avoid code in SPL. -- Tom signature.asc Description: PGP signature

Re: [PATCH v2 127/169] Correct SPL use of QFW_PIO

2023-02-10 Thread Tom Rini
On Sun, Feb 05, 2023 at 03:40:36PM -0700, Simon Glass wrote: > This converts 1 usage of this option to the non-SPL form, since there is > no SPL_QFW_PIO defined in Kconfig > > Signed-off-by: Simon Glass This is used to avoid code in SPL. -- Tom signature.asc Description: PGP signature

Re: [PATCH v2 031/169] Correct SPL use of ATMEL_PIO4

2023-02-10 Thread Eugen Hristev
On 2/10/23 14:41, Tom Rini wrote: On Fri, Feb 10, 2023 at 09:25:22AM +0200, Eugen Hristev wrote: On 2/9/23 19:36, Tom Rini wrote: On Sun, Feb 05, 2023 at 03:36:17PM -0700, Simon Glass wrote: This converts 1 usage of this option to the non-SPL form, since there is no SPL_ATMEL_PIO4 defined in

[PATCH v4 09/11] test: Add a way to set the environment for a pytest

2023-02-10 Thread Simon Glass
This is useful when we need to control a particular environment variable. Add a way to handle this. Signed-off-by: Simon Glass --- Changes in v4: - Add new patch with a way to set the environment for a pytest test/py/multiplexed_log.py | 5 +++-- test/py/u_boot_utils.py| 5 +++-- 2 files

Re: rk3328 broken between v2022.04 and v2022.07 - still not working with v2023.01

2023-02-10 Thread Jonas Karlman
Hi, On 2023-02-10 16:32, Janpieter Sollie wrote: > Op 10/02/2023 om 13:41 schreef Peter Robinson: >> On Fri, Feb 10, 2023 at 12:31 PM Janpieter Sollie >> wrote: >>> Hello, >>> >>> After a few years, I tried updating my u-boot 2021 to a more recent >>> version, as the linux kernel >>> complained

Re: [PATCH 3/3] board: Add new Broadcom Northstar board

2023-02-10 Thread Tom Rini
On Thu, Feb 02, 2023 at 12:37:45AM +0100, Linus Walleij wrote: > This adds a simple Northstar "BRCMNS" board to be used with > the BCM4709x and BCM5301x chips. > > The main intention is to use this with the D-Link DIR-890L > and DIR-885L routers for loading the kernel into RAM from > NAND memory

[PATCH] rockchip: dts: rk3328: fix sdram params

2023-02-10 Thread Jonas Karlman
The rk3328 sdram driver read sdram parameters from the devicetree into a struct rk3328_sdram_params using dev_read_u32_array. After commit 5ab30c3176bf ("ram: rockchip: Update ddr pctl regs for px30") changed the size of struct ddr_pctl_regs, a member of struct rk3328_sdram_params, U-Boot TPL can

Re: [PATCH v3 3/5] test: create dedicated fdt node for ofnode_for_each_prop test

2023-02-10 Thread Tom Rini
On Sun, Jan 22, 2023 at 06:21:23PM +0300, Dzmitry Sankouski wrote: > Property count may change in /buttons node, if more button tests added, > and this will break ofnode_for_each_prop. > Add separate node for mentioned test. > > Signed-off-by: Dzmitry Sankouski > Reviewed-by: Simon Glass

Re: [PATCH v3 5/5] dm: input: add button_kbd driver

2023-02-10 Thread Tom Rini
On Sun, Jan 22, 2023 at 06:21:25PM +0300, Dzmitry Sankouski wrote: > Bootmenu requires an input device with arrows and enter key. > A common smartphone luckily has power, volume up/down buttons, > which may be used for controlling bootmenu. > To use driver, add 'button-kbd' to stdin. > >

Re: [PATCH v2 01/14] qcs404: sysmap: Don't map reserved memory ranges

2023-02-10 Thread Tom Rini
On Wed, Feb 01, 2023 at 07:28:48PM +0530, Sumit Garg wrote: > Currently u-boot maps whole of 1G RAM but there reserved memory ranges on > QCS404 which are reserved for TrustZone, various firmware components etc. > Any access to these reserved memory ranges causes a bus hang issue. So > disable

Re: [PATCH v3 4/5] dm: button: add support for linux_code in button-gpio.c driver

2023-02-10 Thread Tom Rini
On Sun, Jan 22, 2023 at 06:21:24PM +0300, Dzmitry Sankouski wrote: > Linux event code must be used in input devices, using buttons. > > Signed-off-by: Dzmitry Sankouski > Reviewed-by: Simon Glass Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature

Re: [PATCH v2 02/14] qcs404-evb: Enable msm_gpio driver support

2023-02-10 Thread Tom Rini
On Wed, Feb 01, 2023 at 07:28:49PM +0530, Sumit Garg wrote: > Signed-off-by: Sumit Garg Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature

Re: rk3328 broken between v2022.04 and v2022.07 - still not working with v2023.01

2023-02-10 Thread Janpieter Sollie
Op 10/02/2023 om 17:02 schreef Jonas Karlman: Hi, On 2023-02-10 16:32, Janpieter Sollie wrote: Op 10/02/2023 om 13:41 schreef Peter Robinson: On Fri, Feb 10, 2023 at 12:31 PM Janpieter Sollie wrote: Hello, After a few years, I tried updating my u-boot 2021 to a more recent version, as

Re: i.MX8M binman

2023-02-10 Thread Simon Glass
Hi Peng, On Fri, 10 Feb 2023 at 04:55, Peng Fan wrote: > > +Marek, > > I heard that from Marek on IRC, but Marek ask me to reach you. > > Actually I am not sure what is the issue with i.MX8M binman > node. Was this to do with getting the security stuff into binman properly? I do recall trying

[PATCH v4 08/11] CI: Add a check for pre-schema driver model tags

2023-02-10 Thread Simon Glass
These should not be used anymore. Add a check to ensure they don't creek back into U-Boot. Use bootph-... instead. Signed-off-by: Simon Glass --- (no changes since v3) Changes in v3: - Add a new patch to check for pre-schema driver model tags in CI .azure-pipelines.yml | 12

Re: [PATCH 3/3] mmc: mv_sdhci: Depend on DM_MMC

2023-02-10 Thread Simon Glass
On Fri, 10 Feb 2023 at 05:24, Stefan Roese wrote: > > All build targets using this driver already use DM_MMC. So let's depend > this driver on this Kconfig symbol and remove the non-DM driver part. > > Signed-off-by: Stefan Roese > Cc: Tom Rini > Cc: Simon Glass > Cc: Peng Fan > Cc: Jaehoon

Re: [PATCH v2 3/3] fdt: introduce fsapply command

2023-02-10 Thread Simon Glass
Hi, On Fri, 10 Feb 2023 at 04:32, Lothar Waßmann wrote: > > Hi, > > On Fri, 10 Feb 2023 11:02:13 + Andre Przywara wrote: > > Explicitly specifying the exact filenames of devicetree overlays files > > on a U-Boot command line can be quite tedious for users, especially > > when it should be

Re: U-boot not loading NVMe driver on Pi CM4

2023-02-10 Thread Simon Glass
Hi Ignatius, On Thu, 9 Feb 2023 at 17:29, Ignatius Rivaldi wrote: > > Added more debug logging and for some reason the nvme driver tries to bind > to pcie root bridge instead of the SSD > drivers/pinctrl/pinctrl-uclass.c:300-pinctrl_select_state_simple() > pcie_brcm pcie@7d50:

Re: [PATCH v3 2/5] dts: add missing linux,code in gpio-keys

2023-02-10 Thread Tom Rini
On Sun, Jan 22, 2023 at 06:21:22PM +0300, Dzmitry Sankouski wrote: > gpio-keys linux driver enforces user to specify linux,code. > Add missing linux,code before implementing button input support. > > - arch/arm/dts/rk3288-popmetal.dtsi -> KEY_POWER > - arch/arm/dts/rk3288-tinker.dtsi ->

Re: [PATCH] arm: bcm7xxx: Convert to DM_SERIAL

2023-02-10 Thread Tom Rini
On Sat, Feb 04, 2023 at 08:36:57PM -0500, Thomas Fitzsimmons wrote: > Remove ns16550 configuration from header files. Document > DM_SERIAL-required prior stage device tree configuration. As a follow-up, can you please migrate README.bcm7xxx to doc/board/broadcom/bcm7xxx.rst, or whatever you

Re: [PATCH v3 1/5] gpio: qcom: add direction functions for pwrkey

2023-02-10 Thread Tom Rini
On Sun, Jan 22, 2023 at 06:21:21PM +0300, Dzmitry Sankouski wrote: > GPIO button driver requires direction functions to probe > button gpio. Those functions are blank, since pwrkey is not > really gpio, and don't support direction settings. > > Signed-off-by: Dzmitry Sankouski > Reviewed-by:

Re: [PATCH v2 076/169] Correct SPL use of EFI_SECURE_BOOT

2023-02-10 Thread Tom Rini
On Sun, Feb 05, 2023 at 03:39:45PM -0700, Simon Glass wrote: > This converts 1 usage of this option to the non-SPL form, since there is > no SPL_EFI_SECURE_BOOT defined in Kconfig > > Signed-off-by: Simon Glass As I missed this, somehow, in the main branch, applied to u-boot/master, thanks!

Re: [PATCH v2 13/14] i2c: Add support for Qualcomm I2C driver

2023-02-10 Thread Tom Rini
On Wed, Feb 01, 2023 at 07:29:00PM +0530, Sumit Garg wrote: > Add support for Qualcomm I2C QUP driver which is inspired from > corresponding driver in Linux: drivers/i2c/busses/i2c-qup.c. > > Currently this driver only support FIFO polling mode which is sufficient > to support devices like

Re: [PATCH 1/2] env: Complete generic support for writable list

2023-02-10 Thread Tom Rini
On Fri, Feb 03, 2023 at 01:22:51PM +0100, Jan Kiszka wrote: > From: Jan Kiszka > > This completes what 890feecaab72 started by selecting ENV_APPEND and > loading the default env before any other sources. This ensures that load > operations pick up all non-writable vars from the default env and

Re: [PATCH 2/2] env: Couple networking-related variable flags to CONFIG_NET

2023-02-10 Thread Tom Rini
On Fri, Feb 03, 2023 at 01:22:52PM +0100, Jan Kiszka wrote: > From: Jan Kiszka > > Boards may set networking variables programmatically, thus may have > CONFIG_NET on but CONFIG_CMD_NET off. The IOT2050 is an example. > > CC: Joe Hershberger > Signed-off-by: Jan Kiszka > Reviewed-by: Tom

Re: [PATCH v2 14/14] dts: qcs404-evb: Add I2C controller nodes

2023-02-10 Thread Tom Rini
On Wed, Feb 01, 2023 at 07:29:01PM +0530, Sumit Garg wrote: > Signed-off-by: Sumit Garg Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature

Re: [PATCH v3 2/3] net: ftmac100: simplify priv->iobase casting

2023-02-10 Thread Tom Rini
On Fri, Feb 03, 2023 at 10:09:03PM +0300, Sergei Antonov wrote: > Replace 'phys_addr_t iobase' with 'struct ftmac100 *ftmac100' in > struct ftmac100_data. It allows to remove casting in a number of places. > > Since priv->iobase is phys_addr_t, use phys_to_virt() to make > a pointer from it. >

[PATCH v4 01/11] schemas: Add schema for U-Boot driver model 'phase tags'

2023-02-10 Thread Simon Glass
U-Boot has some particular challenges with device tree and devices: - U-Boot has multiple build phases, such as a Secondary Program Loader (SPL) phase which typically runs in a pre-SDRAM environment where code and data space are limited. In particular, there may not be enough space for the

[PATCH v4 10/11] dm: Add support for handling old u-boot,dm- tags

2023-02-10 Thread Simon Glass
Add a CONFIG option to deal with this automatically, printing a warning when U-Boot starts up. This can be useful if the device tree comes from another project. We will maintain this through the 2023.07 release, providing 6 months for people to notice. Signed-off-by: Simon Glass --- Changes in

Re: [PATCH v1 1/1] tools: env: use /run to store lockfile

2023-02-10 Thread Tom Rini
On Fri, Feb 10, 2023 at 05:05:39PM +0800, Tim Lee wrote: > According this issue https://github.com/ppp-project/ppp/issues/339. > Eventually, the dt-utils changed lock directory to fix missing /var/lock > directory error then make dt-utils can run normally. > > We also have a similar issue with

Re: imx8mm USB Mass Storage fails through a HUB

2023-02-10 Thread Fabio Estevam
Hi Tim, On Fri, Feb 10, 2023 at 10:26 AM Marcel Ziswiler wrote: > The second USB port has a on-carrier USB hub both on Dahlia as well as the > Verdin Development board. > > Sorry, that I do not have any good suggestion other than maybe comparing with > our configuration/design. The verdin

Re: [PATCH] arm: kirkwood: Enable uart0 dm-pre-reloc for Kirkwood boards

2023-02-10 Thread Pali Rohár
On Thursday 09 February 2023 17:37:54 Tony Dinh wrote: > Hi Pali, > > On Thu, Feb 9, 2023 at 1:45 PM Pali Rohár wrote: > > > > On Thursday 09 February 2023 13:33:23 Tony Dinh wrote: > > > Hi Stefan, > > > > > > On Wed, Feb 8, 2023 at 11:15 PM Stefan Roese wrote: > > > > > > > > Hi Tony, > > > >

Re: [PATCH v2 000/169] Kconfig: More cleanup of CONFIG options

2023-02-10 Thread Tom Rini
On Sun, Feb 05, 2023 at 03:35:46PM -0700, Simon Glass wrote: > Quite a few CONFIG options without an SPL equivalent are used as if they > have one. For example, we see CONFIG_IS_ENABLED(IOMMU) when there is no > SPL_IOMMU Kconfig option. > > This series adds new features to moveconfig to detect

Re: imx8mm USB Mass Storage fails through a HUB

2023-02-10 Thread Tim Harvey
On Fri, Feb 10, 2023 at 5:26 AM Marcel Ziswiler wrote: > > Hi Tim > > On Thu, 2023-02-09 at 11:58 -0800, Tim Harvey wrote: > > Greetings, > > > > I'm finding that USB Mass Storage devices appear to fail detection on > > imx8mm when hanging off a USB hub whereas other devices such as USB > >

Re: imx8mm USB Mass Storage fails through a HUB

2023-02-10 Thread Marek Vasut
On 2/9/23 20:58, Tim Harvey wrote: Greetings, Hi, I'm finding that USB Mass Storage devices appear to fail detection on imx8mm when hanging off a USB hub whereas other devices such as USB ethernet work fine. u-boot=> usb start && usb tree starting USB... Bus usb@32e4: Bus usb@32e5:

Re: rk3328 broken between v2022.04 and v2022.07 - still not working with v2023.01

2023-02-10 Thread Janpieter Sollie
Op 10/02/2023 om 17:02 schreef Jonas Karlman: Hi, On 2023-02-10 16:32, Janpieter Sollie wrote: Op 10/02/2023 om 13:41 schreef Peter Robinson: On Fri, Feb 10, 2023 at 12:31 PM Janpieter Sollie wrote: Hello, After a few years, I tried updating my u-boot 2021 to a more recent version, as

Re: [PATCH v3 3/3] net: ftmac100: add mii read and write callbacks

2023-02-10 Thread Tom Rini
On Fri, Feb 03, 2023 at 10:09:04PM +0300, Sergei Antonov wrote: > Register mii_bus with read and write callbacks to allow the 'mii' > command to work. Use a timeout of 10 ms to wait for the R/W > operations to complete. > > Signed-off-by: Sergei Antonov > Reviewed-by: Rick Chen > Tested-by:

Re: [PATCH v3 1/3] net: ftmac100: change driver name from nds32_mac to ftmac100

2023-02-10 Thread Tom Rini
On Fri, Feb 03, 2023 at 10:09:02PM +0300, Sergei Antonov wrote: > So it will be named similarly to the related ftgmac100 driver. > The old name 'nds32_mac' is not referred to anywhere in U-Boot. > > Signed-off-by: Sergei Antonov > Reviewed-by: Ramon Fried > Reviewed-by: Rick Chen Applied to

Re: [PATCH v3 08/10] phy: socionext: Add UniPhier USB3 PHY driver

2023-02-10 Thread Marek Vasut
On 2/8/23 10:15, Kunihiko Hayashi wrote: [...] +static int uniphier_usb3phy_init(struct phy *phy) +{ + struct uniphier_usb3phy_priv *priv = dev_get_priv(phy->dev); + int ret; + + ret = clk_enable_bulk(>clks); + if (ret) + goto out_clk; This should be

Re: [PATCH v2 09/14] dts: qcs404-evb: Add ethernet controller node

2023-02-10 Thread Tom Rini
On Wed, Feb 01, 2023 at 07:28:56PM +0530, Sumit Garg wrote: > Signed-off-by: Sumit Garg Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature

Pull request for efi-2023-04-rc2

2023-02-10 Thread Heinrich Schuchardt
Dear Tom, The following changes since commit 81e8a51cee2b265e26272f0c67518c4844baa36c: Merge branch '2023-02-09-minor-updates' (2023-02-09 16:31:36 -0500) are available in the Git repository at: https://source.denx.de/u-boot/custodians/u-boot-efi.git tags/efi-2023-04-rc2 for you to fetch

Re: [PATCH v2 10/14] clock-snapdragon: Add clk_rcg_set_rate() with mnd_width=0

2023-02-10 Thread Tom Rini
On Wed, Feb 01, 2023 at 07:28:57PM +0530, Sumit Garg wrote: > Add clk_rcg_set_rate() which allows to configure clocks without programming > MND values. This is required for configuring I2C clocks on QCS404. > > Co-developed-by: Mike Worsfold > Signed-off-by: Mike Worsfold > Signed-off-by:

Re: [PATCH v2 12/14] pinctrl: qcs404: Enable I2C pinmux options

2023-02-10 Thread Tom Rini
On Wed, Feb 01, 2023 at 07:28:59PM +0530, Sumit Garg wrote: > Co-developed-by: Mike Worsfold > Signed-off-by: Mike Worsfold > Signed-off-by: Sumit Garg Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature

Re: [PATCH v2 07/14] net: dwc_eth_qos: Allow platform to override tx/rx_fifo_sz

2023-02-10 Thread Tom Rini
On Wed, Feb 01, 2023 at 07:28:54PM +0530, Sumit Garg wrote: > The GMAC controller on QCS404 SoC (support added by upcoming patch) fails > to work with maximum tx/rx_fifo_sz supported by the hardware (16K). So > allow platforms to override FIFO size using corresponding DT node > properties. > >

Re: [PATCH v2 08/14] net: dwc_eth_qos: Add Qcom ethernet driver glue layer

2023-02-10 Thread Tom Rini
On Wed, Feb 01, 2023 at 07:28:55PM +0530, Sumit Garg wrote: > The Qualcom ETHQOS hardware supports an RGMII macro which needs to be > configured according to following link speeds: > - SPEED_1000 > - SPEED_100 > - SPEED_10 > > So add a corresponding glue driver to configure RGMII macro. > >

[PATCH v4 00/11] dm: Move to new driver model schema for device tree tags

2023-02-10 Thread Simon Glass
Now that a new schema has been accepted upstream, press it into service in U-Boot. Changes in v4: - Add new patch with a way to set the environment for a pytest - Add a way to support handling old u-boot,dm- tags - Add new patch with a test for the various migration combinations Changes in v3: -

[PATCH v4 11/11] dm: test: Add a test for the various migration combinations

2023-02-10 Thread Simon Glass
Test that: - sandbox shows a warning when an unmigrated DT is used - sandbox fails to run when migration is turned off - sandbox_spl fails to build when migration is turned off Signed-off-by: Simon Glass --- Changes in v4: - Add new patch with a test for the various migration combinations

  1   2   >