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

--i63sme3zsfeqfqhw
Content-Type: text/plain; protected-headers=v1; charset=iso-8859-1
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable
Subject: Re: [PATCH v1 2/6] serial: txx9: Drop usage of uart_match_port()
MIME-Version: 1.0

On Mon, Aug 03, 2026 at 04:46:14PM +0200, Greg Kroah-Hartman wrote:
> On Fri, Jul 31, 2026 at 11:39:05AM +0200, Uwe Kleine-K=F6nig 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.
> >=20
> > Signed-off-by: Uwe Kleine-K=F6nig <[email protected]>
> > ---
> >  drivers/tty/serial/serial_txx9.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >=20
> > diff --git a/drivers/tty/serial/serial_txx9.c b/drivers/tty/serial/seri=
al_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_po=
rt *port)
> >  	mutex_lock(&serial_txx9_mutex);
> >  	for (i =3D 0; i < UART_NR; i++) {
> >  		uart =3D &serial_txx9_ports[i];
> > -		if (uart_match_port(uart, port)) {
> > +		if (uart->iobase =3D=3D port->iobase)
> >  			uart_remove_one_port(&serial_txx9_reg, uart);
> >  			break;
> >  		}
>=20
> Does this even build?

Sigh, it passed my build scripts (which is essentially allmodconfig for
arc, sh, alpha, arm64, x86_64, s390, riscv, mips, powerpc, loongarch,
sparc, m68k and parisc), but this driver is only compiled for certain
mips configs and not allmodconfig :-(

Will respin with a better suitable test build.

Best regards
Uwe

--i63sme3zsfeqfqhw
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQEzBAABCgAdFiEEP4GsaTp6HlmJrf7Tj4D7WH0S/k4FAmpw5JIACgkQj4D7WH0S
/k6IvAgAjae8C55Be9XZiSYRiEkFAhX3+6rrIribu/i94EKIzaCherI5Ey7btJ5C
4lBeqIcaowivanHETZzOYkZIAIU1YmM5wzQRG/0TffdEjWfmFBMsfo7uYJqKzNVd
WYSryaaDrz3/K9t2ggKVdUPZZ/gzv1eP0xOOUUSDBP/HJgkP7vQjvDc1hU3qqRaL
TdLrnd1GtPf7VXmtBwHEX3tbgy+ZfXJpVCoTisLgkb0iTlMY7/TRbzbue5eOczXV
T6nqCbkU4i3gQnTJc4p91F1RpsxfJakr27HdnB1OjmBHf/zY5kW6vmAjrS9iJqBF
JcH/U7OLbJe9x/b+DDm4hADBWhs3oA==
=Lh51
-----END PGP SIGNATURE-----

--i63sme3zsfeqfqhw--