Skip to content

Cypress 5.14 rollup #11367

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 23 commits into from
Aug 29, 2019
Merged

Conversation

kyle-cypress
Copy link

Description

Rollup of several related PRs that modify Cypress targets, rebased onto latest master. This incorporates the changes from the following PRs:
#11321
#11323
#11327
#11320
#11328

Pull request type

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

Reviewers

@ARMmbed/team-cypress

Release Notes

Kyle Kearney and others added 22 commits August 28, 2019 10:56
Fix incorrect file names in a few headers
This avoids a stack overflow if sleep is called for the first time from
the idle thread (which by default has a fairly small stack, and which is
already fairly deep by the time it calls into the usticker adapter)
Enables code examples/end user applications to override if necessary
Add BSP_DESIGN_MODUS component by default to all PSOC6 boards. Applications can remove this if necessary.
For kit targets that inherit from a module, move the kit a top-level
target folder rather than nesting underneath the module folder, to
avoid too-long-path issues on Windows.
Note this only changes the folder layout, not the inheritance.
Include all configurator design files with each BSP
Only include the CM4 startup code and linker scripts with the target for each board.
Allow more flexibile configuration for BLE radio pins
Migrate to using ThisThread::sleep_for instead.
Fixes are for CY8CKIT_062S2_43012 and CY8CPROTO_062_4343W
Enable QSPI and Watchdog
Define additional macros to enable more generic BSP code
Allow deep sleep latency to be set by configurator-generated code
@ciarmcom ciarmcom requested review from maclobdell and a team August 28, 2019 19:00
@ciarmcom
Copy link
Member

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

@0xc0170 0xc0170 removed request for a team August 28, 2019 19:39
@0xc0170
Copy link
Contributor

0xc0170 commented Aug 28, 2019

CI started

PRs look good to me, PRs were reviewed separately

@mbed-ci
Copy link

mbed-ci commented Aug 29, 2019

Test run: FAILED

Summary: 3 of 4 test jobs failed
Build number : 1
Build artifacts

Failed test jobs:

  • jenkins-ci/mbed-os-ci_build-GCC_ARM
  • jenkins-ci/mbed-os-ci_build-ARM
  • jenkins-ci/mbed-os-ci_build-IAR

This was added to TARGET_PSOC6 which TARGET_SEQUANA inherits from,
but TARGET_SEQUANA doesn't implement WATCHDOG
@kyle-cypress
Copy link
Author

@kyle-cypress
Copy link
Author

Pushed 095e8df to address CI failure.

@0xc0170
Copy link
Contributor

0xc0170 commented Aug 29, 2019

CI restarted

@0xc0170
Copy link
Contributor

0xc0170 commented Aug 29, 2019

@kyle-cypress Once we integrate this one, what is the next , I can still few others opened

@mbed-ci
Copy link

mbed-ci commented Aug 29, 2019

Test run: SUCCESS

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

@0xc0170
Copy link
Contributor

0xc0170 commented Aug 29, 2019

One more travis restart and we should be good to go

@0xc0170
Copy link
Contributor

0xc0170 commented Aug 29, 2019

Rollup of several related PRs that modify Cypress targets, rebased onto latest master. This incorporates the changes from the following PRs:
#11321
#11323
#11327
#11320
#11328

They were closed. If these would have been added via merge , Github would automatically close them . I did manually all of them. Please verify

@kyle-cypress
Copy link
Author

@kyle-cypress Once we integrate this one, what is the next , I can still few others opened

Remaining from our side for PSoC targets are:
#11356
#11354
#11353
#11355 (not critical for 5.14)

There shouldn't be any ordering dependencies between these; all are independent changes and now apply cleanly on master.

@kyle-cypress
Copy link
Author

Rollup of several related PRs that modify Cypress targets, rebased onto latest master. This incorporates the changes from the following PRs:
#11321
#11323
#11327
#11320
#11328

They were closed. If these would have been added via merge , Github would automatically close them . I did manually all of them. Please verify

Looks good. Thanks!

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