On Mon, Jul 17, 2023 at 09:28:15PM +0100, Caleb Connolly wrote: > On 17/07/2023 19:23, Eric Chanudet wrote: > > SA8540P-ride is one of the Qualcomm platforms that does not have access > > to UEFI runtime services and on which the RTC registers are read-only, > > as described in: > > https://lore.kernel.org/all/20230202155448.6715-1-johan+linaro@xxxxxxxxxx/ > > > > Reserve four bytes in one of the PMIC registers to hold the RTC offset > > the same way as it was done for sc8280xp-crd which has similar > > limitations: > > commit e67b45582c5e ("arm64: dts: qcom: sc8280xp-crd: enable rtc") > > > > One small difference on SA8540P-ride, the PMIC register bank SDAM6 is > > not writable, so use SDAM7 instead. > > > > Signed-off-by: Eric Chanudet <echanude@xxxxxxxxxx> > > --- > > arch/arm64/boot/dts/qcom/sa8540p-pmics.dtsi | 10 +++++++++- > > arch/arm64/boot/dts/qcom/sa8540p-ride.dts | 15 +++++++++++++++ > > 2 files changed, 24 insertions(+), 1 deletion(-) > > > > diff --git a/arch/arm64/boot/dts/qcom/sa8540p-pmics.dtsi b/arch/arm64/boot/dts/qcom/sa8540p-pmics.dtsi > > index 1221be89b3de..9c5dcad35cce 100644 > > --- a/arch/arm64/boot/dts/qcom/sa8540p-pmics.dtsi > > +++ b/arch/arm64/boot/dts/qcom/sa8540p-pmics.dtsi > > @@ -14,7 +14,7 @@ pmm8540a: pmic@0 { > > #address-cells = <1>; > > #size-cells = <0>; > > > > - rtc@6000 { > > + pmm8540a_rtc: rtc@6000 { > > compatible = "qcom,pm8941-rtc"; > > reg = <0x6000>, <0x6100>; > > reg-names = "rtc", "alarm"; > > @@ -22,6 +22,14 @@ rtc@6000 { > > wakeup-source; > > }; > > > > + pmm8540a_sdam_7: nvram@b610 { > Johan disabled the SDAM node in their series for sc8280xp. Unless it's > used on all sa8540p platforms, you should probably also do that here. > > > > + compatible = "qcom,spmi-sdam"; > > + reg = <0xb610>; > > + #address-cells = <1>; > > + #size-cells = <1>; > > + ranges = <0 0xb610 0xb0>; > status = "disabled"; > > With that fix, > > Reviewed-by: Caleb Connolly <caleb.connolly@xxxxxxxxxx> Thank you for the review. Here is the v2 with the requested change: https://lore.kernel.org/linux-arm-msm/20230718145105.3464105-1-echanude@xxxxxxxxxx/ Best, -- Eric Chanudet
