tools: iio: iio_generic_buffer: rename and change type of force variable
authorEva Rachel Retuya <eraretuya@gmail.com>
Thu, 20 Oct 2016 09:19:23 +0000 (17:19 +0800)
committerJonathan Cameron <jic23@kernel.org>
Sun, 23 Oct 2016 18:34:10 +0000 (19:34 +0100)
Replace the type of 'force' flag from int to bool and at the same time
rename it to 'force_autochannels' for better readability.

Suggested-by: Peter Meerwald-Stadler <pmeerw@pmeerw.net>
Signed-off-by: Eva Rachel Retuya <eraretuya@gmail.com>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
tools/iio/iio_generic_buffer.c

index 31ddd3abe04fdd4009915c0465367f6a93794c33..8ef39ae9858f7f66ac781b4573133a687194c3bf 100644 (file)
@@ -348,7 +348,7 @@ int main(int argc, char **argv)
        int noevents = 0;
        int notrigger = 0;
        char *dummy;
-       int force = 0;
+       bool force_autochannels = false;
 
        struct iio_channel_info *channels = NULL;
 
@@ -362,7 +362,7 @@ int main(int argc, char **argv)
                        break;
                case 'A':
                        autochannels = AUTOCHANNELS_ENABLED;
-                       force = 1;
+                       force_autochannels = true;
                        break;  
                case 'c':
                        errno = 0;
@@ -526,14 +526,15 @@ int main(int argc, char **argv)
                        "diag %s\n", dev_dir_name);
                goto error;
        }
-       if ((num_channels && autochannels == AUTOCHANNELS_ENABLED) && !force) {
+       if ((num_channels && autochannels == AUTOCHANNELS_ENABLED) &&
+           !force_autochannels) {
                fprintf(stderr, "Auto-channels selected but some channels "
                        "are already activated in sysfs\n");
                fprintf(stderr, "Proceeding without activating any channels\n");
        }
 
        if ((!num_channels && autochannels == AUTOCHANNELS_ENABLED) ||
-           ((autochannels == AUTOCHANNELS_ENABLED) && force)) {
+           ((autochannels == AUTOCHANNELS_ENABLED) && force_autochannels)) {
                fprintf(stderr, "Enabling all channels\n");
 
                ret = enable_disable_all_channels(dev_dir_name, 1);