As Senthil Kumar Selvaraj wrote:

> The attached patch to wdt.h gets rid of the huge device specific 
> conditional branches for wdt_enable and wdt_disable.

That's cool.  The current state of affairs used to be a continued
cause for being forgotten upon adding a new devices.

> I do see bigger code at -O0 though. Is that an acceptable tradeoff?

For wdt_enable/wdt_disable, yes, it is.  Only wdt_reset() must be
kept as short as possible, but as this translates directly into
one assembly instruction, that's not an issue.
-- 
cheers, Joerg               .-.-.   --... ...--   -.. .  DL8DTL

http://www.sax.de/~joerg/
Never trust an operating system you don't have sources for. ;-)

_______________________________________________
AVR-libc-dev mailing list
AVR-libc-dev@nongnu.org
https://lists.nongnu.org/mailman/listinfo/avr-libc-dev

Reply via email to