linguini1 commented on code in PR #18505:
URL: https://github.com/apache/nuttx/pull/18505#discussion_r2901642987
##########
drivers/lcd/Kconfig:
##########
@@ -831,10 +831,153 @@ config LCD_ST7796
bool "ST7796 LCD Display"
default n
depends on SPI
- depends on LCD_FRAMEBUFFER
+ select LCD_FRAMEBUFFER
Review Comment:
In general NuttX prefers "depends on" in almost all cases because select
doesn't recursively enable dependencies (biggest issue with kconfig). So this
keyword makes it easier to prevent breakages :)
--
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]
