Update Document for nxp_m2_wifi_bt Shield with Hardware-Rework and Correct Wi-Fi Shell Build Command #90749
Conversation
nirav-agrawal
commented
May 28, 2025
nirav-agrawal
commented
- Update hardware rework details require for Bluetooth HCI UART and I2S interfaces to work with M.2 module.
- Update west build command for Wi-Fi shell app with extra config overlay file.
github-actions
bot
requested review from
avisconti,
erwango,
jfischer-no and
kartben
nirav-agrawal
force-pushed
the
update_nxp_shield_doc
branch
from
296d2d9 to
0aae533
Compare
|
Hi @kartben , I need your help to understand compliance error with this PR. I tried to update the image from PNG to SVG which reduces image size from 5.47mb to 837kb. It seems the size needs smaller than 100kb which seems not possible. can you suggest if there is any alternate to it? |
SVG is not meant to be used for photos :) |
nirav-agrawal
force-pushed
the
update_nxp_shield_doc
branch
from
0aae533 to
0a541ba
Compare
Hi @kartben, thank you so much for your suggestions. now I have used webp after converting it from png to and tried to keep it under 100kb. In future when we have such link available from the company site, we will use it here. :) |
|
@kartben, can you help approve and merge this PR? thank you :) |
Sure, but it hasn't even been open for 24 hours :) I will have a look, but this won't be merged until it's been in review for at least 2 business days anyway. |
|
@nirav-agrawal Can I please ask that you give reviewers time to come back to their PRs in due course instead of pingging every other day? Thanks! :) |
| This shield is validated for use with the host platform listed below, which is actively maintained and supported by NXP. | ||
| While the shield can be integrated with other host platforms, NXP may not actively test or validate those configurations. |
There was a problem hiding this comment.
| This shield is validated for use with the host platform listed below, which is actively maintained and supported by NXP. | |
| While the shield can be integrated with other host platforms, NXP may not actively test or validate those configurations. | |
| This shield is validated and tested for use with the host platform listed below. It can be used with | |
| other host platforms, but the functionality is not guaranteed. |
There was a problem hiding this comment.
Hi @kartben, thank you for your suggestion. I updated doc with suggested lines.
| .. note:: | ||
| To build Wi-Fi shell application, "nxp/overlay_hosted_mcu.conf" config file | ||
| must be passed along with default conf file as mentioned in above build command. |
There was a problem hiding this comment.
This note seems redundant? However, if these are options that are always needed when user wants to do anything Wi-Fi (i.e not just the wifi shell sample specifically), then maybe it can be reworded along these lines?
There was a problem hiding this comment.
Hi @kartben, the given config file is required for any Wi-Fi application. reworded the line as suggested. thank you. :)
- Update hardware rework details require for Bluetooth HCI UART and I2S interfaces to work with M.2 module. - Update west build command for Wi-Fi shell app with extra config overlay file. Signed-off-by: Nirav Agrawal <nirav.agrawal@nxp.com>
nirav-agrawal
dismissed stale reviews from mmahadevan108 and DerekSnell
via
d8b4f52
nirav-agrawal
force-pushed
the
update_nxp_shield_doc
branch
from
0a541ba to
d8b4f52
Compare
|
