diff options
author | Jean Delvare <jdelvare@suse.de> | 2014-02-24 10:03:16 -0500 |
---|---|---|
committer | Borislav Petkov <bp@suse.de> | 2014-02-25 04:13:46 -0500 |
commit | 5fdae419009764337c8f2bf143af57dcadfd228d (patch) | |
tree | a4b86757f7c8d3a0895908742a4c813066dbb80d /drivers/edac | |
parent | 5576fb8696bf8d8ae60278c2aa4dc23eae9eb86e (diff) |
i82875p_edac: Drop redundant call to pci_get_device()
The first call to pci_get_device() in i82875p_probe1() is useless. The
result is immediately reset at the beginning of
i82875p_setup_overfl_dev(), which then issues the same
pci_get_device() call. Dropping the redundant call avoids a device
reference leak.
Signed-off-by: Jean Delvare <jdelvare@suse.de>
Link: http://lkml.kernel.org/r/20140224160316.60e55fb6@endymion.delvare
Cc: Doug Thompson <dougthompson@xmission.com>
Signed-off-by: Borislav Petkov <bp@suse.de>
Diffstat (limited to 'drivers/edac')
-rw-r--r-- | drivers/edac/i82875p_edac.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/edac/i82875p_edac.c b/drivers/edac/i82875p_edac.c index 80573df0a4d7..8d0450b9b9af 100644 --- a/drivers/edac/i82875p_edac.c +++ b/drivers/edac/i82875p_edac.c | |||
@@ -406,8 +406,6 @@ static int i82875p_probe1(struct pci_dev *pdev, int dev_idx) | |||
406 | 406 | ||
407 | edac_dbg(0, "\n"); | 407 | edac_dbg(0, "\n"); |
408 | 408 | ||
409 | ovrfl_pdev = pci_get_device(PCI_VEND_DEV(INTEL, 82875_6), NULL); | ||
410 | |||
411 | if (i82875p_setup_overfl_dev(pdev, &ovrfl_pdev, &ovrfl_window)) | 409 | if (i82875p_setup_overfl_dev(pdev, &ovrfl_pdev, &ovrfl_window)) |
412 | return -ENODEV; | 410 | return -ENODEV; |
413 | drc = readl(ovrfl_window + I82875P_DRC); | 411 | drc = readl(ovrfl_window + I82875P_DRC); |