[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <47ac5ac0-f1d5-4506-aceb-22a02045fe75@quicinc.com>
Date: Fri, 16 Aug 2024 10:48:55 +0530
From: Taniya Das <quic_tdas@...cinc.com>
To: Tengfei Fan <quic_tengfan@...cinc.com>,
Konrad Dybcio
<konrad.dybcio@...aro.org>,
Krzysztof Kozlowski <krzk@...nel.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>,
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/29/2024 4:12 PM, Tengfei Fan wrote:
>
>
> 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.
>
Thank you Tengfei, I will post the patches again.
--
Thanks & Regards,
Taniya Das.
Powered by blists - more mailing lists