serial: mxs-auart: fix baud rate range
authorStefan Wahren <stefan.wahren@i2se.com>
Tue, 11 Aug 2015 11:46:01 +0000 (11:46 +0000)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sat, 15 Aug 2015 00:08:48 +0000 (17:08 -0700)
Currently mxs-auart doesn't care correctly about the baud rate divisor.
According to reference manual the baud rate divisor must be between
0x000000EC and 0x003FFFC0. So calculate the possible baud rate range
and use it for uart_get_baud_rate().

Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com>
Reviewed-by: Fabio Estevam <fabio.estevam@freescale.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/tty/serial/mxs-auart.c

index 8d1cd158ca2433d7c1e2a4e1857e35aabdef80d2..7c7f30809849d24d337f8f2eedf4a83ed23907f1 100644 (file)
 #define AUART_CTRL2_TXE                                (1 << 8)
 #define AUART_CTRL2_UARTEN                     (1 << 0)
 
+#define AUART_LINECTRL_BAUD_DIV_MAX            0x003fffc0
+#define AUART_LINECTRL_BAUD_DIV_MIN            0x000000ec
 #define AUART_LINECTRL_BAUD_DIVINT_SHIFT       16
 #define AUART_LINECTRL_BAUD_DIVINT_MASK                0xffff0000
 #define AUART_LINECTRL_BAUD_DIVINT(v)          (((v) & 0xffff) << 16)
@@ -659,7 +661,7 @@ static void mxs_auart_settermios(struct uart_port *u,
 {
        struct mxs_auart_port *s = to_auart_port(u);
        u32 bm, ctrl, ctrl2, div;
-       unsigned int cflag, baud;
+       unsigned int cflag, baud, baud_min, baud_max;
 
        cflag = termios->c_cflag;
 
@@ -752,7 +754,9 @@ static void mxs_auart_settermios(struct uart_port *u,
        }
 
        /* set baud rate */
-       baud = uart_get_baud_rate(u, termios, old, 0, u->uartclk);
+       baud_min = DIV_ROUND_UP(u->uartclk * 32, AUART_LINECTRL_BAUD_DIV_MAX);
+       baud_max = u->uartclk * 32 / AUART_LINECTRL_BAUD_DIV_MIN;
+       baud = uart_get_baud_rate(u, termios, old, baud_min, baud_max);
        div = u->uartclk * 32 / baud;
        ctrl |= AUART_LINECTRL_BAUD_DIVFRAC(div & 0x3F);
        ctrl |= AUART_LINECTRL_BAUD_DIVINT(div >> 6);