Skip to content

[STM32F3] Add asynchronous serial #2461

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
Aug 23, 2016
Merged

[STM32F3] Add asynchronous serial #2461

merged 3 commits into from
Aug 23, 2016

Conversation

svastm
Copy link
Contributor

@svastm svastm commented Aug 16, 2016

This PR add the asynchronous serial for the following targets:

  • NUCLEO_F303RE
  • DISCO_F334C8
  • NUCLEO_F302R8
  • NUCLEO_F334R8

The struct serial_s move to common_objects.h.

test report

The BUILD_FAIL are due to .text oversize. Cpputest is relatively big.

@0xc0170
Copy link
Contributor

0xc0170 commented Aug 16, 2016

Please can you rebase?

@svastm
Copy link
Contributor Author

svastm commented Aug 16, 2016

Rebased.

@bridadan
Copy link
Contributor

/morph test

@bridadan
Copy link
Contributor

@mbed-bot: TEST

HOST_OSES=ALL
BUILD_TOOLCHAINS=ALL
TARGETS=NUCLEO_F303RE,DISCO_F334C8,NUCLEO_F302R8,NUCLEO_F334R8

@mbed-bot
Copy link

[Build 812]
SUCCESS: Building succeeded and tests were run! Be sure to check the test results

@bridadan
Copy link
Contributor

Just a note, the CI that just posted success (continuous-integration/mbedci) doesn't test much in this context (just ran a build of the HAL), so it'd be good to have more test results from this. But at least the HAL build looks good! 😄

@mbed-bot
Copy link

Result: SUCCESS

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

/morph test

Output

mbed Build Number: 656

All builds and test passed!

@0xc0170 0xc0170 merged commit 22acfbf into ARMmbed:master Aug 23, 2016
@svastm svastm deleted the serial_asynch_f3 branch September 16, 2016 09:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
release-version: 5.1.3
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@svastm @0xc0170 @bridadan @mbed-bot @sg-