Re: [PATCH v2 1/1] efi_loader: typo mstching

2024-04-17 Thread Ilias Apalodimas
On Thu, 18 Apr 2024 at 05:32, Heinrich Schuchardt wrote: > > %s/mstching/matching/ > > Reported-by: E Shattow > Signed-off-by: Heinrich Schuchardt > --- > v2: > correct Reported-by line > --- > lib/efi_loader/efi_var_common.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > >

Re: [PATCH 1/1] sandbox: don't call os_close with invalid file descriptor

2024-04-17 Thread Tom Rini
On Wed, 10 Apr 2024 23:50:34 +0200, Heinrich Schuchardt wrote: > If open() fails it returns -1. Calling close() with this value > makes no sense. Return -EIO instead. > > Applied to u-boot/master, thanks! -- Tom

Re: [PATCH] arm: mach-k3: security: Lower verbosity of cert message for GP

2024-04-17 Thread Tom Rini
On Wed, 10 Apr 2024 13:38:34 -0500, Andrew Davis wrote: > When we find a certificate on an image to be booted on a GP device we > print out a message explaining that the certificate is being skipped. > This message is rather long and is printed for every image. Shorten > the message and make the

Re: [PATCH] configs: am64x_evm_*_defconfig: Increase offsets for eMMC raw boot

2024-04-17 Thread Tom Rini
On Wed, 10 Apr 2024 13:20:16 -0500, Judith Mendez wrote: > EMMC boot can fail due to the size of R5 SPL image growing beyond the > 500KB of memory allocated in eMMC. Update offsets for eMMMC raw boot > to load each binary from the correct address in eMMC according to the > following eMMC layout:

Re: [PATCH v1] configs: arbel: Use generic timer and npcm reset driver

2024-04-17 Thread Tom Rini
On Wed, 10 Apr 2024 13:54:37 +0800, Jim Liu wrote: > Modify defconfig to use generic timer and npcm reset driver > > Applied to u-boot/master, thanks! -- Tom

Re: [PATCH 1/1] fs: fat: fill creation and change date

2024-04-17 Thread Tom Rini
On Tue, 09 Apr 2024 22:06:07 +0200, Heinrich Schuchardt wrote: > The FAT specification requires that the change date is set. > > If a DM RTC device exists, set the creation and change date to the current > date when updating the directory entry. Otherwise use the date 2020-01-01. > > Applied

Re: [PATCH 1/1] fs: fat: convert change month correctly

2024-04-17 Thread Tom Rini
On Tue, 09 Apr 2024 20:04:56 +0200, Heinrich Schuchardt wrote: > The month is stored in 5 - 8. We need to shift it by 5 bits. > > Cf. Microsoft FAT Specification, 2005-08-30 > > Applied to u-boot/master, thanks! -- Tom

Re: [PATCH v4] tools: copyfile: use 64k instead of 512 buffer

2024-04-17 Thread Tom Rini
On Tue, 09 Apr 2024 14:14:34 +0200, Ahelenia Ziemiańska wrote: > This is a trivial but significant optimization: > mkimage took >200ms (and 49489 writes (of which 49456 512)), > now it takes 110ms (and 419 writes (of which 386 64k)). > > sendfile is much more appropriate for this and is

Re: [PATCH] Makefile.lib: find capsule ESL dtsi file with CONFIG_OF_UPSTREAM

2024-04-17 Thread Tom Rini
On Mon, 08 Apr 2024 16:28:43 -0500, Jonathan Humphreys wrote: > When CONFIG_OF_UPSTREAM is enabled, DTS files are in SOC subdirectories (vs > the > top level dts directory), but when CONFIG_EFI_CAPSULE_AUTHENTICATE is enabled, > the dynamically created dtsi file containing the capsule ESL DT

Re: [PATCH 1/1] test: typo curren

2024-04-17 Thread Tom Rini
On Tue, 09 Apr 2024 09:55:49 +0200, Heinrich Schuchardt wrote: > Fix typos in test_eficonfig.py: %s/curren/current/ > > Applied to u-boot/master, thanks! -- Tom

[PATCH 1/1] acpi: set creator_revision in acpi_fill_header

2024-04-17 Thread Heinrich Schuchardt
We should have a single place where we write the default value to the creator revision field. If we ever will have any table created by another tool, we can overwrite the value afterwards. Signed-off-by: Heinrich Schuchardt --- arch/x86/lib/acpi_table.c | 2 -- lib/acpi/acpi_table.c | 2 +-

Re: [PATCH 2/3] pci: dw_imx: add support for IMX8MM

2024-04-17 Thread Marek Vasut
On 4/17/24 10:09 PM, Tim Harvey wrote: Add support for the IMX8MM SoC by adding driver data with the compatible string of the GPR controller. Signed-off-by: Tim Harvey --- drivers/pci/pcie_dw_imx.c | 20 ++-- 1 file changed, 18 insertions(+), 2 deletions(-) diff --git

[PATCH v2 1/1] efi_loader: typo mstching

2024-04-17 Thread Heinrich Schuchardt
%s/mstching/matching/ Reported-by: E Shattow Signed-off-by: Heinrich Schuchardt --- v2: correct Reported-by line --- lib/efi_loader/efi_var_common.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/efi_loader/efi_var_common.c b/lib/efi_loader/efi_var_common.c

[PATCH 1/1] efi_loader: typo mstching

2024-04-17 Thread Heinrich Schuchardt
%s/mstching/matching/ Reported-by: Reported-by: E Shattow Signed-off-by: Heinrich Schuchardt --- lib/efi_loader/efi_var_common.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/efi_loader/efi_var_common.c b/lib/efi_loader/efi_var_common.c index aa8feffd3ec..ed53bcf3939

Re: [PATCH v3 0/2] upstream DT compatibility

2024-04-17 Thread Tom Rini
On Tue, Apr 09, 2024 at 04:49:23PM +0200, Caleb Connolly wrote: > This is a subset of [1]. With more platform maintainers switching to > OF_UPSTREAM I didn't want to get in the way (it has also proven more > difficult than I hoped to remove only the fully compatible header > files). > > This

Re: [RESEND PATCH v3 5/5] power: regulator: tps65941: Add TPS65224 PMIC regulator support

2024-04-17 Thread Jaehoon Chung
On 3/18/24 18:49, Bhargav Raviprakash wrote: > Reuse TPS65941 regulator driver to adds support for > TPS65224 PMIC's regulators. 4 BUCKs and 3 LDOs, where > BUCK1 and BUCK2 can be configured in dual phase mode. > > Signed-off-by: Bhargav Raviprakash > --- >

Re: [RESEND PATCH v3 3/5] power: regulator: tps65941: Added macros for BUCK ID

2024-04-17 Thread Jaehoon Chung
On 3/18/24 18:49, Bhargav Raviprakash wrote: > Adds macros for buck and ldo ids and switched to using switch > case instead of if else in probe functions. Helps in adding > support for TPS65224 PMIC. > > Signed-off-by: Bhargav Raviprakash > Reviewed-by: Dhruva Gole Reviewed-by: Jaehoon Chung

Re: [RESEND PATCH v3 4/5] power: regulator: tps65941: use function callbacks for conversion ops

2024-04-17 Thread Jaehoon Chung
On 3/18/24 18:49, Bhargav Raviprakash wrote: > Use function callbacks for volt2val, val2volt and slewrate lookups. > This makes it easier to add support for TPS65224 PMIC regulators. > > Signed-off-by: Bhargav Raviprakash Reviewed-by: Jaehoon Chung Best Regards, Jaehoon Chung > --- >

Re: [RESEND PATCH v3 2/5] power: pmic: tps65941: Add TI TPS65224 PMIC

2024-04-17 Thread Jaehoon Chung
On 3/18/24 18:49, Bhargav Raviprakash wrote: > Adds compatible and data field values of TPS65224 driver in > TPS65941 PMIC driver. > > Signed-off-by: Bhargav Raviprakash > Reviewed-by: Dhruva Gole Reviewed-by: Jaehoon Chung Best Regards, Jaehoon Chung > --- > drivers/power/pmic/tps65941.c

Re: [RESEND PATCH v3 1/5] power: tps65941: Add macros of TPS65224 PMIC

2024-04-17 Thread Jaehoon Chung
On 3/18/24 18:49, Bhargav Raviprakash wrote: > Re-use the TPS65941 PMIC driver for TPS65224 PMIC. > Add additional macros of TPS65224 to aid in the driver > re-use. > > Signed-off-by: Bhargav Raviprakash > Reviewed-by: Dhruva Gole Reviewed-by: Jaehoon Chung Best Regards, Jaehoon Chung > ---

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 >

Re: [PATCH 00/23] [RFC] Integrate MbedTLS v3.6 LTS with U-Boot

2024-04-17 Thread Tom Rini
On Wed, Apr 17, 2024 at 04:48:14PM -0400, Raymond Mao wrote: > Hi Tom, > > On Tue, 16 Apr 2024 at 15:23, Tom Rini wrote: > > > On Tue, Apr 16, 2024 at 11:59:56AM -0700, Raymond Mao wrote: > > > > > Integrate MbedTLS v3.6 LTS (currently v3.6.0-RC1) with U-Boot. > > > > > > Patch 01 and 02 are

Re: [PATCH 00/23] [RFC] Integrate MbedTLS v3.6 LTS with U-Boot

2024-04-17 Thread Raymond Mao
Hi Tom, On Tue, 16 Apr 2024 at 15:23, Tom Rini wrote: > On Tue, Apr 16, 2024 at 11:59:56AM -0700, Raymond Mao wrote: > > > Integrate MbedTLS v3.6 LTS (currently v3.6.0-RC1) with U-Boot. > > > > Patch 01 and 02 are for introducing MbedTLS release package. > > I have to split it into 2 parts due

[PATCH 1/3] clk: imx8mm: Add support for PCIe clocks

2024-04-17 Thread Tim Harvey
Add support for PCIe clocks required to enable PCIe support on iMX8MM SoC. Signed-off-by: Tim Harvey --- drivers/clk/imx/clk-imx8mm.c | 21 + 1 file changed, 21 insertions(+) diff --git a/drivers/clk/imx/clk-imx8mm.c b/drivers/clk/imx/clk-imx8mm.c index

[PATCH 3/3] imx8mm_venice_defconfig: Enable PCIe/NVMe support

2024-04-17 Thread Tim Harvey
Enable PCIe/NVMe support. Also, enable the reset driver which is a prerequisite for PCIe support. Signed-off-by: Tim Harvey --- configs/imx8mm_venice_defconfig | 6 ++ 1 file changed, 6 insertions(+) diff --git a/configs/imx8mm_venice_defconfig b/configs/imx8mm_venice_defconfig index

[PATCH 2/3] pci: dw_imx: add support for IMX8MM

2024-04-17 Thread Tim Harvey
Add support for the IMX8MM SoC by adding driver data with the compatible string of the GPR controller. Signed-off-by: Tim Harvey --- drivers/pci/pcie_dw_imx.c | 20 ++-- 1 file changed, 18 insertions(+), 2 deletions(-) diff --git a/drivers/pci/pcie_dw_imx.c

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

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

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

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

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

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

[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

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

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,

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

[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

[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

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

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

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

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

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

[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

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

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

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

[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

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

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

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

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

  1   2   >