Skip to content

Cypress: Misc target updates #12909

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 4 commits into from
May 12, 2020
Merged

Cypress: Misc target updates #12909

merged 4 commits into from
May 12, 2020

Conversation

kyle-cypress
Copy link

Summary of changes

  • Fix incorrect ADC range (was 0-2.4V, is now 0 - VDDA) (af39f05)
  • Remove CY8CPROTO_063_BLE (96f9915) and CYW943012P6EVB_01 (d058586) targets
  • Disable QSPIF on CY8CPROTO_062S3_4343W. It is incompatible with the XIP mode currently used for storing wifi firmware in external flash; instantiating a QSPIFBlockDevice would break Wifi firmware download. (a63fe7d)

Impact of changes

  • ADC range is changed for PSoC targets to align with documented API behavior.
  • CY8CPROTO_063_BLE and CYW943012P6EVB_01 targets are no longer available
  • QSPIF component is removed on CY8CPROTO_062S3_4343W

Migration actions required

  • Applications which utilize the analogin driver should update to handle the new range.
  • Applications which target CY8CPROTO_063_BLE or CYW943012P6EVB_01 should migrate to a new, still supported target, or remain on the mbed-os-5.15 release.
  • Applications on CY8CPROTO_062S3_4343W which need QSPIF and do not use wifi functionality can re-enable the QSPIF component in mbed_app.json.

Documentation

None

Pull request type 

[] Patch update (Bug fix / Target update / Docs update / Test update / Refactor)
[] Feature update (New feature / Functionality change / New API)
[x] Major update (Breaking change E.g. Return code change / API behaviour change)

Test results 

[] No Tests required for this change (E.g docs only update)
[] Covered by existing mbed-os tests (Greentea or Unittest)
[x] Tests / results supplied as part of this PR

GT_FT_KIT_062_BLE_GCC.txt
GT_FT_KIT_062_WIFI_BT_GCC.txt
GT_FT_KIT_062S2_43012_GCC.txt
GT_FT_P6S1_43012EVB_01_GCC_NET.txt
GT_FT_P6S1_43012EVB_01_GCC.txt
GT_FT_PROTO_062_4343W_GCC.txt
GT_FT_PROTO_062S3_4343W_GCC_NET.txt
GT_FT_PROTO_062S3_4343W.txt
GT_FT_PROTO_064_SB_GCC.txt

The sleep test failures are known issues that are being worked on.
The failure on CY8CPROTO_064_SB is an intermittent pyocd programming failure.
The failure of tests-integration-net-single on some boards is under investigation but is reproducible on master as well.

Reviewers

@ARMmbed/team-cypress

Kyle Kearney added 4 commits April 30, 2020 11:12
Disable QSPIF on CY8CPROTO_062S3_4343W
a63fe7d 
This target uses the QSPI interface hardware in XIP mode to allow
memory-mapped access to the WiFi interface firmware. This is
incompatible with the MMIO mode upon which QSPIFBlockDevice relies
to send specific commands over the QSPI bus.
Cypress: Fix non-compliant ADC range
af39f05 
The ADC range was previously 0-VBGR*2 (0 - 2.4 V). Change the reference
so that the ADC range is 0 - VDDA, to bring it in line with the documented
behavior of the mbed hal.
Remove CY8CPROTO_063_BLE target
96f9915
Remove CYW943012P6EVB_01 target
d058586
@ciarmcom ciarmcom requested review from maclobdell and a team April 30, 2020 21:00
@ciarmcom
Copy link
Member

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

@mergify mergify bot added needs: CI and removed needs: review labels May 2, 2020
@0xc0170
Copy link
Contributor

0xc0170 commented May 4, 2020

CI started

@mbed-ci
Copy link

mbed-ci commented May 4, 2020

Test run: SUCCESS

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

@0xc0170
Copy link
Contributor

0xc0170 commented May 4, 2020

Will merge after beta1

@kyle-cypress kyle-cypress mentioned this pull request May 7, 2020
Merged
@0xc0170 0xc0170 merged commit 15ade20 into ARMmbed:master May 12, 2020
@mergify mergify bot removed the ready for merge label May 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@0xc0170 0xc0170 0xc0170 approved these changes

@AnttiKauppila AnttiKauppila AnttiKauppila approved these changes

@maclobdell maclobdell Awaiting requested review from maclobdell

Assignees
No one assigned
Labels
release-version: 6.0.0-rc1
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@kyle-cypress @ciarmcom @0xc0170 @mbed-ci @AnttiKauppila