summaryrefslogtreecommitdiff
path: root/drivers/hwmon
diff options
context:
space:
mode:
authorDan Carpenter <dan.carpenter@oracle.com>2020-10-22 10:06:59 +0300
committerGuenter Roeck <linux@roeck-us.net>2020-12-03 04:42:22 +0300
commit92bc2e1f82afa2e456d7416598ffdf3dbf267835 (patch)
tree8aa0af4999cc84fdb491706d2c9f7f8119b2ed54 /drivers/hwmon
parent96eca8c97fccd3c02f61a87b0341a079b4096730 (diff)
downloadlinux-92bc2e1f82afa2e456d7416598ffdf3dbf267835.tar.xz
hwmon: (pmbus/max20730) delete some dead code
The debugfs_create_dir() function never returns NULL. Normal users are not supposed to check the return value so the correct fix is just to delete this check. In the case where the debugfs_create_dir() fails, the function returns NULL. The other debugfs function check for NULL directory and handle it. Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Link: https://lore.kernel.org/r/20201022070659.GA2817762@mwanda Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Diffstat (limited to 'drivers/hwmon')
-rw-r--r--drivers/hwmon/pmbus/max20730.c2
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/hwmon/pmbus/max20730.c b/drivers/hwmon/pmbus/max20730.c
index be83b98411c7..00fea16acab4 100644
--- a/drivers/hwmon/pmbus/max20730.c
+++ b/drivers/hwmon/pmbus/max20730.c
@@ -328,8 +328,6 @@ static int max20730_init_debugfs(struct i2c_client *client,
return -ENOENT;
max20730_dir = debugfs_create_dir(client->name, debugfs);
- if (!max20730_dir)
- return -ENOENT;
for (i = 0; i < MAX20730_DEBUGFS_NUM_ENTRIES; ++i)
psu->debugfs_entries[i] = i;