[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #6965: nxmutex replace nxsem when used as a lock

Thu, 06 Oct 2022 09:01:18 -0700 


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


##########
drivers/sensors/mlx90393.c:
##########
@@ -499,9 +499,9 @@ static ssize_t mlx90393_read(FAR struct file *filep, FAR 
char *buffer,
   data->z_mag = priv->data.z_mag;
   data->temperature = priv->data.temperature;
 
-  /* Give back the semaphore */
+  /* Give back the mutex */
 
-  nxsem_post(&priv->datasem);
+  nxmutex_unlock(&priv->datalock);
 
   return sizeof(FAR struct mlx90393_sensor_data_s);

Review Comment:
   Done



-- 
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

Reply via email to