David Wronek писал(а) 06.07.2023 17:40: > Add support for the RTC which is the same as on other PMICs and add the > resin child node to the PM6150 PON device and disable it by default. > > Signed-off-by: David Wronek <davidwronek@xxxxxxxxx> Actually... (+CC CROS watchers) Doug, this would enable a somewhat dubious RTC for trogdor devices. On qcom firmware the rtc is forever write-protected but you run TF-A so maybe it's more cooperative and you can add "allow-set-time" prop for trogdor, in case qtiseclib doesn't mess with it? Or, alternatively, disable this rtc for trogdor completely so it doesn't cause trouble? In any case, on the WoA firmware and with userspace offset handling, this works beautifully, so: Tested-by: Nikita Travkin <nikita@xxxxxxx> # sc7180, Aspire 1 Thanks! > --- > arch/arm64/boot/dts/qcom/pm6150.dtsi | 15 +++++++++++++++ > 1 file changed, 15 insertions(+) > > diff --git a/arch/arm64/boot/dts/qcom/pm6150.dtsi b/arch/arm64/boot/dts/qcom/pm6150.dtsi > index 2e6afa296141..43c50f890e8b 100644 > --- a/arch/arm64/boot/dts/qcom/pm6150.dtsi > +++ b/arch/arm64/boot/dts/qcom/pm6150.dtsi > @@ -53,6 +53,14 @@ pm6150_pwrkey: pwrkey { > bias-pull-up; > linux,code = <KEY_POWER>; > }; > + > + pm6150_resin: resin { > + compatible = "qcom,pm8941-resin"; > + interrupts = <0x0 0x8 1 IRQ_TYPE_EDGE_BOTH>; > + debounce = <15625>; > + bias-pull-up; > + status = "disabled"; > + }; > }; > > pm6150_temp: temp-alarm@2400 { > @@ -88,6 +96,13 @@ pm6150_adc_tm: adc-tm@3500 { > status = "disabled"; > }; > > + rtc@6000 { > + compatible = "qcom,pm8941-rtc"; > + reg = <0x6000>, <0x6100>; > + reg-names = "rtc", "alarm"; > + interrupts = <0x0 0x61 0x1 IRQ_TYPE_EDGE_RISING>; > + }; > + > pm6150_gpios: gpio@c000 { > compatible = "qcom,pm6150-gpio", "qcom,spmi-gpio"; > reg = <0xc000>;