diff mbox series

arm: dts: am4372: setup rtc as system-power-controller

Message ID 1532498135-9031-1-git-send-email-j-keerthy@ti.com (mailing list archive)
State New, archived
Headers show
Series arm: dts: am4372: setup rtc as system-power-controller | expand

Commit Message

J, KEERTHY July 25, 2018, 5:55 a.m. UTC
RTC alarm2 is connected to pmic_en line and hence can be used to control
the pmic enabling/disabling. Hence add the system-power-controller for rtc
node.

Signed-off-by: Keerthy <j-keerthy@ti.com>
---
 arch/arm/boot/dts/am4372.dtsi | 1 +
 1 file changed, 1 insertion(+)

Comments

Tony Lindgren Aug. 8, 2018, 9:06 a.m. UTC | #1
* Keerthy <j-keerthy@ti.com> [180725 05:59]:
> RTC alarm2 is connected to pmic_en line and hence can be used to control
> the pmic enabling/disabling. Hence add the system-power-controller for rtc
> node.
> 
> Signed-off-by: Keerthy <j-keerthy@ti.com>
> ---
>  arch/arm/boot/dts/am4372.dtsi | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/arch/arm/boot/dts/am4372.dtsi b/arch/arm/boot/dts/am4372.dtsi
> index f0cbd86..d4b7c59 100644
> --- a/arch/arm/boot/dts/am4372.dtsi
> +++ b/arch/arm/boot/dts/am4372.dtsi
> @@ -469,6 +469,7 @@
>  			ti,hwmods = "rtc";
>  			clocks = <&clk_32768_ck>;
>  			clock-names = "int-clk";
> +			system-power-controller;
>  			status = "disabled";
>  		};
>  

Should this be queued as a fix for v4.19-rc series?

Regards,

Tony
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
J, KEERTHY Aug. 8, 2018, 9:08 a.m. UTC | #2
On Wednesday 08 August 2018 02:36 PM, Tony Lindgren wrote:
> * Keerthy <j-keerthy@ti.com> [180725 05:59]:
>> RTC alarm2 is connected to pmic_en line and hence can be used to control
>> the pmic enabling/disabling. Hence add the system-power-controller for rtc
>> node.
>>
>> Signed-off-by: Keerthy <j-keerthy@ti.com>
>> ---
>>  arch/arm/boot/dts/am4372.dtsi | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/arch/arm/boot/dts/am4372.dtsi b/arch/arm/boot/dts/am4372.dtsi
>> index f0cbd86..d4b7c59 100644
>> --- a/arch/arm/boot/dts/am4372.dtsi
>> +++ b/arch/arm/boot/dts/am4372.dtsi
>> @@ -469,6 +469,7 @@
>>  			ti,hwmods = "rtc";
>>  			clocks = <&clk_32768_ck>;
>>  			clock-names = "int-clk";
>> +			system-power-controller;
>>  			status = "disabled";
>>  		};
>>  
> 
> Should this be queued as a fix for v4.19-rc series?

Yes this makes pmic to shutdown during power off.

> 
> Regards,
> 
> Tony
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Tony Lindgren Aug. 17, 2018, 2:28 p.m. UTC | #3
* Keerthy <j-keerthy@ti.com> [180808 09:11]:
> 
> 
> On Wednesday 08 August 2018 02:36 PM, Tony Lindgren wrote:
> > * Keerthy <j-keerthy@ti.com> [180725 05:59]:
> >> RTC alarm2 is connected to pmic_en line and hence can be used to control
> >> the pmic enabling/disabling. Hence add the system-power-controller for rtc
> >> node.
> >>
> >> Signed-off-by: Keerthy <j-keerthy@ti.com>
> >> ---
> >>  arch/arm/boot/dts/am4372.dtsi | 1 +
> >>  1 file changed, 1 insertion(+)
> >>
> >> diff --git a/arch/arm/boot/dts/am4372.dtsi b/arch/arm/boot/dts/am4372.dtsi
> >> index f0cbd86..d4b7c59 100644
> >> --- a/arch/arm/boot/dts/am4372.dtsi
> >> +++ b/arch/arm/boot/dts/am4372.dtsi
> >> @@ -469,6 +469,7 @@
> >>  			ti,hwmods = "rtc";
> >>  			clocks = <&clk_32768_ck>;
> >>  			clock-names = "int-clk";
> >> +			system-power-controller;
> >>  			status = "disabled";
> >>  		};
> >>  
> > 
> > Should this be queued as a fix for v4.19-rc series?
> 
> Yes this makes pmic to shutdown during power off.

OK thanks applying into omap-for-v4.19/fixes.

Tony
diff mbox series

Patch

diff --git a/arch/arm/boot/dts/am4372.dtsi b/arch/arm/boot/dts/am4372.dtsi
index f0cbd86..d4b7c59 100644
--- a/arch/arm/boot/dts/am4372.dtsi
+++ b/arch/arm/boot/dts/am4372.dtsi
@@ -469,6 +469,7 @@ 
 			ti,hwmods = "rtc";
 			clocks = <&clk_32768_ck>;
 			clock-names = "int-clk";
+			system-power-controller;
 			status = "disabled";
 		};