On Sun, Aug 21, 2016 at 10:11:20AM +0800, Chen-Yu Tsai wrote: > + if (id == AXP806_DCDCB) { > + return (((reg & GENMASK(7, 6)) == BIT(6)) || > + ((reg & GENMASK(7, 6)) == BIT(7))); > + } else if (id == AXP806_DCDCC) { > + return ((reg & GENMASK(7, 6)) == BIT(7)); > + } else if (id == AXP806_DCDCE) { > + return !!(reg & BIT(5)); > + } This looks like you're trying to write a switch statement, please do that.
Attachment:
signature.asc
Description: PGP signature