summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTsiChung Liew <Tsi-Chung.Liew@freescale.com>2008-05-29 12:21:54 -0500
committerJohn Rigby <jrigby@freescale.com>2008-07-11 10:45:56 -0600
commit81cc32322acb1b3225ee45606ced48e2a14824dc (patch)
tree27484e115603a42a3f470d0ac471157ff8329e79
parent9b55a2536919f4de1bb1044e6eb8262c2f53bc96 (diff)
downloadu-boot-imx-81cc32322acb1b3225ee45606ced48e2a14824dc.zip
u-boot-imx-81cc32322acb1b3225ee45606ced48e2a14824dc.tar.gz
u-boot-imx-81cc32322acb1b3225ee45606ced48e2a14824dc.tar.bz2
ColdFire: Fix UART baudrate formula
The formula "counter = (u32) (gd->bus_clk / gd->baudrate) / 32" can generate the wrong divisor due to integer division truncation. Round the calculated divisor value by adding 1/2 the baudrate before dividing by the baudrate. Signed-off-by: TsiChung Liew <Tsi-Chung.Liew@freescale.com> Acked-by: Gerald Van Baren <vanbaren@cideas.com>
-rw-r--r--drivers/serial/mcfuart.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/serial/mcfuart.c b/drivers/serial/mcfuart.c
index 88f3eb1..5eb4f45 100644
--- a/drivers/serial/mcfuart.c
+++ b/drivers/serial/mcfuart.c
@@ -63,8 +63,8 @@ int serial_init(void)
uart->umr = UART_UMR_SB_STOP_BITS_1;
/* Setting up BaudRate */
- counter = (u32) (gd->bus_clk / (gd->baudrate));
- counter >>= 5;
+ counter = (u32) ((gd->bus_clk / 32) + (gd->baudrate / 2));
+ counter = counter / gd->baudrate;
/* write to CTUR: divide counter upper byte */
uart->ubg1 = (u8) ((counter & 0xff00) >> 8);