diff options
author | Emil Medve <Emilian.Medve@Freescale.com> | 2008-05-22 15:49:22 -0400 |
---|---|---|
committer | Paul Mackerras <paulus@samba.org> | 2008-06-16 01:00:25 -0400 |
commit | 476ff8a0e3b17fc23994255aa9fd917d599d2ec7 (patch) | |
tree | 6586103ad941312dcb3caa31b7fb162427c04a6f /arch | |
parent | 598056d5af8fef1dbe8f96f5c2b641a528184e5a (diff) |
[POWERPC] Fix return value check logic in debugfs virq_mapping setup
debugfs_create_file() returns a non-NULL (non-zero) value in case of
success, not a NULL value.
This fixes this non-critical boot-time debugging error message:
[ 1.316386] calling irq_debugfs_init+0x0/0x50
[ 1.316399] initcall irq_debugfs_init+0x0/0x50 returned -12 after 0 msecs
[ 1.316411] initcall irq_debugfs_init+0x0/0x50 returned with error code -12
Signed-off-by: Emil Medve <Emilian.Medve@Freescale.com>
Acked-by: Michael Ellerman <michael@ellerman.id.au>
Signed-off-by: Paul Mackerras <paulus@samba.org>
Diffstat (limited to 'arch')
-rw-r--r-- | arch/powerpc/kernel/irq.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/powerpc/kernel/irq.c b/arch/powerpc/kernel/irq.c index 2f73f705d564..bcc249d90c4d 100644 --- a/arch/powerpc/kernel/irq.c +++ b/arch/powerpc/kernel/irq.c | |||
@@ -1073,7 +1073,7 @@ static const struct file_operations virq_debug_fops = { | |||
1073 | static int __init irq_debugfs_init(void) | 1073 | static int __init irq_debugfs_init(void) |
1074 | { | 1074 | { |
1075 | if (debugfs_create_file("virq_mapping", S_IRUGO, powerpc_debugfs_root, | 1075 | if (debugfs_create_file("virq_mapping", S_IRUGO, powerpc_debugfs_root, |
1076 | NULL, &virq_debug_fops)) | 1076 | NULL, &virq_debug_fops) == NULL) |
1077 | return -ENOMEM; | 1077 | return -ENOMEM; |
1078 | 1078 | ||
1079 | return 0; | 1079 | return 0; |