Groups > linux.kernel > #1420814

Re: [PATCH v2 0/4] clocksource: rockchip/timer: Support rktimer for rk3399

Path	csiph.com!news.mixmin.net!aioe.org!gothmog.csi.it!bofh.it!news.nic.it!robomod
From	Daniel Lezcano <daniel.lezcano@linaro.org>
Newsgroups	linux.kernel
Subject	Re: [PATCH v2 0/4] clocksource: rockchip/timer: Support rktimer for rk3399
Date	Mon, 13 Jun 2016 15:10:02 +0200
Message-ID	<rJzMS-6In-29@gated-at.bofh.it> (permalink)
References	<rHhhn-6ua-7@gated-at.bofh.it>
X-Original-To	Caesar Wang <wxt@rock-chips.com>
Dkim-Signature	v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=Xtw8spV0c25DTBaWOLU9mn4qmOhw1hyAcOsTIqfJMOk=; b=XxeBtkVA+bzvJT18b9zzhmT5vB9PGG7d6zvCV5OkAa416S6eRwBn1CEOcEVvEs0b+C ldQ194Cqi6aOtnT6rGBEHgmy9be6FGEwMtWS11C8QiyGiEo/StE2GST8quBFJ6kyKxGs zdVfkW4KF7EbhjRfUe+IBrwExc5mNWJqTjDP0=
X-Google-Dkim-Signature	v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=Xtw8spV0c25DTBaWOLU9mn4qmOhw1hyAcOsTIqfJMOk=; b=iMEHQlaoDcoxMZtJ1P4tKLtbSz1DJ+yj6w0j8zK4PhxC7fJ/bPJ605P2hgIxXpU+BE JM4Fcs9FILvbY5LVUbxAZsEKFBBp5m+sG3nrPVIn9yLFqbjugAEODL85F/ztFEa68yW4 iSQLWXWaJvNkVjlwvCu6qE1mikA23DOs/Fyfx0FOgk3IVh9LZvM6SN3f70ZZ2NxWmUjb d2Ey5HuMUtKly/IIvzODVIXWxcoXf0Axs6aaIMFI4L1xy6Mf2fcKTZ9EHD/P7q3dIoJU OgVmYUqav2m8dklLyM+I7Kq+iqNSYQfCjrzIS8gdgmMQDopIBce97FrZJFKsEsv8WlvS eX/Q==
X-Gm-Message-State	ALyK8tJRXxlHScW/ExFF8wOaMXZXTdPTXRFkhCBR56MrBc2iq8HEpTnhSTGM/4Mrbmpm5yyf
X-Received	by 10.194.81.72 with SMTP id y8mr1122723wjx.83.1465823188135; Mon, 13 Jun 2016 06:06:28 -0700 (PDT)
MIME-Version	1.0
Content-Type	text/plain; charset=us-ascii
Content-Disposition	inline
User-Agent	Mutt/1.5.21 (2010-09-15)
Sender	robomod@news.nic.it
List-ID	<linux-kernel.vger.kernel.org>
X-Mailing-List	linux-kernel@vger.kernel.org
Approved	robomod@news.nic.it
Lines	39
Organization	linux.* mail to news gateway
X-Original-Cc	Heiko Stuebner <heiko@sntech.de>, dianders@chromium.org, briannorris@google.com, smbarber@google.com, linux-rockchip@lists.infradead.org, Thomas Gleixner <tglx@linutronix.de>, cf@rock-chips.com, huangtao@rock-chips.com, devicetree@vger.kernel.org, Xing Zheng <zhengxing@rock-chips.com>, Jianqun Xu <jay.xu@rock-chips.com>, Masahiro Yamada <yamada.masahiro@socionext.com>, Brian Norris <briannorris@chromium.org>, linux-kernel@vger.kernel.org, Shawn Lin <shawn.lin@rock-chips.com>, Rob Herring <robh+dt@kernel.org>, Will Deacon <will.deacon@arm.com>, Mark Rutland <mark.rutland@arm.com>, Catalin Marinas <catalin.marinas@arm.com>, linux-arm-kernel@lists.infradead.org
X-Original-Date	Mon, 13 Jun 2016 15:06:25 +0200
X-Original-Message-ID	<20160613130625.GC10634@linaro.org>
X-Original-References	<1465275273-22076-1-git-send-email-wxt@rock-chips.com>
X-Original-Sender	linux-kernel-owner@vger.kernel.org
Xref	csiph.com linux.kernel:1420814

Show key headers only | View raw

On Tue, Jun 07, 2016 at 12:54:29PM +0800, Caesar Wang wrote:
> This series patches had been tested on rockchip inside kernel.
> In order to support the rk3399 SoC timer and turn off interrupts and IPIs to
> save power in idle.

For my personnal information, are the arch_timer in the same power domain 
than the CPU ? IOW, what is the 'always-on' property in the DT ?

> Okay, it still works bootup on rk3288/other SoCs, even though many socs 
> hasn't used
> the broadcast timer.

Yes, unfortunately the SoC design on rk3288 and the previous ones do not 
allow to use a cpuidle driver with cpu/cluster power down, so obviously the 
broadcast timer is pointless on these boards :)

> History version:
> v1:
> https://lkml.org/lkml/2016/5/25/186
> 
> Easy to test for my borad.
> localhost / # cat /proc/interrupts
> CPU0       CPU1       CPU2       CPU3       CPU4       CPU5
> 1:          0          0          0          0          0          0     GICv3  29 Edge      arch_timer
> ...
> 5:          0          0          0          0          0          0     GICv3 113 Level     rk_timer
> ..
> 
> localhost / # cat /proc/timer_list | grep event_handler
> get "event_handler:  hrtimer_interrupt"
> event_handler:  tick_handle_oneshot_broadcast
> event_handler:  hrtimer_interrupt

What are you trying to demonstrate here ? There are no interrupts for both 
arch_timer and rk_timer.

> That should work for my board.

What do you mean ?

Back to linux.kernel | Previous | Next — Next in thread | Find similar | Unroll thread

Thread

Re: [PATCH v2 0/4] clocksource: rockchip/timer: Support rktimer for  rk3399 Daniel Lezcano <daniel.lezcano@linaro.org> - 2016-06-13 15:10 +0200
  Re: [PATCH v2 0/4] clocksource: rockchip/timer: Support rktimer for  rk3399 "Huang, Tao" <huangtao@rock-chips.com> - 2016-06-14 06:10 +0200
    Re: [PATCH v2 0/4] clocksource: rockchip/timer: Support rktimer for  rk3399 Caesar Wang <wxt@rock-chips.com> - 2016-06-14 11:20 +0200

csiph-web