From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932970Ab3LDS2c (ORCPT ); Wed, 4 Dec 2013 13:28:32 -0500 Received: from mail-pd0-f172.google.com ([209.85.192.172]:39289 "EHLO mail-pd0-f172.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753590Ab3LDS23 (ORCPT ); Wed, 4 Dec 2013 13:28:29 -0500 Date: Wed, 4 Dec 2013 10:28:24 -0800 From: Guenter Roeck To: Sekhar Nori Cc: Ivan Khoronzhuk , Santosh Shilimkar , linux-arm-kernel@lists.infradead.org, mark.rutland@arm.com, wim@iguana.be, linux-watchdog@vger.kernel.org, devicetree@vger.kernel.org, Pawel.Moll@arm.com, swarren@wwwdotorg.org, ijc+devicetree@hellion.org.uk, galak@kernel.crashing.org, rob.herring@calxeda.com, linux-kernel@vger.kernel.org, grant.likely@linaro.org, grygorii.strashko@ti.com Subject: Re: [PATCH v4 1/5] watchdog: davinci: change driver to use WDT core Message-ID: <20131204182824.GA14499@roeck-us.net> References: <1385556508-3935-1-git-send-email-ivan.khoronzhuk@ti.com> <1385556508-3935-2-git-send-email-ivan.khoronzhuk@ti.com> <529F6EBE.5050703@ti.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <529F6EBE.5050703@ti.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Dec 04, 2013 at 11:34:46PM +0530, Sekhar Nori wrote: > On 11/27/2013 6:18 PM, Ivan Khoronzhuk wrote: > > To reduce code duplicate and increase code readability use WDT core > > code to handle WDT interface. > > > > Remove io_lock as the WDT core uses mutex to lock each wdt device. > > Remove wdt_state as the WDT core tracks state with its own variable. > > > > The watchdog_init_timeout() can read timeout value from timeout-sec > > property if the passed value is out of bounds. The heartbeat is > > initialized in next way. If heartbeat is not set thought module > > parameter, try to read it's value from WDT node timeout-sec property. > > If node has no one, use default value. > > > > The heartbeat is hold in wdd->timeout by WDT core, so use it in > > order to set timeout period. > > > > Acked-by: Santosh Shilimkar > > Reviewed-by: Guenter Roeck > > Signed-off-by: Ivan Khoronzhuk > > --- > > drivers/watchdog/Kconfig | 2 + > > drivers/watchdog/davinci_wdt.c | 147 ++++++++++------------------------------ > > 2 files changed, 37 insertions(+), 112 deletions(-) > > > > diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig > > index 5be6e91..eb8c89d 100644 > > --- a/drivers/watchdog/Kconfig > > +++ b/drivers/watchdog/Kconfig > > @@ -271,6 +271,8 @@ config IOP_WATCHDOG > > config DAVINCI_WATCHDOG > > tristate "DaVinci watchdog" > > depends on ARCH_DAVINCI > > + select WATCHDOG_CORE > > + select WATCHDOG_NOWAYOUT > > Its not clear for change log why NOWAYOUT needs to be forced on all the > time for all users of DaVinci watchdog. > Actually, thinking about it, it is not necessary and can be replaced with watchdog_set_nowayout(wdd, 1); In other words, there is no need to force NOWAYOUT on _other_ watchdogs which may be present in the system. Still, you are right, it would be nice to explain in the changelog (or maybe even better as comment in the code) why it is enforced. Guenter