问题描述
以下代码不会更新初始 adcValue。 LED 在不同的程序下正常工作,并且在给定初始 adcValue 的情况下它们也正常工作。他们不响应电位器调整。延迟只是为了让它变慢,没有它它也不起作用。
#define F_cpu 1000000UL
#include <avr/io.h>
#include <avr/interrupt.h>
#include <util/delay.h>
double adcValue = 700;
void startConversion()
{
ADCSRA |= (1 << ADSC);
}
void setupADC()
{
// AVcc with external capacitor as reference,ADC5 as input
ADMUX = (1 << REFS0) | (1 << MUX0) | (1 << MUX2);
// Enabling ADC,acticating interrupts,setting prescaler to 8
ADCSRA = (1 << ADEN) | (1 << ADIE) | (1 << ADPS0) | (1 << ADPS1) ;
//disabling digital input buffer
DIDR0 = (1 << ADC5D);
startConversion();
}
ISR(ADC_vect)
{
adcValue = ADC;
}
int main(void)
{
DDRB = 0b00111111;
setupADC();
sei();
while(1)
{
_delay_ms(500);
if( adcValue < 400)
{
PORTB = 0b00000000;
}
else if ( adcValue < 800)
{
PORTB = 0b00000011;
}
else
{
PORTB = 0b00000111;
}
startConversion();
}
}
解决方案:
现在可以了
volatile double adcValue = 700;
解决方法
adcValue
正在得到优化,因为编译器没有“看到”ISR 中的变量变化(它无法判断 ISR 是否会被调用)。因此,它只会被替换为常量。解决此问题的方法是将其标记为 volatile
,这会告诉编译器不要对变量进行任何假设,并防止编译器对其进行优化。
现在,除此之外,adcValue
是一个双精度类型。通常,您希望避免在没有 FPU 的设备中使用浮点(尤其是双精度)变量。具体来说,整数和浮点类型之间的转换需要很多周期。就像,在数百个周期的数量级上。您可能不希望在您的 ISR 中使用它。
如果您确实需要变量为浮点数,我建议在 ISR 之外进行转换。您将有两个变量,static volatile uint16_t wAdcValue
和需要浮点数的本地变量,您可以分配 float fAdcValue = (float) wAdcValue;
请注意,任何浮点操作都需要更多的处理和闪存使用来处理它。