Skip to content

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

Closed
wants to merge 1 commit into from

Conversation

cmonr
Copy link
Contributor

@cmonr cmonr commented Jan 25, 2018

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:

  • Download mbed-os at 98611c8
  • Compile, flash, and test on NUCLEO_F746ZG.
  • Power cycle the device.
  • Run test again. Sync will fail.

To recover the device:

This reverts commit 4d0535a.

Pull request introduced CI bug that causes a serial sync error once greentea
tests are started.
@cmonr
Copy link
Contributor Author

cmonr commented Jan 25, 2018

/morph test

@cmonr
Copy link
Contributor Author

cmonr commented Jan 25, 2018

@studavekar @0xc0170 @theotherjimmy @adbridge Feel free to check this over.

Immediately jumping to testing this PR.

@cmonr
Copy link
Contributor Author

cmonr commented Jan 25, 2018

@bcostm, keeping you in the loop.

@mbed-ci
Copy link

mbed-ci commented Jan 25, 2018

@cmonr
Copy link
Contributor Author

cmonr commented Jan 25, 2018

I promise that previous morph command was suppose to be build...

/morph build

@theotherjimmy
Copy link
Contributor

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.

@mbed-ci
Copy link

mbed-ci commented Jan 25, 2018

Build : SUCCESS

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

Triggering tests

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

@studavekar
Copy link
Contributor

@mbed-ci
Copy link

mbed-ci commented Jan 25, 2018

@mbed-ci
Copy link

mbed-ci commented Jan 25, 2018

@0xc0170
Copy link
Contributor

0xc0170 commented Jan 25, 2018

/morph test

@0xc0170
Copy link
Contributor

0xc0170 commented Jan 25, 2018

/morph uvisor-test

@bcostm
Copy link
Contributor

bcostm commented Jan 25, 2018

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.

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.

@0xc0170
Copy link
Contributor

0xc0170 commented Jan 25, 2018

In the above descriptions by @cmonr , you need to run the test 2x to get the failure from my understanding.

@mbed-ci
Copy link

mbed-ci commented Jan 25, 2018

Test : FAILURE

Build number : 773
Test logs :http://mbed-os-logs.s3-website-us-west-1.amazonaws.com/?prefix=logs/5927/773

@0xc0170
Copy link
Contributor

0xc0170 commented Jan 25, 2018

Looking at the changes this does not revert the cache settings (entire patch), sending a new one

@0xc0170 0xc0170 closed this Jan 25, 2018
@0xc0170 0xc0170 removed the needs: CI label Jan 25, 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.

6 participants