[PATCH v10 09/23] dt-binding: clock: Document canaan,k210-clk bindings

Damien Le Moal Damien.LeMoal at wdc.com
Thu Dec 17 05:51:20 EST 2020


On 2020/12/17 19:49, Stephen Boyd wrote:
> Quoting Damien Le Moal (2020-12-17 02:43:50)
>>
>> I just did that. It works.
>>
>> Ideally, patches 7, 8 and 9 need to go in together with the clk driver patch.
>> Since the builtin DTB patch precedes the clock driver patch that touches the
>> sysctl driver, I need to rework it a little, keeping the
>> SOC_DECLARE_BUILTIN_DTB() for now. And finally, a small DTS update patch needs
>> to be added too for the sysctl & sysclk nodes update. That would make it a 5
>> patch series for the clock driver addition. Would this work ?
>>
>> Or, you just take patch 9 (clk doc) and patch 13 (clk driver), slightly modified
>> to move the sysctl register definitions into a common header (currently part of
>> patch 7). 2 patches only, without any other change, resulting in the clock
>> driver not being used until the rest of the series goes into 5.12. Do you prefer
>> that solution ?
>>
> 
> I was thinking of just applying this DT binding patch now so it gets
> merged into the next -rc1. Then anyone can use the defines because
> they're in Linus' tree and wherever the dts file ends up can just base
> on -rc1. I probably won't merge the clk driver until v5.12 given that
> the merge window is open.

OK. Makes sense. Sending just the clk binding patch then, with the K210_CLK_ACLK
clock definition left in.

Thanks !

> 


-- 
Damien Le Moal
Western Digital Research



More information about the linux-riscv mailing list