Re: [PATCH] serial: mxs-auart: support CMSPAR termios cflag

From: Stefan Wahren
Date: Tue Dec 06 2016 - 08:37:19 EST


Hi Wolfgang,

> --- a/drivers/tty/serial/mxs-auart.c
> +++ b/drivers/tty/serial/mxs-auart.c
> @@ -95,6 +95,7 @@
> #define AUART_LINECTRL_BAUD_DIVFRAC_SHIFT 8
> #define AUART_LINECTRL_BAUD_DIVFRAC_MASK 0x00003f00
> #define AUART_LINECTRL_BAUD_DIVFRAC(v) (((v) & 0x3f) << 8)
> +#define AUART_LINECTRL_SPS (1 << 7)
> #define AUART_LINECTRL_WLEN_MASK 0x00000060
> #define AUART_LINECTRL_WLEN(v) (((v) & 0x3) << 5)
> #define AUART_LINECTRL_FEN (1 << 4)
> @@ -1010,10 +1011,12 @@ static void mxs_auart_settermios(struct uart_port *u,
> ctrl |= AUART_LINECTRL_WLEN(bm);
>
> /* parity */
> - if (cflag & PARENB) {
> + if (cflag & (PARENB|CMSPAR)) {

does it make sense to enable stick parity in case parity is disabled?

The i.MX28 reference manual doesn't describe this case explicit.

Stefan