From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail-io0-f196.google.com ([209.85.223.196]:36706 "EHLO mail-io0-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752174AbcBZPeu (ORCPT ); Fri, 26 Feb 2016 10:34:50 -0500 Received: by mail-io0-f196.google.com with SMTP id b188so8611466iof.3 for ; Fri, 26 Feb 2016 07:34:50 -0800 (PST) Subject: Re: [PATCH v4 0/6] watchdog: pnx4008: add restart handler From: Sylvain Lemieux To: Guenter Roeck Cc: linux-arm-kernel@lists.infradead.org, arnd@arndb.de, wim@iguana.be, manabian@gmail.com, linux-watchdog@vger.kernel.org, stigge@antcom.de, vz@mleia.com In-Reply-To: <56D062DD.1060707@roeck-us.net> References: <1456425100-28136-1-git-send-email-slemieux.tyco@gmail.com> <56D062DD.1060707@roeck-us.net> Content-Type: text/plain; charset="UTF-8" Date: Fri, 26 Feb 2016 10:34:46 -0500 Message-ID: <1456500886.2194.6.camel@localhost> Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Sender: linux-watchdog-owner@vger.kernel.org List-Id: linux-watchdog@vger.kernel.org On Fri, 2016-02-26 at 06:36 -0800, Guenter Roeck wrote: > On 02/25/2016 10:31 AM, Sylvain Lemieux wrote: > > From: Sylvain Lemieux > > > > The LPC32xx provide the capability to perform a soft or hard reset; > > a soft reset will no reset the external peripheral > > (i.e. RESOUT_N output not asserted). > > > > This changeset migrate the restart code from the "mach-lpc32xx" to > > the "pnx-4008" watchdog driver and add the soft reset capability. > > > > The default behavior (i.e. hard reset) remain unchanged. > > > > The core restart handler support added by this change: > > 2165bf524da5f5e496d - "watchdog: core: add restart handler support" > > is not use in this changeset; it does not support the reboot mode. > > I finally had the time to look into this. Sorry for the late reply. > > Changing the watchdog core to pass the restart mode (or action) > and the cmd/data pointer to the driver would be trivial and should > be done instead. > Thanks for the feedback. Do you want me to rebase this patchset on the patch you will submit to update the core restart handler support? > I wrote a patch to do just that. I'll send it out later today for review, > assuming it passes my internal tests. > > Guenter > > > > Note: > > Patch #5 was generated with the option "--unified=2" to ensure > > it can be apply before or after the changes for the new > > LPC32xx irqchip driver: > > - http://permalink.gmane.org/gmane.linux.drivers.devicetree/144683 > > --- > > Changes from v3 to v4: > > * Update the patch as per the feedback received from: > > Arnd: http://permalink.gmane.org/gmane.linux.watchdog/3329 > > - Switch patch #5 and #6 order. > > > > Changes from v2 to v3: > > * Update the patch as per the feedback received from: > > Joachim: http://permalink.gmane.org/gmane.linux.watchdog/3303 > > - remove explicit clock enable from restart handler. > > - verify "register_restart_handler" return value. > > - add call to unregister_restart_handler". > > - do the restart handling directly in "pnx4008_restart_handler". > > * Move the change to "phy3250.c" into a separate patch and generate > > the new patch with the "--unified=2" option. > > > > Changes from v1 to v2: > > * Rename patch title; > > was "arm: lpc32xx: restart: add support for soft reset" > > * Update the patch as per the feedback received from: > > Joachim: http://permalink.gmane.org/gmane.linux.ports.arm.kernel/479791 > > - migrate restart code from "mach-lpc32xx" to "pnx-4008" driver. > > * Add soft reset support to "pnx-4008" driver instead of "mach-lpc32xx". > > * Use define available in "pnx-4008" when writting to watchdog register. > > * Remove restart handler from "mach-lpc32xx". > > * Remove restart hook from "phy3250.c". > > * Update driver logging during power-on. > > > > Sylvain Lemieux (6): > > watchdog: pnx4008: update logging during power-on > > watchdog: pnx4008: add restart handler > > watchdog: pnx4008: add support for soft reset > > watchdog: pnx4008: restart: support "cmd" from userspace > > arm: lpc32xx: phy3250 remove restart hook > > arm: lpc32xx: remove restart handler > > > > arch/arm/mach-lpc32xx/common.c | 15 ------------ > > arch/arm/mach-lpc32xx/common.h | 1 - > > arch/arm/mach-lpc32xx/phy3250.c | 1 - > > drivers/watchdog/pnx4008_wdt.c | 51 +++++++++++++++++++++++++++++++++++++++-- > > 4 files changed, 49 insertions(+), 19 deletions(-) > > >