diff options
| author | Dave Hansen <dave@sr71.net> | 2009-10-14 09:20:47 -0700 |
|---|---|---|
| committer | Richard Purdie <rpurdie@linux.intel.com> | 2009-12-16 11:30:09 +0000 |
| commit | 09a46db05b2e07484c08c638d7d6bda0523a13a6 (patch) | |
| tree | 550fe0cb15b0da3c168edbc405e0d0fc9255a2f9 /drivers/leds/leds-ss4200.c | |
| parent | 7f131cf3ed96c969d7b092bf629e25c3df50901e (diff) | |
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>
Diffstat (limited to 'drivers/leds/leds-ss4200.c')
| -rw-r--r-- | drivers/leds/leds-ss4200.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/drivers/leds/leds-ss4200.c b/drivers/leds/leds-ss4200.c index 0ec4ec3bc82..c7f634727d6 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); |