| Message ID | 20210106001004.4081508-1-bjorn.andersson@linaro.org (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | arm64: dts: qcom: pm8150: Enable RTC | expand |
On Wed, 6 Jan 2021 at 03:09, Bjorn Andersson <bjorn.andersson@linaro.org> wrote: > > The PM8150 comes with everything the RTC needs, so let's just leave it > enabled instead of having to explicitly enable it for all boards. > In effect this patch enables the RTC on the SM8150 MTP and the SM8250 > HDK. Acked-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> > > Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org> > --- > arch/arm64/boot/dts/qcom/pm8150.dtsi | 4 +--- > arch/arm64/boot/dts/qcom/qrb5165-rb5.dts | 4 ---- > arch/arm64/boot/dts/qcom/sm8250-mtp.dts | 4 ---- > 3 files changed, 1 insertion(+), 11 deletions(-) > > diff --git a/arch/arm64/boot/dts/qcom/pm8150.dtsi b/arch/arm64/boot/dts/qcom/pm8150.dtsi > index a53eccf2b695..027f55d53584 100644 > --- a/arch/arm64/boot/dts/qcom/pm8150.dtsi > +++ b/arch/arm64/boot/dts/qcom/pm8150.dtsi > @@ -97,13 +97,11 @@ die-temp@6 { > }; > }; > > - pm8150_rtc: rtc@6000 { > + rtc@6000 { > compatible = "qcom,pm8941-rtc"; > reg = <0x6000>; > reg-names = "rtc", "alarm"; > interrupts = <0x0 0x61 0x1 IRQ_TYPE_NONE>; > - > - status = "disabled"; > }; > > pm8150_gpios: gpio@c000 { > diff --git a/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts b/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts > index d7799a6fb489..546b7fb9cf55 100644 > --- a/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts > +++ b/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts > @@ -609,10 +609,6 @@ lt9611_rst_pin: lt9611-rst-pin { > }; > }; > > -&pm8150_rtc { > - status = "okay"; > -}; > - > &qupv3_id_0 { > status = "okay"; > }; > diff --git a/arch/arm64/boot/dts/qcom/sm8250-mtp.dts b/arch/arm64/boot/dts/qcom/sm8250-mtp.dts > index 767a2e446248..c291422fa096 100644 > --- a/arch/arm64/boot/dts/qcom/sm8250-mtp.dts > +++ b/arch/arm64/boot/dts/qcom/sm8250-mtp.dts > @@ -392,10 +392,6 @@ &i2c15 { > /* rtc6226 @ 64 */ > }; > > -&pm8150_rtc { > - status = "okay"; > -}; > - > &qupv3_id_0 { > status = "okay"; > }; > -- > 2.29.2 >
On 05-01-21, 16:10, Bjorn Andersson wrote: > The PM8150 comes with everything the RTC needs, so let's just leave it > enabled instead of having to explicitly enable it for all boards. > In effect this patch enables the RTC on the SM8150 MTP and the SM8250 > HDK. Reviewed-by: Vinod Koul <vkoul@kernel.org>
diff --git a/arch/arm64/boot/dts/qcom/pm8150.dtsi b/arch/arm64/boot/dts/qcom/pm8150.dtsi index a53eccf2b695..027f55d53584 100644 --- a/arch/arm64/boot/dts/qcom/pm8150.dtsi +++ b/arch/arm64/boot/dts/qcom/pm8150.dtsi @@ -97,13 +97,11 @@ die-temp@6 { }; }; - pm8150_rtc: rtc@6000 { + rtc@6000 { compatible = "qcom,pm8941-rtc"; reg = <0x6000>; reg-names = "rtc", "alarm"; interrupts = <0x0 0x61 0x1 IRQ_TYPE_NONE>; - - status = "disabled"; }; pm8150_gpios: gpio@c000 { diff --git a/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts b/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts index d7799a6fb489..546b7fb9cf55 100644 --- a/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts +++ b/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts @@ -609,10 +609,6 @@ lt9611_rst_pin: lt9611-rst-pin { }; }; -&pm8150_rtc { - status = "okay"; -}; - &qupv3_id_0 { status = "okay"; }; diff --git a/arch/arm64/boot/dts/qcom/sm8250-mtp.dts b/arch/arm64/boot/dts/qcom/sm8250-mtp.dts index 767a2e446248..c291422fa096 100644 --- a/arch/arm64/boot/dts/qcom/sm8250-mtp.dts +++ b/arch/arm64/boot/dts/qcom/sm8250-mtp.dts @@ -392,10 +392,6 @@ &i2c15 { /* rtc6226 @ 64 */ }; -&pm8150_rtc { - status = "okay"; -}; - &qupv3_id_0 { status = "okay"; };
The PM8150 comes with everything the RTC needs, so let's just leave it enabled instead of having to explicitly enable it for all boards. In effect this patch enables the RTC on the SM8150 MTP and the SM8250 HDK. Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org> --- arch/arm64/boot/dts/qcom/pm8150.dtsi | 4 +--- arch/arm64/boot/dts/qcom/qrb5165-rb5.dts | 4 ---- arch/arm64/boot/dts/qcom/sm8250-mtp.dts | 4 ---- 3 files changed, 1 insertion(+), 11 deletions(-)