Interupt mit Timer

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

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.

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

Mit der Frequenz hast du recht

Da ist eine null zu viel :)


Wie kann man das denn ausrechnen das der nen Tick bei einer Sekunde 
macht??
Also welche formel gibt es dafür sowas allgemein auszurechnen?

Ist die Formel Zufällig:

Qaruzfrequenz / Prescaler (256) = TOP (Pro Sekunde???)

Wäre dann der TOP für einen 8 MHz Quarz bei eine Prescaler von 256
31250? also 7A12 ??

Damit jede sekunde ein tick ist?

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.

ok :)

Finde es als Hex übersichtlicher irgendwie :)

ok also ist die Formel, damit jede Sekunde in Tick ist folgende:

Qarzfrequenz / Prescaler = Top

Und das ganze noch -1


oder?

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

ah ok :)

danke schön

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.

Wie isn nun die richtige formel?????

So???

OCR1A=(8MHz/(256*1Hz))-1=31249

> 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.
Yo, Danke/Pardon

>So???
>OCR1A=(8MHz/(256*1Hz))-1=31249
Jaaa :-)

Die uhr braucht jetzt ewig bis sie anläuft und hängt zwischendrin auch?

Warum das denn?

Und sie läuft viel zu schnell

1 Minute von der BinärUhr entsprechen 47 Sekunden Real.

Wie kann ich das noch optimieren?

Sehe ich das richtig, in der ISR läuft eine for-ever-Schleife?:
>ISR(TIMER1_COMPA_vect) //16 Bit Timer
>{
>  [...]
>  //Uhr hochz�hlen lassen
>
>  for (;;)
>   {
>    _delay_ms(50);

Ein "break" konnte ich nicht finden...
Das dürfte einen Stacküberlauf wg. verschachtelter Interrupts ergeben.

Nochnmal auf deutsch bitte ??? Soll ich die forever schleife löschen?

Sebastian H. wrote:
> ok :)
>
> Finde es als Hex übersichtlicher irgendwie :)

Ehrlich?
Dann solltest du dich mal fragen, warum du den dezimalen
Wert als Kommentar noch dazuschreiben musst.


Was soll den der Unsinn mit der for Schleife in der ISR?

ah cool :)

das delay muss auch noch weg... is noch vom alten code


Werd ich mal machen und dann nochmal schreiben :)

schreib das nur dazu für Leute die meine Codes laden und eben nicht 
sofort erkennen was das ist :)

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.

was stimmt denn mit den ausgabe befehlen nicht??

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 :-)

Mache aus
  PORTA &= 0x00;
  PORTA |= (s2 << 4) | s1;
einfach
  PORTA = (s2 << 4) | s1;

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...

@ Karl heinz Buchegger:
> PORTA = ( Sekunden % 10 ) << 4 ) | ( Sekunden / 10 );

Ne, umgekehrt. ;-)

jetzt funktioniert gar naix mehr :(

Stefan Ernst wrote:
> @ Karl heinz Buchegger:
>> PORTA = ( Sekunden % 10 ) << 4 ) | ( Sekunden / 10 );
>
> Ne, umgekehrt. ;-)

Ähm, ja.

Matthias Lipinsky wrote:

> So hat der Atmel noch weniger zutun...

Der kommt doch um vor Langeweile! :-)

@ Matthias Lipinsky:

> So hat der Atmel noch weniger zutun...

Sorry, aber das stimmt so nicht ganz. Er hat in beiden Fällen gleich 
viel zu tun, deine Schleife wird nur einfach (meistens) schneller und 
damit häufiger abgearbeitet. ;-)

Help es funktioniert gar nix mehr (quellcode in meinem Letzten post)

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?

Ich würde sehr gerne das ganze simulieren. Leider läuft der Simulator 
unter Vista nicht :(

Naja es passiert einfach gar nichts mehr :)

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 :)

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)

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.

komisch

Danke fürs simulieren :)

Dann sollte es auf meinem Prozzesor eigentlich auch laufen ...

Vielleicht is ja mein 16 Mhz Quarz nicht aktiviert? oder Kaputt?

Hab leider kein Oszi sonst würd ich das direkt übernehmen :)

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

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

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.

ich werde morgen mal die beiden Programme draufbrennen und testen.

Ich werde euch dann die ergebnisse hier posten.

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)

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.

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.

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.

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??

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??

http://www.mikrocontroller.net/articles/AVR_Fuses

ok wird gleich getestet

super das wars danke euch vielmals

Jetzt kann ich mich wieder dem Programm zuwenden.


Wird noch etwas dauer. Ich denke ab Mittwoch kann ich mich wieder melden 
(schreibe morgen eine Schlaufgabe in Deutsch.


Danke derweil ich meld mich wieder.

So Programm läuft und Uhr ist nach 1er Stunde noch sehr genau (hab se im 
moment sehr experimentell gebaut).


Werde se mal bis ende der Woche bauen (wenn die bestelleten LEDs da 
sind)

und mich dann nochmal melden