Question

我想读取LEDS旁边的开关，并将LEDS从0循环到按下任何一个开关如果没有按下它们会延迟所有这些延迟。为此我使用了timer0。自从我工作以来 ATmega8515的。我用过INT0。这是我的实施：

#include <avr/io.h>
#include <avr/interrupt.h>
#include <util/delay.h>

#define BIT(n) (1 << n) 

volatile uint8_t led, k, switches, i , j = 1;
            /* uint8_t is the same as unsigned char */
int main(void)
{



    DDRB = 0xff;                /* use all pins on PortB for output */

    led = 1;                    /* init variable representing the LED state */
    PORTB = 0XFF;

    cli( );

    TCCR0|=(1<<CS02) |(1<<CS00);

 //Enable Overflow Interrupt Enable
   TIMSK|=(1<<TOIE0);

 //Initialize Counter
   TCNT0=0;


   GICR = BIT(INT0);
    MCUCR = BIT(ISC01) | BIT(ISC00);

   sei( );

    for ( ; ;);



}


ISR(TIMER0_OVF_vect)
{

  if(switches == 0xff)
  {
    PORTB = ~led;           /* invert the output since a zero means: LED on */
        led <<= 1;              /* move to next LED */
        if (!led)               /* overflow: start with Pin B0 again */
        {
            led = 1;
        }
  }     
  else
  {
    for (k = 0; k< 8;k++)
    {
        j =  switches & (1 << k);

        if(j == 0)
        {
           for(i=1;i<=(k +1);i++)
            {   
                j = 1;
                PORTB =  ~j;
                j = 1 << i;

                _delay_ms(100);     //without this delay it doesnt cycle the LEDS from to whichever switch is pressed
            }   

    }
}
}

但是在ISR中使用延迟循环是一种糟糕的编程习惯。如何使用相同的计时器而不是延迟？

Answer 1

我认为在ISR中，您应该只更新LED状态，在主循环中您可以设置PORTB并读取开关值。在你的代码中，似乎在ISR中占用了这么多时间。

在LEDEGA8515中循环LEDS

1 个答案: