AVR timer1 faktor 10 zu schnell

Im Code ist erst mal so kein Fehler erkennbar, ausser der von Nils 
etdeckte Ziffernsturz. Aber der macht keinen Faktor 10 aus.

Auch die Berechnung stimmt.

Ev. ein Messfehler? Wie gehts weiter? In welchem Umfeld benutzt du den 
Code?

Die Gefahr von unterschiedlichen Zahlen bei der Zuweisung an High und 
Low Register kannst du vermeiden, indem du die Aufdröselung den Compiler 
machen lässt.

1

  TCCR1A = 0x00;

2

  TCCR1B = (1<<WGM12) | (1<<CS12)  | (1<<CS10);  // 1024 ??

3

4

  TCNT1 = 0;

5

  OCR1A = 3599;

6

7

  TIMSK = (1<<OCIE1A);

oliver schrieb:

> Mein timer 1 läuft auf einem ATMEGA 32 mit 3,6864MHz im CTC Mode.
>
> Hier die initialisierung:
>
> void timer1_init()
> {
>   TCCR1A = 0x00;

OK.

>   TCCR1B = (1<<WGM12)

Table 47
4 0 1 0 0 CTC OCR1A Immediate MAX

OK.

> | (1<<CS12)  | (1<<CS10);  // 1024 ??

Table 48
1 0 1 clkI/O/1024 (From prescaler)

OK.

>   TCNT1H = 0;
>   TCNT1L = 0;

OK.

>   OCR1AH = ((3599) >> 8);
>   OCR1AL =  (3559);
                 ^
n. OK.

OCR1A Wert um alle 1s einen Timer/Counter1, Output Compare A Match 
Interrupt zu bekommen: 3686400 / 1024 - 1 = 3599

>   TIMSK = (1<<OCIE1A);

OK.

> leider erhalte ich genau den Faktor 10. zu schnell!!

Wie gemessen? Bist du sicher, dass die nicht im Code gezeigte ISR 
angesprungen wird und nicht der AVR sich alle 100ms resettet?

Der kleine Niels schrieb:
> Da fällt mir auf:
> 3,6864MHz klingt nach STK500 Takt... liegt vielleicht dort der Hund
> begraben?

Hab ich mir auch schon gedacht.
Aber spielen wir das mal durch.
Bei einem Faktor von 10, selbst wenn man nicht rechnet, müsste die 
CPU-Frequenz dann entweder 368kHz oder 36Mhz sein.
Das eine ist zu klein, das andere zu gross. 368kHz würde die CPU zwar 
noch mitmachen, aber ob sie mit 36Mhz noch läuft, bezweifle ich mal.

Faktor 10 ist aber auch ein ungewöhnlicher Faktor.

oliver schrieb:
> der Controller läuft definitiv mit externem takt lt fuses.
>
> gemessen, naja, ich habe eine serielle ausgabe, die mir halt jede
> sekunde etwas anzeigt, anstatt alle 10s.

Moment.
Du willst 10 Sekunden?

Zeig doch mal das komplette Programm

Und was denkst du, soll das hier

1

  if(timer_seconds%10 == ((int)(timer_seconds%10)))

machen?
Das ist für jede beliebige Zahl in timer_seconds immer wahr

1

  if( timer_seconds%10 == 0 )

Machs so

1

    if(value_dB >  THRESHOLD_VALUE) 

2

    {

3

      if(!train_begin)

4

      {

5

        train_begin = 1;

6

        sprintf( s, "T00000000000100000000000101dB%08u", fifo_buffer[0]);

7

        uputs( s );

8

      }  

9

    }

10

    else

11

    {

12

      train_begin = 0;

13

    }

dann klappts auch mit dem 'nur einmal ausgeben'.

Das else ist ein deutliches Indiz, dass der von ihm abhängige Teil in 
irgendeiner Form mit dem if davor verknüpft ist und das man diese beiden 
Ausführungspfade in ihrer Gesamtheit sehen muss um zu verstehen was da 
vor sich geht. Lass dir diese 'Dokumentationsmöglichkeit' nicht 
entgehen, sie hat nur Vorteile. Schon alleine der Umstand, dass die 2-te 
Bedingung damit automatisch das Gegenteil von der ersten ist, ohne dass 
du sie hinschreiben musst, ist den Minderaufwand schon wert.

Wenn du die Variable dann auch noch 'sendOverflowMessage' oder so 
ähnlich nennst, dann kann man das auch lesen. Sonst fragt sich jeder, 
wie da jetzt ein Zug ins Spiel kommt und ob da wohl der ICE damit 
gemeint ist.

1

#ifndef TRUE

2

#define FALSE 0

3

#define TRUE  1

4

#endif

5

6

...

7

8

9

    if( value_dB >  THRESHOLD_VALUE ) 

10

    {

11

      if( sendOverflowMessage )

12

      {

13

        sendOverflowMessage = FALSE;

14

        sprintf( s, "T00000000000100000000000101dB%08u", fifo_buffer[0] );

15

        uputs( s );

16

      }  

17

    }

18

    else

19

    {

20

      sendOverflowMessage = TRUE;

21

    }