Skip to content

ST: STM32WBA2: Add BLE support #108116

Open
HoZHel wants to merge 2 commits intozephyrproject-rtos:mainfrom
HoZHel:Add_WBA2_BLE
Open

ST: STM32WBA2: Add BLE support #108116
HoZHel wants to merge 2 commits intozephyrproject-rtos:mainfrom
HoZHel:Add_WBA2_BLE

Conversation

@HoZHel
Copy link
Copy Markdown
Contributor

@HoZHel HoZHel commented Apr 28, 2026 •
edited
Loading

  • Provide Bluetooth LE support
  • Change the ADC4 input channel to avoid a conflict with LED0
  • Enable ADC4 and die_temp to have linklayer calibration based on the
    temperature

Supplementary PR #364

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 28, 2026 •
edited
Loading

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

Name Old Revision New Revision Diff
hal_stm32 zephyrproject-rtos/hal_stm32@686cf2d zephyrproject-rtos/hal_stm32#364 zephyrproject-rtos/hal_stm32#364/files

Additional metadata changed:

Name URL Submodules West cmds module.yml Blobs
hal_stm32 ✏ 4x ✏, 2x 🆕

⛔ DNM label due to: 1 project with PR revision, 1 project with metadata changes and 6 blob changes

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

@github-actions github-actions Bot added manifest manifest-hal_stm32 DNM (manifest) This PR should not be merged (controlled by action-manifest) Binary Blobs Added labels Apr 28, 2026
@HoZHel HoZHel mentioned this pull request Apr 28, 2026
Open
@HoZHel HoZHel marked this pull request as ready for review April 29, 2026 07:48
etienne-lms
etienne-lms reviewed Apr 29, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@etienne-lms etienne-lms left a comment

Choose a reason for hiding this comment

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

For commit "ST: STM32WBA2: Add BLE support", could you refine the commit message header line? E.g.:

boards: st: nuclo_wba25ce1: Add BLE support

Despite minor comments, LGTM.

Comment thread boards/st/nucleo_wba25ce1/nucleo_wba25ce1.dts Outdated
@HoZHel
Copy link
Copy Markdown
Contributor Author

HoZHel commented Apr 29, 2026

For commit "ST: STM32WBA2: Add BLE support", could you refine the commit message header line? E.g.:

boards: st: nuclo_wba25ce1: Add BLE support

Despite minor comments, LGTM.

Done.

etienne-lms
etienne-lms previously approved these changes Apr 29, 2026
View reviewed changes
juickar
juickar reviewed Apr 30, 2026
View reviewed changes
Comment thread boards/st/nucleo_wba25ce1/nucleo_wba25ce1.dts Outdated
@HoZHel
Copy link
Copy Markdown
Contributor Author

HoZHel commented Apr 30, 2026

Rebased + adding one more commit to include "bluetooth" term in supported features.

@etienne-lms
Copy link
Copy Markdown
Contributor

etienne-lms commented Apr 30, 2026

My typo regarding commit messagee header line:

-boards: st: nuclo_wba25ce1: Add BLE support
+boards: st: nucleo_wba25ce1: Add BLE support

While at it, also worth to squash commit "boards: st: nucleo_wba25ce1: Include Bluetooth term in supported features" into this 1st commit.

Last, nitpicking comment: the commit updating west.yml manifest should be placed first, since needed by the commit that claims nucleo_wba25ce1 board supports BLE tests.

@HoZHel
west.yml: Update west to point to the recent changes for hal_stm32
0a3ea39 
Update west.yml to point to the recent changes for hal_stm32.

Signed-off-by: Ali Hozhabri <ali.hozhabri@st.com>
@HoZHel
boards: st: nuclo_wba25ce1: Add BLE support
5707216 
Provide Bluetooth LE support.

Change the ADC4 input channel to avoid a conflict with LED0.
Enable ADC4 and die_temp to have linklayer calibration based on the
temperature.

Add "bluetooth" term to supported features.

Signed-off-by: Ali Hozhabri <ali.hozhabri@st.com>
@HoZHel
Copy link
Copy Markdown
Contributor Author

HoZHel commented Apr 30, 2026

My typo regarding commit messagee header line:

-boards: st: nuclo_wba25ce1: Add BLE support
+boards: st: nucleo_wba25ce1: Add BLE support

While at it, also worth to squash commit "boards: st: nucleo_wba25ce1: Include Bluetooth term in supported features" into this 1st commit.

Last, nitpicking comment: the commit updating west.yml manifest should be placed first, since needed by the commit that claims nucleo_wba25ce1 board supports BLE tests.

Done.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Apr 30, 2026

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

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

Reviewers

@juickar juickar juickar left review comments

@etienne-lms etienne-lms etienne-lms left review comments

@asm5878 asm5878 Awaiting requested review from asm5878

@djiatsaf-st djiatsaf-st Awaiting requested review from djiatsaf-st

@erwango erwango Awaiting requested review from erwango

@FRASTM FRASTM Awaiting requested review from FRASTM

@gautierg-st gautierg-st Awaiting requested review from gautierg-st

@GeorgeCGV GeorgeCGV Awaiting requested review from GeorgeCGV

@mathieuchopstm mathieuchopstm Awaiting requested review from mathieuchopstm

At least 2 approving reviews are required to merge this pull request.

Assignees

@erwango erwango

Labels

area: Boards/SoCs Binary Blobs Added Binary Blobs Modified DNM (manifest) This PR should not be merged (controlled by action-manifest) manifest manifest-hal_stm32 platform: STM32 ST Micro STM32

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@HoZHel @etienne-lms @juickar @erwango @zephyrbot