diff options
author | Dave Hansen <dave@sr71.net> | 2009-10-14 12:20:47 -0400 |
---|---|---|
committer | Richard Purdie <rpurdie@linux.intel.com> | 2009-12-16 06:30:09 -0500 |
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 0ec4ec3bc823..c7f634727d67 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; | |||
344 | static int __devinit ich7_lpc_probe(struct pci_dev *dev, | 344 | static int __devinit ich7_lpc_probe(struct pci_dev *dev, |
345 | const struct pci_device_id *id) | 345 | const struct pci_device_id *id) |
346 | { | 346 | { |
347 | int status = 0; | 347 | int status; |
348 | u32 gc = 0; | 348 | u32 gc = 0; |
349 | 349 | ||
350 | pci_enable_device(dev); | 350 | status = pci_enable_device(dev); |
351 | if (status) { | ||
352 | dev_err(&dev->dev, "pci_enable_device failed\n"); | ||
353 | return; | ||
354 | } | ||
351 | 355 | ||
352 | nas_gpio_pci_dev = dev; | 356 | nas_gpio_pci_dev = dev; |
353 | status = pci_read_config_dword(dev, PMBASE, &g_pm_io_base); | 357 | status = pci_read_config_dword(dev, PMBASE, &g_pm_io_base); |