From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934393AbdAKXg4 (ORCPT ); Wed, 11 Jan 2017 18:36:56 -0500 Received: from mail-qk0-f194.google.com ([209.85.220.194]:36187 "EHLO mail-qk0-f194.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1761949AbdAKXgx (ORCPT ); Wed, 11 Jan 2017 18:36:53 -0500 MIME-Version: 1.0 In-Reply-To: <1484091325-9199-21-git-send-email-linux@roeck-us.net> References: <1484091325-9199-1-git-send-email-linux@roeck-us.net> <1484091325-9199-21-git-send-email-linux@roeck-us.net> From: Andy Shevchenko Date: Thu, 12 Jan 2017 01:36:52 +0200 Message-ID: Subject: Re: [PATCH 20/62] watchdog: iTCO_wdt: Replace shutdown function with call to watchdog_stop_on_reboot To: Guenter Roeck Cc: Wim Van Sebroeck , linux-watchdog@vger.kernel.org, "linux-kernel@vger.kernel.org" Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Jan 11, 2017 at 1:34 AM, Guenter Roeck wrote: > The shutdown function calls the stop function. > Call watchdog_stop_on_reboot() from probe instead. > > The conversion was done automatically with coccinelle using the > following semantic patches. The semantic patches and the scripts used > to generate this commit log are available at > https://github.com/groeck/coccinelle-patches > > - Replace shutdown function with call to watchdog_stop_on_reboot() > Reviewed-by: Andy Shevchenko > Signed-off-by: Guenter Roeck > --- > drivers/watchdog/iTCO_wdt.c | 9 +-------- > 1 file changed, 1 insertion(+), 8 deletions(-) > > diff --git a/drivers/watchdog/iTCO_wdt.c b/drivers/watchdog/iTCO_wdt.c > index d8bb7bf5e669..3d0abc0d59b4 100644 > --- a/drivers/watchdog/iTCO_wdt.c > +++ b/drivers/watchdog/iTCO_wdt.c > @@ -520,6 +520,7 @@ static int iTCO_wdt_probe(struct platform_device *pdev) > WATCHDOG_TIMEOUT); > } > > + watchdog_stop_on_reboot(&p->wddev); > ret = devm_watchdog_register_device(dev, &p->wddev); > if (ret != 0) { > pr_err("cannot register watchdog device (err=%d)\n", ret); > @@ -543,13 +544,6 @@ static int iTCO_wdt_remove(struct platform_device *pdev) > return 0; > } > > -static void iTCO_wdt_shutdown(struct platform_device *pdev) > -{ > - struct iTCO_wdt_private *p = platform_get_drvdata(pdev); > - > - iTCO_wdt_stop(&p->wddev); > -} > - > #ifdef CONFIG_PM_SLEEP > /* > * Suspend-to-idle requires this, because it stops the ticks and timekeeping, so > @@ -603,7 +597,6 @@ static const struct dev_pm_ops iTCO_wdt_pm = { > static struct platform_driver iTCO_wdt_driver = { > .probe = iTCO_wdt_probe, > .remove = iTCO_wdt_remove, > - .shutdown = iTCO_wdt_shutdown, > .driver = { > .name = DRV_NAME, > .pm = ITCO_WDT_PM_OPS, > -- > 2.7.4 > -- With Best Regards, Andy Shevchenko