diff options
author | Felipe Balbi <balbi@ti.com> | 2015-06-29 20:17:22 -0500 |
---|---|---|
committer | Felipe Balbi <balbi@ti.com> | 2015-07-29 09:59:17 -0500 |
commit | 5a350d53f4c4784cf6bb4d94e7b8d76ca4f96e9f (patch) | |
tree | 11111aca309267db1a2f9b8cbe13cc2e9fe32db7 | |
parent | 5ea75095fe6d683900ccc674fcac375e7df68005 (diff) | |
download | op-kernel-dev-5a350d53f4c4784cf6bb4d94e7b8d76ca4f96e9f.zip op-kernel-dev-5a350d53f4c4784cf6bb4d94e7b8d76ca4f96e9f.tar.gz |
usb: dwc2: gadget: use | instead of + for bitmasks
It's just a lot clearer to use | operator instead of
+ operator.
Caught by coccicheck:
"
drivers/usb/dwc2/gadget.c:2883:26-27: WARNING: sum of probable
bitmasks, consider |
"
Cc: John Youn <John.Youn@synopsys.com>
Signed-off-by: Felipe Balbi <balbi@ti.com>
-rw-r--r-- | drivers/usb/dwc2/gadget.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/usb/dwc2/gadget.c b/drivers/usb/dwc2/gadget.c index 4d47b7c..731b13d 100644 --- a/drivers/usb/dwc2/gadget.c +++ b/drivers/usb/dwc2/gadget.c @@ -2880,7 +2880,7 @@ static int s3c_hsotg_ep_sethalt(struct usb_ep *ep, int value) epctl = readl(hs->regs + epreg); if (value) { - epctl |= DXEPCTL_STALL + DXEPCTL_SNAK; + epctl |= DXEPCTL_STALL | DXEPCTL_SNAK; if (epctl & DXEPCTL_EPENA) epctl |= DXEPCTL_EPDIS; } else { |