[riot-notifications] [RIOT-OS/RIOT] cpu/cc2538: Add Kconfig support (#16719)

Leandro Lanzieri notifications at github.com
Fri Aug 27 09:28:37 CEST 2021


@leandrolanzieri commented on this pull request.



> +config MODULE_CC2538_RF_OBS_SIG
+    bool "Output observation signals"
+    help
+        Say y to output different observation signals from the RF Core
+        to GPIO pins. These signals can be useful for debugging of
+        low-level protocols or control of external PA, LNA, or switches.
+
+        For more information check section 9.2.4 of the CC2538 User's Manual.
+
+endif # MODULE_CC2538_RF
+
+config HAVE_CC2538_RF
+    bool
+    select MODULE_CC2538_RF if MODULE_NETDEV_DEFAULT
+    help
+        Indicates that a cc2538 radio is present.

Missing newline

> @@ -0,0 +1,30 @@
+# Copyright (c) 2021 HAW Hamburg
+#
+# This file is subject to the terms and conditions of the GNU Lesser
+# General Public License v2.1. See the file LICENSE in the top level
+# directory for more details.
+#
+
+menuconfig MODULE_CC2538_RF
+    bool "CC2538 IEEE 802.15.4 radio"
+    depends on TEST_KCONFIG
+    depends on CPU_FAM_CC2538
+
+if MODULE_CC2538_RF
+
+config MODULE_CC2538_RF_OBS_SIG

Does this somehow depend on `MODULE_GPIO`?

> +config MODULE_IEEE802154_SECURITY
+    bool "IEEE 802.15.4 security"
+    select MODULE_CRYPTO
+    select MODULE_CIPHER_MODES
+
+config MODULE_IEEE802154_SUBMAC
+    bool "IEEE 802.15.4 submac"
+    select MODULE_XTIMER

Here and in other modules it would be really useful if we could add at least a sentence of documentation.

-- 
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/16719#pullrequestreview-740217970
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210827/8ac408dd/attachment.htm>


More information about the notifications mailing list