Commit 59b9796d authored Feb 21, 2013 by Mauro Carvalho Chehab

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>

parent 9cbc6d38

drivers/edac/i5100_edac.c

+2 −4

Original line number	Diff line number	Diff line
		@@ -1203,7 +1203,6 @@ static void i5100_remove_one(struct pci_dev *pdev)

		priv = mci->pvt_info;

		if (priv->debugfs)
		debugfs_remove_recursive(priv->debugfs);

		priv->scrub_enable = 0;
		@@ -1246,7 +1245,6 @@ static int __init i5100_init(void)

		static void __exit i5100_exit(void)
		{
		if (i5100_debugfs)
		debugfs_remove(i5100_debugfs);

		pci_unregister_driver(&i5100_driver);

Admin message