diff options
author | Alexey Dobriyan <adobriyan@gmail.com> | 2006-12-07 07:36:21 +0300 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.osdl.org> | 2006-12-07 19:39:33 +0300 |
commit | b4178ab58aa81f4ed3c75c48940682fe3b45d880 (patch) | |
tree | d9bd5827ee1f9a36bf9958a56760f6206272cc2f /drivers/block/paride/comm.c | |
parent | f4330002d11f032559954cbff68a5cad95b6d27f (diff) | |
download | linux-b4178ab58aa81f4ed3c75c48940682fe3b45d880.tar.xz |
[PATCH] paride_register(): shuffle return values
paride_register() returns 1 on success, 0 on failure and module init
code looks like
static int __init foo_init(void)
{
return paride_register(&foo) - 1;
}
which is not what one get used to. Converted to usual 0/-E convention.
In case of kbic driver, unwind registration. It was just
return (paride_register(&k951)||paride_register(&k971))-1;
Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'drivers/block/paride/comm.c')
-rw-r--r-- | drivers/block/paride/comm.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/block/paride/comm.c b/drivers/block/paride/comm.c index c3d4076e8c0e..9bcd35495323 100644 --- a/drivers/block/paride/comm.c +++ b/drivers/block/paride/comm.c @@ -205,7 +205,7 @@ static struct pi_protocol comm = { static int __init comm_init(void) { - return paride_register(&comm)-1; + return paride_register(&comm); } static void __exit comm_exit(void) |