From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail.savoirfairelinux.com ([208.88.110.44]:36337 "EHLO mail.savoirfairelinux.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751201AbbKCSTY (ORCPT ); Tue, 3 Nov 2015 13:19:24 -0500 Date: Tue, 3 Nov 2015 13:19:20 -0500 From: Vivien Didelot To: Guenter Roeck Cc: Damien Riegel , linux-watchdog@vger.kernel.org, Wim Van Sebroeck , kernel@savoirfairelinux.com Subject: Re: [PATCH 00/13] watchdog: factorize restart handler registration Message-ID: <20151103181920.GA23103@ketchup.mtl.sfl> References: <1446567166-10156-1-git-send-email-damien.riegel@savoirfairelinux.com> <5638F812.8010802@roeck-us.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <5638F812.8010802@roeck-us.net> Sender: linux-watchdog-owner@vger.kernel.org List-Id: linux-watchdog@vger.kernel.org On Nov. Tuesday 03 (45) 10:08 AM, Guenter Roeck wrote: > On 11/03/2015 08:12 AM, Damien Riegel wrote: > >Many drivers implements the exact same piece of code to register a > >restart handler. It can be nice to factorize this in the watchdog core. > > > >The first patch adds an optional restart watchdog operation. If a driver > >defines this operation, a restart handler is registered. By default, the > >restart handler priority is set to 0, but a helper function > >watchdog_set_restart_priority is provided to change it. > > > >The following patches bring this change to the current watchdog drivers > >that use watchdog_core. > > > >This change has been compile-tested on da9063, imx2, lpc18xx, imgpdc. > >It has been tested with (not mainlined yet) ts-4800's watchdog driver. > > > >Damien Riegel (13): > > watchdog: core: add restart handler support > > watchdog: bcm47xx_wdt: use core restart handler > > watchdog: da9063_wdt: use core restart handler > > watchdog: digicolor_wdt: use core restart handler > > watchdog: imgpdc_wdt: use core restart handler > > watchdog: imx2_wdt: use core restart handler > > watchdog: lpc18xx_wdt: use core restart handler > > watchdog: meson_wdt: use core restart handler > > watchdog: moxart_wdt: use core restart handler > > watchdog: mtk_wdt: use core restart handler > > watchdog: qcom-wdt: use core restart handler > > watchdog: s3c2410_wdt: use core restart handler > > watchdog: sunxi_wdt: use core restart handler > > > > Damien, > > "v2" in the subject line as well as a change log would have been appreciated. > You are making it (hopefully not intentionally) difficult for maintainers > to separate v1 from v2. You might want to keep in mind that maintainers are > in general quite busy and might thus be a bit annoyed about such additional > (and unnecessary) work. Hi Guenter, I assume the shot here. I suggested no subject prefix, since the first patchset was RFC. Should we use "v2" even after an RFC? Or "v1" maybe? Sorry about that, -v