diff options
| author | Kulikov Vasiliy <[email protected]> | 2010-08-10 18:03:20 -0700 |
|---|---|---|
| committer | Linus Torvalds <[email protected]> | 2010-08-11 08:59:21 -0700 |
| commit | b425d5c82d9b1fcf788688e556e4dfeca76604da (patch) | |
| tree | abec89a7486ca331b2bad256a95b4674379f3aed | |
| parent | 44aa80f005be88fef940fccdd345b313964768f3 (diff) | |
edac: i5400: improve handling of pci_enable_device() return value
-EIO is not the only error code that pci_enable_device() may return, also
the set of errors can be enhanced in future. We should compare return
code with zero, not with concrete error value.
Signed-off-by: Kulikov Vasiliy <[email protected]>
Acked-by: Mauro Carvalho Chehab <[email protected]>
Cc: Jeff Roberson <[email protected]>
Cc: Doug Thompson <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>
| -rw-r--r-- | drivers/edac/i5400_edac.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/edac/i5400_edac.c b/drivers/edac/i5400_edac.c index 010c1d6526f5..38a9be9e1c7c 100644 --- a/drivers/edac/i5400_edac.c +++ b/drivers/edac/i5400_edac.c @@ -1348,7 +1348,7 @@ static int __devinit i5400_init_one(struct pci_dev *pdev, /* wake up device */ rc = pci_enable_device(pdev); - if (rc == -EIO) + if (rc) return rc; /* now probe and enable the device */ |