linux-kernel - Re: Add system timer driver for Mediatek SoCs

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <561c2c41-ea92-3ad8-a1d7-aeef46cdc601@linaro.org>
Date:   Tue, 26 Jun 2018 09:31:47 +0200
From:   Daniel Lezcano <daniel.lezcano@...aro.org>
To:     Stanley Chu <stanley.chu@...iatek.com>
Cc:     Matthias Brugger <matthias.bgg@...il.com>,
        Thomas Gleixner <tglx@...utronix.de>,
        Rob Herring <robh+dt@...nel.org>, linux-kernel@...r.kernel.org,
        linux-mediatek@...ts.infradead.org, devicetree@...r.kernel.org,
        wsd_upstream@...iatek.com
Subject: Re: Add system timer driver for Mediatek SoCs

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.

-- 
 <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog