From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from eusmtp01.atmel.com ([212.144.249.243]:20279 "EHLO eusmtp01.atmel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752234Ab2KTNBG (ORCPT ); Tue, 20 Nov 2012 08:01:06 -0500 Message-ID: <50AB7F0F.1090005@atmel.com> Date: Tue, 20 Nov 2012 14:01:03 +0100 From: Nicolas Ferre MIME-Version: 1.0 To: Fabio Porcedda CC: Wim Van Sebroeck , , , Jean-Christophe PLAGNIOL-VILLARD , Andrew Victor , Jason Cooper , Andrew Lunn , Ben Dooks , Kukjin Kim , Subject: Re: [PATCH v5 6/8] watchdog: at91sam9_wdt: add timeout-sec property binding References: <1353334243-16703-1-git-send-email-fabio.porcedda@gmail.com> <1353334243-16703-7-git-send-email-fabio.porcedda@gmail.com> In-Reply-To: <1353334243-16703-7-git-send-email-fabio.porcedda@gmail.com> Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit Sender: linux-watchdog-owner@vger.kernel.org List-Id: linux-watchdog@vger.kernel.org On 11/19/2012 03:10 PM, Fabio Porcedda : > Tested on at91sam9260 board (evk-pro3). > > Signed-off-by: Fabio Porcedda > Cc: Andrew Victor > Cc: Nicolas Ferre Seems good: Acked-by: Nicolas Ferre > Cc: Jean-Christophe PLAGNIOL-VILLARD > --- > Documentation/devicetree/bindings/watchdog/atmel-wdt.txt | 4 ++++ > drivers/watchdog/Kconfig | 1 + > drivers/watchdog/at91sam9_wdt.c | 16 +++++++++++++--- > 3 files changed, 18 insertions(+), 3 deletions(-) > > diff --git a/Documentation/devicetree/bindings/watchdog/atmel-wdt.txt b/Documentation/devicetree/bindings/watchdog/atmel-wdt.txt > index 2957ebb..fcdd48f 100644 > --- a/Documentation/devicetree/bindings/watchdog/atmel-wdt.txt > +++ b/Documentation/devicetree/bindings/watchdog/atmel-wdt.txt > @@ -7,9 +7,13 @@ Required properties: > - reg: physical base address of the controller and length of memory mapped > region. > > +Optional properties: > +- timeout-sec: contains the watchdog timeout in seconds. > + > Example: > > watchdog@fffffd40 { > compatible = "atmel,at91sam9260-wdt"; > reg = <0xfffffd40 0x10>; > + timeout-sec = <10>; > }; > diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig > index ad1bb93..dda695f 100644 > --- a/drivers/watchdog/Kconfig > +++ b/drivers/watchdog/Kconfig > @@ -114,6 +114,7 @@ config AT91RM9200_WATCHDOG > config AT91SAM9X_WATCHDOG > tristate "AT91SAM9X / AT91CAP9 watchdog" > depends on ARCH_AT91 && !ARCH_AT91RM9200 > + select WATCHDOG_CORE > help > Watchdog timer embedded into AT91SAM9X and AT91CAP9 chips. This will > reboot your system when the timeout is reached. > diff --git a/drivers/watchdog/at91sam9_wdt.c b/drivers/watchdog/at91sam9_wdt.c > index dc42e44..d864dc4 100644 > --- a/drivers/watchdog/at91sam9_wdt.c > +++ b/drivers/watchdog/at91sam9_wdt.c > @@ -57,8 +57,10 @@ > #define WDT_TIMEOUT (HZ/2) > > /* User land timeout */ > +#define MIN_HEARTBEAT 1 > +#define MAX_HEARTBEAT 16 > #define WDT_HEARTBEAT 15 > -static int heartbeat = WDT_HEARTBEAT; > +static int heartbeat; > module_param(heartbeat, int, 0); > MODULE_PARM_DESC(heartbeat, "Watchdog heartbeats in seconds. " > "(default = " __MODULE_STRING(WDT_HEARTBEAT) ")"); > @@ -255,6 +257,12 @@ static struct miscdevice at91wdt_miscdev = { > .fops = &at91wdt_fops, > }; > > +static struct watchdog_device at91wdt_wdd __initdata = { > + .timeout = WDT_HEARTBEAT, > + .min_timeout = MIN_HEARTBEAT, > + .max_timeout = MAX_HEARTBEAT, > +}; > + > static int __init at91wdt_probe(struct platform_device *pdev) > { > struct resource *r; > @@ -273,6 +281,8 @@ static int __init at91wdt_probe(struct platform_device *pdev) > return -ENOMEM; > } > > + watchdog_init_timeout(&at91wdt_wdd, heartbeat, pdev->dev.of_node); > + > /* Set watchdog */ > res = at91_wdt_settimeout(ms_to_ticks(WDT_HW_TIMEOUT * 1000)); > if (res) > @@ -282,12 +292,12 @@ static int __init at91wdt_probe(struct platform_device *pdev) > if (res) > return res; > > - at91wdt_private.next_heartbeat = jiffies + heartbeat * HZ; > + at91wdt_private.next_heartbeat = jiffies + at91wdt_wdd.timeout * HZ; > setup_timer(&at91wdt_private.timer, at91_ping, 0); > mod_timer(&at91wdt_private.timer, jiffies + WDT_TIMEOUT); > > pr_info("enabled (heartbeat=%d sec, nowayout=%d)\n", > - heartbeat, nowayout); > + at91wdt_wdd.timeout, nowayout); > > return 0; > } > -- Nicolas Ferre From mboxrd@z Thu Jan 1 00:00:00 1970 From: nicolas.ferre@atmel.com (Nicolas Ferre) Date: Tue, 20 Nov 2012 14:01:03 +0100 Subject: [PATCH v5 6/8] watchdog: at91sam9_wdt: add timeout-sec property binding In-Reply-To: <1353334243-16703-7-git-send-email-fabio.porcedda@gmail.com> References: <1353334243-16703-1-git-send-email-fabio.porcedda@gmail.com> <1353334243-16703-7-git-send-email-fabio.porcedda@gmail.com> Message-ID: <50AB7F0F.1090005@atmel.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 11/19/2012 03:10 PM, Fabio Porcedda : > Tested on at91sam9260 board (evk-pro3). > > Signed-off-by: Fabio Porcedda > Cc: Andrew Victor > Cc: Nicolas Ferre Seems good: Acked-by: Nicolas Ferre > Cc: Jean-Christophe PLAGNIOL-VILLARD > --- > Documentation/devicetree/bindings/watchdog/atmel-wdt.txt | 4 ++++ > drivers/watchdog/Kconfig | 1 + > drivers/watchdog/at91sam9_wdt.c | 16 +++++++++++++--- > 3 files changed, 18 insertions(+), 3 deletions(-) > > diff --git a/Documentation/devicetree/bindings/watchdog/atmel-wdt.txt b/Documentation/devicetree/bindings/watchdog/atmel-wdt.txt > index 2957ebb..fcdd48f 100644 > --- a/Documentation/devicetree/bindings/watchdog/atmel-wdt.txt > +++ b/Documentation/devicetree/bindings/watchdog/atmel-wdt.txt > @@ -7,9 +7,13 @@ Required properties: > - reg: physical base address of the controller and length of memory mapped > region. > > +Optional properties: > +- timeout-sec: contains the watchdog timeout in seconds. > + > Example: > > watchdog at fffffd40 { > compatible = "atmel,at91sam9260-wdt"; > reg = <0xfffffd40 0x10>; > + timeout-sec = <10>; > }; > diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig > index ad1bb93..dda695f 100644 > --- a/drivers/watchdog/Kconfig > +++ b/drivers/watchdog/Kconfig > @@ -114,6 +114,7 @@ config AT91RM9200_WATCHDOG > config AT91SAM9X_WATCHDOG > tristate "AT91SAM9X / AT91CAP9 watchdog" > depends on ARCH_AT91 && !ARCH_AT91RM9200 > + select WATCHDOG_CORE > help > Watchdog timer embedded into AT91SAM9X and AT91CAP9 chips. This will > reboot your system when the timeout is reached. > diff --git a/drivers/watchdog/at91sam9_wdt.c b/drivers/watchdog/at91sam9_wdt.c > index dc42e44..d864dc4 100644 > --- a/drivers/watchdog/at91sam9_wdt.c > +++ b/drivers/watchdog/at91sam9_wdt.c > @@ -57,8 +57,10 @@ > #define WDT_TIMEOUT (HZ/2) > > /* User land timeout */ > +#define MIN_HEARTBEAT 1 > +#define MAX_HEARTBEAT 16 > #define WDT_HEARTBEAT 15 > -static int heartbeat = WDT_HEARTBEAT; > +static int heartbeat; > module_param(heartbeat, int, 0); > MODULE_PARM_DESC(heartbeat, "Watchdog heartbeats in seconds. " > "(default = " __MODULE_STRING(WDT_HEARTBEAT) ")"); > @@ -255,6 +257,12 @@ static struct miscdevice at91wdt_miscdev = { > .fops = &at91wdt_fops, > }; > > +static struct watchdog_device at91wdt_wdd __initdata = { > + .timeout = WDT_HEARTBEAT, > + .min_timeout = MIN_HEARTBEAT, > + .max_timeout = MAX_HEARTBEAT, > +}; > + > static int __init at91wdt_probe(struct platform_device *pdev) > { > struct resource *r; > @@ -273,6 +281,8 @@ static int __init at91wdt_probe(struct platform_device *pdev) > return -ENOMEM; > } > > + watchdog_init_timeout(&at91wdt_wdd, heartbeat, pdev->dev.of_node); > + > /* Set watchdog */ > res = at91_wdt_settimeout(ms_to_ticks(WDT_HW_TIMEOUT * 1000)); > if (res) > @@ -282,12 +292,12 @@ static int __init at91wdt_probe(struct platform_device *pdev) > if (res) > return res; > > - at91wdt_private.next_heartbeat = jiffies + heartbeat * HZ; > + at91wdt_private.next_heartbeat = jiffies + at91wdt_wdd.timeout * HZ; > setup_timer(&at91wdt_private.timer, at91_ping, 0); > mod_timer(&at91wdt_private.timer, jiffies + WDT_TIMEOUT); > > pr_info("enabled (heartbeat=%d sec, nowayout=%d)\n", > - heartbeat, nowayout); > + at91wdt_wdd.timeout, nowayout); > > return 0; > } > -- Nicolas Ferre From mboxrd@z Thu Jan 1 00:00:00 1970 From: Nicolas Ferre Subject: Re: [PATCH v5 6/8] watchdog: at91sam9_wdt: add timeout-sec property binding Date: Tue, 20 Nov 2012 14:01:03 +0100 Message-ID: <50AB7F0F.1090005@atmel.com> References: <1353334243-16703-1-git-send-email-fabio.porcedda@gmail.com> <1353334243-16703-7-git-send-email-fabio.porcedda@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1353334243-16703-7-git-send-email-fabio.porcedda-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: devicetree-discuss-bounces+gldd-devicetree-discuss=m.gmane.org-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org Sender: "devicetree-discuss" To: Fabio Porcedda Cc: Andrew Lunn , Kukjin Kim , Jason Cooper , devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org, Wim Van Sebroeck , Ben Dooks , Andrew Victor , linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, linux-watchdog-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: devicetree@vger.kernel.org On 11/19/2012 03:10 PM, Fabio Porcedda : > Tested on at91sam9260 board (evk-pro3). > > Signed-off-by: Fabio Porcedda > Cc: Andrew Victor > Cc: Nicolas Ferre Seems good: Acked-by: Nicolas Ferre > Cc: Jean-Christophe PLAGNIOL-VILLARD > --- > Documentation/devicetree/bindings/watchdog/atmel-wdt.txt | 4 ++++ > drivers/watchdog/Kconfig | 1 + > drivers/watchdog/at91sam9_wdt.c | 16 +++++++++++++--- > 3 files changed, 18 insertions(+), 3 deletions(-) > > diff --git a/Documentation/devicetree/bindings/watchdog/atmel-wdt.txt b/Documentation/devicetree/bindings/watchdog/atmel-wdt.txt > index 2957ebb..fcdd48f 100644 > --- a/Documentation/devicetree/bindings/watchdog/atmel-wdt.txt > +++ b/Documentation/devicetree/bindings/watchdog/atmel-wdt.txt > @@ -7,9 +7,13 @@ Required properties: > - reg: physical base address of the controller and length of memory mapped > region. > > +Optional properties: > +- timeout-sec: contains the watchdog timeout in seconds. > + > Example: > > watchdog@fffffd40 { > compatible = "atmel,at91sam9260-wdt"; > reg = <0xfffffd40 0x10>; > + timeout-sec = <10>; > }; > diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig > index ad1bb93..dda695f 100644 > --- a/drivers/watchdog/Kconfig > +++ b/drivers/watchdog/Kconfig > @@ -114,6 +114,7 @@ config AT91RM9200_WATCHDOG > config AT91SAM9X_WATCHDOG > tristate "AT91SAM9X / AT91CAP9 watchdog" > depends on ARCH_AT91 && !ARCH_AT91RM9200 > + select WATCHDOG_CORE > help > Watchdog timer embedded into AT91SAM9X and AT91CAP9 chips. This will > reboot your system when the timeout is reached. > diff --git a/drivers/watchdog/at91sam9_wdt.c b/drivers/watchdog/at91sam9_wdt.c > index dc42e44..d864dc4 100644 > --- a/drivers/watchdog/at91sam9_wdt.c > +++ b/drivers/watchdog/at91sam9_wdt.c > @@ -57,8 +57,10 @@ > #define WDT_TIMEOUT (HZ/2) > > /* User land timeout */ > +#define MIN_HEARTBEAT 1 > +#define MAX_HEARTBEAT 16 > #define WDT_HEARTBEAT 15 > -static int heartbeat = WDT_HEARTBEAT; > +static int heartbeat; > module_param(heartbeat, int, 0); > MODULE_PARM_DESC(heartbeat, "Watchdog heartbeats in seconds. " > "(default = " __MODULE_STRING(WDT_HEARTBEAT) ")"); > @@ -255,6 +257,12 @@ static struct miscdevice at91wdt_miscdev = { > .fops = &at91wdt_fops, > }; > > +static struct watchdog_device at91wdt_wdd __initdata = { > + .timeout = WDT_HEARTBEAT, > + .min_timeout = MIN_HEARTBEAT, > + .max_timeout = MAX_HEARTBEAT, > +}; > + > static int __init at91wdt_probe(struct platform_device *pdev) > { > struct resource *r; > @@ -273,6 +281,8 @@ static int __init at91wdt_probe(struct platform_device *pdev) > return -ENOMEM; > } > > + watchdog_init_timeout(&at91wdt_wdd, heartbeat, pdev->dev.of_node); > + > /* Set watchdog */ > res = at91_wdt_settimeout(ms_to_ticks(WDT_HW_TIMEOUT * 1000)); > if (res) > @@ -282,12 +292,12 @@ static int __init at91wdt_probe(struct platform_device *pdev) > if (res) > return res; > > - at91wdt_private.next_heartbeat = jiffies + heartbeat * HZ; > + at91wdt_private.next_heartbeat = jiffies + at91wdt_wdd.timeout * HZ; > setup_timer(&at91wdt_private.timer, at91_ping, 0); > mod_timer(&at91wdt_private.timer, jiffies + WDT_TIMEOUT); > > pr_info("enabled (heartbeat=%d sec, nowayout=%d)\n", > - heartbeat, nowayout); > + at91wdt_wdd.timeout, nowayout); > > return 0; > } > -- Nicolas Ferre