[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7472: Initialize global mutext/sem by NXMUTEX_INITIALIZER and SEM_INITIALIZER

[GitBox]

xiaoxiang781216 commented on code in PR #7472:
URL: https://github.com/apache/incubator-nuttx/pull/7472#discussion_r1011962768


##########
arch/arm/src/samd5e5/sam_i2c_master.c:
##########
@@ -384,16 +455,17 @@ static const struct i2c_attr_s g_i2c7attr =
   .base      = SAM_SERCOM7_BASE,
 };
 
-static struct sam_i2c_dev_s g_i2c7;
-#endif
-
-struct i2c_ops_s g_i2cops =
+static struct sam_i2c_dev_s g_i2c7 =
 {
-  .transfer = sam_i2c_transfer,
-#ifdef CONFIG_I2C_RESET
-  .reset = sam_i2c_reset,
-#endif
+  .dev       =
+  {
+    .ops     = &g_i2cops,
+  },
+  .attr      = &g_i2c7attr,

Review Comment:
   yes



##########
arch/arm/src/samd5e5/sam_i2c_master.c:
##########
@@ -366,7 +428,16 @@ static const struct i2c_attr_s g_i2c6attr =
   .base      = SAM_SERCOM6_BASE,
 };
 
-static struct sam_i2c_dev_s g_i2c6;
+static struct sam_i2c_dev_s g_i2c6 =
+{
+  .dev       =
+  {
+    .ops     = &g_i2cops,
+  },
+  .attr      = &g_i2c6attr,

Review Comment:
   yes



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org