Wed 13 Nov 2013 05:14:47 PM UTC, original submission:
The implementation of sleep_bod_disable for non-xmega mcu's is as follows (from avr/sleep.h):
#define sleep_bod_disable()
do {
uint8_t tempreg;
_asm_ __volatile__("in %[tempreg], %[mcucr]" "nt"
"ori %[tempreg], %[bods_bodse]" "nt"
"out %[mcucr], %[tempreg]" "nt"
"andi %[tempreg], %[not_bodse]" "nt"
"out %[mcucr], %[tempreg]"
: [tempreg] "=&d" (tempreg)
: [mcucr] "I" _SFR_IO_ADDR(MCUCR),
[bods_bodse] "i" (_BV(BODS) | _BV(BODSE)),
[not_bodse] "i" (~_BV(BODSE)));
} while (0)
However, this hardcodes MCUCR, while attiny13 uses BODCR for these bits (see page 33 of http://www.atmel.com/Images/doc8126.pdf).
A grep on BODSE and BPDSE (a typo, see #40567) shows that tn2313a and tn4313 also have the same problem.
Perhaps adding something like:
#ifdef BODCR
#define BODSE_TEMP_REG BODCR
#else
#define BODSE_TEMP_REG MCUCR
#endif
and then use BODSE_TEMP_REG in the ASM snippet would fix this?
|