Interruptroutine LED

[Jan M. (pietus3)]

Guten Abend.

Ich wende mich an euch nach 2 Tagen Kopfschmerzen und ich hoffe ihr 
könnt mir helfen. Ich versuche mich an Interruptroutinen heranzuwagen 
und möchte dafür eine LED blinken lassen erstmal egal, welches 
Intervall. Ich programmiere unter der Arduino IDE. Der Mikroprozessor 
ist ein ATmega328.

#define F_CPU 1000000UL
 
#include <avr/io.h>
#include <avr/interrupt.h>
#include <util/delay.h>

boolean test;

ISR( TIMER1_COMPA_vect)                // Interruptbehandlungsroutine
{
  if (test ==true){
  
    test = false;
    
  }
  
  else{
  
    test= true;
  
  }    
}

void setup() {
  DDRB = 0b00000000;
  PORTB |= (1<<PORTB5);           // Pullup für PB0 und PB1
 
  TCCR1A = (1<<COM1A0);                 // Togglen bei Compare Match
  TCCR1B = (1<<WGM12) | (1<<CS11);      // CTC-Mode; Prescaler 8
  TIMSK1  = (1<<OCIE1A);                 // Timer-Compare Interrupt an
  test = false;
 
  OCR1A = 1000;                         // Neutralposition ((2500-2312)*0.008ms)=1,5ms)
 
  sei();                                // Interrupts global an

}

void loop() {
  if (test == true ) {       // Impuls-Zeit verlängern
      PORTB |= (1<<PORTB5); 
    }
 
    else{       // Impuls-Zeit verkürzen
      PORTB &= ~(1<<PORTB5); 
    }
    
  
}

Ich danke euch für eure Tipps und Vorschläge.

MFG Pietus3

[TH (Gast)]

Ist das alles? Wo ist deine Main Funktion?

Jan Meyer schrieb:
> Ich versuche mich an Interruptroutinen heranzuwagen

Dafür sieht dein Programm aber schon ganzschön komplex aus!

[holger (Gast)]

>  DDRB = 0b00000000;

Kein Ausgang für die LED definiert.

>boolean test;

volatile boolean test;

>  OCR1A = 1000;                         // Neutralposition 
>((2500-2312)*0.008ms)=1,5ms)

Die "blinkt" dann mit 333Hz. Das kann man nicht sehen.

[Jan Peter Meyer (Gast)]

Meine main function besteht aus Loop und Setup, weil ich eben unter der 
Arduino IDE programmiere.

[Jan Peter Meyer (Gast)]

Danke teste ich sofort morgen Holger

[TH (Gast)]

Jan Peter Meyer schrieb:
> Meine main function besteht aus Loop und Setup, weil ich eben unter der
> Arduino IDE programmiere.

Achso sorry... mit arduino habe ich noch nichts am hut gehabt.

Jan Meyer schrieb:
> TCCR1A = (1<<COM1A0);                 // Togglen bei Compare Match

Du könntest deine LED übrigends auch an PB1 anschließen (vorher aber im 
DDRB auf Ausgang stellen) da der hardwareseitig bei jedem Compare-Match 
schon getoggelt wird. Den Code im loop bräuchtest du also erstmal gar 
nicht.


Gruß

[tommy (Gast)]

Jan Meyer schrieb:
> #define F_CPU 1000000UL

1 MHz ?
Die laufen eigentlich mit 16 MHz (einige auch mit 8 MHz).
Welchen Arduino benutzt Du?

[Jan Peter Meyer (Gast)]

Arduino Uno.

[Ralph S. (jjflash)]

... hmmmmmm grübel... Wenn du schon die Interrupts so deklarierst wie du 
sie deklarierst ... und die DDR Register (müßte das dann nicht 
DD-Register richtigerweise heißen) von Hand setzt (wenn auch wie oben 
genannt falsch: eine 1 definiert am Anschluß einen Ausgang, eine 0 einen 
Eingang)...

Warum ... benutzt man dann noch Arduino wenn das FAST schon reines C 
(nicht C++) ist ?

Wäre das dann nicht einfach einfacher zu handhaben ?

[Jan M. (pietus3)]

Danke jetzt blickt die LED.

Wenn man mal richtig überlegt waren das pure Anfängerfehler. 
Entschuldigung nochmal.

MFG Pietus3

[Karl H. (kbuchegg)]

Exakt.

So was

  DDRB = 0b00000000;

ist fast immer schlecht. Wozu Binärzahlen? Da sieht man so schön - gar 
nichts.

So was

  if (test ==true){

ist auch ein typischer Fall fürs selber ins Knie schiessen.

test ist schon offensichtlich ein boolscher Wert (mit einem bescheuerten 
Namen). Den muss man nicht auf true oder false abfragen.

  if (test)

oder noch besser: die Variable auf einen ordentlichen Namen umnennen

  if (LightOn)

und die Sache ist geritzt.

#define F_CPU 1000000UL
 
#include <avr/io.h>
#include <avr/interrupt.h>
#include <util/delay.h>

boolean LightOn;

#define LED_DDR   DDRB
#define LED_PORT  PORTB
#define LED       PB5

ISR( TIMER1_COMPA_vect)                // Interruptbehandlungsroutine
{
  LightOn = !LightOn;
}

void setup()
{
  LightOn = false;
  LED_DDR |= ( 1 << LED );
  LED_PORT |= ( 1 << LED );
 
  TCCR1A = (1<<COM1A0);                 // Togglen bei Compare Match
  TCCR1B = (1<<WGM12) | (1<<CS11);      // CTC-Mode; Prescaler 8
  TIMSK1  = (1<<OCIE1A);                 // Timer-Compare Interrupt an
  OCR1A = 1000;                         // Neutralposition ((25002312)*0.008ms)=1,5ms)

  sei();                                // Interrupts global an
}

void loop()
{
  if (LightOn)
    LED_PORT |= ( 1 << LED );
  else
    LED_PORT &= ~( 1 << LED );
}

Ein Schimmel ist per Definition ein weißes Pferd. Das muss man nicht 
extra abfragen, ob er weiß ist. Eine boolsche Variable ist per 
Definition schon bool. Die muss man nicht extra abfragen, ob sie true 
ist.

[Svenska (Gast)]

Und wenn man die Variablen, die in einer ISR und im Hauptprogramm 
benutzt werden, dann auch noch als "volatile" deklariert, könnte das 
sogar funktionieren... :-)

[Quack (Gast)]

Karl Heinz schrieb:
> ist auch ein typischer Fall fürs selber ins Knie schiessen.

Das schiesst in gar kein Knie. Explizit ausgeschriebene Vergleiche sind 
guter Stil, da lesbarer. Nachteile gibt es gar keine, ausser, dass es 
kein cooles C ist.

[Karl H. (kbuchegg)]

Quack schrieb:
> Karl Heinz schrieb:
>> ist auch ein typischer Fall fürs selber ins Knie schiessen.
>
> Das schiesst in gar kein Knie. Explizit ausgeschriebene Vergleiche sind
> guter Stil, da lesbarer. Nachteile gibt es gar keine, ausser, dass es
> kein cooles C ist.

ACh
Du schreibst wohl auch

  if( ( i == 5 ) == true )

i == 5 ist ein boolscher Wert. Genauso wie LightOn im obigen Beispiel.


Du schreibst auch

  if( ( PINB & ( 1 << PB4 ) ) == ( 1 << PB4 ) )

weil es doch guter Stil ist, einen expliziten Vergleich zu schreiben, 
obwohl er nicht nötig ist (und nur Fehlerpotential bietet)?

Was bitte ist an

  if( isKeyPressed() )

nicht lesbar und warum wird es durch

  if( isKeyPressed() == true )

lesbarer?
Wie sagst du in Deutsch?
Sagst du: Wenn die Taste gedrückt ist

   if( isKeyPressed() ) )

oder sagst du:
Wenn es wahr ist, dass die Taste gedrückt ist

   if( isKeyPressed() == true )

Die Fragestellung "Ist die Taste gedrückt?" ist für sich alleine schon 
eine perfekte boolsche Abfrage. Die muss man nicht noch mal explizit auf 
true oder false testen. Man gewinnt nichts dadurch, wenn man es tut. Man 
kann es natürlich tun, aber in der Realität ist das einfach nur 
zusätzliche Tipparbeit mit der Gefahr in diesem Zusatz Fehler einbauen 
zu können.
Selbiges hier: Der Inhalt der Variablen LightOn ist die Antwort auf die 
Fragestellung "Soll das Licht eingeschaltet sein?". Der Wert in der 
Variablen (true oder false, wobei der Variablenname so gewählt wird, 
dass er mit den möglichen 2 Werten in der Fragestellung korreliert) ist 
bereits die perfekte boolsche Aussage. Ich seh wirklich keinen Grund, 
warum man einen bereits boolschen Wert nochmal darauf überprüfen soll, 
ob er true ist.

[Falk B. (falk)]

@ Karl Heinz (kbuchegg) (Moderator)

>oder sagst du:
>Wenn es wahr ist, dass die Taste gedrückt ist

>   if( isKeyPressed() == true )

Es ist wahr, dass das recht umständlich ist ;-)

[Oliver S. (oliverso)]

In dem o.a. Fall geht es auch kurz und knapp so:

 test = !test;

Oliver

[ich (Gast)]

Oliver S. schrieb:
> In dem o.a. Fall geht es auch kurz und knapp so:
>  test = !test;
> Oliver

Hi Oliver,

was ist da jetzt anders als die Version von Karlheinz?
Der Variablenname...
Er hat einen aussagekräftigen Namen benutzt und nicht "test", was ja 
alles und nichts aussagen kann.

Karl Heinz schrieb:
> ISR( TIMER1_COMPA_vect)                // Interruptbehandlungsroutine
> {
>   LightOn = !LightOn;
> }