Messages in this thread | | | Date | Fri, 15 Oct 2021 12:43:44 +0300 | From | Dan Carpenter <> | Subject | Re: [PATCH] staging: wlan-ng: Avoid bitwise vs logical OR warning in hfa384x_usb_throttlefn() |
| |
On Thu, Oct 14, 2021 at 02:57:03PM -0700, Nathan Chancellor wrote: > A new warning in clang points out a place in this file where a bitwise > OR is being used with boolean expressions: > > In file included from drivers/staging/wlan-ng/prism2usb.c:2: > drivers/staging/wlan-ng/hfa384x_usb.c:3787:7: warning: use of bitwise '|' with boolean operands [-Wbitwise-instead-of-logical] > ((test_and_clear_bit(THROTTLE_RX, &hw->usb_flags) && > ~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > drivers/staging/wlan-ng/hfa384x_usb.c:3787:7: note: cast one or both operands to int to silence this warning > 1 warning generated.
Both sides of this bitwise OR are bool, so | and || are equivalent logically. Clang should not warn about it.
regards, dan carpenter
| |