linux-kernel - Re: [PATCH 1/3] dt-bindings: clock: Add Qualcomm SC8180X Camera clock controller

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <1b09b837-2478-49a8-a7a1-0a49f0704105@quicinc.com>
Date: Wed, 30 Apr 2025 10:23:55 +0530
From: Satya Priya Kakitapalli <quic_skakitap@...cinc.com>
To: Konrad Dybcio <konrad.dybcio@....qualcomm.com>,
        Bryan O'Donoghue
	<bryan.odonoghue@...aro.org>,
        Bjorn Andersson <andersson@...nel.org>,
        "Michael Turquette" <mturquette@...libre.com>,
        Stephen Boyd
	<sboyd@...nel.org>, "Rob Herring" <robh@...nel.org>,
        Krzysztof Kozlowski
	<krzk+dt@...nel.org>,
        "Conor Dooley" <conor+dt@...nel.org>,
        Konrad Dybcio
	<konradybcio@...nel.org>
CC: Ajit Pandey <quic_ajipan@...cinc.com>,
        Imran Shaik
	<quic_imrashai@...cinc.com>,
        Taniya Das <quic_tdas@...cinc.com>,
        "Jagadeesh
 Kona" <quic_jkona@...cinc.com>,
        <linux-arm-msm@...r.kernel.org>, <linux-clk@...r.kernel.org>,
        <devicetree@...r.kernel.org>, <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 1/3] dt-bindings: clock: Add Qualcomm SC8180X Camera clock
 controller


On 4/25/2025 3:05 PM, Konrad Dybcio wrote:
> On 4/24/25 12:38 PM, Satya Priya Kakitapalli wrote:
>> On 4/22/2025 5:11 PM, Bryan O'Donoghue wrote:
>>> On 22/04/2025 06:42, Satya Priya Kakitapalli wrote:
>>>> Add device tree bindings for the camera clock controller on
>>>> Qualcomm SC8180X platform.
>>>>
>>>> Signed-off-by: Satya Priya Kakitapalli <quic_skakitap@...cinc.com>
>>>> ---
> [...]
>
>>>> +  required-opps:
>>>> +    maxItems: 1
>>>> +    description:
>>>> +      A phandle to an OPP node describing required MMCX performance point.
>>>> +
>>>> +allOf:
>>>> +  - $ref: qcom,gcc.yaml#
>>> A suspicious lack of clock depends here. No AHB clock ? No dependency on gcc ?
>>>
>>> You call out the gcc above.
>>>
>>> Could you please recheck your list of clock dependencies.
>> The dependent GCC clocks are marked always on from gcc probe, hence did not mention the dependency here.
> Let's do what was done on x1e80100 - describe the AHB clock in CAMCC
> bindings regardless of how we handle it.
>
> This way the DT represents the real hw dependency, but the OS takes steps
> to get them out of the way (and then ignores the GCC_CAMERA_AHB_CLK entry
> because the clock is never registered with GCC)


Sure, current gcc bindings header does not have the camera AHB clock 
defined, I'll add the definition and mark the dependency here.


Thanks.


> Konrad