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


##########
drivers/wireless/ieee80211/bcm43xxx/bcmf_core.c:
##########
@@ -771,10 +772,10 @@ void bcmf_core_reset(FAR struct bcmf_sdio_dev_s *sbus,
   bcmf_write_sbregw(sbus, base + BCMA_RESET_CTL, 0);
   bcmf_read_sbregw(sbus, base + BCMA_RESET_CTL, &value);
 
-  up_udelay(1);
+  nxsig_usleep(1);

Review Comment:
   In general I support such changes for ms delays. With us delays we need to 
be more careful since `nxsig_usleep` rely on systick and that is usually as ms 
boundary. Another part is that `nxsig_usleep` will work only after scheduler is 
running, but not on the early init. If all the changes are verified with the HW 
and timings are sufficient than I do not have any objections.



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