anchao commented on code in PR #14923:
URL: https://github.com/apache/nuttx/pull/14923#discussion_r1857784368


##########
drivers/syslog/Kconfig:
##########
@@ -407,5 +407,25 @@ config SYSLOG_REGISTER
        ---help---
                This option will support register the syslog channel 
dynamically.
 
+config SYSLOG_STDOUT

Review Comment:
   1. Neither `printf` nor `syslog` can guarantee the integrity of printing, 
they are have different protection mechanisms in the backend 
implementation(txsem/csection)
   2. The application is an open-source project. It is impossible to change all 
`stdout` to `syslog()`.
   3. This option is a configurable function, and default value is **n**, which 
won't impact anyone.



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to