From: Drew Fustini <fustini@kernel.org>
To: Stephen Boyd <sboyd@kernel.org>
Cc: linux-clk@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-riscv@lists.infradead.org, Yao Zi <ziyao@disroot.org>
Subject: [GIT PULL] clk: thead: Updates for v6.17, part 2
Date: Thu, 24 Jul 2025 13:48:23 -0700 [thread overview]
Message-ID: <aIKcFzsCvvz0PCBW@x1> (raw)
Hi Stephen,
I'm sending this followup PR in case there is still time to include it
for your 6.17 PR. Yao Zi refactored mux clk registration which fixes an
orphan mux clk issue which would occur when booting with mainline uboot.
Thanks,
Drew
The following changes since commit 19272b37aa4f83ca52bdf9c16d5d81bdd1354494:
Linux 6.16-rc1 (2025-06-08 13:44:43 -0700)
are available in the Git repository at:
git://git.kernel.org/pub/scm/linux/kernel/git/fustini/linux.git tags/thead-clk-for-v6.17-p2
for you to fetch changes up to 54edba916e2913b0893b0f6404b73155d48374ea:
clk: thead: th1520-ap: Describe mux clocks with clk_mux (2025-07-22 15:40:54 -0700)
----------------------------------------------------------------
T-HEAD clock changes for v6.17, part 2
One more improvement for the T-HEAD TH1520 clock controller this cycle:
Yao Zi has fixed an issue where the c910 mux clk could end up as an
orphan in CCF when the bootloader reparents it to the c910-i0 mux clk.
The solution is to refactor the handling of mux clocks by embedding a
clk_mux structure directly in ccu_mux. This allows the mux clocks to be
registered with devm_clk_hw_register() without allocating any new clk_hw
pointer which solves the orphan issue.
This change has been tested in linux-next. The LPi4a still boots okay
without clk_ignore_unused and peripherals like serial, emmc and ethernet
are functional. The file /sys/kernel/debug/clk/c910/clk_possible_parents
now correctly outputs: "c910-i0 cpu-pll1"
Link: https://lore.kernel.org/linux-riscv/20250722080535.60027-2-ziyao@disroot.org/
Signed-off-by: Drew Fustini <fustini@kernel.org>
----------------------------------------------------------------
Michal Wilczynski (1):
clk: thead: Mark essential bus clocks as CLK_IGNORE_UNUSED
Yao Zi (2):
clk: thead: th1520-ap: Correctly refer the parent of osc_12m
clk: thead: th1520-ap: Describe mux clocks with clk_mux
drivers/clk/thead/clk-th1520-ap.c | 109 +++++++++++++++++---------------------
1 file changed, 48 insertions(+), 61 deletions(-)
WARNING: multiple messages have this Message-ID (diff)
From: Drew Fustini <fustini@kernel.org>
To: Stephen Boyd <sboyd@kernel.org>
Cc: linux-clk@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-riscv@lists.infradead.org, Yao Zi <ziyao@disroot.org>
Subject: [GIT PULL] clk: thead: Updates for v6.17, part 2
Date: Thu, 24 Jul 2025 13:48:23 -0700 [thread overview]
Message-ID: <aIKcFzsCvvz0PCBW@x1> (raw)
Hi Stephen,
I'm sending this followup PR in case there is still time to include it
for your 6.17 PR. Yao Zi refactored mux clk registration which fixes an
orphan mux clk issue which would occur when booting with mainline uboot.
Thanks,
Drew
The following changes since commit 19272b37aa4f83ca52bdf9c16d5d81bdd1354494:
Linux 6.16-rc1 (2025-06-08 13:44:43 -0700)
are available in the Git repository at:
git://git.kernel.org/pub/scm/linux/kernel/git/fustini/linux.git tags/thead-clk-for-v6.17-p2
for you to fetch changes up to 54edba916e2913b0893b0f6404b73155d48374ea:
clk: thead: th1520-ap: Describe mux clocks with clk_mux (2025-07-22 15:40:54 -0700)
----------------------------------------------------------------
T-HEAD clock changes for v6.17, part 2
One more improvement for the T-HEAD TH1520 clock controller this cycle:
Yao Zi has fixed an issue where the c910 mux clk could end up as an
orphan in CCF when the bootloader reparents it to the c910-i0 mux clk.
The solution is to refactor the handling of mux clocks by embedding a
clk_mux structure directly in ccu_mux. This allows the mux clocks to be
registered with devm_clk_hw_register() without allocating any new clk_hw
pointer which solves the orphan issue.
This change has been tested in linux-next. The LPi4a still boots okay
without clk_ignore_unused and peripherals like serial, emmc and ethernet
are functional. The file /sys/kernel/debug/clk/c910/clk_possible_parents
now correctly outputs: "c910-i0 cpu-pll1"
Link: https://lore.kernel.org/linux-riscv/20250722080535.60027-2-ziyao@disroot.org/
Signed-off-by: Drew Fustini <fustini@kernel.org>
----------------------------------------------------------------
Michal Wilczynski (1):
clk: thead: Mark essential bus clocks as CLK_IGNORE_UNUSED
Yao Zi (2):
clk: thead: th1520-ap: Correctly refer the parent of osc_12m
clk: thead: th1520-ap: Describe mux clocks with clk_mux
drivers/clk/thead/clk-th1520-ap.c | 109 +++++++++++++++++---------------------
1 file changed, 48 insertions(+), 61 deletions(-)
_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv
next reply other threads:[~2025-07-24 20:48 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-07-24 20:48 Drew Fustini [this message]
2025-07-24 20:48 ` [GIT PULL] clk: thead: Updates for v6.17, part 2 Drew Fustini
2025-07-24 21:11 ` Drew Fustini
2025-07-24 21:11 ` Drew Fustini
2025-07-24 22:06 ` Stephen Boyd
2025-07-24 22:06 ` Stephen Boyd
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=aIKcFzsCvvz0PCBW@x1 \
--to=fustini@kernel.org \
--cc=linux-clk@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-riscv@lists.infradead.org \
--cc=sboyd@kernel.org \
--cc=ziyao@disroot.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 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.