Interupt mit Timer

Sebastian H. #841464
Lesenswert?
• ▲
▼ 
Guten Tag,

Würde folgender Code mit einem 8 MHz Quarz funktionieren? das ganze 
solle eine Uhr werden:

#include <avr/interrupt.h>

#define fosc 80000000  //8 MHz Quarz zwischen XTAL 1 und XTAL 2

void init_Timer1(void) {

OCR1A = 0xBB7F;       // TOP = 47999

  TIMSK |= (1<<OCIE1A); // CTC-Interrupt zulassen
  TCCR1B = (1<<CS12) | (1<<WGM12); // Mode 4 (CTC; OCR), Timer starten
PSC = 256


}


ISR(TIMER1_COMPA_vect) {

  // Uhrencode
}


int main( void ) {

  init_Timer1();

  sei();

  while(1) {}

  return 0;
}


Grüße
Sebastian H
Antwort Zitieren Bearbeiten Thread verschieben Thread sperren Anmeldepflicht aktivieren Anpinnen Thread löschen Thread mit anderem zusammenführen Beitrag melden
Karl H. #841466
Lesenswert?
• ▲
▼ 
Sebastian H. wrote:
> Guten Tag,
>
> Würde folgender Code mit einem 8 MHz Quarz funktionieren?

Definiere 'funktionieren'

Wenn du damit 'läuft grundsätzlich' meinst:
Sieht nicht schlecht aus. Was sagt denn dein Simulator, buw der
Prozessor dazu?

Wenn du damit 'kriege ich damit eine exakte Uhr' meinst.
Nein, wahrscheinlich nicht.
> #define fosc 80000000  //8 MHz Quarz zwischen XTAL 1 und XTAL 2
Dein Quarz schwingt nicht mit exakt 8000000 Hz.

http://www.mikrocontroller.net/articles/AVR_-_Die_genaue_Sekunde_/_RTC
Antwort Zitieren Bearbeiten Löschen Beitrag melden
Sebastian H. #841467
Lesenswert?
• ▲
▼ 
hm k :)

Also mit funktionieren meine ich ob der das mach was er soll. Nämlich 
jede Sekunde einen Interupt auslösen damit eine Uhr hochzählt tut er da? 
:)

Das sie nicht sonderlich genau ist ist nich so schlimm. Reicht mir schon 
wenn se nach 1ner Stunde nicht 1 Minute falsch geht :)

Simulator läuft leider unter Vista nicht. Deswegen frage ich hier.

Auf Prozessor teste ich es dann noch :) Wollte aber erst wissen ob es 
funktionieren kann.

Grüße
Sebastian H.
Antwort Zitieren Bearbeiten Löschen Beitrag melden
Martin L. #841477
Lesenswert?
• ▲
▼ 
Sebastian H. wrote:
> #define fosc 80000000  //8 MHz Quarz zwischen XTAL 1 und XTAL 2
-> Ist da nicht eine 0 zuviel? (Für den Fall, daß fosc noch irgendwo 
verwendet wird...).

> OCR1A = 0xBB7F;       // TOP = 47999
>   TCCR1B = (1<<CS12) | (1<<WGM12); // Mode 4 (CTC; OCR), Timer starten
> PSC = 256

Prescaler = 256, TOP = 47999 ergibt bei 8MHz einen Tick von 1,535968 
Sek.(oder hab' ich mich verrechnet?)?!

Gruß

Martin
Antwort Zitieren Bearbeiten Löschen Beitrag melden
Karl H. #841495
Lesenswert?
• ▲
▼ 
Sebastian H. wrote:
> Wäre dann der TOP für einen 8 MHz Quarz bei eine Prescaler von 256
> 31250? also 7A12 ??

Fast: 31250 - 1

Frage: Kriegst du eigentlich Sonderpunkte, wenn du den Top Wert
als Hex Zahl angibst? Wenn auf deinem Taschenrechner 8000000 / 256
31250 ergibt, dann schreib das doch auch so hin. Angenehmer
Nebeneffekt: Du brauchst dann auch keinen Kommentar, der dem
Leser die Dezimalzahl dazu angibt.
Antwort Zitieren Bearbeiten Löschen Beitrag melden
Stefan E. #841510
Lesenswert?
• ▲
▼ 
Martin L. wrote:
> Ich hätte erst das Manual lesen und dann schreiben sollen:
>
> Die Tickrate ist lt. ATMEL (Bsp.: ATMega8):
> f_OCR1A=f_CLK/(2*Prescaler*(1+OCR1A))
>
> In meiner Rechnung hatte ich den Faktor 2 und das +1 unterschlagen...
>
> Für 1 Hz also:
> OCR1A=(8MHz/(2*256*1Hz))-1=15624

Nein, die Formel gibt die Frequenz für einen OC-Pin an. Der toggelt bei 
jedem Timer-Event, daher der Faktor 2. Die Interrupt-Rate ist die Formel 
ohne die 2.
Antwort Zitieren Bearbeiten Löschen Beitrag melden
Stefan E. #841553
Lesenswert?
• ▲
▼ 
Ja, war mir fast klar, dass du deinen kompletten alten Code in die 
Interruptroutine verschieben würdest. Da fehlen bei dir ja noch einige 
C-Grundlagen. Versuche z.B. mal den Unterschied zwischen globalen und 
lokalen Variablen rauszufinden (und "static" ist in diesem Zusammenhang 
auch sehr interessant). Im Übrigen ist die Ausgabe unnötig 
verkompliziert.
Antwort Zitieren Bearbeiten Löschen Beitrag melden
Karl H. #841565
Lesenswert?
• ▲
▼ 
Sebastian H. wrote:
> Nochnmal auf deutsch bitte ??? Soll ich die forever schleife löschen?

Die ISR Funktion wird (mehr oder weniger) exakt genau 1 mal jede
Sekunde aufgerufen. Du brauchst da keine Schleifen oder delays
oder sonstigen Müll. Einfach die Zeit hochzählen und fertig.

Die eigentliche Ausgabe bzw. Abfrage der Eingänge verfrachtest
du in die Hauptschleife in main(). Dort bleibt noch genügend
Zeit dafür übrig, die Anzeige auf den neuesten Stand zu bringen

1
volatile unsigned char s1=0;

2
volatile unsigned char s2=0;

3
volatile unsigned char m1=0;

4
volatile unsigned char m2=0;

5
volatile unsigned char h1=0;

6
volatile unsigned char h2=0;

7


8
ISR(TIMER1_COMPA_vect) //16 Bit Timer

9
{

10
  s1 = s1 + 1;

11
  if (s1 > 9)

12
  {

13
    s1 = 0;

14
    s2 = s2 + 1;

15
    if (s2 == 6)

16
    {

17
      s2 = 0;

18
      m1 = m1 + 1;

19


20
      if (m1 > 9)

21
      {

22
        m1 = 0;

23
        m2 = m2 + 1;

24


25
        if (m2 == 6)

26
        {

27
          m2 = 0;

28
          h1 = h1 + 1;

29


30
          if ((h1 > 9) && (h2 < 2))

31
          {

32
            h1 = 0;

33
            h2 = h2 + 1;

34


35
            if ((h1 == 4) && (h2 == 2))

36
            {

37
              h2 = 0;

38
              h1 = 0;

39
            }

40
          }

41
        }

42
      }

43
    }

44
  }

45
}

46


47
int main()

48
{

49
  ....

50


51


52
  while( 1 ) {

53
    // hier die Ausgabe von den globalen Variablen s1, s2, etc

54


55
    // sowie die Behandlung der Tastendrücke

56
  }

57
}


Persönlich finde ich die Unterteilung der Sekunden in Zehenr
und Einer nicht sehr glücklich. Das verkompliziert das Hochzählen
einfach nur kräftig, ohne dass es bei der Ausgabe viel bringt.
Eine 2 stellige Zahl in Zehner und Einer zu unterteilen ist trivial:

Einer  = Zahl % 10
Zehner = Zahl / 10

Die Division kostet zwar Rechenzeit, aber dein µC fadisiert sich
sowieso, mit dieser Aufgabe. Und vernünfitge Variablennamen haben
auch noch nie geschadet.

1
volatile unsigned char Sekunden;

2
volatile unsigned char Minuten;

3
volatile unsigned char Stunden;

4


5
ISR(TIMER1_COMPA_vect) //16 Bit Timer

6
{

7
  Sekunden++;

8


9
  if( Sekunden == 60 ) {

10
    Sekunden = 0;

11
    Minuten++;

12


13
    if( Minuten == 60 ) {

14
      Minuten = 0;

15
      Stunden++;

16


17
      if( Stunden == 24 ) {

18
        Stunden = 0;

19
      }

20
    }

21
  }

22
}

23


24
int main()

25
{

26
   ...

27


28
   while( 1 ) {

29


30
     PORTA = ( Sekunden % 10 ) << 4 ) | ( Sekunden / 10 );

31
     PORTB = ( Minuten % 10 )  << 4 ) | ( Minuten / 10 );

32
     PORTC = ( Stunden % 10 )  << 4 ) | ( Stunden / 10 );

33


34
     ...

35
   }

36
}


Sieht doch gleich viel besser aus :-)
Antwort Zitieren Bearbeiten Löschen Beitrag melden
Martin L. #841568
Lesenswert?
• ▲
▼ 
Stefan Ernst wrote:
> Versuche z.B. mal den Unterschied zwischen globalen und
> lokalen Variablen rauszufinden.

@ Sebastian:
Wenn Du das Ding nicht noch einmal ganz neu stricken, sondern die ISR 
"retten" möchtest: Die Variablen (s1 etc.) müssen dann entweder "static" 
deklariert werden, weil sie sonst am Blockende ihre Gültigkeit 
verlieren, ODER global definiert werden.
Wenn sie static in der ISR sind, kann nirgendwo sonst im Programm darauf 
zugegriffen werden...
Antwort Zitieren Bearbeiten Löschen Beitrag melden
Matthias L. Gast #841584
Lesenswert?
• ▲
▼ 
1
int main()

2
{

3
   unsigned char SekundenAlt  = 255;

4


5
   ...

6


7
   while( 1 ) {

8
     if ( SekundenAlt != Sekunden )

9
     {

10
        SekundenAlt = Sekunden;

11
        PORTA = ( Sekunden % 10 ) << 4 ) | ( Sekunden / 10 );

12
        PORTB = ( Minuten % 10 )  << 4 ) | ( Minuten / 10 );

13
        PORTC = ( Stunden % 10 )  << 4 ) | ( Stunden / 10 );

14
     }

15
     ...

16
   }

17
}


So hat der Atmel noch weniger zutun...
Antwort Zitieren Bearbeiten Löschen Beitrag melden
Karl H. #841602
Lesenswert?
• ▲
▼ 
Dann stell den Prescaler mal kurzzeitig auf 1, schnapp dir den
Simulator im AVR Studio und simuliere mal durch. Einen
Breakpoint in die ISR und dann mittels Einzelschritten durchsteppen.

(Wenn du schon nicht die Anregung zur Vereinfachung des COdes
annehmen willst).

Ausserdem: Was funktioniert denn nicht?
Antwort Zitieren Bearbeiten Löschen Beitrag melden
Gast Gast #841606
Lesenswert?
• ▲
▼ 
@Sebastian
Jetzt haben verschiedene Leute stundenlang dein Problem geloest.
Dein letzter Quellcode zeigt aber, dass Du immer noch Defizite
in den Grundlagen hast.
Vielleicht wuerde das Durcharbeiten des Tutorials mehr bringen.
Antwort Zitieren Bearbeiten Löschen Beitrag melden
Karl H. #841610
Lesenswert?
• ▲
▼ 
Sebastian H. wrote:
> Ich würde sehr gerne das ganze simulieren. Leider läuft der Simulator
> unter Vista nicht :(

Da musst du dir was einfallen lassen.
Das ist so nämlich auf Dauer kein Zustand.
VMWare mit einem 2-ten WinXP aufsetzen.

Mein Vista ist zwar schon lange von der Maschine runter (ca 1 Jahr)
aber das damalige AVR Studio hatte keine Probleme mit der
Simulation (ausser den üblichen Problemen, die aber nichts mit
Vista zu tun hatten)

> Naja es passiert einfach gar nichts mehr :)
Antwort Zitieren Bearbeiten Löschen Beitrag melden
Karl H. #841621
Lesenswert?
• ▲
▼ 
Sebastian H. wrote:
> Okay werd ich mal machen. Hab glaub ich noch irgendwo ne Version von
> VMWare :)
>
> Würde mich trotzdem freuen, wenn ihr mir offensichtliche fehler schicken
> könntet (learning by doing)

Laut Simulator (eingestellt auf Mega16) wird die ISR aufgerufen.
(Prescaler hab ich mal auf 1 gesetzt).
Innerhalb der ISR werden die Variablen auch hochgezählt und die
Ausgabe (zumindest am PortA) wird ebenfalls gemacht.

Ich könnte das ganze jetzt nich in einen realen Mega16 brennen,
bin mir aber sicher, dass sich dort das gleiche Bild zeigen wird.
Antwort Zitieren Bearbeiten Löschen Beitrag melden
Matthias L. Gast #841626
Lesenswert?
• ▲
▼ 
>Vielleicht is ja mein 16 Mhz Quarz nicht aktiviert? oder Kaputt?


Aus diesem GRund habe ich zu Testzwecken anfangs immer sowas drin: Das 
ist mit nem Oszi nachprüfbar.

1
void main ( void )

2
{

3
  ...

4


5
  while (1)

6
  {

7
    // zum Testen

8
    PORTD ^= 0x80;     // irgendein freier Ausgangspin

9
    ...

10


11
  }

12
}
Antwort Zitieren Bearbeiten Löschen Beitrag melden
Sebastian H. #841633
Lesenswert?
• ▲
▼ 
tut er :)

eine lampe leuchtet (nach ca. 1 Minute)

Aber es passiert dann nie wieder was so als ob der Timer ewig lange 
braucht

Ich werde mal den Prescaler auf 1 stellen und dann nochmal auf den 
ATMega brennen.

muss man den die XTAL extra activieren ?? In den fusebits oder so??

Mal schauen was dann passiert
Antwort Zitieren Bearbeiten Löschen Beitrag melden
Karl H. #841634
Lesenswert?
• ▲
▼ 
Sebastian H. wrote:
> Hab leider kein Oszi sonst würd ich das direkt übernehmen :)

Das siehst du doch, ob die LED angeht oder nicht.

1
#include <avr/io.h>

2


3
int main()

4
{

5
  unsigned char i;

6


7
  DDRD = 0xFF;

8


9
  while( 1 ) {

10
    PORTD = 0xAA;

11
  }

12
}


Am PortD muss jede 2.te Led brennen.
Dann mal den Gegentest mit

   PORTD = 0x55;

Jetzt muessen jeweils die anderen LED brennen.

Dann weist du zumindest mal, ob der Prozessor läuft.
Dann noch einen Test zur Geschwindigkeit

1
#define F_CPU 16000000

2


3
#include <avr/io.h>

4
#include <util/delay.h>

5


6
int main()

7
{

8
  unsigned char i;

9


10
  DDRD = 0xFF;

11


12
  while( 1 ) {

13


14
    PORTD = 0x00;

15
    for( i = 0; i < 100; i++ )

16
      _delay_ms( 10 );

17


18
    PORTD = 0xFF;

19
    for( i = 0; i < 100; i++ )

20
      _delay_ms( 10 );

21
  }

22
}


Die LED muessen im Sekundentakt an und aus gehen. Damit wäre
dann auch abgeklärt, ob die 16Mhz auch wirklich da sind.
Antwort Zitieren Bearbeiten Löschen Beitrag melden
Sebastian H. #841640
Lesenswert?
• ▲
▼ 
Ach ja noch was:


ich habe ja in meinem code:
#define fosc 16000000  //16 MHz Quarz zwischen XTAL 1 und XTAL 2
stehen.


Ist da vielleicht der Fehler.
In einem Programm für eine Textdisplay steht für die CPU frequenz 
folgendes:
#define F_CPU 16000000UL     //Prozessor Frequenz in Hz
(das Display Programm funktioniert)
Antwort Zitieren Bearbeiten Löschen Beitrag melden
Karl H. #841642
Lesenswert?
• ▲
▼ 
Sebastian H. wrote:
> Ach ja noch was:
>
>
> ich habe ja in meinem code:
> #define fosc 16000000  //16 MHz Quarz zwischen XTAL 1 und XTAL 2
> stehen.
>
>
> Ist da vielleicht der Fehler.
> In einem Programm für eine Textdisplay steht für die CPU frequenz
> folgendes:
> #define F_CPU 16000000UL     //Prozessor Frequenz in Hz
> (das Display Programm funktioniert)

In deinem Code gibt es nichts, was F_CPU (oder fosc) benutzen
würde. Von daher ist es gehupft wie gesprungen. Dein Program
würde sich nicht verändern, selbst wenn du fosc rausschmeissen
würdest.
Antwort Zitieren Bearbeiten Löschen Beitrag melden
Karl H. #841645
Lesenswert?
• ▲
▼ 
Sebastian H. wrote:
> Achso ok
>
>
> ich finds nur komisch das es bei dir in der Simulation funktioniert hat,
> es aber auf dem Chip nicht funktioniert.
>
>
> Andere Programme laufen ja auch.

Brenn doch mal das 2.te Programm drauf. Damit siehst du nach,
ob die 16Mhz auch wirklich anliegen.
Entweder die Led gehen alle 1 Sekunde an und aus, oder die brauchen
dazu viel länger. Ist ein ganz simpler Test.

Wenn die LED erst nach jeweils 16 Sekunden umschalten, dann läuft
kein Quarz, sondern der interne Taktgeber mit 1Mhz.
Antwort Zitieren Bearbeiten Löschen Beitrag melden
Sebastian H. #841646
Lesenswert?
• ▲
▼ 
jo werd ich wie gesagt morgen machen und dann posten (hab den Programmer 
jetzt schon aufgeräumt :)


Danke das ihr soviel Zeit investiert habt :)


Ich meld mich dann mal morgen wieder, wenn ich das 2. Programm 
draufgebrannt habe mit den ergebnissen :)


Noch so als Frage:
Muss man den Externen Quarz in den Fusebits aktivieren??
Antwort Zitieren Bearbeiten Löschen Beitrag melden
Sebastian H. #842088
Lesenswert?
• ▲
▼ 
Okay ich hab das ganze gerade mal draufgebrannt. Die LED geht alle 16 
Sekunden an und nach weiteren 16 Sekunden wieder aus.

Also stimmt wohl etwas mit dem Quarz nicht oder?

Er ist folgendermaßen angeschlossen:

XTAL 1----..----------||------------------------GND
          ||Quarz    Kondensatoren 22pf
XTAL 2----..----------||------------------------GND



Muss man den Quarz irgendwie in den Fuses aktivieren??
Antwort Zitieren Bearbeiten Löschen Beitrag melden
Thread beobachten

Antwort schreiben

Bitte melde dich an, um einen Beitrag zu schreiben.

Bestehender Account

oder

Mit Google-Account einloggen

Noch kein Account?

Die Registrierung ist kostenlos und dauert nur eine Minute.

Jetzt registrieren