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

Allow WhdAccessPoint scan results with extended parameters #11662

Merged
merged 1 commit into from Oct 24, 2019

Conversation

@cydriftcloud
Copy link
Contributor

cydriftcloud commented Oct 10, 2019

Add WhdAccessPoint to include additional WHD scan info

  • To save memory, only move assignment is supported for WhdAccessPoint
  • Add scan_whd to scan for WhdAccessPoint with extended parameters such as BSS_TYPE, IE_DATA and IE_LEN
  • Set set_blocking(false) to unsupported by return NSAPI_ERROR_UNSUPPORTED

Description

Pull request type

[x] Fix
[ ] Refactor
[ ] Target update
[ ] Functionality change
[ ] Docs update
[ ] Test update
[ ] Breaking change

Reviewers

Release Notes

@ciarmcom ciarmcom requested review from maclobdell and ARMmbed/mbed-os-maintainers Oct 10, 2019
@ciarmcom

This comment has been minimized.

Copy link
Member

ciarmcom commented Oct 10, 2019

@cydriftcloud

This comment has been minimized.

Copy link
Contributor Author

cydriftcloud commented Oct 18, 2019

@maclobdell Please help to review. Thanks.

Copy link
Member

0xc0170 left a comment

otherwise looks fine to me

- Add WhdAccessPoint to include additional WHD scan info
- To save memory, only move assignment is supported for WhdAccessPoint
- Add scan_whd to scan for WhdAccessPoint
- Set set_blocking(false) to unsupported by return NSAPI_ERROR_UNSUPPORTED
@cydriftcloud cydriftcloud force-pushed the cydriftcloud:pr-dev/sta-extended-scan branch from f0084d8 to 7b16166 Oct 23, 2019
Copy link
Contributor Author

cydriftcloud left a comment

License header added.

@0xc0170

This comment has been minimized.

Copy link
Member

0xc0170 commented Oct 24, 2019

CI started while @ARMmbed/mbed-os-ipcore reviews

@mbed-ci

This comment has been minimized.

Copy link

mbed-ci commented Oct 24, 2019

Test run: SUCCESS

Summary: 11 of 11 test jobs passed
Build number : 1
Build artifacts

@0xc0170 0xc0170 merged commit ab931ee into ARMmbed:master Oct 24, 2019
26 checks passed
26 checks passed
continuous-integration/jenkins/pr-head This commit looks good
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
jenkins-ci/build-ARM Success
Details
jenkins-ci/build-GCC_ARM Success
Details
jenkins-ci/build-IAR Success
Details
jenkins-ci/cloud-client-test Success
Details
jenkins-ci/dynamic-memory-usage RTOS ROM(+0 bytes) RAM(+0 bytes)
Details
jenkins-ci/exporter Success
Details
jenkins-ci/greentea-test Success
Details
jenkins-ci/mbed2-build-ARM Success
Details
jenkins-ci/mbed2-build-GCC_ARM Success
Details
jenkins-ci/mbed2-build-IAR Success
Details
jenkins-ci/unittests Success
Details
travis-ci/astyle Success!
Details
travis-ci/docs Success!
Details
travis-ci/doxy-spellcheck Success!
Details
travis-ci/events Success! Runtime is 8692 cycles.
Details
travis-ci/gitattributestest Success!
Details
travis-ci/include_check Success!
Details
travis-ci/licence_check Success!
Details
travis-ci/littlefs Success! Code size is 8420B.
Details
travis-ci/psa-autogen Success!
Details
travis-ci/tools-py2.7 Success!
Details
travis-ci/tools-py3.5 Success!
Details
travis-ci/tools-py3.6 Success!
Details
travis-ci/tools-py3.7 Success!
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants
You can’t perform that action at this time.