[PATCH v2] 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] 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] smegw01: Remove misuse of CONFIG_ENV_IS_NOWHERE

2023-10-10 Thread Fabio Estevam
nt. Based on the fix done by commit b16fd7f75f6d ("imx6q: acc: Remove misuse of env is nowhere driver"). Signed-off-by: Eduard Strehlau Signed-off-by: Fabio Estevam --- board/storopack/smegw01/smegw01.c | 15 --- configs/smegw01_defconfig | 1 - 2 files changed, 16 dele

[PATCH v3 3/3] mx28evk: Add USB Mass Storage support

2023-10-09 Thread Fabio Estevam
From: Fabio Estevam Select the USB options to allow running "ums 0 mmc 0". Signed-off-by: Fabio Estevam --- Changes since v2: - Removed further unrelated options. (Marek) Changes since v1: - Remove unrelated config options. (Marek) configs/mx28evk_defconfig | 12 1 fi

[PATCH v3 2/3] usb: ehci: mxs: Use regulator_set_enable_if_allowed()

2023-10-09 Thread Fabio Estevam
From: Fabio Estevam Since commit 4fcba5d556b4 ("regulator: implement basic reference counter") the return value of regulator_set_enable() may be EALREADY or EBUSY for fixed/GPIO regulators. Switch to using the more relaxed regulator_set_enable_if_allowed() to continue if regulat

[PATCH v3 1/3] usb: ehci: mxs: Fix the USB node pointer retrieval

2023-10-09 Thread Fabio Estevam
From: Fabio Estevam Use dev_ofnode() to retrieve the USB node pointer from the udevice structure. This fixes the following build error: drivers/usb/host/ehci-mxs.c:143:38: error: 'struct udevice' has no member named 'node_' Signed-off-by: Fabio Estevam Reviewed-by: Marek Vasut --- Changes

[PATCH v2 3/3] mx28evk: Add USB Mass Storage support

2023-10-08 Thread Fabio Estevam
From: Fabio Estevam Select the USB options to allow running "ums 0 mmc 0". Signed-off-by: Fabio Estevam --- Changes since v1: - Remove unrelated config options. (Marek) configs/mx28evk_defconfig | 12 1 file changed, 12 insertions(+) diff --git a/configs/mx28evk_d

[PATCH v2 2/3] usb: ehci: mxs: Use regulator_set_enable_if_allowed()

2023-10-08 Thread Fabio Estevam
From: Fabio Estevam Since commit 4fcba5d556b4 ("regulator: implement basic reference counter") the return value of regulator_set_enable() may be EALREADY or EBUSY for fixed/GPIO regulators. Switch to using the more relaxed regulator_set_enable_if_allowed() to continue if regulat

[PATCH v2 1/3] usb: ehci: mxs: Fix the USB node pointer retrieval

2023-10-08 Thread Fabio Estevam
From: Fabio Estevam Use dev_ofnode() to retrieve the USB node pointer from the udevice structure. This fixes the following build error: drivers/usb/host/ehci-mxs.c:143:38: error: 'struct udevice' has no member named 'node_' Signed-off-by: Fabio Estevam Reviewed-by: Marek Vasut --- Changes

[PATCH 2/3] usb: ehci: mxs: Use regulator_set_enable_if_allowed()

2023-10-07 Thread Fabio Estevam
From: Fabio Estevam Since commit 4fcba5d556b4 ("regulator: implement basic reference counter") the return value of regulator_set_enable() may be EALREADY or EBUSY for fixed/GPIO regulators. Swict to using the more relaxed regulator_set_enable_if_allowed() to continue if regulat

[PATCH 3/3] mx28evk: Add USB Mass Storage support

2023-10-07 Thread Fabio Estevam
From: Fabio Estevam Select the USB options to allow running "ums 0 mmc 0". Signed-off-by: Fabio Estevam --- configs/mx28evk_defconfig | 14 ++ 1 file changed, 14 insertions(+) diff --git a/configs/mx28evk_defconfig b/configs/mx28evk_defconfig index df0cceaea719..15

[PATCH 1/3] usb: ehci: mxs: Fix the USB node pointer retrieval

2023-10-07 Thread Fabio Estevam
From: Fabio Estevam Use dev_ofnode() to retrieve the USB node pointer from the udevice structure. This fixes the following build error: drivers/usb/host/ehci-mxs.c:143:38: error: 'struct udevice' has no member named 'node_' Signed-off-by: Fabio Estevam --- drivers/usb/host/ehci-mxs.c | 7

Re: [RFC PATCH] mxs: Don't enable 4P2 reg if mx28 is powered only from DCDC_BATT without 5V

2023-10-07 Thread Fabio Estevam
Hi Alan, [Adding Lukasz and Cody] On Tue, Aug 30, 2022 at 2:49 PM Alan Kay wrote: > > mxs_power_enable_4p2() was added to mxs_batt_boot() in > 'commit a0f97610757d ("ARM: mxs: Enable DCDC converter for battery boot")' > to enable DCDC converter when board is powered from 5V and has > detected

Re: [PATCH v5] imx: imx8mm: Add support for Mettler-Toledo snowflake board.

2023-10-07 Thread Fabio Estevam
iled. > - Replace a reference to SMARC with the current product name of the >SoM. > > v1?v2: > - Clarify details about the board and whether or not there is an >external SD-card interface. The changelog should go below the --- line. Reviewed-by: Fabio Estevam

Re: [PATCH v1] board: toradex: verdin-imx8mm: set fixed LPDDR4 refresh rate as per errata ERR050805

2023-10-07 Thread Fabio Estevam
with de-rating > 3b111: SDRAM High temperature operating limit exceeded > > [1]: https://www.nxp.com/docs/en/errata/IMX8MM_0N87W.pdf > > Signed-off-by: Andrejs Cainikovs Reviewed-by: Fabio Estevam

Re: [PATCH v2 1/6] board: phytec: Add common PHYTEC SoM detection

2023-10-07 Thread Fabio Estevam
e used for RAM detection and loading device tree overlays > > during kernel start. > > > > Signed-off-by: Teresa Remmet > > Reviewed-by: Yannic Moog > Tested-by: Yannic Moog For the series: Reviewed-by: Fabio Estevam

Re: [PATCH] usbarmory: Add DM_I2C and DM_SERIAL support

2023-10-07 Thread Fabio Estevam
On Thu, Sep 21, 2023 at 8:27 AM Andrej Rosano wrote: > > From: Andrej Rosano > > Use DM_I2C and DM_SERIAL as it is now mandatory. > > Signed-off-by: Andrej Rosano Reviewed-by: Fabio Estevam

Re: [PATCH 2/2] dm: adc: imx93-adc depends on ADC (fix boot)

2023-10-07 Thread Fabio Estevam
epend on CONFIG_ADC. > > Signed-off-by: Sébastien Szymanski Reviewed-by: Fabio Estevam

Re: [PATCH 1/2] arm: dts: imx93-11x11-evk: add bootph-some-ram property

2023-10-07 Thread Fabio Estevam
uot; > > Fix this by adding bootph-some-ram prop as suggested by the commit > above. > > Fixes: 9e644284ab81 ("dm: core: Report bootph-pre-ram/sram node as pre-reloc > after relocation") > Signed-off-by: Sébastien Szymanski Yes, it is getting harder and harder to keep track of all these boot properties. Thanks for fixing it: Reviewed-by: Fabio Estevam

Re: [PATCH] toradex: verdin-imx8mm/imx8mp: Remove bootcmd_mfg

2023-10-07 Thread Fabio Estevam
On Wed, Oct 4, 2023 at 4:14 PM Francesco Dolcini wrote: > > From: Joao Paulo Goncalves > > The bootcmd_mfg env variable is legacy from IMX downstream u-boot branch > and is not needed on mainline. > > Signed-off-by: Joao Paulo Goncalves > Signed-off-by: Francesco Dolc

Re: [PATCH] arm: mach-imx: Makefile: Extend u-boot-nand.imx padding

2023-10-07 Thread Fabio Estevam
alue that makes the > proper padding for these NAND devices. > > [1] > https://lore.kernel.org/all/cac4tdfuqffqzrqfv5age_xtbfy1agr2sepn_fzedexhwjdr...@mail.gmail.com/ > > Signed-off-by: Hiago De Franco Reviewed-by: Fabio Estevam

Re: [PATCH] toradex: colibri_imx7: Enable nand/emmc detection and set boot variant

2023-10-07 Thread Fabio Estevam
tly to RAM. This > allows variant detection for an accurate device tree selection. > > Signed-off-by: Hiago De Franco Reviewed-by: Fabio Estevam

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

2023-10-07 Thread Fabio Estevam
0be The messages: MMC Device 2 not found no mmc device at slot 2 are a bit annoying as there is no SD card in the imx7d-pico-pi. I would prefer if these messages were not shown. Can you try to get rid of it on the imx7d-pico-pi? Tested-by: Fabio Estevam

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

2023-10-05 Thread Fabio Estevam
On Thu, Oct 5, 2023 at 6:40 PM wrote: > > From: Benjamin Szőke > > Take over codes from Techenxion to support mmc autodetect boot for pico-imx7d. > > Signed-off-by: Benjamin Szőke Still not good: WARNING: Possible unwrapped commit description (prefer a maximum 75 chars per line) #57: Take

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

2023-10-04 Thread Fabio Estevam
On Wed, Oct 4, 2023 at 8:35 PM wrote: > > From: Benjamin Szőke > > Take over codes from Techenxion to support mmc autodetect boot for pico-imx7d. > > Signed-off-by: Benjamin Szőke I wanted to test your patch, but there are several issues: checkpatch complains: total: 5 errors, 7 warnings, 5

[PATCH v2] pico-pi-imx7d: Unselect CONFIG_CONSOLE_MUX

2023-10-03 Thread Fabio Estevam
From: Fabio Estevam Unselect CONFIG_CONSOLE_MUX to fix the following error for the input, output and error interfaces: U-Boot 2023.10 (Oct 03 2023 - 21:23:18 -0300) ... In:No input devices available! Out: No output devices available! Err: No error devices available! Net: eth0

[PATCH] pico-pi-imx7d: Unselect CONFIG_CONSOLE_MUX

2023-10-03 Thread Fabio Estevam
From: Fabio Estevam Unselect CONFIG_CONSOLE_MUX to fix the following error for the input, output and error interfaces: U-Boot 2023.10 (Oct 03 2023 - 21:23:18 -0300) CPU: Freescale i.MX7D rev1.2 1000 MHz (running at 792 MHz) CPU: Commercial temperature grade (0C to 95C) at 42C Reset cause

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

2023-09-26 Thread Fabio Estevam
Hi Rogan, On Tue, Sep 26, 2023 at 1:01 AM Rogan Dawes wrote: > > Hi Fabio, > > That prints "LLC", but does not print "Pref". What happens if you unselect DM_SERIAL like this: --- a/configs/imx28-wink-hub_defconfig +++ b/configs/imx28-wink-hub_defconfig @@ -58,4 +58,3 @@ CONFIG_DM_REGULATOR=y

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

2023-09-25 Thread Fabio Estevam
Hi Rogan, On Mon, Sep 25, 2023 at 4:27 PM Rogan Dawes wrote: > > That outputs LLC about a second after the mxsldr terminates (kinda slowly, > the individual characters can be seen being emitted one at a time), and then > about 5 seconds later, "Pref". But nothing after that point. > > At least

[PATCH] smegw01: Fix inverted CONFIG_SYS_BOOT_LOCKED logic

2023-09-25 Thread Fabio Estevam
CONFIG_SYS_BOOT_LOCKED by default. Signed-off-by: Eduard Strehlau Signed-off-by: Fabio Estevam --- Hi Tom, We have just identified this bug. Could you please pick this one directly for U-Boot 2023.10? Thanks board/storopack/smegw01/smegw01.env | 4 ++-- configs/smegw01_defconfig | 2 +- 2 files

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

2023-09-25 Thread Fabio Estevam
On Mon, Sep 25, 2023 at 11:00 AM Rogan Dawes wrote: > I see absolutely nothing in the console. > > I’m wondering whether I can possibly execute the vendor SPL with a modern > u-boot? The old U-Boot vendor does not use SPL. It uses an old mechanism called bootlets. I suggest that we start from

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

2023-09-25 Thread Fabio Estevam
Hi Rogan, On Mon, Sep 25, 2023 at 6:52 AM Rogan Dawes wrote: > > Hi Fabio, > > I used the following diff with "make mx28evk_defconfig", but unfortunately > still get absolutely nothing on the DUART when running "mxsloader u-boot.sb". > In fact, mxsloader returns immediately on trying to load

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

2023-09-24 Thread Fabio Estevam
Hi Rogan, On Sun, Sep 24, 2023 at 5:16 PM Rogan Dawes wrote: > > Hi folks, > > I eventually decided that I did need to figure out the DRAM timings for the > Micron chip on my board, but I found an easier way than actually > "understanding what I'm doing"! :-) > > I pulled the vendor U-Boot

Re: [PATCH] verdin-imx8mp: drop unused tdx easy installer ifdef

2023-09-20 Thread Fabio Estevam
On Mon, Sep 4, 2023 at 11:09 AM Hiago De Franco wrote: > > From: Hiago De Franco > > Drop unused code related to CONFIG_TDX_EASY_INSTALLER, that existed only on > toradex downstream branch. > > Signed-off-by: Hiago De Franco Reviewed-by: Fabio Estevam

Re: [PATCH] imx8mp: binman: rename spl and u-boot nodes

2023-09-20 Thread Fabio Estevam
ame, and the csf.sh script will work for all four SOCs. > > Signed-off-by: Rasmus Villemoes Reviewed-by: Fabio Estevam

Re: [PATCH v2] mx8m: csf.sh: pad csf blob for u-boot.itb to CSF_SIZE minus IVT header

2023-09-20 Thread Fabio Estevam
ctually serve at least as many bytes as the > target expects, or we will hang in rom_api_download_image(). > > Extend the logic in the csf.sh script so that the csf blob is padded > to CONFIG_CSF_SIZE minus the size of the IVT header. > > Signed-off-by: Rasmus Villemoes > --- &g

Re: [PATCH resend 2/2] imx: spl_imx_romapi.c: remove dead code

2023-09-20 Thread Fabio Estevam
On Tue, Sep 19, 2023 at 10:49 AM Rasmus Villemoes wrote: > > These IS_ENABLED(CONFIG_SPL_LOAD_FIT) cases can no longer be reached, > and thus get_fit_image_size() is also redundant. > > Signed-off-by: Rasmus Villemoes Reviewed-by: Fabio Estevam

Re: [PATCH resend 1/2] imx: spl_imx_romapi: avoid tricky use of spl_load_simple_fit() to get full FIT size

2023-09-20 Thread Fabio Estevam
; > This only affects the CONFIG_SPL_LOAD_FIT case - I don't have any > hardware or experience with the CONFIG_SPL_LOAD_IMX_CONTAINER case, so > I leave that alone for now. > > Signed-off-by: Rasmus Villemoes Reviewed-by: Fabio Estevam

[PATCH] smegw01: Use CONFIG_SYS_LOAD_ADDR for loading fitImage

2023-09-20 Thread Fabio Estevam
From: Eduard Strehlau Set CONFIG_SYS_LOAD_ADDR=0x8800 and use this address for loading fitImage. Also pass the standard CONFIG_BOOTFILE option to indicate the fitImage file. Signed-off-by: Eduard Strehlau Signed-off-by: Fabio Estevam --- board/storopack/smegw01/smegw01.env | 14

[PATCH] pico-imx7d: Remove Vanessa from MAINTAINERS

2023-09-19 Thread Fabio Estevam
From: Fabio Estevam Vanessa's NXP e-mail is no longer active. Contacted her offline and she told me that she does not have access to the board anymore and it is OK to remove her from MAINTAINERS. Signed-off-by: Fabio Estevam --- board/technexion/pico-imx7d/MAINTAINERS | 1 - 1 file changed

[PATCH] imx7: Disable CAAM Job Ring 0

2023-09-12 Thread Fabio Estevam
From: Fabio Estevam Trying to boot a fitImage after a successful hab_auth_img operation causes the following error: ## Loading kernel from FIT Image at 8800 ... Using 'conf-imx7d-smegw01.dtb' configuration Trying 'kernel-1' kernel subimage Description: Linux kernel Type

[PATCH 3/3] imx: hab: Explain that ivt_offset is optional

2023-09-06 Thread Fabio Estevam
From: Fabio Estevam The ivt_offset parameter is optional for both hab_auth_img_or_fail and hab_auth_img commands. Document it in their usage texts to make it clearer. Signed-off-by: Fabio Estevam --- arch/arm/mach-imx/hab.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff

[PATCH 2/3] imx: hab: Improve the hab_auth_img_or_fail usage text

2023-09-06 Thread Fabio Estevam
From: Fabio Estevam Split the hab_auth_img_or_fail usage text in two sentences to make it clearer. Signed-off-by: Fabio Estevam --- arch/arm/mach-imx/hab.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/mach-imx/hab.c b/arch/arm/mach-imx/hab.c index 6cacca069d9f

[PATCH 1/3] imx: hab: Allow hab_auth_img_or_fail to be called without ivt_offset

2023-09-06 Thread Fabio Estevam
parameters. Signed-off-by: Eduard Strehlau Signed-off-by: Fabio Estevam --- arch/arm/mach-imx/hab.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/mach-imx/hab.c b/arch/arm/mach-imx/hab.c index 439cdaf07a7c..6cacca069d9f 100644 --- a/arch/arm/mach-imx/hab.c +++ b/arch/ar

Re: [RFC 0/6] firmware: scmi: add SCMI pinctrl protocol support

2023-09-05 Thread Fabio Estevam
Adding Peng Fan, who is working on scmi/pinctrl support for i.MX9: https://lore.kernel.org/all/ZO9GLG5tQynYyAvR@pluto/T/ On Tue, Sep 5, 2023 at 11:41 PM AKASHI Takahiro wrote: > > This is an RFC and meant to get feedback from other developers as > - the specification (pinctrl part) is still in

Re: [PATCH] imx: hab: Use CONFIG_SPL_LOAD_FIT_ADDRESS in the CSF example

2023-08-31 Thread Fabio Estevam
On 31/08/2023 11:56, Marek Vasut wrote: The SPL authenticates image starting from CONFIG_SPL_LOAD_FIT_ADDRESS address, update the csf_fit.txt to match. Signed-off-by: Marek Vasut --- Cc: "NXP i.MX U-Boot Team" Cc: Fabio Estevam Cc: Peng Fan Cc: Rasmus Villemoes Cc: Stefano Babi

Re: [PATCH v1] board: toradex: verdin-imx8mp: enable usb device and fastboot support

2023-08-28 Thread Fabio Estevam
Note that the i.MX 8M Plus recovery mode support is based on the USB > boot stage of the BOOTROM and does NOT require USB SDP SPL aka serial > downloader support. > > Signed-off-by: Marcel Ziswiler Reviewed-by: Fabio Estevam

Re: [PATCH v2 0/6] PHYTEC SoM detection for phyCORE-i.MX8MP

2023-08-24 Thread Fabio Estevam
Hi Yannic, On Thu, Aug 17, 2023 at 5:57 AM Teresa Remmet wrote: > > second version of the series to add support for EEPROM SoM detection used > by different PHYTEC SoMs. The EEPROM data consist of 32 bytes containing > information like PCB revision, RAM size and other SoM specific >

Re: [PATCH v1] board: toradex: verdin-imx8mm: enable usb sdp spl recovery support

2023-08-24 Thread Fabio Estevam
ex Easy Installer FIT > image. > > Signed-off-by: Marcel Ziswiler Yes, this is helpful: Reviewed-by: Fabio Estevam

[PATCH v2 5/5] thermal: imx_tmu: Increase the polling interval

2023-08-23 Thread Fabio Estevam
From: Fabio Estevam Polling every second to check whether the CPU has cooled down is too frequent. Allow more time for the CPU to cool down by increasing the polling interval to 5 seconds by defaut. This value is used in the absence of the 'polling-delay' devicetree property. Signed-off

[PATCH v2 4/5] thermal: imx_tmu: Fix the temperature unit

2023-08-23 Thread Fabio Estevam
From: Fabio Estevam The temperature unit is millidegree Celsius, so divide by 1000 to correctly print the temperature values in Celsius. While at it, also change a typo: "has beyond" to "is beyond". Signed-off-by: Fabio Estevam --- Changes since v1: - Rebased. drivers/t

[PATCH v2 3/5] thermal: imx_tmu: Increase the log level for high temperatures

2023-08-23 Thread Fabio Estevam
From: Fabio Estevam dev_info() message is not printed by default. Increase the log level to dev_crit(). This allows the critical messages related to the temperature getting beyong the alert threshold to be displayed. Signed-off-by: Fabio Estevam --- Changes since v1: - Use dev_crit() instead

[PATCH v2 2/5] thermal: imx_tmu: Fix the polling default

2023-08-23 Thread Fabio Estevam
From: Fabio Estevam When the 'polling-delay' property is not passed via devicetree, pdata->polling_delay keeps at 0. This causes the imx_tmu driver to get stuck inside the busy while() loop when the CPU temperature is above the alert point. Fix this problem by passing a one second poll

[PATCH v2 1/5] imx8mm_evk_defconfig: Select CONFIG_IMX_TMU

2023-08-23 Thread Fabio Estevam
From: Fabio Estevam Select the i.MX8MM thermal driver as it is useful for displaying the CPU temperature and its grading: CPU: Commercial temperature grade (0C to 95C) at 38C It also prevents booting when the temperature is above the alert point. Signed-off-by: Fabio Estevam --- Changes

[PATCH 5/5] thermal: imx_tmu: Increase the polling interval

2023-08-23 Thread Fabio Estevam
From: Fabio Estevam Polling every second to check whether the CPU has cooled down is too frequent. Allow more time for the CPU to cool down by increasing the polling interval to 5 seconds by defaut. This value is used in the absence of the 'polling-delay' devicetree property. Signed-off

[PATCH 4/5] thermal: imx_tmu: Fix th temperature unit

2023-08-23 Thread Fabio Estevam
From: Fabio Estevam The temperature unit is millidegree Celsius, so divide by 1000 to correctly print the temperature values in Celsius. While at it, also change a typo: "has beyond" to "is beyond". Signed-off-by: Fabio Estevam --- drivers/thermal/imx_tmu.c | 4 ++-

[PATCH 3/5] thermal: imx_tmu: Select LOG

2023-08-23 Thread Fabio Estevam
From: Fabio Estevam Without LOG being selected, the dev_info() information from the imx_tmu driver cannot be printed. Select the LOG option, so that no dev_info() messages are lost. Signed-off-by: Fabio Estevam --- drivers/thermal/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git

[PATCH 2/5] thermal: imx_tmu: Fix the polling default

2023-08-23 Thread Fabio Estevam
From: Fabio Estevam When the 'polling-delay' property is not passed via devicetree, pdata->polling_delay keeps at 0. This causes the imx_tmu driver to get stuck inside the busy while() loop when the CPU temperature is above the alert point. Fix this problem by passing a one second poll

[PATCH 1/5] imx8mm_evk_defconfig: Select CONFIG_IMX_TMU

2023-08-23 Thread Fabio Estevam
From: Fabio Estevam Select the i.MX8MM thermal driver as it is useful for displaying the CPU temperature and its grading: CPU: Commercial temperature grade (0C to 95C) at 38C It also prevents booting when the temperature is above the alert point. Signed-off-by: Fabio Estevam --- configs

Re: [PATCH] spl: watchdog: introduce SPL_HW_WATCHDOG

2023-08-22 Thread Fabio Estevam
Hi Oleksandr, On Tue, Aug 22, 2023 at 12:53 PM Oleksandr Suvorov wrote: > ddr-1d-imem-fw { > - filename = "lpddr4_pmu_train_1d_imem_202006.bin"; > + filename = "lpddr4_pmu_train_1d_imem.bin"; > type =

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

2023-08-22 Thread Fabio Estevam
Hi Rogan, On Tue, Aug 22, 2023 at 6:42 AM Rogan Dawes wrote: > > Hi Fabio, > > As far as I am aware, the U-boot source for the Wink hub has not been > made available anywhere. That said, the RAM chip in use on the board Was there a formal request for the U-Boot source code to Wink? As per the

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

2023-08-21 Thread Fabio Estevam
Hi Rogan, On Fri, Aug 18, 2023 at 10:45 AM Rogan Dawes wrote: > > Circling back to this, I wanted to get a modern u-boot running on the > Wink Hub v1 as well. Since it is possible to invoke SDP over USB > (having soldered on a suitable microUSB connector), I have no concern > about bricking it

[PATCH] doc: csf_examples: csf.sh: Remove unneeded export ATF_LOAD_ADDR line

2023-08-15 Thread Fabio Estevam
From: Fabio Estevam Originally, exporting the ATF_LOAD_ADDR was required, but since binman has been used to generate the flash.bin, it is no longer needed to do such manual export. The ATF address is now passed via binman. Remove the unneeded export ATF_LOAD_ADDR line. Signed-off

Re: [PATCH v2] arm: imx: imx8m: imx9: Fix DRAM size calculation due to rom_pointer

2023-08-08 Thread Fabio Estevam
condition to prevent using rom_pointer in SPL. Signed-off-by: Elena Popa --- V2: Added the extra condition also in dram_init() and get_effective_memsize() for consistency. Thanks for the fix. Reviewed-by: Fabio Estevam

[PATCH 3/3] smegw01: Convert to watchdog driver model

2023-08-08 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 2/3] mx7dsabresd: Convert to watchdog driver model

2023-08-08 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/3] imx7d-sdb-u-boot: Pass SPDX-License tag

2023-08-08 Thread Fabio Estevam
From: Fabio Estevam SPDX-License tag is missing and checkpatch complains about it. Add the SPDX-License tag using the same one from imx7d-sdb.dts. Signed-off-by: Fabio Estevam --- arch/arm/dts/imx7d-sdb-u-boot.dtsi | 2 ++ 1 file changed, 2 insertions(+) diff --git a/arch/arm/dts/imx7d-sdb

Re: [PATCH 3/6] ARM: imx: Add weak default reset_cpu()

2023-08-08 Thread Fabio Estevam
Hi Marek, On Mon, Jun 26, 2023 at 5:53 AM Marek Vasut wrote: > > Add weak default reset_cpu() implementation needed by e.g. panic(). > > Signed-off-by: Marek Vasut > --- > Cc: "NXP i.MX U-Boot Team" > Cc: Fabio Estevam > Cc: Stefano Babic > --- > ar

[PATCH 2/2] doc: imx8mp_evk: Use in-tree build in the example

2023-08-07 Thread Fabio Estevam
From: Fabio Estevam To make it consistent with the instructions from other NXP imx8m boards, such as imx8mm-evk and imx8mn-evk, use U-Boot in-tree build in the examples. Signed-off-by: Fabio Estevam --- doc/board/nxp/imx8mp_evk.rst | 16 +--- 1 file changed, 9 insertions(+), 7

[PATCH 1/2] doc: imx8mp_evk: Remove unneeded export ATF_LOAD_ADDR line

2023-08-07 Thread Fabio Estevam
From: Fabio Estevam Originally, exporting the ATF_LOAD_ADDR was required, but since binman has been used to generate the flash.bin, it is no longer needed to do such manual export. The ATF address is now passed via binman in imx8mp-u-boot.dtsi: atf { description = &quo

Re: [PATCH 0/4] imx93: add DWC EQoS MAC driver support

2023-08-01 Thread Fabio Estevam
; u-boot=> ping 192.168.1.51 > Using ethernet@428a device > host 192.168.1.51 is alive > u-boot=> Thanks for the series: Reviewed-by: Fabio Estevam

Re: [PATCH] clk: also return NULL for -ENOENT in devm_clk_get_optional functions

2023-07-24 Thread Fabio Estevam
On Mon, Jul 24, 2023 at 2:32 PM Yang Xiwen via B4 Relay wrote: > > From: Yang Xiwen > > As described by the doc. Which doc? Please mention the details in the commit log.

Re: [PATCH 1/1] dm: adc: imx93-adc depends on ADC (fix boot)

2023-07-24 Thread Fabio Estevam
e7ff54d96303 ("imx93_evk: defconfig: add adc > support") CONFIG_ADC_IMX93 is enabled but CONFIG_ADC is not. > Fix this by making CONFIG_ADC_IMX93 depend on CONFIG_ADC. > > Signed-off-by: Sébastien Szymanski Reviewed-by: Fabio Estevam

Re: patch to submit

2023-07-20 Thread Fabio Estevam
Hi Mark, On Thu, Jul 20, 2023 at 1:55 PM mark roths wrote: > > Avalanche Technology has developed a new MRAM device (MTD). We would like > to submit a patch to u-boot including the Avalanche jedec memory identifier > and some code to allow boot from this device. > > Please point me to the

[PATCH] mxs: Kconfig: Remove TARGET_XFI3 symbol

2023-07-18 Thread Fabio Estevam
From: Fabio Estevam The xfi3 target has been removed by commit 539fba2c10e7 ("arm: Remove xfi3 board"), but it missed to remove an entry from the mxs Kconfig. Remove it. Signed-off-by: Fabio Estevam --- arch/arm/mach-imx/mxs/Kconfig | 3 --- 1 file changed, 3 deletions(-) diff --

[PATCH] mx7dsabresd: Retrieve the second MAC address from fuses

2023-07-11 Thread Fabio Estevam
From: Fabio Estevam Currently, a random MAC address is assigned to eth1 in Linux. Fix this behavor by retrieving the second MAC address from the fuses. Signed-off-by: Fabio Estevam --- board/freescale/mx7dsabresd/mx7dsabresd.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/board

[PATCH 3/3] mx23_olinuxino: Convert to CONFIG_DM_SERIAL

2023-07-11 Thread Fabio Estevam
From: Fabio Estevam The conversion to CONFIG_DM_SERIAL is mandatory, so select this option. Signed-off-by: Fabio Estevam --- arch/arm/Kconfig | 1 - arch/arm/mach-imx/mxs/Kconfig| 1 + configs/mx23_olinuxino_defconfig | 1 + 3 files changed, 2 insertions(+), 1 deletion

[PATCH 2/3] mx23evk: Convert to CONFIG_DM_SERIAL

2023-07-11 Thread Fabio Estevam
From: Fabio Estevam The conversion to CONFIG_DM_SERIAL is mandatory, so select this option. Signed-off-by: Fabio Estevam --- arch/arm/Kconfig | 2 +- arch/arm/mach-imx/mxs/Kconfig | 1 + configs/mx23evk_defconfig | 1 + 3 files changed, 3 insertions(+), 1 deletion(-) diff

[PATCH 1/3] mx28evk: Convert to CONFIG_DM_SERIAL

2023-07-11 Thread Fabio Estevam
From: Fabio Estevam The conversion to CONFIG_DM_SERIAL is mandatory, so select this option. Signed-off-by: Fabio Estevam --- arch/arm/Kconfig | 1 - arch/arm/mach-imx/mxs/Kconfig | 1 + configs/mx28evk_defconfig | 2 +- 3 files changed, 2 insertions(+), 2 deletions(-) diff

Re: R: [RFC] rockchip: rk3308: fix "same-as-spl" bug in boot devices order

2023-07-11 Thread Fabio Estevam
On Tue, Jul 11, 2023 at 5:04 PM Tom Rini wrote: > $ git grep -l bootph arch/arm64/ > arch/arm64/boot/dts/xilinx/zynqmp-clk-ccf.dtsi > arch/arm64/boot/dts/xilinx/zynqmp-sm-k26-revA.dts > arch/arm64/boot/dts/xilinx/zynqmp.dtsi > $ git describe HEAD > v6.5-rc1-6-g3f01e9fed845 but where is bootph

Re: R: [RFC] rockchip: rk3308: fix "same-as-spl" bug in boot devices order

2023-07-11 Thread Fabio Estevam
On Tue, Jul 11, 2023 at 2:14 PM Tom Rini wrote: > One thing I would say is that the bootph changes can go to the upstream > kernel dts files, so that they aren't lost in the future, and we can Are you sure? I don't see bootph devicetree properties being supported in 6.5-rc1.

[PATCH] imx7d-sdb: Sync devicetree with kernel 6.5-rc1

2023-07-11 Thread Fabio Estevam
From: Fabio Estevam Commit 0aea5dda2928 ("ARM: dts: imx7d-sdb-u-boot: Fix usdhc1 UHS operation") did a temporary workaround by addng the usdhc1 fix inside the -u-boot.dtsi file. The imx7d-sdb.dts from kernel 6.5-rc1 contains the proper UHS fix, so sync the devicetree tree

Re: imx8mp: issues getting dtb with pcie enabled to boot

2023-07-09 Thread Fabio Estevam
On Fri, Jul 7, 2023 at 8:48 PM Sahaj Sarup wrote: > For now the fix seems to be `clocks = <_blk_ctrl 0>;` Please submit the fix to Linux. > this gets pcie working under linux but not under u-boot, i'm guessing pcie > for imx8 is not yet implemented? That's correct.

Re: imx8mp: issues getting dtb with pcie enabled to boot

2023-07-07 Thread Fabio Estevam
Adding Marcel and Francesco. On Fri, Jul 7, 2023 at 5:30 PM Sahaj Sarup wrote: > > Hi, > I am working on the imxmp-verdin platform. One of the quirks with > this particular som is that it uses an internal pcie_phy clock instead of > an external one like most imx8mp boards do. > To get this

Re: [bug report] sunxi: H6: no ethernet on Orange Pi One Plus

2023-07-07 Thread Fabio Estevam
On Fri, Jul 7, 2023 at 10:46 AM Anne Macedo wrote: > On u-boot shell: > > gpio set pd6 Good point. You should pass the following options: CONFIG_DM_REGULATOR=y CONFIG_DM_REGULATOR_FIXED=y so that the GPIO3_6 turns on by default and then you would not need to run the "gpio set pd6" command. >

Re: [bug report] sunxi: H6: no ethernet on Orange Pi One Plus

2023-07-05 Thread Fabio Estevam
On Wed, Jul 5, 2023 at 1:31 PM Anne Macedo wrote: > I think I'm on the right path :) > > 1. Included SUNXI_SETUP_REGULATORS=0 to the bl31 make > 2. Changed the phy mode on arch/arm/dts/sun50i-h6-orangepi-one-plus.dts >From rgmii-id to rgmii This does not look like the correct fix. Please

Re: [bug report] sunxi: H6: no ethernet on Orange Pi One Plus

2023-07-05 Thread Fabio Estevam
Hi Anne, On Tue, Jul 4, 2023 at 8:52 PM Anne Macedo wrote: > > Hey! > > I'm trying to bake Linux images for the Orange Pi One Plus using Yocto. > Everything works fine, except for Ethernet. > > On the u-boot prompt: > > => dhcp > No ethernet found. > > After adding: > > CONFIG_SPL_SPI_SUNXI=y >

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

2023-07-04 Thread Fabio Estevam
On 04/07/2023 14:04, Francesco Dolcini wrote: The boards that do not check the return value might start to behave wrongly without an obvious error to help the debugging. Yes, the current implementation of fdt_status_disabled() is fragile, but there's not so much we can do for the upcoming

[PATCH v2] colibri-imx7: Call fdt_increase_size()

2023-07-04 Thread Fabio Estevam
From: Fabio Estevam For changing the USB OTG node status from "okay" to "disabled" more space is needed, so call fdt_increase_size() to avoid the following error: ``` U-Boot 2023.07-rc5-0.0.0-devel+git.580eb31199be (Jun 27 2023 - 13:39:58 +) CPU: Freescale i.MX7S rev

[PATCH] colibri-imx7: Call fdt_increase_size()

2023-07-04 Thread Fabio Estevam
From: Fabio Estevam For changing the USB OTG node status from "okay" to "disabled" more space is needed, so call fdt_increase_size() to avoid the following error: ``` U-Boot 2023.07-rc5-0.0.0-devel+git.580eb31199be (Jun 27 2023 - 13:39:58 +) CPU: Freescale i.MX7S rev

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

2023-07-04 Thread Fabio Estevam
On Tue, Jul 4, 2023 at 12:12 PM Francesco Dolcini wrote: > Now that the situation is pretty much clear I am not overly concerned for > colibri-imx7s. > > Do we consider this something to be worried about for other boards? There are only three boards that check the return value from

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

2023-07-04 Thread Fabio Estevam
Hi Francesco, On Mon, Jul 3, 2023 at 5:49 PM Francesco Dolcini wrote: > If I do this small partial revert > > --- a/arch/arm/dts/imx7d-colibri-eval-v3-u-boot.dtsi > +++ b/arch/arm/dts/imx7d-colibri-eval-v3-u-boot.dtsi > @@ -15,7 +15,8 @@ > pinctrl-0 = <_lcdif_dat >

Re: [PATCH v2] ARM: imx: romapi: Fix signed integer bitwise ops misuse

2023-07-02 Thread Fabio Estevam
On 01/07/2023 22:03, Marek Vasut wrote: Bitwise operations on signed integers are not defined, replace them with per-call checks. Reviewed-by: Peng Fan Signed-off-by: Marek Vasut Reviewed-by: Fabio Estevam

[PATCH v2 3/3] 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

[PATCH v2 2/3] microblaze: u-boot-spl.lds: Pass _image_binary_end

2023-06-30 Thread Fabio Estevam
From: Fabio Estevam Pass _image_binary_end to make a standard way to indicate the end of the text section in SPL. The motivation for this is to have a uniform way to handle the SPL boundary checks. Signed-off-by: Fabio Estevam --- Changes since v1: - Newly introduced to avoid build failure

[PATCH v2 1/3] sunxi: u-boot-spl.lds: Pass _image_binary_end

2023-06-30 Thread Fabio Estevam
From: Fabio Estevam Pass _image_binary_end to make a standard way to indicate the end of the text section in SPL. The motivation for this is to have a uniform way to handle the SPL boundary checks. Signed-off-by: Fabio Estevam --- Changes since v1: - Newly introduced to avoid build failure

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

2023-06-30 Thread Fabio Estevam
On Fri, Jun 30, 2023 at 6:04 PM Tom Rini wrote: > Yeah, just hit re-run when that happens, sadly. Ok, only sunxi failed. If I try: --- a/arch/arm/cpu/arm926ejs/sunxi/u-boot-spl.lds +++ b/arch/arm/cpu/arm926ejs/sunxi/u-boot-spl.lds @@ -37,6 +37,8 @@ SECTIONS __image_copy_end = .;

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

2023-06-30 Thread Fabio Estevam
On Fri, Jun 30, 2023 at 4:11 PM Tom Rini wrote: > Probably? But have you thrown this at CI, or a world build? I threw this at CI, but it failed at test.py qemu_m68k: https://github.com/u-boot/u-boot/pull/327#partial-pull-merging

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

2023-06-30 Thread Fabio Estevam
On Fri, Jun 30, 2023 at 3:36 PM Fabio Estevam wrote: > I tested this with CONFIG_SPL_SEPARATE_BSS=n and it failed. Tested this one with CONFIG_SPL_SEPARATE_BSS enabled and disabled: --- a/common/spl/spl_legacy.c +++ b/common/spl/spl_legacy.c @@ -18,9 +18,17 @@ static v

Re: u-boot compilation failure for Sifive unmatched board

2023-06-30 Thread Fabio Estevam
Hi Satish, On Fri, Jun 30, 2023 at 3:21 PM satish kumar wrote: > > Hi all > This is Satish, compiling u-boot code based on the reference page: > https://github.com/carlosedp/riscv-bringup/blob/master/unmatched/Readme.md#install-toolchain-to-build-kernel > > u-boot is failing with following

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