[PATCH 2/2] board: dh_stm32mp1: Only print board code with CONFIG_SPL_DISPLAY_PRINT

2023-09-27 Thread Harald Seiler
Ensure that the SoM and board code information is only printed when CONFIG_SPL_DISPLAY_PRINT is set. Signed-off-by: Harald Seiler --- board/dhelectronics/dh_stm32mp1/board.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/board/dhelectronics/dh_stm32mp1/board.c b/board

[PATCH 1/2] ram: stm32mp1: Only print RAM config with CONFIG_SPL_DISPLAY_PRINT

2023-09-27 Thread Harald Seiler
Ensure that the RAM configuration line is only printed when CONFIG_SPL_DISPLAY_PRINT is set. Signed-off-by: Harald Seiler --- drivers/ram/stm32mp1/stm32mp1_ram.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/ram/stm32mp1/stm32mp1_ram.c b/drivers/ram/stm32mp1

Re: [PATCH] CI: gitlab: Collect pytest artifacts

2023-02-28 Thread Harald Seiler
Hi Marek, On Tue, 2023-02-28 at 03:25 +0100, Marek Vasut wrote: > On 2/28/23 00:36, Tom Rini wrote: > > [...] > > > > @@ -454,9 +457,4 @@ coreboot test.py: > > > TEST_PY_BD: "coreboot" > > > TEST_PY_TEST_SPEC: "not sleep" > > > TEST_PY_ID: "--id qemu" > > > - artifacts: > > >

Re: [PATCH] usb: gadget: dwc2_udc_otg: implement pullup()

2023-01-10 Thread Harald Seiler
Hi, On Tue, 2023-01-10 at 10:11 +0100, Mattijs Korpershoek wrote: > Pullup is used by the usb framework in order to do software-controlled > usb_gadget_connect() and usb_gadget_disconnect(). > > Implement pullup() for dwc2 using the SOFT_DISCONNECT bit in the dctl > register. > > This is

[PATCH] Revert "time: add weak annotation to timer_read_counter declaration"

2023-01-04 Thread Harald Seiler
again as they are obviously a very big footgun. [1]: https://stackoverflow.com/questions/31203402/gcc-behavior-for-unresolved-weak-functions Fixes: 65ba7add0d60 ("time: add weak annotation to timer_read_counter declaration") Reported-by: Serge Bazanski Signed-off-by: Harald Seiler

Re: TFTP boot using DNS

2022-12-15 Thread Harald Seiler
Hi Valerio, On Thu, 2022-12-15 at 10:27 +0100, Valerio Nappi wrote: > Hello everyone, > > Context: > I'm new to the u-boot project. I am trying to boot a Zynq SoC from an > internal TFTP server, but i have no guarantee that the IP of the > server is going to stay the same. A DNS service is

Re: [PATCH] console: Add option to keep it silent until env is loaded

2022-07-18 Thread Harald Seiler
Hi Simon, On Wed, 2022-07-13 at 09:28 -0600, Simon Glass wrote: > Hi Harald, > > On Tue, 12 Jul 2022 at 05:58, Harald Seiler wrote: > > > > Hi Simon, > > > > On Tue, 2022-07-12 at 04:58 -0600, Simon Glass wrote: > > > Hi Harald, > > > >

Re: [PATCH] console: Add option to keep it silent until env is loaded

2022-07-12 Thread Harald Seiler
Hi Simon, On Tue, 2022-07-12 at 04:58 -0600, Simon Glass wrote: > Hi Harald, > > On Wed, 6 Jul 2022 at 05:19, Harald Seiler wrote: > > > > Add a config-option which forces the console to stay silent until the > > proper environment is loaded from flash. >

[PATCH v2] spl: mmc: Use correct MMC device when loading image

2022-07-11 Thread Harald Seiler
corrupted or missing on the first attempted MMC device. Fixes: e1eb6ada4e38 ("spl: Make image loader infrastructure more universal") Signed-off-by: Harald Seiler --- Notes: Changes in v2: - Make it work with CONFIG_SPL_BLK as well common/spl/spl_mmc.c | 17 +++--

Re: [PATCH] spl: mmc: Use correct MMC device when loading image

2022-07-11 Thread Harald Seiler
Hi, On Mon, 2022-07-11 at 12:18 +0200, Quentin Schulz wrote: > Hi Harald, > > On 7/6/22 12:58, Harald Seiler wrote: > > When attempting to load images from multiple MMC devices in sequence, > > spl_mmc_load() chooses the wrong device from the second attempt onwards

[PATCH] console: Add option to keep it silent until env is loaded

2022-07-06 Thread Harald Seiler
is loaded, the console will be silenced/unsilenced depending on it. If PRE_CONSOLE_BUFFER is also used, the buffer will now be flushed if the console should not be silenced. Signed-off-by: Harald Seiler --- common/Kconfig | 10 ++ common/console.c | 5 + 2 files changed, 15

[PATCH] spl: mmc: Use correct MMC device when loading image

2022-07-06 Thread Harald Seiler
corrupted or missing on the first attempted MMC device. Fixes: e1eb6ada4e38 ("spl: Make image loader infrastructure more universal") Signed-off-by: Harald Seiler --- common/spl/spl_mmc.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/common/spl/spl_mmc.c b/

Re: [PATCH 2/5] imx8mm-cl-iot-gate: Retrieve the DDR type from EEPROM

2022-03-21 Thread Harald Seiler
uilt : 22:28:11, Mar 15 2022 > > Based on the original code from Compulab's U-Boot. > > Tested on a imx8mm-cl-iot-gate board populated with 4GB of RAM. > > Signed-off-by: Fabio Estevam Tested-by: Harald Seiler > --- > board/compulab/imx8mm-cl-iot-gate/ddr/ddr.c | 2

Re: Question on running uboot_testpy with tbot

2022-02-01 Thread Harald Seiler
Hi Simon, On Thu, 2022-01-27 at 09:05 -0700, Simon Glass wrote: > Hi Harald, > [...] > > Thanks for all the info. I tried master and it mostly works, although > Iget the same error: > [...] > │ File >

Re: Question on running uboot_testpy with tbot

2022-01-27 Thread Harald Seiler
Hi Simon, first of all, sorry for the super late response to this :/ I had it on my list for a long time but had trouble finding enough time to revisit the test/py integration properly. You see, the existing test/py integration has bitrotten quite a lot and I couldn't even get the existing code

Re: [PATCH] MAINTAINERS: Add watchdog maintainers entry

2022-01-17 Thread Harald Seiler
and you should have maintainer access to it. Let me know if you need anything else. -- Harald > Signed-off-by: Stefan Roese > Cc: Tom Rini > Cc: Harald Seiler > --- > MAINTAINERS | 8 > 1 file changed, 8 insertions(+) > > diff --git a/MAINTAINERS b/MAINTAINER

[PATCH] mx6: Use imx6_src_get_boot_mode() to check boot device

2021-12-01 Thread Harald Seiler
Use imx6_src_get_boot_mode() instead of manually reading SBMR1. The existing function has proper handling for software overrides of the bootdevice which can happen, for example, when booting from an alternate source using `bmode`. Signed-off-by: Harald Seiler --- arch/arm/mach-imx/mx6/soc.c

[PATCH] imx: spl: Fix typo BMODE_EMI -> BMODE_EIM

2021-12-01 Thread Harald Seiler
The interface for NOR/OneNAND is called "EIM" not "EMI". Fix this. Signed-off-by: Harald Seiler --- arch/arm/include/asm/mach-imx/sys_proto.h | 10 +- arch/arm/mach-imx/spl.c | 4 ++-- 2 files changed, 7 insertions(+), 7 deletions(-) diff --gi

Re: Two jobs at once on denx-vulcan?

2021-09-24 Thread Harald Seiler
Hi Simon, On Mon, 2021-09-20 at 08:06 -0600, Simon Glass wrote: > Hi Harald, > > On Mon, 20 Sept 2021 at 02:12, Harald Seiler wrote: > > > > Hi, > > > > On Sat, 2021-09-18 at 10:37 -0600, Simon Glass wrote: > > > Hi, > > > > > > Is

Re: Two jobs at once on denx-vulcan?

2021-09-20 Thread Harald Seiler
Hi, On Sat, 2021-09-18 at 10:37 -0600, Simon Glass wrote: > Hi, > > Is there something screwy with this? It seems that denx-vulcan does > two builds at once? > > https://source.denx.de/u-boot/custodians/u-boot-dm/-/jobs/323540 Hm, I did some changes to the vulcan runner which might have caused

Re: [PATCH] am33xx: Fix USB for am335x boards

2021-08-10 Thread Harald Seiler
Hi, On Sat, 2021-08-07 at 14:17 +0300, Matwey V. Kornilov wrote: > USB nodes were mistakenly disabled in > > commit 942853dd96df ("arm: dts: Resync BeagleBone device trees") To be precise, the problem is that only half of the device tree files were synced. am33xx.dtsi (and seemingly some

Re: [BISECTED] arm: dts: Resync BeagleBone device trees

2021-08-06 Thread Harald Seiler
On Fri, 2021-08-06 at 12:44 +0100, Peter Robinson wrote: > On Fri, Aug 6, 2021 at 12:39 PM Harald Seiler wrote: > > > > Hi, > > > > On Fri, 2021-08-06 at 13:54 +0300, Matwey V. Kornilov wrote: > > > Hello, > > > > > > I've found th

Re: [BISECTED] arm: dts: Resync BeagleBone device trees

2021-08-06 Thread Harald Seiler
Hi, On Fri, 2021-08-06 at 13:54 +0300, Matwey V. Kornilov wrote: > Hello, > > I've found that the following commit breaks USB on BeagleBone Black > board (am335x based): > > commit 942853dd96df5de1c0a2a61c877c1cf1c24f1e91 > Author: Paul Barker > Date: Mon Jul 12 21:14:09 2021 +0100 > >

Re: Setting serverip from DHCP server

2021-07-02 Thread Harald Seiler
Hi, On Thu, 2021-07-01 at 15:56 +, Gregory Anders wrote: > (Re-sending now that I'm subscribed to the list) > > Hi all, > > I am running U-Boot on an embedded device that is connected via Ethernet > to my workstation. The workstation is running dhcpd and U-Boot is able > to successfully

Re: [PATCH 2/2] ARM: imx: Pick correct eMMC boot partition from ROM log

2021-07-01 Thread Harald Seiler
MC boot > partition selection. This way, the SPL loads the remaining boot > components from the same eMMC boot partition from which it was > started, even in case of the fallback. > > Signed-off-by: Marek Vasut > Cc: Faiz Abbas > Cc: Harald Seiler > Cc: Lokesh Vutla >

Re: [PATCH u-boot v2 00/38] U-Boot LTO (Sandbox + Some ARM boards)

2021-03-12 Thread Harald Seiler
On Fri, 2021-03-12 at 16:17 +0100, Pali Rohár wrote: > On Friday 12 March 2021 16:07:22 Harald Seiler wrote: > > On Fri, 2021-03-12 at 15:26 +0100, Marek Behun wrote: > > > On Fri, 12 Mar 2021 15:21:05 +0100 > > > Harald Seiler wrote: > > > > > > &g

Re: [PATCH u-boot v2 00/38] U-Boot LTO (Sandbox + Some ARM boards)

2021-03-12 Thread Harald Seiler
On Fri, 2021-03-12 at 16:07 +0100, Harald Seiler wrote: > On Fri, 2021-03-12 at 15:26 +0100, Marek Behun wrote: > > On Fri, 12 Mar 2021 15:21:05 +0100 > > Harald Seiler wrote: > > > > > Hi Marek, > > > > > > On Fri, 2021-03-12 at 11:33 +0100, Mare

Re: [PATCH u-boot v2 00/38] U-Boot LTO (Sandbox + Some ARM boards)

2021-03-12 Thread Harald Seiler
On Fri, 2021-03-12 at 15:26 +0100, Marek Behun wrote: > On Fri, 12 Mar 2021 15:21:05 +0100 > Harald Seiler wrote: > > > Hi Marek, > > > > On Fri, 2021-03-12 at 11:33 +0100, Marek Behún wrote: > > > Hello, > > > > > > I am sending

Re: [PATCH u-boot v2 00/38] U-Boot LTO (Sandbox + Some ARM boards)

2021-03-12 Thread Harald Seiler
Hi Marek, On Fri, 2021-03-12 at 11:33 +0100, Marek Behún wrote: > Hello, > > I am sending version 2 of patches adding support for LTO to U-Boot. > > This series was tested by Github/Azure CI at >   https://github.com/u-boot/u-boot/pull/57 > > Code reduction is on average 4.23% for u-boot.bin

Re: [PATCH 0/4] Remove addr parameter from reset_cpu()

2021-03-02 Thread Harald Seiler
Hi, On Tue, 2020-12-15 at 16:47 +0100, Harald Seiler wrote: > Hi, > > this is something I had on my mind for a longer time but never got > around to actually do until now ... A while back, while working on the > patchset that led to commit c5635a032a4b ("ARM: imx8m:

Re: [PATCH 3/4] Revert "lpc32xx: cpu: add support for soft reset"

2020-12-17 Thread Harald Seiler
ld > Sylvain Lemieux > > -Original Message- > From: Harald Seiler > Sent: Tuesday, December 15, 2020 10:48 AM > To: u-boot@lists.denx.de > Cc: Harald Seiler ; Tom Rini ; Simon Glass > ; Sylvain Lemieux > Subject: [PATCH 3/4] Revert "lpc32xx: cpu:

[PATCH 4/4] reset: Remove addr parameter from reset_cpu()

2020-12-15 Thread Harald Seiler
pe argname) + reset_cpu(void) { ... } Signed-off-by: Harald Seiler --- arch/arc/lib/reset.c| 4 ++-- arch/arm/cpu/arm920t/ep93xx/cpu.c | 2 +- arch/arm/cpu/arm920t/imx/timer.c| 2 +- arch/arm/cpu/arm926ejs

[PATCH 2/4] board: ns3: Remove superfluous reset logic

2020-12-15 Thread Harald Seiler
u() parameter across the entire tree in a later patch. Fixes: b5a152e7ca0b ("board: ns3: default reset type to L3") Cc: Bharat Gooty Cc: Rayagonda Kokatanur Signed-off-by: Harald Seiler --- board/broadcom/bcmns3/ns3.c | 22 ++ 1 file changed, 2 insertions(+), 20 de

[PATCH 3/4] Revert "lpc32xx: cpu: add support for soft reset"

2020-12-15 Thread Harald Seiler
perform a hard-reset for now. This is a preparation for removal of the reset_cpu() parameter across the entire tree in a later patch. Fixes: 576007aec9a4 ("lpc32xx: cpu: add support for soft reset") Cc: Sylvain Lemieux Signed-off-by: Harald Seiler --- arch/arm/mach-lpc32xx/

[PATCH 1/4] nds32: Remove dead reset_cpu() implementation

2020-12-15 Thread Harald Seiler
. ae3xx) or realized using a WDT (e.g. ag101). Remove this left-over implementation in preparation for the removal of the `addr` parameter in the entire tree. Cc: Rick Chen Signed-off-by: Harald Seiler --- arch/nds32/cpu/n1213/start.S | 22 -- 1 file changed, 22 deletions

[PATCH 0/4] Remove addr parameter from reset_cpu()

2020-12-15 Thread Harald Seiler
u()" for details. I also ran buildman for all boards I could, to verify that everything still compiles. One notable exception is NDS32 because I couldn't get the compiler to work there ... Regards, Harald Harald Seiler (4): nds32: Remove dead reset_cpu() implementation board: ns3:

Re: [PATCH 6/6] env: Add support for explicit write access list

2020-06-25 Thread Harald Seiler
Hi Marek, On Wed, 2020-06-03 at 02:01 +0200, Marek Vasut wrote: > This option marks any U-Boot variable which does not have explicit 'w' > writeable flag set as read-only. This way the environment can be locked > down and only variables explicitly configured to be writeable can ever > be changed

[PATCH] common: hash: Remove a debug printf statement

2020-06-15 Thread Harald Seiler
Remove a left-over debug printf that was introduced with SHA512 support. Fixes: d16b38f42704 ("Add support for SHA384 and SHA512") Signed-off-by: Harald Seiler --- Notes: Just noticed the SHA512 patch seems to still contain a left-over debug printf. common/hash.c | 2

[PATCH] tools: fw_env: Fix warning when reading too little

2020-05-28 Thread Harald Seiler
. Signed-off-by: Harald Seiler --- tools/env/fw_env.c | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/tools/env/fw_env.c b/tools/env/fw_env.c index 8734663cd4c7..c6378ecf34f6 100644 --- a/tools/env/fw_env.c +++ b/tools/env/fw_env.c @@ -946,11 +946,17 @@ static int

Re: [PATCH] imx8mp_evk: Make SPL binary size smaller

2020-05-07 Thread Harald Seiler
Hello Fabio, On Thu, 2020-05-07 at 09:00 -0300, Fabio Estevam wrote: > Commit f24dea4e1b52 ("ARM: imx8m: Fix reset in SPL on NXP iMX8MP EVK") caused > the u-boot-spl.bin binary size to grow by around 2000 bytes, which makes the > board to fail to boot. Just to check, the addition of the clock

Re: [PATCH 6/8] ARM: imx8m: Fix reset in SPL on NXP iMX8MP EVK

2020-05-05 Thread Harald Seiler
Hello Fabio, On Mon, 2020-05-04 at 12:28 -0300, Fabio Estevam wrote: > On Mon, May 4, 2020 at 12:21 PM Harald Seiler wrote: > > > With or without the revert? > > When I change the defconfig like: > > --- a/configs/imx8mp_evk_defconfig > +++ b/configs/imx8mp_evk

Re: [PATCH 6/8] ARM: imx8m: Fix reset in SPL on NXP iMX8MP EVK

2020-05-04 Thread Harald Seiler
On Mon, 2020-05-04 at 12:18 -0300, Fabio Estevam wrote: > On Mon, May 4, 2020 at 12:05 PM Harald Seiler wrote: > > > "Failed to find clock node. Check device tree" comes from spl_board_init() > > in board/freescale/imx8mp_evk/spl.c; line 56: > > > >

Re: [PATCH 6/8] ARM: imx8m: Fix reset in SPL on NXP iMX8MP EVK

2020-05-04 Thread Harald Seiler
Hello, On Mon, 2020-05-04 at 16:32 +0200, Marek Vasut wrote: > On 5/4/20 4:27 PM, Fabio Estevam wrote: > > Hi, > > > > On Wed, Apr 29, 2020 at 10:05 AM Harald Seiler wrote: > > > From: Marek Vasut > > > > > > Board files should not re-impleme

Re: [PATCH] imx: spl: Fix build-time test for CONFIG_SPL_FS_FAT

2020-04-30 Thread Harald Seiler
Hello Jan, On Wed, 2020-04-29 at 19:23 +0200, Jan Kiszka wrote: > From: Jan Kiszka > > This was already changed in 0c3a9ed409a5 but apparently missed when > adding 9d86dbd9cf9d. I sent a very similar patch [1] as part of my "Fix spl_mmc_boot_mode() implementation for IMX" series [2] which

Re: [PATCH 3/8] ARM: imx8m: Don't use the addr parameter of reset_cpu()

2020-04-29 Thread Harald Seiler
Hello Fabio, On Wed, 2020-04-29 at 10:14 -0300, Fabio Estevam wrote: > Hi Harald, > > On Wed, Apr 29, 2020 at 10:04 AM Harald Seiler wrote: > > From: Claudius Heine > > > > imx8m has the only implementation of reset_cpu() which does not ignore > > the addr pa

[PATCH 6/8] ARM: imx8m: Fix reset in SPL on NXP iMX8MP EVK

2020-04-29 Thread Harald Seiler
to implement the same as the do_reset() hack in the board file, except correctly in the DM/DT framework. Signed-off-by: Marek Vasut Cc: Fabio Estevam Cc: Flavio Suligoi Cc: Harald Seiler Cc: Igor Opaniuk Cc: Marcel Ziswiler Cc: Oleksandr Suvorov Cc: Peng Fan Cc: Stefano Babic --- arch/arm

[PATCH 5/8] ARM: imx8m: Fix reset in SPL on NXP iMX8MN EVK

2020-04-29 Thread Harald Seiler
to implement the same as the do_reset() hack in the board file, except correctly in the DM/DT framework. Signed-off-by: Marek Vasut Cc: Fabio Estevam Cc: Flavio Suligoi Cc: Harald Seiler Cc: Igor Opaniuk Cc: Marcel Ziswiler Cc: Oleksandr Suvorov Cc: Peng Fan Cc: Stefano Babic --- arch/arm

[PATCH 8/8] ARM: reset: use do_reset in SPL/TPL if SYSRESET was not enabled for them

2020-04-29 Thread Harald Seiler
From: Claudius Heine In case CONFIG_SYSRESET is set, do_reset() from reset.c will not be available anywere, even if SYSRESET is disabled for SPL/TPL. do_reset() is called from SPL for instance from the panic handler and PANIC_HANG is not set Signed-off-by: Claudius Heine ---

[PATCH 4/8] ARM: imx8m: Fix reset in SPL on NXP iMX8MM EVK

2020-04-29 Thread Harald Seiler
to implement the same as the do_reset() hack in the board file, except correctly in the DM/DT framework. Signed-off-by: Marek Vasut Cc: Fabio Estevam Cc: Flavio Suligoi Cc: Harald Seiler Cc: Igor Opaniuk Cc: Marcel Ziswiler Cc: Oleksandr Suvorov Cc: Peng Fan Cc: Stefano Babic --- arch/arm

[PATCH 2/8] ARM: imx8m: Fix indentation of reset_cpu() function

2020-04-29 Thread Harald Seiler
Use proper code-style, tabs instead of spaces for indentation. Signed-off-by: Harald Seiler --- arch/arm/mach-imx/imx8m/soc.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/arch/arm/mach-imx/imx8m/soc.c b/arch/arm/mach-imx/imx8m/soc.c index

[PATCH 7/8] ARM: imx8m: Fix reset in SPL on Toradex iMX8MM Verdin

2020-04-29 Thread Harald Seiler
to implement the same as the do_reset() hack in the board file, except correctly in the DM/DT framework. Signed-off-by: Marek Vasut Cc: Fabio Estevam Cc: Flavio Suligoi Cc: Harald Seiler Cc: Igor Opaniuk Cc: Marcel Ziswiler Cc: Oleksandr Suvorov Cc: Peng Fan Cc: Stefano Babic --- arch/arm

[PATCH 3/8] ARM: imx8m: Don't use the addr parameter of reset_cpu()

2020-04-29 Thread Harald Seiler
. Fixes: d2041725e84b ("imx8m: restrict reset_cpu") Co-developed-by: Harald Seiler Signed-off-by: Harald Seiler Signed-off-by: Claudius Heine --- arch/arm/mach-imx/imx8m/soc.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/arch/arm/mach-imx/imx8m/soc.c b/arc

[PATCH 1/8] ARM: imx8m: Do not define do_reset() if sysreset is enabled

2020-04-29 Thread Harald Seiler
From: Marek Vasut The SPL can also be compiled with sysreset drivers just fine, so update the condition to cater for that option. Signed-off-by: Marek Vasut Cc: Fabio Estevam Cc: Flavio Suligoi Cc: Harald Seiler Cc: Igor Opaniuk Cc: Marcel Ziswiler Cc: Oleksandr Suvorov Cc: Peng Fan Cc

[PATCH 0/8] ARM: imx: Fix reset in SPL

2020-04-29 Thread Harald Seiler
: Don't use the addr parameter of reset_cpu() ARM: reset: use do_reset in SPL/TPL if SYSRESET was not enabled for them Harald Seiler (1): ARM: imx8m: Fix indentation of reset_cpu() function Marek Vasut (5): ARM: imx8m: Do not define do_reset() if sysreset is enabled ARM: imx8m: Fix reset

Re: [PATCH 1/5] ARM: imx: Do not define do_reset() if sysreset is enabled

2020-04-29 Thread Harald Seiler
Hello Marek, On Wed, 2020-04-29 at 00:26 +0200, Marek Vasut wrote: > On 4/28/20 11:23 PM, Harald Seiler wrote: > > Hello Marek, > > Hi, > > > On Tue, 2020-04-28 at 16:22 +0200, Marek Vasut wrote: > > > The SPL can also be compiled with sysreset drivers just fi

Re: [PATCH 1/5] ARM: imx: Do not define do_reset() if sysreset is enabled

2020-04-28 Thread Harald Seiler
sut > Cc: Fabio Estevam > Cc: Flavio Suligoi > Cc: Harald Seiler > Cc: Igor Opaniuk > Cc: Marcel Ziswiler > Cc: Oleksandr Suvorov > Cc: Peng Fan > Cc: Stefano Babic > --- > arch/arm/mach-imx/imx8m/soc.c | 2 +- > 1 file changed, 1 insertion(+), 1 delet

Re: [PATCH 0/3] ARM: Fix reset in SPL if SYSRESET is not used

2020-04-23 Thread Harald Seiler
Hello, On Wed, 2020-03-04 at 15:23 +0100, Harald Seiler wrote: > Hello, > > continuing on the discussion around Claudius' patch for fixing reset in SPL > [1] > we have taken a closer look at the issue. To quickly summarize the situation: > > The original patch was

[PATCH 5/5] imx: spl: Fix use of removed SPL_FAT_SUPPORT config

2020-04-23 Thread Harald Seiler
CONFIG_SPL_FAT_SUPPORT was removed in commit 0c3a9ed409a5 ("spl: Kconfig: Replace CONFIG_SPL_FAT_SUPPORT with CONFIG_SPL_FS_FAT"). Fixup a leftover use of the symbol. Fixes: 9d86dbd9cf9d ("imx: spl: implement spl_boot_mode for i.MX7/8/8M") Signed-off-by: Harald Seiler --

[PATCH 0/5] Fix spl_mmc_boot_mode() implementation for IMX

2020-04-23 Thread Harald Seiler
[4]: https://lists.denx.de/pipermail/u-boot/2020-April/405979.html Anatolij Gustschin (1): imx: spl: return boot mode for asked MMC device in spl_mmc_boot_mode() Harald Seiler (4): Revert "imx: Introduce CONFIG_SPL_FORCE_MMC_BOOT to force MMC boot on falcon mode" Revert "

[PATCH 1/5] imx: spl: return boot mode for asked MMC device in spl_mmc_boot_mode()

2020-04-23 Thread Harald Seiler
From: Anatolij Gustschin Boards may extend or re-define the boot list in their board_boot_order() function by modifying spl_boot_list. E.g. a board might boot SPL from a slow SPI NOR flash and then load the U-Boot from an eMMC or SD-card. Or it might use additional MMC boot device in

[PATCH 3/5] Revert "imx: defconfig: Enable CONFIG_SPL_FORCE_MMC_BOOT"

2020-04-23 Thread Harald Seiler
-by: Harald Seiler --- configs/display5_defconfig | 1 - configs/imx28_xea_defconfig | 1 - 2 files changed, 2 deletions(-) diff --git a/configs/display5_defconfig b/configs/display5_defconfig index 9026c17f3fb5..f982191f7f53 100644 --- a/configs/display5_defconfig +++ b/configs/display5_defconfig

[PATCH 4/5] imx: spl: Remove ifdefs in spl_mmc_boot_mode()

2020-04-23 Thread Harald Seiler
It is hard to read code which contains nested ifdef blocks. Replace them with normal if-blocks and the IS_ENABLED() macro. This is not only more readable but also helps as both arms are validated by the compiler in all cases. Signed-off-by: Harald Seiler --- arch/arm/mach-imx/spl.c | 38

[PATCH 2/5] Revert "imx: Introduce CONFIG_SPL_FORCE_MMC_BOOT to force MMC boot on falcon mode"

2020-04-23 Thread Harald Seiler
was introduced in the first place, in commit 2b1cdafa9fdd ("common: Pass the boot device into spl_boot_mode()"). This reverts commit 772b55723bcbe8ebe84f579d9cdc831d8e18579d. Link: https://lists.denx.de/pipermail/u-boot/2020-April/405979.html Signed-off-by: Harald Seiler --- arch/ar

Re: Your message to U-Boot awaits moderator approval

2020-04-20 Thread Harald Seiler
Hello, On Mon, 2020-04-20 at 11:59 +1200, Reuben Dowle wrote: > Hi, > > I have attempted to submit patch to this mailing list twice, and each time > the patch appears to get lost in moderation. If the patch you are talking about is "Add support for SHA384 and SHA512", then I don't think it got

Re: Network functionality in v2020.04 on Raspberry Pi 4

2020-04-17 Thread Harald Seiler
Hello Sasha, adding Matthias Brugger on Cc as he is the maintainer for Raspberry Pi. On Thu, 2020-04-16 at 20:25 +, Jevtic, Sasha wrote: > Hello all, > > I am encountering considerable difficulty with the networking > functionality on Raspberry Pi 4 on the latest release (2020.04). In >

[PATCH] ARM: imx6: DHCOM i.MX6 PDK: Fix usb-otg VBUS regulator

2020-04-16 Thread Harald Seiler
-38 g_dnl_register failed Fix this by adding the relevant GPIO to the regulator node. Fixes: 4ca99fe81aea ("ARM: imx: dh-imx6: Enable DM regulator") Signed-off-by: Harald Seiler --- Notes: This patch currently depends on "ARM: imx6: DHCOM i.MX6 PDK: Convert to DM_ETH&qu

[PATCH v4] ARM: imx6: DHCOM i.MX6 PDK: Convert to DM_ETH

2020-04-15 Thread Harald Seiler
-post-delay-us) but U-Boot currently ignores those and only supports the bus-level reset properties in the FEC node (phy-reset-gpios, phy-reset-duration, phy-reset-post-delay). Signed-off-by: Harald Seiler --- Notes: Changes in v2: - Move reset and VIO to device-tree. - Always enable

Re: [PATCH v3] ARM: imx6: DHCOM i.MX6 PDK: Convert to DM_ETH

2020-04-15 Thread Harald Seiler
Hello Marek, On Wed, 2020-04-15 at 19:02 +0200, Marek Vasut wrote: > On 4/15/20 5:54 PM, Harald Seiler wrote: > > Use DM_ETH instead of legacy networking. Add VIO as a fixed regulator > > to the relevant device-trees and augment the FEC node with properties > &g

Re: [PATCH v3] ARM: imx6: DHCOM i.MX6 PDK: Convert to DM_ETH

2020-04-15 Thread Harald Seiler
Hello Fabio, On Wed, 2020-04-15 at 13:15 -0300, Fabio Estevam wrote: > Hi Harald, > > On Wed, Apr 15, 2020 at 12:54 PM Harald Seiler wrote: > > > +/ { > > + fec_vio: regulator-fec { > > + compatible = "regulator-fixed"; > > +

[PATCH v3] ARM: imx6: DHCOM i.MX6 PDK: Convert to DM_ETH

2020-04-15 Thread Harald Seiler
and only supports the bus-level reset properties in the FEC node. Signed-off-by: Harald Seiler --- Notes: Changes in v2: - Move reset and VIO to device-tree. - Always enable the clock, not just if CONFIG_FEC_MXC=y. Changes in v3: - Rename the dt file to imx6qdl-dhcom

Re: [PATCH v2] ARM: imx6: DHCOM i.MX6 PDK: Convert to DM_ETH

2020-04-15 Thread Harald Seiler
Hello Marek, On Wed, 2020-04-15 at 16:53 +0200, Marek Vasut wrote: > On 4/15/20 4:48 PM, Harald Seiler wrote: > > Use DM_ETH instead of legacy networking. > > Some more descriptive commit message would help. > > [...] > > > diff --git a/arch/arm/dts/imx6qdl-dhc

[PATCH v2] ARM: imx6: DHCOM i.MX6 PDK: Convert to DM_ETH

2020-04-15 Thread Harald Seiler
Use DM_ETH instead of legacy networking. Signed-off-by: Harald Seiler --- Notes: Changes in v2: - Move reset and VIO to device-tree - Always enable the clock, not just if CONFIG_FEC_MXC=y arch/arm/dts/imx6dl-dhcom-pdk2-u-boot.dtsi | 6 +++ arch/arm/dts/imx6q-dhcom-pdk2-u

[PATCH] ARM: imx6: DHCOM i.MX6 PDK: Convert to DM_ETH

2020-04-15 Thread Harald Seiler
Use DM_ETH instead of legacy networking. Signed-off-by: Harald Seiler --- Notes: I am unsure whether I converted 'enough' of the board_eth_init() function to DM. I think the reset GPIO in particular could be handled by the DM driver if I add the following to the device-tree (did

[PATCH 0/2] Rename spl_boot_mode() and spl_boot_partition()

2020-04-15 Thread Harald Seiler
_boot_mode() and spl_mmc_boot_partition() respectively. I will also send a second series to fix the implementation which is affecting me (arch/arm/mach-imx/spl.c). Harald Seiler (2): spl: mmc: Rename spl_boot_mode() to spl_mmc_boot_mode() spl: mmc: Rename spl_boot_partition() to spl_mmc_boot

[PATCH 2/2] spl: mmc: Rename spl_boot_partition() to spl_mmc_boot_partition()

2020-04-15 Thread Harald Seiler
This function is only relevant to the MMC driver so calling it spl_boot_partition() might be confusing. Rename it to spl_mmc_boot_partition() to make its purpose more clear (and bring it in line with spl_mmc_boot_mode()). Signed-off-by: Harald Seiler --- include/spl.h | 14

[PATCH 1/2] spl: mmc: Rename spl_boot_mode() to spl_mmc_boot_mode()

2020-04-15 Thread Harald Seiler
partition). Rename it to spl_mmc_boot_mode() to make it more obvious what this function is about. Link: https://lists.denx.de/pipermail/u-boot/2020-April/405979.html Signed-off-by: Harald Seiler --- include/spl.h | 18 +- common/spl/spl_mmc.c

Re: [PATCH v2] imx: Introduce CONFIG_SPL_FORCE_MMC_BOOT to force MMC boot on falcon mode

2020-04-08 Thread Harald Seiler
Hello Marek, On Wed, 2020-04-08 at 15:45 +0200, Marek Vasut wrote: > On 4/8/20 2:42 PM, Harald Seiler wrote: > > Hello, > > Hi, > > > On Mon, 2019-09-09 at 15:32 +0200, Lukasz Majewski wrote: > > > This change tries to fix the following problem: > >

Re: Basic syntax question

2020-04-08 Thread Harald Seiler
Hello Damien, On Wed, 2020-04-08 at 11:27 +0300, Damien LEFEVRE wrote: > Hi, > > I'm new to u-boot. I'm trying to store the return value of gpio command to > a variable. > > I have the following, (switch released): > Tegra186 (P2771--500) # gpio input FF1 > gpio: pin FF1 (gpio 241) value is

Re: [PATCH v2] imx: Introduce CONFIG_SPL_FORCE_MMC_BOOT to force MMC boot on falcon mode

2020-04-08 Thread Harald Seiler
Hello, On Mon, 2019-09-09 at 15:32 +0200, Lukasz Majewski wrote: > This change tries to fix the following problem: > > - The board boots (to be more precise - ROM loads SPL) from a slow SPI-NOR > memory. > As a result the spl_boot_device() will return SPI-NOR as a boot device > (which is

[PATCH] test/py: mmc: Fix 'mmc info' testcase

2020-03-26 Thread Harald Seiler
Commit 41e30dcf8796 ("cmd: mmc: Make Mode: printout consistent") fixed the layout of `mmc info` output. Reflect this change in the respective testcase. Also fix a typo in the documentation. Fixes: 41e30dcf8796 ("cmd: mmc: Make Mode: printout consistent") Signed-off-by: Hara

Re: [PATCH V3 10/13] net: smc911x: Pass around driver private data

2020-03-25 Thread Harald Seiler
Hi Marek, On Wed, 2020-03-25 at 16:36 +0100, Marek Vasut wrote: > Introduce a private data structure for this driver with embedded > struct eth_device and pass it around. This prepares the driver to > work with both DM and non-DM systems. > > Signed-off-by: Marek Vasut > Cc: Joe Hershberger >

Re: [U-Boot] Sharing a hardware lab

2020-03-23 Thread Harald Seiler
Hello Simon, On Sun, 2020-03-22 at 12:42 -0600, Simon Glass wrote: > Hi Harald, > > On Sun, 22 Mar 2020 at 03:56, Harald Seiler wrote: > > Hi Simon, > > > > On Sat, 2020-03-21 at 13:07 -0600, Simon Glass wrote: [...] > > > > Huh, actually I messed up

Re: [U-Boot] Sharing a hardware lab

2020-03-22 Thread Harald Seiler
Hi Simon, On Sat, 2020-03-21 at 13:07 -0600, Simon Glass wrote: > Hi Harald, > > On Mon, 24 Feb 2020 at 06:27, Harald Seiler wrote: > > Hello Simon, > > > > On Sun, 2020-02-23 at 19:34 -0700, Simon Glass wrote: > > > Hi Heiko, > > > > >

Re: [RFC PATCH 3/9] mksunxi_fit_atf.sh: produce working binaries by default

2020-03-18 Thread Harald Seiler
Hi, On Wed, 2020-03-18 at 10:57 +0100, Petr Štetiar wrote: > At this moment unusable binaries are produced if bl31.bin file is > missing in order to allow passing of various CI tests. This intention of > broken binaries has to be now explicitly confirmed via new > BUILDBOT_BROKEN_BINARIES config

Re: [PATCH 3/3] imx: imx8m: Don't use the addr parameter of reset_cpu

2020-03-04 Thread Harald Seiler
Hello Marek, On Wed, 2020-03-04 at 15:30 +0100, Marek Vasut wrote: > On 3/4/20 3:23 PM, Harald Seiler wrote: > > From: Claudius Heine > > > > imx8m has the only implementation of `reset_cpu` which does not ignore > > the addr parameter and instead gives it some

[PATCH 2/3] imx: imx8m*: Remove do_reset from board files

2020-03-04 Thread Harald Seiler
` calls `reset_cpu` with 0 as the addr parameter while the boards are passing WDOG1_BASE_ADDR. This is ok because the `reset_cpu` implementation uses WDOG1_BASE_ADDR by default if 0 is passed in. Co-Authored-by: Harald Seiler Signed-off-by: Claudius Heine Signed-off-by: Harald Seiler --- board

[PATCH 0/3] ARM: Fix reset in SPL if SYSRESET is not used

2020-03-04 Thread Harald Seiler
Hello, continuing on the discussion around Claudius' patch for fixing reset in SPL [1] we have taken a closer look at the issue. To quickly summarize the situation: The original patch was to enable the generic ARM implementation of `do_reset` if CONFIG_SYSRESET is not enabled in SPL.

[PATCH 3/3] imx: imx8m: Don't use the addr parameter of reset_cpu

2020-03-04 Thread Harald Seiler
. Fixes: d2041725e84b ("imx8m: restrict reset_cpu") Co-Authored-by: Harald Seiler Signed-off-by: Claudius Heine Signed-off-by: Harald Seiler --- arch/arm/mach-imx/imx8m/soc.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/arch/arm/mach-imx/imx8m/soc.c b/arch/ar

[PATCH 1/3] ARM: reset: use do_reset in SPL/TPL if SYSRESET was not enabled for them

2020-03-04 Thread Harald Seiler
From: Claudius Heine In case CONFIG_SYSRESET is set, do_reset from reset.c will not be available anywere, even if SYSRESET is disabled for SPL/TPL. 'do_reset' is called from SPL for instance from the panic handler and PANIC_HANG is not set Signed-off-by: Claudius Heine ---

Re: [U-Boot] Sharing a hardware lab

2020-02-24 Thread Harald Seiler
gt; > > > > > > Is it possible to share this so that others can use the lab when > > > > > > > they > > > > > > > push trees? Is it as simple as adding to the .gitlab-ci.yml file > > > > > > > as I > > >

Re: Boot from different partitions based on some condition

2019-12-12 Thread Harald Seiler
Hello Mats, On Thu, 2019-12-12 at 16:49 +0100, Mats Jansson wrote: > Hi, > > I want to set up for an upgrade of our embedded system, using a > special kernel for the upgrade. The standard will be to load the > system from mtd3 (kernel), mtd4 (devicetree) and mtd5 (rootfs). > > When an update is

Re: gtlab slow?

2019-12-10 Thread Harald Seiler
Hi Simon, On Tue, 2019-12-10 at 05:48 -0700, Simon Glass wrote: > Hi, > > I sometimes find that gitlab.denx.de is very slow. Just now is is > worse than usual - it has taken almost 10 minutes to push a branch and > still not finished. I am getting a 503 error on the web site. > > Any ideas?

Re: [U-Boot] [PATCH 1/4] ARM: dts: dh-imx6: add wdt-reboot node for sysreset driver

2019-11-28 Thread Harald Seiler
Hello Claudius, Fabio, On Thu, 2019-11-28 at 09:49 -0300, Fabio Estevam wrote: > Hi Claudius, > > On Thu, Nov 28, 2019 at 9:07 AM Claudius Heine wrote: > > Signed-off-by: Claudius Heine > > --- > > arch/arm/dts/imx6qdl-dhcom-pdk2.dtsi | 5 + > > 1 file changed, 5 insertions(+) > > > >

Re: [U-Boot] [PATCH 0/4] ARM: imx6: DHCOM i.MX6 PDK: Fixing reset

2019-11-28 Thread Harald Seiler
Hello Claudius, On Thu, 2019-11-28 at 13:06 +0100, Claudius Heine wrote: > Hi, > > currently the reset on the DHCOM i.MX6 board is brocken in u-boot. > > This patchset fixes that by integrating the sysreset and watchdog dm driver. I think you should clarify that reset was broken by commit

[U-Boot] [PATCH v2] spl: imx: Use correct boot-device in spl_boot_mode

2019-11-07 Thread Harald Seiler
uot; error. Supporting this use-case was the original intention for the boot_device parameter, which was introduced in commit 2b1cdafa9fdd ("common: Pass the boot device into spl_boot_mode()"). Fixes: 2b1cdafa9fdd ("common: Pass the boot device into spl_boot_mode()") Signed-off-b

[U-Boot] [PATCH] spl: imx: Use correct boot-device in spl_boot_mode

2019-10-21 Thread Harald Seiler
uot; error. Supporting this use-case was the original intention for the boot_device parameter, which was introduced in commit 2b1cdafa9fdd ("common: Pass the boot device into spl_boot_mode()"). Signed-off-by: Harald Seiler --- arch/arm/mach-imx/spl.c | 2 +- 1 file changed, 1 insertion(+),

Re: [U-Boot] [PATCH 1/3] ARM: imx: Rename imx6_is_bmode_from_gpr9 to imx6_is_bmode_from_gpr10

2019-10-18 Thread Harald Seiler
Hi Claudius, On Wed, 2019-10-16 at 15:27 +0200, Claudius Heine wrote: > The only register used in that function is gpr10, which is used to store > the flag. So naming it after this makes sense. I think the old name had a reason: The _value_ for bmode is stored in GPR9 if this function returns

Re: [U-Boot] GitLab: make pipeline status public

2019-08-08 Thread Harald Seiler
Hi Heinrich, On Wed, 2019-08-07 at 21:26 +0200, Heinrich Schuchardt wrote: > Hello Harald, > > Tom suggested you could help on this issue. > > I suggest that the pipeline status should be public on all custodian > gits, e.g. page > https://gitlab.denx.de/u-boot/custodians/u-boot-efi/pipelines.

Re: [U-Boot] [ANNOUNCEMENT] Switching to gitlab.denx.de

2019-06-24 Thread Harald Seiler
Hello Peng, On Thu, 2019-06-20 at 06:42 +, Peng Fan wrote: > Hi Wolfgang, > [...] > > > > The switch will take place as follows: > > > > 1. We will create user accounts for all custodians at gitlab.denx.de > >You will receive an e-mail notification with your login > >credentials.

Re: [U-Boot] [PATCH] MAINTAINERS: Update git repo links

2019-06-19 Thread Harald Seiler
On Wed, 2019-06-19 at 11:37 +0900, Masahiro Yamada wrote: > On Wed, Jun 19, 2019 at 1:49 AM Wolfgang Denk wrote: > > Dear Bin, > > > > In message <1560872253-6077-1-git-send-email-bmeng...@gmail.com> you wrote: > > > Update all git repo links with the new gitlab ones. > > > > > > Signed-off-by:

  1   2   >