On Fri, Dec 13, 2019 at 01:05:55PM -0700, Bart Van Assche wrote: > On 12/13/19 5:49 AM, Dan Carpenter wrote: > > We know that "check_for_bkops" is non-zero on this side of the || > > because it was checked on the other side. > > How about also removing the superfluous parentheses? Anyway: > Around "(req_link_state == UIC_LINK_OFF_STATE)"? I considered it but some people like them... regards, dan carpenter