diff options
author | Mauro Carvalho Chehab <mchehab@redhat.com> | 2013-02-21 09:01:23 -0500 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2013-02-21 09:06:34 -0500 |
commit | 59b9796d1e0e5edb6eb3d5ae550eac0d53d27adb (patch) | |
tree | 1e07817873d2d3be13bd1428c4fe36efd47315d7 /drivers/edac/i5100_edac.c | |
parent | 9cbc6d38f25ae8fb3efd0b1c14f4f18c1d9f0369 (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.c | 6 |
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 | ||
1247 | static void __exit i5100_exit(void) | 1246 | static 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 | } |