summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDave Hansen <dave@sr71.net>2009-10-14 09:20:47 -0700
committerRichard Purdie <rpurdie@linux.intel.com>2009-12-16 11:30:09 +0000
commit09a46db05b2e07484c08c638d7d6bda0523a13a6 (patch)
tree550fe0cb15b0da3c168edbc405e0d0fc9255a2f9
parent7f131cf3ed96c969d7b092bf629e25c3df50901e (diff)
downloadop-kernel-dev-09a46db05b2e07484c08c638d7d6bda0523a13a6.zip
op-kernel-dev-09a46db05b2e07484c08c638d7d6bda0523a13a6.tar.gz
leds-ss4200: Check pci_enable_device return
pci_enable_result is defined using the __must_check macro but leds-ss4200 is not checking the return value. Signed-off-by: Javier Martinez Canillas <martinez.javier@gmail.com> Signed-off-by: Dave Hansen <dave@sr71.net> Signed-off-by: Richard Purdie <rpurdie@linux.intel.com>
-rw-r--r--drivers/leds/leds-ss4200.c8
1 files changed, 6 insertions, 2 deletions
diff --git a/drivers/leds/leds-ss4200.c b/drivers/leds/leds-ss4200.c
index 0ec4ec3..c7f6347 100644
--- a/drivers/leds/leds-ss4200.c
+++ b/drivers/leds/leds-ss4200.c
@@ -344,10 +344,14 @@ static struct pci_dev *nas_gpio_pci_dev;
static int __devinit ich7_lpc_probe(struct pci_dev *dev,
const struct pci_device_id *id)
{
- int status = 0;
+ int status;
u32 gc = 0;
- pci_enable_device(dev);
+ status = pci_enable_device(dev);
+ if (status) {
+ dev_err(&dev->dev, "pci_enable_device failed\n");
+ return;
+ }
nas_gpio_pci_dev = dev;
status = pci_read_config_dword(dev, PMBASE, &g_pm_io_base);
OpenPOWER on IntegriCloud