gustavonihei commented on a change in pull request #4506: URL: https://github.com/apache/incubator-nuttx/pull/4506#discussion_r705528998
########## File path: drivers/lcd/Kconfig ########## @@ -619,15 +625,15 @@ if LCD_ST7789 ---help--- Specifies the Y offset of the LCD. - config LCD_ST7789_BPP + config LCD_ST77XX_BPP Review comment: I'm concerned about having two `config` options with the same identifier. Have you tested enabling both `LCD_ST7735` and `LCD_ST7789`? This is a valid scenario and I believe will probably result in a crash in `menuconfig`. -- 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