xiaoxiang781216 commented on code in PR #5070: URL: https://github.com/apache/incubator-nuttx/pull/5070#discussion_r999647622
########## arch/arm/src/kinetis/kinetis_i2c.c: ########## @@ -191,7 +191,7 @@ static struct kinetis_i2cdev_s g_i2c0_dev = .config = &kinetis_i2c0_config, .refs = 0, .lock = NXMUTEX_INITIALIZER, - .wait = NXSEM_INITIALIZER(0, PRIOINHERIT_FLAGS_DISABLE), + .wait = SEM_INITIALIZER(0), Review Comment: But, it isn't a issue to use a macro or function which doesn't start with NX or nx prefix, if the the function/macro doesn't touch errno or other per task data(e.g. file handle). -- 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