Messages in this thread | | | Subject | Re: [PATCH] drivers/fmc: remove unneeded NULL check | From | Mukesh Ojha <> | Date | Thu, 28 Mar 2019 15:52:39 +0530 |
| |
On 3/28/2019 2:53 PM, Ding Xiang wrote: > The variable will check in debugfs_remove_recursive, so > the NULL check here is not needed > > Signed-off-by: Ding Xiang <dingxiang@cmss.chinamobile.com>
Reviewed-by: Mukesh Ojha <mojha@codeaurora.org>
-Mukesh
> --- > drivers/fmc/fmc-debug.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/fmc/fmc-debug.c b/drivers/fmc/fmc-debug.c > index 3293072..ab868a3 100644 > --- a/drivers/fmc/fmc-debug.c > +++ b/drivers/fmc/fmc-debug.c > @@ -168,6 +168,5 @@ int fmc_debug_init(struct fmc_device *fmc) > > void fmc_debug_exit(struct fmc_device *fmc) > { > - if (fmc->dbg_dir) > - debugfs_remove_recursive(fmc->dbg_dir); > + debugfs_remove_recursive(fmc->dbg_dir); > }
| |