Re: [PATCH] serial: 8250_hub6: Fix double definition for hub6_match_port()

Uwe Kleine-König <[email protected]> Thu, 30 Jul 2026 21:57:01 +0200
Newsgroups org.kernel.vger.linux-serial,org.kernel.vger.linux-kernel
Message-ID <amusQjI0DOPzL4jM@monoceros>
Hello Hugo,

On Thu, Jul 30, 2026 at 04:49:45PM +0200, Greg Kroah-Hartman wrote:
> On Wed, Jul 29, 2026 at 03:30:57PM +0200, Uwe Kleine-König wrote:
> > I think the real fix would be to just copy uart_match_port() into the
> > two drivers that actually use it (and remove the then dead code paths).
> > Then it would be drivers/tty/serial/8250/8250_core.c using
> > hub6_match_port() only and that can be handled by a proper dependency.
> 
> I'm totally confused, so I'll drop this patch from my review queue and
> wait for you all to figure it out :)

Does my suggestion make sense for you? Do you intend to work on this to
unconfuse Greg?

Best regards
Uwe
signature.asc (application/pgp-signature, 488 B)
-----BEGIN PGP SIGNATURE-----

iQEzBAABCgAdFiEEP4GsaTp6HlmJrf7Tj4D7WH0S/k4FAmprrIoACgkQj4D7WH0S
/k4lAQf/WVX/WIN4b+WCVtKUAw3N9m/XRolz3JDLMsiqNA8R+Z207MWDCfOjFXgM
Me+MfN6kKvfKnbhBKlJtGCyQ6w5C7b0CjVcZ7ua63MCkH2JAGvMQ+34ZfARQt1j/
yqsy8yOdTOlq0SIzuouBg2wvM3nQ8dN07g4SMsYyZ0SBoKBM9boTTq2unz+ZGdzp
6rL4C6bocj7iQ0Oo7R3hGwoYmotpU79hOtRLyLlXfyYnbz5iMEK0nl6RounBqvgx
YmJgbV8Oi8c84+Jy18lEMSNSTJqfdjop5ZFpCk0oIW+tkWSvuNpR0DzgiEb7CHvV
SgubOmsnWNCWyZLhvgsjoelArW1Hkw==
=giPV
-----END PGP SIGNATURE-----