[riot-notifications] [RIOT-OS/RIOT] drivers/{disp_dev, touch_dev}: add auto_init_screen as default module when used (#16485)

Leandro Lanzieri notifications at github.com
Thu May 20 11:29:43 CEST 2021


@leandrolanzieri commented on this pull request.



> @@ -1,7 +1,6 @@
 config APPLICATION
     bool
     default y
-    imply MODULE_AUTO_INIT_SCREEN if BOARD_HAS_DISPLAY

I think specifying
```
config MODULE_DISP_DEV
    bool "Display device generic API"
    depends on TEST_KCONFIG
    imply MODULE_AUTO_INIT_SCREEN
```
(and something similar for `MODULE_TOUCH_DEV` would work. If the dependencies of `MODULE_AUTO_INIT_SCREEN` are not there (e.g. `MODULE_AUTO_INIT` has been disabled) then it's not enabled. The user can also explicitly say `CONFIG_MODULE_AUTO_INIT_SCREEN=n` if it wants it off.

Another option is to do
```
config MODULE_AUTO_INIT_SCREEN
    bool "Auto-initialize screens"
    default y if MODULE_TOUCH_DEV || MODULE_DISP_DEV
```

In both cases we need to know that [this](https://docs.zephyrproject.org/latest/guides/kconfig/tips.html#stuck-symbols-in-menuconfig-and-guiconfig) still applies when using menuconfig.


-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/16485#discussion_r635933240
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210520/5520eb17/attachment-0001.htm>


More information about the notifications mailing list