Skip to content

hal-qspi_test: remove STM workaround #7580

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

Conversation

maciejbocianski
Copy link
Contributor

Description

After feature-qspi branch rebase to latest master, everything works fine and workaround is not needed anymore.

@adustm can you confirm that test without workaround works fine on you side

Related PR: #7325

Pull request type

[X] Fix
[ ] Refactor
[ ] New target
[ ] Feature
[ ] Breaking change

after feature-qspi branch rebase, everything works fine and workaround is not needed anymore
@0xc0170 0xc0170 requested a review from adustm July 23, 2018 13:02
@cmonr
Copy link
Contributor

cmonr commented Jul 23, 2018

/morph build

@adustm
Copy link
Member

adustm commented Jul 23, 2018

Hello @maciejbocianski
Thanks for this PR removing the wait.

I've been able to pass the test tests-mbed_hal-qspi on DISCO_L475VG_IOT01A and the result is the same as with the wait.

Kind regards
Armelle

@mbed-ci
Copy link

mbed-ci commented Jul 23, 2018

Build : SUCCESS

Build number : 2657
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/7580/

Triggering tests

/morph test
/morph uvisor-test
/morph export-build
/morph mbed2-build

@mbed-ci
Copy link

mbed-ci commented Jul 23, 2018

@mbed-ci
Copy link

mbed-ci commented Jul 23, 2018

@cmonr cmonr merged commit 74307ff into ARMmbed:feature-qspi Jul 24, 2018
@0xc0170 0xc0170 removed the needs: CI label Jul 24, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants