Re: [PATCH RESEND 2/7] clk: spacemit: define struct k1_ccu_data

From: Alex Elder
Date: Sun Mar 23 2025 - 18:30:43 EST


On 3/23/25 8:04 AM, Yixun Lan wrote:
On 07:43 Sun 23 Mar , Alex Elder wrote:
On 3/22/25 10:50 AM, Yixun Lan wrote:
Hi Alex:

this patch change relate to clock only, so how about let's fold
it into clk patches (which now has not been merged), so we make
the code right at first place? cause some moving around and renaming

No I don't want to do that.

The clock patches are Haylen's and the are getting closer to
acceptance. Let's not confuse things by adding a bunch of new
functionality. Get those patches in, and mine can follow not
too long after that.


I only mean patch [2/7], not all patches, as it's still clock related
but, either way fine by me if you insist

I see. It would be great for Haylen to implement this, it's a
better way to do it anyway--you can define the number of
elements in the array using ARRAY_SIZE() this way also (rather
than having to count them at runtime).

Haylen is welcome to use my patch as the basis of this, but
if it doesn't get into that code I'll add it.

-Alex