[go: nahoru, domu]

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

[test] test whether the RCP supports data poll #10494

Merged
merged 1 commit into from
Jul 12, 2024

Conversation

zhanglongxia
Copy link
Contributor

This commit adds a test case to the cp-tests to test whether the RCP supports data poll transmitter and receiver.

Copy link
size-report bot commented Jul 9, 2024

Size Report of OpenThread

Merging #10494 into main(f32c18b).

name branch text data bss total
ot-cli-ftd main 466768 856 66364 533988
#10494 466768 856 66364 533988
+/- 0 0 0 0
ot-ncp-ftd main 435860 760 61592 498212
#10494 435860 760 61592 498212
+/- 0 0 0 0
libopenthread-ftd.a main 236060 95 40302 276457
#10494 236060 95 40302 276457
+/- 0 0 0 0
libopenthread-cli-ftd.a main 57580 0 8075 65655
#10494 57580 0 8075 65655
+/- 0 0 0 0
libopenthread-ncp-ftd.a main 32127 0 5924 38051
#10494 32127 0 5924 38051
+/- 0 0 0 0
ot-cli-mtd main 365392 760 51236 417388
#10494 365392 760 51236 417388
+/- 0 0 0 0
ot-ncp-mtd main 348108 760 46480 395348
#10494 348108 760 46480 395348
+/- 0 0 0 0
libopenthread-mtd.a main 158789 0 25190 183979
#10494 158789 0 25190 183979
+/- 0 0 0 0
libopenthread-cli-mtd.a main 39795 0 8059 47854
#10494 39795 0 8059 47854
+/- 0 0 0 0
libopenthread-ncp-mtd.a main 25007 0 5924 30931
#10494 25007 0 5924 30931
+/- 0 0 0 0
ot-cli-ftd-br main 550744 864 131228 682836
#10494 550744 864 131228 682836
+/- 0 0 0 0
libopenthread-ftd-br.a main 324381 100 105142 429623
#10494 324381 100 105142 429623
+/- 0 0 0 0
libopenthread-cli-ftd-br.a main 71524 0 8099 79623
#10494 71524 0 8099 79623
+/- 0 0 0 0
ot-rcp main 62312 564 20636 83512
#10494 62312 564 20636 83512
+/- 0 0 0 0
libopenthread-rcp.a main 9734 0 5060 14794
#10494 9734 0 5060 14794
+/- 0 0 0 0
libopenthread-radio.a main 18991 0 222 19213
#10494 18991 0 222 19213
+/- 0 0 0 0

Copy link
Contributor
@superwhd superwhd left a comment

Choose a reason for hiding this comment

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

LGTM overall

tools/cp-caps/rcp_caps_test.py Outdated Show resolved Hide resolved
tools/cp-caps/rcp_caps_test.py Outdated Show resolved Hide resolved
tools/cp-caps/rcp_caps_test.py Show resolved Hide resolved
tools/cp-caps/rcp_caps_test.py Outdated Show resolved Hide resolved
tools/cp-caps/README.md Outdated Show resolved Hide resolved

```bash
$ DUT_ADB_USB=1269UCKFZTAM95OR REF_CLI_SERIAL=/dev/ttyACM0 python3 ./tools/cp-caps/rcp_caps_test.py -p
Data Poll Receiver --------------------------------------- OK
Copy link
Member

Choose a reason for hiding this comment

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

I think any FTD device should be able to act as a parent and therefore should be capable of processing data polls, so I guess this would be equivalent to is this device FTD?

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, FTD device should be able to act as a parent and therefore should be capable of processing data polls. This test is mainly used to test the device is indeed support data polls.

Copy link
Member
@abtink abtink left a comment

Choose a reason for hiding this comment

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

LGTM. 👍

@zhanglongxia zhanglongxia force-pushed the cp-test-data-poll branch 2 times, most recently from ead3785 to 909a6f4 Compare July 12, 2024 05:48
Copy link
Contributor
@Irving-cl Irving-cl left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@jwhui jwhui merged commit 20c9bcb into openthread:main Jul 12, 2024
103 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.

None yet

5 participants