diff options
| author | Li Zetao <lizetao1@huawei.com> | 2024-09-07 11:09:59 +0800 |
|---|---|---|
| committer | Hans de Goede <hdegoede@redhat.com> | 2024-09-11 14:20:01 +0200 |
| commit | f80d7100f091e96b54beb32dc6d136520453582f (patch) | |
| tree | 90d05368ed493c28fa1659a87e0c3a50be25064d | |
| parent | cedf233530cc375343c5a0b612fe94392f246c99 (diff) | |
platform/olpc: Remove redundant null pointer checks in olpc_ec_setup_debugfs()
Since the debugfs_create_dir() never returns a null pointer, checking
the return value for a null pointer is redundant. Since
debugfs_create_file() can deal with a ERR_PTR() style pointer, drop
the check.
Signed-off-by: Li Zetao <lizetao1@huawei.com>
Link: https://lore.kernel.org/r/20240907031009.3591057-2-lizetao1@huawei.com
Reviewed-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
| -rw-r--r-- | drivers/platform/olpc/olpc-ec.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/drivers/platform/olpc/olpc-ec.c b/drivers/platform/olpc/olpc-ec.c index 921520475ff6..48e9861bb571 100644 --- a/drivers/platform/olpc/olpc-ec.c +++ b/drivers/platform/olpc/olpc-ec.c @@ -332,9 +332,6 @@ static struct dentry *olpc_ec_setup_debugfs(void) struct dentry *dbgfs_dir; dbgfs_dir = debugfs_create_dir("olpc-ec", NULL); - if (IS_ERR_OR_NULL(dbgfs_dir)) - return NULL; - debugfs_create_file("cmd", 0600, dbgfs_dir, NULL, &ec_dbgfs_ops); return dbgfs_dir; |