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: Remove compilation warnings #5327

Merged
merged 3 commits into from Oct 26, 2017

Conversation

Projects
None yet
5 participants
@bcostm
Contributor

bcostm commented Oct 17, 2017

Description

Remove compilation warnings due to unused variables, missing parenthesis and identical left/right operands.

Status

READY

Migrations

NO

@bcostm bcostm changed the title from STM32: Remove GCC_ARM compilation warnings to STM32: Remove GCC_ARM/IAR compilation warnings Oct 18, 2017

@0xc0170 0xc0170 added the needs: CI label Oct 18, 2017

@mbed-ci

This comment has been minimized.

mbed-ci commented Oct 18, 2017

Build : SUCCESS

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

Triggering tests

/morph test
/morph uvisor-test

@mbed-ci

This comment has been minimized.

@0xc0170

This comment has been minimized.

Member

0xc0170 commented Oct 18, 2017

/morph uvisor-test

1 similar comment
@0xc0170

This comment has been minimized.

Member

0xc0170 commented Oct 18, 2017

/morph uvisor-test

@0xc0170

This comment has been minimized.

Member

0xc0170 commented Oct 18, 2017

@orenc17A Please see the latesat CI uVisor build, and restart

@bcostm bcostm changed the title from STM32: Remove GCC_ARM/IAR compilation warnings to STM32: Remove compilation warnings Oct 18, 2017

@0xc0170

This comment has been minimized.

Member

0xc0170 commented Oct 19, 2017

/morph build

1 similar comment
@studavekar

This comment has been minimized.

Collaborator

studavekar commented Oct 19, 2017

/morph build

@mbed-ci

This comment has been minimized.

mbed-ci commented Oct 19, 2017

@adbridge

This comment has been minimized.

Contributor

adbridge commented Oct 24, 2017

[DEBUG] Errors: Error: L6200E: Symbol __asm___7_CAN_cpp___Z7__REV16j multiply defined (by ./BUILD/MAX32620HSP/ARM/mbed-os/drivers/SPISlave.o and ./BUILD/MAX32620HSP/ARM/mbed-os/drivers/CAN.o).
[DEBUG] Errors: Error: L6200E: Symbol __asm___7_CAN_cpp___Z7__REVSHi multiply defined (by ./BUILD/MAX32620HSP/ARM/mbed-os/drivers/SPISlave.o and ./BUILD/MAX32620HSP/ARM/mbed-os/drivers/CAN.o).
[DEBUG] Errors: Error: L6200E: Symbol __asm___7_CAN_cpp___Z5__RRXj multiply defined (by ./BUILD/MAX32620HSP/ARM/mbed-os/drivers/SPISlave.o and ./BUILD/MAX32620HSP/ARM/mbed-os/drivers/CAN.o).
[DEBUG] Errors: Not enough information to list the image map.

Looks like a genuine build issue but not really related to this PR ?

@0xc0170

This comment has been minimized.

Member

0xc0170 commented Oct 24, 2017

/morph build

@mbed-ci

This comment has been minimized.

mbed-ci commented Oct 24, 2017

Build : SUCCESS

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

Triggering tests

/morph test
/morph uvisor-test

@mbed-ci

This comment has been minimized.

@0xc0170 0xc0170 added ready for merge and removed needs: CI labels Oct 24, 2017

@adbridge adbridge merged commit 38ba693 into ARMmbed:master Oct 26, 2017

5 checks passed

AWS-CI uVisor Build & Test Success
Details
ci-morph-build build completed
Details
ci-morph-test test completed
Details
continuous-integration/jenkins/pr-head This commit looks good
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@bcostm bcostm deleted the bcostm:remove_warnings branch Nov 23, 2017

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