Re: [PATCH v1 2/6] serial: txx9: Drop usage of uart_match_port()

Greg Kroah-Hartman <[email protected]> Mon, 3 Aug 2026 16:46:14 +0200
Newsgroups org.kernel.vger.linux-serial,org.kernel.vger.linux-kernel
Message-ID <2026080352-chatter-neurotic-48c3@gregkh>
On Fri, Jul 31, 2026 at 11:39:05AM +0200, Uwe Kleine-König wrote:
> In this driver .iotype is always UPIO_PORT. So uart_match_port() is
> equivalent to comparing .iobase. Remove the call to uart_match_port()
> which then only has a single caller that is handled in the next commit.
> 
> Signed-off-by: Uwe Kleine-König <[email protected]>
> ---
>  drivers/tty/serial/serial_txx9.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/tty/serial/serial_txx9.c b/drivers/tty/serial/serial_txx9.c
> index 5ed20129b44b..2368081c3742 100644
> --- a/drivers/tty/serial/serial_txx9.c
> +++ b/drivers/tty/serial/serial_txx9.c
> @@ -965,7 +965,7 @@ static int serial_txx9_register_port(struct uart_port *port)
>  	mutex_lock(&serial_txx9_mutex);
>  	for (i = 0; i < UART_NR; i++) {
>  		uart = &serial_txx9_ports[i];
> -		if (uart_match_port(uart, port)) {
> +		if (uart->iobase == port->iobase)
>  			uart_remove_one_port(&serial_txx9_reg, uart);
>  			break;
>  		}

Does this even build?

:(

thanks,

greg k-h