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

boards: arm: sensortile_box_pro: add bluenrg-lp chip description #78564

Merged
merged 1 commit into from
Sep 23, 2024

Conversation

avisconti
Copy link
Collaborator

Extend sensortile_box_pro document adding BlueNRG-LP chip description.

This PR replaces #74255

erwango
erwango previously approved these changes Sep 18, 2024
Copy link
Member

@erwango erwango left a comment

Choose a reason for hiding this comment

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

LGTM, but see comments for a rewording proposal.

Comment on lines 221 to 224
The board is equipped with an STMicroelectronics `BlueNRG-LP`_ chip. Before using the board Bluetooth
functionality, a firmware upgrade of the chip may be necessary, in order to run a firmware which is
compatible with the Zephyr BT stack (see `stsw-mkbox-bleco`_ for more information).
The upgrade may be easily performed following the procedure explained in `ble-fw-upg`_.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
The board is equipped with an STMicroelectronics `BlueNRG-LP`_ chip. Before using the board Bluetooth
functionality, a firmware upgrade of the chip may be necessary, in order to run a firmware which is
compatible with the Zephyr BT stack (see `stsw-mkbox-bleco`_ for more information).
The upgrade may be easily performed following the procedure explained in `ble-fw-upg`_.
The board is equipped with an STMicroelectronics `BlueNRG-LP`_ chip. Before running Zephyr Bluetooth samples
on SensorTile.box PRO, it is required to upgrade the BlueNRG chip with a Zephyr BLE stack compatible firmware.
The upgrade may be easily performed using the application provided in `SensorTile.box PRO BLE firmware upgrade package`_.
For more information about BLE binaries for SensorTile.box family, see `stsw-mkbox-bleco`_.

@avisconti
Copy link
Collaborator Author

LGTM, but see comments for a rewording proposal.

ok, seems better

Extend sensortile_box_pro document adding BlueNRG-LP chip description.

Signed-off-by: Armando Visconti <[email protected]>
@avisconti
Copy link
Collaborator Author

@erwango can you possibly review it again? I just applied your suggestions

@avisconti
Copy link
Collaborator Author

Added also @kartben and @fabiobaltieri in the reviewer list. If you can also give a look to this PR... it is only a small documentation change

@mmahadevan108 mmahadevan108 merged commit 3508994 into zephyrproject-rtos:main Sep 23, 2024
16 checks passed
@avisconti avisconti deleted the mkbox-ble-doc branch September 24, 2024 09:04
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