[PATCH 0/7] ARM: dts: stm32: clk: Switch ETHRX clock parent from ETHCK_K to MCO2 on DHCOM SoM

Alexandre TORGUE alexandre.torgue at foss.st.com
Tue Apr 13 08:48:10 BST 2021


Hi Marek

On 4/12/21 8:44 PM, Marek Vasut wrote:
> On 4/12/21 10:09 AM, Alexandre TORGUE wrote:
>> Hi Marek
> 
> Hello Alex,
> 
> [...]
> 
>>> All the above still only discusses the clock part of the problem. 
>>> Even if
>>> the clock cyclic dependencies could be solved, it would be necessary to
>>> resolve legacy dwmac st,eth-clk-sel and st,eth-ref-clk-sel DT properties
>>> and avoid DT ABI break.
>>
>> Thanks for those clear explanations and for this series. As discussed, 
>> this approach looks good to me as it doesn't break our current 
>> strategy for dwmac clock integration. I don't know if those cyclic 
>> redundancies will be fixed one day but we can have a look on dwmac DT 
>> properties (the gain to change them, the effort to keep the backward 
>> compatibility, code readability, ...).
>>
>> Your DT patches looks good. I'll merge them soon.
> +CC Stephen ; the DT patches depend on the clock driver changes. Would 
> it make sense to pick the clock patches through the same tree or how 
> should that be handled ?

In this situation, I prefer to wait that Stephen takes clock patches in 
his tree. Then I'll take DT ones in mine. (I assume that taking only 
clock patches will not break mp1 boot or Ethernet usage).

Regards
Alex


> 
> [...]
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel



More information about the linux-arm-kernel mailing list