diff options
| author | Fabian Frederick <fabf@skynet.be> | 2014-06-28 13:58:17 +0200 |
|---|---|---|
| committer | Lee Jones <lee.jones@linaro.org> | 2014-07-09 14:58:21 +0100 |
| commit | 005d16b6022676ba8c6ae93e2d38cb0e70e0ef92 (patch) | |
| tree | ab78c1b6f3670c14aa30df03c8f057ee2475a4bc | |
| parent | d1fd345e2087f0362c92bd3b0a1cea7fe636ac3a (diff) | |
| download | op-kernel-dev-005d16b6022676ba8c6ae93e2d38cb0e70e0ef92.zip op-kernel-dev-005d16b6022676ba8c6ae93e2d38cb0e70e0ef92.tar.gz | |
mfd: ab8500-debugfs: Remove unnecessary null test before debugfs_remove_recursive
Fix checkpatch warning:
WARNING: debugfs_remove_recursive(NULL) is safe this check is probably not required
Signed-off-by: Fabian Frederick <fabf@skynet.be>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
| -rw-r--r-- | drivers/mfd/ab8500-debugfs.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/mfd/ab8500-debugfs.c b/drivers/mfd/ab8500-debugfs.c index d514e36..8452c15 100644 --- a/drivers/mfd/ab8500-debugfs.c +++ b/drivers/mfd/ab8500-debugfs.c @@ -3191,8 +3191,7 @@ static int ab8500_debug_probe(struct platform_device *plf) return 0; err: - if (ab8500_dir) - debugfs_remove_recursive(ab8500_dir); + debugfs_remove_recursive(ab8500_dir); dev_err(&plf->dev, "failed to create debugfs entries.\n"); return -ENOMEM; |
