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

Alex Elder
2025-03-23T22:30:33Z

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

>