tools:iio: return values directly

Return directly, if no common cleanup is required.

Signed-off-by: Hartmut Knaack <knaack.h@gmx.de>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>

3 files changed, 16 insertions, 21 deletions

diff --git a/tools/iio/generic_buffer.c b/tools/iio/generic_buffer.c
index cf9a4120204f..62ec398922a0 100644
--- a/tools/iio/generic_buffer.c
+++ b/tools/iio/generic_buffer.c
@@ -238,8 +238,7 @@ int main(int argc, char **argv)
         dev_num = find_type_by_name(device_name, "iio:device");
         if (dev_num < 0) {
                 printf("Failed to find the %s\n", device_name);
-                ret = dev_num;
-                goto error_ret;
+                return dev_num;
         }
         printf("iio device number being used is %d\n", dev_num);
 
@@ -410,6 +409,6 @@ error_free_triggername:
                 free(trigger_name);
 error_free_dev_dir_name:
         free(dev_dir_name);
-error_ret:
+
         return ret;
 }
diff --git a/tools/iio/iio_event_monitor.c b/tools/iio/iio_event_monitor.c
index 1316527f3c3c..7f56238b1d66 100644
--- a/tools/iio/iio_event_monitor.c
+++ b/tools/iio/iio_event_monitor.c
@@ -258,8 +258,7 @@ int main(int argc, char **argv)
                         device_name, dev_num);
                 ret = asprintf(&chrdev_name, "/dev/iio:device%d", dev_num);
                 if (ret < 0) {
-                        ret = -ENOMEM;
-                        goto error_ret;
+                        return -ENOMEM;
                 }
         } else {
                 /* If we can't find a IIO device by name assume device_name is a
@@ -312,6 +311,6 @@ int main(int argc, char **argv)
 
 error_free_chrdev_name:
         free(chrdev_name);
-error_ret:
+
         return ret;
 }
diff --git a/tools/iio/iio_utils.c b/tools/iio/iio_utils.c
index 5d5831d7dfd0..0524725279bb 100644
--- a/tools/iio/iio_utils.c
+++ b/tools/iio/iio_utils.c
@@ -107,10 +107,9 @@ int iioutils_get_type(unsigned *is_signed,
         const struct dirent *ent;
 
         ret = asprintf(&scan_el_dir, FORMAT_SCAN_ELEMENTS_DIR, device_dir);
-        if (ret < 0) {
-                ret = -ENOMEM;
-                goto error_ret;
-        }
+        if (ret < 0)
+                return -ENOMEM;
+
         ret = asprintf(&builtname, FORMAT_TYPE_FILE, name);
         if (ret < 0) {
                 ret = -ENOMEM;
@@ -202,7 +201,7 @@ error_free_builtname:
         free(builtname);
 error_free_scan_el_dir:
         free(scan_el_dir);
-error_ret:
+
         return ret;
 }
 
@@ -230,10 +229,9 @@ int iioutils_get_param_float(float *output,
         const struct dirent *ent;
 
         ret = asprintf(&builtname, "%s_%s", name, param_name);
-        if (ret < 0) {
-                ret = -ENOMEM;
-                goto error_ret;
-        }
+        if (ret < 0)
+                return -ENOMEM;
+
         ret = asprintf(&builtname_generic,
                        "%s_%s", generic_name, param_name);
         if (ret < 0) {
@@ -277,7 +275,7 @@ error_free_builtname_generic:
         free(builtname_generic);
 error_free_builtname:
         free(builtname);
-error_ret:
+
         return ret;
 }
 
@@ -326,10 +324,9 @@ int build_channel_array(const char *device_dir,
 
         *counter = 0;
         ret = asprintf(&scan_el_dir, FORMAT_SCAN_ELEMENTS_DIR, device_dir);
-        if (ret < 0) {
-                ret = -ENOMEM;
-                goto error_ret;
-        }
+        if (ret < 0)
+                return -ENOMEM;
+
         dp = opendir(scan_el_dir);
         if (dp == NULL) {
                 ret = -errno;
@@ -526,7 +523,7 @@ error_close_dir:
 
 error_free_name:
         free(scan_el_dir);
-error_ret:
+
         return ret;
 }