diff options
author | Dean Camera <[email protected]> | 2010-07-29 15:14:56 +0000 |
---|---|---|
committer | Dean Camera <[email protected]> | 2010-07-29 15:14:56 +0000 |
commit | ca007f91f2aa959a738649d35becb54cb1efc8b8 (patch) | |
tree | 37c2f79e6fc0c13981082d6cc5e4373473a50080 /Projects/Benito | |
parent | 62022252ea4d2763e15c1150367ca972bc4926bf (diff) | |
download | qmk_firmware-ca007f91f2aa959a738649d35becb54cb1efc8b8.tar.gz qmk_firmware-ca007f91f2aa959a738649d35becb54cb1efc8b8.zip |
Fixed Benito and USBtoSerial projects not turning off the USART before reconfiguring it, which could cause incorrect operation to occur (thanks to Bob Paddock).
Diffstat (limited to 'Projects/Benito')
-rw-r--r-- | Projects/Benito/Benito.c | 14 |
1 files changed, 11 insertions, 3 deletions
diff --git a/Projects/Benito/Benito.c b/Projects/Benito/Benito.c index 71b187bfb4..834554b4fc 100644 --- a/Projects/Benito/Benito.c +++ b/Projects/Benito/Benito.c @@ -235,10 +235,18 @@ void EVENT_CDC_Device_LineEncodingChanged(USB_ClassInfo_CDC_Device_t* const CDCI break; } - UCSR1A = (1 << U2X1); - UCSR1B = ((1 << RXCIE1) | (1 << TXEN1) | (1 << RXEN1)); - UCSR1C = ConfigMask; + /* Must turn off USART before reconfiguring it, otherwise incorrect operation may occur */ + UCSR1A = 0; + UCSR1B = 0; + UCSR1C = 0; + + /* Set the new baud rate before configuring the USART */ UBRR1 = SERIAL_2X_UBBRVAL(CDCInterfaceInfo->State.LineEncoding.BaudRateBPS); + + /* Reconfigure the USART in double speed mode for a wider baud rate range at the expense of accuracy */ + UCSR1A = (1 << U2X1); + UCSR1B = ((1 << RXCIE1) | (1 << TXEN1) | (1 << RXEN1)); + UCSR1C = ConfigMask; } /** ISR to manage the reception of data from the serial port, placing received bytes into a circular buffer |