trivial: cleanup hpfall example code (checkpatch)

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>
This commit is contained in:
Frans Pop 2009-06-22 11:08:36 +02:00 committed by Jiri Kosina
parent cf83011d8f
commit b519c15d4a

View file

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