diff options
author | Thomas Gleixner <tglx@linutronix.de> | 2017-06-24 05:05:59 -0400 |
---|---|---|
committer | Thomas Gleixner <tglx@linutronix.de> | 2017-06-24 05:43:53 -0400 |
commit | c2ce34c0a0e5187195ecade872be950d2611ba68 (patch) | |
tree | b0953c50658cc35611efa5486fc984e5f7f3fbbe /kernel/irq/internals.h | |
parent | 8d9d51b62e8558bbc11c6b978acad001f9ea7a42 (diff) |
genirq/debugfs: Remove pointless NULL pointer check
debugfs_remove() has it's own NULL pointer check. Remove the conditional
and make irq_remove_debugfs_entry() an inline helper
Reported-by: kbuild test robot <fengguang.wu@intel.com>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Diffstat (limited to 'kernel/irq/internals.h')
-rw-r--r-- | kernel/irq/internals.h | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/kernel/irq/internals.h b/kernel/irq/internals.h index 5fd105e252c3..a573e0771baf 100644 --- a/kernel/irq/internals.h +++ b/kernel/irq/internals.h | |||
@@ -318,8 +318,13 @@ static inline bool irq_fixup_move_pending(struct irq_desc *desc, bool fclear) | |||
318 | #endif /* !CONFIG_GENERIC_PENDING_IRQ */ | 318 | #endif /* !CONFIG_GENERIC_PENDING_IRQ */ |
319 | 319 | ||
320 | #ifdef CONFIG_GENERIC_IRQ_DEBUGFS | 320 | #ifdef CONFIG_GENERIC_IRQ_DEBUGFS |
321 | #include <linux/debugfs.h> | ||
322 | |||
321 | void irq_add_debugfs_entry(unsigned int irq, struct irq_desc *desc); | 323 | void irq_add_debugfs_entry(unsigned int irq, struct irq_desc *desc); |
322 | void irq_remove_debugfs_entry(struct irq_desc *desc); | 324 | static inline void irq_remove_debugfs_entry(struct irq_desc *desc) |
325 | { | ||
326 | debugfs_remove(desc->debugfs_file); | ||
327 | } | ||
323 | # ifdef CONFIG_IRQ_DOMAIN | 328 | # ifdef CONFIG_IRQ_DOMAIN |
324 | void irq_domain_debugfs_init(struct dentry *root); | 329 | void irq_domain_debugfs_init(struct dentry *root); |
325 | # else | 330 | # else |