externen timer auslesen (74hc193) und differenz berrechnen

[TomSawer]

Hallo!

Ich habe einen 4bit up/dwn timer an einen Port des Atmega angeschlossen.
Der Timer wird mit 1Khz ausgelesen. Als return möchte ich die differenz als int8_t
von -8 bis +8.
Zudem soll der Code, aufgrund der hohen Frequenz, effizient sein!

So funktionierts schonmal... aber da kann man sicher mächtig optimieren!
Jemand nen Tipp??? Viele Dank!

ISR, 1Khz {
static int old;                // alter Zählerstand
int new, dir, diff;

new = PINC & 0x0F;        // neuer Zählerstand, 4 bit ausmaskiert

if (new > old) {
   diff = new - old;
   dir = 1;
}
else {
   diff = old - new;
   dir = -1;
}

if (diff > 8) {
   diff = (16 - diff);
   dir *= -1;
}

diff *= dir;

old = new;
return diff;
}

[sternst]

Eines vorweg:

ISR, 1Khz {
...
return diff;
}

Hä???


Nun zum Optimieren:

static int old;                // alter Zählerstand
int new, dir, diff;

Warum sind das alles 16-Bit-Variablen, wo du doch nur mit 4 Bit rechnest und du im Text sogar explizit die 8-Bit-Variante int8_t erwähnst?

if (new > old) {
   diff = new - old;
   dir = 1;
}
else {
   diff = old - new;
   dir = -1;
}

if (diff > 8) {
   diff = (16 - diff);
   dir *= -1;
}

diff *= dir;

Das würde ich so schreiben:

diff = new - old;
if (diff > 8)
    diff -= 16;
if (diff < -8)
    diff += 16;

[TomSawer]

oooopsieeee... 2 mal geposted!

[TomSawer]

Super Danke Stefan!

Hatte nur in meinem Beitrag int geschrieben. Weiß nicht warum.
in meinem Program war natürlich alles int8_t!

Macht man solche "Kürzungen" auch mit boolscher algebra oder mit scharfem Verstand und Erfahrung?

Schöne Woche! Tom