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

[GitBox]

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


##########
drivers/rptun/rptun.c:
##########
@@ -462,7 +462,7 @@ static int rptun_notify_wait(FAR struct remoteproc *rproc, 
uint32_t id)
   return 0;
 }
 
-static void *rptun_get_priv_by_rdev(FAR struct rpmsg_device *rdev)
+static FAR void *rptun_get_priv_by_rdev(FAR struct rpmsg_device *rdev)
 {
   struct rpmsg_virtio_device *rvdev;
   struct virtio_device *vdev;

Review Comment:
   ```suggestion
     FAR struct rpmsg_virtio_device *rvdev;
     FAR struct virtio_device *vdev;
   ```



##########
drivers/wireless/ieee80211/bcm43xxx/bcmf_gspi.c:
##########
@@ -337,9 +337,9 @@ static int bcmf_gspi_bus_lowpower(FAR bcmf_gspi_dev_t 
*gbus, bool enable)
  * Name: bcmf_gspi_thread_isr
  ****************************************************************************/
 
-static int bcmf_gspi_thread_isr(int isr, void *context, void *arg)
+static int bcmf_gspi_thread_isr(int isr, FAR void *context, FAR void *arg)
 {
-  FAR bcmf_gspi_dev_t *gbus = (FAR bcmf_gspi_dev_t  *) arg;
+  FAR bcmf_gspi_dev_t *gbus = (FAR bcmf_gspi_dev_t  *)arg;

Review Comment:
   ```suggestion
     FAR bcmf_gspi_dev_t *gbus = (FAR bcmf_gspi_dev_t *)arg;
   ```



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