Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
From:   Andy Duan <fugang.duan@nxp.com>
To:     Philippe Schenker <philippe.schenker@toradex.com>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        "linux-serial@vger.kernel.org" <linux-serial@vger.kernel.org>
CC:     Max Krummenacher <max.krummenacher@toradex.com>,
        Jiri Slaby <jslaby@suse.com>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        Stefan Agner <stefan@agner.ch>,
        dl-linux-imx <linux-imx@nxp.com>,
        Andrey Smirnov <andrew.smirnov@gmail.com>
Subject: RE: [EXT] [PATCH v1 3/3] tty: serial: lpuart: Add RS485 support for
 32-bit uart flavour
Thread-Topic: [EXT] [PATCH v1 3/3] tty: serial: lpuart: Add RS485 support for
 32-bit uart flavour
Thread-Index: AQHVhDUKmWLMalCWlk6sMnHx6FvIsqdeSvHQ
Date:   Thu, 17 Oct 2019 05:14:15 +0000
Message-ID: <VI1PR0402MB36007F03DB38FDA44FED3332FF6D0@VI1PR0402MB3600.eurprd04.prod.outlook.com>
References: <20191016151845.15859-1-philippe.schenker@toradex.com>
 <20191016151845.15859-3-philippe.schenker@toradex.com>
In-Reply-To: <20191016151845.15859-3-philippe.schenker@toradex.com>
Accept-Language: zh-CN, en-US
Content-Language: en-US
received-spf: None (protection.outlook.com: nxp.com does not designate
 permitted sender hosts)
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: 043add86-214d-4425-897d-08d752c0daeb
X-MS-Exchange-CrossTenant-originalarrivaltime: 17 Oct 2019 05:14:15.4953
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: EuzjXQgaZP4CfNBOXxnfaKTWl8r1IyM2Z8c+ZaIEhg5jGkWtpBu3Qhc62Gr+JyXcvFr/cy92I5SpUCuAs5Ad/Q==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR0402MB2878
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

From: Philippe Schenker <philippe.schenker@toradex.com> Sent: Wednesday, Oc=
tober 16, 2019 11:19 PM
> This commits adds RS485 support for LPUART hardware that uses 32-bit
> registers. These are typically found in i.MX8 processors.
>=20
> Signed-off-by: Philippe Schenker <philippe.schenker@toradex.com>

Reviewed-by: Fugang Duan <fugang.duan@nxp.com>
>=20
> ---
>=20
>  drivers/tty/serial/fsl_lpuart.c | 65 ++++++++++++++++++++++++++++++++-
>  1 file changed, 63 insertions(+), 2 deletions(-)
>=20
> diff --git a/drivers/tty/serial/fsl_lpuart.c b/drivers/tty/serial/fsl_lpu=
art.c index
> 346b4a070ce9..22df5f8f48b6 100644
> --- a/drivers/tty/serial/fsl_lpuart.c
> +++ b/drivers/tty/serial/fsl_lpuart.c
> @@ -1280,6 +1280,57 @@ static int lpuart_config_rs485(struct uart_port
> *port,
>         return 0;
>  }
>=20
> +static int lpuart32_config_rs485(struct uart_port *port,
> +                       struct serial_rs485 *rs485) {
> +       struct lpuart_port *sport =3D container_of(port,
> +                       struct lpuart_port, port);
> +
> +       unsigned long modem =3D lpuart32_read(&sport->port, UARTMODIR)
> +                               & ~(UARTMODEM_TXRTSPOL |
> UARTMODEM_TXRTSE);
> +       lpuart32_write(&sport->port, modem, UARTMODIR);
> +
> +       /* clear unsupported configurations */
> +       rs485->delay_rts_before_send =3D 0;
> +       rs485->delay_rts_after_send =3D 0;
> +       rs485->flags &=3D ~SER_RS485_RX_DURING_TX;
> +
> +       if (rs485->flags & SER_RS485_ENABLED) {
> +               /* Enable auto RS-485 RTS mode */
> +               modem |=3D UARTMODEM_TXRTSE;
> +
> +               /*
> +                * RTS needs to be logic HIGH either during transer _or_
> after
> +                * transfer, other variants are not supported by the
> hardware.
> +                */
> +
> +               if (!(rs485->flags & (SER_RS485_RTS_ON_SEND |
> +                               SER_RS485_RTS_AFTER_SEND)))
> +                       rs485->flags |=3D SER_RS485_RTS_ON_SEND;
> +
> +               if (rs485->flags & SER_RS485_RTS_ON_SEND &&
> +                               rs485->flags &
> SER_RS485_RTS_AFTER_SEND)
> +                       rs485->flags &=3D
> ~SER_RS485_RTS_AFTER_SEND;
> +
> +               /*
> +                * The hardware defaults to RTS logic HIGH while
> transfer.
> +                * Switch polarity in case RTS shall be logic HIGH
> +                * after transfer.
> +                * Note: UART is assumed to be active high.
> +                */
> +               if (rs485->flags & SER_RS485_RTS_ON_SEND)
> +                       modem &=3D ~UARTMODEM_TXRTSPOL;
> +               else if (rs485->flags & SER_RS485_RTS_AFTER_SEND)
> +                       modem |=3D UARTMODEM_TXRTSPOL;
> +       }
> +
> +       /* Store the new configuration */
> +       sport->port.rs485 =3D *rs485;
> +
> +       lpuart32_write(&sport->port, modem, UARTMODIR);
> +       return 0;
> +}
> +
>  static unsigned int lpuart_get_mctrl(struct uart_port *port)  {
>         unsigned int temp =3D 0;
> @@ -1878,6 +1929,13 @@ lpuart32_set_termios(struct uart_port *port,
> struct ktermios *termios,
>                 ctrl |=3D UARTCTRL_M;
>         }
>=20
> +       /*
> +        * When auto RS-485 RTS mode is enabled,
> +        * hardware flow control need to be disabled.
> +        */
> +       if (sport->port.rs485.flags & SER_RS485_ENABLED)
> +               termios->c_cflag &=3D ~CRTSCTS;
> +
>         if (termios->c_cflag & CRTSCTS) {
>                 modem |=3D (UARTMODIR_RXRTSE |
> UARTMODIR_TXCTSE);
>         } else {
> @@ -2405,7 +2463,10 @@ static int lpuart_probe(struct platform_device
> *pdev)
>                 sport->port.ops =3D &lpuart_pops;
>         sport->port.flags =3D UPF_BOOT_AUTOCONF;
>=20
> -       sport->port.rs485_config =3D lpuart_config_rs485;
> +       if (lpuart_is_32(sport))
> +               sport->port.rs485_config =3D lpuart32_config_rs485;
> +       else
> +               sport->port.rs485_config =3D lpuart_config_rs485;
>=20
>         sport->ipg_clk =3D devm_clk_get(&pdev->dev, "ipg");
>         if (IS_ERR(sport->ipg_clk)) {
> @@ -2459,7 +2520,7 @@ static int lpuart_probe(struct platform_device
> *pdev)
>             sport->port.rs485.delay_rts_after_send)
>                 dev_err(&pdev->dev, "driver doesn't support RTS
> delays\n");
>=20
> -       lpuart_config_rs485(&sport->port, &sport->port.rs485);
> +       sport->port.rs485_config(&sport->port, &sport->port.rs485);
>=20
>         sport->dma_tx_chan =3D
> dma_request_slave_channel(sport->port.dev, "tx");
>         if (!sport->dma_tx_chan)
> --
> 2.23.0