Skip to content

feat(bsp_selector): Component for selection BSP in menuconfig#705

Merged
espzav merged 1 commit intomasterfrom
feat/bsp_selector
Jan 27, 2026
Merged

feat(bsp_selector): Component for selection BSP in menuconfig#705
espzav merged 1 commit intomasterfrom
feat/bsp_selector

Conversation

@espzav
Copy link
Copy Markdown
Collaborator

@espzav espzav commented Jan 7, 2026

ESP-BSP Pull Request checklist

  • Version of modified component bumped
  • CI passing

Change description

  • Created BSP Selector component - for select BSP from menuconfig
  • It should work from IDF 5.3
  • We keep old bsp_ext.py for CI
  • The CI removes bsp_selector component in examples, which are added to each BSP
  • Added ignore warnings for warning: unknown kconfig symbol* - I didn't find any other solution for it.
    • I tried to remove this configuration option from sdkconfig and add it into presets - not working

Note

Introduces a new components/bsp_selector component to choose BSPs via menuconfig, with Kconfig, CMakeLists.txt fail-safe when none selected, idf_component.yml conditional deps to BSPs, README, and license.

  • Replaces hardcoded BSP deps in examples with bsp_selector; adds per-board sdkconfig.bsp.* files setting CONFIG_BSP_SELECT_* and updates defaults
  • Updates CI: bumps component manager to 2.4.*, uploads components/bsp_selector, adds ignore rule for warning: unknown kconfig symbol*, and adjusts scripts (.github/ci/bsp_example_update.py, examples/bsp_ext.py) to remove bsp_selector when pinning a specific BSP for builds
  • Enhances PR checklist to require adding new BSPs to bsp_selector Kconfig/idf_component.yml

Written by Cursor Bugbot for commit ea54c8f. This will update automatically on new commits. Configure here.

@espzav espzav marked this pull request as draft January 7, 2026 15:07
cursor[bot]

This comment was marked as outdated.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Jan 7, 2026

Test Results

28 tests   28 ✅  1m 32s ⏱️
 5 suites   0 💤
 5 files     0 ❌

Results for commit ea54c8f.

♻️ This comment has been updated with latest results.

@espzav espzav force-pushed the feat/bsp_selector branch 2 times, most recently from 01037a6 to c0347be Compare January 8, 2026 07:59
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jan 8, 2026

⚠️ Build failed for idf_ver=latest index=1

This failure was ignored (continue-on-error enabled).
See details: View build logs

@espzav espzav force-pushed the feat/bsp_selector branch from c0347be to a1806af Compare January 8, 2026 08:21
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jan 8, 2026

⚠️ Build failed for idf_ver=latest index=6

This failure was ignored (continue-on-error enabled).
See details: View build logs

@github-actions
Copy link
Copy Markdown

github-actions bot commented Jan 8, 2026

⚠️ Build failed for idf_ver=latest index=3

This failure was ignored (continue-on-error enabled).
See details: View build logs

@espzav espzav force-pushed the feat/bsp_selector branch 9 times, most recently from 40691f1 to 793b886 Compare January 13, 2026 12:18
@espzav espzav marked this pull request as ready for review January 13, 2026 13:51
@espzav espzav force-pushed the feat/bsp_selector branch from b1b8203 to ce2bd0a Compare January 13, 2026 13:51
@espzav
Copy link
Copy Markdown
Collaborator Author

espzav commented Jan 13, 2026

@tore-espressif @PetrESP PTAL

Comment thread examples/bsp_ext.py Outdated
Comment thread examples/display_audio_photo/sdkconfig.bsp.esp-box Outdated
Comment thread components/bsp_selector/Kconfig
@PetrESP
Copy link
Copy Markdown
Collaborator

PetrESP commented Jan 13, 2026

In my opinion adding this component is a the cleanest solution for this issue and I agree with this concept

@espzav espzav force-pushed the feat/bsp_selector branch from 778b8f3 to 891e126 Compare January 14, 2026 08:35
Comment thread components/bsp_selector/Kconfig
@espzav espzav force-pushed the feat/bsp_selector branch 2 times, most recently from b4a7c6f to 2bea9d3 Compare January 14, 2026 09:35
@PetrESP
Copy link
Copy Markdown
Collaborator

PetrESP commented Jan 15, 2026

Tested locally and it works well on my debian laptop.

Comment thread components/bsp_selector/README.md Outdated
Comment thread components/bsp_selector/Kconfig Outdated
@espzav espzav force-pushed the feat/bsp_selector branch 2 times, most recently from c389b73 to 43975c4 Compare January 21, 2026 09:12
Comment thread examples/display_sensors/sdkconfig.defaults.esp32 Outdated
@espzav espzav force-pushed the feat/bsp_selector branch 2 times, most recently from b8f0348 to f5911cb Compare January 27, 2026 11:34
Comment thread examples/display_camera/sdkconfig.defaults.esp32s2 Outdated
Comment thread examples/mqtt_example/sdkconfig.defaults.esp32 Outdated
Comment thread examples/display_sensors/sdkconfig.bsp.esp32_azure_iot_kit
@espzav espzav force-pushed the feat/bsp_selector branch from f5911cb to ea54c8f Compare January 27, 2026 12:48
Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

Comment thread examples/display_sensors/main/idf_component.yml
@github-actions
Copy link
Copy Markdown

⚠️ Build failure was ignored (continue-on-error enabled).

See details: View build logs

@espzav espzav merged commit 3c3ee9e into master Jan 27, 2026
96 checks passed
@espzav espzav deleted the feat/bsp_selector branch January 27, 2026 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants