usb: phy: tahvo: fix smatch warnings
authorAaro Koskinen <aaro.koskinen@iki.fi>
Sat, 21 Dec 2013 18:37:36 +0000 (20:37 +0200)
committerFelipe Balbi <balbi@ti.com>
Mon, 23 Dec 2013 16:14:33 +0000 (10:14 -0600)
phy-tahvo introduced the following smatch warnings:

drivers/usb/phy/phy-tahvo.c:203 tahvo_usb_set_host() warn: variable dereferenced before check 'otg' (see line 199)
drivers/usb/phy/phy-tahvo.c:235 tahvo_usb_set_peripheral() warn: variable dereferenced before check 'otg' (see line 231)

Fix by deleting bogus NULL pointer checks. The USB framework will always
call us with a valid OTG pointer.

Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Aaro Koskinen <aaro.koskinen@iki.fi>
Signed-off-by: Felipe Balbi <balbi@ti.com>
drivers/usb/phy/phy-tahvo.c

index 8bb833e22d64efe8f93af42317304f8a1405ab19..cc61ee44b911dfa5a0b2842a827b5ebfed285f20 100644 (file)
@@ -200,9 +200,6 @@ static int tahvo_usb_set_host(struct usb_otg *otg, struct usb_bus *host)
 
        dev_dbg(&tu->pt_dev->dev, "%s %p\n", __func__, host);
 
-       if (otg == NULL)
-               return -ENODEV;
-
        mutex_lock(&tu->serialize);
 
        if (host == NULL) {
@@ -232,9 +229,6 @@ static int tahvo_usb_set_peripheral(struct usb_otg *otg,
 
        dev_dbg(&tu->pt_dev->dev, "%s %p\n", __func__, gadget);
 
-       if (!otg)
-               return -ENODEV;
-
        mutex_lock(&tu->serialize);
 
        if (!gadget) {