From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Lezcano Subject: Re: Add system timer driver for Mediatek SoCs Date: Tue, 26 Jun 2018 09:31:47 +0200 Message-ID: <561c2c41-ea92-3ad8-a1d7-aeef46cdc601@linaro.org> References: <1529910601-15005-1-git-send-email-stanley.chu@mediatek.com> <1529917631.17448.22.camel@mtkswgap22> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Return-path: In-Reply-To: <1529917631.17448.22.camel@mtkswgap22> Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org To: Stanley Chu Cc: Matthias Brugger , Thomas Gleixner , Rob Herring , linux-kernel@vger.kernel.org, linux-mediatek@lists.infradead.org, devicetree@vger.kernel.org, wsd_upstream@mediatek.com List-Id: devicetree@vger.kernel.org On 25/06/2018 11:07, Stanley Chu wrote: > On Mon, 2018-06-25 at 09:34 +0200, Daniel Lezcano wrote: >> On 25/06/2018 09:09, Stanley Chu wrote: >>> This patch adds a new driver for system timer on the Mediatek SoCs. >> >> Please elaborate why we need yet another timer. >> >> Is this timer present on all Mediatek platform ? Does it always co-exist >> with the existing one ? >> >> >> >> > Hi Daniel, > > This new "system timer" only exists on recent Mediatek platforms, > for example, MT6765. We will upstream driver first, and then update > device tree on MT6765 later. > > System timer is designed and optimized as a SoC timer for > tick-broadcasting. Besides timer IP has simpler register manipulation > and friendly low-power design. > > There is no plan to remove existed "General Purpose Timer" now. It is unclear what "General Purpose Timer" and "System Timer" differ except they are different timers running on different platforms. Merge them into a single timer-mediatek.c file. -- Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog