path: root/kernel/irq
diff options
authorGreg Kroah-Hartman <gregkh@linuxfoundation.org>2021-02-16 15:36:07 +0100
committerMarc Zyngier <maz@kernel.org>2021-03-08 20:12:08 +0000
commit69dd4503a7e6bae3389b8e028e5768008be8f2d7 (patch)
tree6a989b8f81738ccfc14c7ecc3d6d61e862293851 /kernel/irq
parenta38fd8748464831584a19438cbb3082b5a2dab15 (diff)
irqdomain: Remove debugfs_file from struct irq_domain
There's no need to keep around a dentry pointer to a simple file that debugfs itself can look up when we need to remove it from the system. So simplify the code by deleting the variable and cleaning up the logic around the debugfs file. Cc: Marc Zyngier <maz@kernel.org> Cc: Thomas Gleixner <tglx@linutronix.de> Cc: linux-kernel@vger.kernel.org Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Signed-off-by: Marc Zyngier <maz@kernel.org> Link: https://lore.kernel.org/r/YCvYV53ZdzQSWY6w@kroah.com
Diffstat (limited to 'kernel/irq')
1 files changed, 4 insertions, 5 deletions
diff --git a/kernel/irq/irqdomain.c b/kernel/irq/irqdomain.c
index 288151393a06..d10ab1d689d5 100644
--- a/kernel/irq/irqdomain.c
+++ b/kernel/irq/irqdomain.c
@@ -1898,16 +1898,15 @@ DEFINE_SHOW_ATTRIBUTE(irq_domain_debug);
static void debugfs_add_domain_dir(struct irq_domain *d)
- if (!d->name || !domain_dir || d->debugfs_file)
+ if (!d->name || !domain_dir)
- d->debugfs_file = debugfs_create_file(d->name, 0444, domain_dir, d,
- &irq_domain_debug_fops);
+ debugfs_create_file(d->name, 0444, domain_dir, d,
+ &irq_domain_debug_fops);
static void debugfs_remove_domain_dir(struct irq_domain *d)
- debugfs_remove(d->debugfs_file);
- d->debugfs_file = NULL;
+ debugfs_remove(debugfs_lookup(d->name, domain_dir));
void __init irq_domain_debugfs_init(struct dentry *root)