Re: [PATCH 1/2] clk: thead: th1520-ap: describe gate clocks with clk_gate

From: Drew Fustini
Date: Sat Aug 16 2025 - 19:17:08 EST


On Sat, Aug 16, 2025 at 04:11:47PM +0800, Troy Mitchell wrote:
> On Thu, Aug 14, 2025 at 01:11:35AM +0800, Icenowy Zheng wrote:
> > Similar to previous situation of mux clocks, the gate clocks of
> > clk-th1520-ap drivers are also using a helper that creates a temporary
> > struct clk_hw and abandons the struct clk_hw in struct ccu_common, which
> > prevents clock gates to be clock parents.
> >
> > Do the similar refactor of dropping struct ccu_common and directly use
> > struct clk_gate here.
> >
> > This patch mimics the refactor done on struct ccu_mux at [1].
> >
> > [1] https://lore.kernel.org/all/20250722080535.60027-2-ziyao@xxxxxxxxxxx/
> Should we remove this link to the cover-letter?
> - Troy

Good point. Yes, I think this would be better than a link as it is now
in master:

54edba916e29 ("clk: thead: th1520-ap: Describe mux clocks with clk_mux")

There is already v2 posted so no need for v3 just for this. I can fixup
when applying.

Thanks,
Drew