Re: [PATCH -next] misc/pvpanic: fix return value check in pvpanic_pci_probe()

From: Andy Shevchenko
Date: Tue Mar 30 2021 - 05:28:47 EST


On Tue, Mar 30, 2021 at 09:36:59AM +0800, Qiheng Lin wrote:
> In case of error, the function pci_iomap() returns NULL pointer
> not ERR_PTR(). The IS_ERR() test in the return value check
> should be replaced with NULL test.

Error path there is broken completely, this is only one place you fix, but I
suggest to switch to devm*, pcim* APIs instead,


--
With Best Regards,
Andy Shevchenko