Skip to content

PSOC6: update the psoc6pdl and HAL to the latest version #9762

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

Merged
merged 4 commits into from
Feb 20, 2019
Merged

PSOC6: update the psoc6pdl and HAL to the latest version #9762

merged 4 commits into from
Feb 20, 2019

Conversation

vmedcy
Copy link
Contributor

@vmedcy vmedcy commented Feb 19, 2019

Description

This PR brings numerous updates to the Cypress targets for Mbed OS 5.11.5:

  • Update psoc6pdl to the latest version delivered as part of PSoC 6 SDK with ModusToolbox 1.1
  • Minor updates to the Cypress PSoC 6 HAL and CYW43XXX HCIDriver (internal code review + astyle run)
  • Update BSP generated sources using the latest version of PSoC 6 MCU Device Configurator
  • Switch all BSPs to use WCO clock as LFCLK clock source.

Pull request type

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

Reviewers

Release Notes

Use ModusToolbox Device Configurator 1.1.0.284 to generate the
BSP low-level initialization code. Compatible version of Device
Configurator to be released with ModusToolbox 1.1.

Notable changes:
* rename cycfg_connectivity -> cycfg_routing
* switch LF_CLK clock source from ILO to WCO on
  CY8CPROTO-062-4343W and CYW943012P6EVB-01
@ciarmcom ciarmcom requested review from a team February 19, 2019 18:00
@ciarmcom
Copy link
Member

@vmedcy, thank you for your changes.
@ARMmbed/mbed-os-pan @ARMmbed/mbed-os-maintainers please review.

@cmonr
Copy link
Contributor

cmonr commented Feb 19, 2019

CI started while this undergoes reviews.

Lots of small changes, but curious to see what CI says in the meanwhile

Copy link
Contributor

@cmonr cmonr left a comment

Choose a reason for hiding this comment

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

Thank you for splitting and commenting the PRs.
Makes reviews go by with much less pain.

@cmonr
Copy link
Contributor

cmonr commented Feb 19, 2019

@ARMmbed/mbed-os-pan I believe y'all only need to review cefc745

@ARMmbed/mbed-os-crypto I'm unsure if you review partner crypto implementaions, but if so please take a look at edb944a

@mbed-ci
Copy link

mbed-ci commented Feb 20, 2019

Test run: SUCCESS

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

@yanesca
Copy link
Contributor

yanesca commented Feb 20, 2019

@cmonr We focus on the integration code with Mbed TLS and don't review partner crypto code. However, while reviewing the integration code we usually need to have a superficial look on the driver code. I can't see any integration code in this commit and I don't think we need to review it.

@cmonr
Copy link
Contributor

cmonr commented Feb 20, 2019

@yanesca Thanks for that clarification. Will keep it in mind moving forward.

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.

6 participants