Skip to content

Enabling IW416 and IW612 BT HCI controller with MIMXRT1060EVKC platform for Bluetooth apps to work #85639

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

Conversation

nirav-agrawal
Copy link
Contributor

@nirav-agrawal nirav-agrawal commented Feb 12, 2025

This PR has below updates,

1. Enabled IW416 (with Murata 1XK M2 module) SoC to work with BT HCI driver, perform BT only firmware download using NXP HCI Driver.
a. Adding Firmware blobs and nxp_hal related changes are done to fetch the firmware, perform .bin to .inc conversion, and
include it in inclusion list. We will update west manifest with both PRs once it gets raised here.

2. Enabled MIMXRT1060EVKC platform works with IW416/IW612 controller M2 modules (1XK/2EL) with help of new shield.
a. "nxp_m2_wifi_bt" shield has an overlay which is required for WIFI SDIO and BT HCI driver attachment for both IW416 and IW612
controllers. To support this shield for other platform, board overlay should be added similar to RT1060EVKC overlay which
expose platform specific pin-mux details.

3. Added required kconfigs for Bluetooth samples to work on iMXRT platforms without any initial crashes like stack overflows from various tasks.

This PR has dependency on zephyrproject-rtos/hal_nxp#512. This PR needs to be merged after hal_nxp/PR#512.

Regards,
Nirav

@zephyrbot zephyrbot added area: Shields Shields (add-on boards) platform: NXP Drivers NXP Semiconductors, drivers platform: NXP NXP area: Bluetooth area: Bluetooth Host Bluetooth Host (excluding BR/EDR) area: Bluetooth HCI Bluetooth HCI Driver labels Feb 12, 2025
Copy link

Hello @nirav-agrawal, and thank you very much for your first pull request to the Zephyr project!
Our Continuous Integration pipeline will execute a series of checks on your Pull Request commit messages and code, and you are expected to address any failures by updating the PR. Please take a look at our commit message guidelines to find out how to format your commit messages, and at our contribution workflow to understand how to update your Pull Request. If you haven't already, please make sure to review the project's Contributor Expectations and update (by amending and force-pushing the commits) your pull request if necessary.
If you are stuck or need help please join us on Discord and ask your question there. Additionally, you can escalate the review when applicable. 😊

@nirav-agrawal nirav-agrawal force-pushed the iw416_nw612_bt_enablement_with_rt1060evkc branch from 5809bbb to cfb1eee Compare February 12, 2025 07:28
@Thalley Thalley removed their request for review February 12, 2025 07:41
@nirav-agrawal nirav-agrawal force-pushed the iw416_nw612_bt_enablement_with_rt1060evkc branch from cfb1eee to 1f76e40 Compare February 12, 2025 07:58
@zephyrbot zephyrbot requested a review from theob-pro February 12, 2025 07:59
@nirav-agrawal nirav-agrawal force-pushed the iw416_nw612_bt_enablement_with_rt1060evkc branch from 1f76e40 to 7bb74fc Compare February 17, 2025 04:36

This Zephyr shield is tested with the following M.2 modules and hardware for Wi-Fi and Bluetooth applications:
- Embedded Artist 1XK module - uses Murata 1XK radio module with NXP IW416 chipset
- Embedded Artist 2EL module - uses Murata 2EL radio module with NXP IW612 chipset
Copy link
Member

Choose a reason for hiding this comment

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

Could this be the improper indent?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Error is at line 50. But I can remove extra space and recheck this. thanks

Copy link
Collaborator

Choose a reason for hiding this comment

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

Yeah still not ok :) please see my comment
Bullet lists need to be preceded by an empty line

@nirav-agrawal nirav-agrawal force-pushed the iw416_nw612_bt_enablement_with_rt1060evkc branch 2 times, most recently from 5471da4 to eab146f Compare April 25, 2025 14:52
@mmahadevan108 mmahadevan108 removed the DNM (manifest) This PR should not be merged (controlled by action-manifest) label Apr 25, 2025
mmahadevan108
mmahadevan108 previously approved these changes Apr 25, 2025
dleach02
dleach02 previously approved these changes Apr 25, 2025
Copy link
Collaborator

@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.

Comment on lines 9 to 11
This Zephyr shield is tested with the following M.2 modules and hardware for Wi-Fi and Bluetooth applications:
- Embedded Artist 1XK module - uses Murata 1XK radio module with NXP IW416 chipset
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
This Zephyr shield is tested with the following M.2 modules and hardware for Wi-Fi and Bluetooth applications:
- Embedded Artist 1XK module - uses Murata 1XK radio module with NXP IW416 chipset
This Zephyr shield is tested with the following M.2 modules and hardware for Wi-Fi and Bluetooth applications:
- Embedded Artist 1XK module - uses Murata 1XK radio module with NXP IW416 chipset

Comment on lines 15 to 20
`IW612 NXP Chipset <https://www.nxp.com/products/IW612>`_
`IW416 NXP Chipset <https://www.nxp.com/products/IW416>`_
`2EL Murata Radio Module <https://www.murata.com/en-us/products/connectivitymodule/wi-fi-bluetooth/overview/lineup/type2el>`_
`1XK Murata Radio Module <https://www.murata.com/en-us/products/connectivitymodule/wi-fi-bluetooth/overview/lineup/type1xk>`_
`1XK Embedded Artist Module <https://www.embeddedartists.com/products/1xk-m-2-module>`_
`2EL Embedded Artist Module <https://www.embeddedartists.com/products/2el-m-2-module>`_
Copy link
Collaborator

Choose a reason for hiding this comment

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

Make a proper bullet list - see here how it looks like currently https://builds.zephyrproject.io/zephyr/pr/85639/docs/boards/shields/nxp_m2_wifi_bt/doc/index.html

Comment on lines 45 to 46
- nxp_m2_1xk_wifi_bt : For Wi-Fi/Bluetooth samples to work with NXP IW416 SoC
- nxp_m2_2el_wifi_bt : For Wi-Fi/Bluetooth samples to work with NXP IW612 SoC
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
- nxp_m2_1xk_wifi_bt : For Wi-Fi/Bluetooth samples to work with NXP IW416 SoC
- nxp_m2_2el_wifi_bt : For Wi-Fi/Bluetooth samples to work with NXP IW612 SoC
- ``nxp_m2_1xk_wifi_bt``: For Wi-Fi/Bluetooth samples to work with NXP IW416 SoC
- ``nxp_m2_2el_wifi_bt``: For Wi-Fi/Bluetooth samples to work with NXP IW612 SoC

@dleach02 dleach02 dismissed stale reviews from mmahadevan108 and themself via 6a3cc42 April 25, 2025 18:27
@dleach02 dleach02 force-pushed the iw416_nw612_bt_enablement_with_rt1060evkc branch from eab146f to 6a3cc42 Compare April 25, 2025 18:27
@dleach02
Copy link
Member

@nirav-agrawal, I applied the changes @kartben requested.

@github-actions github-actions bot added the DNM (manifest) This PR should not be merged (controlled by action-manifest) label Apr 25, 2025
kartben
kartben previously approved these changes Apr 25, 2025
Copy link
Collaborator

@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.

+1 for docs/shield, thank you!

DerekSnell
DerekSnell previously approved these changes Apr 25, 2025
Copy link
Collaborator

@DerekSnell DerekSnell left a comment

Choose a reason for hiding this comment

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

Well done, @nirav-agrawal . Thanks for all your work on this.

dleach02
dleach02 previously approved these changes Apr 25, 2025
- Added new shield to add support for WIFI and BT interface.
- Added board overlay for MIMXRT1060EVKC.
- Removed mimxrt1062 overlay from shell example to avoid conflicts.
- Added seperate shield overlay for supported M.2 module to enable
 BT or WIFI or Both.
- Added default kconfigs for each BT and WIFI which enables supported
 modules and SoC as part of enabled shield through build param.
- Added shield document.

Signed-off-by: Nirav Agrawal <[email protected]>
@dleach02 dleach02 dismissed stale reviews from DerekSnell, kartben, and themself via 1604108 April 26, 2025 02:02
@dleach02 dleach02 force-pushed the iw416_nw612_bt_enablement_with_rt1060evkc branch from 6a3cc42 to 1604108 Compare April 26, 2025 02:02
@dleach02 dleach02 removed the DNM (manifest) This PR should not be merged (controlled by action-manifest) label Apr 28, 2025
@MaureenHelm MaureenHelm merged commit bef5abd into zephyrproject-rtos:main Apr 28, 2025
32 of 33 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Bluetooth HCI Bluetooth HCI Driver area: Bluetooth Host Bluetooth Host (excluding BR/EDR) area: Bluetooth area: Shields Shields (add-on boards) area: Wi-Fi Wi-Fi manifest manifest-hal_nxp platform: NXP Drivers NXP Semiconductors, drivers platform: NXP NXP
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants