From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail-io0-f193.google.com ([209.85.223.193]:35826 "EHLO mail-io0-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750973AbcBXRrM (ORCPT ); Wed, 24 Feb 2016 12:47:12 -0500 Received: by mail-io0-f193.google.com with SMTP id z135so3225621iof.2 for ; Wed, 24 Feb 2016 09:47:12 -0800 (PST) From: Sylvain Lemieux To: linux-arm-kernel@lists.infradead.org, wim@iguana.be, manabian@gmail.com Cc: linux-watchdog@vger.kernel.org, linux@roeck-us.net, stigge@antcom.de, vz@mleia.com Subject: [PATCH v3 0/6] watchdog: pnx4008: add restart handler Date: Wed, 24 Feb 2016 12:46:52 -0500 Message-Id: <1456336018-6620-1-git-send-email-slemieux.tyco@gmail.com> Sender: linux-watchdog-owner@vger.kernel.org List-Id: linux-watchdog@vger.kernel.org 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. Note: The patch #6 was generated with the option "--unified=2" to ensure it can be apply before or after the following git pull request for the LPC32xx platform: - http://permalink.gmane.org/gmane.linux.ports.arm.kernel/477137 - http://permalink.gmane.org/gmane.linux.ports.arm.kernel/477138 --- 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: remove restart handler arm: lpc32xx: phy3250 remove restart hook 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(-) -- 1.8.3.1