Skip to content

STM32: avoid STM32 family name check #8960

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

Merged
merged 4 commits into from
Dec 7, 2018
Merged

Conversation

jeromecoutant
Copy link
Collaborator

Description

Hi

This code update should not affect current behavior.

Goal is to replace some STM32 family name check by some feature availability check.

This will make easier the new STM32 family intoduction

@bcostm for STM32H7 :-)

Pull request type

[x] Fix
[ ] Refactor
[ ] Target update
[ ] Functionality change
[ ] Docs update
[ ] Test update
[ ] Breaking change

@0xc0170 0xc0170 requested a review from a team December 4, 2018 10:30
@0xc0170
Copy link
Contributor

0xc0170 commented Dec 6, 2018

Still needs @ARMmbed/team-st-mcd approval

@cmonr
Copy link
Contributor

cmonr commented Dec 6, 2018

CI started

@mbed-ci
Copy link

mbed-ci commented Dec 6, 2018

Test run: FAILED

Summary: 1 of 7 test jobs failed
Build number : 1
Build artifacts

Failed test jobs:

  • jenkins-ci/mbed-os-ci_build-IAR

@cmonr
Copy link
Contributor

cmonr commented Dec 7, 2018

CI restarted.

Something odd happened to the build-IAR job.

@mbed-ci
Copy link

mbed-ci commented Dec 7, 2018

Test run: SUCCESS

Summary: 11 of 11 test jobs passed
Build number : 2
Build artifacts

@cmonr cmonr merged commit c4e913b into ARMmbed:master Dec 7, 2018
@jeromecoutant jeromecoutant deleted the PR_RTC2 branch December 10, 2018 11:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants