Skip to content

Cypress Asset Update #14062

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 14 commits into from
Jan 11, 2021
Merged

Conversation

dustin-crossman
Copy link
Contributor

Summary of changes

This updates a number of Cypress assets to their most recently released versions:

  • udb-sdio-whd to 1.1.1.19666
  • whd-bsp-integration to 1.1.0.19214
  • serial-flash to 1.0.2.17634
  • psoc6hal to 1.4.0.17634
  • core-lib to 1.1.4.17634
  • abstraction-rtos to 1.3.0.17634
  • abstraction-resource to 1.0.0.17634
  • psoc6pdl to 2.0.0.6211

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

GT_FT_P6S1_43438EVB_01_GCC_NET.txt
GT_FT_PROTO_062_4343W_GCC.txt
GT_FT_PROTO_062S3_4343W.txt
GT_FT_PROTO_062S3_4343W_GCC_NET.txt
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_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.txt
GT_FT_P6S1_43012EVB_01_GCC_NET.txt
GT_FT_P6S1_43438EVB_01_GCC.txt

Test Failures and Explanations:

  • connectivity-feature_ble-source-cordio-tests-cordio_hci-driver on multiple boards: Known failure. Fixed in PR: ble cyw43xxx: dev/hci driver terminate #14009
  • storage-blockdevice-tests-tests-blockdevice-general_block_device on multiple boards: Fails on master. Will investigate.
  • connectivity-netsocket-tests-tests-network-wifi on CY8CKIT-062S2-43012: Intermittent failure. Passes when run manually.
[] 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

Reviewers


@ciarmcom
Copy link
Member

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

@ciarmcom ciarmcom requested review from a team December 17, 2020 01:00
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.

targets/TARGET_Cypress/TARGET_PSOC6/COMPONENT_WHD/inc/whd.h and someo ther files contain permission changes (from 100644 to 100755), please review

@mergify mergify bot dismissed 0xc0170’s stale review December 17, 2020 17:10

Pull request has been modified.

@dustin-crossman
Copy link
Contributor Author

@0xc0170 Fixed up a bunch of file modes.

@0xc0170 0xc0170 self-requested a review January 5, 2021 13:02
0xc0170
0xc0170 previously approved these changes Jan 5, 2021
@mergify mergify bot added the needs: CI label Jan 5, 2021
@mergify mergify bot removed the needs: review label Jan 5, 2021
@0xc0170
Copy link
Contributor

0xc0170 commented Jan 5, 2021

CI started

@mergify mergify bot added needs: work and removed needs: CI labels Jan 5, 2021
@mbed-ci
Copy link

mbed-ci commented Jan 5, 2021

Jenkins CI Test : ❌ FAILED

Build Number: 1 | 🔒 Jenkins CI Job | 🌐 Logs & Artifacts

CLICK for Detailed Summary

jobs Status
jenkins-ci/mbed-os-ci_cmake-example-GCC_ARM ✔️
jenkins-ci/mbed-os-ci_cmake-example-ARM ✔️
jenkins-ci/mbed-os-ci_unittests ✔️
jenkins-ci/mbed-os-ci_build-greentea-ARM
jenkins-ci/mbed-os-ci_build-greentea-GCC_ARM
jenkins-ci/mbed-os-ci_build-example-ARM
jenkins-ci/mbed-os-ci_build-example-GCC_ARM
jenkins-ci/mbed-os-ci_build-cloud-example-GCC_ARM ✔️
jenkins-ci/mbed-os-ci_build-cloud-example-ARM ✔️

@0xc0170
Copy link
Contributor

0xc0170 commented Jan 5, 2021

There are compilation errors, see the logs. I found at least one:

Compile [ 80.6%]: whd_resources.c
[Fatal Error] whd_resources.c@22,10: 'wifi_nvram_image.h' file not found

Add label so nvram image is found for 064B0S2 board.
@dustin-crossman
Copy link
Contributor Author

@0xc0170 Fixed up compile errors.

@mergify mergify bot dismissed 0xc0170’s stale review January 5, 2021 20:56

Pull request has been modified.

@adbridge
Copy link
Contributor

adbridge commented Jan 7, 2021

@0xc0170 could you re-review please, I'll start CI in the meantime

@mbed-ci
Copy link

mbed-ci commented Jan 7, 2021

Jenkins CI Test : ✔️ SUCCESS

Build Number: 2 | 🔒 Jenkins CI Job | 🌐 Logs & Artifacts

CLICK for Detailed Summary

jobs Status
jenkins-ci/mbed-os-ci_unittests ✔️
jenkins-ci/mbed-os-ci_cmake-example-ARM ✔️
jenkins-ci/mbed-os-ci_build-greentea-ARM ✔️
jenkins-ci/mbed-os-ci_cmake-example-GCC_ARM ✔️
jenkins-ci/mbed-os-ci_build-greentea-GCC_ARM ✔️
jenkins-ci/mbed-os-ci_build-example-GCC_ARM ✔️
jenkins-ci/mbed-os-ci_build-cloud-example-GCC_ARM ✔️
jenkins-ci/mbed-os-ci_build-example-ARM ✔️
jenkins-ci/mbed-os-ci_build-cloud-example-ARM ✔️
jenkins-ci/mbed-os-ci_cmake-example-test ✔️
jenkins-ci/mbed-os-ci_greentea-test ✔️

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.

7 participants