On 01/19/2013 03:30:30 AM, Albert ARIBAUD wrote:
        /* what I favor */
        clk_is_enabled = ((reg_val >> 9) & 1) ? true: false;
        ip_is_enabled = clk_is_enabled && pwd_is_enabled;
        if (clk_is_enabled) { ...

rather than assigning them 'zero/nonzero', or using bitwise ops on
booleans, or testing against boolean constants (although I concede
that the first line below wins over its counterpart above as far as
concision is concerned).

Conciseness can be improved with "!!((reg_val >> 9) & 1)".

-Scott
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to