linux-kernel - Re: [PATCH v3 7/8] arm64: dts: qcom: Add support for multimedia clock controllers

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: <080b47dc-c8e0-4962-a358-aa4c39e5e868@quicinc.com>
Date: Mon, 29 Jul 2024 18:42:19 +0800
From: Tengfei Fan <quic_tengfan@...cinc.com>
To: Konrad Dybcio <konrad.dybcio@...aro.org>,
        Krzysztof Kozlowski
	<krzk@...nel.org>,
        Taniya Das <quic_tdas@...cinc.com>,
        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>,
        Bartosz Golaszewski
	<bartosz.golaszewski@...aro.org>
CC: <linux-arm-msm@...r.kernel.org>, <linux-clk@...r.kernel.org>,
        <devicetree@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
        <quic_imrashai@...cinc.com>, <quic_jkona@...cinc.com>,
        Tingwei
	<quic_tingweiz@...cinc.com>,
        "Aiqun(Maria) Yu" <quic_aiquny@...cinc.com>
Subject: Re: [PATCH v3 7/8] arm64: dts: qcom: Add support for multimedia clock
 controllers



On 7/16/2024 8:09 PM, Konrad Dybcio wrote:
> On 16.07.2024 9:45 AM, Krzysztof Kozlowski wrote:
>> On 15/07/2024 10:23, Taniya Das wrote:
>>> Add support for video, camera, display0 and display1 clock
>>> controllers on SA8775P platform.
>>>
>>> Signed-off-by: Taniya Das <quic_tdas@...cinc.com>
>>> ---
>>>   arch/arm64/boot/dts/qcom/sa8775p.dtsi | 56 +++++++++++++++++++++++++++++++++++
>>>   1 file changed, 56 insertions(+)
>>>
>>> diff --git a/arch/arm64/boot/dts/qcom/sa8775p.dtsi b/arch/arm64/boot/dts/qcom/sa8775p.dtsi
>>> index 23f1b2e5e624..8fd68a8aa916 100644
>>> --- a/arch/arm64/boot/dts/qcom/sa8775p.dtsi
>>> +++ b/arch/arm64/boot/dts/qcom/sa8775p.dtsi
>>> @@ -2911,6 +2911,47 @@ llcc: system-cache-controller@...0000 {
>>>   			interrupts = <GIC_SPI 580 IRQ_TYPE_LEVEL_HIGH>;
>>>   		};
>>>   
>>> +		videocc: clock-controller@...0000 {
>>> +			compatible = "qcom,sa8775p-videocc";
>>> +			reg = <0x0 0x0abf0000 0x0 0x10000>;
>>> +			clocks = <&gcc GCC_VIDEO_AHB_CLK>,
>>> +				 <&rpmhcc RPMH_CXO_CLK>,
>>> +				 <&rpmhcc RPMH_CXO_CLK_A>,
>>> +				 <&sleep_clk>;
>>> +			power-domains = <&rpmhpd SA8775P_MMCX>;
>>
>> Not sure if these are correct. I had impression the clocks are going
>> away from sa8775p?
> 
> Right, the patches look mostly good, but are still going to be on hold
> until the 8775 situation is cleared out.. We recently had the gigantic
> patchset [1] that shifted things around, and seemingly there was rather
> little closure on that, so we're waiting for the dust to settle and
> people to agree on things..
> 
> Konrad
> 
> [1] https://lore.kernel.org/linux-arm-msm/20240703025850.2172008-1-quic_tengfan@quicinc.com/

After considering the feedback provided on the subject, We have decided
to keep current SA8775p compatible and ABI compatibility in drivers.
Therefore, this patch is still needed, please continue to review this
patch.
Thank you for your input.

-- 
Thx and BRs,
Tengfei Fan