Re: [PATCH v1 1/1] serial: 8250_mid: Balance reference count for PCI DMA device

From: Andy Shevchenko
Date: Tue Feb 15 2022 - 05:32:47 EST


On Tue, Feb 15, 2022 at 12:29:49PM +0200, Andy Shevchenko wrote:
> On Tue, Feb 15, 2022 at 11:11:41AM +0100, Jiri Slaby wrote:
> > On 15. 02. 22, 11:09, Andy Shevchenko wrote:
> > > The pci_get_slot() increases its reference count, the caller
> > > must decrement the reference count by calling pci_dev_put().
>
> > And what about the -EINVAL case?
>
> What about it? The ->probe() calls ->exit() in its error path.
> Or did I miss something?

Or you mean that we call pci_dev_put() on NULL pointer?
This is completely valid case and handled in the callee.

--
With Best Regards,
Andy Shevchenko