Commit d5df8fe3 authored Jan 14, 2015 by Andy Shevchenko Committed by Thomas Gleixner Jan 20, 2015

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>

parent 1b43d712

arch/x86/kernel/pmc_atom.c

+0 −4

Original line number	Diff line number	Diff line
		@@ -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)