From mboxrd@z Thu Jan 1 00:00:00 1970 From: mark.rutland@arm.com (Mark Rutland) Date: Wed, 15 Jul 2015 10:24:37 +0100 Subject: [PATCH] arm: dts: dra7: arch timer sits in always-on power domain In-Reply-To: <1436820101-2172-1-git-send-email-balbi@ti.com> References: <1436820101-2172-1-git-send-email-balbi@ti.com> Message-ID: <20150715092437.GC9627@leverpostej> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Mon, Jul 13, 2015 at 09:41:41PM +0100, Felipe Balbi wrote: > According to DRA7x TRM section 4.3.5 Realtime Counter (Master Counter), > the realtime counter sits in the Wakeup Always-On Power domain. > Furthermore, the counter will automatically switch the 32K clock source > when MPU goes into standby and automatically switch back to SYS_CLK or > ABE_LP when MPU goes out of standby. While the counter is in an always-on domain (the architecture mandates this) I don't think that applies to the timers (i.e. the comparators within a CPU), which are almost certainly not in an always-on domain. I suspect that this is incorrect, and it will be very painful to debug if it is... Thanks, Mark. > > Signed-off-by: Felipe Balbi > --- > arch/arm/boot/dts/dra7.dtsi | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/arch/arm/boot/dts/dra7.dtsi b/arch/arm/boot/dts/dra7.dtsi > index 8f1e25bcecbd..c1e6fd82485f 100644 > --- a/arch/arm/boot/dts/dra7.dtsi > +++ b/arch/arm/boot/dts/dra7.dtsi > @@ -50,6 +50,7 @@ > , > ; > interrupt-parent = <&gic>; > + always-on; > }; > > gic: interrupt-controller at 48211000 { > -- > 2.4.4 > > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel at lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel > From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mark Rutland Subject: Re: [PATCH] arm: dts: dra7: arch timer sits in always-on power domain Date: Wed, 15 Jul 2015 10:24:37 +0100 Message-ID: <20150715092437.GC9627@leverpostej> References: <1436820101-2172-1-git-send-email-balbi@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <1436820101-2172-1-git-send-email-balbi@ti.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Felipe Balbi Cc: Tony Lindgren , Nishanth Menon , Linux OMAP Mailing List , Linux ARM Kernel Mailing List , marc.zyngier@arm.com List-Id: linux-omap@vger.kernel.org On Mon, Jul 13, 2015 at 09:41:41PM +0100, Felipe Balbi wrote: > According to DRA7x TRM section 4.3.5 Realtime Counter (Master Counter), > the realtime counter sits in the Wakeup Always-On Power domain. > Furthermore, the counter will automatically switch the 32K clock source > when MPU goes into standby and automatically switch back to SYS_CLK or > ABE_LP when MPU goes out of standby. While the counter is in an always-on domain (the architecture mandates this) I don't think that applies to the timers (i.e. the comparators within a CPU), which are almost certainly not in an always-on domain. I suspect that this is incorrect, and it will be very painful to debug if it is... Thanks, Mark. > > Signed-off-by: Felipe Balbi > --- > arch/arm/boot/dts/dra7.dtsi | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/arch/arm/boot/dts/dra7.dtsi b/arch/arm/boot/dts/dra7.dtsi > index 8f1e25bcecbd..c1e6fd82485f 100644 > --- a/arch/arm/boot/dts/dra7.dtsi > +++ b/arch/arm/boot/dts/dra7.dtsi > @@ -50,6 +50,7 @@ > , > ; > interrupt-parent = <&gic>; > + always-on; > }; > > gic: interrupt-controller@48211000 { > -- > 2.4.4 > > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel >