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

STM32 : mbed_overrides.c is common for all families #4603

Merged
merged 1 commit into from Jun 27, 2017

Conversation

Projects
None yet
5 participants
@jeromecoutant
Contributor

jeromecoutant commented Jun 21, 2017

Description

mbed_overrides.c file is common for all STM32 families

Note that HAL_Delay redefinition for some family was not useful, so it can be removed.

Status

READY

Tests

OS2: OK
OS5: OK

@0xc0170

+1 for this reduction !

@0xc0170 0xc0170 added the needs: CI label Jun 21, 2017

@studavekar

This comment has been minimized.

Collaborator

studavekar commented Jun 21, 2017

/morph test

@mbed-bot

This comment has been minimized.

mbed-bot commented Jun 21, 2017

Result: FAILURE

Your command has finished executing! Here's what you wrote!

/morph test

Output

mbed Build Number: 601

Test failed!

@theotherjimmy

This comment has been minimized.

Contributor

theotherjimmy commented Jun 26, 2017

/morph test

@theotherjimmy

This comment has been minimized.

Contributor

theotherjimmy commented Jun 27, 2017

It looks like Cam-CI did not report success. @mazimkhan Is this true? Does it need to be restarted?

@mbed-bot

This comment has been minimized.

mbed-bot commented Jun 27, 2017

Result: SUCCESS

Your command has finished executing! Here's what you wrote!

/morph test

Output

mbed Build Number: 654

All builds and test passed!

@theotherjimmy

This comment has been minimized.

Contributor

theotherjimmy commented Jun 27, 2017

retest uvisor

@theotherjimmy theotherjimmy merged commit e907079 into ARMmbed:master Jun 27, 2017

4 checks passed

Cam-CI uvisor Build & Test Success
Details
ci/morph-test Job has completed
Details
continuous-integration/jenkins/pr-head This commit looks good
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@theotherjimmy

This comment has been minimized.

Contributor

theotherjimmy commented Jun 27, 2017

@jeromecoutant Thanks!

@jeromecoutant jeromecoutant deleted the jeromecoutant:PR_OVERRIDE branch Sep 27, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment