From: Dave Hansen <dave@sr71.net>
Date: Wed, 14 Oct 2009 16:20:47 +0000 (-0700)
Subject: leds-ss4200: Check pci_enable_device return
X-Git-Tag: accepted/tizen/common/20141203.182822~13219^2~10
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=09a46db05b2e07484c08c638d7d6bda0523a13a6;p=platform%2Fkernel%2Flinux-arm64.git

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>
---

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);