genirq/debugfs: Remove redundant NULL pointer check

KAGA-KOKO · KAGA-KOKO · commit f610c9d68b1a · 2017-07-07T08:57:57.000+02:00
debugfs_remove() can be called with a NULL pointer. Fixes: 087cdfb ("genirq/debugfs: Add proper debugfs interface") Reported-by: Fengguang Wu <fengguang.wu@intel.com> Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
diff --git a/kernel/irq/irqdomain.c b/kernel/irq/irqdomain.c
@@ -1667,8 +1667,7 @@ static void debugfs_add_domain_dir(struct irq_domain *d)
 
 static void debugfs_remove_domain_dir(struct irq_domain *d)
 {
-	if (d->debugfs_file)
-		debugfs_remove(d->debugfs_file);
+	debugfs_remove(d->debugfs_file);
 }
 
 void __init irq_domain_debugfs_init(struct dentry *root)

Original file line number	Diff line number	Diff line change
`@@ -1667,8 +1667,7 @@ static void debugfs_add_domain_dir(struct irq_domain *d)`
`1667`	`1667`
`1668`	`1668`	`static void debugfs_remove_domain_dir(struct irq_domain *d)`
`1669`	`1669`	`{`
`1670`		`- if (d->debugfs_file)`
`1671`		`- debugfs_remove(d->debugfs_file);`
	`1670`	`+ debugfs_remove(d->debugfs_file);`
`1672`	`1671`	`}`
`1673`	`1672`
`1674`	`1673`	`void __init irq_domain_debugfs_init(struct dentry *root)`