Skip to content

Commit

Permalink
samples/subsys/mgmt/osdp: Fix build issues
Browse files Browse the repository at this point in the history
The samples/subsys/mgmt/osdp utilize GPIO so having it set in the
prj.conf is needed since not all platforms enable GPIO by default.

To address the 'No SOURCES given to Zephyr library: drivers__gpio'
add a 'depends on gpio' to the sample.yaml to only build this on
platforms that have GPIO driver support.

Fixes zephyrproject-rtos#39180

Signed-off-by: Kumar Gala <[email protected]>
  • Loading branch information
galak authored and Rushybrook committed Oct 21, 2021
1 parent 672e9cf commit d288fee
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 0 deletions.
1 change: 1 addition & 0 deletions samples/subsys/mgmt/osdp/control_panel/prj.conf
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@

CONFIG_PRINTK=y
CONFIG_LOG=y
CONFIG_GPIO=y

# OSDP config
CONFIG_OSDP=y
Expand Down
1 change: 1 addition & 0 deletions samples/subsys/mgmt/osdp/control_panel/sample.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ sample:
tests:
sample.mgmt.osdp.control_panel:
tags: osdp
depends_on: gpio
filter: dt_enabled_alias_with_parent_compat("led0", "gpio-leds") and CONFIG_SERIAL
harness: osdp
integration_platforms:
Expand Down
1 change: 1 addition & 0 deletions samples/subsys/mgmt/osdp/peripheral_device/prj.conf
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@

CONFIG_PRINTK=y
CONFIG_LOG=y
CONFIG_GPIO=y

# OSDP config
CONFIG_OSDP=y
Expand Down
1 change: 1 addition & 0 deletions samples/subsys/mgmt/osdp/peripheral_device/sample.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ sample:
tests:
sample.mgmt.osdp.peripheral_device:
tags: osdp
depends_on: gpio
filter: dt_enabled_alias_with_parent_compat("led0", "gpio-leds") and CONFIG_SERIAL
harness: osdp
integration_platforms:
Expand Down

0 comments on commit d288fee

Please sign in to comment.