Re: [PATCH 2/4] watchdog: Add a flag to indicate the watchdog doesn't reboot things

[Hans de Goede <hdegoede@redhat.com>]

Looks good, ack.

Acked-by: Hans de Goede <hdegoede@redhat.com>

On 03/21/2012 04:25 PM, Alan Cox wrote:
> From: Alan Cox<alan@linux.intel.com>
>
> Some watchdogs merely trigger external alarms and controls. In a managed
> environment this is very useful but we want drivers to be able to figure
> out which is which now multiple dogs can be loaded. Thus add an ALARMONLY
> feature flag.
>
> Signed-off-by: Alan Cox<alan@linux.intel.com>
> ---
>
>   include/linux/watchdog.h |    2 ++
>   1 files changed, 2 insertions(+), 0 deletions(-)
>
>
> diff --git a/include/linux/watchdog.h b/include/linux/watchdog.h
> index 0d5c3af..fdd5dbb 100644
> --- a/include/linux/watchdog.h
> +++ b/include/linux/watchdog.h
> @@ -45,6 +45,8 @@ struct watchdog_info {
>   #define	WDIOF_SETTIMEOUT	0x0080  /* Set timeout (in seconds) */
>   #define	WDIOF_MAGICCLOSE	0x0100	/* Supports magic close char */
>   #define	WDIOF_PRETIMEOUT	0x0200  /* Pretimeout (in seconds), get/set */
> +#define WDIOF_ALARMONLY		0x0400	/* Watchdog triggers a management or
> +					   other external alarm not a reboot */
>   #define	WDIOF_KEEPALIVEPING	0x8000	/* Keep alive ping reply */
>
>   #define	WDIOS_DISABLECARD	0x0001	/* Turn off the watchdog timer */
>