Message ID | 20170412133015.23185-1-romain.perier@collabora.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
On Wed, Apr 12, 2017 at 10:30 AM, Romain Perier <romain.perier@collabora.com> wrote: > From: Nandor Han <nandor.han@ge.com> > > Currently, this IRQ is always enabled. Some devices might mux these pins > to other I/Os, like I2C. This could lead to spurious interrupts. > > This commit makes this IRQ optional, by using the field have_rtscts. > > Signed-off-by: Nandor Han <nandor.han@ge.com> > Signed-off-by: Romain Perier <romain.perier@collabora.com> Reviewed-by: Fabio Estevam <fabio.estevam@nxp.com>
Hello all, Le 12/04/2017 à 15:38, Fabio Estevam a écrit : > On Wed, Apr 12, 2017 at 10:30 AM, Romain Perier > <romain.perier@collabora.com> wrote: >> From: Nandor Han <nandor.han@ge.com> >> >> Currently, this IRQ is always enabled. Some devices might mux these pins >> to other I/Os, like I2C. This could lead to spurious interrupts. >> >> This commit makes this IRQ optional, by using the field have_rtscts. >> >> Signed-off-by: Nandor Han <nandor.han@ge.com> >> Signed-off-by: Romain Perier <romain.perier@collabora.com> > Reviewed-by: Fabio Estevam <fabio.estevam@nxp.com> No more feedback on this patch ? Regards, Romain
diff --git a/drivers/tty/serial/imx.c b/drivers/tty/serial/imx.c index e3e152cb..6dd674f 100644 --- a/drivers/tty/serial/imx.c +++ b/drivers/tty/serial/imx.c @@ -1294,7 +1294,9 @@ static int imx_startup(struct uart_port *port) imx_enable_dma(sport); temp = readl(sport->port.membase + UCR1); - temp |= UCR1_RRDYEN | UCR1_RTSDEN | UCR1_UARTEN; + temp |= UCR1_RRDYEN | UCR1_UARTEN; + if (sport->have_rtscts) + temp |= UCR1_RTSDEN; writel(temp, sport->port.membase + UCR1);