Re: [PATCH] spl: spl_legacy: Fix spl_end address

2023-06-30 Thread Fabio Estevam
On Fri, Jun 30, 2023 at 3:26 PM Fabio Estevam wrote: > + if (IS_ENABLED(CONFIG_SPL_SEPARATE_BSS)) > + spl_end = (uintptr_t)_image_binary_end; > + else > + spl_end = (uintptr_t)__bss_end; > + I tested this with CONFIG_SPL_SEPARATE_BSS=

Re: [PATCH] spl: spl_legacy: Fix spl_end address

2023-06-30 Thread Fabio Estevam
On Fri, Jun 30, 2023 at 11:23 AM Fabio Estevam wrote: > diff --git a/common/spl/spl_legacy.c b/common/spl/spl_legacy.c > index d34bc5492e8d..095443c63d8d 100644 > --- a/common/spl/spl_legacy.c > +++ b/common/spl/spl_legacy.c > @@ -19,7 +19,7 @@ > static void spl_parse_legacy_

[PATCH] spl: spl_legacy: Fix spl_end address

2023-06-30 Thread Fabio Estevam
From: Fabio Estevam Currently, spl_end points to the __bss_end address, which is an external RAM address instead of the end of the SPL text section in the internal RAM. This causes boot failures on imx6-colibri, for example: ``` Trying to boot from MMC1 SPL: Image overlaps SPL resetting

Re: v2023.07-rc5 regression: Image overlaps SPL

2023-06-30 Thread Fabio Estevam
On Fri, Jun 30, 2023 at 11:05 AM Tom Rini wrote: > I think this breaks x86, without updating their linker scripts at least. What about this instead? --- a/common/spl/spl_legacy.c +++ b/common/spl/spl_legacy.c @@ -19,7 +19,7 @@ static void spl_parse_legacy_validate(uintptr_t start, uintptr_t

Re: v2023.07-rc5 regression: Image overlaps SPL

2023-06-30 Thread Fabio Estevam
On Fri, Jun 30, 2023 at 10:22 AM Fabio Estevam wrote: > Should we fix spl_end like this? Looking at u-boot-spl.map: 0x009122640x0 common/spl/spl.o 0x00912264. = ALIGN (0x4) 0x00912

Re: v2023.07-rc5 regression: Image overlaps SPL

2023-06-30 Thread Fabio Estevam
Hi Francesco and Marek, On Thu, Jun 29, 2023 at 11:19 AM Francesco Dolcini wrote: > > Hello Marek, > as briefly discussed off-list it looks like > commit 77aed22b48ab ("spl: spl_legacy: Add extra address checks") introduces a > regression on some board/arch, at least colibri and apalis imx6

[PATCH] doc: imx: habv4: Fix typo in 'signing'

2023-06-29 Thread Fabio Estevam
From: Fabio Estevam Fix two occurrences where 'signing' is misspelled. Signed-off-by: Fabio Estevam --- doc/imx/habv4/guides/mx6_mx7_secure_boot.txt | 2 +- doc/imx/habv4/guides/mx6_mx7_spl_secure_boot.txt | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/doc/imx

[PATCH] smegw01: Fix wrong symbol override

2023-06-27 Thread Fabio Estevam
From: Eduard Strehlau board_mmc_get_env_part() is not called as the default implementation of mmc_get_env_part() is used. Fix this problem by directly calling mmc_get_env_part() instead. Signed-off-by: Eduard Strehlau Signed-off-by: Fabio Estevam --- Tom, Stefano is out this week. Could you

[PATCH] smegw01: Fix duplicate bootcmd

2023-06-26 Thread Fabio Estevam
From: Eduard Strehlau Two conflicting bootcmds were included in the environment. Streamline to defining the bootcmd only in the env file. Signed-off-by: Eduard Strehlau Signed-off-by: Fabio Estevam --- Tom, Stefano is out this week. Could you please apply this one directly so that it gets

Re: [PATCH] board: freescale: imx93_evk: Fix MMC environment offset boot conflict.

2023-06-26 Thread Fabio Estevam
he content of the environment data. This will then hang > the system. > > To fix this, move the environment from 0x40 to 0x70 reserving up > to 3 MB at 0x40 for any actual secondary user image container. > > Signed-off-by: Ken Sloat Reviewed-by: Fabio Estevam

Re: Trying to boot custom kernel on Wink Hub (i.MX28)

2023-06-24 Thread Fabio Estevam
On Fri, Jun 23, 2023 at 11:41 AM Fabio Estevam wrote: > > [0.00] Booting Linux on physical CPU 0x0 > > [0.00] Linux version 6.4.0-rc6-00242-g1639fae5132b (rogan@vermin) > > (arm-linux-gnueabi-gcc (Debian 10.2.1-6) 10.2.1 20210110, GNU ld (GNU > > Binutil

Re: Trying to boot custom kernel on Wink Hub (i.MX28)

2023-06-23 Thread Fabio Estevam
Hi Rogan, On Fri, Jun 23, 2023 at 11:25 AM Rogan Dawes wrote: > > I understand the importance now. No, I got to the same result with your dts > as with the evk one, crash after the usbcore modules were registered, and > then binary garbage on the console. On the dts I shared, the USB nodes

Re: Trying to boot custom kernel on Wink Hub (i.MX28)

2023-06-23 Thread Fabio Estevam
On Fri, Jun 23, 2023 at 11:35 AM Rogan Dawes wrote: > > Kernel output up until the crash: > > Starting kernel ... > > [0.00] Booting Linux on physical CPU 0x0 > [0.00] Linux version 6.4.0-rc6-00242-g1639fae5132b (rogan@vermin) > (arm-linux-gnueabi-gcc (Debian 10.2.1-6) 10.2.1

Re: Trying to boot custom kernel on Wink Hub (i.MX28)

2023-06-23 Thread Fabio Estevam
Hi Rogan, On Fri, Jun 23, 2023 at 10:53 AM Rogan Dawes wrote: > > Hi Fabio, > > Thanks so much for the effort put in. I just wanted to confirm that I am able > to get console output when using the imx28-evk.dts file, although it crashes > after a few lines of output. That seems to suggest that

Re: Trying to boot custom kernel on Wink Hub (i.MX28)

2023-06-23 Thread Fabio Estevam
for imx28-wink-hub-v1 and should allow you to boot until it searches for the rootfs. I am glad you managed to start seeing kernel console using a mainline kernel :-) From b5554f5f53076985795a716a5208f58c1242de2d Mon Sep 17 00:00:00 2001 From: Fabio Estevam Date: Fri, 23 Jun 2023 09:11:49 -030

Re: Trying to boot custom kernel on Wink Hub (i.MX28)

2023-06-22 Thread Fabio Estevam
Hi Rogan, On Thu, Jun 22, 2023 at 10:03 PM Fabio Estevam wrote: > > Hi Rogan, > > On Thu, Jun 22, 2023 at 7:43 PM Fabio Estevam wrote: > > > You should also pass the kernel command line: console=ttyAMA0,115200n8 > > > > As far as I recall, the 2.6.35 kerne

Re: Trying to boot custom kernel on Wink Hub (i.MX28)

2023-06-22 Thread Fabio Estevam
Hi Rogan, On Thu, Jun 22, 2023 at 7:43 PM Fabio Estevam wrote: > You should also pass the kernel command line: console=ttyAMA0,115200n8 > > As far as I recall, the 2.6.35 kernel used a different console device. Yes, 2.6.35 uses ttyAM0. Mainline uses ttyAMA0 (note the extra A). Plea

Re: Trying to boot custom kernel on Wink Hub (i.MX28)

2023-06-22 Thread Fabio Estevam
Hi Rogan, On Thu, Jun 22, 2023 at 7:09 PM Rogan Dawes wrote: > > I tried to follow your suggestion, but so far I have been unsuccessful. > > I cloned Linus' latest source code, and then did: > > make ARCH=arm mxs_defconfig > make ARCH=arm CROSS_COMPILE=arm-linux-gnueabi- > cat

Re: [PATCH] arm: dts: imx8m: move CAAM nodes into common u-boot.dtsi

2023-06-22 Thread Fabio Estevam
ed-off-by: Tim Harvey Reviewed-by: Fabio Estevam

Re: [PATCH 3/3] arm: dts: imx8mn: protect the firmware/optee node with ifdef

2023-06-20 Thread Fabio Estevam
On Tue, Jun 20, 2023 at 12:19 PM Tim Harvey wrote: > > There is no need to include the firmware/optee node if the optee > driver is not enabled. > > Signed-off-by: Tim Harvey Reviewed-by: Fabio Estevam

Re: [PATCH 2/3] arm: dts: imx8mp: move firmware/optee node to common imx8mp-u-boot.dtsi

2023-06-20 Thread Fabio Estevam
On Tue, Jun 20, 2023 at 12:19 PM Tim Harvey wrote: > > Move the firmware/optee node to the common imx8mp-u-boot.dtsi and > protect it with an ifdef CONFIG_OPTEE as it is a meaningless node > without the optee driver enabled. > > Signed-off-by: Tim Harvey Reviewed-by: Fabio Estevam

Re: [PATCH 1/3] arm: dts: imx8mm: move firmware/optee node to common imx8mm-u-boot.dtsi

2023-06-20 Thread Fabio Estevam
On Tue, Jun 20, 2023 at 12:19 PM Tim Harvey wrote: > > Move the firmware/optee node to the common imx8mm-u-boot.dtsi and > protect it with an ifdef CONFIG_OPTEE as it is a meaningless node > without the optee driver enabled. > > Signed-off-by: Tim Harvey Reviewed-by: Fabio Estevam

Re: [PATCH] spl: spl_legacy: Add extra address checks

2023-06-19 Thread Fabio Estevam
On Mon, Jun 19, 2023 at 7:41 PM Marek Vasut wrote: > > Check whether the loaded image or entry point does not overlap SPL. > > Signed-off-by: Marek Vasut Reviewed-by: Fabio Estevam

Re: [PATCH 2/2] imx: hab: Simplify the mechanism

2023-06-19 Thread Fabio Estevam
the signing > scripting far simpler. > > Signed-off-by: Marek Vasut Reviewed-by: Fabio Estevam

Re: [PATCH 1/2] imx: hab: Fix a couple of build warnings with DEBUG enabled

2023-06-19 Thread Fabio Estevam
On Mon, Jun 19, 2023 at 7:45 PM Marek Vasut wrote: > > In case the DEBUG is enabled, these three lines warn about cast of > pointer to integer of different size, add the missing casts to fix > the warnings. > > Signed-off-by: Marek Vasut Reviewed-by: Fabio Estevam

Re: Trying to boot custom kernel on Wink Hub (i.MX28)

2023-06-15 Thread Fabio Estevam
dtb by default and then you don't need the steps above. Take a look at board/freescale/mx28eevk support for a reference. Regards, Fabio Estevam

Re: [PATCH] arm: mx6: module_fuse: fix build failure due to wrong argument name

2023-06-15 Thread Fabio Estevam
On Thu, Apr 6, 2023 at 1:17 PM Giulio Benetti wrote: > > nodeoff variable should be variable off returned by fdt_path_offset() so > let's rename it to off. > > Signed-off-by: Giulio Benetti Reviewed-by: Fabio Estevam

Re: [PATCH] mx8m: csf.sh: use vars for keys to avoid file edits when signing

2023-06-12 Thread Fabio Estevam
gt; to these which can be expended from env variables in the csf.sh script. > > The following vars are used: > SKK_TABLE - full path to SRK_1_2_3_4_table.bin Typo: SKK_TABLE --> SRK_TABLE Reviewed-by: Fabio Estevam

Re: [PATCH 3/3] arm: dts: imx8mp: Sync with Linux 6.3

2023-05-30 Thread Fabio Estevam
On Tue, May 30, 2023 at 7:35 PM Adam Ford wrote: > I have it working now. I need some time to clean my stuff and re-base > the imx8mp.dtsi file, but I can submit a patch which fixes the clocks > and re-sync's the device tree with the current stuff from kernel.org. > I should be able to get a

Re: [PATCH] arm: dts: imx8mp-beacon-kit: Enable USB Power domains

2023-05-30 Thread Fabio Estevam
On Tue, May 30, 2023 at 7:49 PM Adam Ford wrote: > > The USB Power domains should not have been removed as it causes > the board to hang if the USB is started. > > Signed-off-by: Adam Ford Reviewed-by: Fabio Estevam

Re: [PATCH 2/2] arm: dts: imx8mp: Sync the DT with kernel 6.4-rc4

2023-05-30 Thread Fabio Estevam
On Tue, May 30, 2023 at 7:46 PM Adam Ford wrote: > > Several changes have been made to the device tree > in the kernel, so update that as well as the > corresponding imx8mp-u-boot.dtsi files to prevent > breaking the booting. > > Signed-off-by: Adam Ford Reviewed-by: Fabio Estevam

Re: [PATCH 1/2] clk: imx8mp: Update clocks based on kernel 6.4-RC4

2023-05-30 Thread Fabio Estevam
bled > to match with the upstream kernel as well. > > Signed-off-by: Adam Ford Reviewed-by: Fabio Estevam

Re: [PATCH 3/3] arm: dts: imx8mp: Sync with Linux 6.3

2023-05-24 Thread Fabio Estevam
Hi Tim, On Fri, May 19, 2023 at 8:00 PM Tim Harvey wrote: > Fabio, > > There's more to be done here also. With this patch, and with the > spba-bus added to u-boot.dtsi, if you try to enable usb (usb start) > you get: > starting USB... > Bus usb@3820: > Enable clock-controller@3038

Re: [PATCH 3/3] arm: dts: imx8mp: Sync with Linux 6.3

2023-05-22 Thread Fabio Estevam
Hi Tim, On Fri, May 19, 2023 at 8:00 PM Tim Harvey wrote: > Fabio, > > There's more to be done here also. With this patch, and with the > spba-bus added to u-boot.dtsi, if you try to enable usb (usb start) > you get: > starting USB... > Bus usb@3820: > Enable clock-controller@3038

[PATCH 2/2] doc: bindings: soft-spi: Remove the usage of deprecated properties

2023-05-18 Thread Fabio Estevam
From: Fabio Estevam According to Documentation/devicetree/bindings/spi/spi-gpio.yaml from Linux, the recommended spio-gpio properties are: sck-gpios, miso-gpios and mosi-gpios. gpio-sck, gpio-mosi and gpio-miso are considered deprecated. Update the bindings to suggest the recommeded

[PATCH 1/2] spi: soft_spi: Support the recommended soft spi properties

2023-05-18 Thread Fabio Estevam
From: Fabio Estevam According to Documentation/devicetree/bindings/spi/spi-gpio.yaml from Linux, the recommended spio-gpio properties are: sck-gpios, miso-gpios and mosi-gpios. gpio-sck, gpio-mosi and gpio-miso are considered deprecated. Currently, U-Boot only supports the deprecated

Re: [PATCH 3/5] freescale: fix header inclusion guard

2023-05-15 Thread Fabio Estevam
Hi Andre, > What do you mean, exactly? > To put *some* context into the commit message, I tried to find out what > this "vsc" device is about, and found "crosspoint switch" somewhere > (doc/README.VSC3316-3308). Is that not what it is about? Or is not a > Freescale device, but just used by

Re: [PATCH 3/5] freescale: fix header inclusion guard

2023-05-15 Thread Fabio Estevam
H Andre, On Mon, May 15, 2023 at 10:52 AM Andre Przywara wrote: > > It seems like the header inclusion guard for some Freescale crosspoint > switch header was misspelled. > > Make the preprocessor symbol for the #ifndef and #define lines the > same, so that the double inclusion protection works

Re: [PATCH 1/5] arm: mxs: Provide Kconfig option to to not use VDD5V as IMX28 PMU source

2023-05-10 Thread Fabio Estevam
Hi Lukasz, The series looks good. Only a minor comment: On Tue, May 9, 2023 at 11:33 AM Lukasz Majewski wrote: > +config SPL_MXS_PMU_MINIMAL_VDD5V_CURRENT > + bool "Force minimal current draw from VDD5V by MX28 PMU" > + default n Please drop "default n" as it is already the

[PATCH] power: imx8m-power-domain: Add delay to align with kernel driver

2023-05-06 Thread Fabio Estevam
From: Fabio Estevam In the imx8m power domain kernel driver, there is an extra udelay(5) prior to requesting the domain to power up: https://github.com/torvalds/linux/blob/v6.3/drivers/soc/imx/gpcv2.c#L347-L375 Haven't observed any issues due to the lack of this delay in U-Boot yet, but better

[PATCH] imx8qm_dmsse20a1: Improve SPL memory configuration

2023-05-04 Thread Fabio Estevam
From: Fabio Estevam Currently, building imx8qm_dmsse20a1_defconfig leads to the following warning: aarch64-linux-ld.bfd: invalid origin for memory region .sdram Fix it by using the same SPL memory layout as done in the imx8mq_evk. Reported-by: Tom Rini Signed-off-by: Fabio Estevam Tested

Re: Pull request: please pull u-boot-imx-20230503

2023-05-03 Thread Fabio Estevam
Hi Tom, On Wed, May 3, 2023 at 4:01 PM Tom Rini wrote: > So, both my HW test and CI is fine, so I've applied this. But please > note we now have this scary message to fix: > +(imx8qm_dmsse20a1) aarch64-linux-ld.bfd: invalid origin for memory region > .sdram I compared the imx8qm_dmsse20a1

Re: [PATCH] board: gateworks: venice: add eraseenv command

2023-05-02 Thread Fabio Estevam
On Tue, May 2, 2023 at 2:46 PM Tim Harvey wrote: > > Add eraseenv command and remove the unnecessary env command. > > Signed-off-by: Tim Harvey Reviewed-by: Fabio Estevam

Re: [PATCH] board: gateworks: venice: add GPIO name lookup

2023-05-02 Thread Fabio Estevam
t=> gpio input pci_wdis# > gpio: pin pci_wdis# (gpio 103) value is 1 > > Signed-off-by: Tim Harvey Reviewed-by: Fabio Estevam

[RFC] power: imx8m-power-domain: Add delay to align with kernel driver

2023-04-28 Thread Fabio Estevam
From: Fabio Estevam In the imx8m power domain kernel driver, there is an extra udelay(5) prior to requesting the domain to power up: https://github.com/torvalds/linux/blob/v6.3/drivers/soc/imx/gpcv2.c#L347-L375 Haven't observed any issues due to the lack of this delay in U-Boot, but better

Re: [PATCH 1/3] arm: dts: imx8mm: Sync with Linux 6.3

2023-04-28 Thread Fabio Estevam
patch which should get > applied through Marek via the usb tree before the dt sync. This works for me, thanks. When you submit it, feel free to add: Tested-by: Fabio Estevam

Re: [PATCH 1/3] arm: dts: imx8mm: Sync with Linux 6.3

2023-04-28 Thread Fabio Estevam
Hi Tim, On Fri, Apr 28, 2023 at 12:48 PM Tim Harvey wrote: > Yes I think that is similar enough to test. In my experience simply > enabling otg2 via dt on imx8mm-evk shows the issue I see here but > Fabio says he sees a hang on 'usb start' even before this dt sync and > I don't know why my

Re: [PATCH 1/3] arm: dts: imx8mm: Sync with Linux 6.3

2023-04-27 Thread Fabio Estevam
On Thu, Apr 27, 2023 at 4:44 PM Tim Harvey wrote: > Fabio, > > Sorry for the confusion. > > This imx8mm dt sync patch will hang on imx8mm boards that use 'both' > usbotg1 and usbotg2. You can reproduce this hang on your imx8mm-evk by > enabling usbotg2 in the dt (the board has it but it is not

Re: [PATCH 1/3] arm: dts: imx8mm: Sync with Linux 6.3

2023-04-27 Thread Fabio Estevam
On Thu, Apr 27, 2023 at 4:19 PM Tim Harvey wrote: > Fabio, > > Sorry, responded to the wrong thread here. The thread regarding imx8mm > hang on usb_stop is resolved with the regulator reference counting > support. > > The dt sync here still needs some work apparently. I am confused now. Which

Re: [PATCH 1/3] arm: dts: imx8mm: Sync with Linux 6.3

2023-04-27 Thread Fabio Estevam
On Thu, Apr 27, 2023 at 3:56 PM Tim Harvey wrote: > Fabio, > > This causes a hang on imx8mm boards when usbotg2 (usb@32e5) is > enabled. You can re-create this on the imx8mm-evk with: I am able to reproduce the hang after enabling usbotg2, but this hang is not caused by the imx8mm.dtsi

Re: imx8mm/imx8mn hang on usb stop / ehci_shutdown

2023-04-27 Thread Fabio Estevam
Hi Tim, On Thu, Apr 27, 2023 at 3:46 PM Tim Harvey wrote: > Fabio, > > Do you have another board to test with that uses both imx8mm usb host > controllers? With your imx8mm dtsi sync added a simple 'usb start' > hangs on my imx8mm-gw71xx board when the second controller is enabled: > starting

Re: [RESEND PATCH 0/3] imx6: clock: add support to get LCD pixel clock rate

2023-04-27 Thread Fabio Estevam
imx6: clock: add support to get LCD pixel clock rate > imx6: clock: print real pixel clock rate For the series: Reviewed-by: Fabio Estevam

Re: [PATCH] arm: imx8m: remove unused and obsolete board_fix_fdt() in SOC context

2023-04-27 Thread Fabio Estevam
Boot copy of imx8mp.dtsi. > > This code was introduced in commit 35bb60787b88. It seems to be some > random import of code from downstream NXP U-Boot, with a commit > message that makes no sense in upstream context. > > Signed-off-by: Hugo Villeneuve Reviewed-by: Fabio Estevam

Re: [PATCH] imx: fix get_boot_device() for imx8

2023-04-27 Thread Fabio Estevam
imx8ulp and imx9. > > Fixes: 787f04bb6a0a ("imx: add USB2_BOOT type") > Signed-off-by: Tim Harvey Reviewed-by: Fabio Estevam

Re: imx8mm/imx8mn hang on usb stop / ehci_shutdown

2023-04-27 Thread Fabio Estevam
Hi Tim, On Thu, Apr 27, 2023 at 2:52 PM Tim Harvey wrote: > Fabio, > > The original issue I was trying to solve should not be visible on > imx8mm-evk unless you changed its usbotg1 to dr_mode = "host" in which > case you should see a hang with a simple 'usb start && usb stop'. Just did the

[PATCH 3/3] arm: dts: imx8mp: Sync with Linux 6.3

2023-04-27 Thread Fabio Estevam
From: Fabio Estevam Sync imx8mp.dtsi and imx8mp-clock.h with Linux 6.3. Signed-off-by: Fabio Estevam --- arch/arm/dts/imx8mp.dtsi | 374 --- include/dt-bindings/clock/imx8mp-clock.h | 14 +- 2 files changed, 270 insertions(+), 118 deletions(-) diff --git

[PATCH 2/3] arm: dts: imx8mn: Sync with Linux 6.3

2023-04-27 Thread Fabio Estevam
From: Fabio Estevam Sync imx8mn.dtsi with Linux 6.3. Signed-off-by: Fabio Estevam --- arch/arm/dts/imx8mn.dtsi | 46 ++-- 1 file changed, 35 insertions(+), 11 deletions(-) diff --git a/arch/arm/dts/imx8mn.dtsi b/arch/arm/dts/imx8mn.dtsi index cb2836bfbd95

[PATCH 1/3] arm: dts: imx8mm: Sync with Linux 6.3

2023-04-27 Thread Fabio Estevam
From: Fabio Estevam Sync imx8mm.dtsi with Linux 6.3. 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 suggested to sync i

Re: imx8mm/imx8mn hang on usb stop / ehci_shutdown

2023-04-27 Thread Fabio Estevam
On Thu, Apr 27, 2023 at 2:55 PM Adam Ford wrote: > Any chance you can update the imx8mn.dtsi as well when you do the > patch? Nano has the same USB controller, it's likely to have the same > issue. Yes, I plan to update imx8mm, imx8mn and imx8mp. Will send the series shortly. Thanks

Re: imx8mm/imx8mn hang on usb stop / ehci_shutdown

2023-04-27 Thread Fabio Estevam
Hi Adam, On Thu, Apr 27, 2023 at 2:42 PM Adam Ford wrote: > There was some re-organization of the power domains in Linux in the > device tree [1]. Do you know if those propagated to U-Boot? Excellent, I just synced the kernel imx8mm.dtsi with U-Boot and it does fix the "ums 0 mmc 1" issue I

Re: imx8mm/imx8mn hang on usb stop / ehci_shutdown

2023-04-27 Thread Fabio Estevam
Hi Tim, On Thu, Apr 27, 2023 at 2:14 PM Tim Harvey wrote: > I haven't heard any response here so I did some more digging. This > hang should be affecting all imx8mm/imx8mn devices that have USB host > controllers configured for host mode (dr_mode = "host") and would > appear for anyone that

[PATCH v5 14/14] smegw01: Convert CFG_EXTRA_ENV_SETTINGS to an env file

2023-04-26 Thread Fabio Estevam
From: Fabio Estevam Currently, it is recommended to move from CFG_EXTRA_ENV_SETTINGS to an env file, so do the conversion. Suggested-by: Tom Rini Signed-off-by: Fabio Estevam --- Changes since v4: - Also add SETUP_BOOT_MENU into the .env file (Tom). Cc: Tom Rini board/storopack/smegw01

[PATCH v5 13/14] smegw01: Fix fallback to altbootcmd

2023-04-26 Thread Fabio Estevam
From: Eduard Strehlau The bootcmd/altbootcmd mechanism is not invoked for bootmenus. Manually compare bootcount and bootlimit to implement fallback on all codepaths. Signed-off-by: Eduard Strehlau Signed-off-by: Fabio Estevam --- Changes since v4: - None configs/smegw01_defconfig | 2 +- 1

[PATCH v5 12/14] smegw01: Disable additional boot menu options

2023-04-26 Thread Fabio Estevam
From: Eduard Strehlau Defaults have changed, we do not want the bootmenu to contain EFI options. Signed-off-by: Eduard Strehlau Signed-off-by: Fabio Estevam --- Changes since v4: - None configs/smegw01_defconfig | 2 ++ 1 file changed, 2 insertions(+) diff --git a/configs/smegw01_defconfig

[PATCH v5 11/14] smegw01: Read the second MAC address

2023-04-26 Thread Fabio Estevam
From: Fabio Estevam Currently, only the first MAC address is read from the fuses. The second MAC address is not read and Linux assigns a random one. To prevent this behavior, read the second MAC address from the fuses and store it into the eth1addr environment variable so that it can be passed

[PATCH v5 10/14] smegw01: Add lockdown U-Boot env support

2023-04-26 Thread Fabio Estevam
From: Eduard Strehlau Add lockdown U-Boot env support so that only certain U-Boot environment variables are allowed to be modified. Signed-off-by: Eduard Strehlau Signed-off-by: Fabio Estevam --- Changes since v4: - None board/storopack/smegw01/Kconfig | 7 + board/storopack/smegw01

[PATCH v5 07/14] smegw01: Enable EMMC boot from multiple partitions

2023-04-26 Thread Fabio Estevam
can overwrite the inactive hardware partition with new bootloader and environment and afterwards switch the eMMC boot partition for an atomic bootloader switch. Signed-off-by: Eduard Strehlau Signed-off-by: Fabio Estevam --- Changes since v4: - None board/storopack/smegw01/smegw01.c | 10

[PATCH v5 09/14] smegw01: Switch to fitImage

2023-04-26 Thread Fabio Estevam
From: Eduard Strehlau Instead of loading the zImage kernel and the devicetree separately, switch to the fitImage format, which is more convenient when working with secure boot, for example. Signed-off-by: Eduard Strehlau Signed-off-by: Fabio Estevam --- Changes since v4: - None include

[PATCH v5 08/14] smegw01: Change default boot device to eMMC

2023-04-26 Thread Fabio Estevam
From: Eduard Strehlau Let eMMC be the default boot medium. Signed-off-by: Eduard Strehlau Signed-off-by: Fabio Estevam --- Changes since v4: - None include/configs/smegw01.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/configs/smegw01.h b/include/configs

[PATCH v5 06/14] smegw01: Only commit to new partition if update was successful

2023-04-26 Thread Fabio Estevam
From: Eduard Strehlau When performing rootfs update via swupdate, it is convenient to check the 'ustate' variable to decide whether the update succeeded or not. Signed-off-by: Eduard Strehlau Signed-off-by: Fabio Estevam --- Changes since v4: - None include/configs/smegw01.h | 16

[PATCH v5 05/14] smegw01: Run altbootcmd in the case of failure

2023-04-26 Thread Fabio Estevam
From: Eduard Strehlau Run the altbootcmd script if any step of bootcmd fails. This ensures that always a valid image can be run. Signed-off-by: Eduard Strehlau Signed-off-by: Fabio Estevam --- Changes since v4: - None configs/smegw01_defconfig | 2 +- include/configs/smegw01.h | 8

[PATCH v5 04/14] smegw01: Add altbootcmd

2023-04-26 Thread Fabio Estevam
From: Eduard Strehlau Add an altbootcmd script, which is convenient way to integrate with swupdate and perform a roll back of the previous working version in the case of update failure. Signed-off-by: Eduard Strehlau Signed-off-by: Fabio Estevam --- Changes since v4: - None configs

[PATCH v5 03/14] smegw01: Select bootcount support

2023-04-26 Thread Fabio Estevam
From: Eduard Strehlau Add automatic boot counter that increases after every reset. After a power-on reset, it will be initialized with 1, and each reboot will increment the value by 1. By default it is disabled if bootlimit isn't set. Signed-off-by: Eduard Strehlau Signed-off-by: Fabio Estevam

[PATCH v5 02/14] smegw01: Select CONFIG_CMD_SQUASHFS

2023-04-26 Thread Fabio Estevam
From: Eduard Strehlau Select CONFIG_CMD_SQUASHFS so that the SquashFS U-Boot commands are available. Signed-off-by: Eduard Strehlau Signed-off-by: Fabio Estevam --- Changes since v4: - None configs/smegw01_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/configs

[PATCH v5 01/14] smegw01: Enable setting additional boot params

2023-04-26 Thread Fabio Estevam
From: Eduard Strehlau Introduce EXTRA_BOOTPARAMS to allow passing additional parameters to kernel command line. This is useful for debugging purposes. Signed-off-by: Eduard Strehlau Signed-off-by: Fabio Estevam --- Changes since v4: - None include/configs/smegw01.h | 8 +++- 1 file

[PATCH v4 14/14] smegw01: Convert CFG_EXTRA_ENV_SETTINGS to an env file

2023-04-26 Thread Fabio Estevam
From: Fabio Estevam Currently, it is recommended to move from CFG_EXTRA_ENV_SETTINGS to an env file, so do the conversion. Suggested-by: Tom Rini Signed-off-by: Fabio Estevam --- Changes since v3: - New patch added as per Tom Rini's suggestion. Cc: Tom Rini board/storopack/smegw01/smegw01

[PATCH v4 13/14] smegw01: Fix fallback to altbootcmd

2023-04-26 Thread Fabio Estevam
From: Eduard Strehlau The bootcmd/altbootcmd mechanism is not invoked for bootmenus. Manually compare bootcount and bootlimit to implement fallback on all codepaths. Signed-off-by: Eduard Strehlau Signed-off-by: Fabio Estevam --- Changes since v3: - None configs/smegw01_defconfig | 2 +- 1

[PATCH v4 12/14] smegw01: Disable additional boot menu options

2023-04-26 Thread Fabio Estevam
From: Eduard Strehlau Defaults have changed, we do not want the bootmenu to contain EFI options. Signed-off-by: Eduard Strehlau Signed-off-by: Fabio Estevam --- Changes since v3: - None configs/smegw01_defconfig | 2 ++ 1 file changed, 2 insertions(+) diff --git a/configs/smegw01_defconfig

[PATCH v4 11/14] smegw01: Read the second MAC address

2023-04-26 Thread Fabio Estevam
From: Fabio Estevam Currently, only the first MAC address is read from the fuses. The second MAC address is not read and Linux assigns a random one. To prevent this behavior, read the second MAC address from the fuses and store it into the eth1addr environment variable so that it can be passed

[PATCH v4 10/14] smegw01: Add lockdown U-Boot env support

2023-04-26 Thread Fabio Estevam
From: Eduard Strehlau Add lockdown U-Boot env support so that only certain U-Boot environment variables are allowed to be modified. Signed-off-by: Eduard Strehlau Signed-off-by: Fabio Estevam --- Changes since v3: - None board/storopack/smegw01/Kconfig | 7 + board/storopack/smegw01

[PATCH v4 09/14] smegw01: Switch to fitImage

2023-04-26 Thread Fabio Estevam
From: Eduard Strehlau Instead of loading the zImage kernel and the devicetree separately, switch to the fitImage format, which is more convenient when working with secure boot, for example. Signed-off-by: Eduard Strehlau Signed-off-by: Fabio Estevam --- Changes since v3: - None include

[PATCH v4 08/14] smegw01: Change default boot device to eMMC

2023-04-26 Thread Fabio Estevam
From: Eduard Strehlau Let eMMC be the default boot medium. Signed-off-by: Eduard Strehlau Signed-off-by: Fabio Estevam --- Changes since v3: - None include/configs/smegw01.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/configs/smegw01.h b/include/configs

[PATCH v4 07/14] smegw01: Enable EMMC boot from multiple partitions

2023-04-26 Thread Fabio Estevam
can overwrite the inactive hardware partition with new bootloader and environment and afterwards switch the eMMC boot partition for an atomic bootloader switch. Signed-off-by: Eduard Strehlau Signed-off-by: Fabio Estevam --- Changes since v3: - None board/storopack/smegw01/smegw01.c | 10

[PATCH v4 06/14] smegw01: Only commit to new partition if update was successful

2023-04-26 Thread Fabio Estevam
From: Eduard Strehlau When performing rootfs update via swupdate, it is convenient to check the 'ustate' variable to decide whether the update succeeded or not. Signed-off-by: Eduard Strehlau Signed-off-by: Fabio Estevam --- Changes since v3: - None include/configs/smegw01.h | 16

[PATCH v4 05/14] smegw01: Run altbootcmd in the case of failure

2023-04-26 Thread Fabio Estevam
From: Eduard Strehlau Run the altbootcmd script if any step of bootcmd fails. This ensures that always a valid image can be run. Signed-off-by: Eduard Strehlau Signed-off-by: Fabio Estevam --- Changes since v3: - None configs/smegw01_defconfig | 2 +- include/configs/smegw01.h | 8

[PATCH v4 04/14] smegw01: Add altbootcmd

2023-04-26 Thread Fabio Estevam
From: Eduard Strehlau Add an altbootcmd script, which is convenient way to integrate with swupdate and perform a roll back of the previous working version in the case of update failure. Signed-off-by: Eduard Strehlau Signed-off-by: Fabio Estevam --- Changes since v3: - None configs

[PATCH v4 03/14] smegw01: Select bootcount support

2023-04-26 Thread Fabio Estevam
From: Eduard Strehlau Add automatic boot counter that increases after every reset. After a power-on reset, it will be initialized with 1, and each reboot will increment the value by 1. By default it is disabled if bootlimit isn't set. Signed-off-by: Eduard Strehlau Signed-off-by: Fabio Estevam

[PATCH v4 02/14] smegw01: Select CONFIG_CMD_SQUASHFS

2023-04-26 Thread Fabio Estevam
From: Eduard Strehlau Select CONFIG_CMD_SQUASHFS so that the SquashFS U-Boot commands are available. Signed-off-by: Eduard Strehlau Signed-off-by: Fabio Estevam --- Changes since v3: - None configs/smegw01_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/configs

[PATCH v4 01/14] smegw01: Enable setting additional boot params

2023-04-26 Thread Fabio Estevam
From: Eduard Strehlau Introduce EXTRA_BOOTPARAMS to allow passing additional parameters to kernel command line. This is useful for debugging purposes. Signed-off-by: Eduard Strehlau Signed-off-by: Fabio Estevam --- Changes since v3: - None include/configs/smegw01.h | 8 +++- 1 file

[PATCH v3 13/13] smegw01: Fix fallback to altbootcmd

2023-04-24 Thread Fabio Estevam
From: Eduard Strehlau The bootcmd/altbootcmd mechanism is not invoked for bootmenus. Manually compare bootcount and bootlimit to implement fallback on all codepaths. Signed-off-by: Eduard Strehlau Signed-off-by: Fabio Estevam --- Changes since v2: - None configs/smegw01_defconfig | 2 +- 1

[PATCH v3 11/13] smegw01: Read the second MAC address

2023-04-24 Thread Fabio Estevam
From: Fabio Estevam Currently, only the first MAC address is read from the fuses. The second MAC address is not read and Linux assigns a random one. To prevent this behavior, read the second MAC address from the fuses and store it into the eth1addr environment variable so that it can be passed

[PATCH v3 12/13] smegw01: Disable additional boot menu options

2023-04-24 Thread Fabio Estevam
From: Eduard Strehlau Defaults have changed, we do not want the bootmenu to contain EFI options. Signed-off-by: Eduard Strehlau Signed-off-by: Fabio Estevam --- Changes since v2: - None configs/smegw01_defconfig | 2 ++ 1 file changed, 2 insertions(+) diff --git a/configs/smegw01_defconfig

[PATCH v3 10/13] smegw01: Add lockdown U-Boot env support

2023-04-24 Thread Fabio Estevam
From: Eduard Strehlau Add lockdown U-Boot env support so that only certain U-Boot environment variables are allowed to be modified. Signed-off-by: Eduard Strehlau Signed-off-by: Fabio Estevam --- Changes since v2: - None board/storopack/smegw01/Kconfig | 7 + board/storopack/smegw01

[PATCH v3 09/13] smegw01: Switch to fitImage

2023-04-24 Thread Fabio Estevam
From: Eduard Strehlau Instead of loading the zImage kernel and the devicetree separately, switch to the fitImage format, which is more convenient when working with secure boot, for example. Signed-off-by: Eduard Strehlau Signed-off-by: Fabio Estevam --- Changes since v2: - None include

[PATCH v3 08/13] smegw01: Change default boot device to eMMC

2023-04-24 Thread Fabio Estevam
From: Eduard Strehlau Let eMMC be the default boot medium. Signed-off-by: Eduard Strehlau Signed-off-by: Fabio Estevam --- Changes since v2: - None include/configs/smegw01.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/configs/smegw01.h b/include/configs

[PATCH v3 07/13] smegw01: Enable EMMC boot from multiple partitions

2023-04-24 Thread Fabio Estevam
can overwrite the inactive hardware partition with new bootloader and environment and afterwards switch the eMMC boot partition for an atomic bootloader switch. Signed-off-by: Eduard Strehlau Signed-off-by: Fabio Estevam --- Changes since v2: - Use the same board_mmc_get_env_part

[PATCH v3 06/13] smegw01: Only commit to new partition if update was successful

2023-04-24 Thread Fabio Estevam
From: Eduard Strehlau When performing rootfs update via swupdate, it is convenient to check the 'ustate' variable to decide whether the update succeeded or not. Signed-off-by: Eduard Strehlau Signed-off-by: Fabio Estevam --- Changes since v2: - None include/configs/smegw01.h | 16

[PATCH v3 05/13] smegw01: Run altbootcmd in the case of failure

2023-04-24 Thread Fabio Estevam
From: Eduard Strehlau Run the altbootcmd script if any step of bootcmd fails. This ensures that always a valid image can be run. Signed-off-by: Eduard Strehlau Signed-off-by: Fabio Estevam --- Changes since v2: - None configs/smegw01_defconfig | 2 +- include/configs/smegw01.h | 8

[PATCH v3 04/13] smegw01: Add altbootcmd

2023-04-24 Thread Fabio Estevam
From: Eduard Strehlau Add an altbootcmd script, which is convenient way to integrate with swupdate and perform a roll back of the previous working version in the case of update failure. Signed-off-by: Eduard Strehlau Signed-off-by: Fabio Estevam --- Changes since v2: - None configs

[PATCH v3 03/13] smegw01: Select bootcount support

2023-04-24 Thread Fabio Estevam
From: Eduard Strehlau Add automatic boot counter that increases after every reset. After a power-on reset, it will be initialized with 1, and each reboot will increment the value by 1. By default it is disabled if bootlimit isn't set. Signed-off-by: Eduard Strehlau Signed-off-by: Fabio Estevam

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