diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2020-10-22 10:06:59 +0300 |
|---|---|---|
| committer | Guenter Roeck <linux@roeck-us.net> | 2020-12-03 04:42:22 +0300 |
| commit | 92bc2e1f82afa2e456d7416598ffdf3dbf267835 (patch) | |
| tree | 8aa0af4999cc84fdb491706d2c9f7f8119b2ed54 /drivers/hwmon | |
| parent | 96eca8c97fccd3c02f61a87b0341a079b4096730 (diff) | |
| download | linux-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.c | 2 |
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; |