Skip to content

STM32: Remove compilation warnings #5327

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Oct 26, 2017
Merged

STM32: Remove compilation warnings #5327

merged 3 commits into from
Oct 26, 2017

Conversation

bcostm
Copy link
Contributor

@bcostm bcostm commented Oct 17, 2017 •
edited
Loading

Description

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

Status

READY

Migrations

NO

@bcostm bcostm changed the title STM32: Remove GCC_ARM compilation warnings STM32: Remove GCC_ARM/IAR compilation warnings Oct 18, 2017
@mbed-ci
Copy link

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
Copy link

mbed-ci commented Oct 18, 2017

Test : SUCCESS

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

@0xc0170
Copy link
Contributor

0xc0170 commented Oct 18, 2017

/morph uvisor-test

1 similar comment
@0xc0170
Copy link
Contributor

0xc0170 commented Oct 18, 2017

/morph uvisor-test

@0xc0170
Copy link
Contributor

0xc0170 commented Oct 18, 2017

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

@bcostm bcostm changed the title STM32: Remove GCC_ARM/IAR compilation warnings STM32: Remove compilation warnings Oct 18, 2017
@0xc0170
Copy link
Contributor

0xc0170 commented Oct 19, 2017

/morph build

1 similar comment
@studavekar
Copy link
Contributor

/morph build

@mbed-ci
Copy link

mbed-ci commented Oct 19, 2017

Build : FAILURE

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

@adbridge
Copy link
Contributor

[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
Copy link
Contributor

0xc0170 commented Oct 24, 2017

/morph build

@mbed-ci
Copy link

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
Copy link

mbed-ci commented Oct 24, 2017

Test : SUCCESS

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

@adbridge adbridge merged commit 38ba693 into ARMmbed:master Oct 26, 2017
@bcostm bcostm deleted the remove_warnings branch November 23, 2017 10:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@0xc0170 0xc0170 0xc0170 approved these changes

Assignees
No one assigned
Labels
release-version: 5.6.4
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@bcostm @mbed-ci @0xc0170 @studavekar @adbridge