Message ID | 20230524165423.21024-1-osmtendev@gmail.com (mailing list archive) |
---|---|
State | Changes Requested |
Headers | show |
Series | adm1266.c: Fix error checking for debugfs_create_dir | expand |
On Wed, May 24, 2023 at 09:54:23PM +0500, Osama Muhammad wrote: > This patch fixes the error checking in adm1266.c in > debugfs_create_dir. The correct way to check if an error occurred > is using 'IS_ERR' inline function. > > Signed-off-by: Osama Muhammad <osmtendev@gmail.com> > --- > drivers/hwmon/pmbus/adm1266.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/hwmon/pmbus/adm1266.c b/drivers/hwmon/pmbus/adm1266.c > index 1ac2b2f4c570..d88b6115b42e 100644 > --- a/drivers/hwmon/pmbus/adm1266.c > +++ b/drivers/hwmon/pmbus/adm1266.c > @@ -340,7 +340,7 @@ static void adm1266_init_debugfs(struct adm1266_data *data) > return; > > data->debugfs_dir = debugfs_create_dir(data->client->name, root); > - if (!data->debugfs_dir) > + if (IS_ERR(data->debugfs_dir)) > return; Just drop the error check entirely instead. It is not needed. Thanks, Guenter > > debugfs_create_devm_seqfile(&data->client->dev, "sequencer_state", data->debugfs_dir,
diff --git a/drivers/hwmon/pmbus/adm1266.c b/drivers/hwmon/pmbus/adm1266.c index 1ac2b2f4c570..d88b6115b42e 100644 --- a/drivers/hwmon/pmbus/adm1266.c +++ b/drivers/hwmon/pmbus/adm1266.c @@ -340,7 +340,7 @@ static void adm1266_init_debugfs(struct adm1266_data *data) return; data->debugfs_dir = debugfs_create_dir(data->client->name, root); - if (!data->debugfs_dir) + if (IS_ERR(data->debugfs_dir)) return; debugfs_create_devm_seqfile(&data->client->dev, "sequencer_state", data->debugfs_dir,
This patch fixes the error checking in adm1266.c in debugfs_create_dir. The correct way to check if an error occurred is using 'IS_ERR' inline function. Signed-off-by: Osama Muhammad <osmtendev@gmail.com> --- drivers/hwmon/pmbus/adm1266.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)