Message ID | 1569293934-31451-1-git-send-email-dingxiang@cmss.chinamobile.com (mailing list archive) |
---|---|
State | Mainlined |
Commit | 0ad82e3a2234dfa38ec835d1772e8c9af176d458 |
Headers | show |
Series | remoteproc: debug: Remove unneeded NULL check | expand |
On Mon 23 Sep 19:58 PDT 2019, Ding Xiang wrote: > debugfs_remove_recursive will do NULL check, so remove > the redundant null check > Nice, applied. Regards, Bjorn > Signed-off-by: Ding Xiang <dingxiang@cmss.chinamobile.com> > --- > drivers/remoteproc/remoteproc_debugfs.c | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/drivers/remoteproc/remoteproc_debugfs.c b/drivers/remoteproc/remoteproc_debugfs.c > index 8cd4a0a..dd93cf0 100644 > --- a/drivers/remoteproc/remoteproc_debugfs.c > +++ b/drivers/remoteproc/remoteproc_debugfs.c > @@ -333,9 +333,6 @@ struct dentry *rproc_create_trace_file(const char *name, struct rproc *rproc, > > void rproc_delete_debug_dir(struct rproc *rproc) > { > - if (!rproc->dbg_dir) > - return; > - > debugfs_remove_recursive(rproc->dbg_dir); > } > > -- > 1.9.1 > > >
diff --git a/drivers/remoteproc/remoteproc_debugfs.c b/drivers/remoteproc/remoteproc_debugfs.c index 8cd4a0a..dd93cf0 100644 --- a/drivers/remoteproc/remoteproc_debugfs.c +++ b/drivers/remoteproc/remoteproc_debugfs.c @@ -333,9 +333,6 @@ struct dentry *rproc_create_trace_file(const char *name, struct rproc *rproc, void rproc_delete_debug_dir(struct rproc *rproc) { - if (!rproc->dbg_dir) - return; - debugfs_remove_recursive(rproc->dbg_dir); }
debugfs_remove_recursive will do NULL check, so remove the redundant null check Signed-off-by: Ding Xiang <dingxiang@cmss.chinamobile.com> --- drivers/remoteproc/remoteproc_debugfs.c | 3 --- 1 file changed, 3 deletions(-)