diff options
author | Tomoya MORINAGA <tomoya.rohm@gmail.com> | 2012-01-11 21:27:06 -0500 |
---|---|---|
committer | Felipe Balbi <balbi@ti.com> | 2012-01-24 04:34:24 -0500 |
commit | c802672cd36cd063bfd54d54c8c34825ab5b2357 (patch) | |
tree | 862c8c34512f314eb8fc57388298484c087b729a /drivers | |
parent | c50a3bff0edb0acd49d8033a12ea4668e09a31ad (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')
-rw-r--r-- | drivers/usb/gadget/pch_udc.c | 4 |
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__, |