From 75d19552dd9ced67eaf7c08f74c123a8cf70984a Mon Sep 17 00:00:00 2001 From: MrKevinWeiss Date: Wed, 24 May 2023 09:13:14 +0200 Subject: [PATCH 1/5] kconfig: Fix tabs/space issue --- kconfigs/Kconfig.features | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/kconfigs/Kconfig.features b/kconfigs/Kconfig.features index 80736af39b..557406b58c 100644 --- a/kconfigs/Kconfig.features +++ b/kconfigs/Kconfig.features @@ -191,9 +191,9 @@ config HAS_PERIPH_GPIO_IRQ config HAS_PERIPH_GPIO_FAST_READ bool help - Indicates that the GPIO peripheral supports a mode in which pin read - operations are faster, usually with a tradeoff against a different - property. + Indicates that the GPIO peripheral supports a mode in which pin read + operations are faster, usually with a tradeoff against a different + property. config HAS_PERIPH_GPIO_TAMPER_WAKE bool From adbf951481630863bbe22ac781769546a7872ede Mon Sep 17 00:00:00 2001 From: MrKevinWeiss Date: Wed, 24 May 2023 09:49:21 +0200 Subject: [PATCH 2/5] cpu/rpx0xx: Fix kconfig model --- cpu/rpx0xx/Kconfig | 14 ++++++++++++++ drivers/periph_common/Kconfig | 10 ++++++++++ 2 files changed, 24 insertions(+) diff --git a/cpu/rpx0xx/Kconfig b/cpu/rpx0xx/Kconfig index 27d3872af7..5ef494173c 100644 --- a/cpu/rpx0xx/Kconfig +++ b/cpu/rpx0xx/Kconfig @@ -17,6 +17,10 @@ config CPU_FAM_RPX0XX select HAS_PERIPH_UART_MODECFG select HAS_PERIPH_UART_RECONFIGURE select HAS_PIO_I2C + select MODULE_PIO_I2C if MODULE_PERIPH_I2C + # Since make has a wildcard for anything starting with pio_ we need to + # build it as new modules get added + select MODULE_PERIPH_PIO if MODULE_PIO_I2C config CPU_FAM default "RPX0XX" if CPU_FAM_RPX0XX @@ -36,4 +40,14 @@ config HAS_CPU_RPX0XX help Indicates that a RPX0XX CPU (e.g. the RP2040) is used +config MODULE_PIO_I2C + bool "Enable PIO I2C module" + depends on HAS_PIO_I2C + +config MODULE_PIO_AUTOSTART_I2C + bool "Enable PIO I2C module autostart" + default y if MODULE_PIO_I2C + depends on HAS_PIO_I2C + + source "$(RIOTCPU)/cortexm_common/Kconfig" diff --git a/drivers/periph_common/Kconfig b/drivers/periph_common/Kconfig index ee2df8edf7..5b5c53d64f 100644 --- a/drivers/periph_common/Kconfig +++ b/drivers/periph_common/Kconfig @@ -90,6 +90,16 @@ config MODULE_PERIPH_INIT_HWRNG rsource "Kconfig.i2c" +config MODULE_PERIPH_PIO + bool "Programmable IO (PIO) peripheral driver" + depends on HAS_PERIPH_PIO + select MODULE_PERIPH_COMMON + +config MODULE_PERIPH_INIT_PIO + bool "Auto initialize programmable IO (PIO) peripheral driver" + default y if MODULE_PERIPH_INIT + depends on MODULE_PERIPH_PIO + config MODULE_PERIPH_PM bool "Power Management (PM) peripheral driver" default y From 208f16085b9fd55ba9197c69fd069d59d1f7e49f Mon Sep 17 00:00:00 2001 From: MrKevinWeiss Date: Wed, 24 May 2023 09:51:46 +0200 Subject: [PATCH 3/5] .murdock: Test rpi-pico with kconfig --- .murdock | 1 + 1 file changed, 1 insertion(+) diff --git a/.murdock b/.murdock index d3643ce2a4..194a4bae4e 100755 --- a/.murdock +++ b/.murdock @@ -96,6 +96,7 @@ pba-d-01-kw2x p-nucleo-wb55 qn9080dk remote-revb +rpi-pico same54-xpro samr21-xpro seeedstudio-gd32 From 1dc6e0e32b55f0293575e1ddd59894adf8cba6f3 Mon Sep 17 00:00:00 2001 From: Alexandre Abadie Date: Tue, 23 May 2023 09:44:32 +0200 Subject: [PATCH 4/5] drivers/nrf24l01p: model in Kconfig --- drivers/Kconfig.net | 1 + drivers/nrf24l01p/Kconfig | 20 ++++++++++++++++++++ 2 files changed, 21 insertions(+) create mode 100644 drivers/nrf24l01p/Kconfig diff --git a/drivers/Kconfig.net b/drivers/Kconfig.net index 4b95e6460a..ff194b8fb3 100644 --- a/drivers/Kconfig.net +++ b/drivers/Kconfig.net @@ -27,6 +27,7 @@ rsource "mcp2515/Kconfig" rsource "mrf24j40/Kconfig" rsource "ncv7356/Kconfig" rsource "netdev_ieee802154_submac/Kconfig" +rsource "nrf24l01p/Kconfig" rsource "pn532/Kconfig" rsource "rn2xx3/Kconfig" rsource "$(RIOTCPU)/sam0_common/sam0_eth/Kconfig" diff --git a/drivers/nrf24l01p/Kconfig b/drivers/nrf24l01p/Kconfig new file mode 100644 index 0000000000..ce970122d8 --- /dev/null +++ b/drivers/nrf24l01p/Kconfig @@ -0,0 +1,20 @@ +# Copyright (c) 2023 Inria +# 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_NRF24L01P + bool + prompt "nrf24l01p radio" if !(MODULE_NETDEV_DEFAULT && HAVE_NRF24L01P) + default (MODULE_NETDEV_DEFAULT && HAVE_NRF24L01P) + depends on TEST_KCONFIG + select MODULE_PERIPH_GPIO + select MODULE_PERIPH_GPIO_IRQ + select MODULE_PERIPH_SPI + select ZTIMER_USEC + +config HAVE_NRF24L01P + bool + help + Indicates that a NRF24L01P radio is present. From 4daa40d68c67700fcc77a7cef1de4c58dec21077 Mon Sep 17 00:00:00 2001 From: Alexandre Abadie Date: Tue, 23 May 2023 09:44:52 +0200 Subject: [PATCH 5/5] tests/drivers/nrf24l01p_lowlevel: add Kconfig config --- tests/drivers/nrf24l01p_lowlevel/app.config.test | 6 ++++++ 1 file changed, 6 insertions(+) create mode 100644 tests/drivers/nrf24l01p_lowlevel/app.config.test diff --git a/tests/drivers/nrf24l01p_lowlevel/app.config.test b/tests/drivers/nrf24l01p_lowlevel/app.config.test new file mode 100644 index 0000000000..179b5a6135 --- /dev/null +++ b/tests/drivers/nrf24l01p_lowlevel/app.config.test @@ -0,0 +1,6 @@ +CONFIG_MODULE_NRF24L01P=y +CONFIG_MODULE_PS=y +CONFIG_MODULE_SHELL=y +CONFIG_MODULE_SHELL_CMDS_DEFAULT=y +CONFIG_MODULE_ZTIMER=y +CONFIG_ZTIMER_USEC=y