* [PATCH] arm64: dts: qcom: sa8775p: pmic: enable rtc
@ 2024-09-02 10:43 Tingguo Cheng
2024-09-06 12:22 ` Krzysztof Kozlowski
2024-11-22 11:00 ` Dmitry Baryshkov
0 siblings, 2 replies; 7+ messages in thread
From: Tingguo Cheng @ 2024-09-02 10:43 UTC (permalink / raw)
To: andersson, konradybcio, robh, krzk+dt, conor+dt
Cc: linux-arm-msm, devicetree, linux-kernel, kernel, quic_kotarake,
quic_kamalw, quic_skakitap, quic_fenglinw, Tingguo Cheng
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
@@ -132,6 +132,14 @@ pmm8654au_0_pon_resin: resin {
};
};
+ pmm8654au_0_rtc: rtc@6100 {
+ compatible = "qcom,pmk8350-rtc";
+ reg = <0x6100>, <0x6200>;
+ reg-names = "rtc", "alarm";
+ interrupts = <0x0 0x62 0x1 IRQ_TYPE_EDGE_RISING>;
+ allow-set-time;
+ };
+
pmm8654au_0_gpios: gpio@8800 {
compatible = "qcom,pmm8654au-gpio", "qcom,spmi-gpio";
reg = <0x8800>;
--
2.34.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH] arm64: dts: qcom: sa8775p: pmic: enable rtc
2024-09-02 10:43 [PATCH] arm64: dts: qcom: sa8775p: pmic: enable rtc Tingguo Cheng
@ 2024-09-06 12:22 ` Krzysztof Kozlowski
2024-09-19 2:23 ` Tingguo Cheng
2024-11-22 11:00 ` Dmitry Baryshkov
1 sibling, 1 reply; 7+ messages in thread
From: Krzysztof Kozlowski @ 2024-09-06 12:22 UTC (permalink / raw)
To: Tingguo Cheng, andersson, konradybcio, robh, krzk+dt, conor+dt
Cc: linux-arm-msm, devicetree, linux-kernel, kernel, quic_kotarake,
quic_kamalw, quic_skakitap, quic_fenglinw
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.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] arm64: dts: qcom: sa8775p: pmic: enable rtc
2024-09-06 12:22 ` Krzysztof Kozlowski
@ 2024-09-19 2:23 ` Tingguo Cheng
2024-11-22 10:38 ` Tingguo Cheng
0 siblings, 1 reply; 7+ messages in thread
From: Tingguo Cheng @ 2024-09-19 2:23 UTC (permalink / raw)
To: Krzysztof Kozlowski, andersson, konradybcio, robh, krzk+dt,
conor+dt
Cc: linux-arm-msm, devicetree, linux-kernel, kernel, quic_kotarake,
quic_kamalw, quic_skakitap, quic_fenglinw
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
>
--
Thank you & BRs
Tingguo
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] arm64: dts: qcom: sa8775p: pmic: enable rtc
2024-09-19 2:23 ` Tingguo Cheng
@ 2024-11-22 10:38 ` Tingguo Cheng
0 siblings, 0 replies; 7+ messages in thread
From: Tingguo Cheng @ 2024-11-22 10:38 UTC (permalink / raw)
To: Krzysztof Kozlowski, andersson, konradybcio, robh, krzk+dt,
conor+dt
Cc: linux-arm-msm, devicetree, linux-kernel, kernel, quic_kotarake,
quic_kamalw, quic_skakitap, quic_fenglinw
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
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] arm64: dts: qcom: sa8775p: pmic: enable rtc
2024-09-02 10:43 [PATCH] arm64: dts: qcom: sa8775p: pmic: enable rtc Tingguo Cheng
2024-09-06 12:22 ` Krzysztof Kozlowski
@ 2024-11-22 11:00 ` Dmitry Baryshkov
2025-01-16 7:44 ` Tingguo Cheng
1 sibling, 1 reply; 7+ messages in thread
From: Dmitry Baryshkov @ 2024-11-22 11:00 UTC (permalink / raw)
To: Tingguo Cheng
Cc: andersson, konradybcio, robh, krzk+dt, conor+dt, linux-arm-msm,
devicetree, linux-kernel, kernel, quic_kotarake, quic_kamalw,
quic_skakitap, quic_fenglinw
On Mon, Sep 02, 2024 at 06:43:02PM +0800, 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(+)
>
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] arm64: dts: qcom: sa8775p: pmic: enable rtc
2024-11-22 11:00 ` Dmitry Baryshkov
@ 2025-01-16 7:44 ` Tingguo Cheng
2025-01-16 9:11 ` Krzysztof Kozlowski
0 siblings, 1 reply; 7+ messages in thread
From: Tingguo Cheng @ 2025-01-16 7:44 UTC (permalink / raw)
To: Dmitry Baryshkov
Cc: andersson, konradybcio, robh, krzk+dt, conor+dt, linux-arm-msm,
devicetree, linux-kernel, kernel, quic_kotarake, quic_kamalw,
quic_skakitap, quic_fenglinw
On 11/22/2024 7:00 PM, Dmitry Baryshkov wrote:
> On Mon, Sep 02, 2024 at 06:43:02PM +0800, 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(+)
>>
>
> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
>
Could you please help apply this patch?
--
Thank you & BRs
Tingguo
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] arm64: dts: qcom: sa8775p: pmic: enable rtc
2025-01-16 7:44 ` Tingguo Cheng
@ 2025-01-16 9:11 ` Krzysztof Kozlowski
0 siblings, 0 replies; 7+ messages in thread
From: Krzysztof Kozlowski @ 2025-01-16 9:11 UTC (permalink / raw)
To: Tingguo Cheng, Dmitry Baryshkov
Cc: andersson, konradybcio, robh, krzk+dt, conor+dt, linux-arm-msm,
devicetree, linux-kernel, kernel, quic_kotarake, quic_kamalw,
quic_skakitap, quic_fenglinw
On 16/01/2025 08:44, Tingguo Cheng wrote:
>
>
> On 11/22/2024 7:00 PM, Dmitry Baryshkov wrote:
>> On Mon, Sep 02, 2024 at 06:43:02PM +0800, 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(+)
>>>
>>
>> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
>>
> Could you please help apply this patch?
Check the patchwork, is it there? What is it its status? Maybe no need
to ping people, because it will change nothing?
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2025-01-16 9:11 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-09-02 10:43 [PATCH] arm64: dts: qcom: sa8775p: pmic: enable rtc Tingguo Cheng
2024-09-06 12:22 ` Krzysztof Kozlowski
2024-09-19 2:23 ` Tingguo Cheng
2024-11-22 10:38 ` Tingguo Cheng
2024-11-22 11:00 ` Dmitry Baryshkov
2025-01-16 7:44 ` Tingguo Cheng
2025-01-16 9:11 ` Krzysztof Kozlowski
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox