All of lore.kernel.org
 help / color / mirror / Atom feed
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: Re: [GIT PULL] clk: thead: Updates for v6.17, part 2
Date: Thu, 24 Jul 2025 14:11:27 -0700	[thread overview]
Message-ID: <aIKhf41ky1gzdOIV@x1> (raw)
In-Reply-To: <aIKcFzsCvvz0PCBW@x1>

On Thu, Jul 24, 2025 at 01:48:23PM -0700, Drew Fustini wrote:
> 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.
[snip]
> 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(-)

My apologies, I failed to just send the delta between thead-clk-for-v6.17 and
thead-clk-for-v6.17-p2.

The following is the correct text of the pull request. Hopefull this is
acceptable?

Thanks,
Drew

--
The following changes since commit d274c77ffa202b70ad01d579f33b73b4de123375:

  clk: thead: th1520-ap: Correctly refer the parent of osc_12m (2025-07-13 12:46:58 -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>

----------------------------------------------------------------
Yao Zi (1):
      clk: thead: th1520-ap: Describe mux clocks with clk_mux

 drivers/clk/thead/clk-th1520-ap.c | 95 +++++++++++++++------------------------
 1 file changed, 37 insertions(+), 58 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: Re: [GIT PULL] clk: thead: Updates for v6.17, part 2
Date: Thu, 24 Jul 2025 14:11:27 -0700	[thread overview]
Message-ID: <aIKhf41ky1gzdOIV@x1> (raw)
In-Reply-To: <aIKcFzsCvvz0PCBW@x1>

On Thu, Jul 24, 2025 at 01:48:23PM -0700, Drew Fustini wrote:
> 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.
[snip]
> 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(-)

My apologies, I failed to just send the delta between thead-clk-for-v6.17 and
thead-clk-for-v6.17-p2.

The following is the correct text of the pull request. Hopefull this is
acceptable?

Thanks,
Drew

--
The following changes since commit d274c77ffa202b70ad01d579f33b73b4de123375:

  clk: thead: th1520-ap: Correctly refer the parent of osc_12m (2025-07-13 12:46:58 -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>

----------------------------------------------------------------
Yao Zi (1):
      clk: thead: th1520-ap: Describe mux clocks with clk_mux

 drivers/clk/thead/clk-th1520-ap.c | 95 +++++++++++++++------------------------
 1 file changed, 37 insertions(+), 58 deletions(-)


_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv

  reply	other threads:[~2025-07-24 21:11 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-07-24 20:48 [GIT PULL] clk: thead: Updates for v6.17, part 2 Drew Fustini
2025-07-24 20:48 ` Drew Fustini
2025-07-24 21:11 ` Drew Fustini [this message]
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=aIKhf41ky1gzdOIV@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.