Question

我一直在尝试通过按相应的按钮来打开和关闭Arduino上的LED。

我正在使用中断来使它发生并且按钮的确被注册了，但是由于某种原因，它并没有改变全局变量的值（int button_pressed1，...）；

应该发生的是，当我按下按钮1时，LED 1应该被打开和关闭，与按钮2和按钮3相同。

我非常感谢您看一下，中断对我来说是很新的，因此可能是一个小小的忽略。 <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;
}

Answer 1

几个基本问题：

与ISR共享的所有变量都应声明为volatile，并具有防止竞争条件的保护。有关详细信息，请参见this。
ISR内不应有繁忙延迟。相反，您应该将计时器中断设置为在特定时间段内再次触发。通常，很难为按钮专门使用GPIO中断，最好使用循环定时器中断进行轮询。您可以使用中断，但是细节here相当复杂。
500us可能没有足够的时间进行反跳。机械信号跳动相对较慢。等待〜10ms左右较为常见。通过向按钮添加电源，然后按下按钮并捕获边缘，您可以很容易地用示波器测量反弹特性。它看起来像一个正弦曲线，您可以轻松地测量它会持续多长时间。
主循环中的500ms延迟将被用户视为“滞后”。他们可能开始怀疑按钮是否损坏。您可能希望至少跳过关闭延迟。

c arduino按钮中断（ISR），使多个LED亮起和熄灭

1 个答案: