Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751937AbdFTQNV (ORCPT ); Tue, 20 Jun 2017 12:13:21 -0400 Received: from mail-ot0-f193.google.com ([74.125.82.193]:35160 "EHLO mail-ot0-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751038AbdFTQNT (ORCPT ); Tue, 20 Jun 2017 12:13:19 -0400 MIME-Version: 1.0 In-Reply-To: <20170620153701.12626-1-clemens.gruber@pqgruber.com> References: <20170620153701.12626-1-clemens.gruber@pqgruber.com> From: Fabio Estevam Date: Tue, 20 Jun 2017 13:13:18 -0300 Message-ID: Subject: Re: [PATCH] serial: imx: disable DMA for RS-485 on i.MX6 SMP To: Clemens Gruber Cc: "linux-serial@vger.kernel.org" , Greg Kroah-Hartman , Fabio Estevam , =?UTF-8?Q?Uwe_Kleine=2DK=C3=B6nig?= , linux-kernel , stable , Fugang Duan Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2455 Lines: 57 Hi Clemens, On Tue, Jun 20, 2017 at 12:37 PM, Clemens Gruber wrote: > DMA support for half-duplex RS-485 never worked correctly on i.MX6Q/D > due to an undiscovered SMP-related bug, instead of the real data being > sent out, the rest of the transmit buffer is sent (xmit->tail jumps over > xmit->head in imx_transmit_buffer and UART_XMIT_SIZE bytes are sent out) > More details: https://lkml.org/lkml/2017/1/4/579 > > Disable it for that configuration until the bug is found and fixed. > > We need to know at probe time if we can enable DMA. (RS-485 could be > enabled after that). Let's therefore only enable DMA if it is not an > i.MX6Q/D UART with uart-has-rtscts in the DT and CONFIG_SMP enabled. > > Fixes: 17b8f2a3fdca ("serial: imx: add support for half duplex rs485") > Cc: > Signed-off-by: Clemens Gruber > --- > drivers/tty/serial/imx.c | 10 ++++++++-- > 1 file changed, 8 insertions(+), 2 deletions(-) > > diff --git a/drivers/tty/serial/imx.c b/drivers/tty/serial/imx.c > index bbefddd92bfe..000949f686a4 100644 > --- a/drivers/tty/serial/imx.c > +++ b/drivers/tty/serial/imx.c > @@ -1277,8 +1277,14 @@ static int imx_startup(struct uart_port *port) > > writel(temp & ~UCR4_DREN, sport->port.membase + UCR4); > > - /* Can we enable the DMA support? */ > - if (!uart_console(port) && !sport->dma_is_inited) > + /* > + * Can we enable the DMA support? > + * RS-485 DMA is broken on i.MX6D/Q SMP systems. Do not use DMA on > + * UARTs with RTS/CTS to prevent misbehavior until the bug is fixed. > + */ > + if (!uart_console(port) && !sport->dma_is_inited && > + !(sport->have_rtscts && is_imx6q_uart(sport) && > + IS_ENABLED(CONFIG_SMP))) > imx_uart_dma_init(sport); The subject gives the impression that the DMA will only be disabled for RS485, but the impact of this change is wider. For example: if I have a mx6q system with a Bluetooth serial connection I can no longer use DMA with your change applied. Ideally we should fix the RS485 DMA bug. If that is not possible, then at least we need to restrict this change to the RS485 case. Maybe we need to pass "linux,rs485-enabled-at-boot-time" in device tree and then use this property to deceide if DMA will be enabled or not: if (!uart_console(port) && !sport->dma_is_inited && !sport->rs485_enabled)