在C中更改全局变量
我正在AVR芯片上运行一个C程序。每当听到串行信号时,它就会运行串行中断ISR(USART_RX_vect)。在这个方法中,它应该打开对在C中更改全局变量,c,avr-gcc,C,Avr Gcc,我正在AVR芯片上运行一个C程序。每当听到串行信号时,它就会运行串行中断ISR(USART_RX_vect)。在这个方法中,它应该打开对=1的更改。然后在我的主循环中,它应该清除LCD并显示,然后再次设置change=0 这是为了阻止它继续进行计算,并在LCD上以每分钟一百万次的速度显示结果 但是,当中断方法将更改变量更改为1时,它似乎不会“全局”更改它,并且在主方法中它始终为0 这里有一些东西是为了调试的目的 /* LCD DEFINES */ #define LED PB5 #define
=1的更改代码>。然后在我的主循环中,它应该清除LCD并显示,然后再次设置change=0
这是为了阻止它继续进行计算,并在LCD上以每分钟一百万次的速度显示结果
但是,当中断方法将更改变量更改为1时,它似乎不会“全局”更改它,并且在主方法中它始终为0
这里有一些东西是为了调试的目的
/* LCD DEFINES */
#define LED PB5
#define output_low(port,pin) port &= ~(1<<pin)
#define output_high(port,pin) port |= (1<<pin)
#define set_input(portdir,pin) portdir &= ~(1<<pin)
#define set_output(portdir,pin) portdir |= (1<<pin)
/* UART SERIAL DEFINES */
#define F_CPU 16000000UL
#define BAUD 9600
#define MYUBRR F_CPU/16/BAUD-1
#define STARTCHAR 'R'
#define ENDCHAR 'E'
char reading;
char inputBuffer[12];
char readStatus;
uint8_t position;
int change;
char output;
int result;
struct Axis
{
uint8_t axisNumber;
uint16_t position;
uint16_t oldPosition;
} axis1, axis2, axis3;
/* SETUP UART */
void USART_Init( unsigned int ubrr)
{
/*Set baud rate */
UBRR0H = (unsigned char)(ubrr>>8);
UBRR0L = (unsigned char)ubrr;
/*Enable receiver and transmitter */
UCSR0B = (1<<RXEN0)|(1<<TXEN0);
/* Set frame format: 8data, 2stop bit */
UCSR0C = (1<<USBS0)|(3<<UCSZ00);
}
void USART_Transmit( unsigned char data )
{
UDR0 = data;
}
unsigned char USART_Receive( void )
{
return UDR0;
}
/*****************************************************************/
int main(void)
{
/* INITALISE SERIAL */
USART_Init(MYUBRR);
/* Turn on Receive Complete Interrupt */
UCSR0B |= (1 << RXCIE0);
/* Turn On GLobal Interrupts */
sei();
position = 0;
change = 0;
/* Initialise LCD */
lcd_init(LCD_DISP_ON); /* Initialize display, cursor off. */
lcd_clrscr();
lcd_puts("READY");
//Turn on LED 13
set_output(PORTB,LED);
output_low(PORTB,LED);
while (1) /* Loop forever */
{
if (change == 1)
{
//If not reading, display the result on the LCD display.
axis1.position = (inputBuffer[0]<< 8) | inputBuffer[1];
axis2.position = (inputBuffer[2]<< 8) | inputBuffer[3];
axis3.position = (inputBuffer[4]<< 8) | inputBuffer[5];
char axis1Printout[12];
char axis2Printout[12];
char axis3Printout[12];
sprintf(axis1Printout,"%u ", axis1.position);
sprintf(axis2Printout,"%u ", axis2.position);
sprintf(axis3Printout,"%u ", axis3.position);
char output[40] = "";
strcat(output, axis1Printout);
strcat(output, axis2Printout);
//strcat(output, axis3Printout);
lcd_clrscr(); /* Clear the screen*/
lcd_puts(output);
_delay_ms(300);
change = 0;
}
}
}
/* INTERRUPTS */
ISR (USART_RX_vect)
{
change = 1;
unsigned char input = USART_Receive();
if (input == 'R')
{
readStatus = 0; //Reading
position = 0;
}
else if ((input != 'E') && (position < 12) && (position > -1))
{
inputBuffer[position] = input;
position++;
}
else if (input == 'E')
{
readStatus = 1; //Stop Reading
position = -1;
output_high(PORTB,LED);
}
}
/*LCD定义*/
#定义LED PB5
#定义输出低(端口、pin)端口&=~(1您需要使用volatile关键字声明更改:
volatile int change;
这告诉两个“线程”(主执行循环和ISR代码)不要“缓存”寄存器中的值,而是始终从内存中检索它
编辑:代码还有一个问题-在您的主循环中,当您将更改设置为0时,您可能已经有另一个中断,该中断本应触发循环再次运行。简单但不保证的修复方法是在检查后立即将更改设置为0。正确的方法是使用锁-但取决于在您的情况下,第一个选项可能会这样做。进行变量声明,以确保更改的值立即写入内存中的变量。中断处理程序共享的对象和应用程序代码在声明中应限定为volatile
如果没有限定符,实现可以假设对象在应用程序代码中不会意外更改,并且可以缓存变量(例如在寄存器中)用于在执行应用程序代码时进行优化。是否确实调用了ISR?我会在最后保留更改,并确保在更改为0之前中断不会更新值。这将防止在主例程读取值时中断损坏值。另一个选项是保留interrupt标志在中断本身中被清除,并保留主例程以重新启用中断。我本人不熟悉AVR,但处理器通常不会在中断时清除中断标志,您需要调用sei()在中断例程结束时?您可以在完成处理后将其保留在主例程中-在我使用的大多数处理器上,它将在禁用全局中断时对中断进行排队,因此如果数据在处理时到达,则在启用它们时将中断。缓冲区长度可能是一个问题。