Merge pull request #11731 from benemorius/tests-periph_pm-ux

tests/periph_pm: make usage more intuitive and move shell commands to sys/

sys/include/pm_layered.h

@@ -49,6 +49,14 @@ extern "C" {
 #define PROVIDES_PM_SET_LOWEST
 #endif
 
+/**
+ * @brief Power Management mode blocker typedef
+ */
+typedef union {
+    uint32_t val_u32;                   /**< power mode blockers u32 */
+    uint8_t val_u8[PM_NUM_MODES];       /**< power mode blockers u8 */
+} pm_blocker_t;
+
 /**
  * @brief   Block a power mode
  *

sys/pm_layered/pm.c

@@ -33,14 +33,6 @@
 #define PM_BLOCKER_INITIAL 0x01010101   /* block all by default */
 #endif
 
-/**
- * @brief Power Management mode typedef
- */
-typedef union {
-    uint32_t val_u32;
-    uint8_t val_u8[PM_NUM_MODES];
-} pm_blocker_t;
-
 /**
  * @brief Global variable for keeping track of blocked modes
  */

sys/shell/commands/Makefile

@@ -8,6 +8,9 @@ endif
 ifneq (,$(filter mci,$(USEMODULE)))
   SRC += sc_disk.c
 endif
+ifneq (,$(filter periph_pm,$(USEMODULE)))
+  SRC += sc_pm.c
+endif
 ifneq (,$(filter ps,$(USEMODULE)))
   SRC += sc_ps.c
 endif

sys/shell/commands/sc_pm.c

@@ -0,0 +1,194 @@
+/*
+ * Copyright (C) 2019 Thomas Stilwell <[email protected]>
+ *
+ * This file is subject to the terms and conditions of the GNU Lesser
+ * General Public License v2.1. See the file LICENSE in the top level
+ * directory for more details.
+ */
+
+/**
+ * @ingroup     sys_shell_commands
+ * @{
+ *
+ * @file
+ * @brief       Shell command to interact with the PM subsystem
+ *
+ * @author      Bas Stottelaar <[email protected]>
+ * @author      Vincent Dupont <[email protected]>
+ * @author      Thomas Stilwell <[email protected]>
+ *
+ * @}
+ */
+
+#include <stdio.h>
+#include <stdlib.h>
+#include <string.h>
+#include "periph/pm.h"
+
+#ifdef MODULE_PM_LAYERED
+#include "pm_layered.h"
+
+extern volatile pm_blocker_t pm_blocker; /* sys/pm_layered/pm.c */
+#endif /* MODULE_PM_LAYERED */
+
+static void _print_usage(void) {
+    puts("Usage:");
+#ifdef MODULE_PM_LAYERED
+    puts("\tpm show: display current blockers for each power mode");
+    puts("\tpm set <mode>: manually set power mode (lasts until WFI returns)");
+    puts("\tpm block <mode>: manually block power mode");
+    puts("\tpm unblock <mode>: manually unblock power mode");
+#endif /* MODULE_PM_LAYERED */
+    puts("\tpm off: call pm_off()");
+}
+
+#ifdef MODULE_PM_LAYERED
+static int check_mode(int argc, char **argv)
+{
+    if (argc != 3) {
+        printf("Usage: %s %s <power mode>\n", argv[0], argv[1]);
+        return -1;
+    }
+
+    return 0;
+}
+
+static int parse_mode(char *argv)
+{
+    uint8_t mode = atoi(argv);
+
+    if (mode >= PM_NUM_MODES) {
+        printf("Error: power mode not in range 0 - %d.\n", PM_NUM_MODES - 1);
+        return -1;
+    }
+
+    return mode;
+}
+
+static int cmd_block(char *arg)
+{
+    int mode = parse_mode(arg);
+    if (mode < 0) {
+        return 1;
+    }
+
+    printf("Blocking power mode %d.\n", mode);
+    fflush(stdout);
+
+    pm_block(mode);
+
+    return 0;
+}
+
+static int cmd_set(char *arg)
+{
+    int mode = parse_mode(arg);
+    if (mode < 0) {
+        return 1;
+    }
+
+    printf("CPU is entering power mode %d.\n", mode);
+    puts("Now waiting for a wakeup event...");
+    fflush(stdout);
+
+    pm_set(mode);
+    /* execution stops here until anything (like shell input) wakes the CPU */
+
+    printf("CPU has returned from power mode %d.\n", mode);
+
+    return 0;
+}
+
+static int cmd_unblock(char *arg)
+{
+    int mode = parse_mode(arg);
+
+    if (mode < 0) {
+        return 1;
+    }
+
+    if (pm_blocker.val_u8[mode] == 0) {
+        printf("Mode %d is already unblocked.\n", mode);
+        return 1;
+    }
+
+    printf("Unblocking power mode %d.\n", mode);
+    fflush(stdout);
+
+    pm_unblock(mode);
+
+    return 0;
+}
+
+static int cmd_show(char *arg)
+{
+    (void)arg;
+    uint8_t lowest_allowed_mode = 0;
+
+    for (unsigned i = 0; i < PM_NUM_MODES; i++) {
+        printf("mode %u blockers: %u \n", i, pm_blocker.val_u8[i]);
+        if (pm_blocker.val_u8[i]) {
+            lowest_allowed_mode = i + 1;
+        }
+    }
+
+    printf("Lowest allowed mode: %u\n", lowest_allowed_mode);
+    return 0;
+}
+#endif /* MODULE_PM_LAYERED */
+
+static int cmd_off(char *arg)
+{
+    (void)arg;
+
+    pm_off();
+
+    return 0;
+}
+
+int _pm_handler(int argc, char **argv)
+{
+#ifdef MODULE_PM_LAYERED
+    if (!strcmp(argv[1], "show")) {
+        if (argc != 2) {
+            puts("usage: pm show: display current blockers for each power mode");
+            return 1;
+        }
+
+        return cmd_show(argv[1]);
+    }
+
+    if (!strcmp(argv[1], "block")) {
+        if (check_mode(argc, argv) != 0) {
+            return 1;
+        }
+
+        return cmd_block(argv[2]);
+    }
+
+    if (!strcmp(argv[1], "unblock")) {
+        if (check_mode(argc, argv) != 0) {
+            return 1;
+        }
+
+        return cmd_unblock(argv[2]);
+    }
+
+    if (!strcmp(argv[1], "set")) {
+        if (check_mode(argc, argv) != 0) {
+            return 1;
+        }
+
+        return cmd_set(argv[2]);
+    }
+#else
+    (void)argc;
+#endif /* MODULE_PM_LAYERED */
+
+    if (!strcmp(argv[1], "off")) {
+        return cmd_off(NULL);
+    }
+
+    _print_usage();
+    return 1;
+}

sys/shell/commands/shell_commands.c

@@ -34,6 +34,10 @@ extern int _id_handler(int argc, char **argv);
 extern int _heap_handler(int argc, char **argv);
 #endif
 
+#ifdef MODULE_PERIPH_PM
+extern int _pm_handler(int argc, char **argv);
+#endif
+
 #ifdef MODULE_PS
 extern int _ps_handler(int argc, char **argv);
 #endif
@@ -180,6 +184,9 @@ const shell_command_t _shell_command_list[] = {
 #ifdef MODULE_HEAP_CMD
     {"heap", "Prints heap statistics.", _heap_handler},
 #endif
+#ifdef MODULE_PERIPH_PM
+    { "pm", "interact with layered PM subsystem", _pm_handler },
+#endif
 #ifdef MODULE_PS
     {"ps", "Prints information about running threads.", _ps_handler},
 #endif

tests/driver_sx127x/Makefile.ci

@@ -5,5 +5,7 @@ BOARD_INSUFFICIENT_MEMORY := \
     arduino-uno \
     atmega328p \
     nucleo-f031k6 \
+    nucleo-f042k6 \
+    nucleo-l031k6 \
     stm32f030f4-demo \
     #

tests/gnrc_udp/Makefile.ci

@@ -7,6 +7,8 @@ BOARD_INSUFFICIENT_MEMORY := \
     arduino-uno \
     atmega1284p \
     atmega328p \
+    blackpill \
+    bluepill \
     calliope-mini \
     chronos \
     derfmega128 \

tests/lwip/Makefile.ci

@@ -11,6 +11,8 @@ BOARD_INSUFFICIENT_MEMORY := \
     nucleo-f334r8 \
     nucleo-l031k6 \
     nucleo-l053r8 \
+    saml10-xpro \
+    saml11-xpro \
     stm32f030f4-demo \
     stm32f0discovery \
     stm32l0538-disco \

tests/periph_pm/Makefile

@@ -8,5 +8,6 @@ FEATURES_OPTIONAL += periph_rtc
 FEATURES_OPTIONAL += periph_gpio_irq
 
 USEMODULE += shell
+USEMODULE += shell_commands
 
 include $(RIOTBASE)/Makefile.include