Question

我最近在玩ATmega2560时遇到了一个问题，我真的不明白它有什么问题。

这是我的代码。

主：

#include "Definitions.h"

int main(void) {
    Initialization();
    while (1) {
        //_delay_ms(1); // or printf...
        //wait for RXC flag
        if (RxFlag) {
            //wait for new byte in
            //PORTB &= ~(1 << PB7);
            PORTB |= (1 << PB7);
            rxcount = 0;
            UDR0 = 'R';
            RxFlag = false;
            TxFlag = false;
        }
    }
}

IRQ：

#include "Definitions.h"

ISR(USART0_RX_vect) {
    while(!(UCSR0A & (1 << RXC0)))
    //wait for RXC flag
        ;
    /* Loop-back test */
    //PORTB |= (1 << PB7);
    //test_data = UDR0;
    TxFlag = true;
    //enableUDRI0();
    //PORTB &= ~(1 << PB7);
    //PORTB |= (1 << PB7);
    RxBuffer[rxcount++] = UDR0;
    if(!(rxcount < RX_BUF_SZ)) {
        //rxcount = 0;
        PORTB |= (1 << PB7);
        RxFlag = true;
    }
}

ISR(USART0_UDRE_vect) {
    while(!(UCSR0A & (1 << UDRE0)))
    //wait for udr to be empty
        ;
    /* Loop-back test */
    UDR0 = 0x30 + rxcount;
    disableUDRI0();
}

问题是，当我从USART收到一些数据时，我试图设置一个可以在主循环中激活if语句的RxFlag。但是直到我在if语句之前取消注释该函数（可能是_delay_ms（）或printf（））它才能工作。

没有意义。我记得的是，我不需要那些功能，它仍然可以设置全局变量来影响主循环。还是我错过了任何细节？请给我一些线索来弄明白，我很困惑。

Answer 1

不可能确定，但我认为这解释了你的问题，所以很可能是错误。

您不包含RXFlag的声明，但我猜测您没有声明volatile。除非你这样做，否则允许编译器假设它知道该变量会发生什么。特别是它知道，除非main()调用任何其他函数，否则RxFlag的值永远不会改变。因此，它可以通过将它移到循环外来优化if表达式。实际上它正在改变你的代码：

#include "Definitions.h"

int main(void)
{
    register bool x;

    Initialization();

    x = RxFlag;

    while (1) 
    {
        //wait for RXC flag
        if (x) //wait for new byte in
        {
            //PORTB &= ~(1 << PB7);
            PORTB |= (1 << PB7);
            rxcount = 0;
            UDR0 = 'R';
            RxFlag = false;
            TxFlag = false;

        }
    }
}

但是，如果main()调用另一个函数，则编译器放弃尝试将RxFlag保留在该调用的寄存器中。另一个函数可能会更改RxFlag所在的寄存器。它必须将其推入堆栈或其他东西。再次阅读它在代码和时间上会更便宜，因此它会在每次循环时读取它。

此错误的正确解决方法是将主线程和ISR之间共享的任何变量声明为volatile。

ATmega2560使用UART中断来控制全局标志

1 个答案: