mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Enzo Adriano <enzo.adriano.code@gmail.com>
To: junhui.liu@pigmoral.tech
Cc: Stephen Boyd <sboyd@kernel.org>,
	Brian Masney <bmasney+clk@redhat.com>,
	Jerome Brunet <jbrunet+clk@baylibre.com>,
	Chen-Yu Tsai <wens@kernel.org>,
	Jernej Skrabec <jernej.skrabec@gmail.com>,
	Samuel Holland <samuel@sholland.org>,
	Andre Przywara <andre.przywara@arm.com>,
	linux-clk@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	linux-sunxi@lists.linux.dev, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v4 7/8] clk: sunxi-ng: a733: Add bus clock gates
Date: Fri, 25 Sep 2026 22:03:13 -0400	[thread overview]
Message-ID: <20260926020313.911537-1-enzo.adriano.code@gmail.com> (raw)
In-Reply-To: <DLLYUYMR3CZT.17I8APGBH9H8S@pigmoral.tech>

Hi Junhui,

Thanks for reproducing the ahb-cpus issue. Adding clk_ignore_unused lets
the Cubie A7S reach an interactive initramfs shell.

I reused the September 21 uninstrumented comparator: v7.3-rc1 plus the
four single-divider patches, CCU v4, the seven pinctrl patches, and the
minimal test DT, with only ahb_cpus_clk marked CLK_IS_CRITICAL. The Image,
DTB and initramfs were byte-identical to that earlier comparator; the
only boot-argument change was adding clk_ignore_unused.

Without the flag, that comparator stopped producing serial output at
"clk: Disabling unused clocks". With it, the confirmation run recorded:

  [    1.263452] clk: Not disabling unused clocks
  [    1.292364] Run /bin/sh as init process
  [    1.526500] mmc0: new high speed SDXC card at address 544c

Commands issued after the shell prompt returned the expected readiness
and completion markers. No panic, oops or SError appeared in the capture.
The RAM-only run was followed by a guarded 120-second power-off recovery,
which restored the vendor kernel and eMMC root.

This narrows the second failure to the path bypassed by clk_ignore_unused,
but does not identify an individual clock: the flag also bypasses the
runtime-PM acquisition and traversal before the per-clock disable calls.
I therefore still cannot add a Tested-by for v4 as posted.

Assisted-by: Codex:gpt-6

Regards,
Enzo

  reply	other threads:[~2026-09-26  2:03 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-09-21 15:37 [PATCH v4 0/8] clk: sunxi-ng: Add support for Allwinner A733 CCU and PRCM Junhui Liu
2026-09-21 15:37 ` [PATCH v4 1/8] dt-bindings: clk: sun60i-a733-ccu: Add Allwinner A733 support Junhui Liu
2026-09-21 15:38 ` [PATCH v4 2/8] clk: sunxi-ng: sdm: Add dual patterns support Junhui Liu
2026-09-21 16:01   ` Chen-Yu Tsai
2026-09-21 15:38 ` [PATCH v4 3/8] clk: sunxi-ng: a733: Add PRCM CCU Junhui Liu
2026-09-21 15:38 ` [PATCH v4 4/8] clk: sunxi-ng: a733: Add PLL clocks support Junhui Liu
2026-09-21 15:38 ` [PATCH v4 5/8] clk: sunxi-ng: a733: Add bus " Junhui Liu
2026-09-21 15:38 ` [PATCH v4 6/8] clk: sunxi-ng: a733: Add mod " Junhui Liu
2026-09-21 15:38 ` [PATCH v4 7/8] clk: sunxi-ng: a733: Add bus clock gates Junhui Liu
2026-09-21 20:05   ` Enzo Adriano
2026-09-22 15:56     ` Junhui Liu
2026-09-26  2:03       ` Enzo Adriano [this message]
2026-09-26  3:51         ` Junhui Liu
2026-09-26 21:02           ` Enzo Adriano
2026-09-21 15:38 ` [PATCH v4 8/8] clk: sunxi-ng: a733: Add reset lines Junhui Liu

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=20260926020313.911537-1-enzo.adriano.code@gmail.com \
    --to=enzo.adriano.code@gmail.com \
    --cc=andre.przywara@arm.com \
    --cc=bmasney+clk@redhat.com \
    --cc=jbrunet+clk@baylibre.com \
    --cc=jernej.skrabec@gmail.com \
    --cc=junhui.liu@pigmoral.tech \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-clk@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-sunxi@lists.linux.dev \
    --cc=samuel@sholland.org \
    --cc=sboyd@kernel.org \
    --cc=wens@kernel.org \
    /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

all inboxes | Powered by JetHome®