On Thu, Nov 29, 2007 at 01:06:26PM -0700, Shaun Jackman wrote: > > Each fuse bit is currently defined as an inverted mask. For example: > #define BOOTRST ~_BV(0) /* Select Reset Vector */ > I would prefer the flags be defined uninverted. > EFUSE = ~(BOOTRST | BOOTSZ0 | BOOTSZ1);
Since this describes _what_ is intended, in positive logic, it is natural and clear, I believe. > looks better to me than > EFUSE = BOORST & BOOTSZ0 & BOOTSZ1; This De Morgan equivalent is aligned with _how_ the physical fuses work, but employing negated logic isn't normal usage. It works, but it's not so convenient. Erik _______________________________________________ AVR-libc-dev mailing list [email protected] http://lists.nongnu.org/mailman/listinfo/avr-libc-dev
