counter Problem (SRF05 Ultraschallsensor C code)

[Besserwessi]

Die Schreibweise mit TCCR1B = (1<<CS11); ist besser, weil man da das ganze Register definiert, also alles richtig einstellt, egal was vorher drin war. Die andere Version wird meistens auch gehen, weil der normale Mode default ist. die Version mit dem |= wird ggf. kürzer weil dafür spezielle ASM Befehle existieren.

So ähnlich ist es mit dem Stoppen des Timers - das kommt hier ohnehin nach dem Auslesen, ist also nicht von Bedeutung. Man könnte es ggf. Probieren den Timer erst zu stoppen, und dann auszulesen. Im Prinzip sollte es aber auch bei laufendem Timer gehen.
An sich macht GCC das auslesen von 16 Bit Registern richtig, man könnte es ggf. auch mal von Hand probieren (erst low Byte lesen), oder den erzeugten Code ansehen.

An sich kann ich da keinen Fehler sehen. Ein versuch wäre ggf. noch den Timer erst auf 0 zu setzen und dann zu starten. Im Prinzip sollte aber beides gehen.

Eine kleine Ungenauigkeit ist da noch drin bei der Ausgabe. Statt itoa sollte man utoa nehmen, denn die variable ist uint16_t. Bis gut 32000 sollte das aber auch noch keinen Unterschied machen.

Die 58 µs/cm kommen übrigens ganz gut hin mit der Laufzeit. Das ist also schon die Zeit bis zu Echo.

[MagicWSmoke]

die Version mit dem |= wird ggf. kürzer weil dafür spezielle ASM Befehle existieren.

Die Timerregister sind meist oberhalb des IO-Adressbereiches, den SBI & CBI erreichen können, diese Opcodes gehen nur bis 0x1F.
Beim ATM8 sitzt TCCR1B auf 0x2E, damit ist die Sequenz: IN, ORI, OUT, bei der direkten Zuweisung dagegen nur: LDI, OUT, ein Takt weniger.
Ports liegen oft im Bereich unter 0x1F, da ist Bit-weises Setzen und Löschen dann tatsächlich schneller.

Die 58 µs/cm kommen übrigens ganz gut hin mit der Laufzeit. Das ist also schon die Zeit bis zu Echo.

Ja, richtig. Ist ja der Weg hin und zurück.