2022-03-11 10:23:29

by Trevor Woerner

Subject: [PATCH 1/3] serial: 8250_fintek.c: Finish support for the F81865

This driver only partially supports the F81865 device. The UART portions
of this SuperIO chip behave exactly like the UART of the F81866, except
that the F81866 has 128-byte FIFOs whereas the F81865 has 16-byte FIFOs.
Therefore fill out the support for the F81865 in the places where it is
missing.

Tested at 1500000 baud on the iEi NANO-PV-D5251-R10 board.

Signed-off-by: Trevor Woerner <[email protected]>
---
drivers/tty/serial/8250/8250_fintek.c | 2 ++
1 file changed, 2 insertions(+)

diff --git a/drivers/tty/serial/8250/8250_fintek.c b/drivers/tty/serial/8250/8250_fintek.c
index 251f0018ae8c..d9f0e546b1a1 100644
--- a/drivers/tty/serial/8250/8250_fintek.c
+++ b/drivers/tty/serial/8250/8250_fintek.c
@@ -316,6 +316,7 @@ static void fintek_8250_set_termios(struct uart_port *port,
break;
case CHIP_ID_F81966:
case CHIP_ID_F81866:
+ case CHIP_ID_F81865:
reg = F81866_UART_CLK;
break;
default:
@@ -363,6 +364,7 @@ static void fintek_8250_set_termios_handler(struct uart_8250_port *uart)
case CHIP_ID_F81216H:
case CHIP_ID_F81966:
case CHIP_ID_F81866:
+ case CHIP_ID_F81865:
uart->port.set_termios = fintek_8250_set_termios;
break;

--
2.34.1.75.gabe6bb3905

2022-03-11 20:54:01

by Trevor Woerner

Subject: Re: [PATCH 1/3] serial: 8250_fintek.c: Finish support for the F81865

On Fri 2022-03-11 @ 02:02:00 AM, Trevor Woerner wrote:
> This driver only partially supports the F81865 device. The UART portions
> of this SuperIO chip behave exactly like the UART of the F81866, except
> that the F81866 has 128-byte FIFOs whereas the F81865 has 16-byte FIFOs.
> Therefore fill out the support for the F81865 in the places where it is
> missing.
>
> Tested at 1500000 baud on the iEi NANO-PV-D5251-R10 board.

Is it too late to send a v2? I'd like to fixup the subject to remove the ".c",
and I should probably add some comments in the driver explaining that the
F81865 is functionally the same as the F81866 (except for the fifo sizes)
which is why the F81865 doesn't have any of its own #define's.

> Signed-off-by: Trevor Woerner <[email protected]>
> ---
> drivers/tty/serial/8250/8250_fintek.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/tty/serial/8250/8250_fintek.c b/drivers/tty/serial/8250/8250_fintek.c
> index 251f0018ae8c..d9f0e546b1a1 100644
> --- a/drivers/tty/serial/8250/8250_fintek.c
> +++ b/drivers/tty/serial/8250/8250_fintek.c
> @@ -316,6 +316,7 @@ static void fintek_8250_set_termios(struct uart_port *port,
> break;
> case CHIP_ID_F81966:
> case CHIP_ID_F81866:
> + case CHIP_ID_F81865:
> reg = F81866_UART_CLK;
> break;
> default:
> @@ -363,6 +364,7 @@ static void fintek_8250_set_termios_handler(struct uart_8250_port *uart)
> case CHIP_ID_F81216H:
> case CHIP_ID_F81966:
> case CHIP_ID_F81866:
> + case CHIP_ID_F81865:
> uart->port.set_termios = fintek_8250_set_termios;
> break;
>
> --
> 2.34.1.75.gabe6bb3905
>