On Tuesday, December 11, 2018 7:53:05 PM CET Kevin Hilman wrote: > Thomas Meyer <tho...@m3y3r.de> writes: > > > NULL check before some freeing functions is not needed. > > > > Signed-off-by: Thomas Meyer <tho...@m3y3r.de> > > Reviewed-by: Kevin Hilman <khil...@baylibre.com> > > > --- > > > > diff -u -p a/drivers/power/avs/smartreflex.c > > b/drivers/power/avs/smartreflex.c > > --- a/drivers/power/avs/smartreflex.c > > +++ b/drivers/power/avs/smartreflex.c > > @@ -1010,8 +1010,7 @@ static int omap_sr_remove(struct platfor > > > > 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); >
Patch applied, thanks!
