Skip to content
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: Build Fails If Serial Device is Removed #8372

Closed
mattbrown015 opened this issue Oct 11, 2018 · 3 comments
Closed

STM32: Build Fails If Serial Device is Removed #8372

mattbrown015 opened this issue Oct 11, 2018 · 3 comments

Comments

@mattbrown015
Copy link
Contributor

mattbrown015 commented Oct 11, 2018

Description

Removing the serial device causes the build to fail because of changes made in STM32: check for UART ongoing transfers before entering deepsleep #7717.

This feels like a minor issue as I guess it is pretty unusual for anyone to remove the serial device. But, it is supported and the problem occurs because serial_api.c is guarded by #if DEVICE_SERIAL.

I noticed because I was messing around wondering what effect removing devices would have on power consumption and code size.

Either the use of serial_is_tx_ongoing should be guarded or a stub provided.

Issue request type

[ ] Question
[ ] Enhancement
[X] Bug

@bcostm
Copy link
Contributor

bcostm commented Oct 12, 2018

Hi
Can you please check if PR #8403 fixes this issue ?

@mattbrown015
Copy link
Contributor Author

Hi @bcostm,

I tried your change from #8403, it worked for me. :-)

I wondered if you needed to repeat the comment, including its 2 spelling mistakes, though?? (I'm sure Skitt's Law applies here!)

@bcostm
Copy link
Contributor

bcostm commented Oct 13, 2018

OK thanks for confirming. I was thinking the same thing to copy or not the comment... ;-) I'll remove it and fix the typo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants