Re: [PATCH] arm64: dts: qcom: sa8775p: pmic: enable rtc

[Tingguo Cheng <quic_tingguoc@quicinc.com>]

On 9/19/2024 10:23 AM, Tingguo Cheng wrote:
> 
> 
> On 9/6/2024 8:22 PM, Krzysztof Kozlowski wrote:
>> On 02/09/2024 12:43, Tingguo Cheng wrote:
>>> Add RTC node, the RTC is controlled by PMIC device via spmi bus.
>>>
>>> Signed-off-by: Tingguo Cheng <quic_tingguoc@quicinc.com>
>>> ---
>>>   arch/arm64/boot/dts/qcom/sa8775p-pmics.dtsi | 8 ++++++++
>>>   1 file changed, 8 insertions(+)
>>>
>>> diff --git a/arch/arm64/boot/dts/qcom/sa8775p-pmics.dtsi b/arch/ 
>>> arm64/boot/dts/qcom/sa8775p-pmics.dtsi
>>> index 1369c3d43f86..47d05b897d5a 100644
>>> --- a/arch/arm64/boot/dts/qcom/sa8775p-pmics.dtsi
>>> +++ b/arch/arm64/boot/dts/qcom/sa8775p-pmics.dtsi
>>
>> We achieved consensus allowing sa8775p to stay, but now Qualcomm changes
>> point of view and insists on new approach of dropping sa8775p. Therefore
>> this change does not make much sense in the new approach.
>>
> The new approach(QCS9100+sa8775p compatible) will still use the same 
> PMIC chip(As usually, include sa8775p-pmics.dtsi as it's pmic part). 
> Therefore, from this point, it does make sense in the new approach.
>  From another point, for old boards, this change should have been there.
> 
>> Best regards,
>> Krzysztof
>>
> 
We have aligned this internally again, "sa8775p-pmics.dtsi" still 
continue to be used for qcs9100-ride. Can you help review/apply this
change? We need this RTC function in some boards.

-- 
Thank you & BRs
Tingguo