Re: [PATCH v1 11/25] ARM: dts: stm32: Don't probe red led at boot for stm32mp157a-dk1-scmi-u-boot

2024-04-17 Thread Patrick DELAUNAY
Hi, On 4/9/24 17:02, Patrice Chotard wrote: red led and button dedicated to fastboot share the same gpio GPIOA13. Led driver is probed early so the corresponding gpio is taken and configured in output which forbid fastboot and stm32prog button usage. To avoid this, remove the "default-state"

Re: [PATCH v1 10/25] ARM: dts: stm32: Add gpio-keys for stm32mp157a-dk1-scmi-u-boot

2024-04-17 Thread Patrick DELAUNAY
Hi, On 4/9/24 17:02, Patrice Chotard wrote: Instead of using "st,fastboot-gpios" and "st,stm32prog-gpios", declare 2 gpio-keys. Signed-off-by: Patrice Chotard --- arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi | 19 +-- 1 file changed, 17 insertions(+), 2 deletions(-) diff

Re: [PATCH v1 12/25] ARM: dts: stm32: Update red led node for stm32mp157a-dk1-scmi-u-boot

2024-04-17 Thread Patrick DELAUNAY
Hi, On 4/9/24 17:02, Patrice Chotard wrote: As indicated in kernel led dt-bindings, label is a deprecated property, so remove it and use red led node's name instead for u-boot,error-led property. Rename "red" led node's name to "led-red". Remove status property which is useless. Signed-off-by:

Re: [PATCH v2] tpm: display warning if using gpio reset with TPM

2024-04-17 Thread Ilias Apalodimas
On Wed, 17 Apr 2024 at 09:48, Miquel Raynal wrote: > > Hi Ilias, > > ilias.apalodi...@linaro.org wrote on Wed, 17 Apr 2024 08:40:14 +0300: > > > Hi Miquel > > > > On Mon, 8 Apr 2024 at 10:23, Miquel Raynal > > wrote: > > > > > > Hello, > > > > > > ilias.apalodi...@linaro.org wrote on Thu, 28

Re: [PATCH 1/2] mmc: stm32_sdmmc2: Add "st,stm32mp25-sdmmc2" compatible

2024-04-17 Thread Patrick DELAUNAY
Hi, On 3/8/24 15:26, Patrice Chotard wrote: From: Patrick Delaunay Add compatible used for STM32MP25 family. Signed-off-by: Patrick Delaunay Signed-off-by: Patrice Chotard --- drivers/mmc/stm32_sdmmc2.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/mmc/stm32_sdmmc2.c

Re: [PATCH v1 22/25] ARM: dts: stm32: Add gpio-keys for stm32mp157c-ed1-scmi-u-boot

2024-04-17 Thread Patrick DELAUNAY
Hi, On 4/9/24 17:02, Patrice Chotard wrote: Add 2 gpio-keys : _ button-user-1 for stm32prog mode activation. _ button-user-2 for fastboot mode activation. Remove proprietary st,fastboot-gpios and st,stm32prog-gpios. Signed-off-by: Patrice Chotard ---

[PATCH 1/2] doc: sort env sub-commands alphabetically

2024-04-17 Thread Heinrich Schuchardt
The 'env' man-page is currently only partially sorted. Signed-off-by: Heinrich Schuchardt --- doc/usage/cmd/env.rst | 34 +- 1 file changed, 17 insertions(+), 17 deletions(-) diff --git a/doc/usage/cmd/env.rst b/doc/usage/cmd/env.rst index

Re: [PATCH v1 13/25] ARM: dts: stm32: Add led-blue for stm32mp157a-dk1-scmi-u-boot

2024-04-17 Thread Patrick DELAUNAY
Hi, On 4/9/24 17:02, Patrice Chotard wrote: As indicated in kernel led dt-bindings, label is a deprecated property, so remove it and use blue led node's name instead for u-boot,boot-led property. Signed-off-by: Patrice Chotard --- arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi | 6 +-

RE: [PATCH v1] arm: dts: verdin-imx8mm/imx8mp: use gpio-hog for sleep moci

2024-04-17 Thread Peng Fan
> Subject: [PATCH v1] arm: dts: verdin-imx8mm/imx8mp: use gpio-hog for > sleep moci > > From: Stefan Eichenberger > > In Linux, we allow sleep moci to be turned off when the carrier board > supports it and the system is in suspend. In U-Boot, however, we want the > sleep moci to be always on. So

Re: [PATCH v1 15/25] ARM: dts: stm32: Don't probe red led at boot for stm32mp157a-dk1-u-boot

2024-04-17 Thread Patrick DELAUNAY
Hi, On 4/9/24 17:02, Patrice Chotard wrote: red led and button dedicated to fastboot share the same gpio GPIOA13. Led driver is probed early so the corresponding gpio is taken and configured in output which forbid fastboot and stm32prog button usage. To avoid this, remove the "default-state"

Re: [PATCH v1 14/25] ARM: dts: stm32: Add gpio-keys for stm32mp157a-dk1-u-boot

2024-04-17 Thread Patrick DELAUNAY
Hi, On 4/9/24 17:02, Patrice Chotard wrote: Instead of using "st,fastboot-gpios" and "st,stm32prog-gpios", declare 2 gpio-keys. Signed-off-by: Patrice Chotard --- arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi | 19 +-- 1 file changed, 17 insertions(+), 2 deletions(-)

Re: [PATCH] rockchip: px30-board-tpl: Use CONFIG_TPL_BANNER_PRINT flag

2024-04-17 Thread Łukasz Czechowski
Hi Quentin, Thanks for the hints. Matching the parent ifdef with non-PX30 TPL implementation is indeed a good idea. I'll send the updated version of the patch. Best regards, Lukasz wt., 16 kwi 2024 o 15:22 Quentin Schulz napisał(a): > > Hi Lukasz, > > Please use scripts/get_maintainer.pl to

[PATCH v2] rockchip: px30-board-tpl: Use CONFIG_TPL_BANNER_PRINT flag

2024-04-17 Thread lukasz . czechowski
From: Lukasz Czechowski Display TPL init information message only when TPL_BANNER_PRINT configuration entry is set. This allows to disable information message in case logs on UART are unwanted. Update parent ifdef condition to check also CONFIG_TPL_SERIAL to match logic of the non-PX30 TPL

Re: [PATCH v2] tpm: display warning if using gpio reset with TPM

2024-04-17 Thread Miquel Raynal
Hi Ilias, ilias.apalodi...@linaro.org wrote on Wed, 17 Apr 2024 08:40:14 +0300: > Hi Miquel > > On Mon, 8 Apr 2024 at 10:23, Miquel Raynal wrote: > > > > Hello, > > > > ilias.apalodi...@linaro.org wrote on Thu, 28 Mar 2024 09:08:37 +0200: > > > > > Thanks Tim > > > > > > On Thu, 28 Mar 2024

Re: [PATCH] stm32mp: Reserve OPTEE area in EFI memory map

2024-04-17 Thread Heinrich Schuchardt
On 17.04.24 09:25, Patrick DELAUNAY wrote: Hi, On 3/8/24 11:12, Patrice Chotard wrote: Since commit 7b78d6438a2b3 ("efi_loader: Reserve unaccessible memory") memory region above ram_top is tagged in EFI memory map as EFI_BOOT_SERVICES_DATA. In case of STM32MP1 platform, above ram_top, there is

Re: [PATCH 01/10] board: ti: am62x: Init DRAM size in R5/A53 SPL

2024-04-17 Thread Chintan Vankar
On 16/04/24 22:30, Tom Rini wrote: On Tue, Apr 16, 2024 at 05:52:58PM +0530, Chintan Vankar wrote: On 12/04/24 03:37, Tom Rini wrote: On Wed, Apr 03, 2024 at 06:18:01PM +0530, Chintan Vankar wrote: On 22/01/24 10:11, Siddharth Vadapalli wrote: On 20/01/24 22:11, Tom Rini wrote: On

Re: [PATCH v1 21/25] ARM: dts: stm32: Add led-blue for stm32mp157c-ed1-u-boot

2024-04-17 Thread Patrick DELAUNAY
Hi, On 4/9/24 17:02, Patrice Chotard wrote: The blue led is used to indicate U-Boot entering / exit indication then Linux heartbeat. Signed-off-by: Patrice Chotard --- arch/arm/dts/stm32mp157c-ed1-u-boot.dtsi | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) Reviewed-by:

Re: [PATCH v1 19/25] ARM: dts: stm32: Don't probe red led at boot for stm32mp157c-ed1-u-boot

2024-04-17 Thread Patrick DELAUNAY
Hi, On 4/9/24 17:02, Patrice Chotard wrote: red led and button dedicated to fastboot share the same gpio GPIOA13. Led driver is probed early so the corresponding gpio is taken and configured in output which forbid fastboot and stm32prog button usage. To avoid this, remove the "default-state"

Re: [PATCH v1 20/25] ARM: dts: stm32: Update red led node for stm32mp157c-ed1-u-boot

2024-04-17 Thread Patrick DELAUNAY
Hi, On 4/9/24 17:02, Patrice Chotard wrote: As indicated in kernel led dt-bindings, label is a deprecated property, so remove it and use led node's name instead for u-boot,error-led property. Rename red led node's name to led-red. Remove status property which is useless. Add compatible =

Re: [PATCH v1 06/25] board: st: stmp32mp1: Use BUTTON UCLASS in board_key_check()

2024-04-17 Thread Patrick DELAUNAY
Hi, On 4/9/24 17:01, Patrice Chotard wrote: Instead of using gpio directly to detect key pressed on button dedicated for fastboot and stm32mprog, make usage of BUTTON UCLASS. Signed-off-by: Patrice Chotard --- board/st/stm32mp1/stm32mp1.c | 68 +--- 1 file

Re: [PATCH v1 05/25] configs: stm32mp1: Enable BUTTON_GPIO flag for stm32mp13_defconfig

2024-04-17 Thread Patrick DELAUNAY
Hi, On 4/9/24 17:01, Patrice Chotard wrote: Enable BUTTON_GPIO flag for STM32MP15. Signed-off-by: Patrice Chotard --- configs/stm32mp13_defconfig | 2 ++ 1 file changed, 2 insertions(+) diff --git a/configs/stm32mp13_defconfig b/configs/stm32mp13_defconfig index db09e63100e..caaabf39ef3

Re: [PATCH v1 16/25] ARM: dts: stm32: Update red led node for stm32mp157a-dk1-u-boot

2024-04-17 Thread Patrick DELAUNAY
Hi, On 4/9/24 17:02, Patrice Chotard wrote: As indicated in kernel led dt-bindings, label is a deprecated property, so remove it and use red led node's name instead for u-boot,error-led property. Rename red led node's name to led-red. Remove status property which is useless. Signed-off-by:

Re: [PATCH v1 17/25] ARM: dts: stm32: Update u-boot,boot-led for stm32mp157a-dk1-u-boot

2024-04-17 Thread Patrick DELAUNAY
Hi, On 4/9/24 17:02, Patrice Chotard wrote: As indicated in kernel led dt-bindings, label is a deprecated property, so remove it and use blue led node's name instead for u-boot,boot-led property. Signed-off-by: Patrice Chotard --- arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi | 6 +- 1

Re: [PATCH v1 18/25] ARM: dts: stm32: Add gpio-keys for stm32mp157c-ed1-u-boot

2024-04-17 Thread Patrick DELAUNAY
Hi, On 4/9/24 17:02, Patrice Chotard wrote: Add 2 gpio-keys : _ button-user-1 for stm32prog mode activation. _ button-user-2 for fastboot mode activation. Remove proprietary st,fastboot-gpios and st,stm32prog-gpios. Signed-off-by: Patrice Chotard ---

Re: [PATCH 2/3] ARM: dts: stm32: Fix partition node name for stm32mp15xx-dhcor-u-boot

2024-04-17 Thread Patrick DELAUNAY
Hi On 3/8/24 14:50, Patrice Chotard wrote: Fix flash@0 partition node name with correct offset. Fixes: 90f992e6a58c ("arm: dts: stm32: Add partitions in flash0 and nand node for stm32mp15xx-dhcom/dhcor") Signed-off-by: Patrice Chotard --- arch/arm/dts/stm32mp15xx-dhcor-u-boot.dtsi | 6

Re: [PATCH 1/3] ARM: dts: stm32: Fix partition node name for stm32mp157c-ev1-u-boot

2024-04-17 Thread Patrick DELAUNAY
Hi, On 3/8/24 14:50, Patrice Chotard wrote: Fix flash@0 and nand@0 partition node name with correct offset. Fixes: e91d3c61767b ("arm: dts: stm32: Add partitions in flash0 and nand node for stm32mp15xx-ev1") Signed-off-by: Patrice Chotard --- arch/arm/dts/stm32mp157c-ev1-u-boot.dtsi | 8

Re: [PATCH v1 08/25] ARM: dts: stm32: Don't probe led-red/led-blue at boot for stm32mp135f-dk-u-boot

2024-04-17 Thread Patrick DELAUNAY
Hi, On 4/9/24 17:01, Patrice Chotard wrote: led-red and button dedicated to fastboot share the same gpio GPIOA13. led-blue and button dedicated to stm32prog share the same gpio GPIOA14. Led driver is probed early so the corresponding gpio is taken and configured in output which forbid fastboot

Re: [PATCH v1 07/25] ARM: dts: stm32: Add gpio-keys for stm32mp135f-dk-u-boot

2024-04-17 Thread Patrick DELAUNAY
Hi On 4/9/24 17:01, Patrice Chotard wrote: Add 2 gpio-keys : _ button-user-1 for stm32prog mode activation. _ update button-user's label (defined in kernel DT) to match label requested in board_key_check() for fastboot mode activation. Signed-off-by: Patrice Chotard ---

Re: [PATCH v1 09/25] ARM: dts: stm32: Clean led-red node for stm32mp135f-dk-u-boot

2024-04-17 Thread Patrick DELAUNAY
Hi, On 4/9/24 17:01, Patrice Chotard wrote: Remove "color" property from led-red node which is not supported by U-Boot. Signed-off-by: Patrice Chotard --- arch/arm/dts/stm32mp135f-dk-u-boot.dtsi | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm/dts/stm32mp135f-dk-u-boot.dtsi

Re: [PATCH] stm32mp: Reserve OPTEE area in EFI memory map

2024-04-17 Thread Patrick DELAUNAY
Hi, On 3/8/24 11:12, Patrice Chotard wrote: Since commit 7b78d6438a2b3 ("efi_loader: Reserve unaccessible memory") memory region above ram_top is tagged in EFI memory map as EFI_BOOT_SERVICES_DATA. In case of STM32MP1 platform, above ram_top, there is one reserved-memory region tagged "no-map"

[PATCH 0/2] doc: env sub-commands, env list of aliases

2024-04-17 Thread Heinrich Schuchardt
* sort env sub-commands alphabetically * update list of aliases for the env command Heinrich Schuchardt (2): doc: sort env sub-commands alphabetically doc: update list of aliases for the env command doc/usage/cmd/env.rst | 35 ++- 1 file changed, 18

[PATCH 2/2] doc: update list of aliases for the env command

2024-04-17 Thread Heinrich Schuchardt
* add link to askenv man-page * add printenv Signed-off-by: Heinrich Schuchardt --- doc/usage/cmd/env.rst | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/doc/usage/cmd/env.rst b/doc/usage/cmd/env.rst index 60a294328e8..040076bcc03 100644 --- a/doc/usage/cmd/env.rst +++

Re: [PATCH 4/4] configs: visionfive2: enable SPL_YMODEM_SUPPORT

2024-04-17 Thread Heinrich Schuchardt
On 17.04.24 03:41, E Shattow wrote: Successful in use w/ 'tio' serial tool and Adafruit CP2102N Friend adapter on Mars CM Lite board in DFRobot mini router carrier. While SPL and u-boot.itb payload are sourced via UART the environment variables are from the environment variable storage as-is.

Re: [PATCH 2/2] mmc: stm32_sdmmc2: Fix AARCH64 compilation warnings

2024-04-17 Thread Patrick DELAUNAY
Hi, On 3/8/24 15:26, Patrice Chotard wrote: When building with AARCH64 defconfig, we got warnings, fix them. Signed-off-by: Patrice Chotard --- drivers/mmc/stm32_sdmmc2.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/mmc/stm32_sdmmc2.c

Re: [PATCH v1 01/25] configs: stm32mp13: Enable FASTBOOT

2024-04-17 Thread Patrick DELAUNAY
Hi, On 4/9/24 17:01, Patrice Chotard wrote: Enable FASTBOOT relative flags for stm32mp13_defconfig. Signed-off-by: Patrice Chotard --- configs/stm32mp13_defconfig | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) Reviewed-by: Patrick Delaunay Thanks Patrick

Re: [PATCH v1 23/25] ARM: dts: stm32: Don't probe red led at boot for stm32mp157c-ed1-scmi-u-boot

2024-04-17 Thread Patrick DELAUNAY
Hi, On 4/9/24 17:02, Patrice Chotard wrote: red led and button dedicated to fastboot share the same gpio GPIOA13. Led driver is probed early so the corresponding gpio is taken and configured in output which forbid fastboot and stm32prog button usage. To avoid this, remove the "default-state"

Re: [PATCH v1 24/25] ARM: dts: stm32: Update red led node for stm32mp157c-ed1-scmi-u-boot

2024-04-17 Thread Patrick DELAUNAY
Hi, On 4/9/24 17:02, Patrice Chotard wrote: As indicated in kernel led dt-bindings, label is a deprecated property, so remove it and use led node's name instead for u-boot,error-led property. Rename red led node's name to led-red. Remove status property which is useless. Add compatible =

[Patch] dwc_eth_qos: Revert regression handling fixed phy

2024-04-17 Thread Nicole Battenfeld
Subject: [PATCH] dwc_eth_qos: Revert regression handling fixed phy In imx8mp operation on eqos with fixed phy I get without that patch: ERROR: no/invalid phy = fixed_phy_create(dev_ofnode(dev)); + eqos->phy_of_node = fixed_node; + }

Re: [PATCH 3/3] ARM: dts: stm32: Fix partition node name for stm32mp15xx-dhcom-u-boot

2024-04-17 Thread Patrick DELAUNAY
Hi, On 3/8/24 14:50, Patrice Chotard wrote: Fix flash@0 partition node name with correct offset. Fixes: 90f992e6a58c ("arm: dts: stm32: Add partitions in flash0 and nand node for stm32mp15xx-dhcom/dhcor") Signed-off-by: Patrice Chotard --- arch/arm/dts/stm32mp15xx-dhcom-u-boot.dtsi | 6

Re: [PATCH v1 02/25] configs: stm32mp1: Enable BUTTON_GPIO flag for stm32mp15_defconfig

2024-04-17 Thread Patrick DELAUNAY
Hi, On 4/9/24 17:01, Patrice Chotard wrote: Enable BUTTON_GPIO flag for STM32MP15. Signed-off-by: Patrice Chotard --- configs/stm32mp15_defconfig | 2 ++ 1 file changed, 2 insertions(+) diff --git a/configs/stm32mp15_defconfig b/configs/stm32mp15_defconfig index 3302b306996..ffe7512650e

Re: [PATCH v1 04/25] configs: stm32mp1: Enable BUTTON_GPIO flag for stm32mp15_trusted_defconfig

2024-04-17 Thread Patrick DELAUNAY
Hi, On 4/9/24 17:01, Patrice Chotard wrote: Enable BUTTON_GPIO flag for STM32MP15. Signed-off-by: Patrice Chotard --- configs/stm32mp15_trusted_defconfig | 2 ++ 1 file changed, 2 insertions(+) diff --git a/configs/stm32mp15_trusted_defconfig b/configs/stm32mp15_trusted_defconfig index

Re: [PATCH v1 03/25] configs: stm32mp1: Enable BUTTON_GPIO flag for stm32mp15_basic_defconfig

2024-04-17 Thread Patrick DELAUNAY
Hi, On 4/9/24 17:01, Patrice Chotard wrote: Enable BUTTON_GPIO flag for STM32MP15. Signed-off-by: Patrice Chotard --- configs/stm32mp15_basic_defconfig | 2 ++ 1 file changed, 2 insertions(+) diff --git a/configs/stm32mp15_basic_defconfig b/configs/stm32mp15_basic_defconfig index

Re: [PATCH] board: ti: j721e: Add support for both esm probe

2024-04-17 Thread J, KEERTHY
On 4/17/2024 10:06 AM, Udit Kumar wrote: At present only MCU domain ESM is probed, this means errors occurred in mcu domain will be propagate to MCU_SAFETY_ERRORn. MCU ESM accepts SOC_SAFETY_ERRORn signal as Error event and propagate to MCU_SAFETY_ERRORn.[0] Therefore adding support to probe

Re: [PATCH v1 25/25] ARM: dts: stm32: Add led-blue for stm32mp157c-ed1-scmi-u-boot

2024-04-17 Thread Patrick DELAUNAY
Hi, On 4/9/24 17:02, Patrice Chotard wrote: The blue led is used to indicate U-Boot entering / exit indication then Linux heartbeat. Signed-off-by: Patrice Chotard --- arch/arm/dts/stm32mp157c-ed1-scmi-u-boot.dtsi | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) Reviewed-by:

Re: [PATCH v2] rockchip: px30-board-tpl: Use CONFIG_TPL_BANNER_PRINT flag

2024-04-17 Thread Quentin Schulz
Hi Lukasz, I would have renamed the commit title to be something like rockchip: px30-board-tpl: sync ifdef guards with full TPL because CONFIG_TPL_BANNER_PRINT isn't really a flag but a Kconfig symbol and the effect of this patch is to not print the banner when the symbol is not selected.

[PATCH v3] rockchip: px30-board-tpl: Sync ifdef guards with full TPL

2024-04-17 Thread lukasz . czechowski
From: Lukasz Czechowski Display TPL init information message only when TPL_BANNER_PRINT configuration entry is set. This allows to disable information message in case logs on UART are unwanted. Update parent ifdef condition to check also CONFIG_TPL_SERIAL to match logic of the non-PX30 TPL

RE: [PATCH 3/5] mmc: am654_sdhci: Add itap_del_ena[] to store itapdlyena bit

2024-04-17 Thread Jaehoon Chung
Hi, > -Original Message- > From: Judith Mendez > Sent: Tuesday, April 16, 2024 6:28 AM > To: Peng Fan ; Jaehoon Chung ; Tom > Rini > Cc: Nitin Yadav ; Simon Glass ; > u-boot@lists.denx.de > Subject: [PATCH 3/5] mmc: am654_sdhci: Add itap_del_ena[] to store itapdlyena > bit > > Set

Re: [PATCH 01/10] board: ti: am62x: Init DRAM size in R5/A53 SPL

2024-04-17 Thread Sughosh Ganu
hi Chintan, On Wed, 17 Apr 2024 at 13:21, Chintan Vankar wrote: > > > > On 16/04/24 22:30, Tom Rini wrote: > > On Tue, Apr 16, 2024 at 05:52:58PM +0530, Chintan Vankar wrote: > >> > >> > >> On 12/04/24 03:37, Tom Rini wrote: > >>> On Wed, Apr 03, 2024 at 06:18:01PM +0530, Chintan Vankar wrote: >

Re: [PATCH v2 2/4] efi_loader: Add OS notifications for SetVariable at runtime

2024-04-17 Thread Heinrich Schuchardt
On 17.04.24 12:19, Ilias Apalodimas wrote: Previous patches enable SetVariable at runtime using a volatile storage backend using EFI_RUNTIME_SERVICES_DATA allocared memory. Since there's no recommendation from the spec on how to notify the OS, add a volatile EFI variable that contains the

Re: [PATCH v2 3/4] efi_loader: add an EFI variable with the file contents

2024-04-17 Thread Heinrich Schuchardt
On 17.04.24 12:19, Ilias Apalodimas wrote: Previous patches enabled SetVariableRT using a RAM backend. Although EBBR [0] defines a variable format we can teach userspace tools and write the altered variables, it's better if we skip the ABI requirements completely. So let's add a new variable,

[PATCH v2 3/4] efi_loader: add an EFI variable with the file contents

2024-04-17 Thread Ilias Apalodimas
Previous patches enabled SetVariableRT using a RAM backend. Although EBBR [0] defines a variable format we can teach userspace tools and write the altered variables, it's better if we skip the ABI requirements completely. So let's add a new variable, in its own namespace called "VarToFile" which

[PATCH v2 2/4] efi_loader: Add OS notifications for SetVariable at runtime

2024-04-17 Thread Ilias Apalodimas
Previous patches enable SetVariable at runtime using a volatile storage backend using EFI_RUNTIME_SERVICES_DATA allocared memory. Since there's no recommendation from the spec on how to notify the OS, add a volatile EFI variable that contains the filename relative to the ESP. OS'es can use that

[PATCH v2 4/4] efi_selftest: add tests for setvariableRT

2024-04-17 Thread Ilias Apalodimas
Since we support SetVariableRT now add the relevant tests - Search for the RTStorageVolatile and VarToFile variables after EBS - Try to update with invalid variales (BS, RT only) - Try to write a variable bigger than our backend storage - Write a variable that fits and check VarToFile has been

Re: [PATCH v2] rockchip: px30-board-tpl: Use CONFIG_TPL_BANNER_PRINT flag

2024-04-17 Thread Łukasz Czechowski
Hi Quentin, Thanks for your response. As for the CONFIG_TPL_SERIAL flag, I agree that it should use 'imply' to be allowed to disable it, to silence the serial in TPL. However, currently, disabling this flag makes the code unbuildable (which could be the reason 'select' is used). I'd keep this

RE: [PATCH 2/2] mmc: stm32_sdmmc2: Fix AARCH64 compilation warnings

2024-04-17 Thread Jaehoon Chung
> -Original Message- > From: Patrick DELAUNAY > Sent: Wednesday, April 17, 2024 6:02 PM > To: Patrice Chotard ; u-boot@lists.denx.de > Cc: U-Boot STM32 ; Jaehoon Chung > ; > Peng Fan ; Sean Anderson ; Simon Glass > ; Tom > Rini > Subject: Re: [PATCH 2/2] mmc: stm32_sdmmc2: Fix

[PATCH] ram: rockchip: px30: Replace misleading log message

2024-04-17 Thread lukasz . czechowski
From: Lukasz Czechowski Remove the log message "out" from sdram_init function which pollutes the console. It brings no meaningful information and might be unwanted in case silencing the console is required. Instead, add a debug log with a more meaningful message, printed only if DEBUG is set.

[PATCH v1 1/1] net: Add drivers for Sysnopsys Ethernet 10G device

2024-04-17 Thread Boon Khai Ng
This driver support the Synopsys Designware Ethernet 10G IP block refer from the driver dwc_eth_qos. The driver MAC register mapping is different between Synopsys QoS IP and Synopsys 10G IP, and thus new file is created meant for Sysnopsys 10G IP. The dwc_eth_xgmac_socfpga.c is specific to a

Re: [PATCH v2 5/5] common: Convert *.c/h from UTF-8 to ASCII enconfing

2024-04-17 Thread Michal Simek
On 4/16/24 18:19, Heinrich Schuchardt wrote: On 16.04.24 18:06, Tom Rini wrote: On Tue, Apr 16, 2024 at 08:55:19AM +0200, Michal Simek wrote: Convert UTF-8 chars to ASCII in cases where make sense. No Copyright or names are converted. Signed-off-by: Michal Simek Reviewed-by: Tom Rini

[PATCH v1 0/1] add driver for Synopsys Ethernet 10G device

2024-04-17 Thread Boon Khai Ng
This driver support the Synopsys Designware Ethernet 10G IP block refer from the driver dwc_eth_qos. The driver MAC register mapping is different between Synopsys QoS IP and Synopsys 10G IP, and thus new file is created meant for Sysnopsys 10G IP. The dwc_eth_xgmac_socfpga.c is specific

RE: [PATCH 1/5] mmc: am654_sdhci: Add tuning algorithm for delay chain

2024-04-17 Thread Jaehoon Chung
> -Original Message- > From: Judith Mendez > Sent: Tuesday, April 16, 2024 6:28 AM > To: Peng Fan ; Jaehoon Chung ; Tom > Rini > Cc: Nitin Yadav ; Simon Glass ; > u-boot@lists.denx.de > Subject: [PATCH 1/5] mmc: am654_sdhci: Add tuning algorithm for delay chain > > Currently the

RE: [PATCH 2/5] mmc: am654_sdhci: Fix OTAP/ITAP delay values

2024-04-17 Thread Jaehoon Chung
Hi Judith, > -Original Message- > From: Judith Mendez > Sent: Tuesday, April 16, 2024 6:28 AM > To: Peng Fan ; Jaehoon Chung ; Tom > Rini > Cc: Nitin Yadav ; Simon Glass ; > u-boot@lists.denx.de > Subject: [PATCH 2/5] mmc: am654_sdhci: Fix OTAP/ITAP delay values > > From: Nitin Yadav

RE: [PATCH 4/5] mmc: am654_sdhci: Set ENDLL=1 for DDR52 mode

2024-04-17 Thread Jaehoon Chung
> -Original Message- > From: Judith Mendez > Sent: Tuesday, April 16, 2024 6:28 AM > To: Peng Fan ; Jaehoon Chung ; Tom > Rini > Cc: Nitin Yadav ; Simon Glass ; > u-boot@lists.denx.de > Subject: [PATCH 4/5] mmc: am654_sdhci: Set ENDLL=1 for DDR52 mode > > According to the device

Re: [PATCH v2 4/4] efi_selftest: add tests for setvariableRT

2024-04-17 Thread Heinrich Schuchardt
On 17.04.24 12:19, Ilias Apalodimas wrote: Since we support SetVariableRT now add the relevant tests - Search for the RTStorageVolatile and VarToFile variables after EBS - Try to update with invalid variales (BS, RT only) - Try to write a variable bigger than our backend storage - Write a

Re: [PATCH v3] rockchip: px30-board-tpl: Sync ifdef guards with full TPL

2024-04-17 Thread Quentin Schulz
Hi Lukasz, On 4/17/24 13:21, lukasz.czechow...@thaumatec.com wrote: From: Lukasz Czechowski Display TPL init information message only when TPL_BANNER_PRINT configuration entry is set. This allows to disable information message in case logs on UART are unwanted. Update parent ifdef condition

RE: [PATCH] mmc: cv1800b: Add transmit tap delay config to fix write error

2024-04-17 Thread Jaehoon Chung
> -Original Message- > From: Kongyang Liu > Sent: Tuesday, April 16, 2024 4:31 PM > To: u-boot@lists.denx.de > Cc: Jaehoon Chung ; Leo Yu-Chi Liang > ; Peng Fan > ; Tom Rini > Subject: [PATCH] mmc: cv1800b: Add transmit tap delay config to fix write > error > > Currently, only the

Re: [PATCH v2 1/4] efi_loader: conditionally enable SetvariableRT

2024-04-17 Thread Heinrich Schuchardt
On 17.04.24 12:19, Ilias Apalodimas wrote: When we store EFI variables on file we don't allow SetVariable at runtime, since the OS doesn't know how to access or write that file. At the same time keeping the U-Boot drivers alive in runtime sections and performing writes from the firmware is

[PATCH v1] arm: dts: verdin-imx8mm/imx8mp: use gpio-hog for sleep moci

2024-04-17 Thread Stefan Eichenberger
From: Stefan Eichenberger In Linux, we allow sleep moci to be turned off when the carrier board supports it and the system is in suspend. In U-Boot, however, we want the sleep moci to be always on. So we use a gpio hog and disable the regulator. This change is necessary because we switched to

[PATCH v1] arm: dts: verdin-am62: use gpio-hog for sleep moci

2024-04-17 Thread Stefan Eichenberger
From: Stefan Eichenberger In Linux, we allow sleep moci to be turned off when the carrier board supports it and the system is in suspend. In U-Boot, however, we want the sleep moci to be always on. So we use a gpio hog and disable the regulator. This change is necessary because we switched to

RE: [RFC PATCH v1 1/1] mmc: snps_sdhci: Add sdhci driver support for TH1520 SoC

2024-04-17 Thread Jaehoon Chung
Hi, > -Original Message- > From: Maxim Kiselev > Sent: Wednesday, April 3, 2024 3:57 AM > To: Sean Anderson > Cc: Tom Rini ; Peng Fan ; Jaehoon Chung > ; > Marek Vasut ; Paul Barker > ; Kever Yang > ; Peter Geis ; Oleksandr > Suvorov > ; Stefan Roese ; > u-boot@lists.denx.de >

Re: [PATCH v2 01/16] board: Define GUIDs for firmware images

2024-04-17 Thread Caleb Connolly
On 12/04/2024 22:58, Jon Humphreys wrote: > Ilias Apalodimas writes: > >> Hi both >> >> On Wed, 10 Apr 2024 at 10:36, Heinrich Schuchardt wrote: >>> >>> On 09.04.24 23:05, Jon Humphreys wrote: Heinrich Schuchardt writes: > On 4/9/24 00:31, Jonathan Humphreys wrote: >>

RE: [PATCH 1/2] mmc: stm32_sdmmc2: Add "st,stm32mp25-sdmmc2" compatible

2024-04-17 Thread Jaehoon Chung
Hi > -Original Message- > From: Patrick DELAUNAY > Sent: Wednesday, April 17, 2024 6:02 PM > To: Patrice Chotard ; u-boot@lists.denx.de > Cc: U-Boot STM32 ; Jaehoon Chung > ; > Peng Fan ; Sean Anderson ; Simon Glass > ; Tom > Rini > Subject: Re: [PATCH 1/2] mmc: stm32_sdmmc2: Add

Re: [PATCH v2 1/4] efi_loader: conditionally enable SetvariableRT

2024-04-17 Thread Ilias Apalodimas
On Wed, 17 Apr 2024 at 15:28, Heinrich Schuchardt wrote: > > On 17.04.24 12:19, Ilias Apalodimas wrote: > > When we store EFI variables on file we don't allow SetVariable at runtime, > > since the OS doesn't know how to access or write that file. At the same > > time keeping the U-Boot drivers

[PATCH v2 1/4] efi_loader: conditionally enable SetvariableRT

2024-04-17 Thread Ilias Apalodimas
When we store EFI variables on file we don't allow SetVariable at runtime, since the OS doesn't know how to access or write that file. At the same time keeping the U-Boot drivers alive in runtime sections and performing writes from the firmware is dangerous -- if at all possible. For GetVariable

[PATCH v2 0/4] Enable SetVariable at runtime

2024-04-17 Thread Ilias Apalodimas
Hi all, This is the new version of [0] The main difference from v1 is that VarToFile is now filled on the fly, during a GetVariable call for it, instead of creating it during SetVariable. The advantage of doing that is memory efficiency, since the buffer comes from the OS now and we don't have

Re: [PATCH v2 4/4] efi_selftest: add tests for setvariableRT

2024-04-17 Thread Ilias Apalodimas
Hi Heinrich, [...] > > > > + memset(v2, 0x1, sizeof(v2)); > > ret = runtime->query_variable_info(EFI_VARIABLE_BOOTSERVICE_ACCESS, > > _storage, _storage, > > _size); > > @@ -63,10 +69,107 @@ static int

[PATCH 2/2] cmd: sbi: add coreboot and oreboot implementation IDs

2024-04-17 Thread Heinrich Schuchardt
Let the sbi command detect the coreboot and oreboot SBI Implementation IDs defined in SBI specification v2.0. Signed-off-by: Heinrich Schuchardt --- cmd/riscv/sbi.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/cmd/riscv/sbi.c b/cmd/riscv/sbi.c index 55507b0aa63..2d8ee7e5bbb 100644 ---

Re: [PATCH v2] tpm: display warning if using gpio reset with TPM

2024-04-17 Thread Ilias Apalodimas
Hi Tim! > > > > > > > > > > The current logic expects a reset gpio and bails out if it cannot get > > > > > it with a (questionable) goto statement. > > > > > > > > > > You want to invert that logic, and expect no gpio, but only if there > > > > > is > > > > > one you want to warn. > > > > > > >

Re: [PATCH v6 0/5] imx93: Conver to OF_UPSTREAM

2024-04-17 Thread Fabio Estevam
On Mon, Apr 15, 2024 at 9:55 AM Peng Fan wrote: > ok. > Do I need to switch back to only convert i.MX93 11x11 EVK to > OF_UPSTREM? Yes, please convert only the imx93 evk board for now. Currently, on the dts/upstream repo in U-Boot master, only the imx93 evk board is present. The Variscite and

[PATCH 1/2] cmd: sbi: add Supervisor Software Events extension

2024-04-17 Thread Heinrich Schuchardt
OpenSBI has implemented the Supervisor Software Events Extension. Allow detecting it in the sbi command. Signed-off-by: Heinrich Schuchardt --- arch/riscv/include/asm/sbi.h | 1 + cmd/riscv/sbi.c | 1 + 2 files changed, 2 insertions(+) diff --git a/arch/riscv/include/asm/sbi.h

[PATCH 1/1] cmd: eficonfig: check initrd path allocation

2024-04-17 Thread Heinrich Schuchardt
After allocating memory for the initrd file path we need to check the initrd buffer pointer is not NULL. Fixes: 87d791423ac6 ("eficonfig: menu-driven addition of UEFI boot option") Signed-off-by: Heinrich Schuchardt --- cmd/eficonfig.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

Re: [PATCH 01/10] board: ti: am62x: Init DRAM size in R5/A53 SPL

2024-04-17 Thread Tom Rini
On Wed, Apr 17, 2024 at 05:48:31PM +0530, Sughosh Ganu wrote: > hi Chintan, > > On Wed, 17 Apr 2024 at 13:21, Chintan Vankar wrote: > > > > > > > > On 16/04/24 22:30, Tom Rini wrote: > > > On Tue, Apr 16, 2024 at 05:52:58PM +0530, Chintan Vankar wrote: > > >> > > >> > > >> On 12/04/24 03:37, Tom

Re: [PATCH v2 3/4] efi_loader: add an EFI variable with the file contents

2024-04-17 Thread Ilias Apalodimas
Hi Heinrich, [...] > > { > > int ret = 0; > > > > diff --git a/lib/efi_loader/efi_runtime.c b/lib/efi_loader/efi_runtime.c > > index c8f7a88ba8db..99ad1f35d8f1 100644 > > --- a/lib/efi_loader/efi_runtime.c > > +++ b/lib/efi_loader/efi_runtime.c > > @@ -130,6 +130,8 @@ efi_status_t

Re: [PATCH] ram: rockchip: px30: Replace misleading log message

2024-04-17 Thread Łukasz Czechowski
Hi Quentin, Maybe I was a little too fast with submitting this patch. My original point of this commit was to get rid of the message "out", that is printed without any context every time the board boots, or at least replace it with something more descriptive, similarly like it is done with other

Re: [Patch] dwc_eth_qos: Revert regression handling fixed phy

2024-04-17 Thread Marek Vasut
On 4/17/24 10:41 AM, Nicole Battenfeld wrote: Subject: [PATCH] dwc_eth_qos: Revert regression handling fixed phy In imx8mp operation on eqos with fixed phy I get without that patch: ERROR: no/invalid Which commit is being reverted here ?

Re: [PATCH v2] tpm: display warning if using gpio reset with TPM

2024-04-17 Thread Tim Harvey
On Wed, Apr 17, 2024 at 12:01 AM Ilias Apalodimas wrote: > > On Wed, 17 Apr 2024 at 09:48, Miquel Raynal wrote: > > > > Hi Ilias, > > > > ilias.apalodi...@linaro.org wrote on Wed, 17 Apr 2024 08:40:14 +0300: > > > > > Hi Miquel > > > > > > On Mon, 8 Apr 2024 at 10:23, Miquel Raynal > > >

Re: [PATCH] ram: rockchip: px30: Replace misleading log message

2024-04-17 Thread Quentin Schulz
Hi Lukasz, On 4/17/24 13:16, lukasz.czechow...@thaumatec.com wrote: From: Lukasz Czechowski Remove the log message "out" from sdram_init function which pollutes the console. It brings no meaningful information and might be unwanted in case silencing the console is required. Instead, add a

Re: [PATCH] ram: rockchip: px30: Replace misleading log message

2024-04-17 Thread Quentin Schulz
BTW, On 4/17/24 13:16, lukasz.czechow...@thaumatec.com wrote: From: Lukasz Czechowski There is a typo in your mail address. I assume your git config user.email may be wrong, because your signed-off-by is correct. Cheers, Quentin

Re: [PATCH] imx8m*-venice: enable TPM support

2024-04-17 Thread Fabio Estevam
Hi Tim, On Wed, Apr 17, 2024 at 2:00 PM Tim Harvey wrote: > Fabio, > > This one got tagged as 'changes requested' in patchwork but I'm not > clear why. Are there any issues with this? Sorry, this was a mistake. I will apply it in the next batch of patches.

Re: [PATCH v1] mtd: rawnand: macronix: OTP access for MX30LFxG18AC

2024-04-17 Thread Michael Nazzareno Trimarchi
Hi Dario did you add those patches in CI and test them again? Michael On Wed, Apr 17, 2024 at 8:44 PM Arseniy Krasnov wrote: > > Hello, > > Sorry, pls ping > > Thanks, Arseniy > > On 13.03.2024 09:46, Michael Nazzareno Trimarchi wrote: > > Hi Dario > > > > Can apply this series and put in CI?

Re: [PATCH 4/4] configs: visionfive2: enable SPL_YMODEM_SUPPORT

2024-04-17 Thread E Shattow
P.S. mis-spoke should have written "Having 'env load' allows to skip that reboot step in-between On Wed, Apr 17, 2024 at 12:28 PM E Shattow wrote: > > On Wed, Apr 17, 2024 at 1:22 AM Heinrich Schuchardt > wrote: > > > > On 17.04.24 03:41, E Shattow wrote: > > > Successful in use w/ 'tio' serial

Re: [PATCH 0/5] zfs: Fix zfs support on aarch64

2024-04-17 Thread Tom Rini
On Sat, 06 Apr 2024 18:47:24 -0700, mwle...@mailtundra.com wrote: > This patch series is needed to get U-Boot to boot from a ZFS filesystem > on an aarch64 computer. Some of the patches are not architecture specific > and would be needed to boot ZFS on other platforms as well. The ZFS > support

[PATCH] imx8m*_venice_defconfig: enable ipv6, wget and tftpput

2024-04-17 Thread Tim Harvey
Enable ipv6, wget, and tftpput support Signed-off-by: Tim Harvey --- configs/imx8mm_venice_defconfig | 5 + configs/imx8mn_venice_defconfig | 5 + configs/imx8mp_venice_defconfig | 5 + 3 files changed, 15 insertions(+) diff --git a/configs/imx8mm_venice_defconfig

[PATCH] .gitignore: Add files produced by TI platform

2024-04-17 Thread Andrea Calabrese
Add files produced by compilation of TI platforms: *.ti-secure(-rom) *.map *-board-config custMpk.pem *.bin_* *.fit *.itb tispl.bin_unsigned Signed-off-by: Andrea Calabrese --- .gitignore | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/.gitignore b/.gitignore index

Re: [Patch] dwc_eth_qos: Revert regression handling fixed phy

2024-04-17 Thread Elmar Psilog
On 4/17/24 13:15, Marek Vasut wrote: On 4/17/24 10:41 AM, Nicole Battenfeld wrote: Subject: [PATCH] dwc_eth_qos: Revert regression handling fixed phy In imx8mp operation on eqos with fixed phy I get without that patch: ERROR: no/invalid Which commit is being reverted here ? True. I just

Re: [PATCH] imx8m*-venice: enable TPM support

2024-04-17 Thread Tim Harvey
On Mon, Mar 25, 2024 at 9:27 AM Tim Harvey wrote: > > Enable support for TPM2 devices. As the ATTPM20P TPM2 used on the > Gateworks Venice boards hangs off the SPI bus we enable SPI support > as well. > > Signed-off-by: Tim Harvey > --- > configs/imx8mm_venice_defconfig | 10 ++ >

Re: [PATCH v1] mtd: rawnand: macronix: OTP access for MX30LFxG18AC

2024-04-17 Thread Arseniy Krasnov
Hello, Sorry, pls ping Thanks, Arseniy On 13.03.2024 09:46, Michael Nazzareno Trimarchi wrote: > Hi Dario > > Can apply this series and put in CI? > > Michael > > On Wed, Mar 13, 2024 at 7:43 AM Arseniy Krasnov > wrote: >> >> Sorry, please ping >> >> Thanks, Arseniy >> >> On 11.02.2024

Re: [PATCH 4/4] configs: visionfive2: enable SPL_YMODEM_SUPPORT

2024-04-17 Thread E Shattow
On Wed, Apr 17, 2024 at 1:22 AM Heinrich Schuchardt wrote: > > On 17.04.24 03:41, E Shattow wrote: > > Successful in use w/ 'tio' serial tool and Adafruit CP2102N Friend > > adapter on Mars CM Lite board in DFRobot mini router carrier. > > > > While SPL and u-boot.itb payload are sourced via UART

[PATCH] board: gateworks: venice: fix dt adjustment for gw73xx baseboard for imx8mp

2024-04-17 Thread Tim Harvey
The GW73xx baseboard needs a PCI dt adjustment for revC/D based on a change of the PCIe switch. Make sure we are only doing this for a pci based ethernet to avoid causing a boot hang when the ethernet1 alias points to eqos or fec. To know this is a pcie device ensure the alias begins with the pcie

Re: [PATCH v2 00/11] net: dwc_eth_qos: Clean up STM32 glue code and add STM32MP13xx support

2024-04-17 Thread Marek Vasut
On 3/26/24 1:07 PM, Marek Vasut wrote: Split off STM32 glue code from the DWMAC driver into separate file, similar to what other SoCs already do, to avoid mixing the ST specifics with generic DWMAC core code. Clean the STM32 DWMAC board code which is currently duplicated in multiple board

Re: [PATCH 00/15] configs: ti: Enable basic settings for SystemReady ACS

2024-04-17 Thread Tom Rini
On Mon, 08 Apr 2024 16:10:45 -0500, Jonathan Humphreys wrote: > Set basic settings needed for System Ready IR ACS testing, for several TI SoC > based platforms: AM64, AM62, AM62p, BeaglePlay, J7, and BeagleboneAI. > > For AM64, AM62, and AM62p, also includes some config cleanup. Should be no >

  1   2   >