AcoSail/kernel_optimize_test
8
0
Fork 0
forked from luck/tmp_suning_uos_patched
Code Pull Requests Activity

iio: core: no need to check return value of debugfs_create functions

Browse Source 
When calling debugfs functions, there is no need to ever check the
return value.  The function can work or not, but the code logic should
never do something different based on this.

Cc: Jonathan Cameron <jic23@kernel.org>
Cc: Hartmut Knaack <knaack.h@gmx.de>
Cc: Lars-Peter Clausen <lars@metafoo.de>
Cc: Peter Meerwald-Stadler <pmeerw@pmeerw.net>
Cc: linux-iio@vger.kernel.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
This commit is contained in:
Greg Kroah-Hartman 2019-06-18 17:54:40 +02:00 committed by Jonathan Cameron
parent 1c349f4fd3
commit 8915aacac4
1 changed files with 8 additions and 27 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

35
drivers/iio/industrialio-core.c
View File

@ -369,39 +369,25 @@ static void iio_device_unregister_debugfs(struct iio_dev *indio_dev)
debugfs_remove_recursive(indio_dev->debugfs_dentry); debugfs_remove_recursive(indio_dev->debugfs_dentry);
} }
static int iio_device_register_debugfs(struct iio_dev *indio_dev) static void iio_device_register_debugfs(struct iio_dev *indio_dev)
{ {
struct dentry *d;
if (indio_dev->info->debugfs_reg_access == NULL) if (indio_dev->info->debugfs_reg_access == NULL)
return 0; return;
if (!iio_debugfs_dentry) if (!iio_debugfs_dentry)
return 0; return;
indio_dev->debugfs_dentry = indio_dev->debugfs_dentry =
debugfs_create_dir(dev_name(&indio_dev->dev), debugfs_create_dir(dev_name(&indio_dev->dev),
iio_debugfs_dentry); iio_debugfs_dentry);
if (indio_dev->debugfs_dentry == NULL) {
dev_warn(indio_dev->dev.parent,
"Failed to create debugfs directory\n");
return -EFAULT;
}
d = debugfs_create_file("direct_reg_access", 0644, debugfs_create_file("direct_reg_access", 0644,
indio_dev->debugfs_dentry, indio_dev->debugfs_dentry, indio_dev,
indio_dev, &iio_debugfs_reg_fops); &iio_debugfs_reg_fops);
if (!d) {
iio_device_unregister_debugfs(indio_dev);
return -ENOMEM;
}
return 0;
} }
#else #else
static int iio_device_register_debugfs(struct iio_dev *indio_dev) static void iio_device_register_debugfs(struct iio_dev *indio_dev)
{ {
return 0;
} }
static void iio_device_unregister_debugfs(struct iio_dev *indio_dev) static void iio_device_unregister_debugfs(struct iio_dev *indio_dev)
@ -1674,12 +1660,7 @@ int __iio_device_register(struct iio_dev *indio_dev, struct module *this_mod)
/* configure elements for the chrdev */ /* configure elements for the chrdev */
indio_dev->dev.devt = MKDEV(MAJOR(iio_devt), indio_dev->id); indio_dev->dev.devt = MKDEV(MAJOR(iio_devt), indio_dev->id);
ret = iio_device_register_debugfs(indio_dev); iio_device_register_debugfs(indio_dev);
if (ret) {
dev_err(indio_dev->dev.parent,
"Failed to register debugfs interfaces\n");
return ret;
}
ret = iio_buffer_alloc_sysfs_and_mask(indio_dev); ret = iio_buffer_alloc_sysfs_and_mask(indio_dev);
if (ret) { if (ret) {