trivial: cleanup hpfall example code (checkpatch)
authorFrans Pop <elendil@planet.nl>
Mon, 22 Jun 2009 09:08:36 +0000 (11:08 +0200)
committerJiri Kosina <jkosina@suse.cz>
Mon, 21 Sep 2009 13:14:52 +0000 (15:14 +0200)
This patch makes hpfall.c conform to kernel coding style.

I have not fixed the C99 // comments on two lines as they
help indicate that those are not actually comments but
incomplete code.

Before:
 total: 10 errors, 6 warnings, 101 lines checked
After:
 total: 2 errors, 0 warnings, 99 lines checked

Signed-off-by: Frans Pop <elendil@planet.nl>
Acked-by: Pavel Machek <pavel@ucw.cz>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
Documentation/hwmon/hpfall.c

index bbea1cc..d2f6711 100644 (file)
@@ -57,45 +57,43 @@ void ignore_me(void)
 {
        protect(0);
        set_led(0);
-
 }
 
-int main(int argc, charargv[])
+int main(int argc, char *argv[])
 {
-       int fd, ret;
+       int fd, ret;
 
-       fd = open("/dev/freefall", O_RDONLY);
-       if (fd < 0) {
-               perror("open");
-               return EXIT_FAILURE;
-       }
+       fd = open("/dev/freefall", O_RDONLY);
+       if (fd < 0) {
+               perror("open");
+               return EXIT_FAILURE;
+       }
 
        signal(SIGALRM, ignore_me);
 
-       for (;;) {
-              unsigned char count;
-
-               ret = read(fd, &count, sizeof(count));
-              alarm(0);
-              if ((ret == -1) && (errno == EINTR)) {
-                      /* Alarm expired, time to unpark the heads */
-                      continue;
-              }
-
-               if (ret != sizeof(count)) {
-                       perror("read");
-                       break;
-               }
-
-              protect(21);
-              set_led(1);
-              if (1 || on_ac() || lid_open()) {
-                      alarm(2);
-              } else {
-                      alarm(20);
-              }
-       }
+       for (;;) {
+               unsigned char count;
+
+               ret = read(fd, &count, sizeof(count));
+               alarm(0);
+               if ((ret == -1) && (errno == EINTR)) {
+                       /* Alarm expired, time to unpark the heads */
+                       continue;
+               }
+
+               if (ret != sizeof(count)) {
+                       perror("read");
+                       break;
+               }
+
+               protect(21);
+               set_led(1);
+               if (1 || on_ac() || lid_open())
+                       alarm(2);
+               else
+                       alarm(20);
+       }
 
-       close(fd);
-       return EXIT_SUCCESS;
+       close(fd);
+       return EXIT_SUCCESS;
 }