aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/usb
diff options
context:
space:
mode:
authorTomoya MORINAGA <tomoya.rohm@gmail.com>2012-01-11 21:27:06 -0500
committerFelipe Balbi <balbi@ti.com>2012-01-24 04:34:24 -0500
commitc802672cd36cd063bfd54d54c8c34825ab5b2357 (patch)
tree862c8c34512f314eb8fc57388298484c087b729a /drivers/usb
parentc50a3bff0edb0acd49d8033a12ea4668e09a31ad (diff)
usb: gadget: pch_udc: Fix wrong return value
ISSUE: If the return value of pch_udc_pcd_init() is False, the return value of this function is unsettled. Since pch_udc_pcd_init() always returns 0, there is not actually the issue. CAUSE: If pch_udc_pcd_init() is True, the variable, retval, is not set for an appropriate value. Signed-off-by: Tomoya MORINAGA <tomoya.rohm@gmail.com> Signed-off-by: Felipe Balbi <balbi@ti.com>
Diffstat (limited to 'drivers/usb')
-rw-r--r--drivers/usb/gadget/pch_udc.c4
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/usb/gadget/pch_udc.c b/drivers/usb/gadget/pch_udc.c
index 6a44c921d13c..96606b6cff45 100644
--- a/drivers/usb/gadget/pch_udc.c
+++ b/drivers/usb/gadget/pch_udc.c
@@ -2915,8 +2915,10 @@ static int pch_udc_probe(struct pci_dev *pdev,
2915 } 2915 }
2916 pch_udc = dev; 2916 pch_udc = dev;
2917 /* initialize the hardware */ 2917 /* initialize the hardware */
2918 if (pch_udc_pcd_init(dev)) 2918 if (pch_udc_pcd_init(dev)) {
2919 retval = -ENODEV;
2919 goto finished; 2920 goto finished;
2921 }
2920 if (request_irq(pdev->irq, pch_udc_isr, IRQF_SHARED, KBUILD_MODNAME, 2922 if (request_irq(pdev->irq, pch_udc_isr, IRQF_SHARED, KBUILD_MODNAME,
2921 dev)) { 2923 dev)) {
2922 dev_err(&pdev->dev, "%s: request_irq(%d) fail\n", __func__, 2924 dev_err(&pdev->dev, "%s: request_irq(%d) fail\n", __func__,