diff options
author | Cornelia Huck <cornelia.huck@de.ibm.com> | 2006-04-11 01:53:45 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-04-11 09:18:37 -0400 |
commit | a7fbf6bba7f1f0616b0223a8b1cba91fffe607f5 (patch) | |
tree | fea61a369fe5efe55fea010a9d3608bd8f0fdc9c | |
parent | da074d0ac8ccae1068dc227ef9893c2510d23bd8 (diff) |
[PATCH] s390: wrong return codes in cio_ignore_proc_init()
cio_ignore_proc_init() returns 1 in case of success and 0 in case of failure.
The caller tests for != 0, so better return 0 in case of success and -ENOENT
in case of failure.
Signed-off-by: Cornelia Huck <cornelia.huck@de.ibm.com>
Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
-rw-r--r-- | drivers/s390/cio/blacklist.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/s390/cio/blacklist.c b/drivers/s390/cio/blacklist.c index cb8e2e672b68..0960bef7b199 100644 --- a/drivers/s390/cio/blacklist.c +++ b/drivers/s390/cio/blacklist.c | |||
@@ -414,11 +414,11 @@ cio_ignore_proc_init (void) | |||
414 | entry = create_proc_entry ("cio_ignore", S_IFREG | S_IRUGO | S_IWUSR, | 414 | entry = create_proc_entry ("cio_ignore", S_IFREG | S_IRUGO | S_IWUSR, |
415 | &proc_root); | 415 | &proc_root); |
416 | if (!entry) | 416 | if (!entry) |
417 | return 0; | 417 | return -ENOENT; |
418 | 418 | ||
419 | entry->proc_fops = &cio_ignore_proc_fops; | 419 | entry->proc_fops = &cio_ignore_proc_fops; |
420 | 420 | ||
421 | return 1; | 421 | return 0; |
422 | } | 422 | } |
423 | 423 | ||
424 | __initcall (cio_ignore_proc_init); | 424 | __initcall (cio_ignore_proc_init); |