diff mbox series

arm64: dts: qcom: pm8150: Enable RTC

Message ID 20210106001004.4081508-1-bjorn.andersson@linaro.org
State Accepted
Commit 40a5aa1f54ab6057230db4c39524df0e49fe730b
Headers show
Series arm64: dts: qcom: pm8150: Enable RTC | expand

Commit Message

Bjorn Andersson Jan. 6, 2021, 12:10 a.m. UTC
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(-)

Comments

Dmitry Baryshkov March 12, 2021, 6:16 p.m. UTC | #1
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

>



-- 
With best wishes
Dmitry
Vinod Koul March 13, 2021, 10:15 a.m. UTC | #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>


-- 
~Vinod
diff mbox series

Patch

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";
 };