Skip to content

Add TARGET_CY8CPROTO_063_BLE #11561

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Oct 11, 2019
Merged

Add TARGET_CY8CPROTO_063_BLE #11561

merged 1 commit into from
Oct 11, 2019

Conversation

@kyle-cypress
Copy link

Description

Add target to support the CY8CPROTO-063-BLE board.
This PR depends on #11542, and #11560 .

Greentea test report (with dependent PRs included):
GT_FT_PROTO_063_BLE_GCC.txt

Pull request type

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

Reviewers

@ARMmbed/team-cypress

Release Notes

@ciarmcom ciarmcom requested review from a team and maclobdell September 24, 2019 19:00
@ciarmcom
Copy link
Member

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

@adbridge
Copy link
Contributor

adbridge commented Oct 7, 2019

@maclobdell could you review this please

maclobdell
maclobdell approved these changes Oct 8, 2019
View reviewed changes
Copy link
Contributor

@maclobdell maclobdell left a comment

Choose a reason for hiding this comment

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

All changes are contained within the target areas for the new target. Once we receive boards, we can test it and officially "Mbed Enabled" it. Looks good to merge now.

@adbridge
Copy link
Contributor

CI started

@mbed-ci
Copy link

mbed-ci commented Oct 11, 2019

Test run: SUCCESS

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

@adbridge adbridge merged commit c094ad8 into ARMmbed:master Oct 11, 2019
@miteshdedhia7 miteshdedhia7 mentioned this pull request Oct 24, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

2 more reviewers

@bulislaw bulislaw bulislaw approved these changes

@maclobdell maclobdell maclobdell approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

release-version: 5.14.1 scope: new-target

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@kyle-cypress @ciarmcom @adbridge @mbed-ci @bulislaw @maclobdell