diff options
author | Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com> | 2017-07-28 12:28:57 +0200 |
---|---|---|
committer | Felipe Balbi <felipe.balbi@linux.intel.com> | 2017-08-03 11:33:05 +0200 |
commit | b7d44c36a6f6d956e1539e0dd42f98b26e5a4684 (patch) | |
tree | 33a3e4b37c21ed3e0b37bbf363033519bfa04e08 /drivers/leds | |
parent | usb: renesas_usbhs: Fix UGCTRL2 value for R-Car Gen3 (diff) | |
download | linux-b7d44c36a6f6d956e1539e0dd42f98b26e5a4684.tar.xz linux-b7d44c36a6f6d956e1539e0dd42f98b26e5a4684.zip |
usb: renesas_usbhs: gadget: fix unused-but-set-variable warning
The commit b8b9c974afee ("usb: renesas_usbhs: gadget: disable all eps
when the driver stops") causes the unused-but-set-variable warning.
But, if the usbhsg_ep_disable() will return non-zero value, udc/core.c
doesn't clear the ep->enabled flag. So, this driver should not return
non-zero value, if the pipe is zero because this means the pipe is
already disabled. Otherwise, the ep->enabled flag is never cleared
when the usbhsg_ep_disable() is called by the renesas_usbhs driver first.
Fixes: b8b9c974afee ("usb: renesas_usbhs: gadget: disable all eps when the driver stops")
Fixes: 11432050f070 ("usb: renesas_usbhs: gadget: fix NULL pointer dereference in ep_disable()")
Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
Signed-off-by: Felipe Balbi <felipe.balbi@linux.intel.com>
Diffstat (limited to 'drivers/leds')
0 files changed, 0 insertions, 0 deletions