From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752408AbdA3LEu (ORCPT ); Mon, 30 Jan 2017 06:04:50 -0500 Received: from mail-wm0-f44.google.com ([74.125.82.44]:37556 "EHLO mail-wm0-f44.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751141AbdA3LEn (ORCPT ); Mon, 30 Jan 2017 06:04:43 -0500 Date: Mon, 30 Jan 2017 12:04:28 +0100 From: Daniel Lezcano To: Alexander Kochetkov Cc: Heiko Stuebner , linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-rockchip@lists.infradead.org, Thomas Gleixner , Mark Rutland , Rob Herring , Russell King , Caesar Wang , Huang Tao Subject: Re: [PATCH v5 3/8] ARM: dts: rockchip: add timer entries to rk3188 SoC Message-ID: <20170130110428.GC2206@mai> References: <1485260203-14216-1-git-send-email-al.kochet@gmail.com> <1485260203-14216-4-git-send-email-al.kochet@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <1485260203-14216-4-git-send-email-al.kochet@gmail.com> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Jan 24, 2017 at 03:16:38PM +0300, Alexander Kochetkov wrote: > The patch add two timers to all rk3188 based boards. > > The first timer is from alive subsystem and it act as a backup > for the local timers at sleep time. It act the same as other > SoC rockchip timers already present in kernel. It is up to the RTC to wake up the system from suspend and there is no idle state stopping the local timers on these SoCs. So, the rockchip timers won't be ever used on the rk3188, right ? > The second timer is from CPU subsystem and act as replacement > for the arm-global-timer clocksource and sched clock. It run > at stable frequency 24MHz. This patch should go after the rockchip timer patches. > Signed-off-by: Alexander Kochetkov > Reviwed-by: Heiko Stübner > --- > arch/arm/boot/dts/rk3188.dtsi | 16 ++++++++++++++++ > 1 file changed, 16 insertions(+) > > diff --git a/arch/arm/boot/dts/rk3188.dtsi b/arch/arm/boot/dts/rk3188.dtsi > index 869e189..bcf8e03 100644 > --- a/arch/arm/boot/dts/rk3188.dtsi > +++ b/arch/arm/boot/dts/rk3188.dtsi > @@ -106,6 +106,22 @@ > }; > }; > > + timer3: timer@2000e000 { > + compatible = "rockchip,rk3188-timer", "rockchip,rk3288-timer"; > + reg = <0x2000e000 0x20>; > + interrupts = ; > + clocks = <&cru SCLK_TIMER3>, <&cru PCLK_TIMER3>; > + clock-names = "timer", "pclk"; > + }; > + > + timer6: timer@200380a0 { > + compatible = "rockchip,rk3188-timer", "rockchip,rk3288-timer"; > + reg = <0x200380a0 0x20>; > + interrupts = ; > + clocks = <&cru SCLK_TIMER6>, <&cru PCLK_TIMER0>; > + clock-names = "timer", "pclk"; > + }; > + > i2s0: i2s@1011a000 { > compatible = "rockchip,rk3188-i2s", "rockchip,rk3066-i2s"; > reg = <0x1011a000 0x2000>; > -- > 1.7.9.5 > -- Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Lezcano Subject: Re: [PATCH v5 3/8] ARM: dts: rockchip: add timer entries to rk3188 SoC Date: Mon, 30 Jan 2017 12:04:28 +0100 Message-ID: <20170130110428.GC2206@mai> References: <1485260203-14216-1-git-send-email-al.kochet@gmail.com> <1485260203-14216-4-git-send-email-al.kochet@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Return-path: Content-Disposition: inline In-Reply-To: <1485260203-14216-4-git-send-email-al.kochet-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Alexander Kochetkov Cc: Heiko Stuebner , linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, Thomas Gleixner , Mark Rutland , Rob Herring , Russell King , Caesar Wang , Huang Tao List-Id: devicetree@vger.kernel.org On Tue, Jan 24, 2017 at 03:16:38PM +0300, Alexander Kochetkov wrote: > The patch add two timers to all rk3188 based boards. > > The first timer is from alive subsystem and it act as a backup > for the local timers at sleep time. It act the same as other > SoC rockchip timers already present in kernel. It is up to the RTC to wake up the system from suspend and there is no idle state stopping the local timers on these SoCs. So, the rockchip timers won't be ever used on the rk3188, right ? > The second timer is from CPU subsystem and act as replacement > for the arm-global-timer clocksource and sched clock. It run > at stable frequency 24MHz. This patch should go after the rockchip timer patches. > Signed-off-by: Alexander Kochetkov > Reviwed-by: Heiko Stübner > --- > arch/arm/boot/dts/rk3188.dtsi | 16 ++++++++++++++++ > 1 file changed, 16 insertions(+) > > diff --git a/arch/arm/boot/dts/rk3188.dtsi b/arch/arm/boot/dts/rk3188.dtsi > index 869e189..bcf8e03 100644 > --- a/arch/arm/boot/dts/rk3188.dtsi > +++ b/arch/arm/boot/dts/rk3188.dtsi > @@ -106,6 +106,22 @@ > }; > }; > > + timer3: timer@2000e000 { > + compatible = "rockchip,rk3188-timer", "rockchip,rk3288-timer"; > + reg = <0x2000e000 0x20>; > + interrupts = ; > + clocks = <&cru SCLK_TIMER3>, <&cru PCLK_TIMER3>; > + clock-names = "timer", "pclk"; > + }; > + > + timer6: timer@200380a0 { > + compatible = "rockchip,rk3188-timer", "rockchip,rk3288-timer"; > + reg = <0x200380a0 0x20>; > + interrupts = ; > + clocks = <&cru SCLK_TIMER6>, <&cru PCLK_TIMER0>; > + clock-names = "timer", "pclk"; > + }; > + > i2s0: i2s@1011a000 { > compatible = "rockchip,rk3188-i2s", "rockchip,rk3066-i2s"; > reg = <0x1011a000 0x2000>; > -- > 1.7.9.5 > -- Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 From: daniel.lezcano@linaro.org (Daniel Lezcano) Date: Mon, 30 Jan 2017 12:04:28 +0100 Subject: [PATCH v5 3/8] ARM: dts: rockchip: add timer entries to rk3188 SoC In-Reply-To: <1485260203-14216-4-git-send-email-al.kochet@gmail.com> References: <1485260203-14216-1-git-send-email-al.kochet@gmail.com> <1485260203-14216-4-git-send-email-al.kochet@gmail.com> Message-ID: <20170130110428.GC2206@mai> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Tue, Jan 24, 2017 at 03:16:38PM +0300, Alexander Kochetkov wrote: > The patch add two timers to all rk3188 based boards. > > The first timer is from alive subsystem and it act as a backup > for the local timers at sleep time. It act the same as other > SoC rockchip timers already present in kernel. It is up to the RTC to wake up the system from suspend and there is no idle state stopping the local timers on these SoCs. So, the rockchip timers won't be ever used on the rk3188, right ? > The second timer is from CPU subsystem and act as replacement > for the arm-global-timer clocksource and sched clock. It run > at stable frequency 24MHz. This patch should go after the rockchip timer patches. > Signed-off-by: Alexander Kochetkov > Reviwed-by: Heiko St?bner > --- > arch/arm/boot/dts/rk3188.dtsi | 16 ++++++++++++++++ > 1 file changed, 16 insertions(+) > > diff --git a/arch/arm/boot/dts/rk3188.dtsi b/arch/arm/boot/dts/rk3188.dtsi > index 869e189..bcf8e03 100644 > --- a/arch/arm/boot/dts/rk3188.dtsi > +++ b/arch/arm/boot/dts/rk3188.dtsi > @@ -106,6 +106,22 @@ > }; > }; > > + timer3: timer at 2000e000 { > + compatible = "rockchip,rk3188-timer", "rockchip,rk3288-timer"; > + reg = <0x2000e000 0x20>; > + interrupts = ; > + clocks = <&cru SCLK_TIMER3>, <&cru PCLK_TIMER3>; > + clock-names = "timer", "pclk"; > + }; > + > + timer6: timer at 200380a0 { > + compatible = "rockchip,rk3188-timer", "rockchip,rk3288-timer"; > + reg = <0x200380a0 0x20>; > + interrupts = ; > + clocks = <&cru SCLK_TIMER6>, <&cru PCLK_TIMER0>; > + clock-names = "timer", "pclk"; > + }; > + > i2s0: i2s at 1011a000 { > compatible = "rockchip,rk3188-i2s", "rockchip,rk3066-i2s"; > reg = <0x1011a000 0x2000>; > -- > 1.7.9.5 > -- Linaro.org ? Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog