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

[K32W] Add commissioning instructions #4754

Merged
merged 2 commits into from
Feb 11, 2021

Conversation

doru91
Copy link
Contributor

@doru91 doru91 commented Feb 9, 2021

Describe how to setup the topology for NXP E-Lock Demo App.

@boring-cyborg boring-cyborg bot added documentation Improvements or additions to documentation examples labels Feb 9, 2021
@doru91 doru91 force-pushed the feature/k32w_br_doc branch 2 times, most recently from 6da0c2c to b5e1504 Compare February 9, 2021 08:28
@doru91 doru91 force-pushed the feature/k32w_br_doc branch from b5e1504 to a1b01af Compare February 9, 2021 09:56
@doru91 doru91 force-pushed the feature/k32w_br_doc branch 3 times, most recently from 6f8265d to 142ba00 Compare February 9, 2021 11:45
@doru91 doru91 force-pushed the feature/k32w_br_doc branch 3 times, most recently from a0071bd to 2f30a02 Compare February 9, 2021 11:51
SDKs" tab; ![Select SDK](../../platform/k32w/doc/images/select-sdk.JPG)
- Import Project CHIP in MCUXpresso IDE as Makefile Project:
<i>File->Import->C/C++->Existing Code as Makefile Project-> Next </i>.
Select Project CHIP folder as the <i>Existing Code Location</i>. In the
Copy link
Contributor

Choose a reason for hiding this comment

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

This "Select Project CHIP folder as Existing Code Location" bit got lost in the rewording, looks like?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removed entirely this part and replaced with a link to the official openthread documentation.

<i>Toolchain for Indexer Settings </i> list, be sure to keep the setting to
<i>none</i>. Click <i>Finish</i>;
- Configure MCU Settings: right click on project from the workspace and go to
<i>Properties->C/C++ Build->MCU Settings</i>. Select K32W061 from the SDK
Copy link
Contributor

Choose a reason for hiding this comment

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

This "Select K32W061" bit also got lost?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removed entirely this part and replaced with a link to the official openthread documentation.

Comment on lines -196 to -198
Untick the <i>Display compatible toolchains only</i> checkbox. In the
drop-down menu named <i>Current toolchain</i>, select <i>NXP MCU Tools</i>.
Click <i>Apply and Close</i>;
Copy link
Contributor

Choose a reason for hiding this comment

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

These bits also got lost?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removed entirely this part and replaced with a link to the official openthread documentation.


3. Navigate to the _Form_ tab then push the _Form_ button using the default
parameters:
![nxp_form_nwk](../../examples/platform/k32w/doc/images/form_web.JPG)
Copy link
Contributor

Choose a reason for hiding this comment

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

Are the default parameters pre-filled, or is the expectation here that the developer will copy the values from the image? If the latter, the values should be presented as text somewhere; not everyone can see the image.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The default parameters are pre-filled.


![chiptool_main_screen](../../examples/platform/k32w/doc/images/chiptool_main_screen.png)

5. In the Thread network settings screen, use the default settings and tap the
Copy link
Contributor

Choose a reason for hiding this comment

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

Again, are these default settings pre-filled or to be copied from the image?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Pre-filled.

@doru91 doru91 force-pushed the feature/k32w_br_doc branch 5 times, most recently from 2356f21 to 1067e9d Compare February 10, 2021 17:20
@woody-apple
Copy link
Contributor

@saurabhst ?

@woody-apple
Copy link
Contributor

@doru91 can you restyle please?

Signed-off-by: Doru Gucea <doru-cristian.gucea@nxp.com>
@doru91 doru91 force-pushed the feature/k32w_br_doc branch from 1067e9d to 4e8457f Compare February 11, 2021 07:29
@doru91
Copy link
Contributor Author

doru91 commented Feb 11, 2021

@doru91 can you restyle please?

Restyled.

@doru91 doru91 requested a review from saurabhst February 11, 2021 09:39
@bzbarsky-apple bzbarsky-apple merged commit 76f067c into project-chip:master Feb 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation examples
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants