summaryrefslogtreecommitdiffstats
path: root/arch
diff options
context:
space:
mode:
authorEmil Medve <Emilian.Medve@Freescale.com>2008-05-23 05:49:22 +1000
committerPaul Mackerras <paulus@samba.org>2008-06-16 15:00:25 +1000
commit476ff8a0e3b17fc23994255aa9fd917d599d2ec7 (patch)
tree6586103ad941312dcb3caa31b7fb162427c04a6f /arch
parent598056d5af8fef1dbe8f96f5c2b641a528184e5a (diff)
downloadop-kernel-dev-476ff8a0e3b17fc23994255aa9fd917d599d2ec7.zip
op-kernel-dev-476ff8a0e3b17fc23994255aa9fd917d599d2ec7.tar.gz
[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.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/powerpc/kernel/irq.c b/arch/powerpc/kernel/irq.c
index 2f73f70..bcc249d 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 = {
static int __init irq_debugfs_init(void)
{
if (debugfs_create_file("virq_mapping", S_IRUGO, powerpc_debugfs_root,
- NULL, &virq_debug_fops))
+ NULL, &virq_debug_fops) == NULL)
return -ENOMEM;
return 0;
OpenPOWER on IntegriCloud