Skip to content
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

Update Cypress Targets and Assets #12167

Merged
merged 4 commits into from
Jan 3, 2020

Conversation

dustin-crossman
Copy link
Contributor

@dustin-crossman dustin-crossman commented Dec 23, 2019

Summary of changes

Update Cypress targets with latest tools/PDL library.
Update PSoC6 core_lib.
Update PSoC6 abstraction rtos.
Update PSoC6 PDL to version 1.4.0

Impact of changes

Migration actions required

Documentation

None


Pull request type

[x] Patch update (Bug fix / Target update / Docs update / Test update / Refactor)
[] Feature update (New feature / Functionality change / New API)
[] 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)
[x] Covered by existing mbed-os tests (Greentea or Unittest)
[] Tests / results supplied as part of this PR

GT_FT_KIT_062_BLE_GCC.txt
GT_FT_KIT_062S2_43012_GCC.txt
GT_FT_KIT_EVB_43012_GCC.txt
GT_FT_P6S1_43012EVB_01_GCC.txt
GT_FT_KIT_062_WIFI_BT_GCC.txt
GT_FT_PROTO_063_BLE_GCC.txt
GT_FT_PROTO_062_4343W_GCC.txt
GT_FT_PROTO_062S3_4343W.txt
GT_FT_P6S1_43438EVB_01_GCC.txt

Network tests:
GT-FT-KIT_062_WIFI_BT-GCC-NET.txt
GT-FT-KIT-062S2-43012-GCC-NET.txt
GT-FT-PROT-062-4343W-GCC-NET.txt
GT_FT_KIT_EVB_43012_GCC-NET.txt
GT_FT_P6S1_43438EVB_01_GCC_NET.txt

Test failures and explanations:

  • The pinmap test failure on the CYW9P6S21_43012EVB_01 is caused because 2 of the ARDUINO pins on that board are not connected. This is being evaluated for proper resolution in a future PR.
  • Kvstore failures on the CY8CPROTO_063_BLE were fixed by applying Improve reliability of KVStore general tests #12060.
  • The mbed_drivers timeout test failure seen on the CY8CPROTO_062_4343W is an intermittent failure seen occasionally on multiple boards. It is a bug in the test that will be addressed in a future PR.
  • All sleep failures are expected due to to a known incompatibility between the sleep tests and our UART driver.

Reviewers


@ciarmcom ciarmcom requested review from maclobdell and a team December 23, 2019 20:00
@ciarmcom
Copy link
Member

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

Copy link
Member

@bulislaw bulislaw left a comment

Choose a reason for hiding this comment

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

Simmed through the changes, nothing raised any flags.

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.

@dustin-crossman Can you add details to the commits what updates are we getting.

Usually this reflects the version (released or internal at least), to keep track what version we have. If updates in this PR are not versioned, at least information about what we are updating.

The description above contains some versions, they should be in the commits as well.

@dustin-crossman dustin-crossman force-pushed the pr/cypress_target_update branch from ba85bb9 to f53e10e Compare January 2, 2020 18:05
@dustin-crossman
Copy link
Contributor Author

@0xc0170 Fixed up commit messages!

@0xc0170
Copy link
Contributor

0xc0170 commented Jan 3, 2020

CI started

@mbed-ci
Copy link

mbed-ci commented Jan 3, 2020

Test run: FAILED

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

Failed test jobs:

  • jenkins-ci/mbed-os-ci_dynamic-memory-usage

@0xc0170
Copy link
Contributor

0xc0170 commented Jan 3, 2020

I restarted dynamic memory usage, we might having an issue there (not related to this PR).

cc @VeliMattiLahtela

@0xc0170 0xc0170 added release-version: 6.0.0-alpha-1 First pre-release version of 6.0.0 and removed needs: CI labels Jan 3, 2020
@0xc0170 0xc0170 merged commit 8310117 into ARMmbed:master Jan 3, 2020
@adbridge adbridge added release-version: 5.15.1 and removed release-version: 6.0.0-alpha-1 First pre-release version of 6.0.0 labels Feb 4, 2020
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