On Fri, Oct 24, 2014 at 4:29 PM, Daniel Thompson <[email protected]> wrote:
> irq_set_irq_wake() treats its second argument as a boolean. It is much > easier to read code when constant booleans are either 0 or 1! > > This particular line of code distracted me somewhat when I was doing a bit of > work in a code browser since it (spuriously) got me worried that I had > misunderstood how irq_set_irq_wake() worked. > > Signed-off-by: Daniel Thompson <[email protected]> > Cc: Linus Walleij <[email protected]> > Cc: Alexandre Courbot <[email protected]> Patch applied with Alexandre's Review tag. Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
