tools: add minutes as a new supported time unit
authorBartosz Golaszewski <bartosz.golaszewski@linaro.org>
Tue, 23 Apr 2024 10:04:52 +0000 (12:04 +0200)
committerBartosz Golaszewski <bartosz.golaszewski@linaro.org>
Wed, 24 Apr 2024 07:55:36 +0000 (09:55 +0200)
Make it more convenient to specify longer time periods in gpio-tools by
introducing minutes as the new time unit.

Link: https://lore.kernel.org/r/20240423100452.32958-5-brgl@bgdev.pl
Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
tools/tools-common.c

index 500e9a2b4202652ed1e9e437ad0963f113364b3e..4340bce36818ffdb16a24765f0048ea9caae0ed3 100644 (file)
@@ -138,10 +138,12 @@ long long parse_period(const char *option)
        }
 
        if (m) {
-               if (*end != 's')
+               if (*end == '\0')
+                       m = 60000000;
+               else if (*end == 's')
+                       end++;
+               else
                        return -1;
-
-               end++;
        } else {
                m = 1000;
        }
@@ -213,7 +215,7 @@ void print_period_help(void)
 {
        printf("\nPeriods:\n");
        printf("    Periods are taken as milliseconds unless units are specified. e.g. 10us.\n");
-       printf("    Supported units are 's', 'ms', and 'us'.\n");
+       printf("    Supported units are 'm', 's', 'ms', and 'us' for minutes, seconds, milliseconds and microseconds respectively.\n");
 }
 
 #define TIME_BUFFER_SIZE 20