[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20210113152347.GA2472428@robh.at.kernel.org>
Date: Wed, 13 Jan 2021 09:23:47 -0600
From: Rob Herring <robh@...nel.org>
To: Jonathan Neuschäfer <j.neuschaefer@....net>
Cc: Rob Herring <robh+dt@...nel.org>,
Patrick Venture <venture@...gle.com>,
Tali Perry <tali.perry1@...il.com>,
Avi Fishman <avifishman70@...il.com>,
linux-kernel@...r.kernel.org,
Daniel Lezcano <daniel.lezcano@...aro.org>,
devicetree@...r.kernel.org, Tomer Maimon <tmaimon77@...il.com>,
Benjamin Fair <benjaminfair@...gle.com>,
Nancy Yuen <yuenn@...gle.com>, openbmc@...ts.ozlabs.org,
Thomas Gleixner <tglx@...utronix.de>
Subject: Re: [PATCH v2] dt-bindings: timer: nuvoton: Clarify that interrupt
of timer 0 should be specified
On Fri, 08 Jan 2021 17:30:04 +0100, Jonathan Neuschäfer wrote:
> The NPCM750 Timer/Watchdog Controller has multiple interrupt lines,
> connected to multiple timers. The driver uses timer 0 for timer
> interrupts, so the interrupt line corresponding to timer 0 should be
> specified in DT.
>
> I removed the mention of "flags for falling edge", because the timer
> controller uses high-level interrupts rather than falling-edge
> interrupts, and whether flags should be specified is up the interrupt
> controller's DT binding.
>
> Signed-off-by: Jonathan Neuschäfer <j.neuschaefer@....net>
> ---
>
> v2:
> - Fix a typo in the word "watchdog"
> ---
> .../devicetree/bindings/timer/nuvoton,npcm7xx-timer.txt | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
Acked-by: Rob Herring <robh@...nel.org>
Powered by blists - more mailing lists