-
Notifications
You must be signed in to change notification settings - Fork 3k
Revert "NUCLEO_F767ZI: Add bootloader support" #5927
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
Conversation
This reverts commit 4d0535a. Pull request introduced CI bug that causes a serial sync error once greentea tests are started.
/morph test |
@studavekar @0xc0170 @theotherjimmy @adbridge Feel free to check this over. Immediately jumping to testing this PR. |
@bcostm, keeping you in the loop. |
Test : FAILUREBuild number : 771 |
I promise that previous morph command was suppose to be build... /morph build |
I don't think this is the PR you are looking for. "bootloader support" just allows you to change the start and size of the application. |
Build : SUCCESSBuild number : 942 Triggering tests/morph test |
Looks like root cause cache setting may be :: https://github.com/ARMmbed/mbed-os/pull/5821/files#diff-862e5175fcbe6ee18fd2f54787555ec7R38 |
Exporter Build : SUCCESSBuild number : 630 |
Test : FAILUREBuild number : 772 |
/morph test |
/morph uvisor-test |
I have also changed the Cache initialization otherwise the application programmed by the bootloader was not running. But the ci-test was pass finally in PR #5821. |
In the above descriptions by @cmonr , you need to run the test 2x to get the failure from my understanding. |
Test : FAILUREBuild number : 773 |
Looking at the changes this does not revert the cache settings (entire patch), sending a new one |
Reverts commit 4d0535a.
Pull request introduced CI bug that causes a serial sync error once greentea tests are started.
Description
After investigation, it was found that the NUCLEO_F767ZI will not sync with greentea, causing our CI tests to slow down and continuously fail. Reverting prior to when bootloader support was added, and reflashing the device appears to solve the problem.
If this is not the only commit causing the issue, other commits will be reverted in this PR.
Status
READY
Migrations
For the time being, this will disable bootloader support for NUCLEO_F767ZI.
Steps to test or reproduce
To reproduce the issue:
To recover the device: