usb: dwc3: keystone: fix error return code
authorJulia Lawall <Julia.Lawall@lip6.fr>
Thu, 20 Nov 2014 17:33:58 +0000 (18:33 +0100)
committerFelipe Balbi <balbi@ti.com>
Thu, 20 Nov 2014 19:35:46 +0000 (13:35 -0600)
Return a negative error code on failure.

A simplified version of the semantic match that finds this problem is as
follows: (http://coccinelle.lip6.fr/)

// <smpl>
@@
identifier ret; expression e1,e2;
@@
(
if (\(ret < 0\|ret != 0\))
 { ... return ret; }
|
ret = 0
)
... when != ret = e1
    when != &ret
*if(...)
{
  ... when != ret = e2
      when forall
 return ret;
}
// </smpl>

Signed-off-by: Julia Lawall <Julia.Lawall@lip6.fr>
Signed-off-by: Felipe Balbi <balbi@ti.com>
drivers/usb/dwc3/dwc3-keystone.c

index dd8d2df6db317d0f084dd5a077bd04d109928673..fe3b9335a74e1d279da69b73e80897ca2a70d5f0 100644 (file)
@@ -123,6 +123,7 @@ static int kdwc3_probe(struct platform_device *pdev)
        irq = platform_get_irq(pdev, 0);
        if (irq < 0) {
                dev_err(&pdev->dev, "missing irq\n");
+               error = irq;
                goto err_irq;
        }