All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Richard <thomas.richard@bootlin.com>
To: Nishanth Menon <nm@ti.com>
Cc: linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	Siddharth Vadapalli <s-vadapalli@ti.com>,
	Vignesh Raghavendra <vigneshr@ti.com>,
	Krzysztof Kozlowski <krzk@kernel.org>,
	Geert Uytterhoeven <geert+renesas@glider.be>,
	Catalin Marinas <catalin.marinas@arm.com>,
	Bjorn Andersson <andersson@kernel.org>,
	alexander.sverdlin@siemens.com, jan.kiszka@siemens.com,
	ayush@beagleboard.org, robertcnelson@gmail.com,
	upstream@lists.phytec.de, francesco.dolcini@toradex.com,
	leonardo.costa@toradex.com, matthias.schiffer@ew.tq-group.com,
	stefano.r@variscite.com, eblanc@baylibre.com, msp@baylibre.com,
	miquel.raynal@bootlin.com, mwalle@kernel.org
Subject: Re: [PATCH] arm64: configs: defconfig: Convert TISCI to modules
Date: Thu, 24 Sep 2026 19:00:02 +0200	[thread overview]
Message-ID: <1bd67ddc-4910-4f7f-a2fc-21d9c58cf655@bootlin.com> (raw)
In-Reply-To: <20260910134409.1713610-1-nm@ti.com>

Hi Nishanth,

On 9/10/26 3:44 PM, Nishanth Menon wrote:
> Now that all relevant drivers have are ready, convert the
> TISCI drivers to modules. This impacts all K3 platforms for
> boot, recommendation (based on empirical evidence) is to use
> deferred_probe_timeout=60 in kernel command line to allow for driver
> probes to complete successfully based on the initrd used.
> 
It's maybe a bit late, but I did test your patch (linux next-20260923)
on J7200. Indeed with default deferred_probe_timeout value I got some
deferred probe.

[   16.869217] platform 2830000.serial: deferred probe pending: (reason
unknown)
[   16.869241] platform 24c0000.timer: deferred probe pending: (reason
unknown)

Using deferred_probe_timeout=60 fixes sometimes deferred probe for
24c0000.timer, but never not for 2830000.serial.

[   68.582069] platform 2830000.serial: deferred probe pending: (reason
unknown)
[   68.582094] platform 24c0000.timer: deferred probe pending: (reason
unknown)
[   68.582128] ti_sci_pm_domains
44083000.system-controller:power-controller: sync_state() pending due to
24c0000.timer
[   68.582144] ti_sci_pm_domains
44083000.system-controller:power-controller: sync_state() pending due to
2830000.serial

But switching all TISCI drivers to 'y' fixed them. Looks like an
ordering issue.

I also did test suspend-to-ram, it works as expected. Fun fact, doing a
suspend-to-ram allows to probe these 2 missing devices during resume. I
guess it confirms a timing/ordering issue during the boot. At the end of
the resume, no device anymore in /sys/kernel/debug/devices_deferred.

[  124.053571] Restarting tasks: Starting
[  124.053861] Restarting tasks: Done
[  124.053907] random: crng reseeded on system resumption
[  124.053970] PM: suspend exit
[  124.058833] 2830000.serial: ttyS0 MMIO:0x0000000002830000 (irq = 700,
base_baud = 3000000) is a 8250

Best Regards,
Thomas


      parent reply	other threads:[~2026-09-24 17:00 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-09-10 13:44 [PATCH] arm64: configs: defconfig: Convert TISCI to modules Nishanth Menon
2026-09-22 14:24 ` Nishanth Menon
2026-09-24 17:00 ` Thomas Richard [this message]

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=1bd67ddc-4910-4f7f-a2fc-21d9c58cf655@bootlin.com \
    --to=thomas.richard@bootlin.com \
    --cc=alexander.sverdlin@siemens.com \
    --cc=andersson@kernel.org \
    --cc=ayush@beagleboard.org \
    --cc=catalin.marinas@arm.com \
    --cc=eblanc@baylibre.com \
    --cc=francesco.dolcini@toradex.com \
    --cc=geert+renesas@glider.be \
    --cc=jan.kiszka@siemens.com \
    --cc=krzk@kernel.org \
    --cc=leonardo.costa@toradex.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=matthias.schiffer@ew.tq-group.com \
    --cc=miquel.raynal@bootlin.com \
    --cc=msp@baylibre.com \
    --cc=mwalle@kernel.org \
    --cc=nm@ti.com \
    --cc=robertcnelson@gmail.com \
    --cc=s-vadapalli@ti.com \
    --cc=stefano.r@variscite.com \
    --cc=upstream@lists.phytec.de \
    --cc=vigneshr@ti.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.