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.

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

CY8CKIT_062S2_43012 Test Fails #13076

Closed
maclobdell opened this issue Jun 6, 2020 · 16 comments · Fixed by #14982
Closed

CY8CKIT_062S2_43012 Test Fails #13076

maclobdell opened this issue Jun 6, 2020 · 16 comments · Fixed by #14982
Labels
devices: cypress type: bug

Comments

@maclobdell
Copy link
Contributor

maclobdell commented Jun 6, 2020 •
edited
Loading

Description of defect

The following test result in FAIL with CY8CKIT_062S2_43012 and Mbed OS 6.0.

| CY8CKIT_062S2_43012-ARMC6   | tests-mbed_hal-sleep                                                         | FAIL   |  16.98 |
| CY8CKIT_062S2_43012-ARMC6   | tests-mbed_hal-sleep_manager                                                 | FAIL   |  15.33 |
| CY8CKIT_062S2_43012-ARMC6   | features-feature_ble-targets-target_cordio-tests-cordio_hci-driver           | FAIL   |  34.17 |
| CY8CKIT_062S2_43012-GCC_ARM | tests-mbed_hal-sleep_manager                                                 | FAIL   |  15.46 |
| CY8CKIT_062S2_43012-GCC_ARM | tests-mbed_hal-sleep                                                         | FAIL   |  16.99 |
| CY8CKIT_062S2_43012-GCC_ARM | features-feature_ble-targets-target_cordio-tests-cordio_hci-driver           | FAIL   |  34.16 |

See full test summaries:
CY8CKIT_062S2_43012_GCC_ARM_results.txt
CY8CKIT_062S2_43012_ARM_results.txt

Target(s) affected by this defect ?

CY8CKIT_062S2_43012

Toolchain(s) (name and version) displaying this defect ?

GCC Arm compiler 9
Arm compiler 6

What version of Mbed-os are you using (tag or sha) ?

mbed-os-6.0.0

What version(s) of tools are you using. List all that apply (E.g. mbed-cli)

mbed-cli 1.10.1
mbed-cloud-sdk 2.0.8
mbed-flasher 0.10.1
mbed-greentea 1.7.4
mbed-host-tests 1.5.10
mbed-ls 1.7.10
mbed-os-tools 0.0.12
full list of python packages

How is this defect reproduced ?

mbed import https://github.com/armmbed/mbed-os
mbed test -m CY8CKIT_062S2_43012 -t GCC_ARM
mbed test -m CY8CKIT_062S2_43012 -t ARM
@ciarmcom
Copy link
Member

ciarmcom commented Jun 6, 2020

@maclobdell thank you for raising this issue.Please take a look at the following comments:

We cannot automatically identify a release based on the version of Mbed OS that you have provided.
Please provide either a single valid sha of the form #abcde12 or #3b8265d70af32261311a06e423ca33434d8d80de
or a single valid release tag of the form mbed-os-x.y.z .

NOTE: If there are fields which are not applicable then please just add 'n/a' or 'None'.This indicates to us that at least all the fields have been considered.
Please update the issue header with the missing information, the issue will not be mirroredto our internal defect tracking system or investigated until this has been fully resolved.

@0xc0170
Copy link
Contributor

0xc0170 commented Jun 8, 2020

cc @ARMmbed/team-cypress

@kyle-cypress
Copy link

Not sure about the cordio failures yet, but the sleep failures are probably the same as #12434 (known issue, fix in progress).

@ciarmcom
Copy link
Member

@maclobdell it has been 5 days since the last reminder. Could you please update the issue header as previously requested?

7 similar comments
@ciarmcom
Copy link
Member

@maclobdell it has been 5 days since the last reminder. Could you please update the issue header as previously requested?

@ciarmcom
Copy link
Member

@maclobdell it has been 5 days since the last reminder. Could you please update the issue header as previously requested?

@ciarmcom
Copy link
Member

@maclobdell it has been 5 days since the last reminder. Could you please update the issue header as previously requested?

@ciarmcom
Copy link
Member

ciarmcom commented Jul 1, 2020

@maclobdell it has been 5 days since the last reminder. Could you please update the issue header as previously requested?

@ciarmcom
Copy link
Member

ciarmcom commented Jul 6, 2020

@maclobdell it has been 5 days since the last reminder. Could you please update the issue header as previously requested?

@ciarmcom
Copy link
Member

@maclobdell it has been 5 days since the last reminder. Could you please update the issue header as previously requested?

@ciarmcom
Copy link
Member

@maclobdell it has been 5 days since the last reminder. Could you please update the issue header as previously requested?

@kyle-cypress kyle-cypress mentioned this issue Jul 16, 2020
Closed
@ciarmcom
Copy link
Member

@maclobdell it has been 5 days since the last reminder. Could you please update the issue header as previously requested?

@maclobdell
Copy link
Contributor Author

I updated the tag to mbed-os-6.0.0. Hopefully that gets the script off my case.

@ciarmcom
Copy link
Member

ciarmcom commented Oct 2, 2020

Thank you for raising this detailed GitHub issue. I am now notifying our internal issue triagers.
Internal Jira reference: https://jira.arm.com/browse/IOTOSM-2268

@ARMmbed ARMmbed deleted a comment from ciarmcom Oct 6, 2020
@adbridge
Copy link
Contributor

Not sure about the cordio failures yet, but the sleep failures are probably the same as #12434 (known issue, fix in progress).

@kyle-cypress as #12434 is now fixed can this be closed ?

@kyle-cypress
Copy link

Yes, I believe this can be closed. The Cordio failure can be tracked in #13295.

@0xc0170 0xc0170 closed this as completed Nov 17, 2020
@npal-cy npal-cy mentioned this issue Aug 5, 2021
Merged
npal-cy pushed a commit to npal-cy/mbed-os that referenced this issue Aug 6, 2021
Fixed MCU deep-sleep locking flow in CYW43XXX Cordio HCI driver. The …
1d6843e 
…CYW43XXX Cordio HCI driver incorrect handles lock/unlock deep-sleep entry, as results it allows to put PSoC6 in to deep-sleep when CYW43xx does not expect this (the host_wake_up pin in ACTIVE_LOW state, which means hat host must be in ACTIVE mode ).

Now, CYW43XXX Cordio HCI driver has the following MCU deep-sleep locking scenarios/flows:
a) A BT device or MCU does not support Low Power mode (MBED configuration does not include MBED_TICKLESS, DEVICE_SLEEP, DEVICE_LPTICKER or CYCFG_BT_LP_ENABLED features). In this case, CyH4TransportDriver locks deep-sleep in the initialize() function and unlocks the terminate() function.

b) A BT device and MCU support Low Power mode. In this case, the control of the unlock/lock of the deep-sleep functionality will be done in bt_host_wake_rise_irq_handler and bt_host_wake_fall_irq_handler handlers. Finally, CyH4TransportDriver unlocks the deep-sleep in terminate() function (if it was locked before) by checking the bt_host_wake_active flag.

Fixes ARMmbed#13076, ARMmbed#13700, ARMmbed#13295.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
devices: cypress type: bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

CYW43XXX Cordio HCI driver: fixed MCU deep-sleep locking flow npal-cy/mbed-os
5 participants
@0xc0170 @maclobdell @adbridge @ciarmcom @kyle-cypress