On Tue, Aug 1, 2017 at 9:13 PM, <[email protected]> wrote: > From: Kuppuswamy Sathyanarayanan <[email protected]> > > This patch adds proper error handling for failure cases in > ipc_pci_probe() function. > > Signed-off-by: Kuppuswamy Sathyanarayanan > <[email protected]> > --- > drivers/platform/x86/intel_pmc_ipc.c | 23 ++++++++++++++++++----- > 1 file changed, 18 insertions(+), 5 deletions(-) > > diff --git a/drivers/platform/x86/intel_pmc_ipc.c > b/drivers/platform/x86/intel_pmc_ipc.c > index bb792a5..7b65237 100644 > --- a/drivers/platform/x86/intel_pmc_ipc.c > +++ b/drivers/platform/x86/intel_pmc_ipc.c > @@ -489,33 +489,46 @@ static int ipc_pci_probe(struct pci_dev *pdev, const > struct pci_device_id *id) > > ret = pci_enable_device(pdev); > if (ret) > - return ret; > + goto release_device;
Instead of doing this way and ping-ponging code in the same series, better to switch to managed PCI functions in the first place. -- With Best Regards, Andy Shevchenko
