AVR Pointer verhält sich anders als Computer

[ZeigerGehenAufDenZeiger (Gast)]

Hallo Forum.

Manchmal zweifle ich an meinen Fähigkeiten.
Ich will den Vergleich Wert für einen 16 bit Timer während der Laufzeit 
ausrechnen und da muss man ja ein High Register setzen und ein Low 
Register.

Wenn ich mache:

OCR1AH = 0xF4;
OCR1AL = 0x24;

Kommt genau 1 Sekunde raus für den Timer (Prescaler 256 und 16MHz) und 
das klappt auch.

Jetzt zur gesagten Berechnung:

void setCompRegister(int ms) {
    uint16_t compareVal = 16000 * ms / 256;
    uint8_t *ptr = (uint8_t *) &compareVal;
    OCR1AH = ptr[1];
    OCR1AL = ptr[0];
}

mit dem Aufruf:
setCompRegister(1000);

Da müsste wieder genau

OCR1AH = 0xF4;
OCR1AL = 0x24;

rauskommen.

Ich habe leider keinen AVR Debugger, aber wenn ich den Code auf dem 
Rechner laufen lasse funktioniert er!
Nur auf dem ATmega328p kommt was anderes raus (viel kleiner, also höhere 
Frequenz)

Was mach ich nur falsch.

[Einer K. (Gast)]

ZeigerGehenAufDenZeiger schrieb:
> Wenn ich mache:
> OCR1AH = 0xF4;
> OCR1AL = 0x24;

OCR1A = 0xF424;
Kann dein Kompiler nicht?

[leo (Gast)]

ZeigerGehenAufDenZeiger schrieb:
> uint16_t compareVal = 16000 * ms / 256;

Hier hast du einen Overflow.

leo

[ZeigerGehenAufDenZeiger (Gast)]

leo schrieb:
> ZeigerGehenAufDenZeiger schrieb:
>> uint16_t compareVal = 16000 * ms / 256;
>
> Hier hast du einen Overflow.
>
> leo

Tatsache. Danke!

Arduino Fanboy D. schrieb:
> OCR1A = 0xF424;
> Kann dein Kompiler nicht?

Ich wusste nicht, dass das so geht. Ich werde es Mal testen.

[Axel S. (a-za-z0-9)]

ZeigerGehenAufDenZeiger schrieb:
> Arduino Fanboy D. schrieb:
>> OCR1A = 0xF424;
>> Kann dein Kompiler nicht?
>
> Ich wusste nicht, dass das so geht.

Nicht nur, daß das so (auch) geht - es ist sogar die empfohlene 
Variante. Denn beim Zugriff auf 16-Bit IO-Register muß man eine 
bestimmte Reihenfolge einhalten. Der Compiler weiß das und macht es 
richtig. Wenn du auf L- und H-Teil separat zugreifst und nicht darauf 
achtest, machst du es in 50% der Fälle falsch herum.

Außerdem ist dein ganzes Pointer-Gedöhns unsinnig. Wenn du einen 16-Bit 
Wert in H- und L-Teil zerlegen willst, dann kannst du das kurz und 
schmerzlos auf dem arithmetischen Weg machen:

uint16_t x= 0x1234;
// ...
uint8_t xl= x & 0xFF;
uint8_t xh= x / 0x100;

Und keine Sorge, der Compiler rechnet da nichts. Er greift direkt auf 
den L- bzw H-Teil der langen Variable zu. Wir leben im 21. Jahrhundert 
und nicht mehr in der Computersteinzeit.

[ZeigerGehenAufDenZeiger (Gast)]

Danke für die ausführliche Antwort! Man lernt ja nie aus...

Axel S. schrieb:
> uint8_t xl= x & 0xFF;
> uint8_t xh= x / 0x100;

Jetzt hast du aber die Reihenfolge falsch :)

[mech (Gast)]

ZeigerGehenAufDenZeiger schrieb:
> Jetzt hast du aber die Reihenfolge falsch :)

Naja berechnen kannst du die ja in der Reihenfolge die dir am besten 
gefällt. Erst beim Schreiben in Register wird's interessant