On Fri, May 24, 2019 at 01:19:26PM +0100, Jeremy Sowden wrote: > kp2000_check_uio_irq contained a pair of nested ifs which each evaluated > a bitwise operation. If both operations yielded true, the function > returned 1; otherwise it returned 0. > > Replaced the whole thing with one return statement that evaluates the > combination of both bitwise operations. > > Signed-off-by: Jeremy Sowden <jeremy@xxxxxxxxxx> > --- > This applies to staging-testing. > > I was in two minds whether to send this patch or something less terse: > > + return (interrupt_active & irq_check_mask) && > + (interrupt_mask_inv & irq_check_mask); Yeah. I would prefer this. regards, dan carpenter _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
