-
Notifications
You must be signed in to change notification settings - Fork 3k
Cypress Cordio Driver Update #11518
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
Cypress Cordio Driver Update #11518
Conversation
@MukundGitHub, thank you for your changes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cypress Cordio Driver Update
- please extend the commit msg
What update is this about. I see Pull output change here, handling sleep, adding delay to deassert and update baudrate
Looks like multiple things changed in one commit.
Updated the Merge comment as requested. The changes are inter-connected - with enabling of 3M baud and logic for enabling Cypress BT Chip to enter Low Power state. The MCU Device Wake Pin is configured properly and some delays added when de-asserting the Device Wake Pin to BT Chip. |
Added capability to have BT Cordio stack working on Cypress BT chips without BT Device Wake and Host Wake Pins configured. |
@MukundGitHub Can you add this to the first commit:
It would be nice to have there some background. Otherwise it looks good to me! |
I don't see the requested change in the commit message, please let us know once done |
Updated the Description with details. Please accept the PR and Merge the changes. |
CI started |
Note to myself: include description in the merge commit when integrating |
Test run: SUCCESSSummary: 11 of 11 test jobs passed |
Description
This change applies to only to the HCI driver for Cypress WiFi+BT (4343W and 43012) combo chipsets.
The Cypress Specific HCI Cordio Driver has been updated to add:
These changes are inter-connected and hence being merged together.
Test: mbed-os-example-ble-LED application and mbed-os-example-ble-Beacon application have been tested with the changes. The applications run as expected.
Pull request type
Reviewers
Release Notes