| Message ID | 20221230085010.717423-1-abel.vesa@linaro.org (mailing list archive) |
|---|---|
| State | Not Applicable |
| Headers | show |
| Series | arm64: dts: qcom: sc8280xp-x13s: Add RTC support | expand |
Hi Abel, On Fri, Dec 30, 2022 at 10:50:10AM +0200, Abel Vesa wrote: > The PMK8350 PMIC has an available RTC block. Describe it in > sc8280xp-pmics dtsi and enable it in Lenovo Thinkpad X13s specific dts. > Mark it as wakeup-source to allow waking the system from sleep. > > Signed-off-by: Abel Vesa <abel.vesa@linaro.org> I'm currently working on proper RTC support which implies reading and storing the time offset from EFI. This patch in itself is pretty much useless as the returned time is just some random time from the epoch (e.g. in the 1970's). So I suggest waiting another week or so until I'm done. Johan
On 22-12-30 10:08:33, Johan Hovold wrote: > Hi Abel, > > On Fri, Dec 30, 2022 at 10:50:10AM +0200, Abel Vesa wrote: > > The PMK8350 PMIC has an available RTC block. Describe it in > > sc8280xp-pmics dtsi and enable it in Lenovo Thinkpad X13s specific dts. > > Mark it as wakeup-source to allow waking the system from sleep. > > > > Signed-off-by: Abel Vesa <abel.vesa@linaro.org> > > I'm currently working on proper RTC support which implies reading and > storing the time offset from EFI. > > This patch in itself is pretty much useless as the returned time is just > some random time from the epoch (e.g. in the 1970's). > > So I suggest waiting another week or so until I'm done. Sure thing. Consider this dropped then. Please CC me when you have something so I can try it out. Thanks, Abel > > Johan
On Fri, Dec 30, 2022 at 11:20:11AM +0200, Abel Vesa wrote: > On 22-12-30 10:08:33, Johan Hovold wrote: > > Hi Abel, > > > > On Fri, Dec 30, 2022 at 10:50:10AM +0200, Abel Vesa wrote: > > > The PMK8350 PMIC has an available RTC block. Describe it in > > > sc8280xp-pmics dtsi and enable it in Lenovo Thinkpad X13s specific dts. > > > Mark it as wakeup-source to allow waking the system from sleep. > > > > > > Signed-off-by: Abel Vesa <abel.vesa@linaro.org> > > > > I'm currently working on proper RTC support which implies reading and > > storing the time offset from EFI. > > > > This patch in itself is pretty much useless as the returned time is just > > some random time from the epoch (e.g. in the 1970's). > > > > So I suggest waiting another week or so until I'm done. > > Sure thing. Consider this dropped then. > > Please CC me when you have something so I can try it out. Sure, I will. Since it depends on UEFI support it may take a while to get everything merged in mainline, but we should have working out-of-tree support shortly. Johan
diff --git a/arch/arm64/boot/dts/qcom/sc8280xp-lenovo-thinkpad-x13s.dts b/arch/arm64/boot/dts/qcom/sc8280xp-lenovo-thinkpad-x13s.dts index 568c6be1ceaa..4f6daa3f8cb9 100644 --- a/arch/arm64/boot/dts/qcom/sc8280xp-lenovo-thinkpad-x13s.dts +++ b/arch/arm64/boot/dts/qcom/sc8280xp-lenovo-thinkpad-x13s.dts @@ -419,6 +419,10 @@ &pmk8280_pon_resin { status = "okay"; }; +&pmk8280_rtc { + status = "okay"; +}; + &pmk8280_vadc { status = "okay"; diff --git a/arch/arm64/boot/dts/qcom/sc8280xp-pmics.dtsi b/arch/arm64/boot/dts/qcom/sc8280xp-pmics.dtsi index f2c0b71b5d8e..00b13f9a7af8 100644 --- a/arch/arm64/boot/dts/qcom/sc8280xp-pmics.dtsi +++ b/arch/arm64/boot/dts/qcom/sc8280xp-pmics.dtsi @@ -95,6 +95,14 @@ pmk8280_adc_tm: adc-tm@3400 { #thermal-sensor-cells = <1>; status = "disabled"; }; + + pmk8280_rtc: rtc@6100 { + compatible = "qcom,pmk8350-rtc"; + reg = <0x6100>, <0x6200>; + reg-names = "rtc", "alarm"; + interrupts = <0x0 0x62 0x1 IRQ_TYPE_EDGE_RISING>; + status = "disabled"; + }; }; pmc8280_1: pmic@1 {
The PMK8350 PMIC has an available RTC block. Describe it in sc8280xp-pmics dtsi and enable it in Lenovo Thinkpad X13s specific dts. Mark it as wakeup-source to allow waking the system from sleep. Signed-off-by: Abel Vesa <abel.vesa@linaro.org> --- .../arm64/boot/dts/qcom/sc8280xp-lenovo-thinkpad-x13s.dts | 4 ++++ arch/arm64/boot/dts/qcom/sc8280xp-pmics.dtsi | 8 ++++++++ 2 files changed, 12 insertions(+)