Commit 80f46d5d authored by Aaro Koskinen's avatar Aaro Koskinen Committed by Felipe Balbi

usb: phy: tahvo: fix smatch warnings

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: default avatarDan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: default avatarAaro Koskinen <aaro.koskinen@iki.fi>
Signed-off-by: default avatarFelipe Balbi <balbi@ti.com>
parent 2ee8ff30
...@@ -200,9 +200,6 @@ static int tahvo_usb_set_host(struct usb_otg *otg, struct usb_bus *host) ...@@ -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); dev_dbg(&tu->pt_dev->dev, "%s %p\n", __func__, host);
if (otg == NULL)
return -ENODEV;
mutex_lock(&tu->serialize); mutex_lock(&tu->serialize);
if (host == NULL) { if (host == NULL) {
...@@ -232,9 +229,6 @@ static int tahvo_usb_set_peripheral(struct usb_otg *otg, ...@@ -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); dev_dbg(&tu->pt_dev->dev, "%s %p\n", __func__, gadget);
if (!otg)
return -ENODEV;
mutex_lock(&tu->serialize); mutex_lock(&tu->serialize);
if (!gadget) { if (!gadget) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment