[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <794c77bb-db9b-4b34-b632-62ea56616213@oss.qualcomm.com>
Date: Fri, 17 Oct 2025 11:26:26 +0800
From: Tingguo Cheng <tingguo.cheng@....qualcomm.com>
To: Konrad Dybcio <konrad.dybcio@....qualcomm.com>,
Bjorn Andersson <andersson@...nel.org>,
Konrad Dybcio <konradybcio@...nel.org>, Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>
Cc: Tingguo Cheng <tingguo.cheng@....qualcomm.com>,
linux-arm-msm@...r.kernel.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org, kernel@....qualcomm.com,
Kamal Wadhwa <kamal.wadhwa@....qualcomm.com>,
Rakesh Kota <rakesh.kota@....qualcomm.com>
Subject: Re: [PATCH] arm64: dts: qcom: lemans: pmic: enable rtc
On 9/25/2025 7:34 PM, Konrad Dybcio wrote:
> On 9/25/25 9:48 AM, Tingguo Cheng wrote:
>> Add RTC node, the RTC is controlled by PMIC device via spmi bus.
> subject: "arm64: dts: qcom: lemans-pmics: foo"
>
> (following the style for the directory)
Posted v2 to modify this.
>> Signed-off-by: Tingguo Cheng <tingguo.cheng@....qualcomm.com>
>> ---
>> arch/arm64/boot/dts/qcom/lemans-pmics.dtsi | 7 +++++++
>> 1 file changed, 7 insertions(+)
>>
>> diff --git a/arch/arm64/boot/dts/qcom/lemans-pmics.dtsi b/arch/arm64/boot/dts/qcom/lemans-pmics.dtsi
>> index 1369c3d43f866de9d8cd5cd4985241b99c0a0454..9e0d05c1b39ce229d5d4310ea1df1bf02e689178 100644
>> --- a/arch/arm64/boot/dts/qcom/lemans-pmics.dtsi
>> +++ b/arch/arm64/boot/dts/qcom/lemans-pmics.dtsi
>> @@ -132,6 +132,13 @@ pmm8654au_0_pon_resin: resin {
>> };
>> };
>>
>> + pmm8654au_0_rtc: rtc@...0 {
>> + compatible = "qcom,pmk8350-rtc";
>> + reg = <0x6100>, <0x6200>;
>> + reg-names = "rtc", "alarm";
> 1 reg and name a line, please
Posted v2 for this:
https://lore.kernel.org/all/20251017-add-rtc-for-lemans-v2-1-0aaf174b25b9@oss.qualcomm.com/
>> + interrupts = <0x0 0x62 0x1 IRQ_TYPE_EDGE_RISING>;
> Do we not need "qcom,no-alarm" on lemans?
No, it's not needed on lemans. "no-alarm" is only needed on compute
targets where windows TAD feature will support it.
>
> Konrad
Powered by blists - more mailing lists