diff options
| author |   Alexey Dobriyan <adobriyan@gmail.com> | 2006-01-10 00:09:16 +0300 |
|---|---|---|
| committer |   Linus Torvalds <torvalds@g5.osdl.org> | 2006-01-09 13:02:51 -0800 |
| commit | 682e852e2638ed0aff84aa51181c9e5d2f939562 (patch) | |
| tree | e1a5221fe0c98d743fbceec2d0932f3556aecf93 /arch | |
| parent | [PATCH] Fix sg_page_malloc() memset (diff) | |
| download | linux-dev-682e852e2638ed0aff84aa51181c9e5d2f939562.tar.xz linux-dev-682e852e2638ed0aff84aa51181c9e5d2f939562.zip | |
[PATCH] Fix more "if ((err = foo() < 0))" typos
Another reason to use:
ret = foo();
if (ret < 0)
goto out;
Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'arch')
| -rw-r--r-- | arch/mips/kernel/vpe.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/arch/mips/kernel/vpe.c b/arch/mips/kernel/vpe.c index 06be405be399..9c89eebc356f 100644 --- a/arch/mips/kernel/vpe.c +++ b/arch/mips/kernel/vpe.c @@ -1171,7 +1171,8 @@ static int __init vpe_module_init(void) return -ENODEV; } - if ((major = register_chrdev(0, module_name, &vpe_fops) < 0)) { + major = register_chrdev(0, module_name, &vpe_fops); + if (major < 0) { printk("VPE loader: unable to register character device\n"); return major; } |