diff options
| author | Fabio Estevam <fabio.estevam@freescale.com> | 2013-11-08 01:09:47 -0200 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-12-03 10:16:12 -0800 |
| commit | 46adcf3d82b44a4a9f147b2b898595d48873aa53 (patch) | |
| tree | 4eb800118326eaacb9da8d2b583932a1efe08080 /drivers/usb | |
| parent | b8c8fc6a47f4bb0691b0b0b39e54c58b4f6cea09 (diff) | |
| download | talos-op-linux-46adcf3d82b44a4a9f147b2b898595d48873aa53.tar.gz talos-op-linux-46adcf3d82b44a4a9f147b2b898595d48873aa53.zip | |
usb: chipidea: host: Only disable the vbus regulator if it is not NULL
Commit 40ed51a4b (usb: chipidea: host: add vbus regulator
control) introduced a smatch complaint because regulator_disable() is called
without checking whether ci->platdata->reg_vbus is not NULL.
Fix this by adding the check.
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/usb')
| -rw-r--r-- | drivers/usb/chipidea/host.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/usb/chipidea/host.c b/drivers/usb/chipidea/host.c index 59e6020ea753..526cd77563d8 100644 --- a/drivers/usb/chipidea/host.c +++ b/drivers/usb/chipidea/host.c @@ -88,7 +88,8 @@ static int host_start(struct ci_hdrc *ci) return ret; disable_reg: - regulator_disable(ci->platdata->reg_vbus); + if (ci->platdata->reg_vbus) + regulator_disable(ci->platdata->reg_vbus); put_hcd: usb_put_hcd(hcd); |
