diff options
author | Wim Van Sebroeck <wim@iguana.be> | 2008-07-18 07:41:17 -0400 |
---|---|---|
committer | Wim Van Sebroeck <wim@iguana.be> | 2008-08-06 09:21:09 -0400 |
commit | 0c06090c9472db0525cb6fe229c3bea33bbbbb3c (patch) | |
tree | b01c21f6ef5649b6f0071dee4a55cb3ba236a7b8 /drivers/watchdog/cpu5wdt.c | |
parent | 5eb82498e3a6da8a979c48945e3c1a85c10ccc25 (diff) |
[WATCHDOG] Coding style - Indentation - part 2
This brings the watchdog drivers into line with coding style.
This patch takes cares of the indentation as described in chapter 1.
Main changes:
* Re-structure the ioctl switch call for all drivers as follows:
switch (cmd) {
case WDIOC_GETSUPPORT:
case WDIOC_GETSTATUS:
case WDIOC_GETBOOTSTATUS:
case WDIOC_GETTEMP:
case WDIOC_SETOPTIONS:
case WDIOC_KEEPALIVE:
case WDIOC_SETTIMEOUT:
case WDIOC_GETTIMEOUT:
case WDIOC_GETTIMELEFT:
default:
}
This to make the migration from the drivers to the uniform watchdog
device driver easier in the future.
Signed-off-by: Wim Van Sebroeck <wim@iguana.be>
Diffstat (limited to 'drivers/watchdog/cpu5wdt.c')
-rw-r--r-- | drivers/watchdog/cpu5wdt.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/drivers/watchdog/cpu5wdt.c b/drivers/watchdog/cpu5wdt.c index ec324e5e1c99..71f6d7eec9a8 100644 --- a/drivers/watchdog/cpu5wdt.c +++ b/drivers/watchdog/cpu5wdt.c | |||
@@ -160,8 +160,9 @@ static long cpu5wdt_ioctl(struct file *file, unsigned int cmd, | |||
160 | }; | 160 | }; |
161 | 161 | ||
162 | switch (cmd) { | 162 | switch (cmd) { |
163 | case WDIOC_KEEPALIVE: | 163 | case WDIOC_GETSUPPORT: |
164 | cpu5wdt_reset(); | 164 | if (copy_to_user(argp, &ident, sizeof(ident))) |
165 | return -EFAULT; | ||
165 | break; | 166 | break; |
166 | case WDIOC_GETSTATUS: | 167 | case WDIOC_GETSTATUS: |
167 | value = inb(port + CPU5WDT_STATUS_REG); | 168 | value = inb(port + CPU5WDT_STATUS_REG); |
@@ -169,10 +170,6 @@ static long cpu5wdt_ioctl(struct file *file, unsigned int cmd, | |||
169 | return put_user(value, p); | 170 | return put_user(value, p); |
170 | case WDIOC_GETBOOTSTATUS: | 171 | case WDIOC_GETBOOTSTATUS: |
171 | return put_user(0, p); | 172 | return put_user(0, p); |
172 | case WDIOC_GETSUPPORT: | ||
173 | if (copy_to_user(argp, &ident, sizeof(ident))) | ||
174 | return -EFAULT; | ||
175 | break; | ||
176 | case WDIOC_SETOPTIONS: | 173 | case WDIOC_SETOPTIONS: |
177 | if (get_user(value, p)) | 174 | if (get_user(value, p)) |
178 | return -EFAULT; | 175 | return -EFAULT; |
@@ -181,6 +178,9 @@ static long cpu5wdt_ioctl(struct file *file, unsigned int cmd, | |||
181 | if (value & WDIOS_DISABLECARD) | 178 | if (value & WDIOS_DISABLECARD) |
182 | cpu5wdt_stop(); | 179 | cpu5wdt_stop(); |
183 | break; | 180 | break; |
181 | case WDIOC_KEEPALIVE: | ||
182 | cpu5wdt_reset(); | ||
183 | break; | ||
184 | default: | 184 | default: |
185 | return -ENOTTY; | 185 | return -ENOTTY; |
186 | } | 186 | } |