Skip to content

Update to latest psoc6csp #11516

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
Sep 20, 2019
Merged

Update to latest psoc6csp #11516

merged 4 commits into from
Sep 20, 2019

Conversation

kyle-cypress
Copy link

Description

Update Cypress targets to the latest PSoC 6 CSP (HAL, RTOS abstraction, Resource Abstraction).
This update includes:

  • Minor bugfixes/code cleanup
  • Re-organization of resource/rtos abstraction folder structure
  • Documentation improvements

Greentea test logs:
CY8CKIT_062_BLE-GCC_ARM.txt
CY8CKIT_062S2_43012-GCC_ARM.txt
CY8CKIT_062S2_43012-GCC_ARM-NET.txt
CY8CKIT_062_WIFI_BT-GCC_ARM.txt
CY8CPROTO_062_4343W-GCC_ARM.txt
CY8CPROTO_062_4343W-GCC_ARM-NET.txt
CYW943012P6EVB_01-GCC_ARM.txt
CYW943012P6EVB_01-GCC_ARM-NET.txt

Pull request type

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

Reviewers

@ARMmbed/team-cypress

Release Notes

Kyle Kearney added 4 commits September 18, 2019 12:06
Replace with multiplication by the floating point reciprocal, to
 produce a floating point result while being more efficient than
 floating point division.
Addresses the issue raised by kjbracey-arm in
ARMmbed#11324 (review)
- Add const and static qualifiers in places where they are
  applicable but missing
- Remove headers for drivers that aren't implemented yet
- Misc minor bugfixes
- Move resource and rtos abstractions into their own folders
- Remove files for abstractions that are not implemented
@ciarmcom ciarmcom requested review from maclobdell and a team September 18, 2019 21:00
@ciarmcom
Copy link
Member

@kyle-cypress, thank you for your changes.
@maclobdell @ARMmbed/mbed-os-maintainers please review.

Copy link
Contributor

@0xc0170 0xc0170 left a comment

Choose a reason for hiding this comment

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

LGTM

Future reference, fix like in 1713e79 should be a separate PR (If it was not on the top of the PR I might have overlooked it easily).

@kyle-cypress
Copy link
Author

LGTM

Future reference, fix like in 1713e79 should be a separate PR (If it was not on the top of the PR I might have overlooked it easily).

Thanks, will keep that in mind for the future.

@0xc0170
Copy link
Contributor

0xc0170 commented Sep 20, 2019

CI started

@mbed-ci
Copy link

mbed-ci commented Sep 20, 2019

Test run: SUCCESS

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

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.

4 participants