2017-11-06 10:19:01

by Tomer Maimon

[permalink] [raw]
Subject: Re: [PATCH v2 0/2] clocksource: npcm: add NPCM7xx timer driver

On 4 November 2017 at 02:28, Brendan Higgins <[email protected]> wrote:
> On Wed, Nov 1, 2017 at 5:16 AM, Tomer Maimon <[email protected]> wrote:
>> Addressed comments from:
>> - Daniel Lezcano: https://www.spinics.net/lists/devicetree/msg196683.html
>>
>> Changes since version 1:
>> - Rename driver name
>> - Removing unnecessary dependencies in configuration
>> - Adding prefix to the macros
>> - No changes to dt-binding documentation since v1
>>
>> Asked Brendon to modify NPCM7xx device tree for support NPCM7xx timer driver
>> https://www.spinics.net/lists/arm-kernel/msg614424.html
>
> Rob Herring asks that we keep the device tree binding the same as I have it.
> Unless, there is some part of the conversation that I am missing.

you are right that he asked to keep the competible as you have in the
device tree but
the reg is incorrect in your device tree, please modify it as follow:

timer0: timer@f0008000 {
compatible = "nuvoton,npcm750-timer";
interrupts = <0 32 4>;
reg = <0xf0008000 0x1000>;
clocks = <&clk NPCM7XX_CLK_TIMER>;
};

>
>>
>> Changes have been tested on the NPCM750 evaluation board.
>>
>> Tomer Maimon (2):
>> clocksource: npcm: add NPCM7xx timer driver
>> dt-binding: timer: document NPCM7xx timer DT bindings
>>
>> .../bindings/timer/nuvoton,npcm7xx-timer.txt | 25 +++
>> drivers/clocksource/Kconfig | 8 +
>> drivers/clocksource/Makefile | 1 +
>> drivers/clocksource/npcm7xx-timer.c | 224 +++++++++++++++++++++
>> 4 files changed, 258 insertions(+)
>> create mode 100644 Documentation/devicetree/bindings/timer/nuvoton,npcm7xx-timer.txt
>> create mode 100644 drivers/clocksource/npcm7xx-timer.c
>>
>> --
>> 2.14.1
>>

Thanks!

From 1583093292194637191@xxx Sat Nov 04 00:30:13 +0000 2017
X-GM-THRID: 1582866100793273296
X-Gmail-Labels: Inbox,Category Forums,HistoricalUnread