Re: [PATCH 1/1] clk: imx8mn: add pwm clocks

2023-12-11 Thread Fabio Estevam
Hi Nicolas, On Mon, Dec 11, 2023 at 1:52 PM Nicolas Heemeryck wrote: > > Based on Linux kernel 6.7-rc4, add necessary clocks for the PWM > controllers. > > Signed-off-by: Nicolas Heemeryck Reviewed-by: Fabio Estevam

Re: [PATCH] imx: imx-hab: Select SPL_DRIVERS_MISC in the SPL case

2023-12-11 Thread Fabio Estevam
Hi Lisandro, On 11/12/2023 11:45, Lisandro Pérez Meyer wrote: I actually tested this, so, should I reply with Reviewed-By? Feel free to reply with your Tested-by tag. Thanks

[PATCH] imx: imx-hab: Select SPL_DRIVERS_MISC in the SPL case

2023-12-11 Thread Fabio Estevam
From: Fabio Estevam Selecting CONFIG_IMX_HAB=y on a SPL target, such as apalis_imx6_defconfig, for example, leads to the following build error: /usr/bin/arm-linux-gnueabihf-ld.bfd: arch/arm/mach-imx/hab.o: in function `imx_hab_is_enabled': arch/arm/mach-imx/hab.c:879: undefined reference

Re: iMX6, CONFIG_IMX_HAB and fuse_read()

2023-12-09 Thread Fabio Estevam
On Sat, Dec 9, 2023 at 11:12 AM Lisandro Pérez Meyer wrote: > That made it! So the question is: is CONFIG_SPL_DRIVERS_MISC something > that should be pulled in automatically by CONFIG_IMX_HAB or by > something else? Yes, I think we need something like that: diff --git

Re: [PATCH 2/2] ARM: imx: Update DRAM timings with inline ECC on Data Modul i.MX8M Plus eDM SBC

2023-12-09 Thread Fabio Estevam
tting at 3600 MTps. > > Enable inline ECC which is necessary to detect ECC errors and collect > statistics by the EDAC driver in Linux. This reduces the DRAM size by > 64 MiB for each 512 MiB of DRAM, so for a 4 GiB device the available > DRAM size becomes 3.5 GiB . > > Signed-off-by: Marek Vasut Reviewed-by: Fabio Estevam

Re: [PATCH 1/2] ARM: imx: Force DRAM regulators into FPWM mode on Data Modul i.MX8M Plus eDM SBC

2023-12-09 Thread Fabio Estevam
> Force DRAM regulators to forced PWM mode only to stop tempting the DRAM. > > Signed-off-by: Marek Vasut Reviewed-by: Fabio Estevam

Re: iMX6, CONFIG_IMX_HAB and fuse_read()

2023-12-08 Thread Fabio Estevam
On Fri, Dec 8, 2023 at 10:15 PM Lisandro Pérez Meyer wrote: > I added IMX_HAB: > > ``` > $ echo "CONFIG_IMX_HAB=y" >> configs/apalis_imx6_defconfig > ``` Please try like this: --- a/configs/apalis_imx6_defconfig +++ b/configs/apalis_imx6_defconfig @@ -21,7 +21,9 @@ CONFIG_OF_LIBFDT_OVERLAY=y

Re: iMX6, CONFIG_IMX_HAB and fuse_read()

2023-12-08 Thread Fabio Estevam
Hi Lisandro, On Fri, Dec 8, 2023 at 3:43 PM Lisandro Pérez Meyer wrote: > > Hi! > > I am currently not subscribed (already filled the form waiting for > confirmation), so please CC me for the moment. > > I am trying to enable Secure Boot/HAB on u-boot 2022.07. In a previous Please try the top

Re: [PATCH] tools: mxsboot: pre-fill buffer with 0xff, not 0

2023-12-03 Thread Fabio Estevam
outfd) > goto err0; > } > > - memset(buf, 0, size); > + memset(buf, 0xff, size); This makes sense: Reviewed-by: Fabio Estevam Thanks

Re: [PATCH] ARM: imx: Enable CAAM on DH i.MX8M Plus DHCOM

2023-12-01 Thread Fabio Estevam
will help that old kernel version function correctly. > > Signed-off-by: Marek Vasut Reviewed-by: Fabio Estevam

Re: [PATCH] ARM: imx: Enable CAAM on Data Modul i.MX8M Mini/Plus eDM SBC

2023-12-01 Thread Fabio Estevam
will help that old kernel version function correctly. > > Signed-off-by: Marek Vasut Reviewed-by: Fabio Estevam

Re: [PATCH] ddr: imx: Add 3600 MTps rate support

2023-12-01 Thread Fabio Estevam
On Fri, Dec 1, 2023 at 10:49 PM Marek Vasut wrote: > > Add PLL settings for DDR 3600 MTps . This is very similar to 3200 MTps > PLL setting, except the divider is not 9 but 8 . > > Signed-off-by: Marek Vasut Reviewed-by: Fabio Estevam

Re: [PATCH] ddr: imx: Handle 3734 in addition to 3733 and 3732 MTps rates

2023-12-01 Thread Fabio Estevam
On Fri, Dec 1, 2023 at 10:48 PM Marek Vasut wrote: > > The new MX8M DDR tool 3.31 now generates a programming file which uses > data rate 3734 instead of 3733 or 3732 . Handle another rounding option . > > Signed-off-by: Marek Vasut Reviewed-by: Fabio Estevam

Re: [PATHv11 00/43] net/lwip: add lwip library for the network stack

2023-11-30 Thread Fabio Estevam
On Thu, Nov 30, 2023 at 10:14 AM Maxim Uvarov wrote: > I will test that. smsc95xx_eth included in some CI tests and it worked. So > some differences on real hardware... > Log says that a ping packet was sent, but no packet on rx. > Might be the start function of smsc95xx_eth called wrongly

Re: [PATHv11 00/43] net/lwip: add lwip library for the network stack

2023-11-30 Thread Fabio Estevam
Hi Maxim, On Thu, Nov 30, 2023 at 5:39 AM Maxim Uvarov wrote: > If networking for some reason doesn't work on that board (I tested only > virtual board). Please try to compile with: Could you also try on a real board? Tom tested on a Raspberry Pi3 and it failed too:

Re: [PATCH] arm: dts: imx8mp-venice-gw72xx: add TPM device

2023-11-29 Thread Fabio Estevam
On Mon, Nov 27, 2023 at 4:37 PM Tim Harvey wrote: > > Add the TPM device found on the GW72xx revision F PCB. > > This hangs off of SPI2, uses gpio1_10 as a CS and gpio1_11 as RST#. > > Signed-off-by: Tim Harvey Reviewed-by: Fabio Estevam

Re: [PATCH] arm: dts: imx8mm-venice-gw72xx: add TPM device

2023-11-29 Thread Fabio Estevam
On Mon, Nov 27, 2023 at 4:37 PM Tim Harvey wrote: > > Add the TPM device found on the GW72xx revision F PCB. > > This hangs off of SPI2, uses gpio1_10 as a CS and gpio1_11 as RST#. > > Signed-off-by: Tim Harvey Reviewed-by: Fabio Estevam

Re: [PATCH] arm64: imx8mp: Enable TCP, IPv6, wget on Data Modul i.MX8M eDM SBC

2023-11-29 Thread Fabio Estevam
ater is enabled to future proof the IP > stack on this platform. > > Signed-off-by: Marek Vasut Reviewed-by: Fabio Estevam

Re: [PATCH] tqma6: Fix DDR configuration

2023-11-29 Thread Fabio Estevam
html > Link: > https://github.com/tq-systems/u-boot-tqmaxx/commit/15eb6abbefbf6916c28467b85485911dad3da6bc > Signed-off-by: Miquel Raynal Yes, better not to write to reserved fields: Reviewed-by: Fabio Estevam

Re: [PATCH 2/2] board: apalis/colibri imx6/imx6ull/imx7: Add fastboot bootcmd support

2023-11-29 Thread Fabio Estevam
configurations is set, it indicates that the board is in recovery mode > and can use either SDP or Fastboot. > > The default option remains the SDP command, but if > CONFIG_CMD_FASTBOOT is set, it changes to 'fastboot usb 0' as > the boot command. > > Signed-off-by: Hiago De Franco Reviewed-by: Fabio Estevam

Re: [PATCH 1/2] apalis/colibri imx6/imx6ull/imx7: defconfig: Enable fastboot support

2023-11-29 Thread Fabio Estevam
00, to make it consistent with the other boards. > > Signed-off-by: Hiago De Franco Reviewed-by: Fabio Estevam

Re: [PATCH] arm64: imx8mp: Inhibit DTC warning on DH i.MX8MP DHCOM rev.100 DTO

2023-11-29 Thread Fabio Estevam
t; it would also require a full new copy of the PHY node, i.e. > ethernet-phy@5 { ... reg = <5>; ... }, to avoid DTC warnings about > mismatch between node unit and reg value. The node unit in SoM DT is > ethernet-phy@7 { ... }; . > > This simpler approach avoids unnecessary dupl

Re: [PATCH 2/2] arm64: imx8mp: Enable TCP, IPv6, wget on DH i.MX8MP DHCOM

2023-11-29 Thread Fabio Estevam
ater is enabled to future proof the IP > stack on this platform. > > Signed-off-by: Marek Vasut Reviewed-by: Fabio Estevam

Re: [PATCH 1/2] arm: imx6: Enable TCP, IPv6, wget on DH i.MX6 DHCOM

2023-11-29 Thread Fabio Estevam
ater is enabled to future proof the IP > stack on this platform. > > Signed-off-by: Marek Vasut Reviewed-by: Fabio Estevam

Re: [PATCH 0/2] librem5: allow using distroboot w/ extlinux files

2023-11-29 Thread Fabio Estevam
env variable > librem5: Add addresses for compressed kernel load For the series: Reviewed-by: Fabio Estevam Angus, Are you happy with this series?

Re: [PATCH] imx8mn_var_som: Add myself to MAINTAINERS

2023-11-29 Thread Fabio Estevam
On Thu, Oct 19, 2023 at 4:42 PM Hugo Villeneuve wrote: > > From: Hugo Villeneuve > > I would like to help maintaining the imx8mn_var_som symphony board. > > Add myself to MAINTAINERS. > > Signed-off-by: Hugo Villeneuve Reviewed-by: Fabio Estevam

Re: [PATCH v2] arm: mxs: Clear CPSR V bit to activate low vectors

2023-11-29 Thread Fabio Estevam
hould handle > exceptions on its own using its own exception handling code, which is located > at 0x0, i.e. at low vectors. Clear the CPSR V bit, so that the CPU would jump > to low vectors on exception instead, and therefore run the U-Boot exception > handling code. > > Signed-off-by: Ma

Re: [PATCH] arm: dts: imx8mp-venice-gw73xx: add TPM device

2023-11-29 Thread Fabio Estevam
On Wed, Oct 18, 2023 at 3:33 PM Tim Harvey wrote: > > Add the TPM device found on the GW73xx revision F PCB. > > This hangs off of SPI2, uses gpio1_10 as a CS and gpio1_11 as RST#. > > Signed-off-by: Tim Harvey Reviewed-by: Fabio Estevam

Re: [PATCH] arm: dts: imx8mm-venice-gw73xx: add TPM device

2023-11-29 Thread Fabio Estevam
On Wed, Oct 18, 2023 at 3:33 PM Tim Harvey wrote: > > Add the TPM device found on the GW73xx revision F PCB. > > This hangs off of SPI2, uses gpio1_10 as a CS and gpio1_11 as RST#. > > Signed-off-by: Tim Harvey Reviewed-by: Fabio Estevam

Re: [PATCH] board: gateworks: venice: add fixup for GW73xx-F+

2023-11-29 Thread Fabio Estevam
On Wed, Oct 18, 2023 at 3:33 PM Tim Harvey wrote: > > GW73xx-F board revision switched back to the original PCIe switch due to > part availability. > > Signed-off-by: Tim Harvey Reviewed-by: Fabio Estevam

Re: [PATCH] imx8mn-var-som: add manufacturer to target description

2023-11-29 Thread Fabio Estevam
g for a particular > manufacturer/board. > > Signed-off-by: Hugo Villeneuve Reviewed-by: Fabio Estevam

Re: TPM SPI issue

2023-11-29 Thread Fabio Estevam
Hi Niek, On Wed, Nov 29, 2023 at 9:51 AM niek.nooij...@omron.com wrote: > > Hi there > > I have the same issue as here: > https://community.infineon.com/t5/OPTIGA-TPM/I-MX8-TPM-SLB9670-SPI-Com-Error/td-p/472533 > > apparently the CS is released in between responses. Can you help me solve >

Re: [PATHv11 00/43] net/lwip: add lwip library for the network stack

2023-11-28 Thread Fabio Estevam
Hi Tim, On Tue, Nov 28, 2023 at 9:10 PM Tim Harvey wrote: > Fabio, > > You won't see this build issue with the entire series applied... only > with the patches up to and including 'net/lwip: integrate lwIP > library'. My point was there are points in the series where > compilation fails which

Re: [PATHv11 00/43] net/lwip: add lwip library for the network stack

2023-11-28 Thread Fabio Estevam
Hi Tim and Maxim, On Tue, Nov 28, 2023 at 8:05 PM Tim Harvey wrote: > Hi Maxim, > > I've tried this series on my imx8mm-venice boards and find that basic > ping/dhcp/tftp breaks: > u-boot=> net list > eth0 : ethernet@30be 00:d0:12:78:f8:01 active > u-boot=> setenv ipaddr 192.168.1.1 >

Re: [PATHv11 07/43] net/lwip: implement wget cmd

2023-11-28 Thread Fabio Estevam
Hi Maxim, On Tue, Nov 28, 2023 at 4:07 PM Maxim Uvarov wrote: > From this log it's a little bit different wget output. My version prints "#" > for status. So I guess it's original wget. You can spend time on debugging it > but this code will be replaced. > With my code I tried big iso

Re: [PATHv11 07/43] net/lwip: implement wget cmd

2023-11-28 Thread Fabio Estevam
Hi Maxim, On Mon, Nov 27, 2023 at 5:46 PM Maxim Uvarov wrote: > > U-Boot recently got support for an alternative network stack using LWIP. > Replace wget command with the LWIP variant while keeping the output and > error messages identical. > > Signed-off-by: Maxim Uvarov Do you know whether

Re: [PATCH 1/1] pico-imx7d: add baseboard SD card boot detect

2023-11-27 Thread Fabio Estevam
On Mon, Nov 27, 2023 at 4:55 PM Szőke Kálmán Benjamin wrote: > > Unused function parameters should be removed > https://rules.sonarsource.com/c/tag/based-on-misra/RSPEC-1172/ >From this same URL: "Exceptions There are some cases when you want to have an unused parameter (usually because the

Re: [PATCH 1/1] pico-imx7d: add baseboard SD card boot detect

2023-11-26 Thread Fabio Estevam
Hi Benjamin, On Sun, Nov 26, 2023 at 8:17 PM Szőke Kálmán Benjamin wrote: > > It can happen when will -Wall be enabled as in others SW projects. My patches > are final, ready and works. I will not remove this "supression", it must be a > part of it if the target is to provide code in good

Re: [PATCH 1/1] pico-imx7d: add baseboard SD card boot detect

2023-11-19 Thread Fabio Estevam
On Sun, Nov 19, 2023 at 8:47 PM Szőke Kálmán Benjamin wrote: > > It will warn, if warning levels is set correctly. I will do not remove it. How can it happen if U-Boot uses warning-1 := -Wextra -Wunused -Wno-unused-parameter ?

Re: [PATCH 1/1] pico-imx7d: add baseboard SD card boot detect

2023-11-19 Thread Fabio Estevam
Hi Benjamin, [Please do not top-post] On Sun, Nov 19, 2023 at 6:08 PM Szőke Kálmán Benjamin wrote: > (void)(devno); is need because devno arguments is an unused variable, need to > use this solution to suppress any potential warnings. No, devno is a function parameter. There is no warning if

Re: [PATCH v1 2/2] imx: spl_imx_romapi: fix emmc fast boot mode case

2023-11-17 Thread Fabio Estevam
Hi Tom, On Fri, Nov 17, 2023 at 11:25 AM Tom Rini wrote: > OK. Fabio, please send me your patchwork username off-list and I'll get > you added and you can start managing patches there. I've started using > some of the b4 + patchwork integration just recently and it's very nice. In the

Re: [PATCH] arm64: imx: imx8mp-beacon: Add support for 4GB Variant

2023-11-16 Thread Fabio Estevam
> > Signed-off-by: Adam Ford Reviewed-by: Fabio Estevam

Re: inconsistent wget behavior

2023-11-15 Thread Fabio Estevam
Hi Paul, On Wed, Nov 8, 2023 at 10:24 PM Fabio Estevam wrote: > I was not able to see a failure when using "tftp" to transfer the same > 420MB file. > > The number of bytes transferred and md5sum always match, so this is a wget > bug. There was another user who

Re: [PATCH 1/1] pico-imx7d: add baseboard SD card boot detect

2023-11-15 Thread Fabio Estevam
evno) > +{ > + int dev_env = 0; > + (void)(devno); Why? Just drop this ' (void)(devno);' line. I tested it successfully, so: Tested-by: Fabio Estevam

Re: [PATCH v12 9/9] doc: uefi: add HTTP Boot support

2023-11-14 Thread Fabio Estevam
l.com/ Do you see the same problem? Thanks, Fabio Estevam

Re: inconsistent wget behavior

2023-11-08 Thread Fabio Estevam
Hi Tom, On Wed, Nov 8, 2023 at 9:17 PM Tom Rini wrote: > Interesting. What if you take wget out of the equation and use "tftp" > to grab a large file with a known checksum, repeatedly (note that "rhash > -C" will give a crc32 in Linux so you can confirm the correct value and > loop a get,

Re: inconsistent wget behavior

2023-11-08 Thread Fabio Estevam
[Adding Tom and Marek] On Wed, Nov 8, 2023 at 7:22 PM Tim Harvey wrote: > > Greetings, > > I've recently given wget/tcp a try and find the results inconsistent. > > Transferring the same file 15 times in a row will eventually give me > an incomplete transfer with packet/byte count wrong but

[PATCH 2/2] warp7: Convert to watchdog driver model

2023-11-06 Thread Fabio Estevam
From: Fabio Estevam Commit 68dcbdd594d4 ("ARM: imx: Add weak default reset_cpu()") caused the 'reset' command in U-Boot to not cause a board reset. Fix it by switching to the watchdog driver model via sysreset, which is the preferred method for implementing the watchdog reset.

[PATCH 1/2] imx7s-warp-u-boot: Fix aliases and chosen nodes indentation

2023-11-06 Thread Fabio Estevam
From: Fabio Estevam The aliases and chosen nodes are currently indented using spaces. Fix them to use the standard tab indentation. Signed-off-by: Fabio Estevam --- arch/arm/dts/imx7s-warp-u-boot.dtsi | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/arch/arm

[PATCH v2 2/2] pico-pi-imx6ul: Convert to watchdog driver model

2023-11-04 Thread Fabio Estevam
From: Fabio Estevam Commit 68dcbdd594d4 ("ARM: imx: Add weak default reset_cpu()") caused the 'reset' command in U-Boot to not cause a board reset. Fix it by switching to the watchdog driver model via sysreset, which is the preferred method for implementing the watchdog reset.

[PATCH v2 1/2] pico-pi-imx6ul: Connvert to DM_SERIAL

2023-11-04 Thread Fabio Estevam
From: Fabio Estevam The conversion to DM_SERIAL is mandatory, so select this option. Signed-off-by: Fabio Estevam --- Changes since v1: - None configs/pico-pi-imx6ul_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/configs/pico-pi-imx6ul_defconfig b/configs/pico-pi

[PATCH 2/2] pico-pi-imx6ul: Convert to watchdog driver model

2023-11-04 Thread Fabio Estevam
From: Fabio Estevam Commit 68dcbdd594d4 ("ARM: imx: Add weak default reset_cpu()") caused the 'reset' command in U-Boot to not cause a board reset. Fix it by switching to the watchdog driver model via sysreset, which is the preferred method for implementing the watchdog reset.

[PATCH 1/2] pico-pi-imx6ul: Connvert to DM_SERIAL

2023-11-04 Thread Fabio Estevam
From: Fabio Estevam The conversion to DM_SERIAL is mandatory, so select this option. Signed-off-by: Fabio Estevam --- configs/pico-pi-imx6ul_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/configs/pico-pi-imx6ul_defconfig b/configs/pico-pi-imx6ul_defconfig index 1eee1e6c2ff5

Re: [PATCH] pico-imx7d: Remove board_phy_config()

2023-11-04 Thread Fabio Estevam
On Sat, Nov 4, 2023 at 7:45 AM Szőke Kálmán Benjamin wrote: > > Other pico SoMs still has board_phy_config() also, you should remove it from > them, too. board_phy_config() could probably be removed from these boards too. To be sure, someone has to test it. If there is a problem with the

[PATCH] imx7d-pico-pi-u-boot.dtsi: Fix aliases indentation

2023-11-03 Thread Fabio Estevam
From: Fabio Estevam The aliases are currently indented using spaces. Fix it to use the standard tab indentation. Signed-off-by: Fabio Estevam --- arch/arm/dts/imx7d-pico-pi-u-boot.dtsi | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/arch/arm/dts/imx7d-pico-pi-u

Re: [PATCH 1/1] pico-imx7d: fix typo in AR8035 comment

2023-11-03 Thread Fabio Estevam
Hi Benjamin, On Fri, Nov 3, 2023 at 6:16 PM wrote: > > From: Benjamin Szőke > > Signed-off-by: Benjamin Szőke > --- > board/technexion/pico-imx7d/pico-imx7d.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/board/technexion/pico-imx7d/pico-imx7d.c >

[PATCH] pico-imx7d: Remove board_phy_config()

2023-11-03 Thread Fabio Estevam
From: Fabio Estevam With Ethernet DM in place, there is no longer the need for having the board_phy_config() anymore. Remove it. Tested on imx7d-pico-pi board: => setenv autoload no => dhcp BOOTP broadcast 1 *** Unhandled DHCP Option in OFFER/ACK: 42 DHCP client bound to address 192.168

Re: [PATCH 1/1] pico-imx7d: fix typo in AR8035 comment

2023-11-03 Thread Fabio Estevam
On Fri, Nov 3, 2023 at 6:59 PM Szőke Kálmán Benjamin wrote: > > You said it should be a separated patch, here it is. Totaly usless to make > big commit descripton for a type fix. Merge it!! Any reason why you dropped the U-Boot list? Yes, I said it should be a separate patch from the original

Re: [PATCH 1/1] pico-imx7d: fix typo in AR8035 comment

2023-11-03 Thread Fabio Estevam
On Fri, Nov 3, 2023 at 6:16 PM wrote: > > From: Benjamin Szőke > > Signed-off-by: Benjamin Szőke Please make sure to run checkpatch and make it happy: ./scripts/checkpatch.pl 0001-pico-imx7d-fix-typo-in-AR8035-comment.patch WARNING: Missing commit description - Add an appropriate one total:

Re: [PATCH 1/1] [u-boot][master][PATCH v6] pico-imx7d: add baseboard SD card boot detect

2023-11-03 Thread Fabio Estevam
On Thu, Nov 2, 2023 at 5:45 PM wrote: > > From: Benjamin Szőke > > Technexion PICO-IMX7 SoM is supporting USDHC3 (eMMC or micro SD on SoM) > and USDHC1 (SD on carrier board) to use on any carrier board like > PICO-NYMPH. Based on the U-Boot version from Technexion it adds > baseboard SD card

Re: [PATCH 1/1] [u-boot][master][PATCH v6] pico-imx7d: add baseboard SD card boot detect

2023-11-03 Thread Fabio Estevam
In the Subject: simply remove the [PATCH 1/1] [u-boot][master] text as it is not useful. On Thu, Nov 2, 2023 at 5:45 PM wrote: > -aliases { > -mmc0 = > -usb0 = > -display0 = > -}; > + aliases { > + mmc0 = > + mmc1 = > +

Re: [PATCH v4] cmd: mmc: Add mmc reg read command for reading card registers

2023-10-31 Thread Fabio Estevam
ble. > > Tested-by: Jaehoon Chung > Reviewed-by: Jaehoon Chung > Signed-off-by: Marek Vasut Reviewed-by: Fabio Estevam

Re: [PATCH v3] cmd: mmc: Add mmc reg read command for reading card registers

2023-10-31 Thread Fabio Estevam
Hi Marek, On Tue, Oct 31, 2023 at 7:16 AM Marek Vasut wrote: > +config CMD_MMC_REG > + bool "Enable support for reading card registers in the mmc command" > + depends on CMD_MMC > + default n "default n" could be dropped, since it is already the default.

[PATCH] pico-pi-imx7d: Convert to watchdog driver model

2023-10-26 Thread Fabio Estevam
From: Fabio Estevam Commit 68dcbdd594d4 ("ARM: imx: Add weak default reset_cpu()") caused the 'reset' command in U-Boot to not cause a board reset. Fix it by switching to the watchdog driver model via sysreset, which is the preferred method for implementing the watchdog reset

Re: [PATCH] MAINTAINERS: Remove non-working address from MAINTAINERS

2023-10-26 Thread Fabio Estevam
Hi Hugo, On Thu, Oct 26, 2023 at 11:38 AM Hugo Villeneuve wrote: > Hi Fabio, > since the change concerns two different boards, that is why I didn't > indicate it. If required, I could send it as two separate patches. Ok, got it. Let's keep it as is then. Thanks

Re: [PATCH v2 1/4] net: dwc_eth_qos: add i.MX93 support

2023-10-26 Thread Fabio Estevam
On Tue, Oct 17, 2023 at 6:45 AM Sébastien Szymanski wrote: > > Add support for DWC EQoS MAC on i.MX93. > > Signed-off-by: Sébastien Szymanski For the series: Reviewed-by: Fabio Estevam

Re: [PATCH] arm: dts: imx8mn-var-som: Fix broken EEPROM read

2023-10-26 Thread Fabio Estevam
gt; also be read outside of SPL. > > Fixes: 9e644284ab81 ("dm: core: Report bootph-pre-ram/sram node as pre-reloc > after relocation") > Signed-off-by: Hugo Villeneuve Reviewed-by: Fabio Estevam

Re: [PATCH] MAINTAINERS: Remove non-working address from MAINTAINERS

2023-10-26 Thread Fabio Estevam
On Mon, Oct 23, 2023 at 1:45 PM Hugo Villeneuve wrote: > > From: Hugo Villeneuve The Subject does indicate which board it refers to. It would be better to have it like: imx8mn_smm_s2: Remove non-working address from MAINTAINERS Reviewed-by: Fabio Estevam

Re: [PATCH v1] apalis-imx8: add USBH_EN gpio hog

2023-10-26 Thread Fabio Estevam
own > resistor value. > > Also, update apalis-imx8_defconfig via savedefconfig. > > Signed-off-by: Andrejs Cainikovs Reviewed-by: Fabio Estevam

Re: [PATCH] imx8mn-var-som: Simplify FEC initialization

2023-10-26 Thread Fabio Estevam
has the EC configuration option > (onboard ethernet PHY), it also has a dedicated crystal/oscillator. > So in all cases, there is no need to enable the RGMII TX clk output. > > This follows a similar change introduced by Fabio Estevam: > Link: > https://lore.kernel.org/u-boot/2

Re: [PATCH 1/5] configs: imx8mp_beacon: Do not set SYS_CONSOLE_IS_IN_ENV

2023-10-26 Thread Fabio Estevam
On Wed, Oct 25, 2023 at 8:05 PM Adam Ford wrote: > > The hardware only supports a specific console port, so remove the > option to change the console location in the environment. > > Signed-off-by: Adam Ford For the series: Reviewed-by: Fabio Estevam

Re: [PATCH v1 2/2] imx: spl_imx_romapi: fix emmc fast boot mode case

2023-10-26 Thread Fabio Estevam
to the ss->pagesize. Fix this by recording the actual bytes downloaded. > > Fixes: 4b4472438f5a ("imx: spl_imx_romapi: avoid tricky use of > spl_load_simple_fit() to get full FIT size") > Signed-off-by: Marcel Ziswiler Thanks for testing mainline U-Boot and fixing it: Reviewed-by: Fabio Estevam

Re: [PATCH v1 1/2] imx: spl_imx_romapi: fix comment about stream(usb) download failure

2023-10-26 Thread Fabio Estevam
On Thu, Oct 26, 2023 at 4:32 AM Marcel Ziswiler wrote: > > From: Marcel Ziswiler > > Fix comment about Stream(USB) download failure. > > Fixes: 1cbebc786276 ("imx: add rom api support") > Signed-off-by: Marcel Ziswiler Reviewed-by: Fabio Estevam

Re: [PATCH 1/1] [u-boot][master][PATCH v5] pico-imx7d: add baseboard SD card boot detect

2023-10-26 Thread Fabio Estevam
On Sat, Oct 21, 2023 at 9:40 AM wrote: > > From: Benjamin Szőke > > Technexion PICO-IMX7 SoM is supporting USDHC3 (eMMC or micro SD on SoM) > and USDHC1 (SD on carrier board) to use on any carrier board like > PICO-NYMPH. This pacth is intend to take over codes from Technexion Typo in "pacth is

Re: [PATCH 1/1] [u-boot][master][PATCH v5] pico-imx7d: add baseboard SD card boot detect

2023-10-26 Thread Fabio Estevam
On Sat, Oct 21, 2023 at 9:40 AM wrote: > > From: Benjamin Szőke > > Technexion PICO-IMX7 SoM is supporting USDHC3 (eMMC or micro SD on SoM) > and USDHC1 (SD on carrier board) to use on any carrier board like > PICO-NYMPH. This pacth is intend to take over codes from Technexion > to support mmc

[PATCH] imx8mp_evk: Select CONFIG_NET_RANDOM_ETHADDR

2023-10-26 Thread Fabio Estevam
From: Fabio Estevam On an early revision of the imx8mp-evk that I have access to, the MAC addresses fuses are not programmed, causing failure to bring the Ethernet interfaces. Fix this problema by selecting CONFIG_NET_RANDOM_ETHADDR so that random MAC addresses are assigned and the Ethernet

Re: [REGRESSION] imx: spl_imx_romapi: boot loops

2023-10-25 Thread Fabio Estevam
Hi Marcel, On Tue, Oct 24, 2023 at 10:16 AM Marcel Ziswiler wrote: > Yes, regular eMMC boot. Serial download also works for me (note this requires > a patch still not merged upstream I tried the top of tree U-Boot on the imx8mp-evk and I was not able to reproduce the error here. Below is the

[PATCH v2] net: eth-uclass: Improve error message when MAC is not found

2023-10-20 Thread Fabio Estevam
From: Fabio Estevam While bringinp up a new board without the MAC fuses programmed, the following error message was observed: Error: ethernet@30bf address not set. Improve the error message to make it clearer the reason of the failure. Signed-off-by: Fabio Estevam --- Changes since v1

Re: Branches in uboot-imx

2023-10-19 Thread Fabio Estevam
Hi Grzegorz, On Thu, Oct 19, 2023 at 11:58 AM Grzegorz Borowiak wrote: > > Hello! > > I just cloned the repository: > > https://github.com/nxp-imx/uboot-imx This is a U-Boot fork maintained by NXP. Only NXP can help you with this one. > and I encountered a rather large amount of branches

[PATCH] clock_imx8mm: Add a stub for imx8mp_fec_interface_init()

2023-10-19 Thread Fabio Estevam
From: Fabio Estevam When CONFIG_DWC_ETH_QOS_IMX=y and CONFIG_FEC_MXC is not selected, the following warning is seen: arch/arm/mach-imx/imx8m/clock_imx8mm.c: In function 'board_interface_eth_init': arch/arm/mach-imx/imx8m/clock_imx8mm.c:914:24: warning: implicit declaration of function

[PATCH] net: eth-uclass: Improve error message when MAC is not found

2023-10-19 Thread Fabio Estevam
From: Fabio Estevam While bringinp up a new board without the MAC fuses programmed, the following error message was observed: Error: ethernet@30bf address not set. Improve the error message to make it clearer the reason of the failure. Signed-off-by: Fabio Estevam --- net/eth-uclass.c

[PATCH v3 2/2] imx8mp_evk: Remove unneeded header files

2023-10-19 Thread Fabio Estevam
From: Fabio Estevam Cleanup the file by removing unneeded header files. Signed-off-by: Fabio Estevam --- Changes since v2: - Also remove and DECLARE_GLOBAL_DATA_PTR. (Hugo) board/freescale/imx8mp_evk/imx8mp_evk.c | 13 - 1 file changed, 13 deletions(-) diff --git a/board

[PATCH v3 1/2] imx8mp_evk: Simplify Ethernet initialization

2023-10-19 Thread Fabio Estevam
From: Fabio Estevam With DM enabled, there is no need for board code to initialize the Ethernet interfaces. The RTL8211FDI Ethernet PHYs have 25MHz oscillator, so there is no need to enable the RGMII TX clk output. Also, there is no need for describing the deprecated phy-reset FEC properties

[PATCH v2 2/2] mx8mp_evk: Remove unneeded header files

2023-10-19 Thread Fabio Estevam
From: Fabio Estevam Cleanup the file by removing unneeded header files. Signed-off-by: Fabio Estevam --- Changes since v1: - Newly introduced. (Hugo) board/freescale/imx8mp_evk/imx8mp_evk.c | 10 -- 1 file changed, 10 deletions(-) diff --git a/board/freescale/imx8mp_evk/imx8mp_evk.c

[PATCH v2 1/2] imx8mp_evk: Simplify Ethernet initialization

2023-10-19 Thread Fabio Estevam
From: Fabio Estevam With DM enabled, there is no need for board code to initialize the Ethernet interfaces. The RTL8211FDI Ethernet PHYs have 25MHz oscillator, so there is no need to enable the RGMII TX clk output. Also, there is no need for describing the deprecated phy-reset FEC properties

[PATCH] imx8mp_evk: Simplify FEC initialization

2023-10-19 Thread Fabio Estevam
From: Fabio Estevam With DM enabled, there is no need for board code to initialize the FEC interface. The RTL8211FDI Ethernet PHYs have 25MHz oscillator, so there is no need to enable the RGMII TX clk output. Also, there is no need for describing the deprecated phy-reset properties in u

[PATCH] phy: phy-imx8mq-usb: Remove .exit operation

2023-10-19 Thread Fabio Estevam
From: Fabio Estevam Currently, when running "ums 0 mmc 2" and breaking it via CTRL + C, the following message is seen: u-boot=> ums 0 mmc 1 UMS: LUN 0, dev mmc 1, hwpart 0, sector 0x0, count 0x1dacc00 CTRL+C - Operation aborted clk usb_phy_root_clk already disabled The

[PATCH v2 3/3] arm: dts: imx8mp: Sync with linux-next 20231019

2023-10-19 Thread Fabio Estevam
From: Fabio Estevam Sync imx8mp.dtsi and imx8mp-clock.h with linux-next 20231019. Signed-off-by: Fabio Estevam --- Changes since v1: - Rebased against latest U-Boot and synced with linux-next 20231019 arch/arm/dts/imx8mp.dtsi | 542 ++- include/dt-bindings

[PATCH v2 2/3] arm: dts: imx8mn: Sync with linux-next 20231019

2023-10-19 Thread Fabio Estevam
From: Fabio Estevam Sync imx8mn.dtsi with linux-next 20231019. Signed-off-by: Fabio Estevam --- Changes since v1: - Rebased against latest U-Boot and synced with linux-next 20231019 arch/arm/dts/imx8mn.dtsi | 167 --- 1 file changed, 154 insertions(+), 13

[PATCH v2 1/3] arm: dts: imx8mm: Sync with linux-next 20231019

2023-10-19 Thread Fabio Estevam
From: Fabio Estevam Sync imx8mm.dtsi with linux-next 20231019. The motivation for doing this sync was a bug when doing "ums 0 mmc 1" on imx8mm-evk. It worked well for the first time, but after doing a CTRL+C and launching the ums again, the command did not work. Adam Ford suggest

[PATCH 2/2] imx8mp_evk: Add myself to MAINTAINERS

2023-10-18 Thread Fabio Estevam
From: Fabio Estevam I would like to help maintaining the imx8mp_evk board. Add myself to MAINTAINERS. Signed-off-by: Fabio Estevam --- board/freescale/imx8mp_evk/MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/board/freescale/imx8mp_evk/MAINTAINERS b/board/freescale

[PATCH 1/2] imx8mp_evk: Convert to DM_PMIC

2023-10-18 Thread Fabio Estevam
From: Fabio Estevam Currently, the imx8mp_evk uses the non-DM code to initialize the PMIC. Convert to DM_PMIC, which is the recommended way to access the PMIC. While at it, fix multi-line comments style. Signed-off-by: Fabio Estevam --- arch/arm/dts/imx8mp-evk-u-boot.dtsi | 18

[PATCH v4] imx8mp-dhcom-pdk3-overlay-rev100: Pass #address-cells/size-cells

2023-10-18 Thread Fabio Estevam
From: Fabio Estevam Pass #address-cells/size-cells in the mdio node to avoid the following DTC warning: DTCOarch/arm/dts/imx8mp-dhcom-pdk3-overlay-rev100.dtbo Warning (reg_format): /fragment@0/__overlay__:reg: property has invalid length (4 bytes) (#address-cells == 2, #size-cells == 1

[PATCH v3] imx8mp-dhcom-pdk3-overlay-rev100: Pass #address-cells/size-cells

2023-10-18 Thread Fabio Estevam
From: Fabio Estevam Pass #address-cells/size-cells in the mdio node to avoid the following DTC warning: DTCOarch/arm/dts/imx8mp-dhcom-pdk3-overlay-rev100.dtbo Warning (reg_format): /fragment@0/__overlay__:reg: property has invalid length (4 bytes) (#address-cells == 2, #size-cells == 1

[PATCH v2] imx8mp-dhcom-pdk3-overlay-rev100: Pass #address-cells/size-cells

2023-10-18 Thread Fabio Estevam
From: Fabio Estevam Pass #address-cells/size-cells in the mdio node to avoid the following DTC warning: DTCOarch/arm/dts/imx8mp-dhcom-pdk3-overlay-rev100.dtbo Warning (reg_format): /fragment@0/__overlay__:reg: property has invalid length (4 bytes) (#address-cells == 2, #size-cells == 1

[PATCH] imx8mp-dhcom-pdk3-overlay-rev100: Pass #address-cells/size-cells

2023-10-18 Thread Fabio Estevam
From: Fabio Estevam Pass #address-cells/size-cells in the mdio node to avoid the following DTC warning: DTCOarch/arm/dts/imx8mp-dhcom-pdk3-overlay-rev100.dtbo Warning (reg_format): /fragment@0/__overlay__:reg: property has invalid length (4 bytes) (#address-cells == 2, #size-cells == 1

Re: failed to build u-boot mt7621_rfb_defconfig

2023-10-17 Thread Fabio Estevam
[Please keep the list on Cc] On Mon, Oct 16, 2023 at 11:14 PM Xiaobo Liu wrote: > > Hi > > I found the nand rfb u-boot mt7621_stage_sram.bin, no spi-nor version was > found. > > https://u-boot.readthedocs.io/en/latest/board/mediatek/mt7621.html > > Where can I find a version of the mt7621

Re: failed to build u-boot mt7621_rfb_defconfig

2023-10-16 Thread Fabio Estevam
Hi Xiaobo, On Mon, Oct 16, 2023 at 8:49 AM Xiaobo Liu wrote: > Image 'u-boot-spl-ddr' is missing external blobs and is non-functional: > stage_bin > > /binman/u-boot-spl-ddr/stage_bin (mt7621_stage_sram.bin): >Missing blob > > Some images are invalid > make: *** [Makefile:1115:

[PATCH v4] imx8mp-evk: Add USB0 OTG support

2023-10-15 Thread Fabio Estevam
From: Fabio Estevam Add USB0 OTG support. Currently, the USB0 OTG nodes are not enabled in the Linux kernel devicetree. For this reason, enable the USB0 OTG nodes inside imx8mp-evk-u-boot.dtsi for now. Also select several useful options such as USB gadget and fastboot. Tested by running &quo

[PATCH v3] imx8mp-evk: Add USB0 OTG support

2023-10-15 Thread Fabio Estevam
From: Fabio Estevam Add USB0 OTG support. Currently, the USB0 OTG nodes are not enabled in the Linux kernel devicetree. For this reason, enable the USB0 OTG nodes inside imx8mp-evk-u-boot.dtsi for now. Also select several useful options such as USB gadget and fastboot. Tested by running &quo

<    1   2   3   4   5   6   7   8   9   10   >