You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mynewt.apache.org by je...@apache.org on 2022/06/15 19:06:14 UTC

[mynewt-core] 03/03: lsm6dsl: Style check changes

This is an automated email from the ASF dual-hosted git repository.

jerzy pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/mynewt-core.git

commit 73c0725b40b84c5e6ad192dbe70014e59c0a0b03
Author: JustineKH <ju...@juul.com>
AuthorDate: Wed Jun 15 07:04:06 2022 -0700

    lsm6dsl: Style check changes
---
 .../sensors/lsm6dsl/include/lsm6dsl/lsm6dsl.h      |  8 +--
 hw/drivers/sensors/lsm6dsl/src/lsm6dsl.c           | 63 +++++++++++-----------
 hw/drivers/sensors/lsm6dsl/src/lsm6dsl_shell.c     | 12 +++--
 3 files changed, 43 insertions(+), 40 deletions(-)

diff --git a/hw/drivers/sensors/lsm6dsl/include/lsm6dsl/lsm6dsl.h b/hw/drivers/sensors/lsm6dsl/include/lsm6dsl/lsm6dsl.h
index 2281144b4..d5b851cca 100644
--- a/hw/drivers/sensors/lsm6dsl/include/lsm6dsl/lsm6dsl.h
+++ b/hw/drivers/sensors/lsm6dsl/include/lsm6dsl/lsm6dsl.h
@@ -120,10 +120,10 @@ struct lsm6dsl_tilt_settings {
 
 struct lsm6dsl_tap_settings {
     /* Axis enabled bitmask */
-    uint8_t en_x: 1;
-    uint8_t en_y: 1;
-    uint8_t en_z: 1;
-    uint8_t en_dtap: 1;
+    uint8_t en_x : 1;
+    uint8_t en_y : 1;
+    uint8_t en_z : 1;
+    uint8_t en_dtap : 1;
 
     /* Threshold for tap recognition */
     int8_t tap_ths;
diff --git a/hw/drivers/sensors/lsm6dsl/src/lsm6dsl.c b/hw/drivers/sensors/lsm6dsl/src/lsm6dsl.c
index 0783ffb21..68496750c 100644
--- a/hw/drivers/sensors/lsm6dsl/src/lsm6dsl.c
+++ b/hw/drivers/sensors/lsm6dsl/src/lsm6dsl.c
@@ -1028,7 +1028,7 @@ lsm6dsl_get_wake_up(struct lsm6dsl *lsm6dsl, struct lsm6dsl_wk_settings *wk)
 
 int
 lsm6dsl_set_tilt(struct lsm6dsl *lsm6dsl,
-                        const struct lsm6dsl_tilt_settings *cfg)
+                 const struct lsm6dsl_tilt_settings *cfg)
 {
     int rc;
     uint8_t en_mask;
@@ -1041,7 +1041,8 @@ lsm6dsl_set_tilt(struct lsm6dsl *lsm6dsl,
             return rc;
         }
         if (cfg->en_wrist_tilt) {
-            rc = lsm6dsl_write_reg(lsm6dsl, LSM6DSL_FUNC_CFG_ACCESS_REG, LSM6DSL_FUNC_CFG_ACCESS_MASK | LSM6DSL_SHUB_REG_ACCESS_MASK);
+            rc = lsm6dsl_write_reg(lsm6dsl, LSM6DSL_FUNC_CFG_ACCESS_REG,
+                                   LSM6DSL_FUNC_CFG_ACCESS_MASK | LSM6DSL_SHUB_REG_ACCESS_MASK);
             if (rc) {
                 return rc;
             }
@@ -2213,35 +2214,35 @@ lsm6dsl_inc_notif_stats(sensor_event_type_t event)
 {
 #if MYNEWT_VAL(LSM6DSL_NOTIF_STATS)
     switch (event) {
-        case SENSOR_EVENT_TYPE_SINGLE_TAP:
-            STATS_INC(g_lsm6dsl_stats, single_tap_notify);
-            break;
-        case SENSOR_EVENT_TYPE_DOUBLE_TAP:
-            STATS_INC(g_lsm6dsl_stats, double_tap_notify);
-            break;
-        case SENSOR_EVENT_TYPE_ORIENT_CHANGE:
-            STATS_INC(g_lsm6dsl_stats, orientation_notify);
-            break;
-        case SENSOR_EVENT_TYPE_SLEEP:
-            STATS_INC(g_lsm6dsl_stats, sleep_notify);
-            break;
-        case SENSOR_EVENT_TYPE_WAKEUP:
-            STATS_INC(g_lsm6dsl_stats, wakeup_notify);
-            break;
-        case SENSOR_EVENT_TYPE_FREE_FALL:
-            STATS_INC(g_lsm6dsl_stats, free_fall_notify);
-            break;
-        case SENSOR_EVENT_TYPE_TILT_CHANGE:
-            STATS_INC(g_lsm6dsl_stats, rel_tilt_notify);
-            break;
-        case SENSOR_EVENT_TYPE_TILT_POS:
-            STATS_INC(g_lsm6dsl_stats, abs_tilt_pos_notify);
-            break;
-        case SENSOR_EVENT_TYPE_TILT_NEG:
-            STATS_INC(g_lsm6dsl_stats, abs_tilt_neg_notify);
-            break;
-        default:
-            break;
+    case SENSOR_EVENT_TYPE_SINGLE_TAP:
+        STATS_INC(g_lsm6dsl_stats, single_tap_notify);
+        break;
+    case SENSOR_EVENT_TYPE_DOUBLE_TAP:
+        STATS_INC(g_lsm6dsl_stats, double_tap_notify);
+        break;
+    case SENSOR_EVENT_TYPE_ORIENT_CHANGE:
+        STATS_INC(g_lsm6dsl_stats, orientation_notify);
+        break;
+    case SENSOR_EVENT_TYPE_SLEEP:
+        STATS_INC(g_lsm6dsl_stats, sleep_notify);
+        break;
+    case SENSOR_EVENT_TYPE_WAKEUP:
+        STATS_INC(g_lsm6dsl_stats, wakeup_notify);
+        break;
+    case SENSOR_EVENT_TYPE_FREE_FALL:
+        STATS_INC(g_lsm6dsl_stats, free_fall_notify);
+        break;
+    case SENSOR_EVENT_TYPE_TILT_CHANGE:
+        STATS_INC(g_lsm6dsl_stats, rel_tilt_notify);
+        break;
+    case SENSOR_EVENT_TYPE_TILT_POS:
+        STATS_INC(g_lsm6dsl_stats, abs_tilt_pos_notify);
+        break;
+    case SENSOR_EVENT_TYPE_TILT_NEG:
+        STATS_INC(g_lsm6dsl_stats, abs_tilt_neg_notify);
+        break;
+    default:
+        break;
     }
 #endif /* LSM6DSL_NOTIF_STATS */
 }
diff --git a/hw/drivers/sensors/lsm6dsl/src/lsm6dsl_shell.c b/hw/drivers/sensors/lsm6dsl/src/lsm6dsl_shell.c
index d34123cd6..fa85cbf28 100644
--- a/hw/drivers/sensors/lsm6dsl/src/lsm6dsl_shell.c
+++ b/hw/drivers/sensors/lsm6dsl/src/lsm6dsl_shell.c
@@ -182,14 +182,15 @@ lsm6dsl_shell_cmd_dump(int argc, char **argv)
             rc = lsm6dsl_read(g_lsm6dsl, reg_name[i].addr, &value, 1);
             if (rc) {
                 console_printf("dump failed %d\n", rc);
-            } else if (all || value != 0){
+            } else if (all || value != 0) {
                 console_printf("%-22s(0x%02X) = 0x%02X\n",
                                reg_name[i].regname, reg_name[i].addr, value);
             }
         }
 
         /* Bank A */
-        rc = lsm6dsl_write_reg(g_lsm6dsl, LSM6DSL_FUNC_CFG_ACCESS_REG, LSM6DSL_FUNC_CFG_ACCESS_MASK);
+        rc = lsm6dsl_write_reg(g_lsm6dsl, LSM6DSL_FUNC_CFG_ACCESS_REG,
+                               LSM6DSL_FUNC_CFG_ACCESS_MASK);
         if (rc) {
             return rc;
         }
@@ -197,7 +198,7 @@ lsm6dsl_shell_cmd_dump(int argc, char **argv)
             rc = lsm6dsl_read(g_lsm6dsl, reg_name_banka[i].addr, &value, 1);
             if (rc) {
                 console_printf("dump failed %d\n", rc);
-            } else if (all || value != 0){
+            } else if (all || value != 0) {
                 console_printf("%-22s(0x%02X) = 0x%02X\n",
                                reg_name_banka[i].regname, reg_name_banka[i].addr, value);
             }
@@ -208,7 +209,8 @@ lsm6dsl_shell_cmd_dump(int argc, char **argv)
         }
 
         /* Bank B */
-        rc = lsm6dsl_write_reg(g_lsm6dsl, LSM6DSL_FUNC_CFG_ACCESS_REG, LSM6DSL_FUNC_CFG_ACCESS_MASK | LSM6DSL_SHUB_REG_ACCESS_MASK);
+        rc = lsm6dsl_write_reg(g_lsm6dsl, LSM6DSL_FUNC_CFG_ACCESS_REG,
+                               LSM6DSL_FUNC_CFG_ACCESS_MASK | LSM6DSL_SHUB_REG_ACCESS_MASK);
         if (rc) {
             return rc;
         }
@@ -216,7 +218,7 @@ lsm6dsl_shell_cmd_dump(int argc, char **argv)
             rc = lsm6dsl_read(g_lsm6dsl, reg_name_bankb[i].addr, &value, 1);
             if (rc) {
                 console_printf("dump failed %d\n", rc);
-            } else if (all || value != 0){
+            } else if (all || value != 0) {
                 console_printf("%-22s(0x%02X) = 0x%02X\n",
                                reg_name_bankb[i].regname, reg_name_bankb[i].addr, value);
             }