linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH][DRIVER][RFC] CPU5 watchdog driver for 2.5
@ 2003-02-10 20:17 Heiko Ronsdorf
  2003-02-10 22:33 ` Rusty Lynch
  0 siblings, 1 reply; 6+ messages in thread
From: Heiko Ronsdorf @ 2003-02-10 20:17 UTC (permalink / raw)
  To: linux-kernel

[-- Attachment #1: Type: text/plain, Size: 222 bytes --]

Hello linux-kernel,

this patch is for CPU5 watchdog hardware (kernel 2.5.59)

see: http://www.sma.de/en/inco/dokuftp/

I would appreciate if you could find the time to have a
look at the code and send a feedback.

Heiko


[-- Attachment #2: cpu5wdt.diff --]
[-- Type: text/plain, Size: 9777 bytes --]

diff -urN linux-vanilla/drivers/char/watchdog/Kconfig linux-patched/drivers/char/watchdog/Kconfig
--- linux-vanilla/drivers/char/watchdog/Kconfig	Fri Feb  7 20:45:30 2003
+++ linux-patched/drivers/char/watchdog/Kconfig	Fri Feb  7 20:52:40 2003
@@ -316,4 +316,14 @@
 	tristate "ICP Wafer 5823 Single Board Computer Watchdog"
 	depends on WATCHDOG
 
+config CPU5_WDT
+	tristate "SMA CPU5 Watchdog"
+	depends on WATCHDOG
+	---help---
+	  TBD.
+	  This driver is also available as a module ( = code which can be
+	  inserted in and removed from the running kernel whenever you want).
+	  The module is called cpu5wdt.o.  If you want to compile it as a
+	  module, say M here and read <file:Documentation/modules.txt>.
+
 endmenu
diff -urN linux-vanilla/drivers/char/watchdog/Makefile linux-patched/drivers/char/watchdog/Makefile
--- linux-vanilla/drivers/char/watchdog/Makefile	Fri Feb  7 20:45:30 2003
+++ linux-patched/drivers/char/watchdog/Makefile	Fri Feb  7 20:52:40 2003
@@ -29,3 +29,4 @@
 obj-$(CONFIG_ALIM7101_WDT) += alim7101_wdt.o
 obj-$(CONFIG_SC1200_WDT) += sc1200wdt.o
 obj-$(CONFIG_WAFER_WDT) += wafer5823wdt.o
+obj-$(CONFIG_CPU5_WDT) += cpu5wdt.o
diff -urN linux-vanilla/drivers/char/watchdog/cpu5wdt.c linux-patched/drivers/char/watchdog/cpu5wdt.c
--- linux-vanilla/drivers/char/watchdog/cpu5wdt.c	Thu Jan  1 01:00:00 1970
+++ linux-patched/drivers/char/watchdog/cpu5wdt.c	Mon Feb 10 20:38:14 2003
@@ -0,0 +1,312 @@
+/*
+ * sma cpu5 watchdog driver
+ *
+ * Copyright (C) 2003 Heiko Ronsdorf <hero@ihg.uni-duisburg.de>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+ *
+ */
+
+#include <linux/config.h>
+#include <linux/module.h>
+#include <linux/types.h>
+#include <linux/errno.h>
+#include <linux/miscdevice.h>
+#include <linux/kernel.h>
+#include <linux/fs.h>
+#include <linux/proc_fs.h>
+#include <linux/init.h>
+#include <linux/ioport.h>
+#include <linux/timer.h>
+#include <asm/io.h>
+#include <asm/uaccess.h>
+
+#include <linux/watchdog.h>
+
+/* adjustable parameters */
+
+static int verbose = 0;
+static int port = 0x91;
+static volatile int ticks = 10000;
+
+#define PFX			"cpu5wdt: "
+
+#define CPU5WDT_EXTENT          0x0A
+
+#define CPU5WDT_STATUS_REG      0x00
+#define CPU5WDT_TIME_A_REG      0x02
+#define CPU5WDT_TIME_B_REG      0x03
+#define CPU5WDT_MODE_REG        0x04
+#define CPU5WDT_TRIGGER_REG     0x07
+#define CPU5WDT_ENABLE_REG      0x08
+#define CPU5WDT_RESET_REG       0x09
+
+#define CPU5WDT_INTERVAL	(HZ/10+1)
+
+/* some device data */
+
+static struct {
+	struct semaphore stop;
+	volatile int running;
+	struct timer_list timer;
+	volatile int queue;
+	int default_ticks;
+	int min_ticks;
+	unsigned long inuse;
+} cpu5wdt_device;
+
+/* generic helper functions */
+
+static void cpu5wdt_trigger(unsigned long unused) {
+
+	if ( verbose > 2 )
+		printk(KERN_DEBUG PFX "trigger at %i ticks\n", ticks);
+
+	if( cpu5wdt_device.running )
+		ticks--;
+
+	/* keep watchdog alive */
+	outb(1, port + CPU5WDT_TRIGGER_REG);
+
+	/* requeue?? */
+	if( cpu5wdt_device.queue && ticks ) {
+		cpu5wdt_device.timer.expires = jiffies + CPU5WDT_INTERVAL;
+		add_timer(&cpu5wdt_device.timer);
+	}
+	else {
+		/* ticks doesn't matter anyway */
+		up(&cpu5wdt_device.stop);
+	}
+
+}
+
+static void cpu5wdt_reset(void) {
+
+	if ( ticks < cpu5wdt_device.min_ticks )
+		cpu5wdt_device.min_ticks = ticks;
+
+	ticks = cpu5wdt_device.default_ticks;
+
+	if ( verbose )
+		printk(KERN_DEBUG PFX "reset (%i ticks)\n", (int) ticks);
+
+}
+
+#ifdef CONFIG_PROC_FS
+static int cpu5wdt_read_proc(char *buf, char **start, off_t offset, int len) {
+	len = sprintf(buf,      "activation:       %i\n", cpu5wdt_device.queue);
+	len += sprintf(buf+len, "status:           %i\n", cpu5wdt_device.running);
+	len += sprintf(buf+len, "current ticks: %i\n", ticks);
+	len += sprintf(buf+len, "min ticks:     %i\n", cpu5wdt_device.min_ticks);
+	return len;
+}
+
+static inline void cpu5wdt_register_proc(void) {
+	create_proc_info_entry("driver/cpu5wdt", 0, NULL, cpu5wdt_read_proc);
+}
+
+static inline void cpu5wdt_unregister_proc(void) {
+	remove_proc_entry("driver/cpu5wdt", NULL);
+}
+#else
+static inline void cpu5wdt_register_proc(void) {}
+static inline void cpu5wdt_unregister_proc(void) {}
+#endif
+
+/* filesystem operations */
+
+static int cpu5wdt_open(struct inode *inode, struct file *file) {
+
+	switch(minor(inode->i_rdev)) {
+		case WATCHDOG_MINOR:
+			if ( test_and_set_bit(0, &cpu5wdt_device.inuse) )
+				return -EBUSY;
+			break;
+		default:
+			return -ENODEV;
+	}
+	return 0;
+
+}
+
+static int cpu5wdt_release(struct inode *inode, struct file *file) {
+
+	if(minor(inode->i_rdev)==WATCHDOG_MINOR) {
+		clear_bit(0, &cpu5wdt_device.inuse);
+	}
+	return 0;
+}
+
+static int cpu5wdt_ioctl(struct inode *inode, struct file *file, unsigned int cmd, unsigned long arg) {
+	unsigned int value;
+  
+	switch(cmd) {
+		case WDIOC_KEEPALIVE:
+			cpu5wdt_reset();
+			break;
+		case WDIOC_START:
+			if ( !cpu5wdt_device.queue ) {
+				cpu5wdt_device.queue = 1;
+				outb(0, port + CPU5WDT_TIME_A_REG);  
+				outb(0, port + CPU5WDT_TIME_B_REG);  
+				outb(1, port + CPU5WDT_MODE_REG);
+				outb(0, port + CPU5WDT_RESET_REG);
+				outb(0, port + CPU5WDT_ENABLE_REG);
+				cpu5wdt_device.timer.expires = jiffies + CPU5WDT_INTERVAL;
+				add_timer(&cpu5wdt_device.timer);
+			}
+			/* if process dies, counter is not decremented */
+			cpu5wdt_device.running++;
+			break;
+		case WDIOC_GETSTATUS:    
+			value = inb(port + CPU5WDT_STATUS_REG); 
+			value = (value >> 2) & 1;
+			if ( copy_to_user((int *)arg, (int *)&value, sizeof(int)) )
+				return -EFAULT;
+			break;
+		case WDIOC_STOP:
+			if ( cpu5wdt_device.running )
+				cpu5wdt_device.running = 0;
+
+			ticks = cpu5wdt_device.default_ticks;
+
+			if ( verbose )
+				printk(KERN_CRIT PFX "stop not possible\n");
+			return -EIO;
+		default:
+    			return -EINVAL;
+	}
+	return 0;
+}
+
+static ssize_t cpu5wdt_write(struct file *file, const char *buf, size_t count, loff_t *ppos) {
+
+	if ( !count )
+		return -EIO;
+	
+	cpu5wdt_reset();
+	return count;
+
+}
+
+static struct file_operations cpu5wdt_fops = {
+	.owner		= THIS_MODULE,
+	.ioctl		= cpu5wdt_ioctl,
+	.open		= cpu5wdt_open,
+	.write		= cpu5wdt_write,
+	.release	= cpu5wdt_release,
+};
+
+static struct miscdevice cpu5wdt_misc = {
+	.minor	= WATCHDOG_MINOR,
+	.name	= "watchdog",
+	.fops	= &cpu5wdt_fops
+};
+
+/* init/exit function */
+
+static int __devinit cpu5wdt_init(void) {
+	unsigned int val;
+	int err;
+
+	if ( verbose )
+		printk(KERN_DEBUG PFX "port=0x%x, verbose=%i\n", port, verbose);
+
+	if ( (err = misc_register(&cpu5wdt_misc)) < 0 ) {
+		printk(KERN_ERR PFX "misc_register failed\n");
+		goto no_misc;
+	}
+
+	if ( !request_region(port, CPU5WDT_EXTENT, PFX) ) {
+		printk(KERN_ERR PFX "request_region failed\n");
+		err = -EBUSY;
+		goto no_port;
+	}
+
+	/* watchdog reboot? */
+	val = inb(port + CPU5WDT_STATUS_REG); 
+	val = (val >> 2) & 1;
+	if ( !val )
+		printk(KERN_INFO PFX "sorry, was my fault\n");
+
+	init_MUTEX_LOCKED(&cpu5wdt_device.stop);
+	cpu5wdt_device.queue = 0;
+	cpu5wdt_device.min_ticks = ticks;
+
+	clear_bit(0, &cpu5wdt_device.inuse);
+
+	cpu5wdt_register_proc();
+
+	init_timer(&cpu5wdt_device.timer);
+	cpu5wdt_device.timer.function = cpu5wdt_trigger;
+	cpu5wdt_device.timer.data = 0;
+
+	cpu5wdt_device.default_ticks = ticks;
+
+	printk(KERN_INFO PFX "init success\n");
+
+	return 0;
+
+no_port:
+	misc_deregister(&cpu5wdt_misc);
+no_misc:
+	return err;
+}
+
+static int __devinit cpu5wdt_init_module(void) {
+
+	return cpu5wdt_init();
+}
+
+static void __devexit cpu5wdt_exit(void) {
+
+	if ( cpu5wdt_device.queue ) {
+		cpu5wdt_device.queue = 0;
+		down(&cpu5wdt_device.stop);
+	}
+
+	cpu5wdt_unregister_proc();
+
+	misc_deregister(&cpu5wdt_misc);
+
+	release_region(port, CPU5WDT_EXTENT);
+
+}
+
+static void __devexit cpu5wdt_exit_module(void) {
+
+	cpu5wdt_exit();
+}
+
+/* module entry points */
+
+module_init(cpu5wdt_init_module);
+module_exit(cpu5wdt_exit_module);
+
+MODULE_AUTHOR("Heiko Ronsdorf <hero@ihg.uni-duisburg.de>");
+MODULE_DESCRIPTION("sma cpu5 watchdog driver");
+MODULE_SUPPORTED_DEVICE("sma cpu5 watchdog");
+MODULE_LICENSE("GPL");
+
+MODULE_PARM(port, "i");
+MODULE_PARM_DESC(port, "base address of watchdog card, default is 0x91");
+
+MODULE_PARM(verbose, "i");
+MODULE_PARM_DESC(verbose, "be verbose, default is 0 (no)");
+
+MODULE_PARM(ticks, "i");
+MODULE_PARM_DESC(ticks, "count down ticks, default is 10000");
+
+EXPORT_NO_SYMBOLS;
diff -urN linux-vanilla/include/linux/watchdog.h linux-patched/include/linux/watchdog.h
--- linux-vanilla/include/linux/watchdog.h	Fri Jan 10 22:08:18 2003
+++ linux-patched/include/linux/watchdog.h	Fri Feb  7 20:52:40 2003
@@ -27,6 +27,8 @@
 #define	WDIOC_KEEPALIVE		_IOR(WATCHDOG_IOCTL_BASE, 5, int)
 #define	WDIOC_SETTIMEOUT        _IOWR(WATCHDOG_IOCTL_BASE, 6, int)
 #define	WDIOC_GETTIMEOUT        _IOR(WATCHDOG_IOCTL_BASE, 7, int)
+#define	WDIOC_START             _IO(WATCHDOG_IOCTL_BASE, 8)
+#define	WDIOC_STOP              _IO(WATCHDOG_IOCTL_BASE, 9)
 
 #define	WDIOF_UNKNOWN		-1	/* Unknown flag error */
 #define	WDIOS_UNKNOWN		-1	/* Unknown status error */

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH][DRIVER][RFC] CPU5 watchdog driver for 2.5
  2003-02-10 20:17 [PATCH][DRIVER][RFC] CPU5 watchdog driver for 2.5 Heiko Ronsdorf
@ 2003-02-10 22:33 ` Rusty Lynch
  2003-02-11 12:26   ` Heiko Ronsdorf
  0 siblings, 1 reply; 6+ messages in thread
From: Rusty Lynch @ 2003-02-10 22:33 UTC (permalink / raw)
  To: Heiko Ronsdorf; +Cc: linux-kernel

On Mon, 2003-02-10 at 12:17, Heiko Ronsdorf wrote:
> Hello linux-kernel,
> 
> this patch is for CPU5 watchdog hardware (kernel 2.5.59)
> 
> see: http://www.sma.de/en/inco/dokuftp/
> 
> I would appreciate if you could find the time to have a
> look at the code and send a feedback.
> 
> Heiko
> 
> ----
> 

Here are some things I noticed from a casual glance:

* Documentation/CodingStyle calls for the opening braces on functions to
be on the next line, like:

int function(int x)
{
        body of function
}

* You could make you driver fit into existing user space deamons by
conforming to Documentation/watchdog-api.txt.  Some things are a little
odd (different then existing wdt drivers) like the way you start and
stop the watchdog.

* I'm pretty sure that in general adding new code to /proc (that has
nothing to do with processes) is frowned on.

    --rustyl

> diff -urN linux-vanilla/drivers/char/watchdog/Kconfig linux-patched/drivers/char/watchdog/Kconfig
> --- linux-vanilla/drivers/char/watchdog/Kconfig	Fri Feb  7 20:45:30 2003
> +++ linux-patched/drivers/char/watchdog/Kconfig	Fri Feb  7 20:52:40 2003
> @@ -316,4 +316,14 @@
>  	tristate "ICP Wafer 5823 Single Board Computer Watchdog"
>  	depends on WATCHDOG
>  
> +config CPU5_WDT
> +	tristate "SMA CPU5 Watchdog"
> +	depends on WATCHDOG
> +	---help---
> +	  TBD.
> +	  This driver is also available as a module ( = code which can be
> +	  inserted in and removed from the running kernel whenever you want).
> +	  The module is called cpu5wdt.o.  If you want to compile it as a
> +	  module, say M here and read <file:Documentation/modules.txt>.
> +
>  endmenu
> diff -urN linux-vanilla/drivers/char/watchdog/Makefile linux-patched/drivers/char/watchdog/Makefile
> --- linux-vanilla/drivers/char/watchdog/Makefile	Fri Feb  7 20:45:30 2003
> +++ linux-patched/drivers/char/watchdog/Makefile	Fri Feb  7 20:52:40 2003
> @@ -29,3 +29,4 @@
>  obj-$(CONFIG_ALIM7101_WDT) += alim7101_wdt.o
>  obj-$(CONFIG_SC1200_WDT) += sc1200wdt.o
>  obj-$(CONFIG_WAFER_WDT) += wafer5823wdt.o
> +obj-$(CONFIG_CPU5_WDT) += cpu5wdt.o
> diff -urN linux-vanilla/drivers/char/watchdog/cpu5wdt.c linux-patched/drivers/char/watchdog/cpu5wdt.c
> --- linux-vanilla/drivers/char/watchdog/cpu5wdt.c	Thu Jan  1 01:00:00 1970
> +++ linux-patched/drivers/char/watchdog/cpu5wdt.c	Mon Feb 10 20:38:14 2003
> @@ -0,0 +1,312 @@
> +/*
> + * sma cpu5 watchdog driver
> + *
> + * Copyright (C) 2003 Heiko Ronsdorf <hero@ihg.uni-duisburg.de>
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License as published by
> + * the Free Software Foundation; either version 2 of the License, or
> + * (at your option) any later version.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> + * GNU General Public License for more details.
> + *
> + * You should have received a copy of the GNU General Public License
> + * along with this program; if not, write to the Free Software
> + * Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
> + *
> + */
> +
> +#include <linux/config.h>
> +#include <linux/module.h>
> +#include <linux/types.h>
> +#include <linux/errno.h>
> +#include <linux/miscdevice.h>
> +#include <linux/kernel.h>
> +#include <linux/fs.h>
> +#include <linux/proc_fs.h>
> +#include <linux/init.h>
> +#include <linux/ioport.h>
> +#include <linux/timer.h>
> +#include <asm/io.h>
> +#include <asm/uaccess.h>
> +
> +#include <linux/watchdog.h>
> +
> +/* adjustable parameters */
> +
> +static int verbose = 0;
> +static int port = 0x91;
> +static volatile int ticks = 10000;
> +
> +#define PFX			"cpu5wdt: "
> +
> +#define CPU5WDT_EXTENT          0x0A
> +
> +#define CPU5WDT_STATUS_REG      0x00
> +#define CPU5WDT_TIME_A_REG      0x02
> +#define CPU5WDT_TIME_B_REG      0x03
> +#define CPU5WDT_MODE_REG        0x04
> +#define CPU5WDT_TRIGGER_REG     0x07
> +#define CPU5WDT_ENABLE_REG      0x08
> +#define CPU5WDT_RESET_REG       0x09
> +
> +#define CPU5WDT_INTERVAL	(HZ/10+1)
> +
> +/* some device data */
> +
> +static struct {
> +	struct semaphore stop;
> +	volatile int running;
> +	struct timer_list timer;
> +	volatile int queue;
> +	int default_ticks;
> +	int min_ticks;
> +	unsigned long inuse;
> +} cpu5wdt_device;
> +
> +/* generic helper functions */
> +
> +static void cpu5wdt_trigger(unsigned long unused) {
> +
> +	if ( verbose > 2 )
> +		printk(KERN_DEBUG PFX "trigger at %i ticks\n", ticks);
> +
> +	if( cpu5wdt_device.running )
> +		ticks--;
> +
> +	/* keep watchdog alive */
> +	outb(1, port + CPU5WDT_TRIGGER_REG);
> +
> +	/* requeue?? */
> +	if( cpu5wdt_device.queue && ticks ) {
> +		cpu5wdt_device.timer.expires = jiffies + CPU5WDT_INTERVAL;
> +		add_timer(&cpu5wdt_device.timer);
> +	}
> +	else {
> +		/* ticks doesn't matter anyway */
> +		up(&cpu5wdt_device.stop);
> +	}
> +
> +}
> +
> +static void cpu5wdt_reset(void) {
> +
> +	if ( ticks < cpu5wdt_device.min_ticks )
> +		cpu5wdt_device.min_ticks = ticks;
> +
> +	ticks = cpu5wdt_device.default_ticks;
> +
> +	if ( verbose )
> +		printk(KERN_DEBUG PFX "reset (%i ticks)\n", (int) ticks);
> +
> +}
> +
> +#ifdef CONFIG_PROC_FS
> +static int cpu5wdt_read_proc(char *buf, char **start, off_t offset, int len) {
> +	len = sprintf(buf,      "activation:       %i\n", cpu5wdt_device.queue);
> +	len += sprintf(buf+len, "status:           %i\n", cpu5wdt_device.running);
> +	len += sprintf(buf+len, "current ticks: %i\n", ticks);
> +	len += sprintf(buf+len, "min ticks:     %i\n", cpu5wdt_device.min_ticks);
> +	return len;
> +}
> +
> +static inline void cpu5wdt_register_proc(void) {
> +	create_proc_info_entry("driver/cpu5wdt", 0, NULL, cpu5wdt_read_proc);
> +}
> +
> +static inline void cpu5wdt_unregister_proc(void) {
> +	remove_proc_entry("driver/cpu5wdt", NULL);
> +}
> +#else
> +static inline void cpu5wdt_register_proc(void) {}
> +static inline void cpu5wdt_unregister_proc(void) {}
> +#endif
> +
> +/* filesystem operations */
> +
> +static int cpu5wdt_open(struct inode *inode, struct file *file) {
> +
> +	switch(minor(inode->i_rdev)) {
> +		case WATCHDOG_MINOR:
> +			if ( test_and_set_bit(0, &cpu5wdt_device.inuse) )
> +				return -EBUSY;
> +			break;
> +		default:
> +			return -ENODEV;
> +	}
> +	return 0;
> +
> +}
> +
> +static int cpu5wdt_release(struct inode *inode, struct file *file) {
> +
> +	if(minor(inode->i_rdev)==WATCHDOG_MINOR) {
> +		clear_bit(0, &cpu5wdt_device.inuse);
> +	}
> +	return 0;
> +}
> +
> +static int cpu5wdt_ioctl(struct inode *inode, struct file *file, unsigned int cmd, unsigned long arg) {
> +	unsigned int value;
> +  
> +	switch(cmd) {
> +		case WDIOC_KEEPALIVE:
> +			cpu5wdt_reset();
> +			break;
> +		case WDIOC_START:
> +			if ( !cpu5wdt_device.queue ) {
> +				cpu5wdt_device.queue = 1;
> +				outb(0, port + CPU5WDT_TIME_A_REG);  
> +				outb(0, port + CPU5WDT_TIME_B_REG);  
> +				outb(1, port + CPU5WDT_MODE_REG);
> +				outb(0, port + CPU5WDT_RESET_REG);
> +				outb(0, port + CPU5WDT_ENABLE_REG);
> +				cpu5wdt_device.timer.expires = jiffies + CPU5WDT_INTERVAL;
> +				add_timer(&cpu5wdt_device.timer);
> +			}
> +			/* if process dies, counter is not decremented */
> +			cpu5wdt_device.running++;
> +			break;
> +		case WDIOC_GETSTATUS:    
> +			value = inb(port + CPU5WDT_STATUS_REG); 
> +			value = (value >> 2) & 1;
> +			if ( copy_to_user((int *)arg, (int *)&value, sizeof(int)) )
> +				return -EFAULT;
> +			break;
> +		case WDIOC_STOP:
> +			if ( cpu5wdt_device.running )
> +				cpu5wdt_device.running = 0;
> +
> +			ticks = cpu5wdt_device.default_ticks;
> +
> +			if ( verbose )
> +				printk(KERN_CRIT PFX "stop not possible\n");
> +			return -EIO;
> +		default:
> +    			return -EINVAL;
> +	}
> +	return 0;
> +}
> +
> +static ssize_t cpu5wdt_write(struct file *file, const char *buf, size_t count, loff_t *ppos) {
> +
> +	if ( !count )
> +		return -EIO;
> +	
> +	cpu5wdt_reset();
> +	return count;
> +
> +}
> +
> +static struct file_operations cpu5wdt_fops = {
> +	.owner		= THIS_MODULE,
> +	.ioctl		= cpu5wdt_ioctl,
> +	.open		= cpu5wdt_open,
> +	.write		= cpu5wdt_write,
> +	.release	= cpu5wdt_release,
> +};
> +
> +static struct miscdevice cpu5wdt_misc = {
> +	.minor	= WATCHDOG_MINOR,
> +	.name	= "watchdog",
> +	.fops	= &cpu5wdt_fops
> +};
> +
> +/* init/exit function */
> +
> +static int __devinit cpu5wdt_init(void) {
> +	unsigned int val;
> +	int err;
> +
> +	if ( verbose )
> +		printk(KERN_DEBUG PFX "port=0x%x, verbose=%i\n", port, verbose);
> +
> +	if ( (err = misc_register(&cpu5wdt_misc)) < 0 ) {
> +		printk(KERN_ERR PFX "misc_register failed\n");
> +		goto no_misc;
> +	}
> +
> +	if ( !request_region(port, CPU5WDT_EXTENT, PFX) ) {
> +		printk(KERN_ERR PFX "request_region failed\n");
> +		err = -EBUSY;
> +		goto no_port;
> +	}
> +
> +	/* watchdog reboot? */
> +	val = inb(port + CPU5WDT_STATUS_REG); 
> +	val = (val >> 2) & 1;
> +	if ( !val )
> +		printk(KERN_INFO PFX "sorry, was my fault\n");
> +
> +	init_MUTEX_LOCKED(&cpu5wdt_device.stop);
> +	cpu5wdt_device.queue = 0;
> +	cpu5wdt_device.min_ticks = ticks;
> +
> +	clear_bit(0, &cpu5wdt_device.inuse);
> +
> +	cpu5wdt_register_proc();
> +
> +	init_timer(&cpu5wdt_device.timer);
> +	cpu5wdt_device.timer.function = cpu5wdt_trigger;
> +	cpu5wdt_device.timer.data = 0;
> +
> +	cpu5wdt_device.default_ticks = ticks;
> +
> +	printk(KERN_INFO PFX "init success\n");
> +
> +	return 0;
> +
> +no_port:
> +	misc_deregister(&cpu5wdt_misc);
> +no_misc:
> +	return err;
> +}
> +
> +static int __devinit cpu5wdt_init_module(void) {
> +
> +	return cpu5wdt_init();
> +}
> +
> +static void __devexit cpu5wdt_exit(void) {
> +
> +	if ( cpu5wdt_device.queue ) {
> +		cpu5wdt_device.queue = 0;
> +		down(&cpu5wdt_device.stop);
> +	}
> +
> +	cpu5wdt_unregister_proc();
> +
> +	misc_deregister(&cpu5wdt_misc);
> +
> +	release_region(port, CPU5WDT_EXTENT);
> +
> +}
> +
> +static void __devexit cpu5wdt_exit_module(void) {
> +
> +	cpu5wdt_exit();
> +}
> +
> +/* module entry points */
> +
> +module_init(cpu5wdt_init_module);
> +module_exit(cpu5wdt_exit_module);
> +
> +MODULE_AUTHOR("Heiko Ronsdorf <hero@ihg.uni-duisburg.de>");
> +MODULE_DESCRIPTION("sma cpu5 watchdog driver");
> +MODULE_SUPPORTED_DEVICE("sma cpu5 watchdog");
> +MODULE_LICENSE("GPL");
> +
> +MODULE_PARM(port, "i");
> +MODULE_PARM_DESC(port, "base address of watchdog card, default is 0x91");
> +
> +MODULE_PARM(verbose, "i");
> +MODULE_PARM_DESC(verbose, "be verbose, default is 0 (no)");
> +
> +MODULE_PARM(ticks, "i");
> +MODULE_PARM_DESC(ticks, "count down ticks, default is 10000");
> +
> +EXPORT_NO_SYMBOLS;
> diff -urN linux-vanilla/include/linux/watchdog.h linux-patched/include/linux/watchdog.h
> --- linux-vanilla/include/linux/watchdog.h	Fri Jan 10 22:08:18 2003
> +++ linux-patched/include/linux/watchdog.h	Fri Feb  7 20:52:40 2003
> @@ -27,6 +27,8 @@
>  #define	WDIOC_KEEPALIVE		_IOR(WATCHDOG_IOCTL_BASE, 5, int)
>  #define	WDIOC_SETTIMEOUT        _IOWR(WATCHDOG_IOCTL_BASE, 6, int)
>  #define	WDIOC_GETTIMEOUT        _IOR(WATCHDOG_IOCTL_BASE, 7, int)
> +#define	WDIOC_START             _IO(WATCHDOG_IOCTL_BASE, 8)
> +#define	WDIOC_STOP              _IO(WATCHDOG_IOCTL_BASE, 9)
>  
>  #define	WDIOF_UNKNOWN		-1	/* Unknown flag error */
>  #define	WDIOS_UNKNOWN		-1	/* Unknown status error */



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH][DRIVER][RFC] CPU5 watchdog driver for 2.5
  2003-02-11 12:26   ` Heiko Ronsdorf
@ 2003-02-11  4:38     ` Rusty Lynch
  2003-02-12 16:25       ` Heiko Ronsdorf
  2003-02-11 13:11     ` Dave Jones
  1 sibling, 1 reply; 6+ messages in thread
From: Rusty Lynch @ 2003-02-11  4:38 UTC (permalink / raw)
  To: Heiko Ronsdorf; +Cc: linux-kernel, mochel

On Tue, 2003-02-11 at 04:26, Heiko Ronsdorf wrote:
> Rusty Lynch schrieb am Mon, Feb 10, 2003 at 02:33:27PM -0800:
<snip>
> > * I'm pretty sure that in general adding new code to /proc (that has
> > nothing to do with processes) is frowned on.
> 
> I don't know how to track the watchdog. Suggestions are welcome.
> 
> Heiko
> 

I CC'ed Patrick Mochel to get his opinion on this, but if I am
interpreting the new driver model documentation correctly, then it would
be appropriate to make an IO controlled watchdog timer a
"platform_device".  That will let user space see your device in sysfs as
$YOUR_SYSFS_ROOT/devices/legacy/watchdog0/, and then you can use
device_create_file() to add a new file inside the the watchdog0
directory.

Here is your cpu5wdt.c file with the procfs stuff replaced with code
that adds an embedded platform_device to your basic data structure,
registers the driver as a platform device, and then creates a file
called $YOUR_SYSFS_ROOT/devices/legacy/watchdog0/statistics that will
read the same as your original procfs file.  (Or at least I think it
will.  It at least compiles :->)

    --rustyl

/*
 * sma cpu5 watchdog driver
 *
 * Copyright (C) 2003 Heiko Ronsdorf <hero@ihg.uni-duisburg.de>
 *
 * This program is free software; you can redistribute it and/or modify
 * it under the terms of the GNU General Public License as published by
 * the Free Software Foundation; either version 2 of the License, or
 * (at your option) any later version.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 * GNU General Public License for more details.
 *
 * You should have received a copy of the GNU General Public License
 * along with this program; if not, write to the Free Software
 * Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
 *
 */

#include <linux/config.h>
#include <linux/module.h>
#include <linux/types.h>
#include <linux/errno.h>
#include <linux/miscdevice.h>
#include <linux/kernel.h>
#include <linux/fs.h>
#include <linux/init.h>
#include <linux/ioport.h>
#include <linux/timer.h>
#include <asm/io.h>
#include <asm/uaccess.h>

#include <linux/watchdog.h>
#include <linux/kobject.h>
#include <linux/device.h>

/* adjustable parameters */

static int verbose = 0;
static int port = 0x91;
static volatile int ticks = 10000;

#define PFX			"cpu5wdt: "

#define CPU5WDT_EXTENT          0x0A

#define CPU5WDT_STATUS_REG      0x00
#define CPU5WDT_TIME_A_REG      0x02
#define CPU5WDT_TIME_B_REG      0x03
#define CPU5WDT_MODE_REG        0x04
#define CPU5WDT_TRIGGER_REG     0x07
#define CPU5WDT_ENABLE_REG      0x08
#define CPU5WDT_RESET_REG       0x09

#define CPU5WDT_INTERVAL	(HZ/10+1)

#define	to_platform_device(n) container_of(n, struct platform_device, dev)
#define	to_wdt_device(n) container_of(n, struct wdt_device, dev)

/* some device data */

struct wdt_device {
	struct semaphore stop;
	volatile int running;
	struct timer_list timer;
	volatile int queue;
	int default_ticks;
	int min_ticks;
	unsigned long inuse;
	struct platform_device dev;
};

static struct wdt_device cpu5wdt_device = {
	.dev = {
		.name = "watchdog",
		.id		= 0,
		.dev		= {
			.name	= "CPU5 Watchdog Device",
		},
	},
};

ssize_t statistics_show(struct device * dev, char * buf)
{
	size_t len;
	struct wdt_device *d = to_wdt_device(to_platform_device(dev));
	len = sprintf(buf,      "activation:       %i\n", d->queue);
	len += sprintf(buf+len, "status:           %i\n", d->running);
	len += sprintf(buf+len, "current ticks: %i\n", ticks);
	len += sprintf(buf+len, "min ticks:     %i\n", d->min_ticks);
	return len;

}
DEVICE_ATTR(statistics,S_IRUGO,statistics_show,NULL);

/* generic helper functions */

static void cpu5wdt_trigger(unsigned long unused)
{
	if ( verbose > 2 )
		printk(KERN_DEBUG PFX "trigger at %i ticks\n", ticks);

	if( cpu5wdt_device.running )
		ticks--;

	/* keep watchdog alive */
	outb(1, port + CPU5WDT_TRIGGER_REG);

	/* requeue?? */
	if( cpu5wdt_device.queue && ticks ) {
		cpu5wdt_device.timer.expires = jiffies + CPU5WDT_INTERVAL;
		add_timer(&cpu5wdt_device.timer);
	}
	else {
		/* ticks doesn't matter anyway */
		up(&cpu5wdt_device.stop);
	}

}

static void cpu5wdt_reset(void)
{
	if ( ticks < cpu5wdt_device.min_ticks )
		cpu5wdt_device.min_ticks = ticks;

	ticks = cpu5wdt_device.default_ticks;

	if ( verbose )
		printk(KERN_DEBUG PFX "reset (%i ticks)\n", (int) ticks);

}

static void cpu5wdt_start(void)
{
	if ( !cpu5wdt_device.queue ) {
		cpu5wdt_device.queue = 1;
		outb(0, port + CPU5WDT_TIME_A_REG);  
		outb(0, port + CPU5WDT_TIME_B_REG);  
		outb(1, port + CPU5WDT_MODE_REG);
		outb(0, port + CPU5WDT_RESET_REG);
		outb(0, port + CPU5WDT_ENABLE_REG);
		cpu5wdt_device.timer.expires = jiffies + CPU5WDT_INTERVAL;
		add_timer(&cpu5wdt_device.timer);
	}
	/* if process dies, counter is not decremented */
	cpu5wdt_device.running++;
}

static int cpu5wdt_stop(void)
{
	if ( cpu5wdt_device.running )
		cpu5wdt_device.running = 0;

	ticks = cpu5wdt_device.default_ticks;

	if ( verbose )
		printk(KERN_CRIT PFX "stop not possible\n");

	return -EIO;
}

/* filesystem operations */

static int cpu5wdt_open(struct inode *inode, struct file *file)
{
	switch(minor(inode->i_rdev)) {
		case WATCHDOG_MINOR:
			if ( test_and_set_bit(0, &cpu5wdt_device.inuse) )
				return -EBUSY;
			break;
		default:
			return -ENODEV;
	}
	return 0;

}

static int cpu5wdt_release(struct inode *inode, struct file *file)
{
	if(minor(inode->i_rdev)==WATCHDOG_MINOR) {
		clear_bit(0, &cpu5wdt_device.inuse);
	}
	return 0;
}

static int cpu5wdt_ioctl(struct inode *inode, struct file *file, unsigned int cmd, unsigned long arg)
{
	unsigned int value;
	static struct watchdog_info ident =
	{
		.options = WDIOF_CARDRESET,
		.identity = "CPU5 WDT"
	};
  
	switch(cmd) {
		case WDIOC_KEEPALIVE:
			cpu5wdt_reset();
			break;
		case WDIOC_GETSTATUS:    
			value = inb(port + CPU5WDT_STATUS_REG); 
			value = (value >> 2) & 1;
			if ( copy_to_user((int *)arg, (int *)&value, sizeof(int)) )
				return -EFAULT;
			break;
		case WDIOC_GETSUPPORT:
			if ( copy_to_user((struct watchdog_info *)arg, &ident, sizeof(ident)) )
				return -EFAULT;
			break;
		case WDIOC_SETOPTIONS:
			if ( copy_from_user(&value, (int *)arg, sizeof(int)) )
				return -EFAULT;
			switch(value) {
				case WDIOS_ENABLECARD:
					cpu5wdt_start();
					break;
				case WDIOS_DISABLECARD:
					return cpu5wdt_stop();
				default:
					return -EINVAL;
			}
			break;
		default:
    			return -EINVAL;
	}
	return 0;
}

static ssize_t cpu5wdt_write(struct file *file, const char *buf, size_t count, loff_t *ppos)
{
	if ( !count )
		return -EIO;
	
	cpu5wdt_reset();
	return count;

}

static struct file_operations cpu5wdt_fops = {
	.owner		= THIS_MODULE,
	.ioctl		= cpu5wdt_ioctl,
	.open		= cpu5wdt_open,
	.write		= cpu5wdt_write,
	.release	= cpu5wdt_release,
};

static struct miscdevice cpu5wdt_misc = {
	.minor	= WATCHDOG_MINOR,
	.name	= "watchdog",
	.fops	= &cpu5wdt_fops
};

/* init/exit function */

static int __devinit cpu5wdt_init(void)
{
	unsigned int val;
	int err;

	if ( verbose )
		printk(KERN_DEBUG PFX "port=0x%x, verbose=%i\n", port, verbose);

	if ( (err = misc_register(&cpu5wdt_misc)) < 0 ) {
		printk(KERN_ERR PFX "misc_register failed\n");
		goto no_misc;
	}

	if ( !request_region(port, CPU5WDT_EXTENT, PFX) ) {
		printk(KERN_ERR PFX "request_region failed\n");
		err = -EBUSY;
		goto no_port;
	}

	/* watchdog reboot? */
	val = inb(port + CPU5WDT_STATUS_REG); 
	val = (val >> 2) & 1;
	if ( !val )
		printk(KERN_INFO PFX "sorry, was my fault\n");

	init_MUTEX_LOCKED(&cpu5wdt_device.stop);
	cpu5wdt_device.queue = 0;
	cpu5wdt_device.min_ticks = ticks;

	clear_bit(0, &cpu5wdt_device.inuse);

	platform_device_register(&cpu5wdt_device.dev);
	device_create_file(&cpu5wdt_device.dev.dev, &dev_attr_statistics);

	init_timer(&cpu5wdt_device.timer);
	cpu5wdt_device.timer.function = cpu5wdt_trigger;
	cpu5wdt_device.timer.data = 0;

	cpu5wdt_device.default_ticks = ticks;

	printk(KERN_INFO PFX "init success\n");

	return 0;

no_port:
	misc_deregister(&cpu5wdt_misc);
no_misc:
	return err;
}

static int __devinit cpu5wdt_init_module(void)
{
	return cpu5wdt_init();
}

static void __devexit cpu5wdt_exit(void)
{
	if ( cpu5wdt_device.queue ) {
		cpu5wdt_device.queue = 0;
		down(&cpu5wdt_device.stop);
	}

	platform_device_unregister(&cpu5wdt_device.dev);
	device_remove_file(&cpu5wdt_device.dev.dev, &dev_attr_statistics);
	misc_deregister(&cpu5wdt_misc);

	release_region(port, CPU5WDT_EXTENT);

}

static void __devexit cpu5wdt_exit_module(void)
{
	cpu5wdt_exit();
}

/* module entry points */

module_init(cpu5wdt_init_module);
module_exit(cpu5wdt_exit_module);

MODULE_AUTHOR("Heiko Ronsdorf <hero@ihg.uni-duisburg.de>");
MODULE_DESCRIPTION("sma cpu5 watchdog driver");
MODULE_SUPPORTED_DEVICE("sma cpu5 watchdog");
MODULE_LICENSE("GPL");

MODULE_PARM(port, "i");
MODULE_PARM_DESC(port, "base address of watchdog card, default is 0x91");

MODULE_PARM(verbose, "i");
MODULE_PARM_DESC(verbose, "be verbose, default is 0 (no)");

MODULE_PARM(ticks, "i");
MODULE_PARM_DESC(ticks, "count down ticks, default is 10000");

EXPORT_NO_SYMBOLS;



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH][DRIVER][RFC] CPU5 watchdog driver for 2.5
  2003-02-10 22:33 ` Rusty Lynch
@ 2003-02-11 12:26   ` Heiko Ronsdorf
  2003-02-11  4:38     ` Rusty Lynch
  2003-02-11 13:11     ` Dave Jones
  0 siblings, 2 replies; 6+ messages in thread
From: Heiko Ronsdorf @ 2003-02-11 12:26 UTC (permalink / raw)
  To: linux-kernel

[-- Attachment #1: Type: text/plain, Size: 779 bytes --]

Rusty Lynch schrieb am Mon, Feb 10, 2003 at 02:33:27PM -0800:
> On Mon, 2003-02-10 at 12:17, Heiko Ronsdorf wrote:
> > Hello linux-kernel,
> > 
> > this patch is for CPU5 watchdog hardware (kernel 2.5.59)

updated for kernel 2.5.60

> * Documentation/CodingStyle calls for the opening braces on functions to
> be on the next line, like:

done

> * You could make you driver fit into existing user space deamons by
> conforming to Documentation/watchdog-api.txt.  Some things are a little
> odd (different then existing wdt drivers) like the way you start and
> stop the watchdog.

done

> * I'm pretty sure that in general adding new code to /proc (that has
> nothing to do with processes) is frowned on.

I don't know how to track the watchdog. Suggestions are welcome.

Heiko


[-- Attachment #2: cpu5wdt-2.5.60-3.diff --]
[-- Type: text/plain, Size: 9634 bytes --]

diff -urN linux-vanilla/drivers/char/watchdog/Kconfig linux-patched/drivers/char/watchdog/Kconfig
--- linux-vanilla/drivers/char/watchdog/Kconfig	Tue Feb 11 11:03:51 2003
+++ linux-patched/drivers/char/watchdog/Kconfig	Mon Feb 10 22:02:12 2003
@@ -354,4 +354,14 @@
 	  Documentation/modules.txt. The module will be called
 	  wafer5823wdt.o
 
+config CPU5_WDT
+	tristate "SMA CPU5 Watchdog"
+	depends on WATCHDOG
+	---help---
+	  TBD.
+	  This driver is also available as a module ( = code which can be
+	  inserted in and removed from the running kernel whenever you want).
+	  The module is called cpu5wdt.o.  If you want to compile it as a
+	  module, say M here and read <file:Documentation/modules.txt>.
+
 endmenu
diff -urN linux-vanilla/drivers/char/watchdog/Makefile linux-patched/drivers/char/watchdog/Makefile
--- linux-vanilla/drivers/char/watchdog/Makefile	Tue Feb 11 11:03:51 2003
+++ linux-patched/drivers/char/watchdog/Makefile	Mon Feb 10 22:02:12 2003
@@ -29,3 +29,4 @@
 obj-$(CONFIG_ALIM7101_WDT) += alim7101_wdt.o
 obj-$(CONFIG_SC1200_WDT) += sc1200wdt.o
 obj-$(CONFIG_WAFER_WDT) += wafer5823wdt.o
+obj-$(CONFIG_CPU5_WDT) += cpu5wdt.o
diff -urN linux-vanilla/drivers/char/watchdog/cpu5wdt.c linux-patched/drivers/char/watchdog/cpu5wdt.c
--- linux-vanilla/drivers/char/watchdog/cpu5wdt.c	Thu Jan  1 01:00:00 1970
+++ linux-patched/drivers/char/watchdog/cpu5wdt.c	Tue Feb 11 10:36:50 2003
@@ -0,0 +1,344 @@
+/*
+ * sma cpu5 watchdog driver
+ *
+ * Copyright (C) 2003 Heiko Ronsdorf <hero@ihg.uni-duisburg.de>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+ *
+ */
+
+#include <linux/config.h>
+#include <linux/module.h>
+#include <linux/types.h>
+#include <linux/errno.h>
+#include <linux/miscdevice.h>
+#include <linux/kernel.h>
+#include <linux/fs.h>
+#include <linux/proc_fs.h>
+#include <linux/init.h>
+#include <linux/ioport.h>
+#include <linux/timer.h>
+#include <asm/io.h>
+#include <asm/uaccess.h>
+
+#include <linux/watchdog.h>
+
+/* adjustable parameters */
+
+static int verbose = 0;
+static int port = 0x91;
+static volatile int ticks = 10000;
+
+#define PFX			"cpu5wdt: "
+
+#define CPU5WDT_EXTENT          0x0A
+
+#define CPU5WDT_STATUS_REG      0x00
+#define CPU5WDT_TIME_A_REG      0x02
+#define CPU5WDT_TIME_B_REG      0x03
+#define CPU5WDT_MODE_REG        0x04
+#define CPU5WDT_TRIGGER_REG     0x07
+#define CPU5WDT_ENABLE_REG      0x08
+#define CPU5WDT_RESET_REG       0x09
+
+#define CPU5WDT_INTERVAL	(HZ/10+1)
+
+/* some device data */
+
+static struct {
+	struct semaphore stop;
+	volatile int running;
+	struct timer_list timer;
+	volatile int queue;
+	int default_ticks;
+	int min_ticks;
+	unsigned long inuse;
+} cpu5wdt_device;
+
+/* generic helper functions */
+
+static void cpu5wdt_trigger(unsigned long unused)
+{
+	if ( verbose > 2 )
+		printk(KERN_DEBUG PFX "trigger at %i ticks\n", ticks);
+
+	if( cpu5wdt_device.running )
+		ticks--;
+
+	/* keep watchdog alive */
+	outb(1, port + CPU5WDT_TRIGGER_REG);
+
+	/* requeue?? */
+	if( cpu5wdt_device.queue && ticks ) {
+		cpu5wdt_device.timer.expires = jiffies + CPU5WDT_INTERVAL;
+		add_timer(&cpu5wdt_device.timer);
+	}
+	else {
+		/* ticks doesn't matter anyway */
+		up(&cpu5wdt_device.stop);
+	}
+
+}
+
+static void cpu5wdt_reset(void)
+{
+	if ( ticks < cpu5wdt_device.min_ticks )
+		cpu5wdt_device.min_ticks = ticks;
+
+	ticks = cpu5wdt_device.default_ticks;
+
+	if ( verbose )
+		printk(KERN_DEBUG PFX "reset (%i ticks)\n", (int) ticks);
+
+}
+
+static void cpu5wdt_start(void)
+{
+	if ( !cpu5wdt_device.queue ) {
+		cpu5wdt_device.queue = 1;
+		outb(0, port + CPU5WDT_TIME_A_REG);  
+		outb(0, port + CPU5WDT_TIME_B_REG);  
+		outb(1, port + CPU5WDT_MODE_REG);
+		outb(0, port + CPU5WDT_RESET_REG);
+		outb(0, port + CPU5WDT_ENABLE_REG);
+		cpu5wdt_device.timer.expires = jiffies + CPU5WDT_INTERVAL;
+		add_timer(&cpu5wdt_device.timer);
+	}
+	/* if process dies, counter is not decremented */
+	cpu5wdt_device.running++;
+}
+
+static int cpu5wdt_stop(void)
+{
+	if ( cpu5wdt_device.running )
+		cpu5wdt_device.running = 0;
+
+	ticks = cpu5wdt_device.default_ticks;
+
+	if ( verbose )
+		printk(KERN_CRIT PFX "stop not possible\n");
+
+	return -EIO;
+}
+
+#ifdef CONFIG_PROC_FS
+static int cpu5wdt_read_proc(char *buf, char **start, off_t offset, int len)
+{
+	len = sprintf(buf,      "activation:       %i\n", cpu5wdt_device.queue);
+	len += sprintf(buf+len, "status:           %i\n", cpu5wdt_device.running);
+	len += sprintf(buf+len, "current ticks: %i\n", ticks);
+	len += sprintf(buf+len, "min ticks:     %i\n", cpu5wdt_device.min_ticks);
+	return len;
+}
+
+static inline void cpu5wdt_register_proc(void)
+{
+	create_proc_info_entry("driver/cpu5wdt", 0, NULL, cpu5wdt_read_proc);
+}
+
+static inline void cpu5wdt_unregister_proc(void) {
+	remove_proc_entry("driver/cpu5wdt", NULL);
+}
+#else
+static inline void cpu5wdt_register_proc(void) {}
+static inline void cpu5wdt_unregister_proc(void) {}
+#endif
+
+/* filesystem operations */
+
+static int cpu5wdt_open(struct inode *inode, struct file *file)
+{
+	switch(minor(inode->i_rdev)) {
+		case WATCHDOG_MINOR:
+			if ( test_and_set_bit(0, &cpu5wdt_device.inuse) )
+				return -EBUSY;
+			break;
+		default:
+			return -ENODEV;
+	}
+	return 0;
+
+}
+
+static int cpu5wdt_release(struct inode *inode, struct file *file)
+{
+	if(minor(inode->i_rdev)==WATCHDOG_MINOR) {
+		clear_bit(0, &cpu5wdt_device.inuse);
+	}
+	return 0;
+}
+
+static int cpu5wdt_ioctl(struct inode *inode, struct file *file, unsigned int cmd, unsigned long arg)
+{
+	unsigned int value;
+	static struct watchdog_info ident =
+	{
+		.options = WDIOF_CARDRESET,
+		.identity = "CPU5 WDT"
+	};
+  
+	switch(cmd) {
+		case WDIOC_KEEPALIVE:
+			cpu5wdt_reset();
+			break;
+		case WDIOC_GETSTATUS:    
+			value = inb(port + CPU5WDT_STATUS_REG); 
+			value = (value >> 2) & 1;
+			if ( copy_to_user((int *)arg, (int *)&value, sizeof(int)) )
+				return -EFAULT;
+			break;
+		case WDIOC_GETSUPPORT:
+			if ( copy_to_user((struct watchdog_info *)arg, &ident, sizeof(ident)) )
+				return -EFAULT;
+			break;
+		case WDIOC_SETOPTIONS:
+			if ( copy_from_user(&value, (int *)arg, sizeof(int)) )
+				return -EFAULT;
+			switch(value) {
+				case WDIOS_ENABLECARD:
+					cpu5wdt_start();
+					break;
+				case WDIOS_DISABLECARD:
+					return cpu5wdt_stop();
+				default:
+					return -EINVAL;
+			}
+			break;
+		default:
+    			return -EINVAL;
+	}
+	return 0;
+}
+
+static ssize_t cpu5wdt_write(struct file *file, const char *buf, size_t count, loff_t *ppos)
+{
+	if ( !count )
+		return -EIO;
+	
+	cpu5wdt_reset();
+	return count;
+
+}
+
+static struct file_operations cpu5wdt_fops = {
+	.owner		= THIS_MODULE,
+	.ioctl		= cpu5wdt_ioctl,
+	.open		= cpu5wdt_open,
+	.write		= cpu5wdt_write,
+	.release	= cpu5wdt_release,
+};
+
+static struct miscdevice cpu5wdt_misc = {
+	.minor	= WATCHDOG_MINOR,
+	.name	= "watchdog",
+	.fops	= &cpu5wdt_fops
+};
+
+/* init/exit function */
+
+static int __devinit cpu5wdt_init(void)
+{
+	unsigned int val;
+	int err;
+
+	if ( verbose )
+		printk(KERN_DEBUG PFX "port=0x%x, verbose=%i\n", port, verbose);
+
+	if ( (err = misc_register(&cpu5wdt_misc)) < 0 ) {
+		printk(KERN_ERR PFX "misc_register failed\n");
+		goto no_misc;
+	}
+
+	if ( !request_region(port, CPU5WDT_EXTENT, PFX) ) {
+		printk(KERN_ERR PFX "request_region failed\n");
+		err = -EBUSY;
+		goto no_port;
+	}
+
+	/* watchdog reboot? */
+	val = inb(port + CPU5WDT_STATUS_REG); 
+	val = (val >> 2) & 1;
+	if ( !val )
+		printk(KERN_INFO PFX "sorry, was my fault\n");
+
+	init_MUTEX_LOCKED(&cpu5wdt_device.stop);
+	cpu5wdt_device.queue = 0;
+	cpu5wdt_device.min_ticks = ticks;
+
+	clear_bit(0, &cpu5wdt_device.inuse);
+
+	cpu5wdt_register_proc();
+
+	init_timer(&cpu5wdt_device.timer);
+	cpu5wdt_device.timer.function = cpu5wdt_trigger;
+	cpu5wdt_device.timer.data = 0;
+
+	cpu5wdt_device.default_ticks = ticks;
+
+	printk(KERN_INFO PFX "init success\n");
+
+	return 0;
+
+no_port:
+	misc_deregister(&cpu5wdt_misc);
+no_misc:
+	return err;
+}
+
+static int __devinit cpu5wdt_init_module(void)
+{
+	return cpu5wdt_init();
+}
+
+static void __devexit cpu5wdt_exit(void)
+{
+	if ( cpu5wdt_device.queue ) {
+		cpu5wdt_device.queue = 0;
+		down(&cpu5wdt_device.stop);
+	}
+
+	cpu5wdt_unregister_proc();
+
+	misc_deregister(&cpu5wdt_misc);
+
+	release_region(port, CPU5WDT_EXTENT);
+
+}
+
+static void __devexit cpu5wdt_exit_module(void)
+{
+	cpu5wdt_exit();
+}
+
+/* module entry points */
+
+module_init(cpu5wdt_init_module);
+module_exit(cpu5wdt_exit_module);
+
+MODULE_AUTHOR("Heiko Ronsdorf <hero@ihg.uni-duisburg.de>");
+MODULE_DESCRIPTION("sma cpu5 watchdog driver");
+MODULE_SUPPORTED_DEVICE("sma cpu5 watchdog");
+MODULE_LICENSE("GPL");
+
+MODULE_PARM(port, "i");
+MODULE_PARM_DESC(port, "base address of watchdog card, default is 0x91");
+
+MODULE_PARM(verbose, "i");
+MODULE_PARM_DESC(verbose, "be verbose, default is 0 (no)");
+
+MODULE_PARM(ticks, "i");
+MODULE_PARM_DESC(ticks, "count down ticks, default is 10000");
+
+EXPORT_NO_SYMBOLS;

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH][DRIVER][RFC] CPU5 watchdog driver for 2.5
  2003-02-11 12:26   ` Heiko Ronsdorf
  2003-02-11  4:38     ` Rusty Lynch
@ 2003-02-11 13:11     ` Dave Jones
  1 sibling, 0 replies; 6+ messages in thread
From: Dave Jones @ 2003-02-11 13:11 UTC (permalink / raw)
  To: Heiko Ronsdorf, linux-kernel

On Tue, Feb 11, 2003 at 01:26:20PM +0100, Heiko Ronsdorf wrote:
 > > * I'm pretty sure that in general adding new code to /proc (that has
 > > nothing to do with processes) is frowned on.
 > 
 > I don't know how to track the watchdog. Suggestions are welcome.

Last week someone suggested a sysfs interface for the watchdog drivers. 
If this is done, it should be in a generic way that all the watchdogs
automatically benefit from rather than duplicate the same sysfs
code in every watchdog driver.

		Dave

-- 
| Dave Jones.        http://www.codemonkey.org.uk
| SuSE Labs

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH][DRIVER][RFC] CPU5 watchdog driver for 2.5
  2003-02-11  4:38     ` Rusty Lynch
@ 2003-02-12 16:25       ` Heiko Ronsdorf
  0 siblings, 0 replies; 6+ messages in thread
From: Heiko Ronsdorf @ 2003-02-12 16:25 UTC (permalink / raw)
  To: Rusty Lynch; +Cc: linux-kernel, mochel

[-- Attachment #1: Type: text/plain, Size: 2283 bytes --]



> Here is your cpu5wdt.c file with the procfs stuff replaced with code
> that adds an embedded platform_device to your basic data structure,
> registers the driver as a platform device, and then creates a file
> called $YOUR_SYSFS_ROOT/devices/legacy/watchdog0/statistics that will
> read the same as your original procfs file.  (Or at least I think it
> will.  It at least compiles :->)

>From Documentation/filesystems/sysfs.txt

"Mixing types, expressing mulitple lines of data, and doing fancy
formatting of data is heavily frowned upon. Doing these things may get
you publically humiliated and your code rewritten without notice."

Luckily it wasn't my suggestion. :-)

Well, in the right manner to use sysfs I have to create a file for
each (4) of the variables. So I have to write 4 (!) functions for
reading my vars, and all look similar. :-( This is over-designed for
no (real) functionality in only one driver, I guess. Only two or three
of the other watchdog drivers provide a "read" on some data, so no
code can shared between them.

There should be some functions/MACROS in sysfs to declare a read/write
operation on an single int/long etc.. Something like:

struct attribute {
	char		* name;
	mode_t		mode;
	void *		data;
};

struct device_attribute {
	struct attribute        attr;
	ssize_t (*show)(struct device * dev, char * buf, void * data);
	ssize_t (*store)(struct device * dev, const char * buf, \
			size_t count, void * data);
};

#define DEVICE_INT_ATTR(_name,_str,_mode,_var)	\
struct device_attribute dev_attr_##_name = {	\
	.attr = {.name = __stringify(_name), .mode = _mode, .data = &var }, \
	.show = _default_int_show,		\
	.store = _default_int_store,		\
};

size_t _default_int_show(struct device *dev, char *buf, void *data) {
	return sprintf("%i\n", *(int *)data);
}

static ssize_t
subsys_attr_show(struct kobject * kobj, struct attribute * attr, char * page)
{
	struct subsystem * s = to_subsys(kobj);
	struct subsys_attribute * sattr = to_sattr(attr);
	ssize_t ret = 0;

	if (sattr->show)
		ret = sattr->show(s,page,attr->data);
	return ret;
}

With these [IMHO] it would be easier to make use of sysfs.

Finally, I've decided to remove the procfs code (and keep a patch for
myself) from the watchdog driver.

Heiko

--
Sorry for my denglisch


[-- Attachment #2: cpu5wdt-2.5.60-4.diff --]
[-- Type: text/plain, Size: 8789 bytes --]

diff -urN linux-vanilla/drivers/char/watchdog/Kconfig linux-patched/drivers/char/watchdog/Kconfig
--- linux-vanilla/drivers/char/watchdog/Kconfig	Tue Feb 11 11:03:51 2003
+++ linux-patched/drivers/char/watchdog/Kconfig	Mon Feb 10 22:02:12 2003
@@ -354,4 +354,14 @@
 	  Documentation/modules.txt. The module will be called
 	  wafer5823wdt.o
 
+config CPU5_WDT
+	tristate "SMA CPU5 Watchdog"
+	depends on WATCHDOG
+	---help---
+	  TBD.
+	  This driver is also available as a module ( = code which can be
+	  inserted in and removed from the running kernel whenever you want).
+	  The module is called cpu5wdt.o.  If you want to compile it as a
+	  module, say M here and read <file:Documentation/modules.txt>.
+
 endmenu
diff -urN linux-vanilla/drivers/char/watchdog/Makefile linux-patched/drivers/char/watchdog/Makefile
--- linux-vanilla/drivers/char/watchdog/Makefile	Tue Feb 11 11:03:51 2003
+++ linux-patched/drivers/char/watchdog/Makefile	Mon Feb 10 22:02:12 2003
@@ -29,3 +29,4 @@
 obj-$(CONFIG_ALIM7101_WDT) += alim7101_wdt.o
 obj-$(CONFIG_SC1200_WDT) += sc1200wdt.o
 obj-$(CONFIG_WAFER_WDT) += wafer5823wdt.o
+obj-$(CONFIG_CPU5_WDT) += cpu5wdt.o
diff -urN linux-vanilla/drivers/char/watchdog/cpu5wdt.c linux-patched/drivers/char/watchdog/cpu5wdt.c
--- linux-vanilla/drivers/char/watchdog/cpu5wdt.c	Thu Jan  1 01:00:00 1970
+++ linux-patched/drivers/char/watchdog/cpu5wdt.c	Wed Feb 12 00:06:05 2003
@@ -0,0 +1,316 @@
+/*
+ * sma cpu5 watchdog driver
+ *
+ * Copyright (C) 2003 Heiko Ronsdorf <hero@ihg.uni-duisburg.de>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+ *
+ */
+
+#include <linux/config.h>
+#include <linux/module.h>
+#include <linux/types.h>
+#include <linux/errno.h>
+#include <linux/miscdevice.h>
+#include <linux/kernel.h>
+#include <linux/fs.h>
+#include <linux/init.h>
+#include <linux/ioport.h>
+#include <linux/timer.h>
+#include <asm/io.h>
+#include <asm/uaccess.h>
+
+#include <linux/watchdog.h>
+
+/* adjustable parameters */
+
+static int verbose = 0;
+static int port = 0x91;
+static volatile int ticks = 10000;
+
+#define PFX			"cpu5wdt: "
+
+#define CPU5WDT_EXTENT          0x0A
+
+#define CPU5WDT_STATUS_REG      0x00
+#define CPU5WDT_TIME_A_REG      0x02
+#define CPU5WDT_TIME_B_REG      0x03
+#define CPU5WDT_MODE_REG        0x04
+#define CPU5WDT_TRIGGER_REG     0x07
+#define CPU5WDT_ENABLE_REG      0x08
+#define CPU5WDT_RESET_REG       0x09
+
+#define CPU5WDT_INTERVAL	(HZ/10+1)
+
+/* some device data */
+
+static struct {
+	struct semaphore stop;
+	volatile int running;
+	struct timer_list timer;
+	volatile int queue;
+	int default_ticks;
+	int min_ticks;
+	unsigned long inuse;
+} cpu5wdt_device;
+
+/* generic helper functions */
+
+static void cpu5wdt_trigger(unsigned long unused)
+{
+	if ( verbose > 2 )
+		printk(KERN_DEBUG PFX "trigger at %i ticks\n", ticks);
+
+	if( cpu5wdt_device.running )
+		ticks--;
+
+	/* keep watchdog alive */
+	outb(1, port + CPU5WDT_TRIGGER_REG);
+
+	/* requeue?? */
+	if( cpu5wdt_device.queue && ticks ) {
+		cpu5wdt_device.timer.expires = jiffies + CPU5WDT_INTERVAL;
+		add_timer(&cpu5wdt_device.timer);
+	}
+	else {
+		/* ticks doesn't matter anyway */
+		up(&cpu5wdt_device.stop);
+	}
+
+}
+
+static void cpu5wdt_reset(void)
+{
+	if ( ticks < cpu5wdt_device.min_ticks )
+		cpu5wdt_device.min_ticks = ticks;
+
+	ticks = cpu5wdt_device.default_ticks;
+
+	if ( verbose )
+		printk(KERN_DEBUG PFX "reset (%i ticks)\n", (int) ticks);
+
+}
+
+static void cpu5wdt_start(void)
+{
+	if ( !cpu5wdt_device.queue ) {
+		cpu5wdt_device.queue = 1;
+		outb(0, port + CPU5WDT_TIME_A_REG);  
+		outb(0, port + CPU5WDT_TIME_B_REG);  
+		outb(1, port + CPU5WDT_MODE_REG);
+		outb(0, port + CPU5WDT_RESET_REG);
+		outb(0, port + CPU5WDT_ENABLE_REG);
+		cpu5wdt_device.timer.expires = jiffies + CPU5WDT_INTERVAL;
+		add_timer(&cpu5wdt_device.timer);
+	}
+	/* if process dies, counter is not decremented */
+	cpu5wdt_device.running++;
+}
+
+static int cpu5wdt_stop(void)
+{
+	if ( cpu5wdt_device.running )
+		cpu5wdt_device.running = 0;
+
+	ticks = cpu5wdt_device.default_ticks;
+
+	if ( verbose )
+		printk(KERN_CRIT PFX "stop not possible\n");
+
+	return -EIO;
+}
+
+/* filesystem operations */
+
+static int cpu5wdt_open(struct inode *inode, struct file *file)
+{
+	switch(minor(inode->i_rdev)) {
+		case WATCHDOG_MINOR:
+			if ( test_and_set_bit(0, &cpu5wdt_device.inuse) )
+				return -EBUSY;
+			break;
+		default:
+			return -ENODEV;
+	}
+	return 0;
+
+}
+
+static int cpu5wdt_release(struct inode *inode, struct file *file)
+{
+	if(minor(inode->i_rdev)==WATCHDOG_MINOR) {
+		clear_bit(0, &cpu5wdt_device.inuse);
+	}
+	return 0;
+}
+
+static int cpu5wdt_ioctl(struct inode *inode, struct file *file, unsigned int cmd, unsigned long arg)
+{
+	unsigned int value;
+	static struct watchdog_info ident =
+	{
+		.options = WDIOF_CARDRESET,
+		.identity = "CPU5 WDT"
+	};
+  
+	switch(cmd) {
+		case WDIOC_KEEPALIVE:
+			cpu5wdt_reset();
+			break;
+		case WDIOC_GETSTATUS:    
+			value = inb(port + CPU5WDT_STATUS_REG); 
+			value = (value >> 2) & 1;
+			if ( copy_to_user((int *)arg, (int *)&value, sizeof(int)) )
+				return -EFAULT;
+			break;
+		case WDIOC_GETSUPPORT:
+			if ( copy_to_user((struct watchdog_info *)arg, &ident, sizeof(ident)) )
+				return -EFAULT;
+			break;
+		case WDIOC_SETOPTIONS:
+			if ( copy_from_user(&value, (int *)arg, sizeof(int)) )
+				return -EFAULT;
+			switch(value) {
+				case WDIOS_ENABLECARD:
+					cpu5wdt_start();
+					break;
+				case WDIOS_DISABLECARD:
+					return cpu5wdt_stop();
+				default:
+					return -EINVAL;
+			}
+			break;
+		default:
+    			return -EINVAL;
+	}
+	return 0;
+}
+
+static ssize_t cpu5wdt_write(struct file *file, const char *buf, size_t count, loff_t *ppos)
+{
+	if ( !count )
+		return -EIO;
+	
+	cpu5wdt_reset();
+	return count;
+
+}
+
+static struct file_operations cpu5wdt_fops = {
+	.owner		= THIS_MODULE,
+	.ioctl		= cpu5wdt_ioctl,
+	.open		= cpu5wdt_open,
+	.write		= cpu5wdt_write,
+	.release	= cpu5wdt_release,
+};
+
+static struct miscdevice cpu5wdt_misc = {
+	.minor	= WATCHDOG_MINOR,
+	.name	= "watchdog",
+	.fops	= &cpu5wdt_fops
+};
+
+/* init/exit function */
+
+static int __devinit cpu5wdt_init(void)
+{
+	unsigned int val;
+	int err;
+
+	if ( verbose )
+		printk(KERN_DEBUG PFX "port=0x%x, verbose=%i\n", port, verbose);
+
+	if ( (err = misc_register(&cpu5wdt_misc)) < 0 ) {
+		printk(KERN_ERR PFX "misc_register failed\n");
+		goto no_misc;
+	}
+
+	if ( !request_region(port, CPU5WDT_EXTENT, PFX) ) {
+		printk(KERN_ERR PFX "request_region failed\n");
+		err = -EBUSY;
+		goto no_port;
+	}
+
+	/* watchdog reboot? */
+	val = inb(port + CPU5WDT_STATUS_REG); 
+	val = (val >> 2) & 1;
+	if ( !val )
+		printk(KERN_INFO PFX "sorry, was my fault\n");
+
+	init_MUTEX_LOCKED(&cpu5wdt_device.stop);
+	cpu5wdt_device.queue = 0;
+	cpu5wdt_device.min_ticks = ticks;
+
+	clear_bit(0, &cpu5wdt_device.inuse);
+
+	init_timer(&cpu5wdt_device.timer);
+	cpu5wdt_device.timer.function = cpu5wdt_trigger;
+	cpu5wdt_device.timer.data = 0;
+
+	cpu5wdt_device.default_ticks = ticks;
+
+	printk(KERN_INFO PFX "init success\n");
+
+	return 0;
+
+no_port:
+	misc_deregister(&cpu5wdt_misc);
+no_misc:
+	return err;
+}
+
+static int __devinit cpu5wdt_init_module(void)
+{
+	return cpu5wdt_init();
+}
+
+static void __devexit cpu5wdt_exit(void)
+{
+	if ( cpu5wdt_device.queue ) {
+		cpu5wdt_device.queue = 0;
+		down(&cpu5wdt_device.stop);
+	}
+
+	misc_deregister(&cpu5wdt_misc);
+
+	release_region(port, CPU5WDT_EXTENT);
+
+}
+
+static void __devexit cpu5wdt_exit_module(void)
+{
+	cpu5wdt_exit();
+}
+
+/* module entry points */
+
+module_init(cpu5wdt_init_module);
+module_exit(cpu5wdt_exit_module);
+
+MODULE_AUTHOR("Heiko Ronsdorf <hero@ihg.uni-duisburg.de>");
+MODULE_DESCRIPTION("sma cpu5 watchdog driver");
+MODULE_SUPPORTED_DEVICE("sma cpu5 watchdog");
+MODULE_LICENSE("GPL");
+
+MODULE_PARM(port, "i");
+MODULE_PARM_DESC(port, "base address of watchdog card, default is 0x91");
+
+MODULE_PARM(verbose, "i");
+MODULE_PARM_DESC(verbose, "be verbose, default is 0 (no)");
+
+MODULE_PARM(ticks, "i");
+MODULE_PARM_DESC(ticks, "count down ticks, default is 10000");
+
+EXPORT_NO_SYMBOLS;

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2003-02-12 16:15 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2003-02-10 20:17 [PATCH][DRIVER][RFC] CPU5 watchdog driver for 2.5 Heiko Ronsdorf
2003-02-10 22:33 ` Rusty Lynch
2003-02-11 12:26   ` Heiko Ronsdorf
2003-02-11  4:38     ` Rusty Lynch
2003-02-12 16:25       ` Heiko Ronsdorf
2003-02-11 13:11     ` Dave Jones

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).