Olimex P28 Board Problem

[Nicolas K. (Gast)]

Hallo Mikrocontroller.net Gemeindschaft

Ich hab mir das AVR Starterkit (inkl. USB Programmer) hier im Shop 
gekauft alles ok bis her.


Aber wenn ich den Quelltext unten compiliere leuchtet einfach die Led 
auf dem Board auch wenn ich nichts gedrückt habe. Das kommt öffters vor 
das der Quelltext nicht so ausgeführt wird wie ich es versucht hab zu 
programmieren.
Im Simulator läuft das Programm richtig ab nur auf dem Mikrocontroller 
nicht.

C/C++ programmier ich schon seit 4 Jahren.

Ich versuche momentan als Übung verschiedenen Tasterzustände zu 
programmieren, um mich in die Materie der uC einzuarbeiten.
Vielleicht hab ich doch etwas falsch gemacht.

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

#include <stdint.h>


uint16_t time = 0;


ISR( TIMER0_OVF_vect );

void blink(int8_t i);


int main(void)
{
  
  uint8_t click = 0;
  


  DDRC |= ( 1 << PC5 );
  DDRD &= ~( 1 << PIND2 );
  
  TIMSK |= ( 1 << TOIE0 );
  
  
  sei();
  
  
    while(1)
    {
    
    
    if( time < 2200 || click != 2)
    {
      
      if( PIND & ( 1 << PIND2 ) )
      {
        
        if( TCCR0 == 0)
        {
          
            TCCR0 |= ( 1 << CS00 ) | ( 1 << CS01 );
          
            click+=1;
          
        }
        
      }else
      {
        
        if( TCCR0 != 0)
        {
          
          TCCR0 = 0x00;
          
        }
        
      }
      
    }else
    {
      
      if( TCCR0 != 0)
      {
        
        TCCR0 = 0x00;
        
      }
      
      
      
      if( time < 2000 )
      {
        
        if( click < 2)
        {
          
          blink( 1 );
          
        }else
        {
          
          blink( 2 );
          
        }
        
      }else
      {
        
        blink( 3 );
        
      }
      
      
      click = 0;
      time = 0;
      
    }
    
    
      
    
    
    
    }
  
}



ISR( TIMER0_OVF_vect )
{
  
  time++;
  
}


 

void blink( int8_t i)
{
  
  for( int8_t j = 0; j < i; j++)
  {
    
    PORTC |= ( 1 << PC5);
    
    _delay_ms( 500 );
    
    PORTC &= ~( 1 << PC5);
    
    _delay_ms( 500 );
    
  }
  
}

[Stefan E. (sternst)]

1. "volatile" fehlt.
2. Taster-Logik ist verkehrt herum.
3. Zugriffe auf time in main() sind nicht atomar.
4. LED-Logik ebenfalls verkehrt herum.

[Nicolas K. (Gast)]

Danke für die Fehleranalyse.
Noch paar fragen:

Was kann ich unter Taster/Led-Logik verstehen ?
Und zugriffe auf time in main() sind nicht atomar. ?

[Stefan E. (sternst)]

Nicolas K. schrieb:
> Was kann ich unter Taster/Led-Logik verstehen ?

Vor dem Schreiben der Software musst du halt mal einen Blick auf die 
Hardware (Schaltplan) werfen. Einfach anzunehmen, dass ein gedrückter 
Taster am Port ein High erzeugt, ist ganz schlecht. Ebenso falsch ist 
es, einfach anzunehmen, dass ein High am Port die LED einschaltet.

Nicolas K. schrieb:
> Und zugriffe auf time in main() sind nicht atomar. ?

Für deine ersten Spielereien hier kannst du das erst mal außer Acht 
lassen. Aber für spätere "richtige" Programme ist dieses Thema SEHR 
wichtig. Eigentlich hätte ich erwartet, dass jemand der
> C/C++ programmier ich schon seit 4 Jahren.
etwas mit dem Begriff "atomar" anfangen kann. Wenn nicht, dann ->Google 
oder ->Wikipedia.

[Nicolas K. (Gast)]

Ok hab mir mal den Schaltplan angeschaut und es ist ein low Signal :D
Und jetzt weiß ich auch was mit atomar gemeint ist.
In c++ wird die Klasse atomic genannt deswegen konnt ich nichts mit 
atomar anfangen :D

Noch mal vielen Dank für die hilfe.