Skip to content

modules: hal_nordic: nrfs: Disabling subscription feature in the temp… #93142

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Aug 21, 2025
Merged

modules: hal_nordic: nrfs: Disabling subscription feature in the temp… #93142

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions drivers/sensor/nordic/temp/Kconfig
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,11 +28,11 @@ config TEMP_NRFS
Enable driver for nRF temperature sensor accessed through the nRF
Services (nrfs) layer.

if TEMP_NRFS
if TEMP_NRFS && NRFS_TEMP_SERVICE_SUBSCRIPTION_ENABLED

module = TEMP_NRFS
thread_priority = 10
thread_stack_size = 1024
source "drivers/sensor/Kconfig.trigger_template"

endif # TEMP_NRFS
endif
8 changes: 8 additions & 0 deletions modules/hal_nordic/nrfs/Kconfig
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,9 @@ config NRFS_HAS_TEMP_SERVICE
config NRFS_HAS_VBUS_DETECTOR_SERVICE
bool

config NRFS_TEMP_SERVICE_HAS_SUBSCRIPTION_SERVICE
bool

config NRFS
bool "nRF Services Support"
select NRFS_LOCAL_DOMAIN if SOC_NRF54H20_CPUAPP || \
Expand Down Expand Up @@ -93,6 +96,11 @@ config NRFS_TEMP_SERVICE_ENABLED
depends on NRFS_HAS_TEMP_SERVICE
default y

config NRFS_TEMP_SERVICE_SUBSCRIPTION_ENABLED
bool "Subscription feature for the temperature service"
depends on NRFS_TEMP_SERVICE_HAS_SUBSCRIPTION_SERVICE
default y

config NRFS_VBUS_DETECTOR_SERVICE_ENABLED
bool "VBUS detector for the USB peripheral"
depends on NRFS_HAS_VBUS_DETECTOR_SERVICE
Expand Down
4 changes: 4 additions & 0 deletions modules/hal_nordic/nrfs/nrfs_config.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,10 @@
#define NRFS_TEMP_SERVICE_ENABLED
#endif

#ifdef CONFIG_NRFS_TEMP_SERVICE_SUBSCRIPTION_ENABLED
#define NRFS_TEMP_SERVICE_SUBSCRIPTION_ENABLED
#endif

#ifdef CONFIG_NRFS_MRAM_SERVICE_ENABLED
#define NRFS_MRAM_SERVICE_ENABLED
#endif
Expand Down
2 changes: 2 additions & 0 deletions soc/nordic/nrf92/Kconfig
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,7 @@ config SOC_NRF9230_ENGB_CPUAPP
select NRFS_HAS_PMIC_SERVICE
select NRFS_HAS_TEMP_SERVICE
select NRFS_HAS_VBUS_DETECTOR_SERVICE
select NRFS_TEMP_SERVICE_HAS_SUBSCRIPTION_SERVICE

config SOC_NRF9230_ENGB_CPURAD
select ARM
Expand All @@ -46,6 +47,7 @@ config SOC_NRF9230_ENGB_CPURAD
select NRFS_HAS_MRAM_SERVICE
select NRFS_HAS_PMIC_SERVICE
select NRFS_HAS_TEMP_SERVICE
select NRFS_TEMP_SERVICE_HAS_SUBSCRIPTION_SERVICE

config SOC_NRF9230_ENGB_CPUPPR
select RISCV_CORE_NORDIC_VPR
Expand Down
1 change: 0 additions & 1 deletion tests/drivers/sensor/temp_sensor/boards/nrf54h20dk_nrf54h20_cpuapp.conf
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,2 +1 @@
CONFIG_NRFS=y
CONFIG_TEMP_NRFS_TRIGGER_OWN_THREAD=y
2 changes: 1 addition & 1 deletion west.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -200,7 +200,7 @@ manifest:
groups:
- hal
- name: hal_nordic
revision: 2f5d4e5868ab573eac932fa4bc142565073c3c04
revision: a6579483deb33112cc763d05a4a3f8085883c1ac
path: modules/hal/nordic
groups:
- hal
Expand Down