Skip to content

cpu: make msp430fxyz use uart stdio#3924

Merged
thomaseichinger merged 1 commit intoRIOT-OS:masterfrom
kaspar030:msp430fxyz_use_uart_stdio
Sep 25, 2015
Merged

cpu: make msp430fxyz use uart stdio#3924
thomaseichinger merged 1 commit intoRIOT-OS:masterfrom
kaspar030:msp430fxyz_use_uart_stdio

Conversation

@kaspar030
Copy link
Copy Markdown
Contributor

msp430fxyz used an ugly-hacked uart_stdio replacement as it was not fast enough to handle bytewise isr-to-thread transfer using ringbuffer at 115kbaud.

Using the improved thread-safe ringbuffer (which avoids interrupt disabling), the msp easily handles higher speeds.

Depends on #3923.

@kaspar030 kaspar030 added Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) Platform: MSP Platform: This PR/issue effects MSP-based platforms State: waiting for other PR State: The PR requires another PR to be merged first labels Sep 22, 2015
@kaspar030 kaspar030 added this to the Release 2015.09 milestone Sep 22, 2015
@kaspar030
Copy link
Copy Markdown
Contributor Author

Without this, everything but shell that using uart is probably broken, and shell loses input echo until "\r" is received.

@kaspar030 kaspar030 modified the milestones: Release NEXT MAJOR, Release 2015.09 Sep 22, 2015
@kaspar030 kaspar030 added CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR and removed State: waiting for other PR State: The PR requires another PR to be merged first labels Sep 25, 2015
@kaspar030 kaspar030 force-pushed the msp430fxyz_use_uart_stdio branch from a6a8ad4 to d9414fd Compare September 25, 2015 09:09
@kaspar030
Copy link
Copy Markdown
Contributor Author

  • rebased.

IMHO this fixes a lot of bugs and is a rather trivial change. Should we get it in the release?

@thomaseichinger
Copy link
Copy Markdown
Member

Successfully tested on z1 and msb-430h. I'm fine with taking this into this release.

@OlegHahm
Copy link
Copy Markdown
Member

👍

@kaspar030
Copy link
Copy Markdown
Contributor Author

Are those ACKs? Then merge at will.

@thomaseichinger
Copy link
Copy Markdown
Member

ACK & Go

thomaseichinger added a commit that referenced this pull request Sep 25, 2015
@thomaseichinger thomaseichinger merged commit 7b3d1a7 into RIOT-OS:master Sep 25, 2015
@OlegHahm OlegHahm modified the milestones: Release 2015.09, Release NEXT MAJOR Sep 25, 2015
@kaspar030 kaspar030 deleted the msp430fxyz_use_uart_stdio branch February 7, 2017 22:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Platform: MSP Platform: This PR/issue effects MSP-based platforms Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants