Re: [PATCH v3 1/3] dt-bindings: clock: qcom,camcc-sm8250: add interconnect provider
From: Gjorgji Rosikopulos (Consultant)
Date: Tue Oct 20 2026 - 03:44:46 EST
Hi Krzysztof,
On 10/20/2026 10:12 AM, Krzysztof Kozlowski wrote:
> On Fri, Oct 16, 2026 at 08:06:59PM +0300, Atanas Filipov wrote:
>> CAM_CC_CAMNOC_AXI_CLK is shared by several camera blocks. Setting its
>
> Clock is not an interconnect, although AXI suggests this could be
> something with bus, but it seems you solve driver problem with
> interconnect description.
>
> If you only need to solve "last writer" problem, why not fixing the
> clock framework to accept the votes? Otherwise this should have
> description why it is an interconnect.
There are two approaches which we have looked in to resolving the sharing of the AXI clock.
1. The drivers newer touch this clock but vote on new icc camera which controls this clock (The v1 and v2) patches.
2. Use icc-clk in the extension in clock framework same approach as:
https://lore.kernel.org/all/20240430064214.2030013-6-quic_varada@xxxxxxxxxxx/
This is already merged and i believe resolved similar issue.
3. Which you are proposing to have clock framework to accept the votes.
Before we jump and implement the v4 it will be really good to have common agreement on which direction to go.
Regards,
~Gjorgji