Re: [PATCH 1/2] tty: moxa: unwind tty driver if PCI registration fails

Myeonghun Pak <[email protected]> Tue, 4 Aug 2026 22:55:25 +0900
Newsgroups org.kernel.vger.stable,org.kernel.vger.linux-kernel,org.kernel.vger.linux-serial
Message-ID <CAGEsz8E7iZZOUXZz+Ga1f+F2yYXxALh4XQQAHQ_rn4O5effVWA@mail.gmail.com>
Hi Jiri and Greg,

Thanks for the review. I agree that removing the whole MOXA driver is
the better direction, which supersedes this patch. Please drop/ignore
this patch.

Best regards,
Myeonghun

2026년 8월 3일 (월) 오후 9:41, Greg Kroah-Hartman <[email protected]>님이 작성:
>
> On Mon, Aug 03, 2026 at 08:06:58AM +0200, Jiri Slaby wrote:
> > On 31. 07. 26, 18:18, Myeonghun Pak wrote:
> > > moxa_init() registers the tty driver before registering the PCI driver.
> > > If pci_register_driver() fails, module initialization returns without
> > > unregistering the tty driver or dropping its reference. moxa_exit() is
> > > not called after a failed module initialization.
> > >
> > > Unregister the tty driver and drop its reference before returning the
> > > PCI registration error.
> > I think, we should remove the whole driver instead:
> > https://lore.kernel.org/all/[email protected]/
>
> I agree, I'll submit a patch to just drop it.
>
> thanks,
>
> greg k-h