[riot-notifications] [RIOT-OS/RIOT] drivers/mtd: fix Kconfig dependencies (#16836)

Leandro Lanzieri notifications at github.com
Mon Sep 13 10:41:44 CEST 2021


@leandrolanzieri commented on this pull request.



> @@ -0,0 +1,7 @@
+# this file enables modules defined in Kconfig. Do not use this file for
+# application configuration. This is only needed during migration.
+CONFIG_MODULE_MTD=y
+CONFIG_MODULE_MTD_WRITE_PAGE=y
+CONFIG_MODULE_OD=y
+CONFIG_MODULE_SHELL=y
+CONFIG_MODULE_SHELL=y

Duplicated

> @@ -5,8 +5,8 @@
 # directory for more details.
 #
 
-config MODULE_MTD_NATIVE
+config HAVE_MTD_NATIVE

We could group all these MTD feature symbols under the single mtd kconfig file, as you're doing with the modules. Also, now that we are at it, it could make sense to add the symbols for the other backends.

-- 
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/16836#pullrequestreview-752459850
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210913/24917730/attachment.htm>


More information about the notifications mailing list