问题描述
我一直在尝试通过按相应的按钮来打开和关闭Arduino上的LED。
我正在使用中断来使它发生并且按钮的确被注册了,但是由于某种原因,它并没有改变全局变量的值(int button_pressed1,...);
应该发生的是,当我按下按钮1时,LED 1应该被打开和关闭,与按钮2和按钮3相同。
我非常感谢您看一下,中断对我来说是很新的,因此可能是一个小小的忽略。
*我省去了button2和3的代码。如果我可以使LED在按钮1上点亮,那么我将能够使其他两个按钮都点亮。
#include <util/delay.h>
#include <avr/io.h>
#include <avr/interrupt.h>
#include "usart.h"
#define LED_DDR DDRB
#define LED_PORT PORTB
#define BUTTON_DDR DDRC
#define BUTTON_PORT PORTC
#define BUTTON_PIN PINC
int button_pressed1 = 0; //globale variabele to turn on functions
ISR(PCINT1_vect)
{
if (bit_is_clear(BUTTON_PIN,PC1))
{
_delay_us(500); //debounce
if (bit_is_clear(BUTTON_PIN,PC1))
{
button_pressed1 = 1;
printf("button 1 pressed\n");
}
}
}
int main()
{
LED_DDR |= _BV(PB2) | _BV(PB3) | _BV(PB4); //registrer pins output(bit = 1)
LED_PORT |= _BV(PB2) | _BV(PB3) | _BV(PB4);
BUTTON_DDR &= ~_BV(PC1) & ~_BV(PC2) & ~_BV(PC3); //registrer inputs(bit = 0)
BUTTON_PORT |= _BV(PC1) | _BV(PC2) | _BV(PC3); // pull up ( bit =1 )
PCICR |= _BV(PCIE1); //type pins doorgeven
PCMSK1 |= _BV(PC1) | _BV(PC2) | _BV(PC3); //pin/button doorgeven aan change mask
initUSART();
sei();
while (1)
{ //infinte loop
if (button_pressed1 == 1)
{
LED_PORT &= ~_BV(PB2); //turn led on
_delay_ms(500);
LED_PORT |= _BV(PB2); //turn led off
_delay_ms(500);
}
}
return 0;
}
解决方法
几个基本问题:
- 与ISR共享的所有变量都应声明为
volatile
,并具有防止竞争条件的保护。有关详细信息,请参见this。 - ISR内不应有繁忙延迟。相反,您应该将计时器中断设置为在特定时间段内再次触发。通常,很难为按钮专门使用GPIO中断,最好使用循环定时器中断进行轮询。您可以使用中断,但是细节here相当复杂。
- 500us可能没有足够的时间进行反跳。机械信号跳动相对较慢。等待〜10ms左右较为常见。通过向按钮添加电源,然后按下按钮并捕获边缘,您可以很容易地用示波器测量反弹特性。它看起来像一个正弦曲线,您可以轻松地测量它会持续多长时间。
- 主循环中的500ms延迟将被用户视为“滞后”。他们可能开始怀疑按钮是否损坏。您可能希望至少跳过关闭延迟。