aboutsummaryrefslogtreecommitdiffstats
path: root/Projects/Benito/Benito.c
diff options
context:
space:
mode:
authorDean Camera <dean@fourwalledcubicle.com>2010-07-29 15:14:56 +0000
committerDean Camera <dean@fourwalledcubicle.com>2010-07-29 15:14:56 +0000
commitca007f91f2aa959a738649d35becb54cb1efc8b8 (patch)
tree37c2f79e6fc0c13981082d6cc5e4373473a50080 /Projects/Benito/Benito.c
parent62022252ea4d2763e15c1150367ca972bc4926bf (diff)
downloadlufa-ca007f91f2aa959a738649d35becb54cb1efc8b8.tar.gz
lufa-ca007f91f2aa959a738649d35becb54cb1efc8b8.tar.bz2
lufa-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/Benito.c')
-rw-r--r--Projects/Benito/Benito.c14
1 files changed, 11 insertions, 3 deletions
diff --git a/Projects/Benito/Benito.c b/Projects/Benito/Benito.c
index 71b187bfb..834554b4f 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