yamt commented on a change in pull request #2347:
URL: https://github.com/apache/incubator-nuttx/pull/2347#discussion_r530844304
##########
File path: fs/spiffs/src/spiffs_core.c
##########
@@ -1534,8 +1534,10 @@ void spiffs_fobj_event(FAR struct spiffs_s *fs,
FAR struct spiffs_file_s *next;
int16_t objid = objid_raw & ~SPIFFS_OBJID_NDXFLAG;
+#ifdef CONFIG_DEBUG_FS_INFO
Review comment:
to match the above variable declaration.
##########
File path: arch/x86_64/src/common/up_assert.c
##########
@@ -250,7 +250,7 @@ void up_assert(const char *filename, int lineno)
board_autoled_on(LED_ASSERTION);
-#if CONFIG_TASK_NAME_SIZE > 0
+#if CONFIG_TASK_NAME_SIZE > 0 && defined(CONFIG_DEBUG_ALERT)
Review comment:
to match the above variable declaration.
##########
File path: drivers/lcd/st7567.c
##########
@@ -729,7 +729,9 @@ static int st7567_getpower(struct lcd_dev_s *dev)
{
struct st7567_dev_s *priv = (struct st7567_dev_s *)dev;
DEBUGASSERT(priv);
- ginfo("powered: %s\n", st7567_powerstring(priv->powered));
+
+ /* ginfo("powered: %s\n", st7567_powerstring(priv->powered)); */
Review comment:
i have no strong opinion either ways.
##########
File path: drivers/lcd/st7567.c
##########
@@ -747,8 +749,10 @@ static int st7567_setpower(struct lcd_dev_s *dev, int
power)
struct st7567_dev_s *priv = (struct st7567_dev_s *)dev;
DEBUGASSERT(priv && (unsigned)power <= CONFIG_LCD_MAXPOWER);
- ginfo("power: %s powered: %s\n",
- st7567_powerstring(power), st7567_powerstring(priv->powered));
+
+ /* ginfo("power: %s powered: %s\n",
Review comment:
i have no strong opinion either ways.
----------------------------------------------------------------
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]
