IIO: industrialio-core: Checking NULL instead of IS_ERR for debugfs_create_dir()
authorAxel Lin <axel.lin@gmail.com>
Thu, 3 May 2012 14:56:58 +0000 (22:56 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 8 May 2012 23:04:11 +0000 (16:04 -0700)
If defined CONFIG_DEBUG_FS, debugfs_create_dir returns NULL on failure.

Signed-off-by: Axel Lin <axel.lin@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/iio/industrialio-core.c

index 52aa44a8e2091272fa1d0714be1966728691db6d..b39a58757907407ad191c3cf60a52d660c01588c 100644 (file)
@@ -227,15 +227,12 @@ static int iio_device_register_debugfs(struct iio_dev *indio_dev)
        if (indio_dev->info->debugfs_reg_access == NULL)
                return 0;
 
-       if (IS_ERR(iio_debugfs_dentry))
+       if (!iio_debugfs_dentry)
                return 0;
 
        indio_dev->debugfs_dentry =
                debugfs_create_dir(dev_name(&indio_dev->dev),
                                   iio_debugfs_dentry);
-       if (IS_ERR(indio_dev->debugfs_dentry))
-               return PTR_ERR(indio_dev->debugfs_dentry);
-
        if (indio_dev->debugfs_dentry == NULL) {
                dev_warn(indio_dev->dev.parent,
                         "Failed to create debugfs directory\n");