Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1536979

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

From Stefan Wahren <stefan.wahren@i2se.com>
Newsgroups linux.kernel
Subject Re: [PATCH] serial: mxs-auart: support CMSPAR termios cflag
Date 2016-12-06 14:40 +0100
Message-ID <sLo8q-2Cu-29@gated-at.bofh.it> (permalink)
Organization linux.* mail to news gateway

Show all headers | View raw


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

Back to linux.kernel | Previous | NextNext in thread | Find similar | Unroll thread


Thread

Re: [PATCH] serial: mxs-auart: support CMSPAR termios cflag Stefan Wahren <stefan.wahren@i2se.com> - 2016-12-06 14:40 +0100
  Re: [PATCH] serial: mxs-auart: support CMSPAR termios cflag Wolfgang Ocker <weo@reccoware.de> - 2016-12-06 17:00 +0100

csiph-web