diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2014-11-25 18:55:57 +0300 |
|---|---|---|
| committer | Felipe Balbi <balbi@ti.com> | 2015-01-12 21:13:26 +0300 |
| commit | 5e841efef7543209c1d62526311fbbcf85f3613d (patch) | |
| tree | 72427121710400a6d2b0f64228d33f82092f3bee /drivers/usb/gadget/udc/bdc/bdc_ep.c | |
| parent | 135b3c4304dacd8bb01f1aa6fe4165126bee3812 (diff) | |
| download | linux-5e841efef7543209c1d62526311fbbcf85f3613d.tar.xz | |
usb: gadget: udc: clean up a printk
We already know what "value" is, so there is no need to check. It
puzzles static checkers to have the unneeded condition.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Felipe Balbi <balbi@ti.com>
Diffstat (limited to 'drivers/usb/gadget/udc/bdc/bdc_ep.c')
| -rw-r--r-- | drivers/usb/gadget/udc/bdc/bdc_ep.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/drivers/usb/gadget/udc/bdc/bdc_ep.c b/drivers/usb/gadget/udc/bdc/bdc_ep.c index ff67ceac77c4..ebf82bd050cf 100644 --- a/drivers/usb/gadget/udc/bdc/bdc_ep.c +++ b/drivers/usb/gadget/udc/bdc/bdc_ep.c @@ -881,8 +881,8 @@ static int ep_set_halt(struct bdc_ep *ep, u32 value) ret = bdc_ep_set_stall(bdc, ep->ep_num); if (ret) - dev_err(bdc->dev, "failed to %s STALL on %s\n", - value ? "set" : "clear", ep->name); + dev_err(bdc->dev, "failed to set STALL on %s\n", + ep->name); else ep->flags |= BDC_EP_STALL; } else { @@ -890,8 +890,8 @@ static int ep_set_halt(struct bdc_ep *ep, u32 value) dev_dbg(bdc->dev, "Before Clear\n"); ret = bdc_ep_clear_stall(bdc, ep->ep_num); if (ret) - dev_err(bdc->dev, "failed to %s STALL on %s\n", - value ? "set" : "clear", ep->name); + dev_err(bdc->dev, "failed to clear STALL on %s\n", + ep->name); else ep->flags &= ~BDC_EP_STALL; dev_dbg(bdc->dev, "After Clear\n"); |