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

Myeonghun Pak <[email protected]>
Newsgroups gmane.linux.kernel,gmane.linux.serial,gmane.linux.kernel.stable
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
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.