From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dmitry Osipenko Subject: Re: [PATCH v3 2/7] clocksource: Add Tegra186 timers support Date: Fri, 3 Apr 2020 23:51:57 +0300 Message-ID: References: <20200403202209.299823-1-thierry.reding@gmail.com> <20200403202209.299823-3-thierry.reding@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Return-path: In-Reply-To: <20200403202209.299823-3-thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> Content-Language: en-US Sender: linux-tegra-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Thierry Reding , Daniel Lezcano , Thomas Gleixner Cc: Rob Herring , Jon Hunter , devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-tegra@vger.kernel.org 03.04.2020 23:22, Thierry Reding пишет: ... > +static irqreturn_t tegra186_timer_irq(int irq, void *data) > +{ > + struct tegra186_timer *tegra = data; > + > + if (watchdog_active(&tegra->wdt->base)) { > + tegra186_wdt_disable(tegra->wdt); > + tegra186_wdt_enable(tegra->wdt); > + } Shouldn't this return IRQ_NONE if watchdog is inactive? > + return IRQ_HANDLED; > +}