diff options
| author | Andy Shevchenko <andriy.shevchenko@linux.intel.com> | 2015-01-14 18:39:32 +0200 |
|---|---|---|
| committer | Thomas Gleixner <tglx@linutronix.de> | 2015-01-20 12:50:14 +0100 |
| commit | d5df8fe34bec4539e259525feabd16efccf16750 (patch) | |
| tree | 5e4979832bd657de4a8ebe5d3b74d191df0e482a | |
| parent | 1b43d7125f3b6f7d46e72da64f65f3187a83b66b (diff) | |
| download | lwn-d5df8fe34bec4539e259525feabd16efccf16750.tar.gz lwn-d5df8fe34bec4539e259525feabd16efccf16750.zip | |
x86: pmc_atom: don%27t check for NULL twice
debugfs_remove_recursive() is NULL-aware, thus, we may safely remove the check
here. There is no need to assing NULL to variable since it will be not used
anywhere.
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Acked-by: Aubrey Li <aubrey.li@linux.intel.com>
Cc: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Cc: Kumar P. Mahesh <mahesh.kumar.p@intel.com>
Link: http://lkml.kernel.org/r/1421253575-22509-3-git-send-email-andriy.shevchenko@linux.intel.com
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
| -rw-r--r-- | arch/x86/kernel/pmc_atom.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/arch/x86/kernel/pmc_atom.c b/arch/x86/kernel/pmc_atom.c index 8bb9a611ca23..1dd6c62d42eb 100644 --- a/arch/x86/kernel/pmc_atom.c +++ b/arch/x86/kernel/pmc_atom.c @@ -202,11 +202,7 @@ static const struct file_operations pmc_sleep_tmr_ops = { static void pmc_dbgfs_unregister(struct pmc_dev *pmc) { - if (!pmc->dbgfs_dir) - return; - debugfs_remove_recursive(pmc->dbgfs_dir); - pmc->dbgfs_dir = NULL; } static int pmc_dbgfs_register(struct pmc_dev *pmc, struct pci_dev *pdev) |