Encoder nach Peter

[Henry (Gast)]

Hallo
In verschiedenen Projekten verwende ich Encoder (Drehgeber) mit RGB 
LEDs. Damit kann ich dann mit dem drehen verschiedene Farben anzeigen 
lassen.
Möchte jetz mal was ganz einfaches machen und das klappt nicht, miss...
Verwende den Code von Peter:

#define F_CPU 16000000UL        // Angabe der Frequenz, wichtig für die Zeit
#include "avr/io.h"            // Einbindung Datei Ausgänge
#include "avr/interrupt.h"        // Einbindung der interrupt.h

#define PHASE_A     (PINB & 1<<PB2)    // Eingang Encoder A  P172
#define PHASE_B     (PINA & 1<<PA0)    // Eingang Encoder B  P172  

#define LEDS_DDR    DDRA        // Ausgang LED
#define LEDS        PORTA        // 

volatile int8_t enc_delta;        // -128 ... 127
static int8_t last;
  
  void encode_init( void )
    {
    int8_t new;
    new = 0;
    if( PHASE_A ) new = 3;
    if( PHASE_B ) new ^= 1;      // Convertiert Gray Code in Binär Code
    last = new;            // power on state
    enc_delta = 0;
    }
  
  void timer_init()            // Timer2 1ms 16MHz
    {
    TCCR2A = (1<<WGM01);
    TCCR2B = (1<<CS01)|(1<<CS00);
    OCR2A = 249;
    TIMSK2 |=(1<<OCIE2A);
    }

  ISR( TIMER2_COMPA_vect )        // 1ms for manual movement
    {
    int8_t new, diff;
    new = 0;
    if( PHASE_A ) new = 3;
    if( PHASE_B ) new ^= 1;      // Konvertiert Gray Code in Binär Code
    diff = last - new;        // difference last - new
    if( diff & 1 ) 
      {                // bit 0 = value (1)
      last = new;          // store new as next last
      enc_delta += (diff & 2) - 1;  // bit 1 = direction (+/-)
      }
    }

  int8_t encode_read1( void )      // lesen einzel Schritt Encoder = schnell
    {
    int8_t val;
    cli();
    val = enc_delta;
    enc_delta = 0;
    sei();
    return val;            // counts since last call
    }

  int8_t encode_read2( void )      // lesen zwei Schritt Encoder = mittel
    {
    int8_t val;
    cli();
    val = enc_delta;
    enc_delta = val & 1;
    sei();
    return val >> 1;
    }

  int8_t encode_read4( void )      // lesen vier Schritt Encoder = langsam
    {
    int8_t val;
    cli();
    val = enc_delta;
    enc_delta = val & 3;
    sei();
    return val >> 2;
    }

  int main( void )            // Hauptschleife
    {
    int32_t val = 0;
    timer_init();
    DDRA |= 0b00100000;        // LED 5 auf Ausgang
    PORTA = 0b00100000;        // LED auf aus 
    encode_init();
  
    sei();
    for(;;)
      {
      val += encode_read2();    // lesen 1,2 oder 4 Schritt Encoder
      LEDS = val;
      }
    }

Die LED schaltet beim betätigen des Encoder leider etwas eigenartig. 
Muss ca. ein komplette Drehung machen wenn es schalten soll, noch eine 
Umdrehung geht sie wieder aus. Wollte igentlich das sie, egal ob links 
oder rechts drehe, kurz leuchtet und wieder aus geht. Es soll mir 
anzeigen ob ich drehe.
Jemand eine Idee dazu?

[W.S. (Gast)]

Henry schrieb:
> Jemand eine Idee dazu?

Ja. Und das ist immer wieder die gleiche: nicht einfach nur copy&paste, 
sondern verstehen.

Oder glaubst du, daß ein Forum ist wie ein Cola-Automat? Oben ne Frage 
reinstecken und unten kommt die fertige Lösung heraus?

W.S.

[Teo D. (teoderix)]

Du zählst den PortA rauf und runter.... Wie ist den DIE LED 
angeschlossen o. sinds mehrer oder oder......? Zeig mal was Du das hat!

[mIstA (Gast)]

Henry schrieb:
> Wollte igentlich das sie, egal ob links
> oder rechts drehe, kurz leuchtet und wieder aus geht.

Und welcher Teil Deines Programms sollte das - Deiner Meinung nach - wie 
bewerkstelligen?

[Hugo H. (hugo_hu)]

Hallo Henry aka Helmut aka Achim,

Deine Erfahrungen

Henry schrieb:
> In verschiedenen Projekten verwende ich Encoder (Drehgeber) mit RGB
> LEDs. Damit kann ich dann mit dem drehen verschiedene Farben anzeigen
> lassen.

kann man "Deinem" Programm nicht entnehmen.

Du hättest den Programm-Kopf auch stehen lassen können, da das Entfernen 
dieses Teils wohl Deine einzige Eigenleistung war:

/************************************************************************/
/*                                                                      */
/*                      Reading rotary encoder                     */
/*                      one, two and four step encoders supported  */
/*                                                                      */
/*              Author: Peter Dannegger                                 */
/*                                                                      */
/************************************************************************/

Beitrag "Drehgeber/Encoder 1-, 2- oder 4-schrittig"

[Al. K. (alterknacker)]

Dann ist es doch alles o.k.
Betreff:   "Encoder nach Peter"

[Egonwalter M. (heiner1234)]

Hugo H. schrieb:
> Hallo Henry aka Helmut aka Achim,
>
> Deine Erfahrungen
>
> Henry schrieb:
>> In verschiedenen Projekten verwende ich Encoder (Drehgeber) mit RGB
>> LEDs. Damit kann ich dann mit dem drehen verschiedene Farben anzeigen
>> lassen.
>
> kann man "Deinem" Programm nicht entnehmen.

Nun, es ist doch bekannt, dass Henry aka Helmut aka Achim immer wieder 
per  copy & paste Programmteile zusammenkopiert, diese Schnipsel aber 
nicht versteht und dann hier um Hilfe bittet. Ok soweit, es wird immer 
irgendwie irgendjemand helfen.
Wenn er aber dann mithilfe Anderer das Programm zu Laufen gebracht und 
dann ein "Tut" erstellt und dieses ins Netz stellt OHNE Angabe von 
Originalquellen - das ist frech.

[Henry (Gast)]

Kann mich nicht erinnern das Peter die Nutzung seiner Programme 
untersagt hat. Im Gegenteil. Ein Encoder mit RGB ist in dieser Frage 
zwar erwähnt aber nicht nach gefragt, habe es nur erwähnt das es geht.
Die LED ist am Pin PA5 angeschlossen.
Hoffe das ihr nicht von mir erwartet, das ich ein Programm schreibe was 
besser ist als das von Peter.
Wer das denkt leidet wohl unter Grössenwahn.

[Henry (Gast)]

Steht sogar im Titel von wem es ist

[Stefan F. (Gast)]

Egonwalter M. schrieb:
> Wenn er aber dann mithilfe Anderer das Programm zu Laufen gebracht und
> dann ein "Tut" erstellt und dieses ins Netz stellt OHNE Angabe von
> Originalquellen - das ist frech.

Henry schrieb:
> Kann mich nicht erinnern das Peter die Nutzung seiner Programme
> untersagt hat.

Derartige Kritik habe ich schon ein paar mal zu Recht einstecken müssen. 
Was verboten ist, und was sich nicht gehört, sind zweierlei.

Solange du dich noch so schwer tust, fremde Codes zu benutzen, 
beschränke dich auf die C Bibliothek und schreibe alles selber, was du 
kannst. Aber bilde dir nicht zu früh ein, dass deine mühsam erarbeiteten 
Codes für Tutorials gut genug sind. Das ist ein häufiger Anfängerfehler, 
den auch ich einige male gemacht habe.

Wenn du etwas funktionierendes zusammen geklöppelt hast, kannst du es 
hier immer noch zu Kontrolle Posten. Ziehe dir dazu ein dickes Fell an, 
schlafe vor jeder Antwort eine Nacht, und verzichte in deinem Tutorial 
nicht auf Quellangaben. Dann wird das schon.

Wir sind alte Meckersäcke, aber das heißt nicht, dass wir nicht gerne 
helfen.

[Heiner (Gast)]

Henry schrieb:
> Kann mich nicht erinnern das Peter die Nutzung seiner Programme
> untersagt hat.

Hat auch niemand angezweifelt, Henry

> Im Gegenteil. Ein Encoder mit RGB ist in dieser Frage
> zwar erwähnt aber nicht nach gefragt, habe es nur erwähnt das es geht.

Was ist dann Dein Problem?

> Die LED ist am Pin PA5 angeschlossen.
> Hoffe das ihr nicht von mir erwartet, das ich ein Programm schreibe was
> besser ist als das von Peter.

Auf keinen Fall nach Deinen bisherigen Leistungen 😉
> Wer das denkt leidet wohl unter Grössenwahn.

Keiner denkt das!

[Uwe K. (ukhl)]

Henry schrieb:
> Die LED ist am Pin PA5 angeschlossen.

Versuche doch mal PA2...

[Stefan F. (Gast)]

Val ist eine 32 Bit Variable, die gibst du auf einem 8 Bit Port aus, von 
dem du wiederum nur ein Bit mitten drin verwendest. Kommt dir das nicht 
seltsam vor?

Da es das Bit 5 ist, ändert es nur alle 32 Impulse seinen Zustand.

[Hugo H. (hugo_hu)]

Henry schrieb:
> Möchte jetz mal was ganz einfaches machen und das klappt nicht, miss...
> Verwende den Code von Peter:

Schau doch mal im Tutorial nach:

https://playground.boxtec.ch/lib/exe/fetch.php/tutorials/encoder.pdf

[Heiner (Gast)]

😂👍

[Henry (Gast)]

Uwe K. schrieb:
> Versuche doch mal PA2...

Wieso PA2?

Hugo H. schrieb:
> Schau doch mal im Tutorial nach:

Das mit RGB im Encoder klappt hier.

Stefan ⛄ F. schrieb:
> Val ist eine 32 Bit Variable, die gibst du auf einem 8 Bit Port aus, von
> dem du wiederum nur ein Bit mitten drin verwendest. Kommt dir das nicht
> seltsam vor?
>
> Da es das Bit 5 ist, ändert es nur alle 32 Impulse seinen Zustand.

Das hatte ich auch bereits geändert und verwende jetzt val

Stefan ⛄ F. schrieb:
> Wenn du etwas funktionierendes zusammen geklöppelt hast, kannst du es
> hier immer noch zu Kontrolle Posten.

Das klingt gut und würde ich sehr gern machen. Doch leider geht es 
einigen hier darum anderen zu zeigen das sie Nieten und schlimmer sind.

Leider haben einige Menschen in den Sozialen Netzwerken mehr damit zu 
tun andere zu beleidigen und mit dem Tot zu bedrohen. Die Auswirkungen 
hat man ja bereits gesehen.

Stefan ⛄ F. schrieb:
> Wir sind alte Meckersäcke, aber das heißt nicht, dass wir nicht gerne
> helfen.

Leider habe ich den Eindruck das das meckern vorgeht. Einige meckern 
lieber als zu helfen

[Hugo H. (hugo_hu)]

Henry schrieb:
> Das mit RGB im Encoder klappt hier.

Da vermutlich auch - ganz nah ...

https://www.mikrocontroller.net/articles/Modulares_Board#Encoder_/_Drehgeber

Henry schrieb:
> Doch leider geht es
> einigen hier darum anderen zu zeigen das sie Nieten und schlimmer sind.

Nein - nur auf Deine Masche hinweisen, Dir alles von Anderen "vorbeten 
zu lassen" um es danach in ein "Tuturial von HJS" zu packen. Wenn ich 
mich richtig erinnere machst Du das seit ca. 10 Jahren so.

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

Henry schrieb:
> Muss ca. ein komplette Drehung machen
Um deine eigene Längsachse? So eine Art Pirouette?
Warum schreibst du nicht einfach, WAS du da drehst und WIE du das an den 
µC angeschlossen hast.

> Möchte jetz mal was ganz einfaches machen und das klappt nicht
Hilft alles nix. Dann musst du noch was einfacheres machen und Schritt 
für Schritt vorwärts gehen.

> Die LED schaltet beim betätigen des Encoder leider etwas eigenartig.
Du selber hast das doch programmiert. Was hast du dir dabei gedacht, als 
du die Zeilen hingeschrieben hast? Welche Reaktion der LED hast du bei 
diesem, deinem Programm erwartet?

> Es soll mir anzeigen ob ich drehe. Jemand eine Idee dazu?
Ja, du brauchst dafür ein anderes Programm (und lustigerweise brauchst 
du für die Lösung dieser Aufgabe nicht mal eine Encoderauswertung). Du 
musst nur erkennen, ob sich an den Pins was ändert und dann jeweils 
einen Timer hochsetzen. Dieser Timer wird ständig herunter gezählt, und 
so lange er ungleich 0 ist, wird die LED angeschaltet.

Ich würde das etwa so machen:

volatile int8_t enc_delta;        // -128 ... 127
static int8_t last;
volatile int8_t timeout_led = 0;
:
:
  ISR( TIMER2_COMPA_vect )        // 1ms for manual movement
  {
    int8_t new, diff;
    new = 0;
    if( PHASE_A ) new = 3;
    if( PHASE_B ) new ^= 1;          // Konvertiert Gray Code in Binär Code
    diff = last - new;               // difference last - new
    if( diff & 1 ) 
    {                                // bit 0 = value (1)
        last = new;                   // store new as next last
        enc_delta += (diff & 2) - 1;  // bit 1 = direction (+/-)

        timeout_led=255;              // Zeitzähler hochsetzen
    }
    if (timeout_led>0) timeout_led--; // Zeit runterzählen
  }
:
:
    for(;;)
    {
      // Zeit abfragen und LED ansteuern
      if (timeout_led>0) LEDS |=   (1<<5); // ein
      else               LEDS &=  ~(1<<5); // aus

      val += encode_read2();    // lesen 1,2 oder 4 Schritt Encoder
    }
:

Ergebnis: die LED leuchtet so lange du drehst und noch etwa eine viertel 
Sekunde länger.

BTW: tolles Ding, das Eiweißgewölle da zwischen den Ohren. Die Benutzung 
ist ausdrücklich nicht verboten.

EDIT:
Henry schrieb:
> Einige meckern lieber als zu helfen
Da hat nach meiner Erfahrung der TO schon auch ein gewichtig Wort 
mitzureden. Wenn der sich helfen lässt und MITDENKT, dann flutscht das 
viel besser.

Henry schrieb:
> Das hatte ich auch bereits geändert und verwende jetzt val
Und wie sollte sich die LED durch diese "Änderung" verhalten? Bringt es 
irgendwas, die Aufgabe mit "val" anzugehen? Warum meinst du, dass das 
der richtige Weg ist?

[Henry (Gast)]

Lothar M. schrieb:
> Um deine eigene Längsachse? So eine Art Pirouette?
> Warum schreibst du nicht einfach, WAS du da drehst und WIE du das an den
> µC angeschlossen hast.

Meine den Encoder damit. Der Anschluss steht im Programm mit drin. B=PA0 
und A=PB2.

Lothar M. schrieb:
> Und wie sollte sich die LED durch diese "Änderung" verhalten? Bringt es
> irgendwas, die Aufgabe mit "val" anzugehen? Warum meinst du, dass das
> der richtige Weg ist?

Die Auswertung von val bringt eine Reaktion der LED, ein kurzes 
aufleuchten bei der Drehung des Encoders. Es ist mir klar das es nicht 
die beste Lösung ist, hatte aber erst mal die Funktin.
Auf deine Lösung bin ich nicht gekommen. Danke. Werde sie jetzt 
auseinander nehmen und mir genau ansehen

[Stefan F. (Gast)]

Henry schrieb:
> Leider habe ich den Eindruck das das meckern vorgeht. Einige meckern
> lieber als zu helfen

Nein, meckern fällt ihnen (mir) nur sehr viel leichter als höflich zu 
sein.

[Henry (Gast)]

Lothar M. schrieb:
> ISR( TIMER2_COMPA_vect )        // 1ms for manual movement
>   {
>     int8_t new, diff;
>     new = 0;
>     if( PHASE_A ) new = 3;
>     if( PHASE_B ) new ^= 1;          // Konvertiert Gray Code in Binär
> Code
>     diff = last - new;               // difference last - new
>     if( diff & 1 )
>     {                                // bit 0 = value (1)
>         last = new;                   // store new as next last
>         enc_delta += (diff & 2) - 1;  // bit 1 = direction (+/-)
>         timeout_led=255;              // Zeitzähler hochsetzen
>     }
>     if (timeout_led>0) timeout_led--; // Zeit runterzählen
>   }

In diesem Teil funktioniert was nicht.
Die Auswertung des Encoders erfolgt mit Phase A und Phase B. Es kann 
sich nur 0, 1, -1, 3 oder -3 ergben (nach Peter). Dabei ist (diff & 1) 
die Änderung. Damit müsste sich bei einer Änderung timeout_led bei 
Bedienung auf 255 stellen. Sobald keine Drehbewegung am Encoder erfolgt 
müsste timeout_led durch timeout_led-- auf 0 gezählt werden. Da der 
Timer mit 1ms läuft und es 255 gemacht wird, ergibt sich eine 
Nachlaufzeit von ca 250ms und die LED muss schalten. Das macht sie 
leider nicht.

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

Henry schrieb:
> Das macht sie leider nicht.
Sondern? Was macht sie stattdessen?

Und zeig auch mal deinen Code (als Dateianhang namens *.c), der zu dem 
Verhalten führt.

[Egonwalter M. (heiner1234)]

Stefan ⛄ F. schrieb:
> Henry schrieb:
>> Leider habe ich den Eindruck das das meckern vorgeht. Einige meckern
>> lieber als zu helfen
>
> Nein, meckern fällt ihnen (mir) nur sehr viel leichter als höflich zu
> sein.

Es geht nicht um "das meckern", sondern vielmehr um die zumeist 
unqualifizierten Aussagen "... es geht nicht".
Wenn man Hilfe erwartet, dann
- schildert man genau, was man zur Fehlersuche bislang getan hat
- schildert man genau den Fehler (wann tritt er auf, welche 
Schaltzustände usw)
- hängt sein komplettes Programm an und nicht irgendwelche Codeschnipsel

das ist das Mindeste - siehe:

Lothar M. schrieb:
> Henry schrieb:
>> Das macht sie leider nicht.
> Sondern? Was macht sie stattdessen?
>
> Und zeig auch mal deinen Code (als Dateianhang namens *.c), der zu dem
> Verhalten führt.

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

Egonwalter M. schrieb:
> was man zur Fehlersuche bislang getan hat
Das Motto heißt hier "teile und herrsche", und da ist die erste Frage:
funktioniert die Ansteuerung der LED?

Ganz offenbar offenbar ist deine LED low-aktiv, denn
Henry schrieb:
> PORTA = 0b00100000;        // LED auf aus
Da passt also meine Polarität schon mal nicht, und die LED müsste 
während des Drehens ausgehen.

Wenn das mit der LED dann geht, kannst du ja einfach mal einen Taster 
afragen und damit den Zähler auf 255 setzen.

Und wenn die LWD dann auf den Taster reagiert, dann siehst du zu, dass 
das Hochsetzen des Zählers furch eine Drehgeberänderung geschieht.

Und wenn du das hast, dann bist du fertig.

[W.S. (Gast)]

Henry schrieb:
> In verschiedenen Projekten verwende ich Encoder (Drehgeber) mit RGB
> LEDs.

Das ist ein bissel weit hergeholt. Genauso könnte man auch sagen "in der 
Spargelzeit verwende ich Spargel und zerlassene Butter mit dem Klo". Das 
ist genauso richtig - unter Weglassung einer Zwischenstufe (genannt 
Mensch, der das zu Mittag ißt).

Also: Sinnvollerweise hält man die verschiedenen Dinge in einer Firmware 
auseinander. Etwa so: Drehgeber - Treiber dafür - Events (ist nach links 
gedreht worden oder so) - Logik (was tun, wenn 1 Schritt nach rechts 
gedreht wurde) - Auswertung (LED leuchten lassen oder nicht).

Wenn du dich in deiner Herangehensweise an so ein Konzept hältst, dann 
kanst du jeden einzelnen Teil separat austesten und kommst so viel eher 
zu deinem Resultat.

Wenn du alles in einen Topf schmeißt, dann kommt da keine Ordnung rein 
und wenn du obendrein sowas postest (oder eine nicht verstandene Routine 
von jemand anderem), dann kann dir keiner helfen, denn durch so ein 
Durcheinander und/oder aus dem Kontext gerissenes Zeug will sich keiner 
freiwillig durchwühlen.

W.S.

[Henry (Gast)]

Konnte den Fehler weiter eingrenzen. Die LED schaltet korrekt. Nach 
Programm start ist die LED ausgeschaltet, beim ersten drehen des 
Encoders schaltet die LED ein und dann nicht wieder aus.
Mit
Lothar M. schrieb:
> if (timeout_led>0)
macht sie leider gar nichts. Der Code:

//***********************************************************************                                                                   
//                      Reading rotary encoder                      
//                      one, two and four step encoders supported   
//              Author: Peter Dannegger     und Lohtar M                                
//************************************************************************

#define F_CPU 16000000UL        // Angabe der Frequenz, wichtig für die Zeit
#include "avr/io.h"            // Einbindung Datei Ausgänge
#include "avr/interrupt.h"        // Einbindung der interrupt.h

#define PHASE_A     (PINB & 1<<PB2)    // Eingang Encoder A  
#define PHASE_B     (PINA & 1<<PA0)    // Eingang Encoder B    

int8_t val=0;
volatile int8_t enc_delta;        // -128 ... 127
static int8_t last;
volatile int8_t timeout_led = 0;    // neu
  
  void encode_init( void )
    {
    int8_t new;
    new = 0;
    if( PHASE_A ) new = 3;
    if( PHASE_B ) new ^= 1;      // Convertiert Gray Code in Binär Code
    last = new;            // power on state
    enc_delta = 0;
    }
  
  void timer_init()            // Timer2 1ms 16MHz
    {
    TCCR2A = (1<<WGM01);
    TCCR2B = (1<<CS01)|(1<<CS00);
    OCR2A = 249;
    TIMSK2 |=(1<<OCIE2A);
    }
  
    ISR( TIMER2_COMPA_vect )        // 1ms for manual movement
      {
      int8_t new, diff;
      new = 0;
      if( PHASE_A ) new = 3;
      if( PHASE_B ) new ^= 1;          // Konvertiert Gray Code in Binär Code
      diff = last - new;               // difference last - new
      if( diff & 1 )
        {                                // bit 0 = value (1)
        last = new;                   // store new as next last
        enc_delta += (diff & 2) - 1;  // bit 1 = direction (+/-)
        timeout_led=255;              // Zeitzähler hochsetzen
        }
      if (timeout_led>0) 
      {
      timeout_led--;        // Zeit runterzählen
      }
      }

  int8_t encode_read1( void )      // lesen einzel Schritt Encoder = schnell
    {
    int8_t val;
    cli();
    val = enc_delta;
    enc_delta = 0;
    sei();
    return val;            // counts since last call
    }

  int8_t encode_read2( void )      // lesen zwei Schritt Encoder = mittel
    {
    int8_t val;
    cli();
    val = enc_delta;
    enc_delta = val & 1;
    sei();
    return val >> 1;
    }

  int8_t encode_read4( void )      // lesen vier Schritt Encoder = langsam
    {
    int8_t val;
    cli();
    val = enc_delta;
    enc_delta = val & 3;
    sei();
    return val >> 2;
    }

  int main( void )            // Hauptschleife
    {
    timer_init();
    DDRA |= 0b00100000;        // LED 5 auf Ausgang
    PORTA = 0b00100000;        // LED auf aus 
    sei();
    for(;;)
      {
      val += encode_read1();    // lesen 1,2 oder 4 Schritt Encoder
      // LED ansteuern
      if (timeout_led==0)
        {
        PORTA |=(1<<PINA5);
        }
      else
        {
        PORTA &=~(1<<PINA5);
        }  
      }
    }

Das funktioniert:

if( diff & 1 )
        {                                // bit 0 = value (1)
        last = new;                   // store new as next last
        enc_delta += (diff & 2) - 1;  // bit 1 = direction (+/-)
        timeout_led=255;              // Zeitzähler hochsetzen
        }
      if (timeout_led>0) 
      {
      timeout_led--;        // Zeit runterzählen
      }

Wird timeout_led nicht wieder auf 255 gesetzt?

[Egonwalter M. (heiner1234)]

W.S. schrieb:
> Henry schrieb:
>> In verschiedenen Projekten verwende ich Encoder (Drehgeber) mit RGB
>> LEDs.
>
> Also: Sinnvollerweise hält man die verschiedenen Dinge in einer Firmware
> auseinander. Etwa so: Drehgeber - Treiber dafür - Events (ist nach links
> gedreht worden oder so) - Logik (was tun, wenn 1 Schritt nach rechts
> gedreht wurde) - Auswertung (LED leuchten lassen oder nicht).
>
> Wenn du dich in deiner Herangehensweise an so ein Konzept hältst, dann
> kanst du jeden einzelnen Teil separat austesten und kommst so viel eher
> zu deinem Resultat.
>
In einem seiner "Tuts" verweist Achim S. aka Henry ja sogar auf diese 
Vorgehensweise:
- Eingabe
- Verarbeitung
- Ausgabe

Deshalb verstehe ich nicht, warum er sich selbst nicht an sein "Tut" 
hält...

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

Henry schrieb:
> Wird timeout_led nicht wieder auf 255 gesetzt?
Natürlich nicht, weil ein int8_t maximal den Wert 127 annehmen kann. 
Mach da mal ein u mit rein. Sagt der Compiler dazu nichts?

[Henry (Gast)]

Wie lange wollt ihr mir noch das vergangene vorwerfen? Habe Fehler 
gemacht uns stehe dazu.

Egonwalter M. schrieb:
> Wenn du dich in deiner Herangehensweise an so ein Konzept hältst, dann
>> kanst du jeden einzelnen Teil separat austesten und kommst so viel eher
>> zu deinem Resultat.

Wenn du so gut bist dann kannst du mir vielleicht sagen wie man in 
diesem Programm einzelne Teile separat austesten kann. Die LED 
funktioniert, habe ich getestet, Encoder funktioniert, habe ich 
getestet. Was kann ich noch tun?

Lothar M. schrieb:
> Mach da mal ein u mit rein. Sagt der Compiler dazu nichts?

Es kam keine Fehlermeldung. Habe ein u reingesetzt und jetzt 
funktioniert es.
Danke für deine Hilfe.

[Heiner (Gast)]

Naja, da war der Code von Lothar M. (Moderator) leider schon falsch, da 
er bei seinem Codeschnipsel
volatile int8_t timeout_led geschrieben hat und den Wert timeout_led = 
255 in der ISR gesetzt hat.

Int8_t reicht aber nur von - … bis + …

Deshalb das “u”

[Heiner (Gast)]

Und nebenbei - das ist ein Paradebeispiel für hirnloses Abschreiben von 
Code OHNE nachzudenken.

Gratuliere Henry 👍

[Egonwalter M. (heiner1234)]

Heiner schrieb:
> Naja, da war der Code von Lothar M. (Moderator) leider schon falsch, da
> er bei seinem Codeschnipsel
> volatile int8_t timeout_led geschrieben hat und den Wert timeout_led =
> 255 in der ISR gesetzt hat.
>
> Int8_t reicht aber nur von - … bis + …
>
> Deshalb das “u”


wenn beim Compiler die Flags fuer Warnungen / Errors richtig gesetzt 
wären, würde er warnen:

"warning: overflow in implicit constant conversion"

(macht er jedenfalls bei mir - AVR Studio4.18 Build 716, WINAVR 2010...)

[Stefan F. (Gast)]

Benutze die Compiler Option -Wall (das kann auch die Arduino IDE)

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

Heiner schrieb:
> da war der Code von Lothar M. (Moderator) leider schon falsch
Ich sehe sowas als kleine Aufgabe zum Mitdenken an. Hat leider offenbar 
nicht funktioniert. Ich hoffe bloß, dass dieser Hinweis auf das 'u' 
einen gewissen Lerneffekt angestoßen hat. Und die Hoffnung stirbt 
bekanntlich zuletzt...  ;-)

[W.S. (Gast)]

Lothar M. schrieb:
> Ich sehe sowas als kleine Aufgabe zum Mitdenken an.

Das war dreist.

W.S.

[S. Landolt (Gast)]

> Das war dreist.
Ja, und wenn ich dann noch an die "Eiweißmasse" denke (wer es 
rechtzeitig sah) ...

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

S. Landolt schrieb:
> wenn ich dann noch an die "Eiweißmasse" denke
Ja nun, das Gehirn besteht nun mal zum größten Teil aus Eiweiß (aka 
Proteine)...

Immerhin hat Henry die Idee meines Codes verstanden und das mit dem uint 
auch noch gelernt.


Dass man natürlich das Programm noch wesentlich besser entkoppeln könnte 
und im Idealfall der Task "LED ansteuern" nichts mitten im Code 
"Drehgeber" zu suchen hat, steht ausser Frage. Da würde ich im echten 
Leben einen Zeitstempel nehmen und dann mit einem fortlaufenden ms-Timer 
arbeiten. und diese ganze Aufgabe in die Hauptschleife verlegen:

:
:
volatile int32_t millis;
:
:
  ISR( TIMER2_COMPA_vect )        // 1ms for manual movement
    {
    millis++;
    :
    :

  int main( void )            // Hauptschleife
    {
    static int32_t lastchg = 0;
    int32_t tiakt;
    int8_t d;

    timer_init();
    DDRA |= 0b00100000;        // LED 5 auf Ausgang
    PORTA = 0b00100000;        // LED auf aus 
    encode_init();
  
    for(;;)
      {
      cli();          // beim lesen von millis: Interrupts blockieren!!
      tiakt = millis; // millis werden pro Durchlauf nur 1x gelesen, 
      sei();          // im Rest der Schleife ändert sich die Zeit nicht.

      d = encode_read2();
      position += d;

      if (d!=0) lastchg = tiakt; // Position geändert, Zeitpunkt merken
      if (tiakt-lastchg < 333) PORTA |=(1<<PINA5);  // Timeout läuft
      else                     PORTA &=~(1<<PINA5); // Timeout abgelaufen

      }
    }

[S. Landolt (Gast)]

an Lothar Miller:
Es war, zumal für einen Moderator, völlig unangebracht - es hieß bei 
Ihnen "Eiweißgewölle", und da zitiere ich den Duden: 'Gewölle': "von 
Greifvögeln herausgewürgter Klumpen unverdaulicher Nahrungsreste"; und 
das, als Moderator, gegenüber einem Forumsmitglied (mag sich dieses auch 
noch so ungeschickt anstellen).

[W.S. (Gast)]

S. Landolt schrieb:
> an Lothar Miller:
> Es war, zumal für einen Moderator, völlig unangebracht...

Laß mal, gelegentlich hat auch ein Moderator mal den Drang, sich etwas 
'starkdeutsch' auszudrücken. Was mich hingegen mehr ärgert, sind 
Ausreden wenn einer sich mal vertut. Letzteres ist aud Dauer 
unvermeidlich und da wäre ein simples 'sorry, hab mich mal vertan' 
weitaus besser.

W.S.

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

S. Landolt schrieb:
> Es war, zumal für einen Moderator, völlig unangebracht
Es ist nun mal ganz einfach so, dass es ein Eiweißgewölle ist und man es 
zum Denken verwenden kann und auch sollte. Punkt.

> gegenüber einem Forumsmitglied (mag sich dieses auch noch so ungeschickt
> anstellen).
Wer hat denn dem Henry geholfen obwohl er sich ungeschickt angestellt 
hat?


W.S. schrieb:
> sind Ausreden wenn einer sich mal vertut.
Also gut: ich gebe es hiermit öffentlich jedem zur Kenntnis, dass ich da 
oben mangels Compiler und Testumgebung einen Fehler reingebracht habe. 
Wer sowas noch nie geschafft hat, kann jetzt gern auf mir herumhacken.

[Egonwalter M. (heiner1234)]

Lothar M. schrieb:
> S. Landolt schrieb:
>> Es war, zumal für einen Moderator, völlig unangebracht
> Es ist nun mal ganz einfach so, dass es ein Eiweißgewölle ist und man es
> zum Denken verwenden kann und auch sollte. Punkt.
>
>> gegenüber einem Forumsmitglied (mag sich dieses auch noch so ungeschickt
>> anstellen).
> Wer hat denn dem Henry geholfen obwohl er sich ungeschickt angestellt
> hat?
>
>
> W.S. schrieb:
>> sind Ausreden wenn einer sich mal vertut.
> Also gut: ich gebe es hiermit öffentlich jedem zur Kenntnis, dass ich da
> oben mangels Compiler und Testumgebung einen Fehler reingebracht habe.
> Wer sowas noch nie geschafft hat, kann jetzt gern auf mir herumhacken.

Ok, kann man nichts dagegen einwenden.
Akzeptiert.