i5100_edac: Remove two checkpatch warnings
The last changeset introduced a few checkpatch warnings:
WARNING: debugfs_remove_recursive(NULL) is safe this check is probably not required
261: FILE: drivers/edac/i5100_edac.c:1207:
+ if (priv->debugfs)
+ debugfs_remove_recursive(priv->debugfs);
WARNING: debugfs_remove(NULL) is safe this check is probably not required
290: FILE: drivers/edac/i5100_edac.c:1250:
+ if (i5100_debugfs)
+ debugfs_remove(i5100_debugfs);
Get rid of them.
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
diff --git a/drivers/edac/i5100_edac.c b/drivers/edac/i5100_edac.c
index ad4cc89..6ed11b1 100644
--- a/drivers/edac/i5100_edac.c
+++ b/drivers/edac/i5100_edac.c
@@ -1203,8 +1203,7 @@
priv = mci->pvt_info;
- if (priv->debugfs)
- debugfs_remove_recursive(priv->debugfs);
+ debugfs_remove_recursive(priv->debugfs);
priv->scrub_enable = 0;
cancel_delayed_work_sync(&(priv->i5100_scrubbing));
@@ -1246,8 +1245,7 @@
static void __exit i5100_exit(void)
{
- if (i5100_debugfs)
- debugfs_remove(i5100_debugfs);
+ debugfs_remove(i5100_debugfs);
pci_unregister_driver(&i5100_driver);
}