Skip to content
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.

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

driver: bt: add BT_BUILD_ONLY_MODE to allow CI tests #80786

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

sylvioalves
Copy link
Collaborator

@sylvioalves sylvioalves commented Nov 3, 2024

Adds an option to enable CI tests without binary blobs. Same as existing "WIFI_BUILD_ONLY_MODE" option.

Read this before: #80785

Adds an option to enable CI tests without binary blobs.
Same as existing "WIFI_BUILD_ONLY_MODE" option.

Signed-off-by: Sylvio Alves <[email protected]>
Adds extra configuration for at least central and peripheral test
cases so that espressif SoC can be properly tested.

Signed-off-by: Sylvio Alves <[email protected]>
@sylvioalves sylvioalves force-pushed the enh/make_bluetooth_build_only_global branch from 311bfd1 to 3128271 Compare November 28, 2024 02:15
@zephyrbot
Copy link
Collaborator

zephyrbot commented Nov 28, 2024

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff
hal_espressif zephyrproject-rtos/hal_espressif@980d61c zephyrproject-rtos/hal_espressif#359 zephyrproject-rtos/hal_espressif#359/files

DNM label due to: 1 project with PR revision

Note: This message is automatically posted and updated by the Manifest GitHub Action.

Remove bluetooth ignore tag so it can run CI tests whenever
platform is filtered.

Signed-off-by: Sylvio Alves <[email protected]>
@sylvioalves sylvioalves force-pushed the enh/make_bluetooth_build_only_global branch from 3128271 to b81a41a Compare November 28, 2024 02:16
@jhedberg
Copy link
Member

As suggested in zephyrproject-rtos/hal_espressif#359 (comment) I think it would make sense to just have one option for all of this, since it's all about doing CI compile testing without blobs present. The specific feature(s) that blobs are needed for seems secondary.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants