public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Francesco Dolcini <francesco@dolcini.it>
To: Marek Vasut <marex@denx.de>, Fabio Estevam <festevam@denx.de>
Cc: u-boot@lists.denx.de, Adam Ford <aford173@gmail.com>,
	Christoph Niedermaier <cniedermaier@dh-electronics.com>,
	Dong Aisheng <aisheng.dong@nxp.com>,
	Hou Zhiqiang <Zhiqiang.Hou@nxp.com>,
	Michael Trimarchi <michael@amarulasolutions.com>,
	Peng Fan <peng.fan@nxp.com>, Tim Harvey <tharvey@gateworks.com>,
	Tom Rini <trini@konsulko.com>,
	uboot-imx@nxp.com
Subject: [REGRESSION] Re: [PATCH v2 16/24] clk: imx: Convert clock-osc-* back to osc_*
Date: Tue, 15 Apr 2025 16:28:46 +0200	[thread overview]
Message-ID: <20250415142846.GA244333@francesco-nb> (raw)
In-Reply-To: <20250323160107.145749-17-marex@denx.de>

Hello Marex, Fabio, all

On Sun, Mar 23, 2025 at 04:58:45PM +0100, Marek Vasut wrote:
> Convert clock-osc-24m back to osc_24m and clock-osc-32k back to osc_32k.
> These are the clock which match clock tables in Linux. This is now
> possible because the clock drivers now resolve clock names based on
> clock-names DT property in the CCM DT node.
> 
> Signed-off-by: Marek Vasut <marex@denx.de>

Current USB fastboot functionality from U-Boot proper in master is broken
on Verdin iMX8M Plus since commit b4734c9c333b ("clk: imx: Convert
clock-osc-* back to osc_*"). Other i.MX SoCs might be affected, I have
not looked into every detail.

  Could not fetch index
  Failed to get PHY0 for usb@38100000
  Could not fetch index
  Failed to get PHY0 for usb@38100000
  No USB device found
  USB init failed: -19

Reverting this commit on top of master is not trivial (there are
conflicts, and I have not looked into the details).

Full log for reference

  U-Boot SPL 2025.04-rc4-00537-gb4734c9c333b (Apr 15 2025 - 16:14:09 +0200)
  Training FAILED
  DDR configured as single rank
  WDT:   Started watchdog@30280000 with servicing every 1000ms (60s timeout)
  SEC0:  RNG instantiated
  Normal Boot
  Trying to boot from BOOTROM
  Boot Stage: USB boot
  Find img info 0x4802ec00, size 1116
  Need continue download 1024
  NOTICE:  Do not release JR0 to NS as it can be used by HAB
  NOTICE:  BL31: v2.12.0(release):v2.12.0-946-g4d913df8f85e
  NOTICE:  BL31: Built : 13:19:07, Apr 15 2025
  
  
  U-Boot 2025.04-rc4-00537-gb4734c9c333b (Apr 15 2025 - 16:14:09 +0200)
  
  CPU:   Freescale i.MX8MP[8] rev1.1 1600 MHz (running at 1200 MHz)
  CPU:   Industrial temperature grade (-40C to 105C) at 56C
  Reset cause: POR
  DRAM:  4 GiB
  Core:  309 devices, 31 uclasses, devicetree: separate
  WDT:   Started watchdog@30280000 with servicing every 1000ms (60s timeout)
  MMC:   FSL_SDHC: 1, FSL_SDHC: 2
  Loading Environment from nowhere... OK
  In:    serial@30880000
  Out:   serial@30880000
  Err:   serial@30880000
  Model: Toradex 0058 Verdin iMX8M Plus Quad 4GB WB IT V1.0B
  Serial#: 06849144
  Carrier: Toradex Dahlia V1.1A, Serial# 10763256
  SEC0:  RNG instantiated
  Setting variant to wifi
  Net:   pca953x gpio-expander@21: Error reading output register
  eth1: ethernet@30be0000, eth0: ethernet@30bf0000 [PRIME]
  Hit any key to stop autoboot:  0
  Could not fetch index
  Failed to get PHY0 for usb@38100000
  Could not fetch index
  Failed to get PHY0 for usb@38100000
  No USB device found
  USB init failed: -19
  Verdin iMX8MP #

Any suggestion?
Francesco


  reply	other threads:[~2025-04-15 14:28 UTC|newest]

Thread overview: 55+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-03-23 15:58 [PATCH v2 00/24] clk: Add clk_resolve_parent_clk() and fix up iMX clock drivers Marek Vasut
2025-03-23 15:58 ` [PATCH v2 01/24] clk: Add clk_resolve_parent_clk() Marek Vasut
2025-03-23 15:58 ` [PATCH v2 02/24] clk: clk-mux: Fold clk_register_mux() Marek Vasut
2025-03-23 15:58 ` [PATCH v2 03/24] clk: clk-mux: Use struct udevice instead of struct device Marek Vasut
2025-03-23 15:58 ` [PATCH v2 04/24] clk: clk-mux: Resolve parent clock by name Marek Vasut
2025-03-23 15:58 ` [PATCH v2 05/24] clk: imx: Pass struct udevice into imx_clk_mux*() Marek Vasut
2025-03-23 15:58 ` [PATCH v2 06/24] clk: imx: Pass struct udevice to clk_register_mux() Marek Vasut
2025-03-23 15:58 ` [PATCH v2 07/24] clk: clk-gate: Use struct udevice instead of struct device Marek Vasut
2025-03-23 15:58 ` [PATCH v2 08/24] clk: clk-gate: Resolve parent clock by name Marek Vasut
2025-03-23 15:58 ` [PATCH v2 09/24] clk: imx: gate2: Use struct udevice instead of struct device Marek Vasut
2025-03-23 15:58 ` [PATCH v2 10/24] clk: imx: gate2: Resolve parent clock by name Marek Vasut
2025-03-23 15:58 ` [PATCH v2 11/24] clk: imx: Pass struct udevice into imx_clk_gate*() Marek Vasut
2025-03-23 15:58 ` [PATCH v2 12/24] clk: imx: Pass struct udevice to clk_register_gate*() Marek Vasut
2025-03-23 15:58 ` [PATCH v2 13/24] clk: clk-composite: Use struct udevice instead of struct device Marek Vasut
2025-03-23 15:58 ` [PATCH v2 14/24] clk: clk-composite: Resolve parent clock by name Marek Vasut
2025-03-30 15:15   ` Heinrich Schuchardt
2025-03-30 16:14     ` Heinrich Schuchardt
2026-01-27  2:55       ` Simon Glass
2026-01-27  4:36         ` E Shattow
2026-02-04  0:23           ` Simon Glass
2026-02-05  3:14             ` E Shattow
2026-02-04  1:26           ` Heinrich Schuchardt
2025-03-23 15:58 ` [PATCH v2 15/24] clk: imx: Pass struct udevice into imx_clk_composite*() Marek Vasut
2025-03-23 15:58 ` [PATCH v2 16/24] clk: imx: Convert clock-osc-* back to osc_* Marek Vasut
2025-04-15 14:28   ` Francesco Dolcini [this message]
2025-04-15 14:43     ` [REGRESSION] " Adam Ford
2025-04-15 15:03       ` Francesco Dolcini
2025-04-15 16:50         ` Marek Vasut
2025-04-15 17:13           ` Fabio Estevam
2025-04-16  4:38             ` Adam Ford
2025-04-16  9:26             ` Francesco Dolcini
2025-04-16 14:18               ` Christoph Niedermaier
2025-04-16 22:34                 ` Adam Ford
2025-04-16 23:35                   ` Adam Ford
2025-04-16 23:47                     ` Marek Vasut
2025-04-16 23:58                       ` Fabio Estevam
2025-04-17  0:40                         ` Fabio Estevam
2025-04-17  7:24                         ` Marek Vasut
2025-04-17 10:51                           ` Adam Ford
2025-04-17 11:30                             ` Marek Vasut
2025-04-17 12:34                               ` Adam Ford
2025-04-18 14:49                                 ` Marek Vasut
2025-04-17 12:33                           ` Fabio Estevam
2025-03-23 15:58 ` [PATCH v2 17/24] clk: imx: Pass struct udevice into imx_clk_pllv3*() Marek Vasut
2025-03-23 15:58 ` [PATCH v2 18/24] clk: imx: pllv3: Resolve parent clock by name Marek Vasut
2025-03-23 15:58 ` [PATCH v2 19/24] clk: clk-divider: Use struct udevice instead of struct device Marek Vasut
2025-03-23 15:58 ` [PATCH v2 20/24] clk: imx: Pass struct udevice into imx_clk_divider*() Marek Vasut
2025-03-23 15:58 ` [PATCH v2 21/24] clk: clk-divider: Resolve parent clock by name Marek Vasut
2025-03-23 15:58 ` [PATCH v2 22/24] clk: clk-fixed-factor: Use struct udevice instead of struct device Marek Vasut
2025-03-23 15:58 ` [PATCH v2 23/24] clk: clk-fixed-factor: Resolve parent clock by name Marek Vasut
2025-03-23 15:58 ` [PATCH v2 24/24] clk: imx: Pass struct udevice into imx_clk_fixed_factor*() Marek Vasut
2025-03-24 23:33 ` [PATCH v2 00/24] clk: Add clk_resolve_parent_clk() and fix up iMX clock drivers Adam Ford
2025-03-24 23:41   ` Marek Vasut
2025-03-25  7:50 ` Peng Fan
2025-03-25 12:55 ` Fabio Estevam

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20250415142846.GA244333@francesco-nb \
    --to=francesco@dolcini.it \
    --cc=Zhiqiang.Hou@nxp.com \
    --cc=aford173@gmail.com \
    --cc=aisheng.dong@nxp.com \
    --cc=cniedermaier@dh-electronics.com \
    --cc=festevam@denx.de \
    --cc=marex@denx.de \
    --cc=michael@amarulasolutions.com \
    --cc=peng.fan@nxp.com \
    --cc=tharvey@gateworks.com \
    --cc=trini@konsulko.com \
    --cc=u-boot@lists.denx.de \
    --cc=uboot-imx@nxp.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox