Use IS_ERR_OR_NULL() instead of the original !xxx || IS_ERR(xxx) logic.
This will simplify the code and enhance readability.
Signed-off-by: Liao Yuanhong <liaoyuanhong@xxxxxxxx>
---
drivers/s390/block/dasd.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/s390/block/dasd.c b/drivers/s390/block/dasd.c
index 506a947d00a5..78a9c8d781b3 100644
--- a/drivers/s390/block/dasd.c
+++ b/drivers/s390/block/dasd.c
@@ -215,7 +215,7 @@ static struct dentry *dasd_debugfs_setup(const char *name,
if (!base_dentry)
return NULL;
pde = debugfs_create_dir(name, base_dentry);
- if (!pde || IS_ERR(pde))
+ if (IS_ERR_OR_NULL(pde))
return NULL;
return pde;
}
@@ -1089,11 +1089,11 @@ static void dasd_statistics_createroot(void)
dasd_debugfs_root_entry = NULL;
pde = debugfs_create_dir("dasd", NULL);
- if (!pde || IS_ERR(pde))
+ if (IS_ERR_OR_NULL(pde))
goto error;
dasd_debugfs_root_entry = pde;
pde = debugfs_create_dir("global", dasd_debugfs_root_entry);
- if (!pde || IS_ERR(pde))
+ if (IS_ERR_OR_NULL(pde))
goto error;
dasd_debugfs_global_entry = pde;
dasd_profile_init(&dasd_global_profile, dasd_debugfs_global_entry);
--
2.34.1
