From: "Mathieu Dubois-Briand" <mathieu.dubois-briand@bootlin.com>
To: "Andreas Kemnade" <andreas@kemnade.info>
Cc: "Tero Kristo" <kristo@kernel.org>,
"Michael Turquette" <mturquette@baylibre.com>,
"Stephen Boyd" <sboyd@kernel.org>,
"Brian Masney" <bmasney@redhat.com>,
"Tony Lindgren" <tony@atomide.com>,
"Thomas Petazzoni" <thomas.petazzoni@bootlin.com>,
"Théo Lebrun" <theo.lebrun@bootlin.com>,
"Grégory Clement" <gregory.clement@bootlin.com>,
linux-omap@vger.kernel.org, linux-clk@vger.kernel.org,
linux-kernel@vger.kernel.org, hns@goldelico.com
Subject: Re: [PATCH v2 0/2] clk: ti: mux: resolve parent clocks by DT index, not by name
Date: Fri, 04 Sep 2026 09:50:17 +0200 [thread overview]
Message-ID: <DL6D90JSIJCC.FE91IM9ILYSI@bootlin.com> (raw)
In-Reply-To: <20260903115308.2a569a30@kemnade.info>
On Thu Sep 3, 2026 at 11:53 AM CEST, Andreas Kemnade wrote:
> We have:
> localhost:/sys/kernel/debug/clk# grep '(' */clk_possible_parents
> clkout2_src_ck/clk_possible_parents:(missing) (missing) (missing) (missing)
> gpt10_fck/clk_possible_parents:(missing) (missing)
> gpt11_fck/clk_possible_parents:(missing) (missing)
> gpt2_fck/clk_possible_parents:(missing) (missing)
> gpt3_fck/clk_possible_parents:(missing) (missing)
> gpt4_fck/clk_possible_parents:(missing) (missing)
> gpt5_fck/clk_possible_parents:(missing) (missing)
> gpt6_fck/clk_possible_parents:(missing) (missing)
> gpt7_fck/clk_possible_parents:(missing) (missing)
> gpt8_fck/clk_possible_parents:(missing) (missing)
> gpt9_fck/clk_possible_parents:(missing) (missing)
> mcbsp1_fck/clk_possible_parents:(missing) (missing)
> mcbsp2_fck/clk_possible_parents:(missing) (missing)
> mcbsp3_fck/clk_possible_parents:(missing) (missing)
> mcbsp4_fck/clk_possible_parents:(missing) (missing)
> mcbsp5_fck/clk_possible_parents:(missing) (missing)
> sgx_fck/clk_possible_parents:(missing) (missing) (missing) (missing) (missing) (missing) (missing) (missing)
> usim_fck/clk_possible_parents:(missing) (missing) (missing) (missing) (missing) (missing) (missing) (missing) (missing) (missing)
> localhost:/sys/kernel/debug/clk#
>
> All these clocks are composite clocks.
>
> so the early-registration of timer-ti-dm-systimer seems not to be
> the main issue here. Looking around what might be affected,
> mux clocks build into composite clocks:
> :~/linux/arch/arm/boot/dts/ti/omap$ grep -l ti,composite-mux *.dts*
> omap2420-clocks.dtsi
> omap2430-clocks.dtsi
> omap24xx-clocks.dtsi
> omap36xx-am35xx-omap3430es2plus-clocks.dtsi
> omap36xx-omap3430es2plus-clocks.dtsi
> omap3xxx-clocks.dtsi
> omap44xx-clocks.dtsi
> omap54xx-clocks.dtsi
>
> So havoc all over the place, just worst in omap3 because more critical
> clocks are affected.
>
> So I think we need either a revert of these patches or a some
> kind of quirk in clk_hw_register_composite_pdata to pass throuch something
> between the mux component and the composite.
>
> Regrads,
> Andreas
Hi Andreas,
Thanks for taking the time to investigate this issue.
I believe I won't have much time to dive into it in the coming days, so
I fear you are right and we will have to revert this patch for now.
Except if someone else want to have a look at this.
I will try to get some OMAP3 board and come back to it a bit later.
Thanks,
Mathieu
--
Mathieu Dubois-Briand, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
next prev parent reply other threads:[~2026-09-04 7:50 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-07-27 7:41 [PATCH v2 0/2] clk: ti: mux: resolve parent clocks by DT index, not by name Mathieu Dubois-Briand
2026-07-27 7:41 ` [PATCH v2 1/2] " Mathieu Dubois-Briand
2026-07-28 14:11 ` Brian Masney
2026-08-14 5:43 ` Stephen Boyd
2026-08-18 16:02 ` Geert Uytterhoeven
2026-08-18 21:13 ` Brian Masney
2026-08-19 5:39 ` Mathieu Dubois-Briand
2026-08-19 12:32 ` Geert Uytterhoeven
2026-08-19 15:12 ` Mathieu Dubois-Briand
2026-08-19 19:13 ` Geert Uytterhoeven
2026-08-19 21:25 ` Brian Masney
2026-07-27 7:41 ` [PATCH v2 2/2] clk: ti: composite: " Mathieu Dubois-Briand
2026-07-28 14:11 ` Brian Masney
2026-08-14 5:43 ` Stephen Boyd
2026-09-02 6:26 ` [PATCH v2 0/2] clk: ti: mux: " Andreas Kemnade
2026-09-02 8:30 ` Mathieu Dubois-Briand
2026-09-02 9:15 ` Andreas Kemnade
2026-09-02 22:26 ` Andreas Kemnade
2026-09-03 9:53 ` Andreas Kemnade
2026-09-04 7:50 ` Mathieu Dubois-Briand [this message]
2026-09-04 13:41 ` Andreas Kemnade
2026-09-02 8:34 ` Andreas Kemnade
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=DL6D90JSIJCC.FE91IM9ILYSI@bootlin.com \
--to=mathieu.dubois-briand@bootlin.com \
--cc=andreas@kemnade.info \
--cc=bmasney@redhat.com \
--cc=gregory.clement@bootlin.com \
--cc=hns@goldelico.com \
--cc=kristo@kernel.org \
--cc=linux-clk@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-omap@vger.kernel.org \
--cc=mturquette@baylibre.com \
--cc=sboyd@kernel.org \
--cc=theo.lebrun@bootlin.com \
--cc=thomas.petazzoni@bootlin.com \
--cc=tony@atomide.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.