Messages in this thread Patch in this message | | | From | Fabian Frederick <> | Subject | [PATCH V2] PM / AVS: SmartReflex: remove unnecessary NULL test before debugfs_remove_recursive | Date | Sat, 28 Jun 2014 15:48:33 +0200 |
| |
Fix checkpatch warning: WARNING: debugfs_remove_recursive(NULL) is safe this check is probably not required
Cc: Kevin Hilman <khilman@kernel.org> Cc: Nishanth Menon <nm@ti.com> Cc: linux-pm@vger.kernel.org Signed-off-by: Fabian Frederick <fabf@skynet.be> ---
V2: -Subject update (suggested by Nishanth Menon) -Blank line before debugfs_remove not after
drivers/power/avs/smartreflex.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/power/avs/smartreflex.c b/drivers/power/avs/smartreflex.c index db9973b..ed67616 100644 --- a/drivers/power/avs/smartreflex.c +++ b/drivers/power/avs/smartreflex.c @@ -1001,9 +1001,8 @@ static int omap_sr_remove(struct platform_device *pdev) if (sr_info->autocomp_active) sr_stop_vddautocomp(sr_info); - if (sr_info->dbg_dir) - debugfs_remove_recursive(sr_info->dbg_dir); + debugfs_remove_recursive(sr_info->dbg_dir); pm_runtime_disable(&pdev->dev); list_del(&sr_info->node); return 0; -- 1.8.4.5
| |