32 Bit Timer Attiny (Zeitstempel)

Pate schrieb:
> cli();
>
>     time=(uint32_t)(TCNT1)+pSystemtimer->Time;
Was wenn "zwischen" diesen beiden Zeilen der Timer überläuft, TCNT1 also 
0 zurückliefert, aber der Timerinterrupt noch nicht ausgeführt wurde, 
also pSystemtimer->Time noch nicht um 0x10000 erhöht wurde? Dann erfährt 
deine Zeit einen Sprung rückwärts...

ATOMIC_BLOCK mach in einer ISR wenig sinn.


> time=(uint32_t)(TCNT1)+pSystemtimer->Time;
wenn TCNT1 überläuft während die Berechnung läuft könnte das zu 
Problemen kommen. Da hilft auch das cli und sei nicht weil er Trotzdem 
weiterzählt.

Achso, dann muss ich das Überlaufbit mit betrachten.

Das Atomic war eher eine Verzweiflungstat...

Pate schrieb:
> Achso, dann muss ich das Überlaufbit mit betrachten.
Das bringt leider auch nix, was wenn der Überlauf kommt zwischen der 
Abfrage des Bits und Abfrage von TCNT1 - egal in welcher Reihenfolge 
dies stattfindet?
Eigentlich müsstest du den Timer temporär anhalten...

Ich habe es mal wie folgt angepasst:

1

uint32_t Systemtimer::SystemtimerGetTime()

2

{

3

  uint32_t time;

4

  cli();

5

  if (TIFR1 & (1<<TOV1))

6

  {

7

    pSystemtimer->Time+=0x10000;

8

    TIFR1=(1<<TOV1);

9

  }

10

  time=(uint32_t)(TCNT1)+pSystemtimer->Time;

11

12

  sei();

13

  return time;

14

}

Und laut Oszi, gibt es keine Ausreißer mehr, nur ein Jitter durch die 
If-Abfrage.

Zu früh gefreut, ich muss wohl doch den Timer anhalten.

Wie könnte ich denn das Problem lösen?

Wie wäre es mit einem zweiten Timer-Register, bei dem du per 
Flag-Register die Aktualisierung stoppst, wenn du Auslesen möchtest? 
Oder lass deinen Code zum Auslesen mit dem IRQ synchronisieren. Da gibt 
es viele Möglichkeiten, Timer anhalten würde ich nie machen, wenn's denn 
genau sein soll.

PS: Mit ARM wäre das nicht passiert (SCNRT) ;)

Das ein Timer soo viel ärger machen kann...

Das mit der Synchronisierung versuche ich derzeit. Ich habe mir in der 
Klasse eine Funktion gebaut, die Reibungslos funktioniert. Nur wenn ich 
mir eine Warte FUnktion in der Main baue, gehts daneben...

1

void Systemtimer::SystemtimerWait(uint32_t Wait_us)

2

{  

3

  uint32_t nexttime=this->SystemtimerGetTime()+Wait_us;

4

5

  while(this->SystemtimerGetTime()<nexttime)

6

  {

7

  }

8

9

}

greg schrieb:
> PS: Mit ARM wäre das nicht passiert (SCNRT) ;)

Wie, ARM baut auch Timer? Du meinst 'systick'?

Die Überlaufe eines Timers sind doch kein Problem, man muß sie nur 
beachten!

In der Routine "ISR(TIM1_CAPT_vect)" kann man sehen, wie es geht:
http://www.mino-elektronik.de/fmeter/fm_software.htm#bsp11

Georg G. schrieb:
> Wie wäre es mit "zweimal ablesen". Dann kannst du per Plausibilität
> entscheiden, ob und bei welcher Ablesung dein Timer übergelaufen ist.

Auch das kann man machen, wenn es nicht zeitkritisch ist.

Und wenn du den Timer stoppst und anschließend zu TCNT1 x addierst, 
wobei x die Anzahl der Takte ist, die es von Stoppen bis nach der 
Berechnung braucht (x könnte man aus dem Assemblerlisting berechnen)?

WIe soll ich denn das Capture Event auslösen?

So jetzt gehts stabil, habe nochmal die Timer32.c übernommen und auf 
meine Sourcen angepasst.

1

uint32_t Systemtimer::SystemtimerGetTime()

2

{

3

4

    uint32_t val;

5

    uint8_t tifr;

6

7

    cli();

8

    val = pSystemtimer->Time + TCNT1;

9

    tifr = TIFR1;          // read interrupt flags

10

    sei();

11

    if( (tifr & 1<<TOV1) && !(val & 0x8000) )

12

    val += 32768;         

13

14

}

Thread kann geschlossen werden :)

Die Timer32, produziert den gleichen Fehler... Schon wieder zu früh 
gefreut...


uint32_t Systemtimer::SystemtimerGetTime()
{

    uint32_t val;
    uint8_t tifr;

    cli();
    val = pSystemtimer->Time + TCNT1;
    tifr = TIFR1;          // read interrupt flags
    sei();
    if( (tifr & 1<<TOV1) && !(val & 0x8000) )
    val += 32768;
  return val;

}

Egal was ist da Eintrage, jedesmal kommt der Fehler...

1

uint32_t Systemtimer::SystemtimerGetTime()

2

{

3

4

    uint32_t val;

5

    uint8_t tifr;

6

7

    cli();

8

    val = pSystemtimer->Time + TCNT1;

9

    tifr = TIFR1;

10

    sei();

11

12

    if( (tifr & 1<<TOV1) && !(val & 0x80) )

13

    val += 256;    

14

          bzw.

15

          if( (tifr & 1<<TOV1) && !(val & 0x8000) )

16

    val += 32768;    

17

          bzw.

18

          if( (tifr & 1<<TOV1) && !(val & 0x80) )

19

    val += 32768;   

20

21

  return val;

22

}

Pate schrieb:
> Egal was ist da Eintrage, jedesmal kommt der Fehler...
> uint32_t Systemtimer::SystemtimerGetTime()
> {
>
>     uint32_t val;
>     uint8_t tifr;
>
>     cli();
>     val = pSystemtimer->Time + TCNT1;
>     tifr = TIFR1;
>     sei();
>
>     if( (tifr & 1<<TOV1) && !(val & 0x80) )
>     val += 256;
>           bzw.
>           if( (tifr & 1<<TOV1) && !(val & 0x8000) )
>     val += 32768;
>           bzw.
>           if( (tifr & 1<<TOV1) && !(val & 0x80) )
>     val += 32768;
>
>   return val;
> }

Habe es jetzt mit dem doppelten getestet:

1

uint32_t Systemtimer::SystemtimerGetTime()

2

{

3

4

    uint32_t val;

5

    uint8_t tifr;

6

7

    cli();

8

    val = pSystemtimer->Time + TCNT1;

9

    tifr = TIFR1;

10

    sei();

11

    if( (tifr & 1<<TOV1) && !(val & 0x8000) )

12

    val += 65536;       

13

  return val;

14

}

Jetzt taucht kein Fehler mehr auf. 15000 Passes keine Fails :)