gustavonihei commented on a change in pull request #3807:
URL: https://github.com/apache/incubator-nuttx/pull/3807#discussion_r642495462
##########
File path: drivers/syslog/Kconfig
##########
@@ -195,7 +190,7 @@ config SYSLOG_CHAR
config RAMLOG_SYSLOG
bool "Use RAMLOG for SYSLOG"
- depends on RAMLOG && !ARCH_SYSLOG
+ depends on RAMLOG
Review comment:
I believe it is better to be explicit with Kconfig default values.
This ways we ensure the consistency of our Kconfig use across the codebase.
Furthermore, it is easier to understand when you see the default field.
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
