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

boards/telosb: changed default baudrate to 9600 #5215

Merged
merged 1 commit into from
Apr 1, 2016

Conversation

haukepetersen
Copy link
Contributor

supposedly fixes #4356

I don't have a board to test here, but I assume this should fix the issue of dropped characters on the telosb, as the main clock is currently configured with ~2MHz, and this is not enough to process UART interrupts at 115200baud...

@haukepetersen haukepetersen 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 labels Mar 31, 2016
@haukepetersen haukepetersen added this to the Release 2016.04 milestone Mar 31, 2016
@haukepetersen
Copy link
Contributor Author

added the new baudrate also as pyterm parameter to the Makefile and tested this -> confirmed working:

2016-04-01 11:22:47,794 - INFO # RIOT MSP430 hardware initialization complete.
2016-04-01 11:22:47,795 - INFO # 
2016-04-01 11:22:47,892 - INFO # main(): This is RIOT! (Version: 2016.03-devel-1546-g5c6aa-prag-fix_telosb_baudrate)
2016-04-01 11:22:47,912 - INFO # Welcome to RIOT!
help
2016-04-01 11:22:51,992 - INFO # > help
2016-04-01 11:22:52,030 - INFO # Command              Description
2016-04-01 11:22:52,076 - INFO # ---------------------------------------
2016-04-01 11:22:52,118 - INFO # reboot               Reboot the node
2016-04-01 11:22:52,191 - INFO # ps                   Prints information about running threads.
2016-04-01 11:22:52,270 - INFO # saul                 interact with sensors and actuators using SAUL
ps
2016-04-01 11:22:54,280 - INFO # > ps
2016-04-01 11:22:54,369 - INFO #    pid | name                 | state    Q | pri | stack ( used) | location   
2016-04-01 11:22:54,453 - INFO #      1 | idle                 | pending  Q |  15 |    96 (   60) | 0x1124 
2016-04-01 11:22:54,537 - INFO #      2 | main                 | running  Q |   7 |   512 (  384) | 0x1184 
2016-04-01 11:22:54,610 - INFO #        | SUM                  |            |     |   608 (  444)
helphelphelp
2016-04-01 11:22:57,739 - INFO # > helphelphelp
2016-04-01 11:22:57,785 - INFO # shell: command not found: helphelphelp

@haukepetersen haukepetersen added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Apr 1, 2016
@cgundogan
Copy link
Member

Untested ACK. The changes look sane, but I have no hardware around atm.

@cgundogan cgundogan merged commit 9f764cf into RIOT-OS:master Apr 1, 2016
@haukepetersen haukepetersen deleted the fix_telosb_baudrate branch April 1, 2016 11:08
@OlegHahm
Copy link
Member

OlegHahm commented Apr 4, 2016

Just out of curiosity: shouldn't work 38400 baud work as well? At least it does for the bootloader.

@haukepetersen
Copy link
Contributor Author

it probably would, but I decided to go with the safe value here. For the mid-term, we should re-work the clock initialization code for the msp430, let the telosb than run with something around 8MHz and then re-evaluate this value...

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.

boards/telosb: serial input loses characters
3 participants