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 | 41ed16fa47350661da01443b8241bf6ca8080fd7 (diff) | |
| download | op-kernel-dev-682e852e2638ed0aff84aa51181c9e5d2f939562.zip op-kernel-dev-682e852e2638ed0aff84aa51181c9e5d2f939562.tar.gz | |
[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 06be405..9c89eeb 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; } |
