variablen deklaration

kirby Gast #76757
Lesenswert?
• ▲
▼ 
hallo leute

wäre nett wenn jemand rat weiss. nun zum problem:
//Globals
static volatile uint16_t Sec;
.
.
.

SIGNAL(SIG_OVERFLOW1)            // handler for tcnt1 overflow
interrupt
{
    PORTB ^= (_BV(PINB0) | _BV(PINB1)  ); //toggle bits
  Sec++;
  if(Sec == 60)
  {
    Sec = 0;
    Min++;
  }
  if(Cls == 1)
    Dauer++;
  if(Sec == 5)
    PORTD|= _BV(PIND6);
  if(Sec == 10)
    PORTD&= _BV(PIND6); //clear bit PIND6


    TCNT1 = TIMER_1_CNT;         // reset counter to get this interrupt
again
}

folgendes phaenomen ereignet sich. wenn 5 sec vorbei sind schaltet er
brav  die led an, wenn 10 sec vorbei sind gehen auf einmal  von PORTB
zwei leds an die auf PINB2 +3 liegen. das ganze läuft auf einem atmel
AT90S2313


hier noch die timer init:
    TCCR1A = 0x00;               // disable PWM and Compare Output
Mode
    TCCR1B = TMC16_CK1024;       // use CLK/1024 prescale value
    TCNT1  = TIMER_1_CNT         // reset TCNT1 high/low byte
    TIMSK  = _BV(TOIE1);         // enable TCNT1 overflow,bit 8 set on
1

und dann natürlich sei().

wäre nett wenn jemand rat wüsste

thx


kirby
Antwort Zitieren Bearbeiten Thread verschieben Thread sperren Anmeldepflicht aktivieren Anpinnen Thread löschen Thread mit anderem zusammenführen Beitrag melden
kirby Gast #76758
Lesenswert?
• ▲
▼ 
hi ,

ich habe das problem erkannt
es lag nicht an der variablen deklaration sondern am clearen von dem
bit, ich hatte vergessen das bit zu invertieren bevor ich es  verunde.
oh mannnnnnn

also &=~
anstatt &=

trotzdem thx


kirby
Antwort Zitieren Bearbeiten Löschen Beitrag melden
Jörg Wunsch Gast #76759
Lesenswert?
• ▲
▼ 
Übrigens: `volatile' Variablen werden wirklich in keiner Weise
optimiert.

Wenn Du `Sec' nur innerhalb des Interrupthandlers brauchst,
deklariere
die Variable besser nicht volatile, sondern static innerhalb dieser
Funktion.

Wenn Du sie auch extern brauchst, ist sowas besser:

static volatile uint16_t Sec;
...
SIGNAL(...)
{
  uint16_t localsec = Sec;

  if (++localsec == 60)
    ...

  ...
  Sec = localsec;
}

Auf diese Weise wird der Zugriff auf Sec, dessen Optimierung dem
Compiler untersagt ist, nur zweimal gemacht, alles andere darf der
Compiler lokal in Registern optimieren.
Antwort Zitieren Bearbeiten Löschen Beitrag melden
Stefan Kleinwort Gast #76760
Lesenswert?
• ▲
▼ 
Anmerkung zu Jörgs Programm:

... und auch wenn der C-Code 2 Zeilen länger ist: der resultierende
Code wird deswegen höchstens kürzer, aber nie länger werden, das diese
Aktionen der Compiler in jedem Fall machen muss.

Stefan
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