diff options
author | Alexey Dobriyan <adobriyan@gmail.com> | 2006-12-06 23:36:21 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.osdl.org> | 2006-12-07 11:39:33 -0500 |
commit | b4178ab58aa81f4ed3c75c48940682fe3b45d880 (patch) | |
tree | d9bd5827ee1f9a36bf9958a56760f6206272cc2f /drivers/block/paride/comm.c | |
parent | f4330002d11f032559954cbff68a5cad95b6d27f (diff) |
[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 = { | |||
205 | 205 | ||
206 | static int __init comm_init(void) | 206 | static int __init comm_init(void) |
207 | { | 207 | { |
208 | return paride_register(&comm)-1; | 208 | return paride_register(&comm); |
209 | } | 209 | } |
210 | 210 | ||
211 | static void __exit comm_exit(void) | 211 | static void __exit comm_exit(void) |