Skip to content

Commit 8876a3b

Browse files
Rafal-Nordickartben
authored andcommitted
modules: hal_nordic: nrfs: Disabling subscription
- Code optimization for platforms which don't use subscription feature in the temperature service. - Test adaptation to code changes Signed-off-by: Rafal Dyla <[email protected]>
1 parent 347fb0a commit 8876a3b

File tree

6 files changed

+17
-4
lines changed

6 files changed

+17
-4
lines changed

drivers/sensor/nordic/temp/Kconfig

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,11 +28,11 @@ config TEMP_NRFS
2828
Enable driver for nRF temperature sensor accessed through the nRF
2929
Services (nrfs) layer.
3030

31-
if TEMP_NRFS
31+
if TEMP_NRFS && NRFS_TEMP_SERVICE_SUBSCRIPTION_ENABLED
3232

3333
module = TEMP_NRFS
3434
thread_priority = 10
3535
thread_stack_size = 1024
3636
source "drivers/sensor/Kconfig.trigger_template"
3737

38-
endif # TEMP_NRFS
38+
endif

modules/hal_nordic/nrfs/Kconfig

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,9 @@ config NRFS_HAS_TEMP_SERVICE
4646
config NRFS_HAS_VBUS_DETECTOR_SERVICE
4747
bool
4848

49+
config NRFS_TEMP_SERVICE_HAS_SUBSCRIPTION_SERVICE
50+
bool
51+
4952
config NRFS
5053
bool "nRF Services Support"
5154
select NRFS_LOCAL_DOMAIN if SOC_NRF54H20_CPUAPP || \
@@ -93,6 +96,11 @@ config NRFS_TEMP_SERVICE_ENABLED
9396
depends on NRFS_HAS_TEMP_SERVICE
9497
default y
9598

99+
config NRFS_TEMP_SERVICE_SUBSCRIPTION_ENABLED
100+
bool "Subscription feature for the temperature service"
101+
depends on NRFS_TEMP_SERVICE_HAS_SUBSCRIPTION_SERVICE
102+
default y
103+
96104
config NRFS_VBUS_DETECTOR_SERVICE_ENABLED
97105
bool "VBUS detector for the USB peripheral"
98106
depends on NRFS_HAS_VBUS_DETECTOR_SERVICE

modules/hal_nordic/nrfs/nrfs_config.h

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,10 @@
1212
#define NRFS_TEMP_SERVICE_ENABLED
1313
#endif
1414

15+
#ifdef CONFIG_NRFS_TEMP_SERVICE_SUBSCRIPTION_ENABLED
16+
#define NRFS_TEMP_SERVICE_SUBSCRIPTION_ENABLED
17+
#endif
18+
1519
#ifdef CONFIG_NRFS_MRAM_SERVICE_ENABLED
1620
#define NRFS_MRAM_SERVICE_ENABLED
1721
#endif

soc/nordic/nrf92/Kconfig

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@ config SOC_NRF9230_ENGB_CPUAPP
2828
select NRFS_HAS_PMIC_SERVICE
2929
select NRFS_HAS_TEMP_SERVICE
3030
select NRFS_HAS_VBUS_DETECTOR_SERVICE
31+
select NRFS_TEMP_SERVICE_HAS_SUBSCRIPTION_SERVICE
3132

3233
config SOC_NRF9230_ENGB_CPURAD
3334
select ARM
@@ -46,6 +47,7 @@ config SOC_NRF9230_ENGB_CPURAD
4647
select NRFS_HAS_MRAM_SERVICE
4748
select NRFS_HAS_PMIC_SERVICE
4849
select NRFS_HAS_TEMP_SERVICE
50+
select NRFS_TEMP_SERVICE_HAS_SUBSCRIPTION_SERVICE
4951

5052
config SOC_NRF9230_ENGB_CPUPPR
5153
select RISCV_CORE_NORDIC_VPR
Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,2 +1 @@
11
CONFIG_NRFS=y
2-
CONFIG_TEMP_NRFS_TRIGGER_OWN_THREAD=y

west.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -200,7 +200,7 @@ manifest:
200200
groups:
201201
- hal
202202
- name: hal_nordic
203-
revision: 2f5d4e5868ab573eac932fa4bc142565073c3c04
203+
revision: a6579483deb33112cc763d05a4a3f8085883c1ac
204204
path: modules/hal/nordic
205205
groups:
206206
- hal

0 commit comments

Comments
 (0)