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

Reply via email to