diff options
| author | Randy Dunlap <randy.dunlap@oracle.com> | 2007-05-08 00:29:01 -0700 |
|---|---|---|
| committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-05-08 11:15:08 -0700 |
| commit | 10f8a59813ee8bb41fb1d72ed2ec12a1c9f66da2 (patch) | |
| tree | fee52684c48e327f3f34a14da71c5e8bd2d12dcc | |
| parent | 7e80d0d0b64f5c00b0ac7e623d96189309c298ca (diff) | |
| download | lwn-10f8a59813ee8bb41fb1d72ed2ec12a1c9f66da2.tar.gz lwn-10f8a59813ee8bb41fb1d72ed2ec12a1c9f66da2.zip | |
parport_serial: fix PCI must_checks
drivers/parport/parport_serial.c:402: warning: ignoring return value of 'pci_enable_device', declared with attribute warn_unused_result
Signed-off-by: Randy Dunlap <randy.dunlap@oracle.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
| -rw-r--r-- | drivers/parport/parport_serial.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/drivers/parport/parport_serial.c b/drivers/parport/parport_serial.c index e06223cf6c26..90ea3b8b99b0 100644 --- a/drivers/parport/parport_serial.c +++ b/drivers/parport/parport_serial.c @@ -392,6 +392,7 @@ static int parport_serial_pci_suspend(struct pci_dev *dev, pm_message_t state) static int parport_serial_pci_resume(struct pci_dev *dev) { struct parport_serial_private *priv = pci_get_drvdata(dev); + int err; pci_set_power_state(dev, PCI_D0); pci_restore_state(dev); @@ -399,7 +400,12 @@ static int parport_serial_pci_resume(struct pci_dev *dev) /* * The device may have been disabled. Re-enable it. */ - pci_enable_device(dev); + err = pci_enable_device(dev); + if (err) { + printk(KERN_ERR "parport_serial: %s: error enabling " + "device for resume (%d)\n", pci_name(dev), err); + return err; + } if (priv->serial) pciserial_resume_ports(priv->serial); |