Skip to content

Conversation

@marwaiehm-st
Copy link
Contributor

@marwaiehm-st marwaiehm-st commented Apr 10, 2025

Enabling support for STM32WBA6

@github-actions
Copy link

github-actions bot commented Apr 10, 2025

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

Name Old Revision New Revision Diff
hal_stm32 zephyrproject-rtos/hal_stm32@46c5ca5 zephyrproject-rtos/hal_stm32@f467d8e (main) zephyrproject-rtos/[email protected]

All manifest checks OK

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) labels Apr 10, 2025
@marwaiehm-st marwaiehm-st force-pushed the upstream_wbax branch 7 times, most recently from 0fdc5b0 to 98aef86 Compare April 10, 2025 13:16
@marwaiehm-st marwaiehm-st changed the title Upstream the support for WBA6 Upstream the support for STM32WBA6 Apr 10, 2025
@marwaiehm-st marwaiehm-st marked this pull request as ready for review April 10, 2025 15:30
@github-actions github-actions bot added platform: STM32 ST Micro STM32 area: Flash area: Clock Control area: ADC Analog-to-Digital Converter (ADC) labels Apr 10, 2025
nordicjm
nordicjm previously approved these changes Apr 15, 2025
FRASTM added 4 commits April 15, 2025 09:52
update modules hal stm32 for stm32wb and stm32wba

Signed-off-by: Francois Ramu <[email protected]>
This commit adds support for the STM32WBA65x MCU.

Signed-off-by: Francois Ramu <[email protected]>
This commit adds the Device Tree include files
for the STM32WBA65x device
Adding GPIO D/E/G banks.
Renaming JTAG reset pin.

Signed-off-by: Francois Ramu <[email protected]>
Add the clock domain definition for the stm32wba CCIPR

Signed-off-by: Francois Ramu <[email protected]>
@github-actions github-actions bot removed the DNM (manifest) This PR should not be merged (controlled by action-manifest) label Apr 15, 2025
@marwaiehm-st
Copy link
Contributor Author

Updated the west.yml with HAL STM32 merged commit f467d8e

nordicjm
nordicjm previously approved these changes Apr 15, 2025
erwango
erwango previously approved these changes Apr 15, 2025
benothmn-st
benothmn-st previously approved these changes Apr 15, 2025
Comment on lines 2 to 4
ST Nucleo WBA65RI
#################
Copy link
Contributor

Choose a reason for hiding this comment

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

sphinx directive adds this automatically

Suggested change
ST Nucleo WBA65RI
#################

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

Comment on lines 40 to 43

.. image:: img/nucleo_wba65ri.webp
:align: center
:alt: Nucleo WBA65RI
Copy link
Contributor

Choose a reason for hiding this comment

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

sphinx directive adds this automatically

Suggested change
.. image:: img/nucleo_wba65ri.webp
:align: center
:alt: Nucleo WBA65RI

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

Comment on lines +156 to +155

&rng {
status = "okay";
};
Copy link
Contributor

Choose a reason for hiding this comment

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

duplicate entry

Suggested change
&rng {
status = "okay";
};

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@@ -0,0 +1,4 @@
board_runner_args(stm32cubeprogrammer "--port=swd" "--reset-mode=hw")
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
board_runner_args(stm32cubeprogrammer "--port=swd" "--reset-mode=hw")
# SPDX-License-Identifier: Apache-2.0
board_runner_args(stm32cubeprogrammer "--port=swd" "--reset-mode=hw")

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

-----------

Nucleo WBA65RI board has 3 U(S)ARTs. The Zephyr console output is assigned to USART1.
Default settings are 115500 8N1.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Default settings are 115500 8N1.
Default settings are 115200 8N1.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@@ -0,0 +1,26 @@
# Note: Using OpenOCD using nucloe_wba65ri requires using openocd fork.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
# Note: Using OpenOCD using nucloe_wba65ri requires using openocd fork.
# Note: Using OpenOCD using nucleo_wba65ri requires using openocd fork.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

- I2C_1_SCL : PB2
- I2C_1_SDA : PB1
- USER_PB : PC13
- LD1 : PB4
Copy link
Contributor

Choose a reason for hiding this comment

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

this is not what devicetree says?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, it should be LD1 : PD8

FRASTM and others added 3 commits April 15, 2025 14:27
Introduce the stm32WBA56 nucleo board.
HSE32 divided by 2 is the source clock.

Signed-off-by: Francois Ramu <[email protected]>
-The stm32wba6x has Dual Bank memory. Change the flash driver
 to support this OPTion given by presence of the
 DUAL_BANK bit (21) in the FLASH_OPTR register.

-Flash erase with 2 banks: Add the control of the BKER
 bit of the FLASH_NSCR1 to select BANK1 or 2 of
 the internal flash depending
 on the page number >127 for BANK2

Signed-off-by: Francois Ramu <[email protected]>
Add nucleo_wba65ri overlay file

Signed-off-by: IBEN EL HADJ MESSAOUD Marwa <[email protected]>
Copy link
Contributor

@kartben kartben left a comment

Choose a reason for hiding this comment

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

thanks for addressing my comments so quickly :)

@matt-wood-ct
Copy link
Contributor

Awesome work, very keen to use this part with Zephyr 😃

@kartben kartben merged commit a2cb852 into zephyrproject-rtos:main Apr 15, 2025
27 checks passed
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.

10 participants