diff options
author | John Crispin <blogic@openwrt.org> | 2007-12-13 19:55:14 +0000 |
---|---|---|
committer | John Crispin <blogic@openwrt.org> | 2007-12-13 19:55:14 +0000 |
commit | 0a830f80b41e85adb5ef1c9cab209124d02288d1 (patch) | |
tree | a4a7a49ec6ed86f505f7cf96dd5b7158476317bb /target/linux/danube | |
parent | 0b0b514d58d24359451e9f12df3897172d0e669f (diff) | |
download | master-187ad058-0a830f80b41e85adb5ef1c9cab209124d02288d1.tar.gz master-187ad058-0a830f80b41e85adb5ef1c9cab209124d02288d1.tar.bz2 master-187ad058-0a830f80b41e85adb5ef1c9cab209124d02288d1.zip |
cleanup danube watchdog timer
git-svn-id: svn://svn.openwrt.org/openwrt/trunk@9729 3c298f89-4303-0410-b956-a3cf2f4a3e73
Diffstat (limited to 'target/linux/danube')
-rw-r--r-- | target/linux/danube/files/drivers/char/watchdog/danube_wdt.c | 45 |
1 files changed, 18 insertions, 27 deletions
diff --git a/target/linux/danube/files/drivers/char/watchdog/danube_wdt.c b/target/linux/danube/files/drivers/char/watchdog/danube_wdt.c index 59901b7bec..cf01a9d793 100644 --- a/target/linux/danube/files/drivers/char/watchdog/danube_wdt.c +++ b/target/linux/danube/files/drivers/char/watchdog/danube_wdt.c @@ -110,7 +110,7 @@ danube_wdt_disable (void) /* passed LPEN or DSEN */ void -danube_wdt_low_power_or_debug (int en, int type) +danube_wdt_enable_feature (int en, int type) { unsigned int wdt_cr = 0; @@ -168,28 +168,27 @@ danube_wdt_ioctl (struct inode *inode, struct file *file, unsigned int cmd, unsigned long arg) { int result = 0; - int en = 0; - int istatus; - int pwl, clkdiv; static int timeout = -1; + unsigned int user_arg; + + if ((cmd != DANUBE_WDT_IOC_STOP) && (cmd != DANUBE_WDT_IOC_PING) && (cmd != DANUBE_WDT_IOC_GET_STATUS)) + { + if (copy_from_user((void *) &user_arg, (void *) arg, sizeof (int))) + result = -EINVAL; + } switch (cmd) { case DANUBE_WDT_IOC_START: printk("enable watch dog timer!\n"); - if (copy_from_user((void *) &timeout, (void *) arg, sizeof (int))) - { - printk ("invalid argument\n"); - result = -EINVAL; - } else { - if ((result = danube_wdt_enable (timeout)) < 0) - timeout = -1; - } + if ((result = danube_wdt_enable(user_arg)) < 0) + timeout = -1; + else + timeout = user_arg; break; case DANUBE_WDT_IOC_STOP: printk("disable watch dog timer\n"); - timeout = -1; danube_wdt_disable(); break; @@ -201,32 +200,24 @@ danube_wdt_ioctl (struct inode *inode, struct file *file, unsigned int cmd, break; case DANUBE_WDT_IOC_GET_STATUS: - istatus = readl(DANUBE_BIU_WDT_SR); - copy_to_user((int *) arg, (int *) &istatus, sizeof (int)); + user_arg = readl(DANUBE_BIU_WDT_SR); + copy_to_user((int*)arg, (int*)&user_arg, sizeof(int)); break; case DANUBE_WDT_IOC_SET_PWL: - if (copy_from_user((void *) &pwl, (void *) arg, sizeof (int))) - result = -EINVAL; - danube_wdt_prewarning_limit(pwl); + danube_wdt_prewarning_limit(user_arg); break; case DANUBE_WDT_IOC_SET_DSEN: - if (copy_from_user((void *) &en, (void *) arg, sizeof (int))) - result = -EINVAL; - danube_wdt_low_power_or_debug(en, DANUBE_BIU_WDT_CR_DSEN); + danube_wdt_enable_feature(user_arg, DANUBE_BIU_WDT_CR_DSEN); break; case DANUBE_WDT_IOC_SET_LPEN: - if (copy_from_user((void *) &en, (void *) arg, sizeof (int))) - result = -EINVAL; - danube_wdt_low_power_or_debug(en, DANUBE_BIU_WDT_CR_LPEN); + danube_wdt_enable_feature(user_arg, DANUBE_BIU_WDT_CR_LPEN); break; case DANUBE_WDT_IOC_SET_CLKDIV: - if (copy_from_user((void *) &clkdiv, (void *) arg, sizeof (int))) - result = -EINVAL; - danube_wdt_set_clkdiv(clkdiv); + danube_wdt_set_clkdiv(user_arg); break; } |