[riot-notifications] [RIOT-OS/RIOT] cpu/nrf52: initial kconfig modeling (no netif) (#16837)

MrKevinWeiss notifications at github.com
Mon Sep 20 14:08:11 CEST 2021


@MrKevinWeiss commented on this pull request.



> +choice
+    bool "Backend"
+    depends on MODULE_NETDEV_DEFAULT
+    default NRF802154
+    default NRFBLE
+
+config NRF802154
+    bool "nrf802154"
+    select MODULE_NRF802154
+
+config NRFBLE
+    bool "nrfble"
+    select MODULE_NRFBLE
+
+config NRFMIN
+    bool "nrfmin"
+    select MODULE_NRFMIN
+
+endchoice

I started #16845 with the HAVE_* but think maybe that is not the best way to move forward.  I think the stage of discussion we are at is:

The boards/cpu  can select things based on other modules enabled (in my case `STDIO`, in your case `NETDEV_DEFAULT`) if selecting is simple and don't have many dependencies or so many options.
The `HAVE_*` is like `FEATURES_PROVIDED` for the features that are not provided in the makefile.

I am still pretty murky of the issues that can arise so it may be good to get together and work through the corner cases.

-- 
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/16837#discussion_r712105637
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210920/8f245c21/attachment.htm>


More information about the notifications mailing list