Timer atmega 128

[zrmba]

ehm ich studiere elektrotechnik, und ja ich habe auch keine großartigen programmierkenntnisse, wir müssen aber auch nur sehr wenig programmieren.. :-D

- - - Aktualisiert - - -

der neue quellcode sieht dann bei mir so aus:

[Jimmybot]

Bitte poste dein nächsten Code hier im Forum "lesbar"... also beim antworten den Button erweitern (unten rechts) klicken und dann im Menü "Code einfügen" benutzen und deinen Code reinkopieren. Dann kann man auch schneller lesen, ohne erst was runterladen zu müssen.

Zu dein Code:

Ich sortiere ihn mal

#include <avr/io.h>   // AVR headerfile, in dem die Ansteuerung der I/Os definiert wird
#define F_CPU 16000000UL // Definiert die CPU-Frequenz auf 16MHz (entspricht Frequenz des verwendeten Quarzes)
#include <util/delay.h>  // AVR Headerfile, das die Delay-Funktion definiert
#include <avr/interrupt.h>  // Interrupt Bibliothek

#define preload_value_1  62500    // 1. Vorladewert
#define preload_value_2  562500   // 2. Vorladewert

void init_timer_1 (void)
{
  // hier den Timer1 initialisieren
}

ISR (TIMER1_OVF_vect)
{
  // Hier dein Code um zwischen 1 ms und 9 ms zu unterscheiden   
}

int main (void)
{
  // Portrichtung konfigurieren
  DDRB |=(1<<PB2);           // Output 1
  DDRB |=(1<<PB3);           // Output 2

  // Funktion aufrufen um den Timer1 zu initialisieren
  init_Timer_1();

  // Interrups global freigeben
  sei();

  // Arbeitsschleife
  while(1)
  {
  }
}

Den Inhalt von deiner ISR und init-Funktion habe ich mal weg gelassen.

Aber mal eine andere Frage... du sollst die ungeraden LEDs 1ms und die geraden 9ms leuchten lassen...


Alle wieviel ms soll sich der Vorgang wiederholen?


Wie wills du das Ergebnis überprüfen? Denn 1ms und 9 ms sieht man nicht bzw. kaum. Hast du ein Oszilloskop um deine Arbeit zu prüfen?

[Ceos]

okay grundlegendes(sorry wenn ich gerade nciht feinfühliger formulieren kann):

deine ISR() hat nichts in der main() zu suchen und das mit der enfdlosschleife war jetzt ncith ganz verkehrt aber dir fehlt definitiv ein wenig grundlagenwissen

dein controller arbeitet den code der main schleife ab, es sei denn ein interrupt tritt auf der abgearbeitet werden kann/muss

wenn der controller das ende der main erreicht ist er in einem sog, undefiniertem zustand, sowas sollte grundsätzlich vermieden werden ist aber bei neuern controllern nicht mehr wirklich relevant da die compiler schon sorge dafür tragen dass der controller in einem definierten sicheren zustand bleibt

also merke: niemals die main verlassen, immer eine while(TRUE) endlosschleife in der main() in der sich wiederholender code abgearbeitet wird

du möchtest LEDs in festen zeitintervallen laufen lassen, das geht mit ISR auf 2 arten:

Version 1: dein timer läuft in einem festen zeitintervall über (1mS) und jedesmal im OVF interrupt zählst du eine variable timer hoch (timer++ in der ISR, mehr nicht), in der main() schleife fragst du diese timer variable ab und reagierst entsprechend und steuerst deine LEDS (z.B. wenn (timer < 9) dann LED1 an, wenn (timer < 18 ) LED2 an und timer = 0 zum zurücksetzen des timers)

Version 2: die elegantere lösung ist es die LED steuerung gleich innerhalb der ISR zu machen, also in der ISR dann so z.B.: wenn (timer < 9) dann LED1 an und timer++, wenn (timer < 18 ) LED2 an und timer = 0 zum zurücksetzen des timers

Version 2 ist eleganter denn die while(true) in der main() kann leer bleiben und andere aufgaben übernehmen, aber bedenke dass eine ISR immer dazuführt dass der code der main an einer quasi beliebigen stelle unterbrochen werden kann und so vielleicht auch mal dein if in der main eine variable prüft, die im nachfolgenden prozessortakt schon wieder geändert wird udn der inhalt deiner if bedingugn eventuell ungültig wird!

also merke ISRs sind elegant um aufgaben die von externen triggern (zeit ist in dem kontext der trigger) abhängig sind zu verarbeiten, sollten aber immer so geschrieben werden dass der code in der main davon während der ausführung nicht behindert werden kann

solltest du in der main mal trotzdem ein KURZES stück code ausführen dass mit der variablen der ISR interagieren muss kannst du kurz vorher ein cli(); und danach ein sei machen also z.B:
cli();
<deine bedingung und der code der abhängig ist von der variable>
sei();

um alle interrupts kurzzeitig abzuschalten, aber dann kann es passieren dass deine timings nciht eganz exakt passen weil die ISR eventuell ein kleines stück verzögert werden dadurch


ich weis das war jetzt viel theorie auf einmal aber du brauchst mal ein wenig verständnis, weil dein code im moment eher danach aussieht als ob du rumrätselst was wohin gehört

----------------------------
EDIT: blöde auto smilies XD

[zrmba]

@ jimmybot : ja richtig, ich soll die zeiten mit einem oszilloskop überprüfen.
ja die ungeraden LED´s sollen 1ms leuchten und die geraden 9ms und das soll sich sofort abwechseln.

- - - Aktualisiert - - -

danke auch für die korrektur da habe ich ein paar sachen falsch gemacht stimmt.

- - - Aktualisiert - - -

auch danke an ceos
ja das stimmt wohl, dass ich eher rumrätsel, ich habe mich, wie man sieht, auch noch nicht großartig mit der programmierung von microcontrollern beschäftigt, aber die neue if anweisungen für die beiden led ansteuerungen kann ich so gut nachvollziehen, danke.

[oberallgeier]

.. dass ich eher rumrätsel, ich habe mich, wie man sieht, auch noch nicht großartig mit der programmierung von microcontrollern beschäftigt ..

Nur für mein Verständnis: was missfällt Dir denn bei meinem Codebeispiel von oben? Nur damit ich mich nicht wieder einmische, wenns eh unnötig ist.