aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/edac/i5100_edac.c
diff options
context:
space:
mode:
authorMauro Carvalho Chehab <mchehab@redhat.com>2013-02-21 09:01:23 -0500
committerMauro Carvalho Chehab <mchehab@redhat.com>2013-02-21 09:06:34 -0500
commit59b9796d1e0e5edb6eb3d5ae550eac0d53d27adb (patch)
tree1e07817873d2d3be13bd1428c4fe36efd47315d7 /drivers/edac/i5100_edac.c
parent9cbc6d38f25ae8fb3efd0b1c14f4f18c1d9f0369 (diff)
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>
Diffstat (limited to 'drivers/edac/i5100_edac.c')
-rw-r--r--drivers/edac/i5100_edac.c6
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/edac/i5100_edac.c b/drivers/edac/i5100_edac.c
index ad4cc898dc60..6ed11b1881b7 100644
--- a/drivers/edac/i5100_edac.c
+++ b/drivers/edac/i5100_edac.c
@@ -1203,8 +1203,7 @@ static void i5100_remove_one(struct pci_dev *pdev)
1203 1203
1204 priv = mci->pvt_info; 1204 priv = mci->pvt_info;
1205 1205
1206 if (priv->debugfs) 1206 debugfs_remove_recursive(priv->debugfs);
1207 debugfs_remove_recursive(priv->debugfs);
1208 1207
1209 priv->scrub_enable = 0; 1208 priv->scrub_enable = 0;
1210 cancel_delayed_work_sync(&(priv->i5100_scrubbing)); 1209 cancel_delayed_work_sync(&(priv->i5100_scrubbing));
@@ -1246,8 +1245,7 @@ static int __init i5100_init(void)
1246 1245
1247static void __exit i5100_exit(void) 1246static void __exit i5100_exit(void)
1248{ 1247{
1249 if (i5100_debugfs) 1248 debugfs_remove(i5100_debugfs);
1250 debugfs_remove(i5100_debugfs);
1251 1249
1252 pci_unregister_driver(&i5100_driver); 1250 pci_unregister_driver(&i5100_driver);
1253} 1251}