Messages in this thread | ![/](/images/icornerl.gif) | | From | "Rafael J. Wysocki" <> | Subject | Re: [PATCH] PM / AVS: SmartReflex: NULL check before some freeing functions is not needed. | Date | Mon, 17 Dec 2018 12:45:53 +0100 |
| |
On Tuesday, December 11, 2018 7:53:05 PM CET Kevin Hilman wrote: > Thomas Meyer <thomas@m3y3r.de> writes: > > > NULL check before some freeing functions is not needed. > > > > Signed-off-by: Thomas Meyer <thomas@m3y3r.de> > > Reviewed-by: Kevin Hilman <khilman@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!
| ![\](/images/icornerr.gif) |